Skip to content

feat(web): add synchronized browser reviews - #707

Open
benvinegar wants to merge 20 commits into
mainfrom
feat/browser-review
Open

feat(web): add synchronized browser reviews#707
benvinegar wants to merge 20 commits into
mainfrom
feat/browser-review

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Summary

  • add a browser-native, continuous multi-file review surface backed by the same authoritative runtime, document, state, notes, actions, reloads, and resources as the terminal UI
  • render browser diffs and the navigation tree with Pierre, including synchronized selection, filtering, context expansion, note editing, file search, large-review windowing, and watch/reload behavior
  • expose capability-authenticated loopback reviews through --web, --no-open, and hunk session open, with explicit browser-only Tailscale access through --tailscale
  • embed offline browser assets and harden the broker with strict Host/Origin validation, CSP, bounded resources/SSE, immutable generations, backpressure, and daemon compatibility refreshes

Validation

  • bun run typecheck
  • bun run lint
  • bun run format:check
  • bun test — 2,185 passed, 12 skipped
  • bun run website:test:browser — 62 passed, 2 compiled-only skips
  • PTY, TTY, compiled-binary, offline-asset, package, resource/cache memory, and real terminal/browser/session synchronization checks
  • real 185-file Tailscale review in desktop and mobile Chromium with no console, page, resource, HTTP, or horizontal-overflow errors
  • independent correctness and security review gates

Release

Includes a minor changeset for hunkdiff.

This PR description was generated by Pi using gpt-5.6-sol

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
hunk-web Ignored Ignored Preview Aug 11, 2026 5:00pm

Request Review

# Conflicts:
#	docs/extensions.md
#	docs/keybindings.md
#	scripts/build-prebuilt-artifact.test.ts
#	src/app/startup.ts
#	src/ui/App.tsx
#	src/ui/hooks/useReviewController.test.tsx
#	src/ui/lib/appCommands.test.ts
@socket-security

socket-security Bot commented Aug 10, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​types/​react-dom@​19.2.31001007588100
Added@​pierre/​trees@​1.0.0-beta.6791009995100

View full report

@benvinegar
benvinegar marked this pull request as ready for review August 11, 2026 13:33
@greptile-apps

greptile-apps Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

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

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

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