Skip to content

docs: project state consolidation report ahead of restart - #95

Merged
IanMayo merged 2 commits into
mainfrom
claude/dormant-project-consolidation-q5qpr9
Sep 5, 2026
Merged

IanMayo merged 2 commits into
mainfrom
claude/dormant-project-consolidation-q5qpr9

Conversation

@IanMayo

@IanMayo IanMayo commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Consolidation audit of the dormant codebase, written for an incoming maintainer ahead of a formal restart. Adds docs/PROJECT_STATE.md and applies three trivial fixes. Existing documents were audited, not rewritten.

What was run (all on this branch, Node 22)

typecheck, lint, format, unit + integration with coverage, test:gaps, build + size-check, build-storybook, Playwright E2E in default and ENCRYPT_STORAGE=true modes, analyze:e2e-gaps. Chromatic assessed from CI logs (no token locally).

Headline findings (details in the report)

  • Code is healthy: 831 unit, 84 integration, 105 + 104 E2E tests green; bundle 37.6 KB / 40 KB.
  • Chromatic / Storybook build has been broken since the June refactor (orphan qd-error-banner story) and masked by continue-on-error. PR-preview Storybook silently skips for the same reason.
  • Coverage thresholds (80%) are never evaluated in CI and would fail (62% unit lines, 40% integration). E2E job is commented out.
  • "Erase All Data" clears sessionStorage only; StorageService.clearAll() has no callers.
  • 14 of 20 checked Technical_Design.md claims no longer hold (events, components, flags, init() API, cross-tab sync).
  • Toolbar unlock reads #instructor.password.hash, which the DITA template never emits.
  • Four fully-built modules have no importers; spec 009 has three open tasks for an E2E file never written.

The report ends with an 18-item restart backlog ranked by risk with effort tags.

Changes

  • docs/PROJECT_STATE.md — new report
  • docs/test-coverage-report.md — regenerated (was dated 2025-11-26)
  • CODE_REVIEW_REFACTORING_HOTSPOTS.md — "superseded by spec 012" banner only
  • CLAUDE.md — one index line pointing at the report
  • temp-e2e/analysis-test.html — stray file removed

Verification

npm run typecheck, lint, format:check, test:unit, test:integration, build, size-check all pass locally.

🤖 Generated with Claude Code

https://claude.ai/code/session_015u1mfVebscuZA3YDc6hc87


Generated by Claude Code

- Regenerate docs/test-coverage-report.md via `npm run analyze:e2e-gaps`
  (previous report dated 2025-11-26; now 15 spec files / 92 tests)
- Remove stray temp-e2e/analysis-test.html left at repo root
- Add a "superseded by spec 012" banner to CODE_REVIEW_REFACTORING_HOTSPOTS.md
  so readers know its file paths describe the pre-refactor codebase

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015u1mfVebscuZA3YDc6hc87
Audit of the dormant codebase ahead of a formal restart, written for an
incoming maintainer. Covers: verification results from running every
check locally (typecheck, lint, coverage, gaps, build, size, Storybook
build, E2E in both storage modes), architecture drift against
Technical_Design.md, a classification of every root/docs markdown file,
test and visual-regression status, a maintainer-documentation gap
assessment, and a risk-ranked restart backlog.

Key findings recorded: Chromatic/Storybook build broken since the June
refactor and masked by continue-on-error; coverage thresholds never
evaluated in CI; E2E job disabled; "Erase All Data" clears sessionStorage
only; 14 of 20 design-doc claims no longer hold.

Also links the report from the Key Documentation list in CLAUDE.md.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015u1mfVebscuZA3YDc6hc87
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

🚀 PR Preview Deployment

✅ Your changes are being built and deployed to a preview environment.

📋 Live PR Preview Links:

ℹ️ About PR Previews:

📦 Production Demo (main branch):

🧪 Testing Locally:

npm install
npm run build
# Open demo/quiz-index.html in your browser (use file:// or http-server)

This comment will be automatically updated when you push new commits.

github-actions Bot added a commit that referenced this pull request Sep 5, 2026
@IanMayo
IanMayo merged commit 91ea1a4 into main Sep 5, 2026
7 checks passed
@IanMayo
IanMayo deleted the claude/dormant-project-consolidation-q5qpr9 branch September 5, 2026 07:53
github-actions Bot added a commit that referenced this pull request Sep 5, 2026
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.

2 participants