create-agdf 0.12.0 → 0.13.1
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/generated/plugins/agdf/.claude-plugin/plugin.json +2 -2
- package/generated/plugins/agdf/.codex-plugin/plugin.json +9 -6
- package/generated/plugins/agdf/meta/agdf-plugin.definition.json +41 -5
- package/generated/plugins/agdf/runtime/create-agdf/generated/plugins/agdf/meta/agdf-plugin.definition.json +41 -5
- package/generated/plugins/agdf/runtime/create-agdf/package.json +4 -68
- package/generated/plugins/agdf/runtime/runtime-manifest.json +2 -2
- package/generated/plugins/agdf/scripts/check-runtime-integrity.mjs +27 -37
- package/generated/plugins/agdf/submission/openai/availability.md +16 -0
- package/generated/plugins/agdf/submission/openai/capability-matrix.json +49 -0
- package/generated/plugins/agdf/submission/openai/release-notes.md +16 -0
- package/generated/plugins/agdf/submission/openai/reviewer-cases.json +116 -0
- package/generated/submissions/openai/agdf/.codex-plugin/plugin.json +43 -0
- package/generated/submissions/openai/agdf/assets/agdf-icon.svg +16 -0
- package/generated/submissions/openai/agdf/assets/agdf-logo.svg +16 -0
- package/generated/submissions/openai/agdf/meta/agdf-agent-router.md +129 -0
- package/generated/submissions/openai/agdf/meta/agdf-constitution.md +329 -0
- package/generated/submissions/openai/agdf/meta/agdf-interaction-locales.json +514 -0
- package/generated/submissions/openai/agdf/meta/agdf-plugin.definition.json +263 -0
- package/generated/submissions/openai/agdf/meta/agdf-runtime-contract.md +16 -0
- package/generated/submissions/openai/agdf/meta/agdf-tenets.md +201 -0
- package/generated/submissions/openai/agdf/meta/contracts/closeout.md +29 -0
- package/generated/submissions/openai/agdf/meta/contracts/context-graph.md +48 -0
- package/generated/submissions/openai/agdf/meta/contracts/control-scaffold.md +115 -0
- package/generated/submissions/openai/agdf/meta/contracts/gate-transition.md +165 -0
- package/generated/submissions/openai/agdf/meta/contracts/interaction.md +476 -0
- package/generated/submissions/openai/agdf/meta/contracts/modes.md +183 -0
- package/generated/submissions/openai/agdf/meta/contracts/quality.md +157 -0
- package/generated/submissions/openai/agdf/meta/contracts/task-target-resolution.md +102 -0
- package/generated/submissions/openai/agdf/skills/brownfield-analysis/SKILL.md +175 -0
- package/generated/submissions/openai/agdf/skills/brownfield-analysis/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/clean-implementation-review/SKILL.md +126 -0
- package/generated/submissions/openai/agdf/skills/clean-implementation-review/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/code-review/SKILL.md +119 -0
- package/generated/submissions/openai/agdf/skills/code-review/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/delivery-closeout/SKILL.md +118 -0
- package/generated/submissions/openai/agdf/skills/delivery-closeout/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/delivery-path-search/SKILL.md +94 -0
- package/generated/submissions/openai/agdf/skills/delivery-path-search/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/gate-check/SKILL.md +238 -0
- package/generated/submissions/openai/agdf/skills/gate-check/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/qa-gate/SKILL.md +137 -0
- package/generated/submissions/openai/agdf/skills/qa-gate/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/release-or/SKILL.md +137 -0
- package/generated/submissions/openai/agdf/skills/release-or/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/task-plan-review/SKILL.md +157 -0
- package/generated/submissions/openai/agdf/skills/task-plan-review/help.md +23 -0
- package/generated/submissions/openai/agdf/skills/ux-intent-definition/SKILL.md +94 -0
- package/generated/submissions/openai/agdf/skills/ux-intent-definition/help.md +17 -0
- package/generated/submissions/openai/agdf/submission/openai/availability.md +16 -0
- package/generated/submissions/openai/agdf/submission/openai/capability-matrix.json +49 -0
- package/generated/submissions/openai/agdf/submission/openai/inventory.json +175 -0
- package/generated/submissions/openai/agdf/submission/openai/listing.json +38 -0
- package/generated/submissions/openai/agdf/submission/openai/readiness.json +40 -0
- package/generated/submissions/openai/agdf/submission/openai/readiness.md +24 -0
- package/generated/submissions/openai/agdf/submission/openai/release-notes.md +16 -0
- package/generated/submissions/openai/agdf/submission/openai/reviewer-cases.json +116 -0
- package/lib/installers/local-marketplace.js +20 -19
- package/lib/public-plugin/builder.js +82 -0
- package/lib/public-plugin/contract.js +102 -0
- package/lib/public-plugin/manifest.js +72 -0
- package/lib/public-plugin/report.js +33 -0
- package/lib/public-plugin/validator.js +101 -0
- package/package.json +4 -2
|
@@ -0,0 +1,137 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: qa-gate
|
|
3
|
+
description: Use this skill before any QA decision to determine whether implementation can be classified as pass, revise, or block based on TP coverage, Brownfield fit, solution integrity, evidence, and open blockers. Use it after CD+Tests, after reviews, and before UAT or release decisions.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# qa-gate
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
Make the formal QA gate decision.
|
|
10
|
+
|
|
11
|
+
This skill is the only final QA decision point for `pass | revise | block`.
|
|
12
|
+
|
|
13
|
+
It answers:
|
|
14
|
+
|
|
15
|
+
- whether `QA pass` is allowed
|
|
16
|
+
- whether the implementation must be revised
|
|
17
|
+
- whether a hard blocker remains
|
|
18
|
+
- whether relevant TP tasks are sufficiently verified
|
|
19
|
+
- whether P0/P1 tasks are complete
|
|
20
|
+
- whether Brownfield fit is sufficient
|
|
21
|
+
- whether solution integrity is sufficient
|
|
22
|
+
- whether blockers, critical risks, or missing evidence remain
|
|
23
|
+
|
|
24
|
+
## Runtime Contract
|
|
25
|
+
Use these focused runtime-contract modules:
|
|
26
|
+
|
|
27
|
+
- `../../meta/contracts/quality.md`
|
|
28
|
+
- `../../meta/contracts/context-graph.md`
|
|
29
|
+
- `../../meta/contracts/gate-transition.md`
|
|
30
|
+
|
|
31
|
+
QA-specific `decision` is exactly `pass | revise | block`.
|
|
32
|
+
`pass` is allowed only when TP coverage, Brownfield fit, solution integrity, and relevant documentation/Context Graph impact are sufficiently evidenced.
|
|
33
|
+
`sot_drift` must not pass silently as a warning.
|
|
34
|
+
Consume applicable findings through `../../meta/contracts/quality.md` §Normalized Review Gaps.
|
|
35
|
+
QA must not maintain a second mapping or reclassify review findings.
|
|
36
|
+
|
|
37
|
+
When post-CD+Tests review evidence exists, present the Runtime Contract's derived Quality
|
|
38
|
+
Readiness projection before the detailed QA report: Plan coverage, Solution integrity, Code
|
|
39
|
+
quality and QA decision in that order. Name `qa-gate` as the sole decision owner, show one
|
|
40
|
+
decisive reason and one permissible next action, and keep detailed reports as evidence links or
|
|
41
|
+
on-demand detail. The projection is non-authorizing and must not replace the Run Status Card,
|
|
42
|
+
Gate Transition Card or the durable QA report.
|
|
43
|
+
|
|
44
|
+
## Rules
|
|
45
|
+
1. No QA pass without strong evidence.
|
|
46
|
+
2. TP is the reference, not merely working code.
|
|
47
|
+
3. P0/P1 gaps block `pass`.
|
|
48
|
+
4. Brownfield fit is mandatory.
|
|
49
|
+
5. UI surface integrity is mandatory for UI-impacting changes.
|
|
50
|
+
6. Solution integrity is mandatory.
|
|
51
|
+
7. Always output exactly one decision: `pass`, `revise`, or `block`.
|
|
52
|
+
8. Open risks, missing tests, partial TP coverage, or side effects must be visible.
|
|
53
|
+
9. Applicable UX Intent Fidelity rows must all be `fulfilled` with suitable visible evidence before `pass`.
|
|
54
|
+
10. A `requirements_gap` routes to PRD revision; QA must not invent or accept the missing criterion.
|
|
55
|
+
11. Any applicable normalized finding that is `open`, missing, unknown, contradictory or supported by
|
|
56
|
+
insufficient evidence prevents `pass`.
|
|
57
|
+
12. A `resolved` finding counts only when its durable review evidence proves the routed correction or decision.
|
|
58
|
+
|
|
59
|
+
## When To Use
|
|
60
|
+
- after `CD+Tests`
|
|
61
|
+
- after `code-review`
|
|
62
|
+
- after `task-plan-review`
|
|
63
|
+
- after `clean-implementation-review`
|
|
64
|
+
- before UAT
|
|
65
|
+
- before any claim that implementation is done, releasable, or QA-ready
|
|
66
|
+
- when evidence from Brownfield, TP, or clean-review skills exists
|
|
67
|
+
|
|
68
|
+
Important:
|
|
69
|
+
|
|
70
|
+
- `CD+Tests` is implementation and test status only.
|
|
71
|
+
- Without review evidence and TP coverage, `QA pass` is not allowed.
|
|
72
|
+
|
|
73
|
+
## Inputs
|
|
74
|
+
Use what is available:
|
|
75
|
+
|
|
76
|
+
- approved TP and TP Review
|
|
77
|
+
- Brownfield Analysis
|
|
78
|
+
- Clean Implementation Review
|
|
79
|
+
- `code-review` or Code Review Report
|
|
80
|
+
- test/build results
|
|
81
|
+
- documentation impact review
|
|
82
|
+
- runtime or UI evidence
|
|
83
|
+
- known blockers and risks
|
|
84
|
+
|
|
85
|
+
If evidence is missing, lower the decision accordingly.
|
|
86
|
+
|
|
87
|
+
## Workflow
|
|
88
|
+
1. Confirm the relevant gate context.
|
|
89
|
+
2. Verify TP coverage.
|
|
90
|
+
3. Verify P0/P1 completion.
|
|
91
|
+
4. Verify Brownfield fit.
|
|
92
|
+
5. Verify UX Intent Fidelity when applicable, including visible evidence for visible-behavior claims.
|
|
93
|
+
6. Consume normalized findings without reclassification; fail closed on every open or invalid row.
|
|
94
|
+
7. Verify solution integrity.
|
|
95
|
+
8. Verify tests and evidence strength.
|
|
96
|
+
9. Verify documentation and Context Graph impact if relevant.
|
|
97
|
+
10. Decide:
|
|
98
|
+
- `pass`
|
|
99
|
+
- `revise`
|
|
100
|
+
- `block`
|
|
101
|
+
11. State the single required next step.
|
|
102
|
+
|
|
103
|
+
## Output
|
|
104
|
+
Use this shape:
|
|
105
|
+
|
|
106
|
+
```text
|
|
107
|
+
## QA Gate
|
|
108
|
+
- decision: pass | revise | block
|
|
109
|
+
- evidence:
|
|
110
|
+
- missing_evidence:
|
|
111
|
+
- risks:
|
|
112
|
+
- required_next_step:
|
|
113
|
+
- impact_codes:
|
|
114
|
+
```
|
|
115
|
+
|
|
116
|
+
If Context Graph impact is relevant, include the fields from `../../meta/contracts/context-graph.md`.
|
|
117
|
+
|
|
118
|
+
### Compact Chat Output
|
|
119
|
+
|
|
120
|
+
At `pass`: one line — `QA: pass — <one-line summary>`. Reference the durable QA report
|
|
121
|
+
path. Do not show the Quality Readiness projection or evidence inventory in chat.
|
|
122
|
+
At `revise`/`block`: show the decisive dimension, reason and next action.
|
|
123
|
+
|
|
124
|
+
## Decision Guidance
|
|
125
|
+
- `pass`: required TP tasks are done, evidence is strong, no blocking Brownfield or solution-integrity risk remains.
|
|
126
|
+
- `revise`: implementation is plausible but missing evidence, partial tasks, UX gaps, or fixable integrity issues remain.
|
|
127
|
+
- `block`: hard prerequisite, approval, product semantics, security/compliance, SoT drift, or critical behaviour is unresolved.
|
|
128
|
+
|
|
129
|
+
## Forbidden
|
|
130
|
+
This skill must not:
|
|
131
|
+
|
|
132
|
+
- grant QA pass from a green build alone
|
|
133
|
+
- hide partial task completion
|
|
134
|
+
- replace UAT
|
|
135
|
+
- treat missing reviews as completed
|
|
136
|
+
- downgrade SoT drift to harmless warning
|
|
137
|
+
- silently repair, normalize or reroute a review finding
|
|
@@ -0,0 +1,23 @@
|
|
|
1
|
+
## NAME
|
|
2
|
+
|
|
3
|
+
`qa-gate` - QA gate
|
|
4
|
+
|
|
5
|
+
## SYNOPSIS
|
|
6
|
+
|
|
7
|
+
`/qa-gate` [`--help`|`-h`]
|
|
8
|
+
|
|
9
|
+
## DESCRIPTION
|
|
10
|
+
|
|
11
|
+
Makes the formal QA gate decision (`pass | revise | block`) based on TP coverage, Brownfield fit, solution integrity, evidence, and open blockers.
|
|
12
|
+
|
|
13
|
+
## EXAMPLES
|
|
14
|
+
|
|
15
|
+
Trigger the skill:
|
|
16
|
+
|
|
17
|
+
```text
|
|
18
|
+
> /qa-gate
|
|
19
|
+
```
|
|
20
|
+
|
|
21
|
+
## SEE ALSO
|
|
22
|
+
|
|
23
|
+
See the full SKILL.md for detailed workflow, output format, and decision logic.
|
|
@@ -0,0 +1,137 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: release-or
|
|
3
|
+
description: Use this skill at the end of every relevant run to produce a compact, auditable Orchestration Report. It summarizes gate status, delivered and intentionally not delivered work, TP coverage, Brownfield fit, solution integrity, open risks, and the next permissible step.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# release-or
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
Produce the Orchestration Report (OR) as the mandatory closeout for a relevant run.
|
|
10
|
+
|
|
11
|
+
It reports:
|
|
12
|
+
|
|
13
|
+
- current user gate
|
|
14
|
+
- whether the run is `pass`, `revise`, `block`, or not yet QA-decided
|
|
15
|
+
- what was delivered
|
|
16
|
+
- what was intentionally not delivered
|
|
17
|
+
- missing approvals
|
|
18
|
+
- TP coverage status
|
|
19
|
+
- Brownfield fit
|
|
20
|
+
- solution integrity
|
|
21
|
+
- documentation impact
|
|
22
|
+
- Context Graph impact
|
|
23
|
+
- open risks, blockers, retained fallbacks, and exit criteria
|
|
24
|
+
- next permissible step
|
|
25
|
+
- whether further quality follow-up or a separate delivery closeout is useful
|
|
26
|
+
|
|
27
|
+
## Runtime Contract
|
|
28
|
+
Use these focused runtime-contract modules:
|
|
29
|
+
|
|
30
|
+
- `../../meta/contracts/closeout.md`
|
|
31
|
+
- `../../meta/contracts/quality.md`
|
|
32
|
+
- `../../meta/contracts/context-graph.md`
|
|
33
|
+
- `../../meta/contracts/control-scaffold.md`
|
|
34
|
+
|
|
35
|
+
OR-specific output must make gate status, delivered and intentionally not delivered content, missing approvals, missing evidence, risks, retained fallbacks, and the next permissible step visible.
|
|
36
|
+
When `.agdf/control/` is present, persist or link the OR under `.agdf/control/artefacts/<key>/OR.md` and reference it from the selected canonical `RUN_STATE.md` or `MASTER_BACKLOG.md` when it is steering-relevant.
|
|
37
|
+
Use the Runtime Contract definition of `Relevant Run` to decide whether OR is mandatory or whether a Quick Task mini-closeout is enough.
|
|
38
|
+
|
|
39
|
+
## Rules
|
|
40
|
+
1. OR is always allowed and mandatory for relevant runs as defined in the Runtime Contract.
|
|
41
|
+
2. OR is an audit report, not a blocking gate.
|
|
42
|
+
3. Do not leak artefacts from blocked later gates.
|
|
43
|
+
4. Use OR-lite when early gates block.
|
|
44
|
+
5. Use OR-full only within the allowed scope.
|
|
45
|
+
6. Report status, not marketing language.
|
|
46
|
+
7. Name missing approvals, partial implementation, risks, fallbacks, Brownfield issues, and open defects.
|
|
47
|
+
8. Retained fallbacks require visible exit criteria and cleanup path.
|
|
48
|
+
9. End with the immediately permissible next step.
|
|
49
|
+
10. `CD+Tests` is not completion.
|
|
50
|
+
11. When moving a backlog item to Completed or Superseded, use the canonical human-readable table and link the final OR with a document-relative Markdown link.
|
|
51
|
+
12. For an executed `verified_change`, a lifecycle-consistent complete Mini-Closeout in the linked `VERIFIED_CHANGE.md` is the compact OR target; do not require a separate `OR.md` by ritual.
|
|
52
|
+
13. Keep machine status normalization in the CLI; do not write internal snake_case status values into the human-facing backlog.
|
|
53
|
+
14. Reconcile Context Graph impact before clean closeout: report `context_graph_reconciliation: resolved | not_applicable | open_gap`.
|
|
54
|
+
15. If Context Graph work remains unresolved, report it as an explicit open gap and do not describe the run as cleanly handoff-ready.
|
|
55
|
+
16. After writing or updating `MASTER_BACKLOG.md`, run `doctor --json` (or the locally available equivalent) and resolve any `AGDF_BACKLOG_STATUS_UNKNOWN` or `AGDF_BACKLOG_ARTEFACT_LABEL_UNKNOWN` finding before closeout. See the AGDF control scaffold's `MASTER_BACKLOG.md` template Rules section for the canonical status/artefact label vocabulary.
|
|
56
|
+
|
|
57
|
+
## When To Use
|
|
58
|
+
- at the end of every relevant run
|
|
59
|
+
- after artefact creation
|
|
60
|
+
- when a gate blocks
|
|
61
|
+
- after `CD+Tests`
|
|
62
|
+
- after `code-review`
|
|
63
|
+
- after QA
|
|
64
|
+
- after UAT
|
|
65
|
+
- whenever a compact audit closeout is needed
|
|
66
|
+
|
|
67
|
+
Do not force OR onto a pure explanation, read-only inspection, small review, or local debugging step that produces no durable state change and no gate consequence.
|
|
68
|
+
|
|
69
|
+
## Inputs
|
|
70
|
+
Use what is available:
|
|
71
|
+
|
|
72
|
+
- current gate
|
|
73
|
+
- existing approvals
|
|
74
|
+
- artefacts created in this run
|
|
75
|
+
- intentionally not delivered content
|
|
76
|
+
- Task Plan
|
|
77
|
+
- `task-plan-review`
|
|
78
|
+
- `brownfield-analysis`
|
|
79
|
+
- `clean-implementation-review`
|
|
80
|
+
- `code-review`
|
|
81
|
+
- QA Report
|
|
82
|
+
- UAT Report
|
|
83
|
+
- documentation impact review
|
|
84
|
+
- Context Graph impact
|
|
85
|
+
- known risks, defects, fallbacks, and open questions
|
|
86
|
+
- test/build status and relevant metrics
|
|
87
|
+
|
|
88
|
+
If information is missing, state the gap instead of guessing.
|
|
89
|
+
|
|
90
|
+
## Workflow
|
|
91
|
+
1. Determine the current gate and earliest blocker.
|
|
92
|
+
2. Determine report depth:
|
|
93
|
+
- `OR-lite`: gate status, allowed/forbidden outputs, missing approval, next step
|
|
94
|
+
- `OR-full`: also delivered artefacts, TP coverage, Brownfield fit, solution integrity, risks, fallbacks, QA/UAT status
|
|
95
|
+
3. Record the selected `Report mode` and durable OR target or link when `.agdf/control/` exists.
|
|
96
|
+
4. Record delivered vs intentionally not delivered content.
|
|
97
|
+
5. Summarize TP coverage if a TP exists.
|
|
98
|
+
6. Summarize Brownfield fit and solution integrity if reviewed.
|
|
99
|
+
7. Summarize tests and verification.
|
|
100
|
+
8. Summarize documentation and Context Graph impact if relevant.
|
|
101
|
+
9. Record Context Graph reconciliation as `resolved`, `not_applicable`, or `open_gap`.
|
|
102
|
+
10. Name retained fallbacks and exit criteria.
|
|
103
|
+
11. Set exactly one next permissible step.
|
|
104
|
+
12. Set exactly one quality outlook.
|
|
105
|
+
13. State whether `delivery-closeout` is the next operational handoff step when code changes exist and the delivery state allows it.
|
|
106
|
+
|
|
107
|
+
## Output
|
|
108
|
+
Use a compact structure:
|
|
109
|
+
|
|
110
|
+
```text
|
|
111
|
+
## OR
|
|
112
|
+
- gate:
|
|
113
|
+
- report_mode:
|
|
114
|
+
- artefact:
|
|
115
|
+
- status:
|
|
116
|
+
- delivered:
|
|
117
|
+
- intentionally_not_delivered:
|
|
118
|
+
- evidence:
|
|
119
|
+
- missing_evidence:
|
|
120
|
+
- risks:
|
|
121
|
+
- retained_fallbacks:
|
|
122
|
+
- required_next_step:
|
|
123
|
+
- quality_outlook:
|
|
124
|
+
```
|
|
125
|
+
|
|
126
|
+
When Context Graph impact is relevant, include the fields from `../../meta/contracts/context-graph.md`.
|
|
127
|
+
Include `context_graph_reconciliation` whenever Context Graph impact is present. If the value is `open_gap`, required next step must resolve or explicitly retain that gap before commit-ready or release-ready handoff.
|
|
128
|
+
|
|
129
|
+
## Forbidden
|
|
130
|
+
This skill must not:
|
|
131
|
+
|
|
132
|
+
- decide QA in place of `qa-gate`
|
|
133
|
+
- call `CD+Tests` complete, QA-ready, or release-ready
|
|
134
|
+
- hide missing approvals
|
|
135
|
+
- provide later-gate content while an earlier gate blocks
|
|
136
|
+
- produce the operative commit/PR handoff owned by `delivery-closeout`
|
|
137
|
+
- execute commit, push, or PR actions
|
|
@@ -0,0 +1,23 @@
|
|
|
1
|
+
## NAME
|
|
2
|
+
|
|
3
|
+
`release-or` - release OR
|
|
4
|
+
|
|
5
|
+
## SYNOPSIS
|
|
6
|
+
|
|
7
|
+
`/release-or` [`--help`|`-h`]
|
|
8
|
+
|
|
9
|
+
## DESCRIPTION
|
|
10
|
+
|
|
11
|
+
Creates the mandatory Orchestration Report (OR) for a run. Summarizes gate status, delivered artefacts, TP coverage, Brownfield fit, solution integrity, open points, and the next permissible step.
|
|
12
|
+
|
|
13
|
+
## EXAMPLES
|
|
14
|
+
|
|
15
|
+
Trigger the skill:
|
|
16
|
+
|
|
17
|
+
```text
|
|
18
|
+
> /release-or
|
|
19
|
+
```
|
|
20
|
+
|
|
21
|
+
## SEE ALSO
|
|
22
|
+
|
|
23
|
+
See the full SKILL.md for detailed workflow, output format, and decision logic.
|
|
@@ -0,0 +1,157 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: task-plan-review
|
|
3
|
+
description: Use this skill after code changes and before QA to verify whether the approved Task Plan was actually fulfilled. It checks each task_id against implementation, acceptance criteria, tests, visible evidence, and deviations, and provides TP coverage for QA.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# task-plan-review
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
Verify whether implementation actually satisfies the approved Task Plan.
|
|
10
|
+
|
|
11
|
+
For each relevant task, this skill determines:
|
|
12
|
+
|
|
13
|
+
- whether the task is fully, partially, or not completed
|
|
14
|
+
- which acceptance criteria are done, partial, missing, or not verifiable
|
|
15
|
+
- which files, tests, runtime evidence, or UI evidence support the finding
|
|
16
|
+
- evidence strength
|
|
17
|
+
- which gaps must be handed to `qa-gate`
|
|
18
|
+
|
|
19
|
+
This skill provides TP coverage. It is not the final QA decision.
|
|
20
|
+
|
|
21
|
+
## Runtime Contract
|
|
22
|
+
Use these focused runtime-contract modules:
|
|
23
|
+
|
|
24
|
+
- `../../meta/contracts/quality.md`
|
|
25
|
+
- `../../meta/contracts/context-graph.md`
|
|
26
|
+
- `../../meta/contracts/gate-transition.md`
|
|
27
|
+
|
|
28
|
+
TP-specific output must evaluate every relevant `task_id` with completion status, AC coverage, evidence, missing evidence, and QA-relevant gaps.
|
|
29
|
+
If `context_graph_required_action` is not `none`, the affected task is complete only if follow-up, evidence, or an intentionally open gap is visible.
|
|
30
|
+
Applicable review gaps must use `../../meta/contracts/quality.md` §Normalized Review Gaps. This skill
|
|
31
|
+
must not maintain its own type-to-route mapping.
|
|
32
|
+
|
|
33
|
+
## Rules
|
|
34
|
+
1. The Task Plan is the reference, not plausible code.
|
|
35
|
+
2. No assumptions without evidence.
|
|
36
|
+
3. Evaluate each relevant `task_id` individually.
|
|
37
|
+
4. `fully_done` requires strong evidence.
|
|
38
|
+
5. A green build alone does not prove TP completion.
|
|
39
|
+
6. UI/state/render/recovery tasks need visible evidence, not only code.
|
|
40
|
+
7. Partial implementation must be marked `partially_done`.
|
|
41
|
+
8. P0/P1 gaps must be handed to QA.
|
|
42
|
+
9. Out-of-scope changes must be reported.
|
|
43
|
+
10. When evidence is unclear, fail closed to at least `partially_done`.
|
|
44
|
+
11. When approved PRD UX criteria apply, verify both PRD-to-TP coverage and TP-to-code/surface fulfilment.
|
|
45
|
+
12. Never invent a missing UX requirement. Report `requirements_gap` and route it to PRD revision.
|
|
46
|
+
13. Visible behavior requires visible evidence; code evidence alone is `not_verifiable`.
|
|
47
|
+
14. A fulfilled UX Intent Fidelity row uses `none`; every non-fulfilled applicable row uses one
|
|
48
|
+
normalized gap type and a contract-valid route.
|
|
49
|
+
15. Missing, unknown or contradictory classifications fail closed and stay open.
|
|
50
|
+
|
|
51
|
+
## Clarifications
|
|
52
|
+
### TP Slice Before Target State
|
|
53
|
+
If the TP explicitly defines a preparatory, additive, or limited slice, evaluate that exact scope.
|
|
54
|
+
`fully_done` is allowed for a completed slice even if the larger target state remains open.
|
|
55
|
+
|
|
56
|
+
### Missing Priorities
|
|
57
|
+
If no priorities are defined, do not invent them. Mark the gap and treat QA relevance conservatively.
|
|
58
|
+
|
|
59
|
+
### Missing Acceptance Criteria
|
|
60
|
+
If ACs are unclear, derive only obvious expectations from title, goal, and scope. Mark AC basis as incomplete and lower evidence confidence as needed.
|
|
61
|
+
|
|
62
|
+
## When To Use
|
|
63
|
+
- after `CD+Tests`
|
|
64
|
+
- before `QA`
|
|
65
|
+
- when TP completion is unclear
|
|
66
|
+
- when the user asks for TP fulfilment
|
|
67
|
+
- when UI/state/runtime invariants are affected
|
|
68
|
+
- when several agents or work steps touched the same TP
|
|
69
|
+
|
|
70
|
+
## Inputs
|
|
71
|
+
Use what is available:
|
|
72
|
+
|
|
73
|
+
- Task Plan
|
|
74
|
+
- `task_id`, `story_id`, priorities
|
|
75
|
+
- acceptance criteria or success goals
|
|
76
|
+
- changed files
|
|
77
|
+
- tests and test results
|
|
78
|
+
- build results
|
|
79
|
+
- runtime/UI evidence
|
|
80
|
+
- known deviations and scope decisions
|
|
81
|
+
|
|
82
|
+
If TP or relevant tasks are missing, do not claim reliable TP coverage.
|
|
83
|
+
|
|
84
|
+
## Output
|
|
85
|
+
Use this compact structure:
|
|
86
|
+
|
|
87
|
+
```text
|
|
88
|
+
## TP Coverage
|
|
89
|
+
| task_id | status | evidence | missing_evidence | QA impact |
|
|
90
|
+
|---|---|---|---|---|
|
|
91
|
+
|
|
92
|
+
## Summary
|
|
93
|
+
- fully_done:
|
|
94
|
+
- partially_done:
|
|
95
|
+
- not_done:
|
|
96
|
+
- out_of_scope_changes:
|
|
97
|
+
- risks:
|
|
98
|
+
- required_next_step:
|
|
99
|
+
```
|
|
100
|
+
|
|
101
|
+
When applicable, append:
|
|
102
|
+
|
|
103
|
+
```text
|
|
104
|
+
## UX Intent Fidelity
|
|
105
|
+
| prd_criterion | working_mode_state | task_id | visible_evidence | fidelity_status | gap_type |
|
|
106
|
+
|---|---|---|---|---|---|
|
|
107
|
+
```
|
|
108
|
+
|
|
109
|
+
Use `fulfilled | partial | missing | not_verifiable` for `fidelity_status` and
|
|
110
|
+
the shared Quality Contract for `gap_type`. `none` is allowed only on a fulfilled row.
|
|
111
|
+
|
|
112
|
+
When an applicable gap needs routing detail, append:
|
|
113
|
+
|
|
114
|
+
```text
|
|
115
|
+
## Normalized Findings
|
|
116
|
+
| finding_id | gap_type | routing_target | gap_status | evidence | required_next_step |
|
|
117
|
+
|---|---|---|---|---|---|
|
|
118
|
+
```
|
|
119
|
+
|
|
120
|
+
Use only contract-defined values. Do not reconstruct the type-to-route mapping locally.
|
|
121
|
+
|
|
122
|
+
Completion status:
|
|
123
|
+
|
|
124
|
+
- `fully_done`
|
|
125
|
+
- `partially_done`
|
|
126
|
+
- `not_done`
|
|
127
|
+
|
|
128
|
+
AC status:
|
|
129
|
+
|
|
130
|
+
- `done`
|
|
131
|
+
- `partial`
|
|
132
|
+
- `missing`
|
|
133
|
+
- `not_verifiable`
|
|
134
|
+
|
|
135
|
+
Evidence confidence:
|
|
136
|
+
|
|
137
|
+
- `high`
|
|
138
|
+
- `medium`
|
|
139
|
+
- `low`
|
|
140
|
+
|
|
141
|
+
### Compact Chat Output
|
|
142
|
+
|
|
143
|
+
At `pass`: one line — `TP Review: pass — <X>/<total> tasks fully_done`.
|
|
144
|
+
At `revise`/`block`: show incomplete tasks with missing evidence.
|
|
145
|
+
|
|
146
|
+
## Forbidden
|
|
147
|
+
This skill must not:
|
|
148
|
+
|
|
149
|
+
- decide final QA
|
|
150
|
+
- mark tasks fully done from code existence alone
|
|
151
|
+
- hide partial work
|
|
152
|
+
- upgrade missing evidence into confidence
|
|
153
|
+
- reinterpret the TP scope
|
|
154
|
+
- create or silently repair product requirements
|
|
155
|
+
- treat code existence as visible UX evidence
|
|
156
|
+
- emit `none` as a normalized finding type
|
|
157
|
+
- silently repair or reclassify an invalid finding
|
|
@@ -0,0 +1,23 @@
|
|
|
1
|
+
## NAME
|
|
2
|
+
|
|
3
|
+
`task-plan-review` - task plan review
|
|
4
|
+
|
|
5
|
+
## SYNOPSIS
|
|
6
|
+
|
|
7
|
+
`/task-plan-review` [`--help`|`-h`]
|
|
8
|
+
|
|
9
|
+
## DESCRIPTION
|
|
10
|
+
|
|
11
|
+
Checks after code changes and before QA whether the approved Task Plan tasks were fully, partially, or not completed. Provides reliable TP coverage as input for the QA gate decision.
|
|
12
|
+
|
|
13
|
+
## EXAMPLES
|
|
14
|
+
|
|
15
|
+
Trigger the skill:
|
|
16
|
+
|
|
17
|
+
```text
|
|
18
|
+
> /task-plan-review
|
|
19
|
+
```
|
|
20
|
+
|
|
21
|
+
## SEE ALSO
|
|
22
|
+
|
|
23
|
+
See the full SKILL.md for detailed workflow, output format, and decision logic.
|
|
@@ -0,0 +1,94 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: ux-intent-definition
|
|
3
|
+
description: Use this skill after approved UR and post-UR routing when UI/UX impact is medium or high, or low with ambiguous product semantics, to define user intent, success, working modes, effective state, visible state ownership, activation, blockers, recovery and transitions as structured PRD input; fail closed when evidence or product decisions are unresolved. This internal analysis never grants gate permission or replaces the approved PRD.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# ux-intent-definition
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Create structured, testable UX intent input before PRD approval so implementation reviews verify an
|
|
11
|
+
approved product promise instead of inventing requirements after code exists.
|
|
12
|
+
|
|
13
|
+
## Runtime Contract
|
|
14
|
+
|
|
15
|
+
Use:
|
|
16
|
+
|
|
17
|
+
- `../../meta/contracts/gate-transition.md` for impact routing, authority and revision behavior;
|
|
18
|
+
- `../../meta/contracts/quality.md` for evidence and fail-closed output discipline.
|
|
19
|
+
|
|
20
|
+
## Authority
|
|
21
|
+
|
|
22
|
+
This is an internal analytical skill, not a user gate. It cannot approve artefacts, grant
|
|
23
|
+
implementation permission, change approved UR/PRD intent, prescribe technical design or become a
|
|
24
|
+
parallel product source of truth. Proposed criteria become authoritative only when incorporated into
|
|
25
|
+
and approved as PRD content. Technical storage, derivation and component ownership remain SD concerns.
|
|
26
|
+
|
|
27
|
+
## Inputs
|
|
28
|
+
|
|
29
|
+
- approved UR and its intent, success signal, primary action and persona/context;
|
|
30
|
+
- post-UR `delivery_context`, `ui_ux_impact`, reason and required flag;
|
|
31
|
+
- existing behavior/ownership evidence for Brownfield work;
|
|
32
|
+
- known working modes, state semantics, activation, blockers, recovery and transitions.
|
|
33
|
+
|
|
34
|
+
## Workflow
|
|
35
|
+
|
|
36
|
+
1. Confirm the approved UR and routing evidence. If required evidence is absent or conflicts, block.
|
|
37
|
+
2. Identify the primary user intent, observable success and primary decision/action.
|
|
38
|
+
3. For every relevant working mode, define effective state and visible state types.
|
|
39
|
+
4. Separate `effective_state_authority_by_mode` from
|
|
40
|
+
`primary_state_presentation_owner_by_mode`; do not turn either into technical ownership.
|
|
41
|
+
5. Define activation/deactivation, blockers with visible next actions, actionable recovery (including
|
|
42
|
+
visible retry for recoverable transient failure) and relevant state transitions.
|
|
43
|
+
6. Propose observable PRD acceptance criteria and identify open product questions.
|
|
44
|
+
7. Return exactly one decision: `ready | blocked | not_applicable`.
|
|
45
|
+
|
|
46
|
+
## Decision Rules
|
|
47
|
+
|
|
48
|
+
- `ready`: every required output is reliable enough for PRD drafting.
|
|
49
|
+
- `blocked`: user intent, modes, effective state, authority, activation, blockers, recovery, transitions
|
|
50
|
+
or Brownfield evidence is missing, contradictory or materially ambiguous.
|
|
51
|
+
- `not_applicable`: routing proves that no relevant user-facing intent, state or recovery behavior changes.
|
|
52
|
+
|
|
53
|
+
A conflict with approved UR routes to UR revision. A material product change discovered after PRD
|
|
54
|
+
approval routes to PRD revision, or UR revision when intent/scope changes. Never resolve the conflict
|
|
55
|
+
silently. A required blocked result prevents PRD readiness.
|
|
56
|
+
|
|
57
|
+
## Output
|
|
58
|
+
|
|
59
|
+
Populate each field or use an explicit, justified `not_applicable`:
|
|
60
|
+
|
|
61
|
+
```text
|
|
62
|
+
- decision: ready | blocked | not_applicable
|
|
63
|
+
- blocking_reason:
|
|
64
|
+
- primary_user_intent:
|
|
65
|
+
- success_signal:
|
|
66
|
+
- primary_decision_or_action:
|
|
67
|
+
- working_modes:
|
|
68
|
+
- effective_state_by_mode:
|
|
69
|
+
- visible_state_types:
|
|
70
|
+
- effective_state_authority_by_mode:
|
|
71
|
+
- primary_state_presentation_owner_by_mode:
|
|
72
|
+
- activation_paths:
|
|
73
|
+
- blockers:
|
|
74
|
+
- recovery_paths:
|
|
75
|
+
- relevant_state_transitions:
|
|
76
|
+
- proposed_prd_acceptance_criteria:
|
|
77
|
+
- open_product_questions:
|
|
78
|
+
- affected_outputs:
|
|
79
|
+
- evidence:
|
|
80
|
+
- missing_evidence:
|
|
81
|
+
- required_next_step:
|
|
82
|
+
```
|
|
83
|
+
|
|
84
|
+
For `blocked`, `blocking_reason`, `open_product_questions`, `affected_outputs`, `evidence`,
|
|
85
|
+
`missing_evidence` and `required_next_step` are mandatory. When durable control state exists, write
|
|
86
|
+
the result to `.agdf/control/artefacts/<key>/UX_INTENT_DEFINITION.md` without a Gate or approval field.
|
|
87
|
+
|
|
88
|
+
## Forbidden
|
|
89
|
+
|
|
90
|
+
- invent product intent or silently choose between conflicting modes/state authorities;
|
|
91
|
+
- bypass UR/PRD revision or claim PRD readiness from a blocked result;
|
|
92
|
+
- add a gate, approval value or implementation permission;
|
|
93
|
+
- prescribe components, persistence, endpoints, styles or other technical design;
|
|
94
|
+
- treat supporting analysis as authoritative beside the approved PRD.
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
## NAME
|
|
2
|
+
|
|
3
|
+
`ux-intent-definition` - define testable UX intent before PRD approval
|
|
4
|
+
|
|
5
|
+
## SYNOPSIS
|
|
6
|
+
|
|
7
|
+
`/ux-intent-definition` [`--help`|`-h`]
|
|
8
|
+
|
|
9
|
+
## DESCRIPTION
|
|
10
|
+
|
|
11
|
+
Produces structured `ready | blocked | not_applicable` PRD input for user intent, success, working
|
|
12
|
+
modes, effective state, visible ownership, activation, blockers, recovery and transitions. It is an
|
|
13
|
+
internal non-authorizing analysis and fails closed when product semantics or evidence are unresolved.
|
|
14
|
+
|
|
15
|
+
## SEE ALSO
|
|
16
|
+
|
|
17
|
+
See SKILL.md for triggers, authority, required fields and decision rules.
|
|
@@ -0,0 +1,16 @@
|
|
|
1
|
+
# Availability Decision Record
|
|
2
|
+
|
|
3
|
+
- release: `0.13.1`
|
|
4
|
+
- decision_state: `pending`
|
|
5
|
+
- decision_owner: verified publisher
|
|
6
|
+
- countries: not selected
|
|
7
|
+
- regions: not selected
|
|
8
|
+
- workspace_contexts: not selected
|
|
9
|
+
|
|
10
|
+
Availability is deliberately not assumed to be global. Before submission, the verified publisher
|
|
11
|
+
must select the supported countries, regions and workspace contexts in the OpenAI portal after
|
|
12
|
+
confirming public legal routes and best-effort support capacity. The saved portal state must then be
|
|
13
|
+
read back and reconciled with the canonical distribution record.
|
|
14
|
+
|
|
15
|
+
This pending record blocks submission readiness but does not invalidate the deterministic repository
|
|
16
|
+
candidate. Changing availability, submitting or publishing requires its own explicit authorization.
|
|
@@ -0,0 +1,49 @@
|
|
|
1
|
+
{
|
|
2
|
+
"schemaVersion": 1,
|
|
3
|
+
"releaseVersion": "0.13.1",
|
|
4
|
+
"evidenceStates": [
|
|
5
|
+
"common",
|
|
6
|
+
"codex-specific",
|
|
7
|
+
"chatgpt-specific",
|
|
8
|
+
"advisory",
|
|
9
|
+
"unavailable",
|
|
10
|
+
"unverified"
|
|
11
|
+
],
|
|
12
|
+
"capabilities": [
|
|
13
|
+
{
|
|
14
|
+
"id": "proportional-fit",
|
|
15
|
+
"state": "common",
|
|
16
|
+
"description": "Assess whether AGDF is proportionate and recommend the lightest safe path.",
|
|
17
|
+
"evidenceClass": "repository",
|
|
18
|
+
"evidenceRef": "skills/gate-check/SKILL.md"
|
|
19
|
+
},
|
|
20
|
+
{
|
|
21
|
+
"id": "governed-repository-delivery",
|
|
22
|
+
"state": "codex-specific",
|
|
23
|
+
"description": "Use repository files, durable control state and local validation where the Codex host grants the required access.",
|
|
24
|
+
"evidenceClass": "unverified",
|
|
25
|
+
"evidenceRef": "PPD-L02"
|
|
26
|
+
},
|
|
27
|
+
{
|
|
28
|
+
"id": "codex-hooks",
|
|
29
|
+
"state": "codex-specific",
|
|
30
|
+
"description": "Lifecycle hooks are a Codex-specific enhancement and remain host-trust-dependent.",
|
|
31
|
+
"evidenceClass": "unverified",
|
|
32
|
+
"evidenceRef": "PPD-L02"
|
|
33
|
+
},
|
|
34
|
+
{
|
|
35
|
+
"id": "chatgpt-portable-skills",
|
|
36
|
+
"state": "advisory",
|
|
37
|
+
"description": "Fit assessment, explanation, planning and review may use the portable skills core; repository enforcement requires direct host evidence.",
|
|
38
|
+
"evidenceClass": "unverified",
|
|
39
|
+
"evidenceRef": "PPD-L03"
|
|
40
|
+
},
|
|
41
|
+
{
|
|
42
|
+
"id": "agdf-mcp-service",
|
|
43
|
+
"state": "unavailable",
|
|
44
|
+
"description": "The initial release provides no AGDF-operated MCP server, hosted account or service authentication.",
|
|
45
|
+
"evidenceClass": "bundle",
|
|
46
|
+
"evidenceRef": "candidate inventory"
|
|
47
|
+
}
|
|
48
|
+
]
|
|
49
|
+
}
|