Repository navigation
[pr-attention-pulse] Add scoped PR attention dashboards - #69298
Merged
Merged
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
One or more issues must be addressed before approval.
Get a fresh assessment by requesting another Copilot review.
Review tier: Lite
Findings: 4
Open findings (4)
What changed in this PR
Adds a fork-only, manually dispatched PR attention dashboard using deterministic collection, sanitization, rendering, validation, and issue publication.
Changes:
- Adds the workflow and generated lock file.
- Adds trusted sanitizer, renderer, and publication validator.
- Adds comprehensive tests and fixtures.
| File | Description |
|---|---|
| .github/workflows/pr-attention-pulse/Validate-PRAttentionPulseOutput.ps1 | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse/Sanitize-PRAttentionPulse.ps1 | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse/Render-PRAttentionPulse.ps1 | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse/PRAttentionPulseContract.psm1 | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse/Compile-PRAttentionPulse.ps1 | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse.md | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/Test-PulsePublicationCommand.cjs | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/Test-PulseIssueUpdate.cjs | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/Test-PRAttentionPulse.ps1 | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/wrong-repository.json | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/presentation/published-34643961191.pulse.json | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/presentation/published-34643961191.expected.txt | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/presentation/provenance.json | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/presentation/normal-legacy.expected.txt | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/presentation/incomplete-query.expected.txt | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/presentation/complete-zero.expected.txt | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/presentation/bounded-empty-verification.expected.txt | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/normal-legacy.json | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/malformed.json | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/incomplete.json | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/incompatible.json | Updated as part of this pull request. |
| .github/workflows/pr-attention-pulse-tests/fixtures/complete-zero.json | Updated as part of this pull request. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
maraf
approved these changes
Sep 16, 2026
kotlarmilos
approved these changes
Sep 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Add a deterministic PR attention dashboard workflow with a maintained Blazor scope and a repository-wide baseline.
Description
This adds PR Attention Pulse after its
.github/skills/pr-attention-queuedependency was merged in #69199. It demonstrates collapsible per-area composition without claiming a final multi-area taxonomy:blazorskill preset: the defined label/path union, including the incidental-path floor. Its dashboard starts collapsed with<details>.Within both areas, candidate presentation remains deterministic: an exact sanitized
community-contributionreason adds a fixed Community marker beside the author, rows display days open without claiming inactivity, and collapsed summaries distinguish displayed (shown) counts from the full inventory and coverage audit.The deterministic publication architecture is:
dotnet/aspnetcoreand verifies that permanent dashboard [pr-attention-pulse] ASP.NET Core PR Attention Pulse #69328 is an open issue (not a pull request) whose title starts with[pr-attention-pulse].dotnet/aspnetcore: exact-Preset blazor -DisablePersonalInbox -OutputFormat Jsonand exact-AllRepo -DisablePersonalInbox -OutputFormat Jsoninvocations.[dotnet/aspnetcore#N](https://github.com/dotnet/aspnetcore/pull/N).update-issuehandler, trusted request serializer, and private validator all use immutable issue number69328. The safe-output job repeats the repository/type/state/title validation immediately before its single body-only update.No model-authored ranking, prose, URL, target, or interpretation is accepted. All GitHub text is untrusted. The contract preserves candidate order/counts, escapes table cells, enforces the size limit, and rejects malformed or duplicate links, mismatched labels and targets, arbitrary URLs, wrong repositories, issue paths, query/fragment suffixes, candidate-supplied URL text, changed issue targets, and malformed area/status metadata.
Security boundaries
The generated gh-aw v0.88.7 runtime expands the source
bash: ["cat"]declaration to its required baseline shell surface. This PR does not claim that the effective shell is cat-only or that only three physical files exist in the runtime.The enforceable boundary is data minimization:
/tmp/gh-aw/baseand repository-derived/tmp/gh-aw/.github/agentsand/tmp/gh-aw/.github/skills, then asserts they are absent..pr-attention-pulse.pulse-input.json,pulse-body.md, andpulse-request.json. Workspace.github,.git, raw queue data, and every unexpected root or task-directory entry fail closed.GH_TOKEN,GH_AW_GITHUB_TOKEN,GITHUB_MCP_SERVER_TOKEN,GITHUB_TOKEN,OTEL_EXPORTER_OTLP_HEADERS, andGH_AW_OTLP_ENDPOINTS. The v0.88.7 compatibility adapter uses only a deterministic non-secret job output; tests assert no sentinel is exported and no secret-bearing workflow value is copied into model environment.Setup and execution
The permanent dashboard already exists at #69328. Merging this workflow does not run it: the workflow remains manual-only and must be explicitly dispatched in
dotnet/aspnetcore. The repository guard prevents a fork run from reaching inference or publication with the same issue number. This PR does not close #69328.Candidate verification
Latest follow-up commit
bc490da6eac3ef56c26d409f11d559be72266fdabuilds on456a5e87d407d9b140fb9b7d5e6dbb3621267c20and changes only candidate presentation and its focused tests/goldens.cb566918bd65cbdeab39c015a882ee229552853ca876446057ad266a8fbdcff1). The updated local preview preserves the exact ordinal sequence of all 26 clickable PR references, both collapsed scopes, six sections per scope, candidate membership/order/rank/title/next actor/reasons/blockers/discussion/census/overflow/caps, and changes only the approved marker, age, and summary presentation..github/skills/pr-attention-queue/tests/Test-PRAttentionQueue.ps1suite passed..github/aw/actions-lock.jsonpruning side effect was restored byte-for-byte fromHEAD; it is not in the commit.git diff --checkandgit diff --cached --checkpassed. The follow-up changes exactly 6 Pulse presentation/test files and no workflow policy, skill, collector, sanitizer, schema, package,global.json,NuGet.config, shared registry, or unrelated file.Hosted proof
The original imported source SHA completed successfully on the fork in run 34863123658, attempt 1, and published to PureWeen/aspnetcore#58.
723d8601; all required jobs succeeded.b1f310822a0b2005cb8907e0c51c7e349e76249c50f50634754cc9408554a828.gpt-5.6-sol; steering was false, no actual fallback occurred, and total usage was 22.02204 AI credits.That run is historical proof for the original revision. The current upstream-only fixed-target revision has not been dispatched, and this PR intentionally does not close or mutate either dashboard issue.