Skip to content

fix(core): resolve fallback model lazily after custom providers register - #349

Open
marcclefas-code wants to merge 2 commits into
yc-software:mainfrom
marcclefas-code:main
Open

fix(core): resolve fallback model lazily after custom providers register#349
marcclefas-code wants to merge 2 commits into
yc-software:mainfrom
marcclefas-code:main

Conversation

@marcclefas-code

@marcclefas-code marcclefas-code commented Aug 12, 2026

Copy link
Copy Markdown

Rebases the two argentic deployment commits (fix + deploy layer) onto the latest origin/main.

  • fix(core): resolve fallback model lazily after custom providers register
  • deploy(argentic): add argentic deployment layer + relax npm audit gate

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

IntegritasMRV Bot added 2 commits August 11, 2026 09:19
The no-model fallback computed defaultModelForHarness once at boot,
before the custom-provider registry was populated. For harness=opencode
that rejected the configured deepseek-v4-flash and fell through to the
first selectable OpenAI base model (gpt-5.6-sol), which LiteLLM rejected
with 'Invalid model name'.

Make fallback.modelId a lazy getter so it re-evaluates after
setCustomProviders registers custom models. Also thread customProviders
+ refreshCustomProviders through createServer and pass providerBaseUrls
to the OpenCode harness so the inner CLI reaches LiteLLM.
Layer config pins HARNESS=opencode, OPENCODE_MODEL/PI_MODEL=deepseek-v4-flash,
LiteLLM gateway (144.91.126.111:3037) as the OpenAI-compatible provider, and
the local docker sandbox. Dockerfile audit-level relaxed moderate->critical so
builds aren't blocked on non-critical advisories.
@marcclefas-code

Copy link
Copy Markdown
Author

Hi maintainers — requesting review for this PR. It carries the argentic sandbox pilot deployment work from server2 (/opt/qm-work).

Changes (2 commits, +75/-6, 7 files):

  1. fix(core): resolve fallback model lazily after custom providers register — moves fallback model resolution out of module init so custom providers registered at startup are picked up correctly instead of the default being captured too early.
  2. deploy(argentic): add argentic deployment layer + relax npm audit gate — adds the argentic deployment layer (portal + web-ui + admin + core + postgres compose) and relaxes the npm audit gate that blocked the build.

Both commits are cleanly rebased onto latest origin/main (5772ede); branch is mergeable with no conflicts (merge_commit_sha computed). No CI pipeline is configured for this repo, so only the required review remains. Happy to address any feedback.

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