M10: gloss /path/to/, git apply, and review-lab as throwaway #61

2026-06-22T16:45:13-04:00

claude commented

2026-06-22 16:45:13 -04:00

Module 10's lab embedded literal /path/to/ placeholders and a first-use git apply with no explanation — a clarity/consistency gap versus the locked Module 1 exemplar.

Changes

/path/to/ gloss (Part A & B): added the course-wide "use your real course path in place of /path/to/" gloss that M04/M07/M26 already carry. M10 was the one module using the placeholder with no instruction to substitute. (Kept the /path/to/ convention itself — it's used across M04–M07, M21, M22, M26.)
git apply glossed on first use: "lays the AI's proposed change onto this branch as if it were its PR, so you can read it before deciding whether to keep it — exactly what you'd be doing in a real PR review." git apply appears nowhere else in the course. Avoids implying a merge (the lab ends in "request changes").
review-lab identified as throwaway: noted it's a throwaway repo separate from tasks-app, deletable when done, and that nothing here touches the main app.

Verification

Confirmed git apply of lab/ai-change.patch still applies cleanly onto the base app (the documented mechanic).
No build/test harness exists in this content repo.

Acceptance criteria

No paste-ready command contains an unresolved /path/to/ placeholder without explanation.
git apply is glossed on first use.
review-lab is identified as a throwaway repo distinct from tasks-app.

Closes #53

🤖 Generated with Claude Code

https://claude.ai/code/session_01CfDmDAQL7xuSRm2L4qApvn

Module 10's lab embedded literal `/path/to/` placeholders and a first-use `git apply` with no explanation — a clarity/consistency gap versus the locked Module 1 exemplar. ## Changes - **`/path/to/` gloss (Part A & B):** added the course-wide "use your real course path in place of `/path/to/`" gloss that M04/M07/M26 already carry. M10 was the one module using the placeholder with no instruction to substitute. (Kept the `/path/to/` convention itself — it's used across M04–M07, M21, M22, M26.) - **`git apply` glossed on first use:** "lays the AI's proposed change onto this branch as if it were its PR, so you can read it before deciding whether to keep it — exactly what you'd be doing in a real PR review." `git apply` appears nowhere else in the course. Avoids implying a merge (the lab ends in "request changes"). - **review-lab identified as throwaway:** noted it's a throwaway repo *separate* from `tasks-app`, deletable when done, and that nothing here touches the main app. ## Verification - Confirmed `git apply` of `lab/ai-change.patch` still applies cleanly onto the base app (the documented mechanic). - No build/test harness exists in this content repo. ## Acceptance criteria - [x] No paste-ready command contains an unresolved `/path/to/` placeholder without explanation. - [x] `git apply` is glossed on first use. - [x] review-lab is identified as a throwaway repo distinct from tasks-app. Closes #53 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01CfDmDAQL7xuSRm2L4qApvn

claude added 1 commit 2026-06-22 16:45:13 -04:00

M10: gloss /path/to/, git apply, and review-lab as throwaway (#53 ) 67af52e37f

Module 10's lab embedded literal `/path/to/` placeholders and a first-use
`git apply` with no explanation, unlike the Module 1 exemplar. This adds the
course-wide "use your real course path in place of /path/to/" gloss (matching
M04/M07/M26), a half-sentence gloss on git apply, and a note that review-lab
is a throwaway repo separate from tasks-app.

Closes #53

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
Claude-Session: https://claude.ai/code/session_01CfDmDAQL7xuSRm2L4qApvn

claude merged commit 90012ca711 into main

2026-06-22 16:52:31 -04:00

claude deleted branch claude/issue-53

2026-06-22 16:52:31 -04:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: justin/ai-workflow-course#61