Skip to content

fix: validate config check#41

Merged
srobinson merged 4 commits into
mainfrom
fix/config-check-validation
Jun 20, 2026
Merged

fix: validate config check#41
srobinson merged 4 commits into
mainfrom
fix/config-check-validation

Conversation

@srobinson

Copy link
Copy Markdown
Owner

Summary

  • centralize config validation rules and reuse them for runtime loading plus config check diagnostics
  • make config check report malformed TOML, invalid values, effective coerced values, aiSummarization, and per-field validity
  • emit all concrete schema fields in generated TOML, including embeddings retry/HNSW settings and paths.cacheDir

Test plan

  • npx --yes pnpm@10.28.0 check
  • npx --yes pnpm@10.28.0 build
  • npx --yes pnpm@10.28.0 test
  • manual invalid provider check returned exit code 1 with effective provider openai and one warning

@srobinson srobinson merged commit 769ae53 into main Jun 20, 2026
8 checks passed
@srobinson srobinson deleted the fix/config-check-validation branch June 20, 2026 03:02
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.

1 participant