Skip to content

CR-037: Add model manifest warning CLI surface#28

Merged
coreytshaffer merged 1 commit into
mainfrom
cr-037-model-manifest-warning-cli
Jun 17, 2026
Merged

CR-037: Add model manifest warning CLI surface#28
coreytshaffer merged 1 commit into
mainfrom
cr-037-model-manifest-warning-cli

Conversation

@coreytshaffer

Copy link
Copy Markdown
Owner

CR-037: Model Manifest Warning CLI Surface

Status

Implemented

Scope

Implement only:

tc model warn --manifest <path> --route <path>

This change:

  • loads a model route manifest JSON file
  • loads a route metadata JSON file
  • calls compare_route_to_manifest(route_payload, manifest)
  • prints a concise pass/warn summary
  • exits 0 when warnings exist
  • adds a small route payload fixture for reviewer demos

This change does not probe Ollama, LM Studio, Qwen Cloud, filesystems, or live
model artifacts. It does not block runtime routing, mutate the ledger, or wire
the warning layer into runtime execution yet.

Acceptance Criteria

  • tc model warn --manifest <path> --route <path> loads both JSON files.
  • Calls compare_route_to_manifest(route_payload, manifest).
  • Matching manifest and route metadata prints a pass summary.
  • Mismatched metadata prints warnings but still exits 0.
  • Missing or malformed files exit nonzero.
  • No backend probing or routing enforcement.
  • No ledger mutation.
  • Add focused tests and a small example route payload fixture.

Validation

python -m py_compile triage_core\model_manifest.py triage_core\tc_cli.py
python -m pytest tests\test_model_manifest.py tests\test_model_cli.py -q
python -m pytest -q
tc model warn --manifest docs\security\examples\model_route_manifest_local_ollama.json --route docs\security\examples\route_payload_local_ollama.json
$LASTEXITCODE
tc model warn --manifest docs\security\examples\model_route_manifest_cloud_qwen.json --route docs\security\examples\route_payload_local_ollama.json
$LASTEXITCODE
git diff --check
git status --short

@coreytshaffer coreytshaffer merged commit 0031ef0 into main Jun 17, 2026
3 checks passed
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