@event4u/agent-config 1.20.0 → 1.22.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.agent-src/commands/agents.md +1 -1
- package/.agent-src/commands/bug-fix.md +2 -1
- package/.agent-src/commands/bug-investigate.md +3 -2
- package/.agent-src/commands/challenge-me/vision.md +348 -0
- package/.agent-src/commands/challenge-me/with-docs.md +333 -0
- package/.agent-src/commands/challenge-me.md +61 -0
- package/.agent-src/commands/chat-history/import.md +60 -64
- package/.agent-src/commands/compress.md +12 -0
- package/.agent-src/commands/context/create.md +2 -2
- package/.agent-src/commands/context.md +1 -1
- package/.agent-src/commands/copilot-agents.md +1 -1
- package/.agent-src/commands/council/default.md +69 -10
- package/.agent-src/commands/council.md +1 -1
- package/.agent-src/commands/create-pr.md +7 -3
- package/.agent-src/commands/e2e-heal.md +1 -1
- package/.agent-src/commands/e2e-plan.md +1 -1
- package/.agent-src/commands/feature/dev.md +3 -3
- package/.agent-src/commands/feature.md +1 -1
- package/.agent-src/commands/fix/seeder.md +2 -2
- package/.agent-src/commands/fix.md +1 -1
- package/.agent-src/commands/grill-me.md +38 -0
- package/.agent-src/commands/jira-ticket.md +1 -1
- package/.agent-src/commands/judge/steps.md +1 -1
- package/.agent-src/commands/judge.md +2 -2
- package/.agent-src/commands/memory.md +1 -1
- package/.agent-src/commands/mode.md +5 -5
- package/.agent-src/commands/module.md +1 -1
- package/.agent-src/commands/onboard.md +4 -4
- package/.agent-src/commands/optimize/augmentignore.md +1 -1
- package/.agent-src/commands/optimize-prompt.md +61 -0
- package/.agent-src/commands/optimize.md +1 -1
- package/.agent-src/commands/override.md +1 -1
- package/.agent-src/commands/review-changes.md +1 -1
- package/.agent-src/commands/review-routing.md +1 -1
- package/.agent-src/commands/roadmap/ai-council.md +183 -0
- package/.agent-src/commands/roadmap/create.md +6 -1
- package/.agent-src/commands/roadmap/process-full.md +58 -0
- package/.agent-src/commands/roadmap/process-phase.md +69 -0
- package/.agent-src/commands/roadmap/process-step.md +57 -0
- package/.agent-src/commands/roadmap.md +45 -17
- package/.agent-src/commands/set-cost-profile.md +3 -3
- package/.agent-src/commands/sync-agent-settings.md +2 -2
- package/.agent-src/commands/tests/create.md +2 -2
- package/.agent-src/commands/tests.md +1 -1
- package/.agent-src/commands/threat-model.md +5 -4
- package/.agent-src/contexts/augment-infrastructure.md +1 -1
- package/.agent-src/contexts/authority/commit-mechanics.md +14 -1
- package/.agent-src/contexts/authority/destructive-mechanics.md +14 -1
- package/.agent-src/contexts/authority/scope-mechanics.md +5 -0
- package/.agent-src/contexts/communication/rules-auto/guidelines-mechanics.md +76 -0
- package/.agent-src/contexts/communication/rules-auto/slash-command-routing-policy-mechanics.md +54 -19
- package/.agent-src/contexts/communication/rules-auto/think-before-action-mechanics.md +98 -0
- package/.agent-src/contexts/communication/rules-auto/token-efficiency-mechanics.md +93 -0
- package/.agent-src/contexts/communication/rules-auto/user-interaction-mechanics.md +128 -5
- package/.agent-src/contexts/execution/autonomy-mechanics.md +44 -0
- package/.agent-src/contexts/execution/roadmap-process-loop.md +125 -0
- package/.agent-src/contexts/model-recommendations.md +2 -2
- package/.agent-src/contexts/override-system.md +1 -1
- package/.agent-src/contexts/skills-and-commands.md +1 -1
- package/.agent-src/personas/product-owner.md +2 -2
- package/.agent-src/personas/qa.md +1 -1
- package/.agent-src/rules/agent-authority.md +5 -6
- package/.agent-src/rules/agent-docs.md +11 -53
- package/.agent-src/rules/analysis-skill-routing.md +10 -40
- package/.agent-src/rules/architecture.md +6 -1
- package/.agent-src/rules/artifact-drafting-protocol.md +5 -0
- package/.agent-src/rules/artifact-engagement-recording.md +23 -59
- package/.agent-src/rules/ask-when-uncertain.md +24 -47
- package/.agent-src/rules/augment-portability.md +14 -62
- package/.agent-src/rules/augment-source-of-truth.md +10 -1
- package/.agent-src/rules/autonomous-execution.md +17 -98
- package/.agent-src/rules/capture-learnings.md +9 -80
- package/.agent-src/rules/cli-output-handling.md +12 -42
- package/.agent-src/rules/command-suggestion-policy.md +25 -73
- package/.agent-src/rules/commit-conventions.md +9 -58
- package/.agent-src/rules/commit-policy.md +16 -47
- package/.agent-src/rules/context-hygiene.md +5 -0
- package/.agent-src/rules/direct-answers.md +21 -50
- package/.agent-src/rules/docker-commands.md +11 -45
- package/.agent-src/rules/docs-sync.md +10 -56
- package/.agent-src/rules/downstream-changes.md +5 -0
- package/.agent-src/rules/e2e-testing.md +9 -44
- package/.agent-src/rules/guidelines.md +13 -75
- package/.agent-src/rules/improve-before-implement.md +11 -2
- package/.agent-src/rules/invite-challenge.md +71 -0
- package/.agent-src/rules/language-and-tone.md +41 -106
- package/.agent-src/rules/laravel-translations.md +11 -40
- package/.agent-src/rules/markdown-safe-codeblocks.md +4 -0
- package/.agent-src/rules/minimal-safe-diff.md +4 -0
- package/.agent-src/rules/missing-tool-handling.md +4 -0
- package/.agent-src/rules/model-recommendation.md +9 -61
- package/.agent-src/rules/no-attribution-footers.md +5 -0
- package/.agent-src/rules/no-cheap-questions.md +11 -27
- package/.agent-src/rules/no-council-references.md +76 -0
- package/.agent-src/rules/no-roadmap-references.md +7 -0
- package/.agent-src/rules/non-destructive-by-default.md +13 -43
- package/.agent-src/rules/onboarding-gate.md +9 -117
- package/.agent-src/rules/package-ci-checks.md +10 -37
- package/.agent-src/rules/php-coding.md +10 -55
- package/.agent-src/rules/preservation-guard.md +9 -0
- package/.agent-src/rules/review-routing-awareness.md +9 -97
- package/.agent-src/rules/reviewer-awareness.md +8 -83
- package/.agent-src/rules/roadmap-progress-sync.md +7 -170
- package/.agent-src/rules/role-mode-adherence.md +6 -2
- package/.agent-src/rules/rule-type-governance.md +8 -66
- package/.agent-src/rules/runtime-safety.md +5 -0
- package/.agent-src/rules/scope-control.md +17 -62
- package/.agent-src/rules/security-sensitive-stop.md +7 -1
- package/.agent-src/rules/size-enforcement.md +6 -1
- package/.agent-src/rules/skill-improvement-trigger.md +9 -49
- package/.agent-src/rules/skill-quality.md +7 -113
- package/.agent-src/rules/slash-command-routing-policy.md +11 -63
- package/.agent-src/rules/think-before-action.md +22 -87
- package/.agent-src/rules/token-efficiency.md +10 -74
- package/.agent-src/rules/token-optimizer-maintenance.md +68 -0
- package/.agent-src/rules/tool-safety.md +4 -0
- package/.agent-src/rules/ui-audit-gate.md +25 -61
- package/.agent-src/rules/upstream-proposal.md +9 -67
- package/.agent-src/rules/user-interaction.md +22 -108
- package/.agent-src/rules/verify-before-complete.md +1 -1
- package/.agent-src/skills/adversarial-review/SKILL.md +1 -0
- package/.agent-src/skills/agent-docs-writing/SKILL.md +1 -1
- package/.agent-src/skills/ai-council/SKILL.md +197 -8
- package/.agent-src/skills/analysis-autonomous-mode/SKILL.md +1 -1
- package/.agent-src/skills/analysis-skill-router/SKILL.md +3 -3
- package/.agent-src/skills/artisan-commands/SKILL.md +2 -2
- package/.agent-src/skills/authz-review/SKILL.md +1 -1
- package/.agent-src/skills/aws-infrastructure/SKILL.md +5 -5
- package/.agent-src/skills/blast-radius-analyzer/SKILL.md +8 -8
- package/.agent-src/skills/bug-analyzer/SKILL.md +6 -5
- package/.agent-src/skills/code-refactoring/SKILL.md +4 -4
- package/.agent-src/skills/code-review/SKILL.md +2 -2
- package/.agent-src/skills/command-writing/SKILL.md +11 -0
- package/.agent-src/skills/composer-packages/SKILL.md +2 -2
- package/.agent-src/skills/context-authoring/SKILL.md +11 -0
- package/.agent-src/skills/context-document/SKILL.md +1 -1
- package/.agent-src/skills/copilot-agents-optimization/SKILL.md +23 -0
- package/.agent-src/skills/copilot-config/SKILL.md +1 -1
- package/.agent-src/skills/dependency-upgrade/SKILL.md +2 -2
- package/.agent-src/skills/devcontainer/SKILL.md +2 -2
- package/.agent-src/skills/developer-like-execution/SKILL.md +1 -1
- package/.agent-src/skills/docker/SKILL.md +1 -1
- package/.agent-src/skills/dto-creator/SKILL.md +1 -1
- package/.agent-src/skills/estimate-ticket/SKILL.md +2 -2
- package/.agent-src/skills/fe-design/SKILL.md +4 -4
- package/.agent-src/skills/feature-planning/SKILL.md +5 -5
- package/.agent-src/skills/funnel-analysis/SKILL.md +1 -1
- package/.agent-src/skills/laravel/SKILL.md +1 -1
- package/.agent-src/skills/laravel-notifications/SKILL.md +5 -5
- package/.agent-src/skills/laravel-pennant/SKILL.md +1 -1
- package/.agent-src/skills/laravel-pulse/SKILL.md +4 -4
- package/.agent-src/skills/laravel-reverb/SKILL.md +2 -2
- package/.agent-src/skills/laravel-scheduling/SKILL.md +1 -1
- package/.agent-src/skills/migration-creator/SKILL.md +7 -7
- package/.agent-src/skills/multi-tenancy/SKILL.md +8 -8
- package/.agent-src/skills/performance-analysis/SKILL.md +3 -3
- package/.agent-src/skills/pest-testing/SKILL.md +6 -6
- package/.agent-src/skills/php-service/SKILL.md +2 -2
- package/.agent-src/skills/project-analysis-hypothesis-driven/SKILL.md +3 -3
- package/.agent-src/skills/project-analysis-react/SKILL.md +1 -1
- package/.agent-src/skills/project-analysis-symfony/SKILL.md +1 -1
- package/.agent-src/skills/project-analysis-zend-laminas/SKILL.md +2 -2
- package/.agent-src/skills/project-analyzer/SKILL.md +4 -4
- package/.agent-src/skills/prompt-optimizer/SKILL.md +108 -0
- package/.agent-src/skills/readme-reviewer/SKILL.md +1 -1
- package/.agent-src/skills/roadmap-management/SKILL.md +7 -7
- package/.agent-src/skills/rule-writing/SKILL.md +33 -0
- package/.agent-src/skills/sentry-integration/SKILL.md +1 -1
- package/.agent-src/skills/skill-writing/SKILL.md +14 -0
- package/.agent-src/skills/systematic-debugging/SKILL.md +22 -2
- package/.agent-src/skills/technical-specification/SKILL.md +58 -1
- package/.agent-src/skills/terraform/SKILL.md +2 -2
- package/.agent-src/skills/terragrunt/SKILL.md +8 -8
- package/.agent-src/skills/test-performance/SKILL.md +5 -5
- package/.agent-src/skills/threat-modeling/SKILL.md +3 -2
- package/.agent-src/skills/token-optimizer/SKILL.md +110 -0
- package/.agent-src/skills/universal-project-analysis/SKILL.md +1 -1
- package/.agent-src/templates/AGENTS.md +1 -1
- package/.agent-src/templates/agent-settings.md +35 -19
- package/.agent-src/templates/command.md +17 -1
- package/.agent-src/templates/contexts/tenant-boundaries.md +2 -2
- package/.agent-src/templates/contexts.md +1 -1
- package/.agent-src/templates/copilot-instructions.md +21 -0
- package/.agent-src/templates/copilot-review-instructions.md +76 -0
- package/.agent-src/templates/features.md +1 -1
- package/.agent-src/templates/roadmaps.md +10 -2
- package/.agent-src/templates/rule.md +129 -0
- package/.agent-src/templates/skill.md +17 -0
- package/.claude-plugin/marketplace.json +12 -2
- package/AGENTS.md +32 -5
- package/CHANGELOG.md +107 -3
- package/README.md +22 -21
- package/config/agent-settings.template.yml +66 -10
- package/config/gitignore-block.txt +7 -0
- package/docs/architecture.md +86 -5
- package/docs/catalog.md +16 -6
- package/docs/contracts/agent-memory-contract.md +1 -1
- package/docs/contracts/command-clusters.md +45 -1
- package/docs/contracts/context-paths.md +2 -1
- package/docs/contracts/file-ownership-matrix.json +354 -500
- package/docs/contracts/iron-law-overrides.txt +25 -0
- package/docs/contracts/kernel-membership.md +273 -0
- package/docs/contracts/load-context-schema.md +26 -11
- package/docs/contracts/pilot/agent-authority.md +24 -0
- package/docs/contracts/pilot/direct-answers.md +70 -0
- package/docs/contracts/pilot/language-and-tone.md +63 -0
- package/docs/contracts/rule-classification.md +170 -0
- package/docs/contracts/rule-router.md +153 -0
- package/docs/customization.md +17 -6
- package/docs/decisions/ADR-001-kernel-swap-deferred.md +109 -0
- package/docs/decisions/ADR-002-kernel-bucket-overrides.md +124 -0
- package/docs/decisions/ADR-003-flat-cluster-subs-and-colon-syntax.md +126 -0
- package/docs/decisions/ADR-rule-kernel-and-router.md +122 -0
- package/docs/getting-started.md +2 -2
- package/docs/guidelines/agent-infra/naming.md +1 -1
- package/docs/guidelines/agent-infra/roadmap-progress-mechanics.md +176 -0
- package/docs/guidelines/agent-infra/rule-type-governance.md +73 -0
- package/docs/guidelines/agent-infra/size-and-scope.md +13 -2
- package/docs/guidelines/agent-infra/skill-quality-checklist.md +119 -0
- package/docs/guidelines/augment-portability-patterns.md +68 -0
- package/docs/guidelines/php/php-coding-patterns.md +62 -0
- package/package.json +1 -1
- package/scripts/_p43_bodies.py +235 -0
- package/scripts/_p43_compress.py +118 -0
- package/scripts/_p4_migrate.py +199 -0
- package/scripts/_phase2_shim_helper.py +1 -1
- package/scripts/_pilot_council_question.py +57 -0
- package/scripts/_pilot_measure.py +53 -0
- package/scripts/ai_council/session.py +107 -5
- package/scripts/build_linear_digest.py +3 -5
- package/scripts/check_always_budget.py +39 -6
- package/scripts/check_compressed_paths.py +213 -0
- package/scripts/check_compression.py +15 -0
- package/scripts/check_context_paths.py +1 -0
- package/scripts/check_council_layout.py +105 -0
- package/scripts/check_council_references.py +145 -0
- package/scripts/check_portability.py +2 -0
- package/scripts/check_references.py +2 -0
- package/scripts/check_token_optimizer_freshness.py +131 -0
- package/scripts/compile_router.py +148 -0
- package/scripts/compress.py +219 -11
- package/scripts/council_cli.py +132 -11
- package/scripts/council_prune.py +81 -0
- package/scripts/count_token_optimizer_usage.sh +54 -0
- package/scripts/install.sh +44 -2
- package/scripts/iron_law_sha.py +98 -0
- package/scripts/lint_load_context.py +35 -5
- package/scripts/measure_rule_budget.py +314 -0
- package/scripts/migrate_command_suggestions.py +2 -2
- package/scripts/prototype_lint_contradictions.py +150 -0
- package/scripts/schemas/command.schema.json +5 -0
- package/scripts/schemas/rule.schema.json +60 -6
- package/scripts/schemas/skill.schema.json +5 -0
- package/scripts/skill_linter.py +197 -7
- package/scripts/smoke_path_resolution.py +93 -0
- package/scripts/validate_frontmatter.py +41 -1
- package/.agent-src/commands/roadmap/execute.md +0 -109
- package/.agent-src/contexts/communication/rules-auto/artifact-engagement-recording-mechanics.md +0 -72
- package/.agent-src/contexts/communication/rules-auto/augment-portability-mechanics.md +0 -79
- package/.agent-src/contexts/communication/rules-auto/cli-output-handling-mechanics.md +0 -87
- package/.agent-src/contexts/communication/rules-auto/command-suggestion-policy-mechanics.md +0 -62
- package/.agent-src/contexts/communication/rules-auto/docs-sync-mechanics.md +0 -78
- package/.agent-src/contexts/communication/rules-auto/package-ci-checks-mechanics.md +0 -85
- package/.agent-src/contexts/communication/rules-auto/review-routing-awareness-mechanics.md +0 -65
- package/.agent-src/contexts/communication/rules-auto/roadmap-progress-sync-mechanics.md +0 -78
- package/.agent-src/contexts/communication/rules-auto/ui-audit-gate-mechanics.md +0 -53
- /package/{docs → .agent-src/contexts}/contracts/artifact-engagement-flow.md +0 -0
- /package/{docs → .agent-src/contexts}/contracts/command-suggestion-flow.md +0 -0
|
@@ -0,0 +1,183 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: roadmap:ai-council
|
|
3
|
+
cluster: roadmap
|
|
4
|
+
sub: ai-council
|
|
5
|
+
skills: [ai-council, agent-docs-writing, roadmap-management]
|
|
6
|
+
description: Challenge a roadmap with the AI council (deep tier) and refactor from convergence findings. Wraps `/council default` pinned to `--input-mode roadmap --depth deep`; patches surface as numbered options.
|
|
7
|
+
disable-model-invocation: true
|
|
8
|
+
council_depth: deep
|
|
9
|
+
suggestion:
|
|
10
|
+
eligible: true
|
|
11
|
+
trigger_description: "council on roadmap, challenge this roadmap, stress-test the plan, refactor roadmap from council findings"
|
|
12
|
+
trigger_context: "existing agents/roadmaps/*.md the user wants reviewed before execution"
|
|
13
|
+
---
|
|
14
|
+
|
|
15
|
+
# /roadmap:ai-council
|
|
16
|
+
|
|
17
|
+
Council-driven challenge + refactor scope of the
|
|
18
|
+
[`/roadmap`](../roadmap.md) cluster. Pins the input mode to
|
|
19
|
+
`roadmap` and the depth tier to `deep` (architecture / refactor
|
|
20
|
+
artefact), then drives the user through applying convergence
|
|
21
|
+
findings as numbered patches against the roadmap file.
|
|
22
|
+
|
|
23
|
+
**Source of truth:** `.agent-src.uncompressed/` — never read or edit
|
|
24
|
+
`.agent-src/` or `.augment/` directly.
|
|
25
|
+
|
|
26
|
+
## Instructions
|
|
27
|
+
|
|
28
|
+
### 1. Resolve the target roadmap
|
|
29
|
+
|
|
30
|
+
Parse the argument as a roadmap path or filename:
|
|
31
|
+
|
|
32
|
+
- `/roadmap:ai-council agents/roadmaps/<name>.md` — explicit path.
|
|
33
|
+
- `/roadmap:ai-council <name>` — fuzzy match against
|
|
34
|
+
`agents/roadmaps/*.md`; if multiple match, list and ask
|
|
35
|
+
(one question per turn per `ask-when-uncertain`).
|
|
36
|
+
- No argument → list `agents/roadmaps/*.md` and ask.
|
|
37
|
+
|
|
38
|
+
Capture the **original ask** verbatim — the user's framing sentence
|
|
39
|
+
that triggered this council run (e.g. *"review this roadmap before
|
|
40
|
+
I execute it"*). This flows into `--original-ask`.
|
|
41
|
+
|
|
42
|
+
### 2. Run the `/council default` flow with these pinned flags
|
|
43
|
+
|
|
44
|
+
Follow [`/council default`](../council/default.md) Steps 2–4
|
|
45
|
+
**verbatim**, with these arguments fixed:
|
|
46
|
+
|
|
47
|
+
- `--input-mode roadmap`
|
|
48
|
+
- `--depth deep` (this command declares `council_depth: deep` in
|
|
49
|
+
frontmatter; the host translates it into `--depth deep`)
|
|
50
|
+
- `--output agents/council-responses/<roadmap-stem>-roadmap.json`
|
|
51
|
+
(overwrite if it exists; the previous run is the predecessor for
|
|
52
|
+
this iteration)
|
|
53
|
+
- `--original-ask "<captured-ask>"`
|
|
54
|
+
|
|
55
|
+
`--depth deep` floors rounds at
|
|
56
|
+
`max(ai_council.deep_min_rounds, ai_council.min_rounds)` (default
|
|
57
|
+
`3`). Do **not** pass `--rounds` unless the user explicitly asked
|
|
58
|
+
for a different count.
|
|
59
|
+
|
|
60
|
+
The cost gate from `/council default` Step 3 still applies — billable
|
|
61
|
+
members require user confirmation **even under `personal.autonomy: on`**
|
|
62
|
+
(per the deep tier surcharge — typical cost ~$0.05–0.13 vs. ~$0.02
|
|
63
|
+
for the standard tier).
|
|
64
|
+
|
|
65
|
+
### 3. Render the report
|
|
66
|
+
|
|
67
|
+
Run `./agent-config council:render <output.json>` and write the
|
|
68
|
+
**Convergence / Divergence** section per
|
|
69
|
+
[`/council default § Render`](../council/default.md). Do **not** end
|
|
70
|
+
with `/council default`'s generic numbered-options block — the
|
|
71
|
+
refactor flow in Step 4 replaces it.
|
|
72
|
+
|
|
73
|
+
### 4. Append a Council review block to the roadmap
|
|
74
|
+
|
|
75
|
+
Open the roadmap file and append (do **not** overwrite existing
|
|
76
|
+
content):
|
|
77
|
+
|
|
78
|
+
```markdown
|
|
79
|
+
|
|
80
|
+
## Council review (<UTC date>)
|
|
81
|
+
|
|
82
|
+
<Convergence section verbatim>
|
|
83
|
+
|
|
84
|
+
### Convergence findings
|
|
85
|
+
|
|
86
|
+
1. **<Finding 1 title>** — <one-line summary> · trace: §<member-section>
|
|
87
|
+
2. **<Finding 2 title>** — <one-line summary> · trace: §<member-section>
|
|
88
|
+
…
|
|
89
|
+
|
|
90
|
+
### Divergences (no consensus)
|
|
91
|
+
|
|
92
|
+
- **<Topic>** — <Member A says X, Member B says Y; user decides>
|
|
93
|
+
|
|
94
|
+
### Predecessor council trace
|
|
95
|
+
|
|
96
|
+
`agents/council-responses/<roadmap-stem>-roadmap.json` (this run).
|
|
97
|
+
```
|
|
98
|
+
|
|
99
|
+
Run `./agent-config roadmap:progress` after the append. The block
|
|
100
|
+
adds no `[ ]` checkboxes, so the dashboard counts stay flat.
|
|
101
|
+
|
|
102
|
+
### 5. Apply the critical-evaluation lens, then surface verdicted patches
|
|
103
|
+
|
|
104
|
+
Before drafting any patch, run every finding from Step 4 through the
|
|
105
|
+
*Critical evaluation* checklist from the
|
|
106
|
+
[`ai-council` skill](../../skills/ai-council/SKILL.md#critical-evaluation--convener-skeptic-stance):
|
|
107
|
+
|
|
108
|
+
- **Codebase fit** — does the finding match the actual roadmap content, file paths, scripts, contracts cited in the roadmap? (`view` / `codebase-retrieval`)
|
|
109
|
+
- **Locked-decision conflict** — does it contradict an ADR (`docs/decisions/`), a contract (`docs/contracts/`), a kernel rule, or an earlier locked decision in **this** roadmap?
|
|
110
|
+
- **Already addressed** — is the finding already covered by an existing step, AC, or phase in the roadmap?
|
|
111
|
+
- **Cost / benefit** — does the patch's scope vs. roadmap value clear the bar?
|
|
112
|
+
- **Hallucination** — does the finding cite a file, function, phase, or step that does not exist?
|
|
113
|
+
|
|
114
|
+
For every finding, attach a verdict — **`accept`**, **`accept-with-modification`**, **`reject`**, or **`needs-input`** — with a one-line reason citing host evidence (file:line, ADR, contract, roadmap step).
|
|
115
|
+
|
|
116
|
+
Append a **Host verdict** sub-block under the Council review block in the roadmap:
|
|
117
|
+
|
|
118
|
+
```markdown
|
|
119
|
+
### Host verdict
|
|
120
|
+
|
|
121
|
+
| # | Finding | Verdict | Reason |
|
|
122
|
+
|---|---|---|---|
|
|
123
|
+
| 1 | <one-line> | `accept` | matches `agents/roadmaps/<this>.md` Phase X step Y |
|
|
124
|
+
| 2 | <one-line> | `accept-with-modification` | narrow scope to phase Z — global change contradicts AC §N |
|
|
125
|
+
| 3 | <one-line> | `reject` | contradicts ADR `docs/decisions/<adr>.md` |
|
|
126
|
+
| 4 | <one-line> | `needs-input` | open question — user picks below |
|
|
127
|
+
```
|
|
128
|
+
|
|
129
|
+
Then surface a single numbered-options block per [`user-interaction`](../../rules/user-interaction.md), carrying the verdict per option:
|
|
130
|
+
|
|
131
|
+
> 1. `[accept]` Apply finding 1 — <one-line patch summary>
|
|
132
|
+
> 2. `[accept-with-modification]` Apply finding 2 (modified) — <one-line patch summary + adjustment>
|
|
133
|
+
> 3. `[reject]` Skip finding 3 — <one-line reason> (override available below)
|
|
134
|
+
> 4. `[needs-input]` <open question for finding 4>
|
|
135
|
+
> …
|
|
136
|
+
> N. Apply all `accept` findings (recommended only if non-conflicting)
|
|
137
|
+
> N+1. Override host verdict — apply a finding the host rejected (specify number)
|
|
138
|
+
> N+2. Skip — leave Council review block + Host verdict as advisory only
|
|
139
|
+
|
|
140
|
+
The user picks one or more numbers (`1,3,5` is allowed). Apply each selected patch via `str-replace-editor` against the roadmap, then re-run `./agent-config roadmap:progress` once at the end so the dashboard reflects the new step / AC count.
|
|
141
|
+
|
|
142
|
+
**Verdict ≠ filter.** Every finding stays visible in the Host verdict block with its verdict and reason — the user can override at any time. The host filters its **own** recommendation; it does not hide council output.
|
|
143
|
+
|
|
144
|
+
### 6. Hard floor — text + roadmap edits only
|
|
145
|
+
|
|
146
|
+
`/roadmap:ai-council` may:
|
|
147
|
+
|
|
148
|
+
- write `agents/council-responses/<…>.json`
|
|
149
|
+
- append the Council review block to the named roadmap
|
|
150
|
+
- apply user-picked patches to the same roadmap
|
|
151
|
+
- regenerate `agents/roadmaps-progress.md`
|
|
152
|
+
|
|
153
|
+
It does **NOT**:
|
|
154
|
+
|
|
155
|
+
- edit any other roadmap, command, rule, or skill file
|
|
156
|
+
- commit, push, or open a PR
|
|
157
|
+
- run `git` beyond `git diff` (read-only)
|
|
158
|
+
|
|
159
|
+
## Rules
|
|
160
|
+
|
|
161
|
+
- **One roadmap per invocation.** Re-run for the next file.
|
|
162
|
+
- **Critical evaluation is mandatory** — every council finding gets
|
|
163
|
+
a host verdict (`accept` / `accept-with-modification` / `reject` /
|
|
164
|
+
`needs-input`) with one-line evidence before any patch is drafted.
|
|
165
|
+
Convergence ≠ correctness; the council never saw the codebase. See
|
|
166
|
+
[`ai-council § Critical evaluation`](../../skills/ai-council/SKILL.md#critical-evaluation--convener-skeptic-stance).
|
|
167
|
+
- **Decline = silence** ([`scope-control`](../../rules/scope-control.md)) —
|
|
168
|
+
if the user picks "Skip — advisory only", the Council review block
|
|
169
|
+
+ Host verdict stay in the roadmap, but no patches are applied. Do
|
|
170
|
+
not re-ask the question on the same task.
|
|
171
|
+
- **Cost gate is non-negotiable** — the deep tier costs more than
|
|
172
|
+
standard; confirm before every billable run, even with
|
|
173
|
+
`personal.autonomy: on`.
|
|
174
|
+
- **No commit.** Patches land in the working tree only; commit
|
|
175
|
+
decisions stay with the user per
|
|
176
|
+
[`commit-policy`](../../rules/commit-policy.md).
|
|
177
|
+
|
|
178
|
+
## See also
|
|
179
|
+
|
|
180
|
+
- [`/roadmap`](../roadmap.md) — cluster orchestrator
|
|
181
|
+
- [`/council default`](../council/default.md) — base flow this command wraps
|
|
182
|
+
- [`ai-council`](../../skills/ai-council/SKILL.md) — neutrality, redaction, deep tier
|
|
183
|
+
- [`scripts/council_cli.py`](../../../scripts/council_cli.py) — CLI entry point
|
|
@@ -168,7 +168,12 @@ If the user picks **2** → continue.
|
|
|
168
168
|
|
|
169
169
|
After saving (and any council review), ask the user (in their language) whether to start executing the roadmap immediately.
|
|
170
170
|
|
|
171
|
-
If yes → switch to
|
|
171
|
+
If yes → switch to [`/roadmap:process-phase`](process-phase.md) with
|
|
172
|
+
the newly created file (the default execution scope of the `/roadmap`
|
|
173
|
+
cluster). Offer [`process-step`](process-step.md) and
|
|
174
|
+
[`process-full`](process-full.md) as alternatives. The legacy
|
|
175
|
+
`/roadmap execute` command was removed — autonomous execution is the
|
|
176
|
+
only path now.
|
|
172
177
|
|
|
173
178
|
### Rules
|
|
174
179
|
|
|
@@ -0,0 +1,58 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: roadmap:process-full
|
|
3
|
+
cluster: roadmap
|
|
4
|
+
sub: process-full
|
|
5
|
+
skills: [agent-docs-writing, ai-council, roadmap-management]
|
|
6
|
+
description: Autonomously process every open step across every phase of a roadmap until the file is fully closed. Largest execution scope of the /roadmap cluster — runs continuously across phase boundaries.
|
|
7
|
+
disable-model-invocation: true
|
|
8
|
+
suggestion:
|
|
9
|
+
eligible: true
|
|
10
|
+
trigger_description: "process the whole roadmap, finish the roadmap, komplette roadmap abarbeiten"
|
|
11
|
+
trigger_context: "existing agents/roadmaps/*.md and user wants the entire file done end-to-end"
|
|
12
|
+
---
|
|
13
|
+
|
|
14
|
+
# /roadmap:process-full
|
|
15
|
+
|
|
16
|
+
Whole-roadmap execution scope of the [`/roadmap`](../roadmap.md)
|
|
17
|
+
cluster. Same canonical loop as
|
|
18
|
+
[`/roadmap:process-phase`](process-phase.md), but does **not** stop at
|
|
19
|
+
phase boundaries — continues until every step is closed (or a halt
|
|
20
|
+
condition fires).
|
|
21
|
+
|
|
22
|
+
## Instructions
|
|
23
|
+
|
|
24
|
+
Run the canonical loop in
|
|
25
|
+
[`contexts/execution/roadmap-process-loop`](../../contexts/execution/roadmap-process-loop.md)
|
|
26
|
+
with the **scope delta below**.
|
|
27
|
+
|
|
28
|
+
## Scope delta
|
|
29
|
+
|
|
30
|
+
- **Working set:** every open step across every phase, in document
|
|
31
|
+
order.
|
|
32
|
+
- **Stop after:** the entire roadmap reaches `count_open == 0`, or a
|
|
33
|
+
halt condition fires (Hard-Floor, council-off + ambiguity,
|
|
34
|
+
security-sensitive, scope-out-of-roadmap, test/quality red).
|
|
35
|
+
- **Phase boundary handling:** at every phase boundary, run the
|
|
36
|
+
per-phase quality pipeline when `quality_cadence: per_phase` (or
|
|
37
|
+
`per_step`). On red → stop, surface, do **not** silently roll into
|
|
38
|
+
the next phase.
|
|
39
|
+
- **Final archival:** when the roadmap is fully closed, run the
|
|
40
|
+
archival check from
|
|
41
|
+
[`roadmap-process-loop § 6`](../../contexts/execution/roadmap-process-loop.md#6-final-report-and-archival).
|
|
42
|
+
|
|
43
|
+
## Rules
|
|
44
|
+
|
|
45
|
+
- **No silent acceleration past a halt.** Every halt condition stops
|
|
46
|
+
the run; the user resumes on the next turn.
|
|
47
|
+
- **Phase quality pipeline runs at every phase boundary** when cadence
|
|
48
|
+
is `per_phase` or `per_step`. `end_of_roadmap` skips per-phase and
|
|
49
|
+
runs only at the final archival check.
|
|
50
|
+
- All other rules from
|
|
51
|
+
[`process-phase § Rules`](process-phase.md#rules) apply unchanged.
|
|
52
|
+
|
|
53
|
+
## See also
|
|
54
|
+
|
|
55
|
+
- [`/roadmap`](../roadmap.md) — cluster orchestrator
|
|
56
|
+
- [`/roadmap:process-step`](process-step.md) — single-step variant
|
|
57
|
+
- [`/roadmap:process-phase`](process-phase.md) — default scope, single phase
|
|
58
|
+
- [`roadmap-process-loop`](../../contexts/execution/roadmap-process-loop.md) — canonical mechanics
|
|
@@ -0,0 +1,69 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: roadmap:process-phase
|
|
3
|
+
cluster: roadmap
|
|
4
|
+
sub: process-phase
|
|
5
|
+
skills: [agent-docs-writing, ai-council, roadmap-management]
|
|
6
|
+
description: Autonomously process every open step in the next or current phase of a roadmap, then stop. Default execution scope of the /roadmap cluster.
|
|
7
|
+
disable-model-invocation: true
|
|
8
|
+
suggestion:
|
|
9
|
+
eligible: true
|
|
10
|
+
trigger_description: "process the next phase, finish this phase autonomously, eine phase abarbeiten"
|
|
11
|
+
trigger_context: "existing agents/roadmaps/*.md and user wants the next phase done end-to-end"
|
|
12
|
+
---
|
|
13
|
+
|
|
14
|
+
# /roadmap:process-phase
|
|
15
|
+
|
|
16
|
+
Default execution scope of the [`/roadmap`](../roadmap.md) cluster.
|
|
17
|
+
Sibling of [`/roadmap:process-step`](process-step.md) and
|
|
18
|
+
[`/roadmap:process-full`](process-full.md). Replaces the legacy
|
|
19
|
+
`/roadmap execute` (which paused for confirmation before every step).
|
|
20
|
+
|
|
21
|
+
## Instructions
|
|
22
|
+
|
|
23
|
+
Run the canonical loop in
|
|
24
|
+
[`contexts/execution/roadmap-process-loop`](../../contexts/execution/roadmap-process-loop.md)
|
|
25
|
+
with the **scope delta below**. The loop file owns roadmap discovery,
|
|
26
|
+
pre-run summary, cadence resolution, commit-step pre-scan, the step
|
|
27
|
+
loop with AI-council branching, halt conditions, and the archival
|
|
28
|
+
check.
|
|
29
|
+
|
|
30
|
+
## Scope delta
|
|
31
|
+
|
|
32
|
+
- **Working set:** all open steps in the **first phase with
|
|
33
|
+
`count_open > 0`**. If every phase is closed → report "Roadmap
|
|
34
|
+
already complete." and run the archival check from
|
|
35
|
+
[`roadmap-process-loop § 6`](../../contexts/execution/roadmap-process-loop.md#6-final-report-and-archival).
|
|
36
|
+
- **Stop after:** the phase boundary. Do **not** advance into the next
|
|
37
|
+
phase. Use [`/roadmap:process-full`](process-full.md) for continuous
|
|
38
|
+
execution across phases.
|
|
39
|
+
- **Quality cadence at the boundary:** run the per-phase pipeline when
|
|
40
|
+
`quality_cadence: per_phase` (or `per_step`). Skip when
|
|
41
|
+
`end_of_roadmap`.
|
|
42
|
+
|
|
43
|
+
## Rules
|
|
44
|
+
|
|
45
|
+
- **Autonomous within the phase, never beyond.** The user picks
|
|
46
|
+
`process-step` for one step or `process-full` for the whole roadmap.
|
|
47
|
+
- **No commit, push, branch, PR, tag, or bulk-destructive op** without
|
|
48
|
+
explicit permission this turn — see
|
|
49
|
+
[`commit-policy`](../../rules/commit-policy.md) and
|
|
50
|
+
[`scope-control § git-ops`](../../rules/scope-control.md#git-operations--permission-gated).
|
|
51
|
+
Roadmap-listed commit steps follow the single-upfront-ask flow in
|
|
52
|
+
[`roadmap-process-loop § 3`](../../contexts/execution/roadmap-process-loop.md#3-commit-step-pre-scan--one-upfront-ask).
|
|
53
|
+
- **Every checkbox edit syncs the dashboard in the same response** per
|
|
54
|
+
[`roadmap-progress-sync`](../../rules/roadmap-progress-sync.md).
|
|
55
|
+
- **AI-council consultations run silently when council is on.** No
|
|
56
|
+
per-call confirmation. The opt-in covers the whole run.
|
|
57
|
+
- **Decline = silence.** Once the user said "skip council", do not
|
|
58
|
+
re-offer for the rest of this run.
|
|
59
|
+
- **Halt cleanly on Hard-Floor or true ambiguity.** Surface state,
|
|
60
|
+
wait. Resume on the user's next turn from the same checkbox.
|
|
61
|
+
|
|
62
|
+
## See also
|
|
63
|
+
|
|
64
|
+
- [`/roadmap`](../roadmap.md) — cluster orchestrator
|
|
65
|
+
- [`/roadmap:process-step`](process-step.md) — single-step variant
|
|
66
|
+
- [`/roadmap:process-full`](process-full.md) — across-phases variant
|
|
67
|
+
- [`/roadmap:create`](create.md) — sibling, scaffolds roadmaps
|
|
68
|
+
- [`roadmap-process-loop`](../../contexts/execution/roadmap-process-loop.md) — canonical mechanics
|
|
69
|
+
- [`roadmap-management`](../../skills/roadmap-management/SKILL.md) — checkbox + archival mechanics
|
|
@@ -0,0 +1,57 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: roadmap:process-step
|
|
3
|
+
cluster: roadmap
|
|
4
|
+
sub: process-step
|
|
5
|
+
skills: [agent-docs-writing, ai-council, roadmap-management]
|
|
6
|
+
description: Autonomously process the single next open step of a roadmap and stop. Smallest execution scope of the /roadmap cluster — one step in, one step out.
|
|
7
|
+
disable-model-invocation: true
|
|
8
|
+
suggestion:
|
|
9
|
+
eligible: true
|
|
10
|
+
trigger_description: "process the next step, do the next roadmap step, einen schritt abarbeiten"
|
|
11
|
+
trigger_context: "existing agents/roadmaps/*.md and user wants exactly one step done autonomously"
|
|
12
|
+
---
|
|
13
|
+
|
|
14
|
+
# /roadmap:process-step
|
|
15
|
+
|
|
16
|
+
One-step execution scope of the [`/roadmap`](../roadmap.md) cluster.
|
|
17
|
+
Same canonical loop as [`/roadmap:process-phase`](process-phase.md),
|
|
18
|
+
bounded to a single iteration.
|
|
19
|
+
|
|
20
|
+
## Instructions
|
|
21
|
+
|
|
22
|
+
Run the canonical loop in
|
|
23
|
+
[`contexts/execution/roadmap-process-loop`](../../contexts/execution/roadmap-process-loop.md)
|
|
24
|
+
with the **scope delta below**.
|
|
25
|
+
|
|
26
|
+
## Scope delta
|
|
27
|
+
|
|
28
|
+
- **Working set:** the **first checkbox `[ ]` in document order**
|
|
29
|
+
inside the first phase with `count_open > 0`. If every step is
|
|
30
|
+
closed → report "Roadmap already complete." and run the archival
|
|
31
|
+
check from
|
|
32
|
+
[`roadmap-process-loop § 6`](../../contexts/execution/roadmap-process-loop.md#6-final-report-and-archival).
|
|
33
|
+
- **Stop after:** one full iteration of
|
|
34
|
+
[`roadmap-process-loop § 5`](../../contexts/execution/roadmap-process-loop.md#5-step-loop)
|
|
35
|
+
(sub-steps 1–7). After the checkbox edit + dashboard regen, **stop**.
|
|
36
|
+
- **Quality cadence:** run the per-step pipeline only when
|
|
37
|
+
`quality_cadence: per_step`. Skip otherwise.
|
|
38
|
+
- **Phase boundary:** if this single step happens to close the phase,
|
|
39
|
+
do **not** advance. Report the phase as complete and stop.
|
|
40
|
+
- **Roadmap boundary:** if this single step happens to close the
|
|
41
|
+
entire roadmap, run the archival check before reporting.
|
|
42
|
+
|
|
43
|
+
## Rules
|
|
44
|
+
|
|
45
|
+
- **Stop after one step**, even if the next step is trivial. The user
|
|
46
|
+
picks `process-phase` or `process-full` when they want more.
|
|
47
|
+
- All other rules from
|
|
48
|
+
[`process-phase § Rules`](process-phase.md#rules) apply unchanged:
|
|
49
|
+
Hard-Floor, no auto-commit, dashboard sync, AI-council silent-when-on,
|
|
50
|
+
decline = silence.
|
|
51
|
+
|
|
52
|
+
## See also
|
|
53
|
+
|
|
54
|
+
- [`/roadmap`](../roadmap.md) — cluster orchestrator
|
|
55
|
+
- [`/roadmap:process-phase`](process-phase.md) — default scope, single phase
|
|
56
|
+
- [`/roadmap:process-full`](process-full.md) — across-phases variant
|
|
57
|
+
- [`roadmap-process-loop`](../../contexts/execution/roadmap-process-loop.md) — canonical mechanics
|
|
@@ -1,44 +1,72 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: roadmap
|
|
3
|
-
description: Roadmap orchestrator — routes to create
|
|
3
|
+
description: Roadmap orchestrator — routes to create (authoring) and process-step / process-phase / process-full (autonomous execution).
|
|
4
4
|
cluster: roadmap
|
|
5
5
|
disable-model-invocation: true
|
|
6
6
|
suggestion:
|
|
7
7
|
eligible: true
|
|
8
|
-
trigger_description: "create a roadmap,
|
|
9
|
-
trigger_context: "user wants to scaffold or
|
|
8
|
+
trigger_description: "create a roadmap, process a roadmap, work through a roadmap autonomously, plan or abarbeiten"
|
|
9
|
+
trigger_context: "user wants to scaffold or autonomously execute a roadmap under agents/roadmaps/"
|
|
10
10
|
---
|
|
11
11
|
|
|
12
12
|
# /roadmap
|
|
13
13
|
|
|
14
|
-
Top-level orchestrator for the `/roadmap` family.
|
|
15
|
-
|
|
14
|
+
Top-level orchestrator for the `/roadmap` family. Carries authoring
|
|
15
|
+
(`create`) and the three autonomous-execution scopes (`process-step`,
|
|
16
|
+
`process-phase`, `process-full`). The legacy `/roadmap execute` (which
|
|
17
|
+
paused for confirmation before every step) was removed —
|
|
18
|
+
`process-phase` is the default execution scope.
|
|
16
19
|
|
|
17
20
|
## Sub-commands
|
|
18
21
|
|
|
19
22
|
| Sub-command | Routes to | Purpose |
|
|
20
23
|
|---|---|---|
|
|
21
|
-
| `/roadmap
|
|
22
|
-
| `/roadmap
|
|
24
|
+
| `/roadmap:create` | `commands/roadmap/create.md` | Interactively scaffold a new roadmap in `agents/roadmaps/` |
|
|
25
|
+
| `/roadmap:ai-council` | `commands/roadmap/ai-council.md` | Challenge an existing roadmap with the AI council (deep tier) and refactor from convergence findings |
|
|
26
|
+
| `/roadmap:process-step` | `commands/roadmap/process-step.md` | Autonomously process the next open step, then stop |
|
|
27
|
+
| `/roadmap:process-phase` (**default execution scope**) | `commands/roadmap/process-phase.md` | Autonomously process every open step in the current phase |
|
|
28
|
+
| `/roadmap:process-full` | `commands/roadmap/process-full.md` | Autonomously process every open step across every phase |
|
|
23
29
|
|
|
24
30
|
Sub-command names match the locked contract in
|
|
25
|
-
[`docs/contracts/command-clusters.md`](
|
|
31
|
+
[`docs/contracts/command-clusters.md`](../docs/contracts/command-clusters.md).
|
|
32
|
+
`:` and space are equivalent at the cluster boundary — see
|
|
33
|
+
[`slash-command-routing-policy-mechanics`](../contexts/communication/rules-auto/slash-command-routing-policy-mechanics.md#routing-semantics).
|
|
34
|
+
The three `process-*` subs share the canonical loop in
|
|
35
|
+
[`contexts/execution/roadmap-process-loop`](../contexts/execution/roadmap-process-loop.md);
|
|
36
|
+
each only binds a scope delta.
|
|
26
37
|
|
|
27
38
|
## Dispatch
|
|
28
39
|
|
|
29
|
-
1. Parse the user's argument: `/roadmap
|
|
40
|
+
1. Parse the user's argument: `/roadmap[:<sub>] [args]` or
|
|
41
|
+
`/roadmap <sub> [args]`.
|
|
30
42
|
2. Look up the sub-command in the table above.
|
|
31
|
-
3. Load the body of the routed file and follow its `## Instructions`
|
|
32
|
-
verbatim with the remaining args.
|
|
33
|
-
4.
|
|
43
|
+
3. Load the body of the routed file and follow its `## Instructions`
|
|
44
|
+
section verbatim with the remaining args.
|
|
45
|
+
4. **Legacy forwarding:**
|
|
46
|
+
- `/roadmap execute` or `/roadmap-execute` → forward to
|
|
47
|
+
[`/roadmap:process-phase`](roadmap/process-phase.md) (default
|
|
48
|
+
scope) with a one-time migration notice.
|
|
49
|
+
- `/roadmap-process[:<sub>]` (legacy top-level cluster) → forward
|
|
50
|
+
to `/roadmap:process-<sub>` with a one-time migration notice.
|
|
51
|
+
5. If the sub-command is unknown or missing, print the table above
|
|
52
|
+
and ask:
|
|
34
53
|
|
|
35
54
|
> 1. create — scaffold a new roadmap interactively
|
|
36
|
-
> 2.
|
|
55
|
+
> 2. ai-council — challenge + refactor an existing roadmap (deep tier)
|
|
56
|
+
> 3. process-step — process the next open step, then stop
|
|
57
|
+
> 4. process-phase — process the current phase (default)
|
|
58
|
+
> 5. process-full — process every open step across every phase
|
|
37
59
|
|
|
38
60
|
## Rules
|
|
39
61
|
|
|
40
|
-
- **Do NOT commit, push, or open a PR** unless the sub-command
|
|
41
|
-
authorizes it.
|
|
62
|
+
- **Do NOT commit, push, or open a PR** unless the sub-command
|
|
63
|
+
explicitly authorizes it. Roadmap-listed commit steps follow the
|
|
64
|
+
single-upfront-ask flow in
|
|
65
|
+
[`roadmap-process-loop § 3`](../contexts/execution/roadmap-process-loop.md#3-commit-step-pre-scan--one-upfront-ask).
|
|
42
66
|
- **Do NOT chain sub-commands.** One `/roadmap <sub>` per turn.
|
|
43
|
-
- If the user invokes `/roadmap` with no argument, **show the menu** —
|
|
44
|
-
not guess which sub-command they meant.
|
|
67
|
+
- If the user invokes `/roadmap` with no argument, **show the menu** —
|
|
68
|
+
do not guess which sub-command they meant.
|
|
69
|
+
- Execution intents (*"work through the roadmap"*, *"abarbeiten"*,
|
|
70
|
+
*"finish this phase"*) default to
|
|
71
|
+
[`/roadmap:process-phase`](roadmap/process-phase.md) unless the user
|
|
72
|
+
named a different scope.
|
|
@@ -25,7 +25,7 @@ the [`agent-settings` template](../templates/agent-settings.md#cost-profiles):
|
|
|
25
25
|
- For first-run setup use [`/onboard`](onboard.md).
|
|
26
26
|
- For any other single-value change, edit `.agent-settings.yml`
|
|
27
27
|
directly or ask the agent — the merge rules live in
|
|
28
|
-
[`layered-settings`](
|
|
28
|
+
[`layered-settings`](../docs/guidelines/agent-infra/layered-settings.md#section-aware-merge-rules).
|
|
29
29
|
- For role modes use [`/mode`](mode.md) — different concept (sets
|
|
30
30
|
`roles.active_role`, not `cost_profile`).
|
|
31
31
|
|
|
@@ -73,7 +73,7 @@ value directly — still echo the old → new line in step 6.
|
|
|
73
73
|
### 5. Write the value
|
|
74
74
|
|
|
75
75
|
Update `cost_profile` in `.agent-settings.yml` using the
|
|
76
|
-
[section-aware merge rules](
|
|
76
|
+
[section-aware merge rules](../docs/guidelines/agent-infra/layered-settings.md#section-aware-merge-rules)
|
|
77
77
|
(preserve comments, preserve key order, touch only the changed field).
|
|
78
78
|
|
|
79
79
|
If the user picked "Keep current", do nothing and stop.
|
|
@@ -107,6 +107,6 @@ flip. Cost behaviour on those surfaces is governed by the platform itself.
|
|
|
107
107
|
## See also
|
|
108
108
|
|
|
109
109
|
- [`agent-settings`](../templates/agent-settings.md) — profile matrix and settings reference
|
|
110
|
-
- [`layered-settings`](
|
|
110
|
+
- [`layered-settings`](../docs/guidelines/agent-infra/layered-settings.md) — merge rules for settings edits
|
|
111
111
|
- [`onboard`](onboard.md) — first-run setup (includes profile confirmation)
|
|
112
112
|
- [`mode`](mode.md) — role-mode setter (different concept)
|
|
@@ -15,7 +15,7 @@ Reconciles `.agent-settings.yml` with the shipped template
|
|
|
15
15
|
(`config/agent-settings.template.yml`) and the selected cost-profile
|
|
16
16
|
preset (`config/profiles/{profile}.ini`). Applies the section-aware
|
|
17
17
|
merge rules documented in
|
|
18
|
-
[`layered-settings`](
|
|
18
|
+
[`layered-settings`](../docs/guidelines/agent-infra/layered-settings.md):
|
|
19
19
|
|
|
20
20
|
- Template section order wins — keys reorder to match.
|
|
21
21
|
- Existing user scalar values are preserved.
|
|
@@ -129,6 +129,6 @@ is a local-agent concern.
|
|
|
129
129
|
- [`scripts/sync_agent_settings.py`](../../../scripts/sync_agent_settings.py) — the helper
|
|
130
130
|
- [`config/agent-settings.template.yml`](../../../config/agent-settings.template.yml) — canonical template
|
|
131
131
|
- [`config/profiles/`](../../../config/profiles/) — profile presets
|
|
132
|
-
- [`layered-settings`](
|
|
132
|
+
- [`layered-settings`](../docs/guidelines/agent-infra/layered-settings.md) — the merge rules this command enforces
|
|
133
133
|
- [`scripts/install.py`](../../../scripts/install.py) — first-install path; this command handles the update path
|
|
134
134
|
- [`/sync-gitignore`](sync-gitignore.md) — sibling command for the `.gitignore` block
|
|
@@ -59,7 +59,7 @@ suggestion:
|
|
|
59
59
|
- Place tests in the matching directory structure under `tests/` (mirror the source structure).
|
|
60
60
|
- Group related tests with `describe` blocks (Pest) or separate test methods (PHPUnit).
|
|
61
61
|
- Use data providers for testing multiple input/output combinations.
|
|
62
|
-
- Mock external
|
|
62
|
+
- Mock external deps (database, HTTP, file system) — don't test infrastructure.
|
|
63
63
|
|
|
64
64
|
### 6. Verify
|
|
65
65
|
|
|
@@ -70,7 +70,7 @@ suggestion:
|
|
|
70
70
|
|
|
71
71
|
- **Do NOT commit or push.**
|
|
72
72
|
- **Quality over quantity** — 5 meaningful tests beat 20 trivial ones.
|
|
73
|
-
- If a class is hard to test (too many
|
|
73
|
+
- If a class is hard to test (too many deps, global state), flag it and suggest a refactoring approach instead of writing brittle
|
|
74
74
|
tests.
|
|
75
75
|
|
|
76
76
|
## See also
|
|
@@ -22,7 +22,7 @@ commands with a single entry point + sub-command dispatch.
|
|
|
22
22
|
| `/tests execute` | `commands/tests/execute.md` | Run PHP tests inside the Docker container |
|
|
23
23
|
|
|
24
24
|
Sub-command names match the locked contract in
|
|
25
|
-
[`docs/contracts/command-clusters.md`](
|
|
25
|
+
[`docs/contracts/command-clusters.md`](../docs/contracts/command-clusters.md).
|
|
26
26
|
|
|
27
27
|
## Dispatch
|
|
28
28
|
|
|
@@ -3,6 +3,7 @@ name: threat-model
|
|
|
3
3
|
skills: [threat-modeling, authz-review, security-sensitive-stop]
|
|
4
4
|
description: Run a pre-implementation threat model on a proposed change — enumerates abuse cases, trust boundaries, and authorization gaps before the first line of code is written
|
|
5
5
|
disable-model-invocation: true
|
|
6
|
+
council_depth: deep
|
|
6
7
|
suggestion:
|
|
7
8
|
eligible: true
|
|
8
9
|
trigger_description: "threat model this change, what could go wrong security-wise"
|
|
@@ -71,7 +72,7 @@ Produce one combined report with these sections, in order:
|
|
|
71
72
|
5. **Required controls** — the minimum set of validations, authorization
|
|
72
73
|
checks, logging, and negative tests that must exist before the change ships
|
|
73
74
|
6. **Open questions** — anything the skills flagged as uncertain and the user
|
|
74
|
-
must answer before
|
|
75
|
+
must answer before impl starts
|
|
75
76
|
|
|
76
77
|
### 5. Decide next step
|
|
77
78
|
|
|
@@ -83,7 +84,7 @@ After the report, ask:
|
|
|
83
84
|
> 3. Stop here — threat model is the deliverable, no implementation yet
|
|
84
85
|
```
|
|
85
86
|
|
|
86
|
-
- On **1**: hand off to the
|
|
87
|
+
- On **1**: hand off to the impl flow (e.g., `feature-plan`,
|
|
87
88
|
`bug-fix`, or direct edit) with the required-controls list pinned
|
|
88
89
|
- On **2**: re-gather context and re-dispatch
|
|
89
90
|
- On **3**: save the report as the deliverable, stop
|
|
@@ -106,7 +107,7 @@ After the report, ask:
|
|
|
106
107
|
- NEVER merge `threat-modeling` and `authz-review` outputs into a single
|
|
107
108
|
block — each skill owns its format
|
|
108
109
|
- NEVER write production code in the same turn as this command — the
|
|
109
|
-
deliverable is the report;
|
|
110
|
+
deliverable is the report; impl is a separate step
|
|
110
111
|
- NEVER mark the change "safe" if any 🔴 abuse case has no control
|
|
111
112
|
|
|
112
113
|
## See also
|
|
@@ -116,4 +117,4 @@ After the report, ask:
|
|
|
116
117
|
- [`data-flow-mapper`](../skills/data-flow-mapper/SKILL.md) — trace specific data through the change
|
|
117
118
|
- [`blast-radius-analyzer`](../skills/blast-radius-analyzer/SKILL.md) — enumerate affected call sites
|
|
118
119
|
- [`security-sensitive-stop`](../rules/security-sensitive-stop.md) — the trigger rule
|
|
119
|
-
- [`minimal-safe-diff`](../rules/minimal-safe-diff.md) — keep the
|
|
120
|
+
- [`minimal-safe-diff`](../rules/minimal-safe-diff.md) — keep the impl scoped
|
|
@@ -106,7 +106,7 @@ Commands organized by workflow:
|
|
|
106
106
|
| **Bugs** | `bug-investigate`, `bug-fix` |
|
|
107
107
|
| **Contexts** | `context-create`, `context-refactor` |
|
|
108
108
|
| **Modules** | `module-create`, `module-explore` |
|
|
109
|
-
| **Roadmaps** | `roadmap
|
|
109
|
+
| **Roadmaps** | `roadmap:create`, `roadmap:process-step`, `roadmap:process-phase`, `roadmap:process-full` |
|
|
110
110
|
| **Quality** | `quality-fix`, `review-changes`, `prepare-for-review`, `update-form-request-messages`, `fix-seeder` |
|
|
111
111
|
| **CI/PR** | `fix-ci`, `create-pr`, `create-pr-description`, `fix-pr-comments`, `fix-pr-bot-comments`, `fix-pr-developer-comments` |
|
|
112
112
|
| **Testing** | `tests-create`, `tests-execute` |
|
|
@@ -8,6 +8,19 @@ in autonomous vs. non-autonomous mode.
|
|
|
8
8
|
**Size budget:** ≤ 3,000 chars. Tracked under Phase 6 of
|
|
9
9
|
`road-to-pr-34-followups`.
|
|
10
10
|
|
|
11
|
+
## The four commit exceptions — restated
|
|
12
|
+
|
|
13
|
+
Outside the four below, no commit is allowed and no commit ask is allowed.
|
|
14
|
+
|
|
15
|
+
1. **User says so this turn** — "commit this now", "commit it", "go
|
|
16
|
+
ahead and commit". Single commit, not standing.
|
|
17
|
+
2. **Standing instruction not yet revoked** — earlier "commit after
|
|
18
|
+
every phase"; cache the instruction and commit per its terms.
|
|
19
|
+
3. **Commit command invoked** — `/commit` (split + confirm per commit)
|
|
20
|
+
or `/commit:in-chunks` (auto-split, Hard Floor still applies).
|
|
21
|
+
4. **Roadmap authorization** — roadmap lists explicit commit steps and
|
|
22
|
+
the user invoked roadmap execution; each commit matches a step.
|
|
23
|
+
|
|
11
24
|
## Hard Floor still applies — bulk deletions and infra changes
|
|
12
25
|
|
|
13
26
|
Even when one of the four `commit-policy` exceptions authorizes a
|
|
@@ -19,7 +32,7 @@ Hard Floor still fires when the diff:
|
|
|
19
32
|
- Touches Terraform / Pulumi / k8s manifests / Ansible / cloud-config
|
|
20
33
|
|
|
21
34
|
In those cases, **surface the diff** (paths + counts) and confirm
|
|
22
|
-
this turn before committing — even under `/commit
|
|
35
|
+
this turn before committing — even under `/commit:in-chunks`,
|
|
23
36
|
roadmap pre-scan authorization, or an explicit "commit this now". The
|
|
24
37
|
four exceptions cover *whether* commits happen; the Hard Floor covers
|
|
25
38
|
*which diffs* still need a separate confirmation.
|
|
@@ -9,6 +9,19 @@ context holds everything an agent reaches for once those have fired.
|
|
|
9
9
|
**Size budget:** ≤ 3,500 chars. Tracked under Phase 7.4 of
|
|
10
10
|
`road-to-pr-34-followups`.
|
|
11
11
|
|
|
12
|
+
## Iron Law — restatement
|
|
13
|
+
|
|
14
|
+
The rule's Iron Law ("HARD FLOOR OVERRIDES EVERYTHING") is the universal
|
|
15
|
+
non-destructive-by-default safety net. No autonomy setting, roadmap step,
|
|
16
|
+
or standing instruction can lift it. This is the same Iron Law restated
|
|
17
|
+
in `commit-policy` (row 6 of its trigger table) and in `scope-control`
|
|
18
|
+
(production / infra / bulk-destructive subset).
|
|
19
|
+
|
|
20
|
+
The trigger surface covers production-branch merges, deploys / releases,
|
|
21
|
+
production data and infrastructure changes, pushes to remote, and
|
|
22
|
+
whimsical or unscoped bulk-destructive operations. Authorization is
|
|
23
|
+
"user said so this turn", never inferred from a previous turn.
|
|
24
|
+
|
|
12
25
|
## Bulk deletions during WIP — allowed if task-connected
|
|
13
26
|
|
|
14
27
|
Deletions inside an **active, user-stated task** are allowed in the
|
|
@@ -58,7 +71,7 @@ the diff (paths + counts), get confirmation, then commit.
|
|
|
58
71
|
- Committing a diff that removes a directory, deletes ≥5 unrelated
|
|
59
72
|
files, or touches Terraform / k8s manifests / Ansible without
|
|
60
73
|
surfacing the diff first — even when [`commit-policy`](../../rules/commit-policy.md)
|
|
61
|
-
otherwise authorizes commits (e.g. `/commit
|
|
74
|
+
otherwise authorizes commits (e.g. `/commit:in-chunks`, roadmap
|
|
62
75
|
pre-scan, an explicit "commit this now"). Bulk-deletion / infra
|
|
63
76
|
commits need their own ask, every time.
|
|
64
77
|
- Reading a roadmap step listing files to delete as authorization to
|