create-agdf 0.2.8 → 0.3.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/README.md +27 -0
- package/bin/create-agdf.js +63 -10
- package/generated/.agdf/control/README.md +5 -5
- package/generated/.github/skills/agdf-gate-check/SKILL.md +4 -4
- package/generated/.opencode/AGDF.md +102 -0
- package/generated/.opencode/README.md +19 -0
- package/generated/.opencode/agdf-runtime-contract.md +219 -0
- package/generated/.opencode/agents/agdf-brownfield-analysis.md +142 -0
- package/generated/.opencode/agents/agdf-clean-implementation-review.md +106 -0
- package/generated/.opencode/agents/agdf-code-review.md +100 -0
- package/generated/.opencode/agents/agdf-delivery-closeout.md +115 -0
- package/generated/.opencode/agents/agdf-gate-check.md +155 -0
- package/generated/.opencode/agents/agdf-qa-gate.md +116 -0
- package/generated/.opencode/agents/agdf-release-or.md +130 -0
- package/generated/.opencode/agents/agdf-task-plan-review.md +121 -0
- package/generated/opencode.json +13 -0
- package/generated/plugins/agdf/.codex-plugin/plugin.json +3 -1
- package/generated/plugins/agdf/assets/agdf-icon.svg +16 -0
- package/generated/plugins/agdf/assets/agdf-logo.svg +16 -0
- package/generated/plugins/agdf/control/README.md +5 -5
- package/generated/plugins/agdf/hooks/session-start.sh +2 -2
- package/generated/plugins/agdf/meta/agdf-plugin.definition.json +13 -1
- package/generated/plugins/agdf/skills/gate-check/SKILL.md +4 -4
- package/opencode-plugin.js +33 -0
- package/package.json +9 -2
|
@@ -0,0 +1,142 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: after gate-check permits Brownfield Review or implementation preparation, before non-trivial changes in existing systems
|
|
3
|
+
mode: subagent
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
---
|
|
7
|
+
name: agdf-brownfield-analysis
|
|
8
|
+
description: Use this skill after gate-check or live AGDF control state confirms that Brownfield Review or implementation preparation is allowed. Use it after Approval: UR as the lightweight Brownfield Review after G-00 before PRD when existing systems may be affected, and again before non-trivial implementation after TP. Do not use it as the first AGDF skill for a fresh "I want to build/change X" prompt when approval, scope, or next allowed action is unclear.
|
|
9
|
+
---
|
|
10
|
+
|
|
11
|
+
# brownfield-analysis
|
|
12
|
+
|
|
13
|
+
## Purpose
|
|
14
|
+
Ensure delivery in an existing system is Brownfield-oriented, not Greenfield-style.
|
|
15
|
+
|
|
16
|
+
The skill answers:
|
|
17
|
+
|
|
18
|
+
- which existing artefacts matter
|
|
19
|
+
- what already exists fully or partially
|
|
20
|
+
- what can be reused, extended, or refactored
|
|
21
|
+
- whether new artefacts are really needed
|
|
22
|
+
- architecture, compatibility, migration, and regression impact
|
|
23
|
+
- the minimal clean next step
|
|
24
|
+
- whether Context Graph impact exists without automatically creating a node
|
|
25
|
+
|
|
26
|
+
## Runtime Contract
|
|
27
|
+
Use `../agdf-runtime-contract.md` for Quality Contract output, Context Graph fields, gate terms, and non-duplication rules.
|
|
28
|
+
|
|
29
|
+
Brownfield-specific output must make evidence, missing existing-system view, parallel-structure risk, reuse strategy, and the minimal next step visible.
|
|
30
|
+
When `.agdf/control/` is present, persist or link `post_ur_review` output under `.agdf/control/artefacts/<key>/BROWNFIELD_REVIEW.md`.
|
|
31
|
+
|
|
32
|
+
## Modes
|
|
33
|
+
|
|
34
|
+
- `post_ur_review`: use after approved durable UR to size and route the work. Output must decide `quick_task`, `structured_slice`, `structured_delivery`, or `block` with scope reason and evidence.
|
|
35
|
+
- `pre_implementation_analysis`: use after approved durable TP to verify the implementation path before `CD+Tests`. Output must focus on reuse path, owners, regression risk, test impact and minimal clean implementation.
|
|
36
|
+
|
|
37
|
+
Do not mix both modes silently. Name the active mode in the output.
|
|
38
|
+
|
|
39
|
+
## Rules
|
|
40
|
+
1. Brownfield first: understand the existing codebase before PRD/SD decisions when existing-system impact is possible, and again before implementation.
|
|
41
|
+
2. Brownfield Review after `Approval: UR` is a sizing and routing step. It must visibly decide `quick_task`, `structured_slice`, `structured_delivery`, or `block` before PRD depth or implementation is chosen.
|
|
42
|
+
3. Reuse-before-create: prefer existing modules, services, components, tables, endpoints, tests, and configuration.
|
|
43
|
+
4. Minimal clean slice: choose the smallest durable intervention, not merely the smallest technical diff.
|
|
44
|
+
5. No silent parallel structures.
|
|
45
|
+
6. Existing architecture, naming, error handling, logging, security, and test conventions are binding unless deviation is justified.
|
|
46
|
+
7. What is not visibly evidenced does not count as existing.
|
|
47
|
+
8. Async execute paths must not become a second product-policy decision point.
|
|
48
|
+
9. Visible state ownership must be checked for chat, rendering, scrolling, recovery, and status problems.
|
|
49
|
+
10. SoT/runtime/product-semantics drift must be named; if official behaviour must be decided first, recommend UR or equivalent product direction.
|
|
50
|
+
11. Large UI surfaces and state hooks must be checked for mixed render, view-model, derived state, orchestration, persistence, and recovery ownership.
|
|
51
|
+
12. Context Graph impact must be curated; it is not a review log or version index.
|
|
52
|
+
13. Specification archive migrations must follow the archive index if present.
|
|
53
|
+
|
|
54
|
+
## When To Use
|
|
55
|
+
- after `agdf-gate-check` permits `Brownfield Review` or live `.agdf/control/AGDF_RUN.md` names Brownfield Review as the next allowed action
|
|
56
|
+
- after `agdf-gate-check` permits implementation preparation or live `.agdf/control/AGDF_RUN.md` already names Brownfield Analysis as the next allowed action
|
|
57
|
+
- after `Approval: UR` when existing owners, SoT, contracts, policies, persistence, runtime paths, tests, UI, UX or architecture may affect PRD/SD scope
|
|
58
|
+
- before `CD+Tests`
|
|
59
|
+
- before changes to an existing repository
|
|
60
|
+
- bug fixes in existing modules
|
|
61
|
+
- feature extensions in existing architecture
|
|
62
|
+
- refactorings
|
|
63
|
+
- unclear existing coverage
|
|
64
|
+
- risk of new parallel structures
|
|
65
|
+
- a TP is clear but its connection to the existing system is not
|
|
66
|
+
|
|
67
|
+
## Inputs
|
|
68
|
+
Use what is available:
|
|
69
|
+
|
|
70
|
+
- approved UR, Task Plan, `task_id`, `story_id`
|
|
71
|
+
- project structure
|
|
72
|
+
- affected files, modules, services, APIs, data models
|
|
73
|
+
- existing tests
|
|
74
|
+
- architecture notes from PRD or SD
|
|
75
|
+
- repository conventions
|
|
76
|
+
|
|
77
|
+
If inputs are missing, work only from observable evidence and mark gaps explicitly.
|
|
78
|
+
If gate status, approval, scope or next allowed action is unclear, stop and route to `agdf-gate-check` instead of recommending PRD/SD detail or starting implementation.
|
|
79
|
+
When used as Brownfield Review after `Approval: UR`, do not recommend PRD, SD, TP, or implementation until the Mode/Slice Decision is explicit, evidenced and recorded.
|
|
80
|
+
|
|
81
|
+
## Workflow
|
|
82
|
+
1. Identify affected tasks and scope.
|
|
83
|
+
2. Find relevant existing artefacts.
|
|
84
|
+
3. Assess current coverage before new implementation:
|
|
85
|
+
- `fully_done`
|
|
86
|
+
- `partially_done`
|
|
87
|
+
- `not_done`
|
|
88
|
+
4. Choose reuse strategy:
|
|
89
|
+
- `extend`
|
|
90
|
+
- `refactor`
|
|
91
|
+
- `replace`
|
|
92
|
+
- `new`
|
|
93
|
+
5. Assess change impact:
|
|
94
|
+
- files/modules
|
|
95
|
+
- interfaces
|
|
96
|
+
- data model/migrations
|
|
97
|
+
- backwards compatibility
|
|
98
|
+
- regression tests
|
|
99
|
+
- side effects
|
|
100
|
+
6. Check parallel-structure risk.
|
|
101
|
+
7. Check SoT/runtime/product-semantics drift.
|
|
102
|
+
8. Check primary visible ownership for UI/status/recovery paths.
|
|
103
|
+
9. Check UI monolith risk for large surfaces or central hooks.
|
|
104
|
+
10. Check Context Graph impact according to `../agdf-runtime-contract.md`.
|
|
105
|
+
11. Recommend the minimal clean implementation path.
|
|
106
|
+
|
|
107
|
+
## Output
|
|
108
|
+
Use a concise structure:
|
|
109
|
+
|
|
110
|
+
```text
|
|
111
|
+
## Brownfield Analysis
|
|
112
|
+
- mode: post_ur_review | pre_implementation_analysis
|
|
113
|
+
- decision: pass | revise | block | not_applicable
|
|
114
|
+
- mode_slice_decision: quick_task | structured_slice | structured_delivery | block
|
|
115
|
+
- required_next_gate: none | PRD | SD | TP | Brownfield Analysis
|
|
116
|
+
- artefact: .agdf/control/artefacts/<key>/BROWNFIELD_REVIEW.md | none
|
|
117
|
+
- scope:
|
|
118
|
+
- evidence:
|
|
119
|
+
- transparency: why later artefacts are skipped, shortened or required
|
|
120
|
+
- missing_evidence:
|
|
121
|
+
- current_coverage:
|
|
122
|
+
- reuse_strategy:
|
|
123
|
+
- risks:
|
|
124
|
+
- context_graph_impact:
|
|
125
|
+
- required_next_step:
|
|
126
|
+
```
|
|
127
|
+
|
|
128
|
+
## Pass / Revise / Block Guidance
|
|
129
|
+
- `pass`: existing owners are understood, reuse path is clear, and no blocking drift or parallel structure risk remains.
|
|
130
|
+
- `revise`: relevant evidence, tests, ownership, or reuse path is incomplete.
|
|
131
|
+
- `block`: implementation would create a second SoT, second owner, unresolved product semantics, or hard unmitigated risk.
|
|
132
|
+
|
|
133
|
+
## Forbidden
|
|
134
|
+
This skill must not:
|
|
135
|
+
|
|
136
|
+
- propose a Greenfield replacement without evidence
|
|
137
|
+
- silently accept overlapping ownership
|
|
138
|
+
- convert product-semantics drift into a mere refactor
|
|
139
|
+
- create Context Graph nodes automatically
|
|
140
|
+
- treat archive links as active current specification signals
|
|
141
|
+
- recommend broad PRD/SD/TP or implementation by reflex before Mode/Slice Decision is recorded
|
|
142
|
+
|
|
@@ -0,0 +1,106 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: inspect workarounds, fallbacks, parallel structures or symptom fixes
|
|
3
|
+
mode: subagent
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
---
|
|
7
|
+
name: agdf-clean-implementation-review
|
|
8
|
+
description: Use this skill to determine whether an implementation is a clean primary solution or whether fallbacks, workarounds, guards, defaults, shims, or parallel structures have made it unnecessarily complex. Use it after code changes, before QA, or whenever symptom treatment is suspected.
|
|
9
|
+
---
|
|
10
|
+
|
|
11
|
+
# clean-implementation-review
|
|
12
|
+
|
|
13
|
+
## Purpose
|
|
14
|
+
Evaluate implementation integrity.
|
|
15
|
+
|
|
16
|
+
This skill answers:
|
|
17
|
+
|
|
18
|
+
- was the task solved with a clean primary solution
|
|
19
|
+
- was the root cause fixed or only masked
|
|
20
|
+
- were unnecessary fallbacks, workarounds, guards, defaults, or shims introduced
|
|
21
|
+
- were unnecessary parallel structures introduced
|
|
22
|
+
- does the solution fit the existing architecture
|
|
23
|
+
- is it maintainable or workaround-heavy
|
|
24
|
+
|
|
25
|
+
## Runtime Contract
|
|
26
|
+
Use `../agdf-runtime-contract.md` for Quality Contract output, Context Graph fields, gate terms, and non-duplication rules.
|
|
27
|
+
|
|
28
|
+
Clean-review-specific output must make the primary solution, fallbacks, workarounds, parallel structures, exit criteria, and next cleanup/review step visible.
|
|
29
|
+
|
|
30
|
+
## Rules
|
|
31
|
+
1. Clean primary solution before fallback.
|
|
32
|
+
2. Root cause before symptom masking.
|
|
33
|
+
3. Fallbacks are exceptions, not target architecture.
|
|
34
|
+
4. Every retained fallback needs rationale, target state, and exit condition.
|
|
35
|
+
5. No silent parallel structures.
|
|
36
|
+
6. Brownfield fit is mandatory.
|
|
37
|
+
7. Maintainability matters; "works somehow" is not enough.
|
|
38
|
+
8. Async execute paths must not decide product rules again.
|
|
39
|
+
9. Shared finalization or merge rules belong in one owner.
|
|
40
|
+
|
|
41
|
+
## When To Use
|
|
42
|
+
- after `CD+Tests`
|
|
43
|
+
- before QA
|
|
44
|
+
- when a solution contains many guards, defaults, or special paths
|
|
45
|
+
- when wrappers or helper layers were introduced
|
|
46
|
+
- when symptom treatment is suspected
|
|
47
|
+
- when fallback, retry, or compatibility-shim logic was added
|
|
48
|
+
- when code works but the architecture looks questionable
|
|
49
|
+
|
|
50
|
+
## Inputs
|
|
51
|
+
Use what is available:
|
|
52
|
+
|
|
53
|
+
- Task Plan
|
|
54
|
+
- Code Deliverables
|
|
55
|
+
- changed files
|
|
56
|
+
- tests and test results
|
|
57
|
+
- `agdf-code-review` or Code Review Report
|
|
58
|
+
- QA Report
|
|
59
|
+
- Brownfield Analysis findings
|
|
60
|
+
- PRD or SD architecture notes
|
|
61
|
+
|
|
62
|
+
If evidence is missing, mark it explicitly.
|
|
63
|
+
|
|
64
|
+
## Workflow
|
|
65
|
+
1. Understand the intended change.
|
|
66
|
+
2. Identify the primary implementation path.
|
|
67
|
+
3. Identify fallbacks, guards, defaults, shims, retries, wrappers, and catch-all branches.
|
|
68
|
+
4. Decide whether each is justified.
|
|
69
|
+
5. Check for parallel ownership.
|
|
70
|
+
6. Check Brownfield fit.
|
|
71
|
+
7. Assess whether the implementation fixes the root cause.
|
|
72
|
+
8. Assign a decision:
|
|
73
|
+
- `pass`
|
|
74
|
+
- `revise`
|
|
75
|
+
- `block`
|
|
76
|
+
- `not_applicable`
|
|
77
|
+
|
|
78
|
+
## Output
|
|
79
|
+
Use this compact structure:
|
|
80
|
+
|
|
81
|
+
```text
|
|
82
|
+
## Clean Implementation Review
|
|
83
|
+
- decision:
|
|
84
|
+
- primary_solution:
|
|
85
|
+
- evidence:
|
|
86
|
+
- fallbacks_retained:
|
|
87
|
+
- workaround_or_shim_risk:
|
|
88
|
+
- parallel_structure_risk:
|
|
89
|
+
- brownfield_fit:
|
|
90
|
+
- missing_evidence:
|
|
91
|
+
- required_next_step:
|
|
92
|
+
```
|
|
93
|
+
|
|
94
|
+
## Pass / Revise / Block Guidance
|
|
95
|
+
- `pass`: primary solution is clean, tested, and integrated with existing owners.
|
|
96
|
+
- `revise`: solution works but contains avoidable fallback, unclear ownership, or missing evidence.
|
|
97
|
+
- `block`: solution creates a second SoT, a second owner, unbounded fallback, or masks unresolved product semantics.
|
|
98
|
+
|
|
99
|
+
## Forbidden
|
|
100
|
+
This skill must not:
|
|
101
|
+
|
|
102
|
+
- accept fallback-heavy logic as target architecture
|
|
103
|
+
- hide unjustified workarounds
|
|
104
|
+
- treat green tests as proof of solution integrity
|
|
105
|
+
- decide final QA
|
|
106
|
+
|
|
@@ -0,0 +1,100 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: mandatory CR step after code changes, focused on defects, regression and security findings
|
|
3
|
+
mode: subagent
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
---
|
|
7
|
+
name: agdf-code-review
|
|
8
|
+
description: Use this skill after code changes and before QA to produce the mandatory Code Review Report for correctness, regression, security, and maintainability findings in the actual diff. It standardizes the CR step without replacing TP review, clean review, or QA.
|
|
9
|
+
---
|
|
10
|
+
|
|
11
|
+
# code-review
|
|
12
|
+
|
|
13
|
+
## Purpose
|
|
14
|
+
Produce the mandatory Code Review Report (CR) for a relevant run.
|
|
15
|
+
|
|
16
|
+
It answers:
|
|
17
|
+
|
|
18
|
+
- whether the changed code shows functional defects, security issues, regression risks, or maintainability problems that matter
|
|
19
|
+
- whether findings are blocking, revisable, or advisory
|
|
20
|
+
- which concrete evidence in files, diffs, tests, or runtime observations supports each finding
|
|
21
|
+
- which issues must be fixed before QA
|
|
22
|
+
- whether Brownfield, TP, or clean-review follow-up remains open
|
|
23
|
+
|
|
24
|
+
## Runtime Contract
|
|
25
|
+
Use `../agdf-runtime-contract.md` for Quality Contract output, Context Graph fields, gate terms, and non-duplication rules.
|
|
26
|
+
|
|
27
|
+
Code-review-specific output must make actual findings, missing review scope, evidence strength, and the next required step visible.
|
|
28
|
+
|
|
29
|
+
## Rules
|
|
30
|
+
1. Review the actual diff and impacted code, not intent alone.
|
|
31
|
+
2. Findings require concrete evidence.
|
|
32
|
+
3. Prioritize correctness, safety, data integrity, compatibility, and maintainability over style.
|
|
33
|
+
4. Do not generate style-only noise.
|
|
34
|
+
5. Missing review scope or evidence must stay visible.
|
|
35
|
+
6. Brownfield owners and existing conventions are binding unless deviation is justified.
|
|
36
|
+
7. Distinguish blocking defects from fixable review findings.
|
|
37
|
+
8. CR does not grant QA pass.
|
|
38
|
+
|
|
39
|
+
## When To Use
|
|
40
|
+
- after `CD+Tests`
|
|
41
|
+
- before `QA`
|
|
42
|
+
- when code changes exist
|
|
43
|
+
- when multiple agents or contributors touched the same area
|
|
44
|
+
- when a commit-near change needs explicit review evidence
|
|
45
|
+
|
|
46
|
+
## Inputs
|
|
47
|
+
Use what is available:
|
|
48
|
+
|
|
49
|
+
- changed files and diff
|
|
50
|
+
- affected neighbouring code
|
|
51
|
+
- test/build results
|
|
52
|
+
- Brownfield Analysis
|
|
53
|
+
- Task Plan and TP Review
|
|
54
|
+
- Clean Implementation Review
|
|
55
|
+
- runtime or UI evidence
|
|
56
|
+
- known risks or follow-up decisions
|
|
57
|
+
|
|
58
|
+
If no code changes are present, do not invent a CR result.
|
|
59
|
+
|
|
60
|
+
## Workflow
|
|
61
|
+
1. Inspect the changed files and directly impacted neighbours.
|
|
62
|
+
2. Check correctness, edge cases, and error paths.
|
|
63
|
+
3. Check security, data handling, and state ownership risks.
|
|
64
|
+
4. Check regression and compatibility risks.
|
|
65
|
+
5. Check maintainability only where it creates real defect or change-risk exposure.
|
|
66
|
+
6. Classify the overall result:
|
|
67
|
+
- `pass`
|
|
68
|
+
- `revise`
|
|
69
|
+
- `block`
|
|
70
|
+
- `not_applicable`
|
|
71
|
+
7. Set exactly one required next step.
|
|
72
|
+
|
|
73
|
+
## Output
|
|
74
|
+
Use this compact structure:
|
|
75
|
+
|
|
76
|
+
```text
|
|
77
|
+
## Code Review
|
|
78
|
+
- decision: pass | revise | block | not_applicable
|
|
79
|
+
- findings:
|
|
80
|
+
- [severity] file/path - issue - evidence
|
|
81
|
+
- missing_evidence:
|
|
82
|
+
- risks:
|
|
83
|
+
- required_next_step:
|
|
84
|
+
```
|
|
85
|
+
|
|
86
|
+
## Decision Guidance
|
|
87
|
+
- `pass`: no meaningful review finding remains evident in reviewed scope.
|
|
88
|
+
- `revise`: fixable correctness, regression, or maintainability issues remain.
|
|
89
|
+
- `block`: a hard defect, security risk, data-integrity risk, or unresolved critical uncertainty remains.
|
|
90
|
+
- `not_applicable`: no code changes were present for review.
|
|
91
|
+
|
|
92
|
+
## Forbidden
|
|
93
|
+
This skill must not:
|
|
94
|
+
|
|
95
|
+
- decide final QA
|
|
96
|
+
- accept a green build as sufficient review evidence
|
|
97
|
+
- hide missing review scope
|
|
98
|
+
- generate style-only findings
|
|
99
|
+
- duplicate TP Review or QA as if they were the same step
|
|
100
|
+
|
|
@@ -0,0 +1,115 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: commit/PR-near handoff after QA/OR/UAT
|
|
3
|
+
mode: subagent
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
---
|
|
7
|
+
name: agdf-delivery-closeout
|
|
8
|
+
description: Use this skill for the final delivery handoff after QA, OR, or UAT when a run should move toward commit, push, or PR. It standardizes commit-ready summaries, UAT-gated commit offers, and the next delivery step without replacing QA or OR decisions.
|
|
9
|
+
---
|
|
10
|
+
|
|
11
|
+
# delivery-closeout
|
|
12
|
+
|
|
13
|
+
## Purpose
|
|
14
|
+
Standardize the operational delivery closeout.
|
|
15
|
+
|
|
16
|
+
It answers:
|
|
17
|
+
|
|
18
|
+
- what operational delivery status applies after QA, OR, or UAT
|
|
19
|
+
- whether a commit-ready Git summary should be provided
|
|
20
|
+
- whether a commit should be actively offered after `Approval: UAT`
|
|
21
|
+
- which next delivery step makes sense: commit, push, PR, rollout, or none
|
|
22
|
+
- whether evidence-based technical follow-up remains useful
|
|
23
|
+
- whether Context Graph follow-up from OR/QA remains open
|
|
24
|
+
|
|
25
|
+
This skill does not replace `agdf-qa-gate`, `agdf-release-or`, user approvals, or gate decisions.
|
|
26
|
+
|
|
27
|
+
## Runtime Contract
|
|
28
|
+
Use `../agdf-runtime-contract.md` for closeout discipline, gate terms, Context Graph fields, and non-duplication rules.
|
|
29
|
+
|
|
30
|
+
## Rules
|
|
31
|
+
1. Delivery follows gate clarity.
|
|
32
|
+
2. Never perform commit, push, or PR automatically.
|
|
33
|
+
3. Use one handoff format:
|
|
34
|
+
- `Commit title`
|
|
35
|
+
- `Commit body`
|
|
36
|
+
- optional `Migration/Rollout note`
|
|
37
|
+
4. `Approval: UAT` unlocks an active commit offer when code changes exist.
|
|
38
|
+
5. No commit handoff for runs without code changes.
|
|
39
|
+
6. Always include exactly one next step and one quality outlook.
|
|
40
|
+
|
|
41
|
+
## When To Use
|
|
42
|
+
- after `QA pass` with code changes
|
|
43
|
+
- after OR when delivery handoff should be clean
|
|
44
|
+
- after `Approval: UAT`
|
|
45
|
+
- when the user asks for commit, push, or PR handoff
|
|
46
|
+
|
|
47
|
+
Do not use as a substitute for `agdf-qa-gate`, `agdf-release-or`, or `agdf-gate-check`.
|
|
48
|
+
|
|
49
|
+
## Inputs
|
|
50
|
+
Use what is available:
|
|
51
|
+
|
|
52
|
+
- QA Report
|
|
53
|
+
- OR
|
|
54
|
+
- current gate status
|
|
55
|
+
- whether code changes exist
|
|
56
|
+
- existing commit-ready summary
|
|
57
|
+
- rollout or migration notes
|
|
58
|
+
- open technical risks or documentation gaps
|
|
59
|
+
- Context Graph impact from OR/QA/review
|
|
60
|
+
|
|
61
|
+
If code-change status is unclear, do not claim a commit handoff.
|
|
62
|
+
|
|
63
|
+
## Workflow
|
|
64
|
+
1. Classify the closeout:
|
|
65
|
+
- `qa_pass_with_code`
|
|
66
|
+
- `qa_pass_without_code`
|
|
67
|
+
- `uat_approved_with_code`
|
|
68
|
+
- `uat_approved_without_code`
|
|
69
|
+
- `non_delivery_closeout`
|
|
70
|
+
2. Derive Git handoff only when code changes exist.
|
|
71
|
+
3. Set exactly one next delivery step:
|
|
72
|
+
- offer commit
|
|
73
|
+
- offer push
|
|
74
|
+
- offer PR
|
|
75
|
+
- no further delivery step
|
|
76
|
+
- run QA/review
|
|
77
|
+
- obtain UAT/approval
|
|
78
|
+
- perform documentation follow-up
|
|
79
|
+
4. Set quality outlook:
|
|
80
|
+
- no further technical follow-up
|
|
81
|
+
- more tests
|
|
82
|
+
- refactoring
|
|
83
|
+
- documentation sharpening
|
|
84
|
+
- monitoring/runtime verification
|
|
85
|
+
5. If `Approval: UAT` exists and code changes were delivered, actively offer the commit but do not run it.
|
|
86
|
+
|
|
87
|
+
## Output
|
|
88
|
+
For code-changing runs:
|
|
89
|
+
|
|
90
|
+
```text
|
|
91
|
+
Delivery status:
|
|
92
|
+
Next step:
|
|
93
|
+
Quality outlook:
|
|
94
|
+
Commit title:
|
|
95
|
+
Commit body:
|
|
96
|
+
Migration/Rollout note:
|
|
97
|
+
```
|
|
98
|
+
|
|
99
|
+
For non-code runs:
|
|
100
|
+
|
|
101
|
+
```text
|
|
102
|
+
Delivery status:
|
|
103
|
+
Next step:
|
|
104
|
+
Quality outlook:
|
|
105
|
+
```
|
|
106
|
+
|
|
107
|
+
## Forbidden
|
|
108
|
+
This skill must not:
|
|
109
|
+
|
|
110
|
+
- decide QA pass
|
|
111
|
+
- replace OR
|
|
112
|
+
- execute commit, push, or PR
|
|
113
|
+
- invent a commit handoff for non-code runs
|
|
114
|
+
- imply new gate approvals
|
|
115
|
+
|
|
@@ -0,0 +1,155 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: new build/change intent, unclear approval, Structured Delivery, later-gate artefact requested
|
|
3
|
+
mode: subagent
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
---
|
|
7
|
+
name: agdf-gate-check
|
|
8
|
+
description: Use this skill as the default first AGDF skill for any new build, change, extension, refactor, feature, CLI, app, fix with product semantics, Structured Delivery request, unclear approval, later-gate artefact request, or unclear next permissible step. Use it before brownfield-analysis, implementation, formal artefacts, QA, or release when approval/evidence/next action is not already explicit.
|
|
9
|
+
---
|
|
10
|
+
|
|
11
|
+
# gate-check
|
|
12
|
+
|
|
13
|
+
## Purpose
|
|
14
|
+
Determine the earliest blocking user approval gate and derive:
|
|
15
|
+
|
|
16
|
+
- the active or blocking gate
|
|
17
|
+
- whether the process is open or blocked
|
|
18
|
+
- currently allowed outputs
|
|
19
|
+
- currently forbidden outputs
|
|
20
|
+
- the exact missing approval
|
|
21
|
+
- the next permissible step
|
|
22
|
+
|
|
23
|
+
This skill must not create later artefacts such as PRD, SD, TP, CD, CR, QA, or UAT when the gate does not allow them.
|
|
24
|
+
|
|
25
|
+
## Runtime Contract
|
|
26
|
+
Use `../agdf-runtime-contract.md` for gate terms, closeout discipline, and non-duplication rules.
|
|
27
|
+
The canonical gate order and transition model live only in the Runtime Contract. This skill evaluates the current state against that model; it must not maintain a second complete gate table.
|
|
28
|
+
|
|
29
|
+
## Agent-Native Control Path
|
|
30
|
+
|
|
31
|
+
AGDF is agent-native first and CLI-verifiable by design.
|
|
32
|
+
This skill is the primary operating path for gate judgement.
|
|
33
|
+
|
|
34
|
+
First inspect the durable control state directly:
|
|
35
|
+
|
|
36
|
+
- `.agdf/control/AGDF_RUN.md`
|
|
37
|
+
- `.agdf/control/MASTER_BACKLOG.md`
|
|
38
|
+
- `.agdf/control/SOT_REGISTRY.md`
|
|
39
|
+
- `.agdf/control/CONTEXT_GRAPH.md`
|
|
40
|
+
- `.agdf/control/AGENT_QUALITY_CONTRACTS.json`
|
|
41
|
+
|
|
42
|
+
If required control files are missing, do not automatically create a full `.agdf/control/` scaffold for a fresh small request.
|
|
43
|
+
The default first action is to draft the minimal UR in the response and request `Approval: UR`.
|
|
44
|
+
Use `init` only when the user explicitly asks for durable AGDF control state, the repository already uses `.agdf/control/` as its live working state, or a deterministic CLI/CI setup path is being executed.
|
|
45
|
+
|
|
46
|
+
Use the executable control path when a machine-readable check is requested, when the gate state is ambiguous, when CI or PR evidence is needed, or when a repository-local automation needs JSON output:
|
|
47
|
+
|
|
48
|
+
```bash
|
|
49
|
+
npx --yes agdf@latest init
|
|
50
|
+
npx --yes agdf@latest doctor --json
|
|
51
|
+
npx --yes agdf@latest gate-check --json
|
|
52
|
+
npx --yes agdf@latest delivery-map --json
|
|
53
|
+
```
|
|
54
|
+
|
|
55
|
+
`init` creates the control scaffold. `doctor` checks whether `.agdf/control/` is actionable. `agdf-gate-check` consumes that result and `AGDF_RUN.md` to report the operative process decision: `open | blocked`, current gate, blocking reason, missing approval, allowed outputs, forbidden outputs, next allowed action and evidence references.
|
|
56
|
+
`delivery-map` reports the durable delivery picture: active artefacts, approvals, Artefact Chain relationships, evidence refs, missing evidence, risks, Context Graph gate effect and machine-readable findings.
|
|
57
|
+
|
|
58
|
+
The CLI reports are validators and JSON evidence, not the primary user experience, not a required ritual for normal work and not a second rule system. If a report says `blocked`, do not continue with later-gate artefacts until the reported blocker is resolved.
|
|
59
|
+
|
|
60
|
+
## Rules
|
|
61
|
+
1. Fail closed when a required approval or artefact status is missing.
|
|
62
|
+
2. The earliest blocking gate wins.
|
|
63
|
+
3. A user approval is valid only as `Approval: <GateName>`.
|
|
64
|
+
4. Treat `Freigabe: <GateName>` as a legacy alias only when reviewing older German runs.
|
|
65
|
+
5. Implicit consent is not approval. "ok", "go ahead", "do it", "approved", "continue", "leg los" and similar phrases do not unlock gates.
|
|
66
|
+
6. Do not preview later artefacts while a gate blocks.
|
|
67
|
+
7. Internal process steps are not user approval gates.
|
|
68
|
+
8. OR-lite is allowed if it does not leak blocked content.
|
|
69
|
+
9. SoT/runtime/product-semantics drift can trigger an early product gate, usually `UR`.
|
|
70
|
+
10. Approval of one user gate permits work on the next allowed gate artefact or required internal step only; it never skips directly to implementation.
|
|
71
|
+
11. New product semantics, functional change or user-visible behaviour change requires a durable UR in `.agdf/control/` or a linked authoritative repository SoT before Brownfield Review, PRD, SD, TP, Brownfield Analysis or implementation.
|
|
72
|
+
12. Approval text and durable artefact presence are separate requirements for UR, PRD, SD, TP and QA report decisions. Approval text without the corresponding persisted or linked artefact keeps the current gate at that gate.
|
|
73
|
+
13. After Brownfield Review, decide the process size before drafting PRD or implementing: `quick_task | structured_slice | structured_delivery | block`.
|
|
74
|
+
14. The Mode/Slice Decision must be visible and evidenced before any Quick Task execution, PRD shortcut or implementation. A decision value without scope reason and evidence is still missing.
|
|
75
|
+
15. Missing or incomplete control state must not push setup work back to the user. For a fresh request, draft the current minimal artefact in the response and request the exact approval. Initialize or write `.agdf/control/` only when durable control state is explicitly requested, already live for the repository, or required for a deterministic CLI/CI setup path. Implementation remains forbidden.
|
|
76
|
+
|
|
77
|
+
## Gate Evaluation
|
|
78
|
+
|
|
79
|
+
If `Approval: UR` is present, do not say implementation is the next step.
|
|
80
|
+
The next step is Brownfield Review when Brownfield, ownership, runtime, policy, persistence, architecture, UI or UX impact is possible; after that, the next step is a Mode/Slice Decision. Do not assume the full PRD/SD/TP chain before the existing-system impact is understood.
|
|
81
|
+
Use the Runtime Contract's Gate Transition Model to derive the current gate, allowed outputs, forbidden outputs and missing approval.
|
|
82
|
+
|
|
83
|
+
## When To Use
|
|
84
|
+
- new user intent to build, add, change, extend, refactor or deliver something
|
|
85
|
+
- Structured Delivery before first artefact creation
|
|
86
|
+
- unclear current gate
|
|
87
|
+
- user says "continue" or similar without exact approval
|
|
88
|
+
- code or a later-gate artefact is requested
|
|
89
|
+
- implementation permission is unclear
|
|
90
|
+
- QA, UAT, or release permission is unclear
|
|
91
|
+
|
|
92
|
+
Not required for simple Quick Tasks without new product scope and without formal artefacts.
|
|
93
|
+
|
|
94
|
+
For a fresh prompt such as "I want to build a small CLI", use this skill first.
|
|
95
|
+
Do not route directly to `agdf-brownfield-analysis` or implementation until this skill or live AGDF control state says that implementation preparation is the next allowed action.
|
|
96
|
+
|
|
97
|
+
## Inputs
|
|
98
|
+
Use what is available:
|
|
99
|
+
|
|
100
|
+
- current user request
|
|
101
|
+
- existing exact approvals
|
|
102
|
+
- status of UR, PRD, SD, TP
|
|
103
|
+
- Brownfield Review status
|
|
104
|
+
- Mode/Slice Decision
|
|
105
|
+
- Brownfield Analysis status
|
|
106
|
+
- CD+Tests, CR, QA, UAT status
|
|
107
|
+
- signs of documentation/runtime/product-semantics drift
|
|
108
|
+
|
|
109
|
+
If a status is not explicit, do not assume it is satisfied.
|
|
110
|
+
|
|
111
|
+
## Workflow
|
|
112
|
+
1. Check exact approvals.
|
|
113
|
+
2. Check artefact status.
|
|
114
|
+
3. Determine the earliest blocking user gate or internal mandatory step.
|
|
115
|
+
4. Derive allowed and forbidden outputs.
|
|
116
|
+
5. Name the exact missing approval, if any.
|
|
117
|
+
6. If consent was only implicit, say it is not yet approval and provide the exact formula.
|
|
118
|
+
7. Ensure the next step follows the gate transition table. In particular, never jump from `Approval: UR` to implementation.
|
|
119
|
+
8. Treat a generic "start", "continue" or "leg los" request as a request to perform only the current next allowed action.
|
|
120
|
+
9. After Brownfield Review, choose the smallest safe process path before creating later artefacts.
|
|
121
|
+
10. If the selected path is not visibly recorded with scope reason and evidence, keep the run at `Mode/Slice Decision`.
|
|
122
|
+
11. When `.agdf/control/` is missing or incomplete, make the next allowed artefact action explicit. For a fresh request, that means draft the minimal UR in the response, then request `Approval: UR`. Do not write a full control scaffold unless durable control state was explicitly requested or is already the repository's live AGDF working state.
|
|
123
|
+
|
|
124
|
+
## Output
|
|
125
|
+
Keep the result short and operational:
|
|
126
|
+
|
|
127
|
+
- **Current gate:** `<GateName or internal step>`
|
|
128
|
+
- **Status:** `open | blocked`
|
|
129
|
+
- **Allowed:** `<allowed outputs>`
|
|
130
|
+
- **Forbidden:** `<forbidden outputs>`
|
|
131
|
+
- **Missing approval:** `Approval: <GateName> | none`
|
|
132
|
+
- **Next step:** `<single permissible next step>`
|
|
133
|
+
|
|
134
|
+
If this skill creates or updates control artefacts, do not paste full file bodies into the chat.
|
|
135
|
+
List paths, summarize the decision, name the blocker or approval needed, and keep the durable content in the files.
|
|
136
|
+
|
|
137
|
+
## Forbidden
|
|
138
|
+
This skill must not:
|
|
139
|
+
|
|
140
|
+
- create PRD before UR approval
|
|
141
|
+
- create PRD before required Brownfield Review is done or explicitly not_applicable
|
|
142
|
+
- create PRD before Mode/Slice Decision is recorded
|
|
143
|
+
- create SD before PRD approval
|
|
144
|
+
- create TP while earlier gates block
|
|
145
|
+
- suggest implementation immediately after `Approval: UR`
|
|
146
|
+
- suggest implementation before Brownfield Review has explicitly selected `quick_task`
|
|
147
|
+
- suggest implementation immediately after `Approval: PRD`
|
|
148
|
+
- suggest implementation immediately after `Approval: SD`
|
|
149
|
+
- treat "ok", "leg los", "go ahead", "approved" or similar wording as gate approval
|
|
150
|
+
- perform full Brownfield Analysis unless explicitly requested
|
|
151
|
+
- provide implementation snippets while implementation is gated
|
|
152
|
+
- paste full `.agdf/control/` files, templates or artefact bodies into the chat unless the user explicitly asks for the full content
|
|
153
|
+
- present QA or UAT as passed without evidence
|
|
154
|
+
- present release as allowed without QA pass and UAT approval where required
|
|
155
|
+
|