Skip to content

Commit f6db45b

Browse files
committed
docs(agents): refine dependency manifest rule to require approval before changes
1 parent ffec09d commit f6db45b

10 files changed

Lines changed: 10 additions & 10 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.
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.
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
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.
7272

7373
---
7474

.github/agents/changelog-agent.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ 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.
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.
2121

2222
---
2323

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ 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.
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.
3939

4040
---
4141

.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.
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.
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.
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.
123123

124124
---
125125

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

Lines changed: 1 addition & 1 deletion
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
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.
6969

7070
---
7171

.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 add, remove, or change entries in `package.json`, `package-lock.json`, or any other dependency manifest or lock file.
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.
3535

3636
---
3737

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ Do not add extra scenarios unless they are explicitly requested, clearly require
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.
5959
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.
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.
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.
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.
126126

127127
---
128128

0 commit comments

Comments
 (0)