Repository navigation
Make Beginner step text read whole on the card and word paths relative to project/ (349) - #236
Merged
Merged
Conversation
…e to project/ (349)
…dry-run found (349)
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 this changes
Low-severity step text from the live Beginner run (issue 07), lesson by lesson, with the Guide page and the step file saying the same thing:
project/(b-2 Y4, capstone T5, b-5 Y3): the notes now say "relative toproject/", and thatgit status --porcelainprints paths from the folder aboveproject/, such asproject/src/links.js, which the check accepts too. b-5 Y3 namesproject/.gitignoreand says to leave the copy's second.gitignoreas it is. Newbeginner-process.test.mjscases prove both forms pass b-2 and the capstone. b-2 Y4's beat-2 note now starts with a capital.nothing to commit, working tree clean.Esc; b-4 Y5 names.practice/b-4-after-rewind.txt; b-5 W4 asks for one restart, inproject/.FIX_PATHandNOT_FOUNDnow say "fix PATH as Troubleshoot installation describes (lesson 1's Guide page links it under Check the install), then retry."beginner-failures.jsonchanges only those four b-1 messages.CHANGED_SINCE_LEGACY.code:block with Copy, as the page shows it; b-5 W1 adds the page'scat CLAUDE.mdbeat; b-3 W1's lone "Explore" goal is a full instruction.Not changed: the legacy Terminal app copy
practice-template/learn/lessons/b-4.mdstill says "the step 2 prompt";template.mjskeeps it out of learner copies.Checklist
Claim table
Filled by a separate agent that tried to refute each changed claim:
/modelpicker shows "Requires usage credits" on the Fable row." So a shared header model says nothing about the rest of the picker. This is an inference from these lines; no single docs sentence states it.Escwhenever Claude heads the wrong way, then say what you want instead."Esc| Interrupt Claude, or close a dialog | Stop the current response or tool call mid-turn so you can redirect. Claude keeps the work done so far." How Claude Code works also says: "PressEscto stop Claude immediately. The running tool call is canceled and Claude waits for your next instruction."command not foundornot recognizederror when runningclaude, the install directory isn't in your PATH." Lesson 1's step 2, "Check the install", links it as "fix it with Troubleshoot installation". Note: the page's actual title is "# Troubleshoot installation and login", so the short name "Troubleshoot installation" is the guide's own label, used the same way across lesson 1.claudein your copy'sproject/folder, so it loads CLAUDE.md with your rule", then find it with/context allunder Memory filesCLAUDE.mdandCLAUDE.local.mdfrom your current working directory and every directory above it." and "CLAUDE.md and CLAUDE.local.md files in the directory hierarchy above the working directory are loaded at launch." Also: "To confirm the file loaded, run/context allin a session and check the list under Memory files. In fullscreen mode,/contextalone collapses that list."CLAUDE.local.mdin theproject/folder next toCLAUDE.md, and addCLAUDE.local.mdtoproject/.gitignoreCLAUDE.local.mdat the project root. It loads alongsideCLAUDE.mdand is treated the same way. AddCLAUDE.local.mdto your.gitignoreso it isn't committed." The table row reads "./CLAUDE.local.md| Personal project-specific preferences; add to.gitignore". Becauseproject/is the session's working directory, this matches.git status --porcelainprints each path from the root of the repository, the folder aboveproject/, such asproject/src/links.js: the check accepts either form."planPathsstrips theproject/prefix and a leading./, and the commit's new tests in beginner-process.test.mjs coversrc/config.js,project/src/config.jsand./project/src/config.jsfor b-2, and both forms for b-capstone./rewind, or pressEsctwice when the prompt input is empty, to open the rewind menu." and "Restore code and conversation: revert both code and conversation to that point"--resume,-r| Resume a specific session by ID or name" and, under--name, "You can resume a named session withclaude --resume <name>." The words 'the refactor and its commit' describe what that conversation contains, not a documented behavior. The source line for the unchanged why ('claude --continuereopens the most recent session in this folder') is "Load the most recent conversation in the current directory".Dry-run
Live runs from this branch (f00139c) with real Claude Code v2.1.309 at start and end of each, a clean configuration, macOS, driven only by what the app shows (logs
~/.learn-runs/logs/rel349-b1/,rel349-b4/,rel349-b5/). b-4 and b-5 were played to the end: app andnpm run checkpassed (4 of 4, 5 of 5). b-1 covered W2 to W5.claudeoff PATH: "... If it fails there too, fix PATH as Troubleshoot installation describes (lesson 1's Guide page links it under Check the install), then retry." The app's Read the full lesson shows that link in step 2. After restarting from a normal PATH, W2 passed./rewindlisted that prompt just above "(current)", and after Restore codeb-4-after-rewind.txtheld onlyM project/src/links.js. Y5 names.practice/b-4-after-rewind.txt./exit; W6's expect matched the shell; Y3 namedproject/.gitignoreand the second.gitignore, and the Check passed with onlyproject/.gitignoreedited.point 1gutter on b-5 W2's code block has hidden "explained line:" text with nothing after it.