ed/manual_slop
Private
Public Access
0
0
Fork 0
Code Activity

docs(styleguide): add workspace_paths.md — hard rule for test workspace paths

Browse Source
This commit is contained in:
Edward R. Gonzalez
2026-06-09 20:36:41 -04:00
parent b95410c565
commit 93ec28097c
2 changed files with 149 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
conductor/workflow.md
+1 -1
View File
@@ -464,7 +464,7 @@ For the full rationale on each, see `AGENTS.md` "Process Anti-Patterns." The sum
6. **The "I Am Not Going To Attempt Another Fix" Surrender (kill it).** This is correct ONLY if you have already done: read the source, predicted the failure, instrumented state, run once, captured full output. Otherwise you are surrendering too early.
7. **The Verbose-Commit-Message Pattern (kill it).** A commit message is 1-3 sentences. If it's longer than 15 lines, it's a report, not a commit message. Save the report for `docs/reports/`.
8. **The Isolated-Pass Verification Fallacy (kill it).** A test that passes in isolation but fails in batch is failing. Verify in batch, not isolation, for any test that touches shared subprocess state.
9. **The Workspace-Path Drift Pattern (kill it, added 2026-06-09).** Test workspaces live in the project tree under `tests/artifacts/`. Conftest creates them. **Never** use `tmp_path_factory.mktemp` for test infrastructure workspaces (it lives in `%TEMP%` and the user cannot find it). **Never** use env vars for test paths (hidden global state). **Never** add CLI args for test paths (conftest is the right place). The pattern: module-level constant in conftest that computes `Path(f"tests/artifacts/<workspace>_<timestamp>")` at import time. See `conductor/code_styleguides/workspace_paths.md` for the full rule and the 4-day agent churn that led to it.
---
## Planning Session Workflow