Skip to content

CI: run the suite as python -m pytest - #66

Merged
tzhouam merged 1 commit into
mainfrom
docs/ci-workflow-and-doc-map
Aug 12, 2026
Merged

CI: run the suite as python -m pytest#66
tzhouam merged 1 commit into
mainfrom
docs/ci-workflow-and-doc-map

Conversation

@tzhouam

@tzhouam tzhouam commented Aug 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Reduced scope: the workflow and doc refresh originally in this PR already landed on main as 47fdf33. What remains is the one fix its first run surfaced.

Bare pytest omits the repo root from sys.path, and test_release_audit.py imports the root-level tools/ package — so collection dies in 20s (ModuleNotFoundError: No module named 'tools') on every run. python -m pytest inserts cwd, and is the exact command DEVELOPMENT.md documents. Both suite invocations updated.

With #64 merged (the config.py ratchet fix), this is the last known blocker to a green suite job.

🤖 Generated with Claude Code

@tzhouam
tzhouam force-pushed the docs/ci-workflow-and-doc-map branch from 47fdf33 to e547218 Compare August 12, 2026 07:54
@tzhouam

tzhouam commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

The failing suite check is not this PR's content: main itself is red on test_repo_neutral_core (config.py carries 3 repo literals against the recorded ceiling of 2) — invisible until this workflow existed, because no CI ran the suite and local runs used working trees carrying the fix. The fix is in #64 (config resource-probe glob markers + ratchet update). Merge #64 first, then re-run this PR's checks. Earlier collection error (No module named 'tools') is already fixed here by running python -m pytest.

tzhouam
CI workflow + developer doc map refresh
775c535 
- .github/workflows/test.yml: full pytest, --help/doctor smoke, wheel
  installed into a clean venv to prove the four data trees actually ship
  (test_packaged_runtime only asserts pyproject TEXT), and both knowledge
  linters — on every push and PR. Signal only for now: making it a merge
  gate needs the suite job set as a required status check in repo settings,
  which no commit can do.
- DEVELOPMENT.md documents that workflow and its limits; CODE_TOUR updated
  to cover all 80 source files with the staged reading route.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>


Run the suite as `python -m pytest`: bare `pytest` omits the repo root from
sys.path, and upstream test_release_audit.py imports the root-level tools/
package — the exact command DEVELOPMENT.md documents.
@tzhouam
tzhouam force-pushed the docs/ci-workflow-and-doc-map branch from e547218 to 775c535 Compare August 12, 2026 08:20
@tzhouam tzhouam changed the title CI workflow + developer doc map refresh CI: run the suite as python -m pytest Aug 12, 2026
@tzhouam tzhouam mentioned this pull request Aug 12, 2026
Merged
@tzhouam

tzhouam commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

Update: #64 accidentally merged this PR's original CI/docs content instead of the Direct/MCP work (branch ref mishap on my side — details in #67). Consequences for CI: main's red suite job needs BOTH #67 (config.py ratchet fix, the actual #64 content re-landed) and this PR (python -m pytest). Merge in either order; both are full-suite green against current main.

@tzhouam
tzhouam merged commit be52f10 into main Aug 12, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@tzhouam