Catalog refactor and security/quality hardening pass (external/eyrie snapshot)#60
Merged
Merged
Conversation
…deployment, onboarding)
…RSION - Remove broken internal/version/ package (embedding stale 0.1.0 copy) - Add root version.go with //go:embed VERSION (resolves to root VERSION) - Revert client/client.go to SetVersion pattern (version set by root init) - Remove legacy migration code (migrate.go, EnsureDeploymentConfigV2, etc.) - Add deploymentOwnerProviderID for poolside/groq-direct routing
api.poolside.ai does not resolve — Poolside runs on Baseten (inference.poolside.ai). Verified with curl that models endpoint returns available models (laguna-m.1, laguna-xs.2).
Both providers were missing FetchPoolside and FetchGroq functions and were not registered in the live.Registry map, causing 'live: unknown fetcher' errors during catalog discovery. Also updates the parity test assertion from 19 to 21 providers.
Adds mock HTTP server tests, no-key tests, and JSON parsing tests for both FetchPoolside and FetchGroq, following the same pattern as existing provider tests (grok, kimi, etc.).
Adds ClinePass as an OpenAI-compatible provider using the https://api.cline.bot/api/v1 endpoint with CLINE_API_KEY auth. Wires through all layers: provider spec, env config, profile, runtime profile, routing, live fetcher, client registry, compat config, deployment, and tests.
Cline API does not expose a GET /models endpoint, so credential probing fails with HTTP 404. Changed ProbeKind to ProbeNone (key is saved without validation). Live fetcher returns a curated static list of 10 known ClinePass models instead of hitting the API.
…side, Groq, MiniMax
- Catalog v1 spec parsing, provider discovery/refresh, live enrichment, and deployment routing refactor - Remove deprecated legacy catalog shim - Fix stale ProtocolID field references in default_catalog_test.go and model_policy_test.go to match the renamed Deployment.APIProtocolID field
5 tasks
…client and config
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.
Summary
This branch is off an older, diverged checkout of eyrie used as hawk's
external/eyriegit submodule (pinned at544a924). The same catalog refactor has already been applied to the up-to-date eyrie repo in PR #59; this PR brings it to this older submodule snapshot until it can be re-synced to eyrie's current main.ProtocolIDfield references in test files to match the renamedDeployment.APIProtocolIDfield (caught viago veton this snapshot)Test plan
GOWORK=off go build ./...GOWORK=off go vet ./...GOWORK=off go test ./...