Updates wurk overrides for the merged CI workflow - #24
Merged
Conversation
.github/workflows/ci.yml landed on main (sui-56w), so the claims in .claude/wurk/commit.md and .claude/wurk/mr.md that no CI exists are now false. Both files told an unattended agent that its local gate run was the only check a branch would ever get, which is no longer true - CI replays the full gate on pushes to main and on pull requests. Rewrites the two sections to say what CI now does and what it still does not: it runs after a push rather than before a commit, checks nothing about commit messages, and adds no human reviewer. The local full green stays the commit trigger, and requests still wait for no human review. The matching claims in CLAUDE.md and .quality.exs live in operator-only files and are reported separately rather than edited here. Refs: sui-68k
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #23 - merge #23 first.
What
.claude/wurk/commit.mdand.claude/wurk/mr.mdboth asserted that thisrepository has no CI. Since #22 merged,
.github/workflows/ci.ymlruns thefull quality gate (read from
.claude/wurk.json'sgate.full) on pushes tomainand on pull requests, so both claims were false - and an unattendedagent decides what evidence it needs from what these files tell it.
commit.md: the "No CI means the gate is the whole check" section becomes"CI replays the gate; the local run is still the trigger", and the
commit-msg-hook rationale now says CI checks nothing about commit messages
rather than that no CI exists. The local full green stays the commit
trigger.
mr.md: "The request is a record, not a review gate" becomes "The requestis a record and a CI run, not a human review gate" - a check does run when
a request opens now, but there is still no human reviewer, and a skipped
local gate is still a hard stop.
Deliberately not in this PR
The bead names two more files carrying the same claim, and reserves both for
the operator: the
CLAUDE.mdagent-authority section ("There is no CI and nosecond reviewer") and the
.quality.exsheader ("There is no CI. ..."). Theyare reported on the bead with proposed wording rather than edited here.
Refs: sui-68k