Root Hygiene

Do not use this skill for general cleanup unrelated to test/temp/cache behavior.

Rules

1. Treat root clutter as a behavior problem first, not an ignore-file problem.
2. Do not introduce TemporaryDirectory(dir=Path.cwd()) in this repository.
3. For top-level tests, prefer fixtures from tests/conftest.py:
   • project_temp_root
   • project_temp_dir
   • overridden tmp_path
4. For ms-core tests, prefer fixtures from ms-core/tests/conftest.py:
   • project_temp_root
   • project_temp_dir
5. Use .gitignore only as a second line of defense after fixing the write location.
6. If old temp folders remain because of local ACL damage, report that clearly instead of pretending cleanup succeeded.

Workflow

1. Inspect before editing

Run:

git status --short --branch
Get-ChildItem -Force | Where-Object { $_.Name -like 'tmp*' -or $_.Name -like '.pytest*' -or $_.Name -eq '.tmp' -or $_.Name -eq '__pycache__' }
Get-ChildItem -Force ms-core | Where-Object { $_.Name -like 'tmp*' -or $_.Name -like '.pytest*' -or $_.Name -eq '.tmp' -or $_.Name -eq '__pycache__' }

If the task involves temp creation, inspect:

• tests/conftest.py
• ms-core/tests/conftest.py
• pyproject.toml
• ms-core/pyproject.toml
• scripts/clean_local_artifacts.ps1

2. Fix the source of root clutter

Common fixes:

• replace TemporaryDirectory(dir=Path.cwd()) with project_temp_dir()
• use the repo fixtures instead of builtin temp helpers that are unstable in this Windows environment
• route benchmark outputs and cache roots into the test-owned temp tree
• avoid re-enabling pytest cache provider unless you have a strong reason

3. Update ignore rules only if needed

Expected ignored local artifacts include:

• .tmp/
• .pytest-local-temp/
• tmp*/

Mirror ignore changes in ms-core/.gitignore when ms-core produces the same class of artifacts.

4. Verify with focused checks first

Top-level focused check:

$env:PYTHONPATH='ms-core/src'; pytest tests/test_root_hygiene.py -v --tb=short

Top-level full check:

$env:PYTHONPATH='ms-core/src'; pytest tests/ -v --tb=short -x

ms-core check:

pytest ms-core/tests -v --tb=short -x

5. Clean local artifacts

Use:

powershell -ExecutionPolicy Bypass -File scripts\clean_local_artifacts.ps1

This script is reliable for normal writable temp folders created after the hygiene changes.

If cleanup fails for old folders, call out the exact directories and note that local ACL state is blocking removal.

Output Expectations

When reporting the result:

• state which files were causing root clutter
• state which fixture or path policy now owns temp creation
• include the verification commands you ran
• mention any leftover historical temp directories separately from new behavior

Reference Files

• tests/conftest.py
• tests/test_root_hygiene.py
• ms-core/tests/conftest.py
• ms-core/tests/test_root_hygiene.py
• scripts/clean_local_artifacts.ps1