Skip to content

Remove client-side auto-review from app package #204

Closed

Description

@OpenCodeEngineer

Track removal of all client-side auto-review traces from packages/app only.

Scope:

  • Delete src/pages/session/auto-review.ts and test
  • Remove all auto-review logic/fields/effects from src/pages/session.tsx
  • Remove autoReview from settings context and UI
  • Remove i18n strings and prompt-input toggle
  • Clean imports/dead code
  • Validate with bun typecheck in packages/app and packages/opencode

Do not modify server implementation under packages/opencode beyond verification.

Activity

  1. OpenCodeEngineer commented on May 15, 2026

    @OpenCodeEngineer
    Author

    Completed local implementation (no commit) to remove client-side auto-review from packages/app.

    Changes made:

    • Deleted src/pages/session/auto-review.ts and src/pages/session/auto-review.test.ts
    • Removed all auto-review imports/types/state/effects/references from src/pages/session.tsx while preserving general followup flow
    • Removed models.autoReview from src/context/settings.tsx (types/defaults/accessors/setter)
    • Removed auto-review ModelsSection from src/components/settings-general.tsx
    • Removed auto-review i18n strings (and orphaned models section key) from src/i18n/en.ts
    • Confirmed no auto-review toggle exists in src/components/prompt-input.tsx

    Validation:

    • cd packages/app && bun typecheck ✅
    • cd packages/opencode && bun typecheck ✅
  2. added
    tech-debtMaintenance or cleanup work
    uxUser experience or UI behavior
    on Jun 21, 2026
  3. dzianisv commented on Jun 21, 2026

    @dzianisv
    Owner

    The auto-review feature exists in current dev but the UX/direction is contested (#200 wants improvement, #203/#204 want removal). This is a product decision, not a code fix. Closing as deferred — needs product owner call on whether auto-review stays or goes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    priority:lowLow prioritytech-debtMaintenance or cleanup workuxUser experience or UI behavior

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions