Skip to content

feat: add detected models and second-model review - #186

Merged
jeremymcs merged 1 commit into
mainfrom
jeremymcs/more-models-and-review
Aug 30, 2026
Merged

feat: add detected models and second-model review#186
jeremymcs merged 1 commit into
mainfrom
jeremymcs/more-models-and-review

Conversation

@jeremymcs

Copy link
Copy Markdown
Owner

Summary

  • discover model choices from installed Codex and Claude CLIs while preserving saved selections
  • add an opt-in second-model review selector and run the chosen reviewer before commit/push
  • persist review settings and cover discovery, configuration, storage, UI wiring, and babysitter orchestration

Verification

  • npm run test:all — 836 passed
  • npm run check
  • npm run lint
  • npm run build
  • npm run build:public-docs
  • git diff --check

Critical-path coverage: focused regression tests cover reviewer selection, model propagation, failure handling, and merge-conflict review-before-commit ordering.

Notes

  • The repository-required Claude /simplify command was attempted twice with Opus, but stalled without output and was terminated; it made no worktree changes.
  • Screenshot not attached: this environment did not produce an uploadable browser artifact for the Settings control.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Documentation Updated

This PR includes documentation changes:

docs/public/agent-dispatch.md
docs/public/configuration.md

The docs build succeeded. Changes will be deployed to GitHub Pages when merged to main.

@jeremymcs
jeremymcs merged commit 2b3603d into main Aug 30, 2026
8 checks passed
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