Skip to content

feat(extensions): show delegated review metadata - #986

Merged
benvinegar merged 6 commits into
mainfrom
feat/delegated-review-metadata
Sep 5, 2026
Merged

feat(extensions): show delegated review metadata#986
benvinegar merged 6 commits into
mainfrom
feat/delegated-review-metadata

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Summary

  • add bounded, provider-neutral review descriptors to extension CLI patch delegation
  • preserve delegated metadata across same-source refreshes and clear it for unrelated reloads
  • expose review metadata to extension panes and session/broker projections
  • add a bundled two-row hunk:review-info pane above the review stream for change requests
  • update the GitHub PR example to fetch and display PR metadata through the public API

Architecture

The descriptor stays in the session/bootstrap envelope rather than ReviewDocumentV1 or changeset transforms. Validation is shared by extension delegation and browser-safe protocol parsing. The bundled pane registers through the same public pane API as third-party extensions.

Validation

  • bun run typecheck
  • full unit suite: 2,099 passed
  • PTY integration: 142 passed
  • session tests: 13 passed
  • review conformance: 111 passed
  • bun run deps:check
  • source-boundary checks
  • formatting and package-consumer checks

Follow-up

After API v17 ships, modem-dev/hunk-gh can emit these descriptors for installed GitHub PR reviews.

This PR description was generated by Pi using GPT-5.6 Sol

@greptile-apps

greptile-apps Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Too many files changed for review (51 files, 50 file limit).

Bypass the limit by tagging @greptile-apps to review.

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
hunk-web Ignored Ignored Preview Sep 5, 2026 1:04am UTC

Request Review

@benvinegar
benvinegar merged commit 7d27181 into main Sep 5, 2026
12 checks passed
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