feat: recover auto-review feature (supervisor + cross-review) by OpenCodeEngineer · Pull Request #3 · OpenCodeEngineer/opencode

OpenCodeEngineer · 2026-05-12T03:34:05Z

Issue for this PR

Type of change

Bug fix
New feature
Refactor / code improvement
Documentation

What does this PR do?

Recovers the auto-review feature that was lost during the upstream/dev rebase. The feature enables automatic supervisor and cross-review followups after an assistant completes a coding task.

The auto-review flow works as follows:

When enabled in Settings → Models, after the assistant completes a response, a supervisor review is automatically queued using a different model
After the supervisor review completes, the session is summarized, then a cross-review is queued with yet another model
The flow stops when the reviewer outputs "Task completed." or after 3 retries per phase

Backend: Added auto_review config field (Effect Schema) with optional model in provider/model format.

Settings: Added models section with autoReview toggle, defaultModel, and reviewModel selectors.

Session: Added ~200 lines of orchestration: resolveReview() picks a review model different from the one that produced the code, queueAutoReview() queues the followup with the right prompt, and a createEffect watches for completed assistant messages to trigger the flow.

Docs: Updated FORK.md with feature entry anomalyco#14 and rebase conflict zone entries.

How did you verify your code works?

All 14 packages pass bun typecheck (tsgo --noEmit)
All 10 existing auto-review unit tests pass (bun test src/pages/session/auto-review.test.ts)
The auto-review module (auto-review.ts) was already present and unchanged — this PR wires it into the session page and settings

Screenshots / recordings

N/A — this is a backend + state management change. The UI toggle appears in Settings → Models.

Checklist

I have tested my changes locally
I have not included unrelated changes in this PR

…anifest assets (anomalyco#25698) Co-authored-by: Kit Langton <kit.langton@gmail.com>

Port cleanly-compatible fork features onto upstream/dev base: App UI: - Saved prompt suggestions component - Recent sessions sidebar with hierarchy view - Auto-review model selection utility - Recent session organization utility Backend: - Edge TTS synthesis module (node-edge-tts) - TTS HTTP route (/tts/edge) - Session auto-resume module Import paths updated for upstream (@opencode-ai/core/util/). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- encode recent sidebar session slugs with workspace directory - trim/validate TTS input and reject whitespace-only payloads - route /tts through HttpApi backend in server handler - clean up generated TTS temp directories recursively - add whitespace route coverage test - update handoff.md progress Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- add serve-autoresume effect to recover interrupted/unanswered sessions - run per-session resume inside InstanceStore context - provide WorkspaceRef when present and skip busy sessions - start worker in serve command via background scoped fiber - update handoff.md with current state and next steps Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- move /tts/edge into Effect HttpApi global routes + handlers - remove createHttpApi Hono bypass that caused OpenAPI parity drift - exclude /tts/* from injected instance query params in public OpenAPI transform - update handoff.md with regression/fix progress Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- log TTS HttpApi parity fix commit and test outcomes - mark auto-review todo complete with app test coverage - record remaining full-suite failures for next triage Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- note full-suite failures confirmed on upstream/dev as environment-related - document models snapshot artifact and safe.bareRepository test impacts - mark failure triage complete in handoff plan Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…olding in sidebar Restore features lost during rebase on upstream/dev: - Recently Used models group in model picker dialog - Project/workspace prefix labels in recent sessions sidebar - Collapsible parent/child session tree with depth indentation - Session hover preview cards with MessageNav - workspaceKey helper replacing pathKey for directory normalization - childMapByParent helper for building session parent-child maps Also adds FORK.md documenting fork-specific features and rebase survival checklist to prevent future feature loss. Fixes anomalyco#189 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…t sidebar - Add /recent route to router (before /:dir catch-all) - Navigate to /recent when clicking Recent Sessions tile - Fix recent sidebar slug to use base64-encoded directory instead of literal 'recent' - Fix relative navigate path in message hover selection (missing leading /) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add browser smoke test table, deployment quick reference, and document /recent route, TTS, auto-resume, multi-instance features. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…chas Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Add models section to settings context (autoReview, defaultModel, reviewModel) - Import auto-review module functions and AssistantMessage type - Extend FollowupItem with autoReviewSource and autoReviewPhase fields - Add ReviewModel and ReviewState types - Add pending, autoReview, review fields to followup persisted store - Add auto-review helpers: resolveReview, queueAutoReview, isAutoReviewPrompt - Add createEffect for supervisor/cross-review orchestration loop Closes anomalyco#196 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

github-actions · 2026-05-12T03:34:18Z

This pull request has been automatically closed.

opencode-agent Bot and others added 30 commits May 3, 2026 18:24

chore: generate

28112fb

Add v2 session failure events (anomalyco#25628)

7749d8e

chore: generate

a9dc0fa

fix(server): serve embedded UI from bunfs (anomalyco#25632)

6312c55

chore: generate

755cd56

refactor(cli): effectify provider commands (anomalyco#25633)

825ab2e

fix(app): preserve auth token credentials (anomalyco#25636)

ca6150d

Effectify plugin agent regression test (anomalyco#25646)

c2b1974

fix: allow Codex Spark with Codex OAuth (anomalyco#25640)

ce89bcb

feat(server): pty websocket auth tickets (anomalyco#25660)

7bc26da

chore: generate

9f708e7

fix(app): prevent terminal recovery loops (anomalyco#25710)

a366128

chore: generate

67047fa

fix(opencode): strip transfer-encoding in UI proxy and allow public m…

1251a87

…anifest assets (anomalyco#25698) Co-authored-by: Kit Langton <kit.langton@gmail.com>

test(server): regression reproducers for anomalyco#25698 (anomalyco#2…

6e9f10a

…5714)

chore: generate

2c819f2

fix: ensure anthropic sdk properly resolves when using azure (anomaly…

c1f607d

…co#25721)

sync

1aed6b1

chore(docs): rename firmware provider to frogbot (anomalyco#25453)

b70e270

fix(worktree): fork workspace worktree boot (anomalyco#25723)

25dc6f0

fix(server): provide fresh ConfigProvider per HttpApi listener (anoma…

fb07c20

…lyco#25726)

test(agent): skip InstanceBootstrap in plugin-agent regression test (a…

007b57f

…nomalyco#25737)

sync

5720883

fix: ensure effect server middleware properly parses errors (anomalyc…

d431a0e

…o#25717)

sync release versions for v1.14.34

4b65b1e

fix(vcs): preserve batched patch boundaries (anomalyco#25787)

6a5e329

chore: generate

f14784d

sync release versions for v1.14.35

6b85277

Run UI unit tests in CI (anomalyco#25792)

ca2411d

update: normalize download asset names to match new naming convention (…

84afd2b

…anomalyco#25796)

opencode-agent Bot and others added 28 commits May 9, 2026 04:31

chore: generate

b2baddc

feat: port autopilot loop and exit tool

677902b

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: port infra memory retention hardening

a2de23d

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

feat: port scheduler cron heartbeat runtime

a52a6ed

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs: update handoff with infra and scheduler status

d24eee6

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: stabilize post-rebase runtime regressions

333a6e0

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs: record rebase publish and VM deploy status

132ef1e

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: keep orphan sessions visible in recents

6090ef1

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: restore recent rail in sidebar layout

6cbfb9e

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs(fork): add comprehensive post-rebase verification checklist

a2e9c40

Add browser smoke test table, deployment quick reference, and document /recent route, TTS, auto-resume, multi-instance features. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs(fork): add session auto-title/rename to rebase checklist

292fc5d

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs(fork): complete deployment reference with service config and got…

9cd6ece

…chas Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: restore voice, rename tool, and recent folding regressions

418c0c2

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs(fork): refresh regression checklist and remaining backup-only gaps

2d99e1f

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix(opencode): port memory leak hardening for queue, mcp, and bus

5289d75

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs(fork): mark memory leak hardening as recovered

4d7e377

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

fix: restore local install workflow and rebase guardrails

8a8e8c1

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add auto_review config schema

e436846

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

docs(fork): add auto-review feature entry and rebase checklist

00e19a4

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

github-actions Bot closed this May 12, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: recover auto-review feature (supervisor + cross-review)#3

feat: recover auto-review feature (supervisor + cross-review)#3
OpenCodeEngineer wants to merge 1691 commits into
devfrom
fix/recover-auto-review

OpenCodeEngineer commented May 12, 2026

Uh oh!

github-actions Bot commented May 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Conversation

OpenCodeEngineer commented May 12, 2026

Issue for this PR

Type of change

What does this PR do?

How did you verify your code works?

Screenshots / recordings

Checklist

Uh oh!

github-actions Bot commented May 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants