Skip to content

Port to omp 18.3 SDK (settings registry, namespaced AuthStorage, pi-tui moves) - #94

Merged
ddallabenetta merged 1 commit into
ddallabenetta:mainfrom
jr551:port-omp-18.3-upstream
Oct 14, 2026
Merged

ddallabenetta merged 1 commit into
ddallabenetta:mainfrom
jr551:port-omp-18.3-upstream

Conversation

@jr551

@jr551 jr551 commented Sep 26, 2026

Copy link
Copy Markdown

What

Ports omp-web to @oh-my-pi/* ~18.3.2. omp 18.2 replaced the path-keyed Settings API with typed registry handles, namespaced AuthStorage (credentials/keys/oauth), and moved theme/thinking/goal-tool modules to @oh-my-pi/pi-tui.

  • lib/omp-settings.ts is the single path→handle adapter so the rest of the codebase doesn't scatter registry calls.
  • Also guards NO_PROXY for loopback: Bun 1.4 proxies loopback requests, which broke local providers behind HTTP_PROXY.

Unblocks the stalled automation/omp-18.x draft PRs (#77–#93), which all fail typecheck/tests on 18.x; this branch passes tsc --noEmit and bun test (534 pass / 1 skipped) on 18.3.2.

Test

bun run typecheck clean, bun test 534/535, bun run lint clean on the branch.

Disclosure

AI-assisted port (Claude Opus 5.5) authored and verified in a live deployment; the same port already ships in the fork's 0.5.x releases.

Co-Authored-By: Claude Opus 5.5 (1M context) noreply@anthropic.com

…, pi-tui moves)

omp 18.2 replaced the path-keyed Settings API with typed registry handles,
namespaced AuthStorage (credentials/keys/oauth) and moved theme, thinking and
goal-tool modules to @oh-my-pi/pi-tui. lib/omp-settings.ts is the single
path->handle adapter.

Also guards NO_PROXY for loopback: Bun 1.4 proxies loopback requests, which
broke local providers behind HTTP_PROXY.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@socket-security

Copy link
Copy Markdown

@ddallabenetta
ddallabenetta merged commit bb1ce3e into ddallabenetta:main Oct 14, 2026
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.

3 participants