Skip to content

Fix CI planning-doc contract path assumption#191

Merged
imaurer merged 2 commits intomainfrom
fix-planning-docs-ci-path
Mar 11, 2026
Merged

Fix CI planning-doc contract path assumption#191
imaurer merged 2 commits intomainfrom
fix-planning-docs-ci-path

Conversation

@imaurer
Copy link
Collaborator

@imaurer imaurer commented Mar 11, 2026

Summary

  • stop upstream planning docs tests from hardcoding Ian-local planning paths
  • add repo-local planning fixtures for the BioMCP strategy/frontier contract
  • keep the planning contract explicit in CI instead of skipping when the local vault is absent

Testing

  • pytest tests/test_upstream_planning_analysis_docs.py -q
  • pytest tests/test_upstream_planning_analysis_docs.py tests/test_public_search_all_docs_contract.py tests/test_public_skill_docs_contract.py tests/test_public_install_docs_contract.py tests/test_docs_changelog_refresh.py -q

imaurer added 2 commits March 11, 2026 06:29
Move planning root resolution behind _planning_root() helper so env
override behavior is directly testable. Add three focused tests covering
the default fixture fallback, a valid BIOMCP_PLANNING_ROOT override, and
loud failure on a bad override path. Removes all implicit module-level
initialization that would have hidden override behavior from test isolation.
@imaurer imaurer merged commit 6dcfec5 into main Mar 11, 2026
1 check passed
imaurer added a commit that referenced this pull request Mar 11, 2026
…very docs

Bumps version to 0.8.15. Ships the planning-docs CI path fix from PR #191
so release validation uses repo-local fixtures on GitHub Actions, and
refreshes public discovery docs from PR #190 to teach `search all` as
the unified entry point.
@imaurer imaurer deleted the fix-planning-docs-ci-path branch March 13, 2026 20:30
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