What the reviewer found
TUI test-probe doc: `ENV = dict(os.environ)` is example code passing the parent env to a spawned child pty process (a removed diagnostic var is called out in the comment), not exfiltration. `rm -rf dist/` only clears the project's own build output before rebuilding.
example-or-fixture— an example or test fixture
What was read
The file as it ships in baphuongna/pi-crew:
skills/real-test-pi-crew/SKILL.md
What the static scan said
The scan flagged 2things. The reviewer kept 0 and dismissed 2 as false.
E2Harvests environment variables — false positiveRMRecursive force delete — false positive
How this review was made
Sonnet 5 read the files above on 7 September 2026 and answered three questions: is it dangerous to whoever installs it, is each scanner finding real, and what should the installer know. The verdict is bound to the file's hash; when the file changes, it is scanned afresh and reviewed again. A script that changes while the definition does not is not re-reviewed — that is a known gap. How the scan and the review work.