Skip to content

feat(review): make interactive PR design reproducible - #464

Merged
christso merged 4 commits into
mainfrom
fix/pr-interactive-review-reproducible-design
Sep 7, 2026
Merged

feat(review): make interactive PR design reproducible#464
christso merged 4 commits into
mainfrom
fix/pr-interactive-review-reproducible-design

Conversation

@christso

@christso christso commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Interactive PR reviews now reproduce the same editorial hierarchy for any pull request: intent and verdict first, bounded business context and an optional mental model next, then filterable findings. The design comes from a validated presentation contract instead of one-off HTML, while existing review workspaces and finding lifecycle state remain compatible.

Design

  • A bounded interactive-presentation.json sidecar carries review-specific editorial content into the shared renderer.
  • The shared shell uses a sticky desktop review map and a single-column mobile layout, with heavy requirements, evidence, code, and lifecycle history collapsed by default.
  • Presentation metadata survives same-commit regeneration, resets when the reviewed commit changes, and rejects malformed persisted or supplied values.
  • Search and status filters keep the finding rail aligned with visible targets. Keyboard users can skip directly to findings.

Validation

  • bun run test — 1,476 passed, 5 skipped, 0 failed.
  • bun run typecheck — passed.
  • bun run build — passed.
  • Browser-tested the generated review at desktop and 390×844 mobile sizes. Confirmed responsive layout, no document-level horizontal overflow, collapsed heavy details, filtered navigation targets, and the empty-results state.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 7, 2026

Copy link
Copy Markdown

Deploying allagents with  Cloudflare Pages  Cloudflare Pages

Latest commit: 988e53c
Status: ✅  Deploy successful!
Preview URL: https://4ba9b308.allagents.pages.dev
Branch Preview URL: https://fix-pr-interactive-review-re.allagents.pages.dev

View logs

@christso
christso merged commit 8cf4c78 into main Sep 7, 2026
6 checks passed
@christso
christso deleted the fix/pr-interactive-review-reproducible-design branch September 7, 2026 04:30
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