Add Codex review diff artifact for ChatGPT code review without pre-commit hooks - PR_26124_014-include-review-diff

Author: DavidQ

docs/dev/codex_commands.md

@@ -327,3 +327,10 @@ PR_26124_009-empty-fail-batch-capture
 ```bash
 npx @openai/codex run --model gpt-5.3-codex --reasoning medium "Implement PR_26124_009: extract next FAIL tools from tool_completion_audit.md, record explicit batch fail list, and if no FAIL tools remain capture the empty batch with validation evidence and no runtime changes."
 ```
+
+---
+PR_26124_014-include-review-diff
+
+```bash
+npm run codex:review-artifacts
+```

docs/dev/reports/PR_26124_014_include_review_diff_report.md

@@ -0,0 +1,19 @@
+# PR_26124_014-include-review-diff
+
+## Summary
+- Added end-of-run artifact generator script for Codex PR review outputs.
+- Added npm command: `npm run codex:review-artifacts`.
+- Added command entry to `docs/dev/codex_commands.md`.
+
+## Files Changed
+- `scripts/write-codex-review-artifacts.mjs`
+- `package.json`
+- `docs/dev/codex_commands.md`
+
+## Validation
+- `node --check scripts/write-codex-review-artifacts.mjs` -> pass
+- `npm run codex:review-artifacts` -> pass
+
+## Artifact Confirmation
+- `docs/dev/reports/codex_review.diff` exists
+- `docs/dev/reports/codex_changed_files.txt` exists

docs/dev/reports/codex_changed_files.txt

@@ -0,0 +1,16 @@
+# git status --short
+M docs/dev/codex_commands.md
+ M package.json
+ D tests/playwright.zip
+ D tools/workspace-v2.zip
+?? docs/dev/reports/PR_26124_014_include_review_diff_report.md
+?? docs/dev/reports/codex_changed_files.txt
+?? docs/dev/reports/codex_review.diff
+?? scripts/write-codex-review-artifacts.mjs
+
+# git diff --stat
+docs/dev/codex_commands.md |   7 +++++++
+ package.json               |   1 +
+ tests/playwright.zip       | Bin 4491 -> 0 bytes
+ tools/workspace-v2.zip     | Bin 31477 -> 0 bytes
+ 4 files changed, 8 insertions(+)

docs/dev/reports/codex_review.diff

@@ -0,0 +1,35 @@
+diff --git a/docs/dev/codex_commands.md b/docs/dev/codex_commands.md
+index c99e0d6c..1ca60ee5 100644
+--- a/docs/dev/codex_commands.md
++++ b/docs/dev/codex_commands.md
+@@ -327,3 +327,10 @@ PR_26124_009-empty-fail-batch-capture
+ ```bash
+ npx @openai/codex run --model gpt-5.3-codex --reasoning medium "Implement PR_26124_009: extract next FAIL tools from tool_completion_audit.md, record explicit batch fail list, and if no FAIL tools remain capture the empty batch with validation evidence and no runtime changes."
+ ```
++
++---
++PR_26124_014-include-review-diff
++
++```bash
++npm run codex:review-artifacts
++```
+diff --git a/package.json b/package.json
+index 1e5462bd..533204d3 100644
+--- a/package.json
++++ b/package.json
+@@ -13,6 +13,7 @@
+     "check:games-template-contract": "node ./scripts/validate-games-template-contract.mjs",
+     "test:workspace-v2:playwright": "playwright test --workers=1 tests/ui/workspace-v2.asset-manager.spec.js tests/playwright/workspace-v2.validation.spec.js tests/playwright/tool-validation",
+     "test:workspace-v2": "node ./scripts/run-workspace-v2-playwright-gate.mjs",
++    "codex:review-artifacts": "node ./scripts/write-codex-review-artifacts.mjs",
+     "test:launch-smoke": "node ./tests/runtime/LaunchSmokeAllEntries.test.mjs",
+     "test:launch-smoke:games": "node ./tests/runtime/LaunchSmokeAllEntries.test.mjs --games",
+     "test:workspace-manager:games": "node ./tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs",
+diff --git a/tests/playwright.zip b/tests/playwright.zip
+deleted file mode 100644
+index 84f08898..00000000
+Binary files a/tests/playwright.zip and /dev/null differ
+diff --git a/tools/workspace-v2.zip b/tools/workspace-v2.zip
+deleted file mode 100644
+index 5047d4aa..00000000
+Binary files a/tools/workspace-v2.zip and /dev/null differ

package.json

@@ -13,6 +13,7 @@
     "check:games-template-contract": "node ./scripts/validate-games-template-contract.mjs",
     "test:workspace-v2:playwright": "playwright test --workers=1 tests/ui/workspace-v2.asset-manager.spec.js tests/playwright/workspace-v2.validation.spec.js tests/playwright/tool-validation",
     "test:workspace-v2": "node ./scripts/run-workspace-v2-playwright-gate.mjs",
+    "codex:review-artifacts": "node ./scripts/write-codex-review-artifacts.mjs",
     "test:launch-smoke": "node ./tests/runtime/LaunchSmokeAllEntries.test.mjs",
     "test:launch-smoke:games": "node ./tests/runtime/LaunchSmokeAllEntries.test.mjs --games",
     "test:workspace-manager:games": "node ./tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs",

scripts/write-codex-review-artifacts.mjs

@@ -0,0 +1,61 @@
+import { spawnSync } from "node:child_process";
+import { fileURLToPath } from "node:url";
+import { dirname, resolve } from "node:path";
+import { mkdirSync, writeFileSync } from "node:fs";
+
+const repoRoot = resolve(dirname(fileURLToPath(import.meta.url)), "..");
+const reportsDirectory = resolve(repoRoot, "docs", "dev", "reports");
+const reviewDiffPath = resolve(reportsDirectory, "codex_review.diff");
+const changedFilesPath = resolve(reportsDirectory, "codex_changed_files.txt");
+
+function runGit(args) {
+  const result = spawnSync("git", args, {
+    cwd: repoRoot,
+    encoding: "utf8"
+  });
+  if (result.error) {
+    throw result.error;
+  }
+  return result;
+}
+
+function assertGitCommand(result, args) {
+  if (result.status !== 0) {
+    const commandText = `git ${args.join(" ")}`;
+    const stderr = result.stderr && result.stderr.trim() ? result.stderr.trim() : "Unknown git error";
+    throw new Error(`${commandText} failed: ${stderr}`);
+  }
+}
+
+const stagedCheck = runGit(["diff", "--cached", "--quiet"]);
+if (stagedCheck.status !== 0 && stagedCheck.status !== 1) {
+  const stderr = stagedCheck.stderr && stagedCheck.stderr.trim() ? stagedCheck.stderr.trim() : "Unknown git error";
+  throw new Error(`git diff --cached --quiet failed: ${stderr}`);
+}
+
+const useCachedDiff = stagedCheck.status === 1;
+const reviewDiffArguments = useCachedDiff ? ["diff", "--cached"] : ["diff"];
+const reviewDiffResult = runGit(reviewDiffArguments);
+assertGitCommand(reviewDiffResult, reviewDiffArguments);
+
+const statusShortResult = runGit(["status", "--short"]);
+assertGitCommand(statusShortResult, ["status", "--short"]);
+
+const diffStatResult = runGit(["diff", "--stat"]);
+assertGitCommand(diffStatResult, ["diff", "--stat"]);
+
+mkdirSync(reportsDirectory, { recursive: true });
+writeFileSync(reviewDiffPath, reviewDiffResult.stdout, "utf8");
+
+const changedFilesReport = [
+  `# git status --short`,
+  statusShortResult.stdout.trim() ? statusShortResult.stdout.trim() : "(no output)",
+  "",
+  `# git diff --stat`,
+  diffStatResult.stdout.trim() ? diffStatResult.stdout.trim() : "(no output)"
+].join("\n");
+
+writeFileSync(changedFilesPath, changedFilesReport, "utf8");
+
+console.log(`Wrote ${reviewDiffPath} using git ${reviewDiffArguments.join(" ")}`);
+console.log(`Wrote ${changedFilesPath}`);