Skip to content

Add a complete Git practice course with validated lesson states - #2

Merged
dazzo31 merged 2 commits into
mainfrom
feat/learning-course
Nov 7, 2026
Merged

dazzo31 merged 2 commits into
mainfrom
feat/learning-course

Conversation

@dazzo31

@dazzo31 dazzo31 commented Nov 7, 2026 •

Copy link
Copy Markdown
Owner

Description

GitUnstuck offered concept illustrations without a complete practice sequence. This adds optional setup and six lessons covering inspection, staging, history, branches, remotes and recovery, with predictions, independent tasks and exact reference returns.

For example, a learner stages Version 2, saves Version 3, and verifies that committing records Version 2 while Version 3 remains on disk. Shared learning.js definitions drive source rendering, the self-contained production build and real-Git outcome checks. Branch/remote diagrams reflect checked repository states; existing recovery routes remain available.

Includes responsive lesson navigation, clipboard fallback, lesson-embedded optional visual tools, reference printing, honest build identity, and updated authoring documentation. The staging visual opens at the current lesson checkpoint; the branch explorer lives inside the branches lesson. The overview has one course path, and old visual deep links still work. See implementation evidence and plan.

Related issue

No linked issue. Implements the supplied learning improvement plan.

Type of change

  • New feature
  • Content update (recipe, guidance, or wording)
  • Documentation

Testing

  • npm ci
  • npm test: 194 analyzer assertions, 33 recovery scenarios, all seven lesson fixtures/independent tasks, negative states, commit-parent/remote-reference checks, and 14 published setup executions across Git Bash and Windows PowerShell 5.1
  • npm run build:check and npm run build
  • Source and built Chromium journeys: all lessons, exact reference return/reload/history, legacy routes, analyzer, search/filter, clipboard success/failure, privacy sentinel and reference print
  • 320/390/768/1440 CSS-pixel presentation checks in both themes, keyboard/focus, reduced motion, sampled contrast, expanded answers and desktop CSS zoom 200%
  • git diff 7115b5d..HEAD --check

Local environment: Windows, Node 26.12.1, Git 2.56.0.windows.4, Chromium 151.0.7922.34. Checked-in CI uses Ubuntu/Node 22. The original draft head f281d45 passed; CI for the latest refinement is tracked separately. Runtime changes were tested before committing; subsequent preparation changed documentation only. The evidence file identifies the actual dirty-build fingerprint tested locally.

Checklist

  • No new dependencies or build steps added; existing build embeds the learning module
  • Pasted input stays in the browser; no new transmission or persistence
  • Recovery prerequisites, consequences and source citations retained; recovery.js unchanged
  • Full accessibility validation: available automated/keyboard checks passed, but a real screen-reader pass remains outstanding

Remaining validation

42 plan tasks pass. Actual beginner sessions and follow-up transfer checks remain HUMAN_PENDING; prompts and an anonymous observation template are ready in the evidence file. Actual Git 2.23 runtime, a second browser engine and browser-menu zoom remain unverified. CSS zoom is narrower evidence, and automated checks do not establish teaching effectiveness or full accessibility conformance.

Draft for review. No merge or deployment is requested; live release verification remains pending authorization.

@dazzo31
dazzo31 marked this pull request as ready for review November 7, 2026 13:52
@dazzo31
dazzo31 merged commit 5678788 into main Nov 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant