Repository navigation
feat(loop-init): add API Route provider - #697
DennyHo0917 wants to merge 1 commit into
Conversation
Signed-off-by: dennyho0917 <dennyho0917@gmail.com>
|
This PR changes paths that must run the real Fork PRs from first-time contributors start with those workflows waiting for approval. A maintainer needs to open the Checks tab and click Approve and run workflows. Until that happens, branch protection will show the PR as blocked even after a review. Content-only PRs ( — loop-engineering fork-pr-gate |
|
Leaving this open. The merge state is BLOCKED, only the fork gate check is green, and there is no approving review. |
|
The Validate and Audit runs are approved. This stays open: it is a behavior change, and it still needs those checks to pass and an approving review before it can merge. |
Summary
Add an optional API Route provider to
loop-init --with-foundry, following the existing provider selection approach in #653.Related issue
None.
Changes
templates/pattern-template.md+ updatedregistry.yaml)tools/--model-provider api-routeemitsmodel/api-routewith the OpenAI-compatible endpointhttps://global.api-route.com/v1. It defaults togpt-6.1-soland accepts an explicit model ID without embedding a static model catalog or unverified context limits. The selected ID is quoted in YAML. Source and compiled CLI are included, with setup documentation and four CLI regression tests.API keys are configured in the consuming runtime and are never written into the scaffold. This preset documents the verified OpenAI-compatible endpoint only.
Checklist (from CONTRIBUTING)
STATE.md*examples use.examplesuffix — not applicable; no state examples changeddocs/safety.md— not applicable; no safety content changedtools/<pkg>:cd tools/<pkg> && npm ci && npm test— executed; see the existing baseline failure belownpm ci && npm run validate:registry— not applicable; registry unchangedTesting / Dogfood
npx @cobusgreyling/loop audit .(or package-local tests) on affected pathsManual review of generated state / skill output if scaffolding changed
npm ciandnpm testundertools/loop-init: TypeScript build succeeds; 72/73 tests pass, including all four new API Route cases.node --test --test-name-pattern 'API Route' test/cli.test.mjs: 4/4 pass.The sole full-suite failure is the existing
loop-init prints Loop Ready score after scaffoldassertion: it expects/Loop Ready:/, while output contains=== Loop Ready score ===. I reproduced the same failure using the unchanged base commit's compiled CLI, then restored the new build.Root
npm run check:loop-init: passes.Generated a Codex
ci-sweeperscaffold, parsed.foundry/stack.yaml, and used its actual endpoint/model with the OpenAI SDK and an environment-supplied API Route key:gpt-6.1-solreturnedAPI Route OK. This validates the generated provider configuration, not a complete loop execution.git diff --check: passes.I am affiliated with API Route. This is an optional provider preset; existing provider defaults and dependencies are unchanged.