Add ASP.NET Core PR attention canvas - #69063
Draft
PureWeen wants to merge 7 commits into
Draft
Conversation
This was referenced Sep 4, 2026
PureWeen
force-pushed
the
pureween-pr-attention-canvas
branch
4 times, most recently
from
September 5, 2026 17:56
5829374 to
dda8fa4
Compare
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
force-pushed
the
pureween-pr-attention-canvas
branch
from
September 5, 2026 17:59
dda8fa4 to
2eddd4e
Compare
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.
Summary
digestRankrather than deriving a second order in the UI1.0.0producersDiscussion-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
textContentand 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
Validation
pwsh .github/skills/pr-attention-queue/tests/Test-PRAttentionQueue.ps1passes strengthened fixture coverage for author disposition without formal review, polite actionable owner feedback, initial owner objections, incomplete data, current unresolved threads, and resolved/outdated thread controls.node --test .github/extensions/aspnetcore-team-app/*.test.mjspasses all 24 tests.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 tomainafter 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.