Skip to content

fix(strategies,skills): idempotency guards + SKILL.md cutover + version bump (#387, #389, #390)#393

Merged
MScottAdams merged 1 commit intophase2/vbrief-cutoverfrom
agent2/fix/387-389-390-strategy-skill-updates
Apr 15, 2026
Merged

fix(strategies,skills): idempotency guards + SKILL.md cutover + version bump (#387, #389, #390)#393
MScottAdams merged 1 commit intophase2/vbrief-cutoverfrom
agent2/fix/387-389-390-strategy-skill-updates

Conversation

@MScottAdams
Copy link
Copy Markdown
Collaborator

@MScottAdams MScottAdams commented Apr 15, 2026

Summary

Three strategy/skill fixes for the v0.20 vBRIEF cutover:

Task A — Artifact Idempotency Guards (#387)

  • Created strategies/artifact-guards.md with shared guard rules (DRY pattern)
    • Spec-Generating Guard (Full): status-aware check before writing to specification.vbrief.json or PROJECT-DEFINITION.vbrief.json — refuses overwrite of approved/completed, warns on draft/running, preserves existing narratives
    • Preparatory Guard (Light): checks for existing scope vBRIEFs with same feature/scope name before writing to �brief/proposed/
  • Added ! guard references to 7 strategies: speckit (Phase 1+2), enterprise (Stage 1+3), rapid (Step 3), bdd (Step 4), discuss (Output), research (Output), map (Mapping Artifacts)

Task B — SKILL.md Platform Entry Point Cutover (#389)

  • Updated root SKILL.md for v0.20 model:
    • deft-setup → deft-directive-setup (3 occurrences)
    • PROJECT.md → PROJECT-DEFINITION.vbrief.json (6 occurrences)
    • SPECIFICATION.md → scope vBRIEFs in lifecycle folders
    • Updated precedence diagram and Missing Config Auto-Setup flow

Task C — Version Bump (#390)

  • Bumped deft_version from �.19.0 to �.20.0 in skills/deft-directive-setup/SKILL.md (4 occurrences)

Checklist

  • ask check passes (1471 passed, 2 xfailed)
  • CHANGELOG.md updated under [Unreleased]
  • Conventional commit message
  • No changes to constrained files (scripts/, tests/, README.md, etc.)

Closes #387
Closes #389
Closes #390

@MScottAdams
fix(strategies,skills): add idempotency guards, update SKILL.md platf…
4111e9c 
…orm entry, bump deft_version to 0.20.0 (#387, #389, #390)

- Create strategies/artifact-guards.md with shared Spec-Generating Guard (full

  status check for specification.vbrief.json and PROJECT-DEFINITION.vbrief.json)

  and Preparatory Guard (light check for existing scope vBRIEFs)

- Add guard references to 7 strategies: speckit, enterprise, rapid (full);

  bdd, discuss, research, map (light)

- Rewrite root SKILL.md for v0.20 model: deft-setup -> deft-directive-setup,

  PROJECT.md -> PROJECT-DEFINITION.vbrief.json, SPECIFICATION.md -> scope vBRIEFs,

  update precedence diagram and Missing Config Auto-Setup flow

- Bump deft_version 0.19.0 -> 0.20.0 in deft-directive-setup SKILL.md (4 places)
@MScottAdams
Copy link
Copy Markdown
Collaborator Author

MScottAdams commented Apr 15, 2026

@greptileai review

1 similar comment
@MScottAdams
Copy link
Copy Markdown
Collaborator Author

MScottAdams commented Apr 15, 2026

@greptileai review

@MScottAdams
Copy link
Copy Markdown
Collaborator Author

MScottAdams commented Apr 15, 2026 •
edited
Loading

@greptileai

@MScottAdams
Copy link
Copy Markdown
Collaborator Author

MScottAdams commented Apr 15, 2026

@greptileai review

2 similar comments
@MScottAdams
Copy link
Copy Markdown
Collaborator Author

MScottAdams commented Apr 15, 2026

@greptileai review

@MScottAdams
Copy link
Copy Markdown
Collaborator Author

MScottAdams commented Apr 15, 2026 •
edited
Loading

@greptileai review

@MScottAdams MScottAdams merged commit c6eeff6 into phase2/vbrief-cutover Apr 15, 2026
2 checks passed
@MScottAdams MScottAdams deleted the agent2/fix/387-389-390-strategy-skill-updates branch April 15, 2026 18:38
This was referenced Apr 15, 2026
Closed
Closed
Closed
MScottAdams added a commit that referenced this pull request Apr 15, 2026
@MScottAdams
chore(vbrief): move 25 completed Tier 6 vBRIEFs from active/ to compl…
e22d2a8 
…eted/

- Move 13 original Tier 6 vBRIEFs (issues #361-#372, #345, #359) to completed/
- Move 12 merge-blocker vBRIEFs (issues #358, #381-#391) to completed/
- Update plan.status to completed and add updated timestamp on all 25
- All referenced issues are closed; PRs #374-#379, #393-#395 merged
@MScottAdams MScottAdams mentioned this pull request Apr 15, 2026
Open
44 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@MScottAdams