Skip to content

Commit 4caca87

Browse files
chore(agents): move dependency change policy to AGENTS.md and trim instructions
1 parent f6db45b commit 4caca87

11 files changed

Lines changed: 12 additions & 16 deletions

.github/agents/bug-fixing-implementer-agent.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ If the bug requires SCSS, theme wiring, or style-test changes, do not implement
114114
- Do not update `README.md` — the `component-readme-agent` handles that.
115115
- Do not create migration schematics — the `migration-agent` handles that.
116116
- Do not update `CHANGELOG.md` — the `changelog-agent` handles that.
117-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
117+
- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, etc.). Ask for approval first if a dependency change is truly required.
118118

119119
---
120120

.github/agents/bug-fixing-orchestrator-agent.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ You do NOT write tests, production code, or detailed implementation instructions
6868
- Do not write tests or production code
6969
- Do not specify exact test cases, exact implementations, or exact file changes
7070
- Do not over-constrain the handoff prompts — give scope and root cause, not specs
71-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
71+
- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, etc.). Ask for approval first if a dependency change is truly required.
7272

7373
---
7474

.github/agents/changelog-agent.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ You update `CHANGELOG.md` at the repository root for Ignite UI for Angular. Only
1717
## What You Do NOT Do
1818

1919
- Do not modify production source code.
20-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
2120

2221
---
2322

.github/agents/component-readme-agent.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ You do not implement production code, write tests, write migrations, or update `
3535
- Do not rewrite the whole README if only one area changed.
3636
- Do not update `CHANGELOG.md`.
3737
- Do not document private/internal implementation details.
38-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
3938

4039
---
4140

.github/agents/demo-sample-agent.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ You do not implement the library change itself, create new samples or demo folde
4040
- Do not update component `README.md`.
4141
- Do not update `CHANGELOG.md`.
4242
- Do not create migrations.
43-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
43+
- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, etc.). Ask for approval first if a dependency change is truly required.
4444

4545
---
4646

.github/agents/feature-implementer-agent.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ Update component agent skills if you need to guide other agents on how to use th
119119
- Do not update `README.md` — the `component-readme-agent` handles that.
120120
- Do not create migration schematics — the `migration-agent` handles that.
121121
- Do not update `CHANGELOG.md` — the `changelog-agent` handles that.
122-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
122+
- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, etc.). Ask for approval first if a dependency change is truly required.
123123

124124
---
125125

.github/agents/feature-orchestrator-agent.md

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ You do NOT write tests, production code, or detailed acceptance criteria. Each s
6565
- Do not write detailed acceptance criteria that downstream agents must encode literally
6666
- Do not specify exact test cases, exact implementations, or exact file changes
6767
- Do not over-constrain the handoff prompts — give scope, not specs
68-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
68+
- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, etc.). Ask for approval first if a dependency change is truly required.
6969

7070
---
7171

@@ -188,17 +188,15 @@ Use agents in this order:
188188
Only invoke `demo-sample-agent` if the user explicitly requested a demo/sample update.
189189
If the user declined, skip that handoff and continue with the remaining agents.
190190

191-
If the feature is purely theming or styling, route directly from `tdd-test-writer-agent` to `theming-styles-agent` and skip the general
192-
implementer.
191+
If the feature is purely theming or styling, route directly from `tdd-test-writer-agent` to `theming-styles-agent` and skip the general implementer.
193192

194193
### Step 5 — Verify Completeness
195194

196195
After all agents finish, check:
197196

198197
- Were all affected areas covered?
199198
- Were public exports updated?
200-
- Were theming and style changes delegated when SCSS or theme wiring was
201-
affected?
199+
- Were theming and style changes delegated when SCSS or theme wiring was affected?
202200
- Was the component README updated if needed?
203201
- Was `CHANGELOG.md` updated if needed?
204202
- Do migrations exist for any breaking changes?

.github/agents/migration-agent.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ You create **`ng update` migration schematics** for breaking changes in Ignite U
3131

3232
## What You Do NOT Do
3333

34-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
34+
- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, etc.). Ask for approval first if a dependency change is truly required.
3535

3636
---
3737

.github/agents/tdd-test-writer-agent.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@ Do not add extra scenarios unless they are explicitly requested, clearly require
5656
- No execution-order dependency.
5757
6. **Never write production code.**
5858
- Only test code in this phase to ensure the test does not give false negatives. The test MUST fail if the functionality is missing, or not behaving as intended.
59-
7. **Never modify dependency files.**
60-
- `package.json`, `package-lock.json`, `yarn.lock`, `pnpm-lock.yaml`, and similar manifest or lock files are off-limits. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
59+
7. **Do not modify dependency files.**
60+
- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, `yarn.lock`, `pnpm-lock.yaml`, etc.). Ask for approval first if a dependency change is truly required.
6161

6262
---
6363

.github/agents/theming-styles-agent.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ If the task is purely documentation or planning, say clearly that style validati
122122
- Do not update component `README.md`.
123123
- Do not create migrations.
124124
- Do not update `CHANGELOG.md`.
125-
- Do not modify `package.json`, `package-lock.json`, or any other dependency manifest or lock file. If a dependency change appears truly necessary, ask for approval first. Never commit `package-lock.json` unless you have been explicitly approved to introduce a new dependency — committing unintended lock file changes can break builds.
125+
- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, etc.). Ask for approval first if a dependency change is truly required.
126126

127127
---
128128

0 commit comments

Comments
 (0)