Skip to content

feat(sldd): require draft artifact approval gates#12

Open
dearrudam wants to merge 1 commit into
mainfrom
feature/sldd-skills-draft-artifact-approval
Open

feat(sldd): require draft artifact approval gates#12
dearrudam wants to merge 1 commit into
mainfrom
feature/sldd-skills-draft-artifact-approval

Conversation

@dearrudam

Copy link
Copy Markdown
Contributor

Summary

  • Save Markdown-producing SLDD artifacts as reviewable drafts while keeping steps pending until explicit approval.
  • Clarify that artifact existence never satisfies gates; journal status complete is required to advance.
  • Preserve Step 04/05 evidence-only behavior, Step 88 non-mutating defaults, and workflow-set scaffold approval boundaries.

Validation

  • python -m json.tool skills/sldd/schema/_spec-journal.schema.json >/dev/null
  • git diff --check
  • static invariant checks for draft artifact approval policy and workflow-set exceptions
  • reviewer subagent pass: no blockers

Save Markdown-producing step artifacts as reviewable drafts while keeping gates pending until explicit approval marks the journal step complete.

Preserve Red/Green evidence-only phases, Step 88 non-mutating behavior, and workflow-set scaffold approval boundaries.

Co-authored-by: Pi Harness <pi-harness@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant