Walkthrough

Review at a glance

Full review workflow now includes controls, suggestions, and conversation history.

Summary

Seeded server review summary.

Review status

Mock review engine completed a tuned review using repository defaults.

Mock review engine

Review v1

Effort

3/5

Changed files summary

  • - Adds sequence diagram rendering and fallback logic.
  • - Includes issue-linked context in walkthrough output.
  • - Provides fix suggestions for safety and maintainability.

Poem

Threads hum into form / intent sharpens into patches / review keeps us true.

Open Related Issue

Sequence diagram

Generated Mermaid output is rendered as a sequence walkthrough card.

User
->>
PromptRequest
submit
PromptRequest
->>
ReviewEngine
review
ReviewEngine
-->>
User
findings
sequenceDiagram
User->>PromptRequest: submit
PromptRequest->>ReviewEngine: review
ReviewEngine-->>User: findings

Review categories and rules

Phase 5 adds categorization, path-aware configuration, and repository guidance ingestion.

Categories

Bug risk - 1

Correctness and runtime safety findings.

Security - 1

Sanitization and injection findings.

Applied rules

path-filter:dist/*guideline:Prefer hydration-safe routes
- path-filter (dist/*): Skip generated files during review.
- guideline: Prefer hydration-safe flows in route components.

Issue assessment & status feed

Phase 6 links issue readiness, review progress, and local-review handoff in one surface.

Linked issue assessment

Funding and issue scope are aligned with this Prompt Request, so the change is ready for a first-pass review.

readyScore 4/5

Review status feed

Review v1 queued

Collected issue context and changed paths.

Diff review complete

Generated findings and routing hints.

Suggested labels and reviewers

Phase 6 turns routing hints into a first-class review routing panel.

Labels

review:aiarea:prompts

Reviewers

- arch-review
- security-review

Related work

Let issue votes fund Prompt Requests directly

issue - linked

Local review endpoint

CLI-compatible handoff keeps the web review and local review in sync.

bun run local-review --prompt "Add sequence diagrams to review walkthroughs" --format markdown
{
  "promptTitle": "Add sequence diagrams to review walkthroughs",
  "provider": "mock",
  "reviewVersion": 1
}
Open Local Review Surface

Inline findings

Findings support one-click resolve and keep severity metadata for prioritization.

bug-riskbug-riskProtect walkthrough fallback renderingsrc/reviews/renderWalkthrough.ts:41

Fallback handling should protect walkthrough render when Mermaid parse fails.

Suggested fix: Wrap parse/render in try/catch and render raw text on failure.
try { renderMermaid(diagram) } catch { renderFallback(diagram) }
confidence: high
securitysecurityEscape markdown interpolationsrc/server/review/report.ts:67

Escape markdown interpolation for user-controlled fields.

Suggested fix: Sanitize labels before rendering markdown and diagrams.
const safeLabel = escapeMarkdown(label)
confidence: high

Review chat

Ask AI questions inline; answers are persisted as review conversation messages.

ai

Initial review complete. Focus on fallback rendering and markdown sanitization.

now