Add Docker image test script, fix stale API key env vars, rewrite DOC… - #47
Merged
Conversation
…KER.md scripts/test-docker.sh smoke tests a published or locally built image end to end: pulls it and reports whether the digest changed, checks the multi-arch manifest, non-root uid, HEALTHCHECK/CMD/volume declarations, bun + dotnet + llms-py versions, `llms --init` producing valid JSON, `llms ls`, then starts a container on a free port and polls for HTTP 200 and a healthy status. Exits non-zero on any failure so it can gate a release. The API key env vars in .env.example and both compose files had drifted from providers.json — GROK_API_KEY, ZAI_API_KEY and GOOGLE_FREE_API_KEY no longer exist (they are XAI_API_KEY, ZHIPU_API_KEY and GOOGLE_API_KEY), and 10 providers were missing entirely. .env.example is now generated from the catalogue, with the docs URL for each provider's key. Both compose files switch to `env_file: .env` so new providers don't need a per-key entry, add `host.docker.internal:host-gateway` so the container can reach a local Ollama or LM Studio, and publish on 127.0.0.1 by default. DOCKER.md is rewritten around the llmspy.org/install.sh one-line installer, with the manual docker run / compose paths kept for people who want them. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01GWaChcrVtD7gXw759pbkuv
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.
…KER.md
scripts/test-docker.sh smoke tests a published or locally built image end to end: pulls it and reports whether the digest changed, checks the multi-arch manifest, non-root uid, HEALTHCHECK/CMD/volume declarations, bun + dotnet + llms-py versions,
llms --initproducing valid JSON,llms ls, then starts a container on a free port and polls for HTTP 200 and a healthy status. Exits non-zero on any failure so it can gate a release.The API key env vars in .env.example and both compose files had drifted from providers.json — GROK_API_KEY, ZAI_API_KEY and GOOGLE_FREE_API_KEY no longer exist (they are XAI_API_KEY, ZHIPU_API_KEY and GOOGLE_API_KEY), and 10 providers were missing entirely. .env.example is now generated from the catalogue, with the docs URL for each provider's key.
Both compose files switch to
env_file: .envso new providers don't need a per-key entry, addhost.docker.internal:host-gatewayso the container can reach a local Ollama or LM Studio, and publish on 127.0.0.1 by default.DOCKER.md is rewritten around the llmspy.org/install.sh one-line installer, with the manual docker run / compose paths kept for people who want them.
Claude-Session: https://claude.ai/code/session_01GWaChcrVtD7gXw759pbkuv