Repository navigation
docs: upgrade guide, hook reinstall recovery, and agent guide task done - #31
Draft
Deep-Sixed wants to merge 2 commits into
Draft
Deep-Sixed wants to merge 2 commits into
Deep-Sixed wants to merge 2 commits into
Conversation
- README: one Upgrading section in migration order, replacing three scattered notes. Adds the missing 006 note (hubs reject every push until `hook install` is re-run), says 007-009 need nothing, and explains the 003 gap. - README: document recovery when the interpreter path baked into the hooks moves - `hook install` refuses to overwrite its own outdated hooks, so delete both and reinstall. - README: "Eight concepts" -> nine; `status` prints `uri:` and `running: false`; add task done to the audited actions; note that Git invokes `hook pre-receive`/`hook reference-transaction`. - agent guide: step 6 now distinguishes `task done` (finished), handoff (someone continues) and release (stepping away), so agents stop putting finished work back in the queue. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SHGwY6PLGwr7kQY63qQGq6
Resolve the README conflict by keeping the consolidated Upgrading section and folding main's new 010 (protected ref full names) and 011 (quorum consumer separation) notes into it in migration order. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SHGwY6PLGwr7kQY63qQGq6
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
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.
Fixes from a second doc review of
README.mdanddocs/agent-guide.md. Docs only; no code changes.Operational gaps
006_ref_updates.sql. Hubs set up before it reject every push untilquorumgit hook installis re-run to add thereference-transactionhook. The README's upgrade notes, previously scattered, are now a single Upgrading section in migration order covering 004 to 011, with an explanation of why there is no003(that number was used only on branches that were never merged). The 010 and 011 notes that arrived frommainin Compose ref policies and preserve quorum executor separation #35 are included in the same list.hook install. If that interpreter moves (a Python upgrade, a rebuilt venv, a reinstall), pushes fail, andhook installrefuses to overwrite its own now-different hooks ("Existing QuorumGit-managed hook differs"). The fix is documented: delete both hook files, then reinstall. I reproduced both the refusal and the recovery against a real bare hub, before and again after mergingmain.Agent guide
releasefinished work, which returns the task toopen. It now distinguishes three cases:task donefor finished work,handoffwhen someone should continue, andreleasewhen stepping away from unfinished work.Accuracy
statusis described as it behaves: it labels the pathuri:, and printsrunning: falseand exits 1 when no store exists. Verified by running it.task doneis added to the list of audited actions.hook pre-receiveandhook reference-transaction, so users never run them.Merge
main(#32, #34, #35) is merged in. The one conflict was in the README upgrade notes, resolved by foldingmain's 010 and 011 notes into the consolidated section.After the merge:
ruff checkis clean,pyrightreports 0 errors, andpytest testsgives 283 passed.🤖 Generated with Claude Code
https://claude.ai/code/session_01SHGwY6PLGwr7kQY63qQGq6