Skip to content

[dashboard-language] Built-in pages: validate DLS-PAGE-014 data-state exposure - #130

Merged
pelikhan merged 2 commits into
mainfrom
dashboard-builtin-page-014-data-state-566f197836a32c4d
Aug 29, 2026
Merged

[dashboard-language] Built-in pages: validate DLS-PAGE-014 data-state exposure#130
pelikhan merged 2 commits into
mainfrom
dashboard-builtin-page-014-data-state-566f197836a32c4d

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 — DLS-PAGE-014 conservative built-in data-state exposure validation.

Specification coverage

  • Section 10 Built-in Pages
  • Requirement IDs: DLS-PAGE-014

What shipped

  • Added conservative implementation-local built-in page definition support for definition.data-state.
  • Validated that built-in page definitions declare independent availability, completeness, and freshness exposure markers.
  • Rejected unknown built-in data-state axes and non-canonical marker values.
  • Expanded validator unit coverage for accepted and rejected DLS-PAGE-014 cases.
  • Updated PLAN.md and README.md to record the slice and the remaining ambiguity in the specification vocabulary.

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

  • End-to-end browser tests remain blocked by missing Playwright browser provisioning in the runner.

Next milestone

Built-in pages — next conservative Section 10 slice for runs page link coverage without inventing presenter semantics.

Generated by 🧩 Daily Dashboard Language Renderer · pi · gpt54 · 59.2 AIC · ⌖ 7.5 AIC · ⊞ 5.4K ·

@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 00:09
@pelikhan

Copy link
Copy Markdown
Contributor

@copilot update agentic workflow that generated this bug to allow updates to README.md and PLAN.md

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

@copilot update agentic workflow that generated this bug to allow updates to README.md and PLAN.md

Updated in 05bd57e. The renderer now explicitly permits pages/dashboard/README.md and pages/dashboard/PLAN.md; its compiled lock and contract test were updated.

Copilot AI requested a review from pelikhan August 29, 2026 00:15
@pelikhan
pelikhan merged commit 40eea63 into main Aug 29, 2026
2 checks passed
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.

2 participants