docs: sync CLAUDE.md with unified event-driven work flow#84
Merged
Conversation
The two-phase preliminary/thorough plan state machine was replaced by a single unified work task (tasks/work.py, prompts/work.j2) where Claude decides the next action. CLAUDE.md still described the old lifecycle, deleted task files, removed prompts, stale state fields, and the is_plan_approved gate — and since CLAUDE.md is loaded as Clayde's system prompt, that staleness misleads Clayde about its own architecture. Updates: project tree, State Machine -> Work Loop, single Work Task section, Safety Gates, logger names. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.
What
CLAUDE.mdstill documented the old two-phasepreliminary → thorough → 👍plan state machine, deleted task files (tasks/plan.py,implement.py,review.py), removed prompts (preliminary_plan.j2,thorough_plan.j2,address_review.j2, …), the droppedis_plan_approvedgate, and stale state fields (last_seen_comment_id,last_seen_review_id,preliminary_comment_id).Reality on
main: a single unified work task (tasks/work.py+prompts/work.j2) where Claude decides the next action (ask / plan / implement / open PR / address review). Verified: those stale terms appear nowhere insrc/.Why it matters
CLAUDE.mdis loaded as Clayde's system prompt, so the stale doc actively misleads Clayde about its own architecture.Changes (docs only)
work.py/work.j2is_plan_approved, addget_new_visible_commentsScope: only the staleness already landed on
main. The assigned-PR handling from #82 (pr_work.py) is intentionally not documented here — it should be documented by #82 when it merges.🤖 Generated with Claude Code