Skip to content

feat(settings): add generic OpenAI provider#154

Merged
buremba merged 1 commit into
lobu-ai:mainfrom
hsinchao-athena:add-openai-provider
Mar 23, 2026
Merged

feat(settings): add generic OpenAI provider#154
buremba merged 1 commit into
lobu-ai:mainfrom
hsinchao-athena:add-openai-provider

Conversation

@hsinchao-athena
Copy link
Copy Markdown
Contributor

Description

Addresses the worker crash: Failed to extract accountId from token when using standard OpenAI API keys with the default "ChatGPT" platform connection.

Currently, the "ChatGPT" integration (chatgpt provider ID) is hardcoded to use the openai-codex engine from @mariozechner/pi-ai, which explicitly expects OAuth JWT Device-Code flow tokens. Supplying standard API keys (sk-...) causes a parsing error when the pi-ai library tries to extract the accountId.

This PR appends a new generic OpenAI provider configuration block to config/system-skills.json. This registers a distinct "OpenAI" connection option in the dashboard that safely routes to the standard openai provider slug in pi-ai ("sdkCompat": "openai"), bypassing the JWT decoding.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Refactoring

Testing

  • Ran bun test and all tests pass
  • Ran bun run format and bun run lint
  • Verified changes work in Docker Compose (make dev)

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

Related Issues

Closes #(issue number)

Additional Notes

The system-skills.json patch adds the new generic provider while preserving all existing integrations.
(Note: bun run lint surfaced a few pre-existing a11y and key index warnings in packages/landing/... which are unrelated to this JSON patch).

@hsinchao-athena hsinchao-athena marked this pull request as draft March 6, 2026 17:15
@hsinchao-athena hsinchao-athena marked this pull request as ready for review March 6, 2026 17:25
@buremba buremba merged commit fcae8c3 into lobu-ai:main Mar 23, 2026
@buremba
Copy link
Copy Markdown
Member

buremba commented Mar 23, 2026

Thanks for the contribution! This was a missing piece — we had no way for users to configure standard OpenAI API keys through the settings UI. Merged!

buremba added a commit that referenced this pull request May 17, 2026
Includes:
- #150 PATH-based agent detection (replaces MCP session inference)
- #152 auto-pick default agent when defaultAgent is unset
- #155 distinct SF Symbol icons per CLI agent kind
- #154 WATCHERS section in menubar with last-fired + manual trigger
buremba added a commit that referenced this pull request May 17, 2026
Pulls in #823 (revert goals primitive), #824 (auto-provision default agent
+ manual trigger), #825 (owletto submodule bump to #154). Submodule
merged origin/main into our feat/chrome-debugger-executor branch
separately; bumps the parent's submodule pointer to the merge SHA so
both the executor work and main's recent owletto changes (Mac app
LobuClient + connection-settings UI rework + drop default-form-layout)
ship together.
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.

2 participants