Repository navigation
Harden configuration validation and move catalog data under the package - #1
Merged
Merged
Conversation
…ge" -m "Settings now validates backend selectors and numeric bounds at the process boundary (fail closed) and gains explicit knobs for context size, catalog limits, and the Groq/DSPy backend. Catalog JSON moves from top-level data/ into src/meridian/data so it ships as package data. CI adds a format check and a build step; Redis ports bind to localhost only.
brunovicco
added a commit
that referenced
this pull request
Jul 18, 2026
main squash-merged an equivalent of this branch's own "Harden configuration validation and move catalog data under the package" commit (PR #1, 1f68ec7) under a different hash, so git had no shared history to fast-forward through. Resolved by keeping this branch's side everywhere it is a strict superset of main's (flat-demo Makefile target and README sections, the enriched knowledge-base content, the dspy.Refine warn_on_type_mismatch fix, the router tie-break tests) - verified the pre-merge diffs of the two equivalent commits were byte-identical before doing so. No conflicts touched application logic outside the two files already conflicting on wording.
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
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
Settingsnow validates backend selectors and numeric bounds at the processboundary and fails closed on misconfiguration, instead of silently degrading.
Groq/DSPy backend (model, API base, optional compiled router artifact).
data/intosrc/meridian/data/catalogso it ships as package data (
pyproject.tomlpackage-data entry added).bind to
127.0.0.1only.Test plan
make check(format, lint, typecheck, test) passesmake demoruns end-to-end with fake providers and no credentialsmake redis-up && MERIDIAN_BACKEND=redis make demostill finds the catalog data after the moveMERIDIAN_BACKEND/MERIDIAN_LLM_BACKENDvalues raise a clearerror instead of degrading silently