Skip to content

feat(deck): copy a section's PRs as a Markdown list from its header - #49

Merged
mthines merged 6 commits into
mainfrom
feat/copy-section-pr-list
Oct 19, 2026
Merged

mthines merged 6 commits into
mainfrom
feat/copy-section-pr-list

Conversation

@mthines

@mthines mthines commented Oct 19, 2026

Copy link
Copy Markdown
Owner

Why

To hand a group of PRs to an agent ("which of these are ready to merge?") you currently copy them one by one. This makes the whole group one click.

What changed

  • Hovering a deck section header (Needs attention, Waiting, Pinned, …) or an Inbox role header (Your PRs / Needs your review) shows a copy button; right-click offers Copy PR List (N).
  • New pure PRListExport builds the list: one [repo#n](url) title line per PR plus a status line (author, draft, CI, review, conflict, unresolved threads, size, branch → base). Stack members are included even when collapsed.
  • Hover state lives in a per-header HoverReveal, so hovering never re-renders the deck.
  • New copy_pr_list triage telemetry value, documented in docs/telemetry.md.

How to verify

Build Debug, hover Needs attention, click the icon → toast "Copied N PRs: …"; paste. PRListExportChecks.run() asserts the format at launch.

@mthines

mthines commented Oct 19, 2026 •

Copy link
Copy Markdown
Owner Author

✅ No issues found

Both prior suggestions are fixed: role-header copy now follows on-screen order (cd987f0) and docs/keyboard.md documents the feature (0f762e7). The Debug build passes.

Checked: 9 of 9 changed files read · 2 possible issues → 0 confirmed → 0 posted

Progress: open review threads 0 → 0 across the last 2 reviews

Review details
Gate Status Details
Description vs. code ✅ Description matches the code: hover/right-click copy, PRListExport format, HoverReveal, telemetry value
Prior review feedback ✅ Earlier review comments are resolved.
Documentation ✅ docs/keyboard.md, README, CLAUDE.md, and docs/telemetry.md document the feature and the copy_pr_list value
Self-review signals ✅ No debug logs, leftover TODOs, or unreviewed stubs.
Code review ✅ The multi-lens review found no blocking issues.

Found

Quality — produced 2 → posted inline 0 · cleared 0 · carried forward 0 · deferred 0 · below-bar 0
Dropped — 2 below-bar
Standards — ran — CLAUDE.md (48 bullets): pbxproj wiring (3 places), DEBUG self-check wired in AppDelegate, telemetry value bounded and documented; no findings
Measurability — ran — copy_pr_list triage interaction recorded via TelemetryService.recordTriageInteraction; no findings

Run

full · 12 lines in delta · tier deep · depth checkout
Integrations — none
CI — still pending: test · 1 passed
Memories — 1 indexed · 1 used (1 rule)

  • rule reviewer-comment-relevance::nitpick:doc-overstates-menu-gating-scope

Nothing to report — severity, reach diagram (impact graph found no consumer or importer), 0 files skipped.

pr-reviewer · commit 0f762e7 · full review · how these findings are produced · updated Oct 19, 2026 2:36pm UTC

@mthines
mthines marked this pull request as ready for review October 19, 2026 14:36
@mthines
mthines merged commit 9600c8d into main Oct 19, 2026
4 checks passed
@dash0-dev

dash0-dev Bot commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

Darkplane Automerge is enabled for this repository (mode: Auto-approve).

Trigger an evaluation for this pull request when it is ready:

Run evaluation

Evaluations on this repository start manually. To have Darkplane Automerge evaluate each pull request on its own, set Trigger to Automatic in the repository's configuration.

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.

1 participant