feat(cli): dawn verify runs an environment preflight (Node floor, Docker, provider-derived keys)#358
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 4.8 <[email protected]>
…derived, not hardcoded OPENAI_API_KEY) Co-Authored-By: Claude Opus 4.8 <[email protected]>
Co-Authored-By: Claude Opus 4.8 <[email protected]>
Co-Authored-By: Claude Opus 4.8 <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
…ormalize node version Add the 5th 'runtime' check (Node floor probe) as the last entry in checks[], bump counts.passed/total 4->5, and normalize the live process.versions.node in both fixture normalizers to <version:node> so the fixture stays stable across Node patch releases. Also reconcile the deps check's missingEnvVars to [] to match reality: the harness overwrites the scaffolded route with a provider-less agent/graph, so collectRouteProviders finds no provider and the provider-derived env check (263f609) requires no API key. Co-Authored-By: Claude Opus 4.8 <[email protected]>
Contributor
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.
dawn checkvalidates config;dawn verifyvalidates app integrity — but neither checked the environment the app runs in. This extendsdawn verify(no new command — keeps the CLI surface minimal) with aruntimepreflight and fixes a real bug in its key check.What's new
runtimecheck — verify now has 5 checks (app,routes,typegen,deps,runtime):process.versions.node, pure numeric semver compare) — the real floor below whichnode:sqlite(used by@dawn-ai/sqlite-storage/memory) breaks. A stale Node fails verify (non-zero exit).preflight()(reused, not reimplemented).RECOMMENDED_ENV_VARS = ["OPENAI_API_KEY"], so it nagged every app aboutOPENAI_API_KEYand never checked the key the app actually needs. Now it infers each route's provider from itsmodelid (reusing the SDK'sinferProvider) and checks only those keys: an Anthropic-only app requiresANTHROPIC_API_KEY(notOPENAI_API_KEY); multi-provider → the union; ollama → none. Still a warning (not a failure), env-file resolution preserved.--jsonincludes the newruntimeobject; the human summary prints aRuntime: Node … OKline. Full local verify green (build 23/23 · typecheck 24/24 · lint 20/20 · test 1171 passed · check-docs). Changeset: patch@dawn-ai/cli. Docs:cli.mdxverify section + a preflight one-liner in getting-started/configuration.Error codes (
DAWN_E5101/DAWN_E2002) are marked with TODOs to wire once the error-code registry (#357) merges.🤖 Generated with Claude Code