Skip to content

[dashboard-language] Built-in pages: runs render slice - #137

Merged
pelikhan merged 1 commit into
mainfrom
dashboard-built-in-runs-render-71326f0ddd9a7d6b
Aug 29, 2026
Merged

[dashboard-language] Built-in pages: runs render slice#137
pelikhan merged 1 commit into
mainfrom
dashboard-built-in-runs-render-71326f0ddd9a7d6b

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Caution

Protected files were modified in this change.
This pull request is in request_review mode and requires explicit human scrutiny before merge.

Protected files: README.md

Milestone

Built-in pages

Specification coverage

  • Section 10 Built-in Pages
  • DLS-PAGE-006
  • DLS-PAGE-014

What shipped

  • Added visible presenter rendering for the built-in runs page.
  • Rendered run status counts, run conclusion counts, downstream outcome counts, and a runs table with scope, rollout mode, engine, requested model, resolved model, started time, outcome count, and run links.
  • Reused the keyed-list DOM primitive for deterministic run-row reconciliation.
  • Added jsdom unit coverage for deterministic presenter output.
  • Replaced the browser smoke test with a Playwright scenario covering the rendered runs page.
  • Updated PLAN.md and README.md for the new slice.

Gates

  • npm install
  • npm run typecheck
  • npm run lint
  • npm test
  • npm run test:e2e ⚠️ blocked: Playwright Chromium executable is not provisioned in this environment (browserType.launch: Executable doesn't exist)

Blockers

  • Playwright package-level browser execution remains blocked by missing Chromium in the runner, so the end-to-end spec was added and exercised up to browser launch failure only.

Next milestone

Built-in pages: render one additional Section 10 built-in page from the declarative definitions, or extract the first reusable presentation component needed by that next rendering slice.

Generated by 🧩 Daily Dashboard Language Renderer · pi · gpt54 · 43.6 AIC · ⌖ 9.73 AIC · ⊞ 5.6K ·

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Protected files were modified in this pull request and require manual scrutiny before merge.

Please verify that each protected-file change is intentional, policy-compliant, and safe:

  • Protected files: README.md

@pelikhan
pelikhan marked this pull request as ready for review August 29, 2026 01:41
@pelikhan
pelikhan merged commit 6f5e130 into main Aug 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant