Skip to content

feat(runs): add validation gate summary#79

Merged
ncolesummers merged 1 commit into
mainfrom
codex/76-gate-summary-ui
Jul 9, 2026
Merged

feat(runs): add validation gate summary#79
ncolesummers merged 1 commit into
mainfrom
codex/76-gate-summary-ui

Conversation

@ncolesummers

Copy link
Copy Markdown
Owner

Summary

Closes #76.

Adds the run-detail validation gate summary to /runs, backed by persisted validation_report_result artifact metadata. The UI renders pass/fail/skipped gates with StatusBadge, explicit empty/error states, and safe raw artifact link handling for present, missing, and invalid URIs.

What Changed

  • Projected RunRecord.validationSummary from completed validation report artifacts.
  • Added ValidationGateSummary and composed it into the run detail before review/artifact evidence.
  • Extended safe external link handling for artifact://validation/... raw artifacts.
  • Updated fixtures and seed data for mixed, empty, invalid-link, and no-raw-artifact states.
  • Added PGlite, component, safe URL, Storybook, and e2e coverage.

Screenshots

Mixed validation gates:
runs-mixed.png

Empty validation state:
runs-empty.png

Invalid and missing raw artifact handling:
runs-invalid-raw-link.png

Mobile dark mode:
runs-mobile-dark.png

Validation

Red-first evidence was collected before implementation for:

  • bun run test tests/unit/runs/run-record.test.ts
  • bun run test tests/unit/portal/components.test.tsx
  • bun run storybook:build
  • bun run test:e2e -- tests/e2e/portal.spec.ts

Green checks after implementation:

  • bun run test tests/unit/runs/run-record.test.ts
  • bun run test tests/unit/portal/components.test.tsx
  • bun run test tests/unit/portal/safe-url.test.ts
  • bun run storybook:build
  • bun run test:e2e -- tests/e2e/portal.spec.ts
  • bun run build
  • bun run validate

Adversarial QA/security review was run read-only after the first green pass; findings around artifact URI allowlisting, newest malformed reports, and fixture/seed parity were fixed and covered by regression tests.

Co-authored-by: OpenAI Codex (GPT-5) codex@openai.com

Project validation report metadata into run records and render the run-detail validation gate summary with safe raw artifact links.

Co-authored-by: OpenAI Codex (GPT-5) <codex@openai.com>
@vercel

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
loopworks Ready Ready Preview, Comment Jul 9, 2026 2:48am

Request Review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ncolesummers ncolesummers marked this pull request as ready for review July 9, 2026 03:24
@ncolesummers ncolesummers merged commit 5ab30b6 into main Jul 9, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Loopworks Backlog Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Validation gate summary UI (pass/fail/skipped)

3 participants