Skip to content

Add ASP.NET Core PR attention canvas - #69063

Draft
PureWeen wants to merge 7 commits into
dotnet:pureween-pr-attention-queue-skillfrom
PureWeen:pureween-pr-attention-canvas
Draft

Add ASP.NET Core PR attention canvas#69063
PureWeen wants to merge 7 commits into
dotnet:pureween-pr-attention-queue-skillfrom
PureWeen:pureween-pr-attention-canvas

Conversation

@PureWeen

@PureWeen PureWeen commented Sep 4, 2026

Copy link
Copy Markdown
Member

Summary

  • adds the project-scoped ASP.NET Core Team App canvas backed by the deterministic PR Attention skill in Add pull request attention queue skill #69040
  • presents separate Review now and Needs rescue lanes, a compact Ready to merge strip, expandable secondary classifications, and a dedicated Verify discussion lane
  • renders visible items by the engine-provided digestRank rather than deriving a second order in the UI
  • supports explicit digest-only author exclusion while preserving the full matched census
  • validates additive JSON fields while remaining compatible with earlier 1.0.0 producers
  • provides only read-only Open PR, Review, and Investigate rescue actions through opaque server-owned item IDs, stale-safe snapshots, and same-origin loopback protection

Discussion-evidence boundary

The canvas does not reinterpret the deterministic classification or next actor. It displays bounded top-level discussion evidence and current thread state supplied by the skill when a leading raw ReviewNow candidate must be checked before ordinary review.

A Verify discussion card exposes only Open PR. Review actions are denied for verification-needed and not-assessed entries. It is used when the skill finds an explicit author close/continue disposition, actionable or unknown non-author discussion, incomplete data, or a current unresolved inline thread without collected thread-comment text. Resolved and outdated threads remain visible factual context but do not infer ownership. Comment excerpts render through DOM textContent and never reach action prompts.

Thus #67146 is held for its author disposition and #68037 for later non-author discussion, while clear candidates can remain in ordinary Review now. This is not an LLM judgment, an automatic bucket rewrite, or a new mutation surface.

Screenshot

ASP.NET Core Team App showing the live Blazor PR attention queue

Validation

Stack

This PR is stacked on #69040. Its temporary base branch mirrors the current #69040 head at 3e1700c7d9, so it remains a 13-file canvas-only diff. Review this after #69040 and retarget it to main after the base PR lands.

The source intentionally assesses only leading candidates and caps top-level comments and thread state at 50 per candidate. It does not perform full pagination, retrieve inline-thread comment text, or invoke a semantic model. Current unresolved inline feedback is routed to human verification instead of being called clear.

@PureWeen
PureWeen force-pushed the pureween-pr-attention-canvas branch 4 times, most recently from 5829374 to dda8fa4 Compare September 5, 2026 17:56
Copilot AI added 7 commits September 5, 2026 12:59
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>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@PureWeen
PureWeen force-pushed the pureween-pr-attention-canvas branch from dda8fa4 to 2eddd4e Compare September 5, 2026 17:59
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.

2 participants