Skip to content

feat(ui): add ProgressTracker learning dashboard#148

Open
bntvllnt wants to merge 3 commits intofeat/storybookfrom
feat/issue-19-feat-component-progress-tracker
Open

feat(ui): add ProgressTracker learning dashboard#148
bntvllnt wants to merge 3 commits intofeat/storybookfrom
feat/issue-19-feat-component-progress-tracker

Conversation

@bntvllnt
Copy link
Copy Markdown
Collaborator

Summary

  • add a new compound ProgressTracker component family for curriculum progress dashboards
  • ship overview, module, stats, and badge subcomponents with checklist-backed persistence support via persistKey
  • add Storybook docs, unit coverage, and a Playwright component snapshot for the new learning UI

Validation

  • pnpm -F @vllnt/ui exec vitest run src/components/progress-tracker/progress-tracker.test.tsx
  • pnpm -F @vllnt/ui exec eslint src/components/index.ts src/components/progress-tracker/index.ts src/components/progress-tracker/progress-tracker.tsx src/components/progress-tracker/progress-tracker.test.tsx --max-warnings 0
  • pnpm -F @vllnt/ui exec tsx scripts/verify-stories.ts
  • pnpm -F @vllnt/ui exec playwright test -c playwright-ct.config.ts src/components/progress-tracker/progress-tracker.visual.tsx --update-snapshots
  • pnpm -F @vllnt/ui build

Closes #19

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
storybook Ready Ready Preview, Comment Apr 25, 2026 5:49am
ui.vllnt.ai Ready Ready Preview, Comment Apr 25, 2026 5:49am

Request Review

@bntvllnt
Copy link
Copy Markdown
Collaborator Author

Status

  • ready for final human review
  • blockers: none

Validation

  • pnpm -C packages/ui exec vitest run src/components/progress-tracker/progress-tracker.test.tsx
  • pnpm -C packages/ui exec eslint src/components/progress-tracker/progress-tracker.tsx src/components/progress-tracker/progress-tracker.test.tsx src/components/checklist/checklist.tsx src/components/checklist/index.ts src/components/progress-tracker/index.ts src/components/index.ts --no-warn-ignored
  • pnpm -F @vllnt/ui exec tsx scripts/verify-stories.ts
  • pnpm -F @vllnt/ui exec playwright test -c playwright-ct.config.ts src/components/progress-tracker/progress-tracker.visual.tsx
  • pnpm -F @vllnt/ui build
  • ℹ️ focused tsc --noEmit grep for touched files returned no progress-tracker hits; package-wide raw output remained too noisy/slow to use as the authoritative signal in this cron run

Claude review

  • Ran on the narrowed live diff.
  • No confirmed blocking finding remains on the current head.
  • Claude did surface stale concerns from an earlier diff state (polling-based progress sync / locked modules still navigable), but both were manually verified fixed in the current head before this update.

Codex review

  • Ran on the live diff.
  • No confirmed blocking finding remains on the current head.
  • Warn: packages/ui/src/components/progress-tracker/progress-tracker.mdx:15 still points install docs at https://ui.vllnt.com/r/progress-tracker.json while the current registry app routes/quick-add flow resolve under ui.vllnt.ai; treat that as follow-up doc drift unless .com is intentionally redirected.
  • A second Codex warning about exporting CHECKLIST_PROGRESS_EVENT from the package root was reviewed and not accepted as a blocker; that event is intentionally kept internal to the checklist/progress-tracker integration path.

Branch / PR state

  • pushed fix commit for checklist-driven progress resync and locked-module non-interactivity
  • merged latest feat/storybook into this branch to clear the index.ts export conflict
  • PR is now marked ready for review

Action for bntvllnt

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