Skip to content

docs(ui): capture Start admin coverage - #645

Merged
zacjones93 merged 6 commits into
codex/demo-competitions-semanticsfrom
codex/ui-start-admin-coverage
Jul 15, 2026
Merged

docs(ui): capture Start admin coverage#645
zacjones93 merged 6 commits into
codex/demo-competitions-semanticsfrom
codex/ui-start-admin-coverage

Conversation

@zacjones93

@zacjones93 zacjones93 commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • verifies 16 Start platform-admin scenarios across eight routes at desktop 1440×900 and mobile 390×844
  • adds 80 hash-pinned browser artifacts plus one exact-revision capture manifest
  • updates the page-coverage plan, generated ledgers, reusable-pattern audit, and LAT documentation

Routes

  • /admin
  • /admin/competitions
  • /admin/demo-competitions
  • /admin/docs
  • /admin/docs/new
  • /admin/entitlements
  • /admin/organizer-requests
  • /admin/teams

Evidence result

  • exact revision: 739ca5370f292aac9f20a1ae7389c6e6e07e9e4c
  • 16/16 requested URLs, viewports, and light themes matched
  • zero page errors, console errors, responses >=400, or horizontal overflow
  • demo-competition hierarchy verifies h1 -> h2 -> h3 and valid list nesting
  • text evidence excludes headers, cookies, credentials, query values, database URLs, absolute repository paths, and synthetic emails
  • ledger totals: 204 verified / 26 blocked / 128 pending (358 scenarios)

Validation

  • pnpm check:page-coverage
  • lat check
  • manifest/artifact hash, privacy, freshness, orphan, viewport, theme, heading, console, network, overflow, and screenshot-dimension checks
  • pre-push monorepo lint and type-check
  • visual contact-sheet review of all 16 screenshots

GitNexus indexed this exact worktree, but its MCP scope read could not open the new index because the MCP build expects storage version 40 while the CLI wrote version 42. No application symbols changed in this docs/evidence-only PR.

Stacked on the admin hydration, dashboard-heading, and demo-semantics fixes.


Summary by cubic

Captured Start platform-admin coverage across eight routes on desktop (1440×900) and mobile (390×844) light themes with a hash-pinned manifest; 16/16 scenarios verified with zero errors and ledgers updated (204 verified, 128 pending, 26 blocked). Also added a shared @repo/ui/metric primitive and refactored organizer empty states to @repo/ui/empty-state, updating Crew/Start consumers and tests.

  • New Features

    • Introduced @repo/ui/metric composition; added app adapters and Storybook; preserved compact typography variants.
  • Refactors

    • Replaced OrganizerEmptyState with @repo/ui/empty-state in registration questions, event division mapper, invite sources, and venue manager.
    • Adopted Metric for summary counts in Crew import tabs, volunteer import flow, heat import preview, template panel, and copy-prior-event panel.
    • Added unit tests for empty states and metric labels; updated UI inventory and generator to include metric.

Written for commit d6316c1. Summary will update on new commits.

Review in cubic

@coderabbitai

coderabbitai Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0d099a8d-be3a-46e8-998d-b35ffc63b5df

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/ui-start-admin-coverage

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

This was referenced Jul 11, 2026
@zacjones93

Copy link
Copy Markdown
Contributor Author

Independent exact-base review: LGTM at f1f2431 (base 739ca53). I found no actionable issues. The diff is docs/evidence-only and changes exactly the eight intended Start admin route records. The manifest contains 16 captures (8 routes x desktop 1440x900/mobile 390x844, light), all pinned to the base deployment revision with matching requested/final URLs and admin/populated plan metadata. All 80 artifact refs are unique, present, hash-valid, and fully accounted for; the manifest hash matches every ledger reference, with no missing or orphan files. Artifact inspection found no serialized headers/cookies/credentials/form values/query values/database URLs/absolute local paths/synthetic emails; zero console errors, page errors, responses >=400, query-bearing network paths, or horizontal overflow. Every DOM capture has one h1; demo evidence shows h1 -> h2 -> h3 and nested lists containing listitems. Both 16-image contact sheets are visually clean. Counts reconcile from 188/26/136 over 350 scenarios to 204/26/128 over 358. pnpm check:page-coverage, lat check, and git diff --check pass at the exact head. All GitHub CI and E2E checks are green. GitNexus symbol mapping was unavailable due the documented storage-version mismatch, but no application symbols changed.

@zacjones93
zacjones93 merged commit d6316c1 into codex/demo-competitions-semantics Jul 15, 2026
24 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