feat: recover auto-review feature (supervisor + cross-review)#2
Closed
OpenCodeEngineer wants to merge 1691 commits into
Closed
feat: recover auto-review feature (supervisor + cross-review)#2OpenCodeEngineer wants to merge 1691 commits into
OpenCodeEngineer wants to merge 1691 commits into
Conversation
…anifest assets (anomalyco#25698) Co-authored-by: Kit Langton <kit.langton@gmail.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>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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>
|
This pull request has been automatically closed. |
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Recovers the auto-review feature that was lost during the
upstream/devrebase. This restores the full supervisor → summarize → cross-review orchestration loop.Changes
Backend (
packages/opencode/src/config/config.ts)auto_reviewconfig field with optionalmodel(provider/model format) using Effect SchemaFrontend Settings (
packages/app/src/context/settings.tsx)modelssection:autoReviewtoggle,defaultModel,reviewModelselectorsSession Flow (
packages/app/src/pages/session.tsx)AssistantMessagetypeFollowupItemwithautoReviewSourceandautoReviewPhaseReviewModelandReviewStatetypespending,autoReview,reviewfields to followup persisted storeresolveReview(),queueAutoReview(), andisAutoReviewPrompt()helperscreateEffectthat orchestrates: supervisor review → summarize → cross-review (retry cap: 3 per phase)Documentation (
FORK.md)panic: status service not initialized. Call status.InitService() at application startup.when running opencode anomalyco/opencode#15settings.tsxandsession.tsxHow to verify
Typecheck
✅ All 14 packages pass
bun typecheck✅ All 10 auto-review tests pass
Closes anomalyco#194