Skip to content

feat(cli): add modeldock verify --all - #228

Open
utkarsha741 wants to merge 1 commit into
OpenAgentHQ:mainfrom
utkarsha741:feature/verify-all
Open

feat(cli): add modeldock verify --all#228
utkarsha741 wants to merge 1 commit into
OpenAgentHQ:mainfrom
utkarsha741:feature/verify-all

Conversation

@utkarsha741

Copy link
Copy Markdown

Summary

Adds modeldock verify --all to bulk-verify every installed model,
reusing the existing ModelManager.verify() logic per model. Prints
a rich summary table (name + OK/FAILED) and supports --json output
for scripting.

Closes #41

Changes

  • New verify CLI command (src/modeldock/cli/commands/verify.py),
    registered in app.py.
  • render_verify() added to console.py for the summary table.
  • Unit tests covering --all, explicit model names, empty-installed
    edge case, invalid flag combos, exit codes, and --json output.

Testing

  • pytest -q (593 passed, 4 skipped — pre-existing Ollama skips)
  • ruff format --check, mypy --strict, bandit all clean on
    changed files

Checklist

  • Branch named per Git Workflow (feature/verify-all)
  • Not developed on main
  • Code follows AGENT.md standards (type hints, Pydantic v2, no
    generic Exception, no business logic in CLI)
  • domain/ and ports/ stay pure (untouched by this change)
  • Quality gates pass locally: ruff, mypy --strict, bandit, pytest
  • Docs not updated (no public API/config change beyond the new
    CLI flag itself)
  • pyproject.toml/__init__.py versions unchanged (not a release)

Adds --all flag to bulk-verify every installed model, reusing
ModelManager.verify() per model. Prints a rich summary table
(name + OK/FAILED) and supports --json output.

Closes OpenAgentHQ#41
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.

CLI: add modeldock verify --all

1 participant