Skip to content

Add docs-verification node to workflow DAG#287

Merged
srid merged 3 commits intomasterfrom
add-docs-verification-node
Apr 2, 2026
Merged

Add docs-verification node to workflow DAG#287
srid merged 3 commits intomasterfrom
add-docs-verification-node

Conversation

@srid
Copy link
Copy Markdown
Member

@srid srid commented Apr 2, 2026

The /workflow do DAG runs end-to-end but never checks whether documentation still matches the code after structural changes. PR #286 (tmux shim) shipped new server modules without updating the README architecture section — exactly the kind of drift CLAUDE.md warns about.

This adds a docs check-then-fix loop between update-pr and done, mirroring the existing police/police-fix and ci/ci-fix patterns. The node reads README.md's Architecture section, compares against git diff master...HEAD, and updates any stale sections before completing.

Also strengthens workflow.md with a "never stop between nodes" rule to prevent the orchestrator from pausing after skill nodes return.

  • sync
  • understand
  • hickey
  • branch
  • implement
  • e2e
  • fmt
  • commit
  • police
  • test
  • ci
  • update-pr
  • docs
  • done

srid added 3 commits April 1, 2026 20:25
Add `docs` and `docs-fix` nodes between `update-pr` and `done` that
verify README/CLAUDE.md architecture docs are still accurate after
structural changes. Mirrors the existing police-fix loop pattern.

Also add "never stop between nodes" rule to workflow.md to prevent
the orchestrator from pausing after skill nodes.
The example progress bar was missing the new `docs` node.
@srid srid marked this pull request as ready for review April 2, 2026 00:29
@srid
Copy link
Copy Markdown
Member Author

srid commented Apr 2, 2026

#280

@srid srid merged commit 900b591 into master Apr 2, 2026
9 of 10 checks passed
@srid srid deleted the add-docs-verification-node branch April 2, 2026 00:30
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