Skip to content

feat(cli): add CLI tools for pi, omp, letta, codewhale and jcode - #6318

Merged
diegosouzapw merged 21 commits into
diegosouzapw:release/v3.8.47from
hamsa0x7:feat/cli-additions
Jul 10, 2026
Merged

feat(cli): add CLI tools for pi, omp, letta, codewhale and jcode#6318
diegosouzapw merged 21 commits into
diegosouzapw:release/v3.8.47from
hamsa0x7:feat/cli-additions

Conversation

@hamsa0x7

@hamsa0x7 hamsa0x7 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds detection and settings support for five new CLI tools: pi, omp, letta, codewhale, and jcode. Includes new UI components (ApiKeySelect, BaseUrlSelect, JcodeToolCard), settings routes with Zod validation, and provider icon assets.

Problem

The CLI tools system needed support for emerging agent frameworks and CLI applications. Previously, pi, omp, letta, codewhale, and jcode had no detection, settings routes, or UI integration.

Files changed

  • src/app/(dashboard)/dashboard/cli-code/components/ApiKeySelect.tsx — Reusable API key dropdown component
  • src/app/(dashboard)/dashboard/cli-code/components/BaseUrlSelect.tsx — Reusable base URL dropdown with tunnel/tailscale/cloud support
  • src/app/(dashboard)/dashboard/cli-code/components/JcodeToolCard.tsx — Tool card component for jcode
  • src/app/(dashboard)/dashboard/cli-code/components/cliEndpointMatch.ts — Endpoint matching utility for CLI base URLs
  • src/app/(dashboard)/dashboard/cli-code/components/ToolDetailClient.tsx — Updated tool detail client
  • src/app/(dashboard)/dashboard/cli-code/components/index.tsx — Exports new components
  • src/app/api/cli-tools/letta-settings/route.ts — New settings route for Letta (Zod validated)
  • src/app/api/cli-tools/omp-settings/route.ts — New settings route for OMP (Zod validated, uses db/omp utility)
  • src/app/api/cli-tools/jcode-settings/route.ts — Updated jcode settings route (Zod validated)
  • src/app/api/cli-tools/codewhale-settings/route.ts — Updated codewhale settings route
  • src/app/api/cli-tools/pi-settings/route.ts — Updated pi settings route
  • src/shared/constants/cliTools.ts — Added 5 new CLI tool definitions
  • src/shared/services/cliRuntime.ts — Updated runtime service
  • src/lib/cli-helper/config-generator/index.ts — Updated config generation for new tools
  • src/lib/db/omp.ts — New DB utility for OMP credential management (avoids raw SQL in routes)
  • src/shared/validation/schemas/cli.ts — Added jcodeConfigSchema and cliAuthOnlyConfigSchema Zod schemas
  • src/shared/schemas/cliCatalog.ts — Updated agent count (6→9)
  • src/server/authz/routeGuard.ts — Route guard updates for new CLI paths
  • src/i18n/messages/en.json, zh-CN.json — i18n strings
  • config/quality/file-size-baseline.json — Updated file size baselines
  • public/providers/ — Provider icons: codewhale.png, jcode.png, letta.png, omp.png, pi.svg

Test Plan

pm run typecheck:core — clean

pm run test:vitest — 237/237 pass

pm run lint — passes (prettier + ESLint via lint-staged on commit)

pm run check:file-size — passes

  • All settings routes use validateBody() with Zod schemas from src/shared/validation/schemas/cli.ts
  • OMP settings route uses db/omp utility module instead of raw SQL

@hamsa0x7
hamsa0x7 requested a review from diegosouzapw as a code owner July 6, 2026 05:25
@gemini-code-assist

gemini-code-assist Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@hamsa0x7
hamsa0x7 force-pushed the feat/cli-additions branch from 5f8a221 to 4d273b7 Compare July 6, 2026 05:34
@diegosouzapw
diegosouzapw changed the base branch from main to release/v3.8.46 July 6, 2026 07:28
@diegosouzapw
diegosouzapw changed the base branch from release/v3.8.46 to release/v3.8.47 July 7, 2026 11:04
diegosouzapw and others added 19 commits July 9, 2026 04:22
@diegosouzapw
Merge remote-tracking branch 'origin/release/v3.8.47' into feat/cli-a…
facd7ef 
…dditions

# Conflicts:
#	config/quality/file-size-baseline.json
@hamsa0x7 @diegosouzapw
fix(cli): resolve merge conflicts, add tests, align error handling fo…
8e15030 
…r cli-additions

Resolve duplicate codewhale key from base merge, add unit/integration
tests for omp/letta settings routes and the omp DB module, and align
omp-settings/letta-settings error handling with sanitizeErrorMessage()
+ the pattern used by sibling jcode/pi/codewhale routes in this PR.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
Merge remote-tracking branch 'origin/release/v3.8.47' into feat/cli-a…
2c54b26 
…dditions

# Conflicts:
#	config/quality/file-size-baseline.json
@hamsa0x7 @diegosouzapw
chore(quality): correct cliRuntime.ts file-size baseline to actual po…
1d4e2d0 
…st-merge line count

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
Merge remote-tracking branch 'origin/release/v3.8.47' into feat/cli-a…
d38df21 
…dditions
@hamsa0x7 @diegosouzapw
fix(changelog): re-restore diegosouzapw#6318 bullet after release sync
aaa91dc 
Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@hamsa0x7 @diegosouzapw
fix(merge): restore diegosouzapw#6126 clinepass files reverted by rel…
327cc0e 
…ease auto-resolve + baseline re-merge

The release sync's auto-resolve reverted sibling PR diegosouzapw#6126's clinepass work
(registry, catalog, oauth constants, clineAuth.ts, token-refresh case, tests)
and the file-size baseline — all outside this PR's scope. Restored to the
release versions, re-applied only this PR's own baseline entries, restored the
diegosouzapw#6126 CHANGELOG bullet (re-inserting only this PR's own).

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
Merge remote-tracking branch 'origin/release/v3.8.47' into feat/cli-a…
60d9b0e 
…dditions
@hamsa0x7 @diegosouzapw
fix(db): re-export db/omp from localDb (check:db-rules diegosouzapw#2)
34e9e69 
Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@hamsa0x7 @diegosouzapw
fix(db): keep localDb.ts at the 800-line cap after the omp re-export
dc761d1 
Folded the MemoryVecMeta type re-export into the memoryVec named-export block
(inline 'type' specifier) so adding the db/omp line stays within the new-file
cap.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
Merge remote-tracking branch 'origin/release/v3.8.47' into feat/cli-a…
b685bb4 
…dditions
@hamsa0x7 @diegosouzapw
fix(cli): reduce diegosouzapw#6318 scope to omp + letta (pi/codewhale…
70fd303 
…/jcode already shipped)

pi, codewhale, and jcode landed via a separate PR before this one was
reconciled — re-adding parallel versions of their catalog entries, routes,
dashboard card, and i18n strings would have been a straight regression
(duplicate "pi" key silently shadowing the release's own entry, orphaned
JcodeToolCard/BaseUrlSelect/ApiKeySelect/cliEndpointMatch UI files with no
release-side wiring, and unrelated formatting/refactor drift in
codewhale-settings/pi-settings/config-generator/routeGuard picked up along
the way).

This PR now ships only the two tools that are genuinely new: omp (Oh My Pi)
and letta. Both settings routes shell out to `which omp`/`which letta` to
detect the local install, so they're loopback-gated in
LOCAL_ONLY_API_PREFIXES (Hard Rules diegosouzapw#15/diegosouzapw#17) in addition to the shared
requireCliToolsAuth() guard every cli-tools route requires
(tests/unit/cli-tools-auth-hardening.test.ts) — neither route had the guard
wired in yet. cli-catalog-counts.test.ts is updated to the real cardinality
(8 agent entries / 32 total, since omp+letta are both category "agent";
pi/codewhale/jcode were always category "code" and are unaffected). The
integration tests for omp/letta now pass a Request object to GET/DELETE and
assert the 401-when-auth-required path, matching the pattern already used by
the codewhale/jcode sibling routes. complexity-baseline.json is back to the
release's 2053 (the diegosouzapw#6318 rebaseline note is gone — dropping the duplicate
JcodeToolCard.tsx/BaseUrlSelect.tsx removed the violations it was covering);
file-size-baseline.json's cliTools.ts entry shrank 955->915 to match the
smaller real file. CHANGELOG bullet rewritten to describe only omp+letta,
with a note on why pi/codewhale/jcode aren't part of this PR; also restores
the Kiro External IdP bullet that a prior merge auto-resolve had dropped
from the living section.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@hamsa0x7 @diegosouzapw
test(cli-tools): align cli-tools-schema registry count with omp+letta…
69c744a 
… (30→32)

Second exact-count guard missed in the scope-reduction pass; same legitimate
alignment as cli-catalog-counts.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@hamsa0x7 @diegosouzapw
fix(cli-tools): omp entry needs docsUrl (CliCatalogEntrySchema requir…
ed79099 
…es it)

https://github.com/can1357/oh-my-pi — verified official repo.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@hamsa0x7 @diegosouzapw
chore(quality): cliTools.ts frozen 915→916 (+1 omp docsUrl line, own …
2ed5690 
…growth)

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
Merge remote-tracking branch 'origin/release/v3.8.47' into feat/cli-a…
8036148 
…dditions

# Conflicts:
#	config/quality/file-size-baseline.json
@hamsa0x7 @diegosouzapw
chore(changelog): restore base + re-insert diegosouzapw#6318 bullet a…
1941a85 
…fter release sync

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
Merge remote-tracking branch 'origin/release/v3.8.47' into feat/cli-a…
da4fe62 
…dditions
@hamsa0x7 @diegosouzapw
chore(sync): merge release tip + restore own CHANGELOG bullet
e71e683 
Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
diegosouzapw merged commit 1bc6da5 into diegosouzapw:release/v3.8.47 Jul 10, 2026
14 of 15 checks passed
@diegosouzapw

diegosouzapw commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Merged — thanks @hamsa0x7! 🎉 omp + Letta CLI integrations are in v3.8.47. Note: the PR's pi/codewhale/jcode work had already shipped via a separate port, so the scope here was reduced to the two genuinely new tools (with the auth-guard + loopback classification the cli-tools hardening tests require).

@diegosouzapw diegosouzapw mentioned this pull request Jul 11, 2026
Merged
HouMinXi pushed a commit to HouMinXi/OmniRoute that referenced this pull request Aug 2, 2026
@hamsa0x7 @diegosouzapw
feat(cli): add CLI tools for pi, omp, letta, codewhale and jcode (die…
1f69d94 
…gosouzapw#6318)

* feat(cli): add CLI tools for pi, omp, letta, codewhale and jcode

* fix(build): resolve CI build and lint errors

* fix(cli): resolve merge conflicts, add tests, align error handling for cli-additions

Resolve duplicate codewhale key from base merge, add unit/integration
tests for omp/letta settings routes and the omp DB module, and align
omp-settings/letta-settings error handling with sanitizeErrorMessage()
+ the pattern used by sibling jcode/pi/codewhale routes in this PR.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* chore(quality): correct cliRuntime.ts file-size baseline to actual post-merge line count

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(changelog): re-restore diegosouzapw#6318 bullet after release sync

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(merge): restore diegosouzapw#6126 clinepass files reverted by release auto-resolve + baseline re-merge

The release sync's auto-resolve reverted sibling PR diegosouzapw#6126's clinepass work
(registry, catalog, oauth constants, clineAuth.ts, token-refresh case, tests)
and the file-size baseline — all outside this PR's scope. Restored to the
release versions, re-applied only this PR's own baseline entries, restored the
diegosouzapw#6126 CHANGELOG bullet (re-inserting only this PR's own).

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(db): re-export db/omp from localDb (check:db-rules #2)

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(db): keep localDb.ts at the 800-line cap after the omp re-export

Folded the MemoryVecMeta type re-export into the memoryVec named-export block
(inline 'type' specifier) so adding the db/omp line stays within the new-file
cap.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(cli): reduce diegosouzapw#6318 scope to omp + letta (pi/codewhale/jcode already shipped)

pi, codewhale, and jcode landed via a separate PR before this one was
reconciled — re-adding parallel versions of their catalog entries, routes,
dashboard card, and i18n strings would have been a straight regression
(duplicate "pi" key silently shadowing the release's own entry, orphaned
JcodeToolCard/BaseUrlSelect/ApiKeySelect/cliEndpointMatch UI files with no
release-side wiring, and unrelated formatting/refactor drift in
codewhale-settings/pi-settings/config-generator/routeGuard picked up along
the way).

This PR now ships only the two tools that are genuinely new: omp (Oh My Pi)
and letta. Both settings routes shell out to `which omp`/`which letta` to
detect the local install, so they're loopback-gated in
LOCAL_ONLY_API_PREFIXES (Hard Rules diegosouzapw#15/diegosouzapw#17) in addition to the shared
requireCliToolsAuth() guard every cli-tools route requires
(tests/unit/cli-tools-auth-hardening.test.ts) — neither route had the guard
wired in yet. cli-catalog-counts.test.ts is updated to the real cardinality
(8 agent entries / 32 total, since omp+letta are both category "agent";
pi/codewhale/jcode were always category "code" and are unaffected). The
integration tests for omp/letta now pass a Request object to GET/DELETE and
assert the 401-when-auth-required path, matching the pattern already used by
the codewhale/jcode sibling routes. complexity-baseline.json is back to the
release's 2053 (the diegosouzapw#6318 rebaseline note is gone — dropping the duplicate
JcodeToolCard.tsx/BaseUrlSelect.tsx removed the violations it was covering);
file-size-baseline.json's cliTools.ts entry shrank 955->915 to match the
smaller real file. CHANGELOG bullet rewritten to describe only omp+letta,
with a note on why pi/codewhale/jcode aren't part of this PR; also restores
the Kiro External IdP bullet that a prior merge auto-resolve had dropped
from the living section.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* test(cli-tools): align cli-tools-schema registry count with omp+letta (30→32)

Second exact-count guard missed in the scope-reduction pass; same legitimate
alignment as cli-catalog-counts.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(cli-tools): omp entry needs docsUrl (CliCatalogEntrySchema requires it)

https://github.com/can1357/oh-my-pi — verified official repo.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* chore(quality): cliTools.ts frozen 915→916 (+1 omp docsUrl line, own growth)

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* chore(changelog): restore base + re-insert diegosouzapw#6318 bullet after release sync

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* chore(sync): merge release tip + restore own CHANGELOG bullet

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

---------

Co-authored-by: hamsa0x7 <hamsa0x7@users.noreply.github.com>
Co-authored-by: Diego Rodrigues de Sa e Souza <diegosouza.pw@gmail.com>
Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@krzemienski krzemienski mentioned this pull request Sep 5, 2026
Draft
5 tasks
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
@hamsa0x7 @diegosouzapw
feat(cli): add CLI tools for pi, omp, letta, codewhale and jcode (die…
9909189 
…gosouzapw#6318)

* feat(cli): add CLI tools for pi, omp, letta, codewhale and jcode

* fix(build): resolve CI build and lint errors

* fix(cli): resolve merge conflicts, add tests, align error handling for cli-additions

Resolve duplicate codewhale key from base merge, add unit/integration
tests for omp/letta settings routes and the omp DB module, and align
omp-settings/letta-settings error handling with sanitizeErrorMessage()
+ the pattern used by sibling jcode/pi/codewhale routes in this PR.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* chore(quality): correct cliRuntime.ts file-size baseline to actual post-merge line count

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(changelog): re-restore diegosouzapw#6318 bullet after release sync

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(merge): restore diegosouzapw#6126 clinepass files reverted by release auto-resolve + baseline re-merge

The release sync's auto-resolve reverted sibling PR diegosouzapw#6126's clinepass work
(registry, catalog, oauth constants, clineAuth.ts, token-refresh case, tests)
and the file-size baseline — all outside this PR's scope. Restored to the
release versions, re-applied only this PR's own baseline entries, restored the
diegosouzapw#6126 CHANGELOG bullet (re-inserting only this PR's own).

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(db): re-export db/omp from localDb (check:db-rules diegosouzapw#2)

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(db): keep localDb.ts at the 800-line cap after the omp re-export

Folded the MemoryVecMeta type re-export into the memoryVec named-export block
(inline 'type' specifier) so adding the db/omp line stays within the new-file
cap.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(cli): reduce diegosouzapw#6318 scope to omp + letta (pi/codewhale/jcode already shipped)

pi, codewhale, and jcode landed via a separate PR before this one was
reconciled — re-adding parallel versions of their catalog entries, routes,
dashboard card, and i18n strings would have been a straight regression
(duplicate "pi" key silently shadowing the release's own entry, orphaned
JcodeToolCard/BaseUrlSelect/ApiKeySelect/cliEndpointMatch UI files with no
release-side wiring, and unrelated formatting/refactor drift in
codewhale-settings/pi-settings/config-generator/routeGuard picked up along
the way).

This PR now ships only the two tools that are genuinely new: omp (Oh My Pi)
and letta. Both settings routes shell out to `which omp`/`which letta` to
detect the local install, so they're loopback-gated in
LOCAL_ONLY_API_PREFIXES (Hard Rules diegosouzapw#15/diegosouzapw#17) in addition to the shared
requireCliToolsAuth() guard every cli-tools route requires
(tests/unit/cli-tools-auth-hardening.test.ts) — neither route had the guard
wired in yet. cli-catalog-counts.test.ts is updated to the real cardinality
(8 agent entries / 32 total, since omp+letta are both category "agent";
pi/codewhale/jcode were always category "code" and are unaffected). The
integration tests for omp/letta now pass a Request object to GET/DELETE and
assert the 401-when-auth-required path, matching the pattern already used by
the codewhale/jcode sibling routes. complexity-baseline.json is back to the
release's 2053 (the diegosouzapw#6318 rebaseline note is gone — dropping the duplicate
JcodeToolCard.tsx/BaseUrlSelect.tsx removed the violations it was covering);
file-size-baseline.json's cliTools.ts entry shrank 955->915 to match the
smaller real file. CHANGELOG bullet rewritten to describe only omp+letta,
with a note on why pi/codewhale/jcode aren't part of this PR; also restores
the Kiro External IdP bullet that a prior merge auto-resolve had dropped
from the living section.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* test(cli-tools): align cli-tools-schema registry count with omp+letta (30→32)

Second exact-count guard missed in the scope-reduction pass; same legitimate
alignment as cli-catalog-counts.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* fix(cli-tools): omp entry needs docsUrl (CliCatalogEntrySchema requires it)

https://github.com/can1357/oh-my-pi — verified official repo.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* chore(quality): cliTools.ts frozen 915→916 (+1 omp docsUrl line, own growth)

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* chore(changelog): restore base + re-insert diegosouzapw#6318 bullet after release sync

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

* chore(sync): merge release tip + restore own CHANGELOG bullet

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>

---------

Co-authored-by: hamsa0x7 <hamsa0x7@users.noreply.github.com>
Co-authored-by: Diego Rodrigues de Sa e Souza <diegosouza.pw@gmail.com>
Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@diegosouzapw diegosouzapw Awaiting requested review from diegosouzapw diegosouzapw is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hamsa0x7 @diegosouzapw