Repository navigation
Facilitator: verify the exercise 1 helpers, and that reset never rolls main back - #2
Open
alexiocassanifm wants to merge 4 commits into
Open
alexiocassanifm wants to merge 4 commits into
alexiocassanifm wants to merge 4 commits into
Conversation
Each step now says what to type in the terminal and what to paste in Claude Code, with what to expect. Where the lesson is a design choice (the loop's stop, its interval, where the review writes), a variant prompt changes that one thing. Windows runs natively: `py workshop`, Git for Windows for Claude Code's shell, no jq in any handout. `workshop` gains `events` (reads the stream-json log, UTF-16 too) and `hook` / `log-tool` (the exercise 1 hook, with no jq). A committed .claude/settings.json pre-approves the test, git and gh commands: without it, the exercise 1 headless run was denied every test it tried. Claude Code ignores that file until the folder is trusted, so README gains the step and doctor checks it. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UMGFBjF5h1nKda8qUaoRVV
…3 named /goal starts working as soon as it is set, so the condition carries the task and there is no second prompt to land mid-run. The allow-list keeps the gh verbs that read or comment: pre-approving `gh pr *` also pre-approved `gh pr merge --admin`, the act exercise 5 says only a prompt forbids. Exercise 5's duplicate filing happens in the same copy, the only place its search can see. On Windows, doctor checks the `python3` the loop gate calls from Git Bash, and README says exercise 3 run 2 is untested there. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UMGFBjF5h1nKda8qUaoRVV
…s main back verify.sh drives `events` (UTF-8 and UTF-16), the hook command `hook` writes, and `log-tool`, and fails when a starting branch built from an older main would replay that main's files over a copy's. Rehearsed: green on rebuilt branches, and the new check fails on the stale ones. The guide says the branches are rebuilt in the same sitting as every change to main, and what is untested on Windows. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UMGFBjF5h1nKda8qUaoRVV
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.
Companion to the handouts PR into
main(its commits are included here through the merge).verify.shdrivesworkshop events(UTF-8 and UTF-16), the hook commandworkshop hookwrites (successes andPostToolUseFailure), and checks that nothing is left for git to commit.verify.shfails when a starting branch would rollmain's files back:resetreplays the oldmain→exN-startdiff. Rehearsed in a scratch clone: green on rebuilt branches, andVERIFY FAIL: reset ex2 rolls main's files backon the current ones.main, and what is untested on Windows.Order: merge the
mainPR, merge this one, thenpython3 tools/build_branches.py,verify.sh, and force-pushex2-start ex3-goal-start ex3-loop-start ex4-start.🤖 Generated with Claude Code
https://claude.ai/code/session_01UMGFBjF5h1nKda8qUaoRVV