docs: sync all documentation with the current implementation#8
Merged
Conversation
The README and CONTENT_GUIDE had drifted from the shipped product. Correct every stale count and capability claim against the source of truth: - 225 lessons (was 124): 115 across 9 role paths + 110 across 11 verticals, counted from Curriculum.swift and Content/Lessons/. - 20 certification exams (was "9 role"): 9 role + 11 vertical. - 5,127 native known-answer checks (was "2,500+"), from ./Scripts/test.sh. - Web demo is 13 interactive explorables incl. Pixel Setup (was listed as 3), 147 web tests; added a web build/test quickstart. - Latest release is v1.1.1 (Status said "BidLab 1.0"); link docs/ACCEPTANCE.md. - CONTENT_GUIDE: track list now includes adops, ai, and all 11 verticals; the widget catalog adds the pixel widgets; the stale "only 3 are interactive" note is replaced (all catalog types render live except monteCarlo) with the WidgetRegistry + knownWidgets-whitelist gotcha. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The public web demo footer claimed "124 interactive lessons ... certification exams". Update to 225 lessons and 20 certification exams to match the app. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…0 web) #9 raised the native suite to 5,271 checks and #10 the web suite to 180; update the README and ACCEPTANCE.md to match, and add the CI coverage-gate story (web 100%, native 100% of reachable code). ACCEPTANCE.md's web engine-suite row now includes the added boundary tests (47 -> 80). Co-Authored-By: Claude Opus 4.8 <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.
A full documentation pass: review the codebase and correct every stale fact so the docs match what actually ships on
main.What was stale → corrected (all verified against source)
Curriculum.swift,ls Content/Lessonsls Content/Exams./Scripts/test.shweb/src/App.tsxNAVcd web && npm testdocs/ACCEPTANCE.mdFiles
README.md— every count above; expanded the web-demo description to all 13 explorables; added a web build/test quickstart; linked the acceptance-evidence report.Content/CONTENT_GUIDE.md—tracklist now includesadops,ai, and all 11vertical-*tracks (was missing); widget catalog addspixelSetup/pixelCalculators; replaced the stale "only 3 widgets are interactive" note (all catalog types now render live exceptmonteCarlo) and documented theWidgetRegistry+knownWidgetswhitelist gotcha.web/src/App.tsx— corrected the public demo footer ("124 ... certification exams" → "225 ... 20 certification exams").Verification
cd web && npm run build→ typecheck + vite build cleancd web && npm test→ 147 passed./Scripts/test.sh→ 5,127 native checks passed (Swift untouched in this PR)124/2,500stale stats remain; all referenced screenshots andog.pngexist.Deliberately not touched
reviews/*.mdare point-in-time review-cycle snapshots (e.g. "Cycle 1 baseline"), not living docs — rewriting them would falsify history.SettingsSurface.swift:130) still reads "BidLab 1.0". Picking the next version number is a release decision, so it is flagged for you rather than guessed here.🤖 Generated with Claude Code