Add final coverage, review evidence, HTML handler, ZIP return, and tool registration rules - PR_26126_060-project-instructions-final-gaps

Author: DavidQ

docs/dev/PROJECT_INSTRUCTIONS.md

@@ -618,3 +618,43 @@ Do NOT require:
 - full feature coverage
 - 100% code coverage
 - performance requirements
+
+## CODE COVERAGE CONTRACT
+
+When runtime JavaScript changes, Codex must produce a Playwright V8 coverage report.
+
+The coverage report must list changed runtime JavaScript files.
+
+Coverage report lines must start with coverage percentage in this format:
+
+`(xx%) <file-path> - <details>`
+
+Coverage is advisory unless a PR explicitly defines thresholds.
+
+Do not require 100% coverage.
+
+## CODEX ZIP RETURN CONTRACT
+
+Codex must include the repo-structured ZIP in returned artifacts for user and ChatGPT review.
+
+The ZIP must still follow the CODEX ZIP STANDARD.
+
+## CODE REVIEW EVIDENCE RULE
+
+ChatGPT must not claim code review was completed unless it inspected uploaded source, ZIP contents, or `codex_review.diff`.
+
+Pattern-based or process-based review must be labeled as such.
+
+## HTML INLINE HANDLER RESTRICTION
+
+HTML must not contain inline event handlers such as `onclick`, `onchange`, `oninput`, `onsubmit`, or similar.
+
+Event wiring must live in external JavaScript classes or modules.
+
+## FIRST-CLASS TOOL REGISTRATION RULE
+
+New first-class tools must include registry, index, and NAV wiring where applicable.
+
+New first-class tools must include Playwright launch coverage.
+
+Tool registration must not rely on hidden defaults or silent fallback.

docs/dev/reports/project_instructions_final_gaps.txt

@@ -0,0 +1,59 @@
+PR_26126_060-project-instructions-final-gaps
+
+Purpose:
+Append final project instruction gaps for code coverage, ZIP return wording, review evidence, HTML inline handler restrictions, and first-class tool registration.
+
+Appended sections:
+
+## CODE COVERAGE CONTRACT
+
+When runtime JavaScript changes, Codex must produce a Playwright V8 coverage report.
+
+The coverage report must list changed runtime JavaScript files.
+
+Coverage report lines must start with coverage percentage in this format:
+
+`(xx%) <file-path> - <details>`
+
+Coverage is advisory unless a PR explicitly defines thresholds.
+
+Do not require 100% coverage.
+
+## CODEX ZIP RETURN CONTRACT
+
+Codex must include the repo-structured ZIP in returned artifacts for user and ChatGPT review.
+
+The ZIP must still follow the CODEX ZIP STANDARD.
+
+## CODE REVIEW EVIDENCE RULE
+
+ChatGPT must not claim code review was completed unless it inspected uploaded source, ZIP contents, or `codex_review.diff`.
+
+Pattern-based or process-based review must be labeled as such.
+
+## HTML INLINE HANDLER RESTRICTION
+
+HTML must not contain inline event handlers such as `onclick`, `onchange`, `oninput`, `onsubmit`, or similar.
+
+Event wiring must live in external JavaScript classes or modules.
+
+## FIRST-CLASS TOOL REGISTRATION RULE
+
+New first-class tools must include registry, index, and NAV wiring where applicable.
+
+New first-class tools must include Playwright launch coverage.
+
+Tool registration must not rely on hidden defaults or silent fallback.
+
+
+Validation notes:
+- docs/dev/PROJECT_INSTRUCTIONS.md was the only target document edited.
+- Existing content was preserved; new sections were appended at the end of the file.
+- No roadmap, code, samples, schema, or start_of_day files were modified for this PR.
+- Playwright impacted: No.
+- No Playwright impact. This PR is docs/workflow only.
+
+Manual test notes:
+- Open docs/dev/PROJECT_INSTRUCTIONS.md and confirm the new final-gap sections appear at the end of the file.
+- Confirm the Code Coverage Contract states the Playwright V8 report format and advisory nature.
+- Confirm the ZIP, code review evidence, HTML inline handler, and first-class tool registration rules are clear and copy/paste ready.