[codex] Make watchlist skill runtime py-free

[dd3ok]

Summary

• Move the WATCHLIST validator out of the installable skill bundle to tools/validate_watchlist.py.
• Move maintainer self-check prompts out of the runtime bundle to docs/maintainers/self-checks.md.
• Keep runtime guidance py-free and hot-path focused: SKILL.md keeps trigger, storage, add/review, safety, and validation decisions while delegating field/status details to references.
• Clarify storage disambiguation so agents ask only when both root and private WATCHLIST files exist and scope remains unclear.
• Harden package/test guards for .py, .pyw, .pyc, .pyo, case variants, repository-only docs/, wrapper SystemExit, and self-check exclusion.

Runtime size

• .agents/skills/watchlist-md/SKILL.md: 5,483 bytes -> 4,416 bytes
• SKILL.md: 128-ish line structure -> 100 lines under the repo test measurement
• Installable skill bundle: no .py, .pyw, .pyc, .pyo, scripts/, __pycache__, tooling references, or self-checks.md matches

Validation

• PYTHONDONTWRITEBYTECODE=1 python -m unittest discover -s evals -p 'test_*.py' - 92 tests passed
• python evals/check_watchlist.py examples/WATCHLIST.example.md --strict-format --strict-safety --require-archive-section
• python evals/check_watchlist.py .agents/skills/watchlist-md/assets/WATCHLIST.template.md --strict-format --strict-safety --require-archive-section
• python tools/validate_watchlist.py examples/WATCHLIST.example.md --strict-format --strict-safety --require-archive-section
• python tools/validate_watchlist.py .agents/skills/watchlist-md/assets/WATCHLIST.template.md --strict-format --strict-safety --require-archive-section
• python evals/check_release_metadata.py
• python evals/check_policy_markers.py
• python evals/check_semantic_cases.py
• python evals/check_skill_package.py
• runtime bundle scan: NO_RUNTIME_PYTHON_OR_TOOLING_MATCHES

Review follow-up

• Addressed SystemExit wrapper handling.
• Added .pyw, .pyc, .pyo, uppercase suffix, uppercase SCRIPTS/, uppercase TOOLS/, and docs/ package guards.
• Extended tests for Python-family artifacts, case-variant artifacts, repository-only docs, wrapper return behavior, runtime bundle contents, and the stricter 100-line SKILL.md cap.
• Made references/format.md runtime-neutral by removing direct tools/validate_watchlist.py guidance.
• Tightened the final storage wording nit and added updating to the skill trigger description.
• Ran multiple subagent review passes; final gate reported no blocking issues.

[gemini-code-assist]

Code Review

This pull request refactors the project to make the installable skill bundle entirely Python-free. It moves the WATCHLIST validator from the skill directory to the repository's maintainer tools (tools/validate_watchlist.py), introduces a new manual format reference (references/format.md), and updates documentation, tests, and packaging checks accordingly. The reviewer's feedback identifies opportunities to improve the robustness of the validator execution by handling SystemExit exceptions, and suggests extending the packaging checks and tests to explicitly forbid other Python-related extensions like .pyw.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.