Skip to content

[codex] wire omnigent desktop provider - #19

Merged
jasonkneen merged 1 commit into
mainfrom
polly/desktop-omnigent-wiring
Jul 1, 2026
Merged

jasonkneen merged 1 commit into
mainfrom
polly/desktop-omnigent-wiring

Conversation

@jasonkneen

Copy link
Copy Markdown
Owner

Summary

  • Add Omnigent to the desktop built-in chat provider catalog and picker with the daemon-compatible omnigent:default agent option.
  • Route Omnigent chat turns through daemon /chat/job/* only, including a guard so it cannot fall through to the in-process runtime path.
  • Add AppSettings.omnigent defaults matching the daemon settings shape and forward that object in /chat/job/start requests.
  • Add focused desktop wiring coverage for the picker/settings/pass-through contract.

Validation

  • node --test test/model-filter.test.ts test/hermes-provider-models.test.ts test/omnigent-desktop-wiring.test.ts passes.
  • npm run build passes.
  • npm run typecheck currently fails on unrelated baseline issues: packages/codesurf-daemon/src/client.ts unused DaemonPersona, missing nested vitest types for packages/contex-relay, and existing renderer unused/type errors in App.tsx, AppCanvasConnections.tsx, useAppSessionOrchestration.ts, useAppWorkspaceOrchestration.ts, and useTitleTooltips.ts.

Deferred

  • I did not add a full Omnigent settings editor; the visible Settings > Providers panel is for generation providers, not chat providers. The required type/defaults and daemon pass-through are included.

@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8f0f4935-e1c2-41f7-bf4f-1dd25f9628a0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch polly/desktop-omnigent-wiring

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jasonkneen
jasonkneen merged commit 6807973 into main Jul 1, 2026
1 of 2 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