Skip to content

fix: update dot-project spec hash and mcp expected tools count#3

Open
Jaydeep869 wants to merge 1 commit intoMarc-cn:feature/reproducibility-pluginfrom
Jaydeep869:feature/reproducibility-plugin
Open

fix: update dot-project spec hash and mcp expected tools count#3
Jaydeep869 wants to merge 1 commit intoMarc-cn:feature/reproducibility-pluginfrom
Jaydeep869:feature/reproducibility-plugin

Conversation

@Jaydeep869
Copy link
Copy Markdown

Summary

Fixes CI test failures occurring in the PR. Updated the test_mcp_server.py integration test to expect 18 MCP tools instead of 15, accounting for the new tools added recently. Additionally, updated the .project/ spec hash to resolve a synchronization failure in test_dot_project_upstream.py caused by an upstream CNCF spec change.

Type of Change

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)

Framework Changes Checklist

If this PR modifies the darnit framework (packages/darnit/):

  • Updated framework spec (openspec/specs/framework-design/spec.md) if behavior changed
  • Ran uv run python scripts/validate_sync.py --verbose and it passes
  • Ran uv run python scripts/generate_docs.py and committed any doc changes

Control/TOML Changes Checklist

If this PR modifies controls or TOML configuration:

  • Control metadata defined in TOML (not Python code)
  • SARIF fields (description, severity, help_url) included where appropriate
  • Ran validation to confirm TOML schema compliance

Testing

  • Tests pass locally (uv run pytest tests/ -v)
  • Added tests for new functionality (if applicable)
  • Linting passes (uv run ruff check .)

Additional Notes

The CI was failing on two specific tests:

  1. tests/integration/test_mcp_server.py - Fixed by updating the expected tools list and count (from 15 to 18).
  2. tests/darnit/context/test_dot_project_upstream.py - Fixed by running --update-hash to sync the new upstream .project spec hash.

@Jaydeep869
Copy link
Copy Markdown
Author

resolve kusari-oss#138

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