Repository navigation
fix: price the Claude 5 generation, refresh the default LLM allowlist (0.3.5) - #18
Merged
Merged
Conversation
… (0.3.5) omadia core moved its Anthropic catalog to Opus 5 / Sonnet 5 (byte5ai/omadia #1058) and now discovers models live from the vendor API. Two things here still named the previous generation: - usageTelemetry.ts: added exact price entries for claude-opus-5 ($5/$25) and claude-sonnet-5 ($2/$10). Sonnet 5 is CHEAPER than 4.6, so without the exact entry the generic 'sonnet' family fallback would have over-billed it at the old $3/$15 rate. The 4.x entries stay (still billable for pinned deployments). - docker-compose.dev-platform.yaml: DEV_PLATFORM_LLM_ALLOWED_MODELS default now lists the Claude 5 ids first and keeps Opus 4.8; the old `claude-sonnet-4-8` entry never existed as a model id and is gone. This stays an exact-match allowlist of what the CLI reports (a policy input), so it cannot be a class ref. Version 0.3.4 -> 0.3.5 in manifest.yaml, package.json and the lockfile entry (by hand, per scripts/check-lock-sync.mjs). check:lock, typecheck and the plugin test suite pass. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018UfJ6mYnCjhMb4x2jSNJCW
The "Build @omadia/plugin-api from the sibling checkout" step has been red
on every run since core main moved to vite 8 / vitest 4.1: npm 10.9's
arborist walks core's peer-dependency web even under --no-workspaces and
crashes in #loadPeerSet ("Cannot read properties of null (reading
'edgesOut')") before typescript is installed. Reproduced locally against
core main 9d723e39 and its parent d82b42c3 — unrelated to this PR.
The step installs two leaf packages and needs no peer resolution, so
--legacy-peer-deps skips the walk; verified locally: install succeeds and
`tsc -p packages/plugin-api` emits index.d.ts.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018UfJ6mYnCjhMb4x2jSNJCW
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
omadia core moved its Anthropic catalog to Opus 5 / Sonnet 5 (byte5ai/omadia#1058) and now discovers models live from the vendor API. Two things here still named the previous generation:
usageTelemetry.ts: exact price entries forclaude-opus-5($5/$25) andclaude-sonnet-5($2/$10). Sonnet 5 is cheaper than 4.6 — without the exact entry the genericsonnetfamily fallback would over-bill it at $3/$15. 4.x entries stay.docker-compose.dev-platform.yaml:DEV_PLATFORM_LLM_ALLOWED_MODELSdefault lists the Claude 5 ids first and keeps Opus 4.8; the oldclaude-sonnet-4-8entry never existed. This remains an exact-match allowlist of what the CLI reports (policy input), so it cannot be a class ref.Version 0.3.4 → 0.3.5 in manifest.yaml, package.json and the lockfile entry (by hand, per
scripts/check-lock-sync.mjs).Verified:
check:lock,npm run typecheck -w packages/plugin,npm run test -w packages/plugin— all pass.🤖 Generated with Claude Code
https://claude.ai/code/session_018UfJ6mYnCjhMb4x2jSNJCW
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.