Skip to content

Add inference URL override - #30

Merged
jeremyfelder merged 8 commits into
mainfrom
feat/inference-url-override
Oct 26, 2026
Merged

jeremyfelder merged 8 commits into
mainfrom
feat/inference-url-override

Conversation

@jeremyfelder

Copy link
Copy Markdown
Collaborator

Adds override for inference url separate from the base url

zro-inference
zro-inference previously approved these changes Oct 26, 2026

@zro-inference zro-inference left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR adds an inference URL override (--inference-url / ZRO_INFERENCE_URL) plus a no-auth mode (--no-auth / ZRO_NO_AUTH=1), threading endpoint resolution through args parsing, the run pipeline, model catalog handling, and every agent tool adapter, with supporting docs, changelog, and new tests (tests/endpoints.test.ts, tests/no-auth.test.ts, plus args/run/tool test updates). Note on coverage: all four specialist analyzers (correctness, security, test-coverage, adversarial) timed out and returned no findings for this run, so this approval reflects an absence of reported blockers rather than a clean bill of health — a follow-up human or automated pass over the no-auth credential-handling and endpoint-precedence paths would be worthwhile.

JenClaude command reference

Reply with @MoonMathJen on this PR to trigger commands:

Command Example What it does
Re-review @MoonMathJen rereview Re-review after changes (full or scoped — see below)
Stop @MoonMathJen stop Cancel in-progress review
Stop + summarize @MoonMathJen stop and summarize Post partial findings so far
Focus specialty @MoonMathJen only security Review only the named specialty
Skip specialty @MoonMathJen skip test-coverage Review everything except the named specialty
Change model @MoonMathJen pr-review-model: kimi-k2.7-coder Run this review with a specific model
Add context @MoonMathJen focus on auth edge cases Free-text guidance passed to reviewers

Specialties: correctness, security, test-coverage, adversarial

Any text in the comment beyond a recognized command is passed to the reviewers as context. Use it to point them at areas that matter, explain what changed since the last review, or call out constraints (e.g. "skip generated files", "the auth flow is the riskiest part here"). Hard review rules still win over reviewer guidance.


JenClaude metrics: cost $0.000, time 44m 03s, tokens 38,258.

@jeremyfelder
jeremyfelder merged commit b4c138b into main Oct 26, 2026
26 checks passed
@jeremyfelder
jeremyfelder deleted the feat/inference-url-override branch October 26, 2026 15:43
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