@webpieces/pr-gate 0.3.354 → 0.3.356

package/src/scripts/workflow/build-affected.js

@@ -1,55 +1,23 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.BuildGateOptions = exports.DEFAULT_BUILD_COMMAND = void 0;
-exports.resolveBuildCommand = resolveBuildCommand;
-exports.runBuildAffected = runBuildAffected;
-exports.runConfiguredBuildGate = runConfiguredBuildGate;
-exports.runBuildGate = runBuildGate;
+exports.BuildAffected = exports.BuildGateOptions = exports.DEFAULT_BUILD_COMMAND = void 0;
+const tslib_1 = require("tslib");
 const child_process_1 = require("child_process");
 const rules_config_1 = require("@webpieces/rules-config");
-// Single source of truth for the build gate. The PR flow's advisory gate (`wp-start-upsert-pr`)
-// and authoritative merge gate (`wp-finish-upsert-pr`) both run THIS, so the two can never drift.
-// nx `affected` only rebuilds changed projects, so this stays fast on a large monorepo.
-// `--base=$(git merge-base origin/main HEAD)` (the fork point) instead of `--base=origin/main`:
-// origin/main rebuilds projects touched by OTHER people's merged PRs (your branch still holds the
-// pre-merge versions, so nx sees them as "affected"). The fork point scopes affected to only YOUR
-// branch's changes. The `$(...)` resolves because runBuildAffected runs with shell: true.
+const core_context_1 = require("@webpieces/core-context");
+const inversify_1 = require("inversify");
+// Single source of truth for the build gate. The PR flow's advisory gate (`wp-start-upsert-pr`) and
+// authoritative merge gate (`wp-finish-upsert-pr`) both run THIS, so the two can never drift. nx
+// `affected` only rebuilds changed projects. `--base=$(git merge-base origin/main HEAD)` (the fork
+// point) instead of `--base=origin/main`: origin/main rebuilds projects touched by OTHER people's
+// merged PRs. The fork point scopes affected to only YOUR branch's changes. The `$(...)` resolves
+// because runBuildAffected runs with shell: true.
 exports.DEFAULT_BUILD_COMMAND = 'pnpm nx affected --target=ci --base=$(git merge-base origin/main HEAD)';
-/**
- * Resolve the exact build command this gate will run for a repo: the project's configured
- * PrGateConfig.buildCommand, or the default affected-ci command when none is set. Callers
- * print this so the AI knows precisely which command to run locally to get the gate passing.
- */
-function resolveBuildCommand(repoRoot) {
-    const configured = (0, rules_config_1.loadAndValidate)(repoRoot).prGate.buildCommand;
-    return configured !== undefined && configured.trim() !== '' ? configured : exports.DEFAULT_BUILD_COMMAND;
-}
-/**
- * Run the build gate. Returns the process exit code (0 = pass). `buildCommand` overrides
- * the default (sourced from PrGateConfig.buildCommand by callers); empty/undefined uses the
- * default affected-ci command.
- */
-function runBuildAffected(repoRoot, buildCommand) {
-    const cmd = buildCommand !== undefined && buildCommand.trim() !== '' ? buildCommand : exports.DEFAULT_BUILD_COMMAND;
-    process.stdout.write(`\n▶ Build gate: ${cmd}\n\n`);
-    const result = (0, child_process_1.spawnSync)(cmd, { stdio: 'inherit', cwd: repoRoot, shell: true });
-    return result.status ?? 1;
-}
-/**
- * Run the build gate using the project's configured command (PrGateConfig.buildCommand). The
- * finish-upsert-pr gate and wp-start-upsert-pr both call THIS so they provably build with the same
- * command — the resolution the AI validates is built exactly as the PR command builds it.
- * Returns the exit code; callers print their own re-run hint.
- */
-function runConfiguredBuildGate(repoRoot) {
-    return runBuildAffected(repoRoot, (0, rules_config_1.loadAndValidate)(repoRoot).prGate.buildCommand);
-}
 const SEP = '━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n';
 /**
- * The two facts that differ between the wp-start (advisory) and wp-finish (authoritative) build
- * gates. Everything else — resolve command, run it, print the re-run hint, exit non-zero on failure
- * — is identical, so it lives once in runBuildGate and the difference is passed in here. A class (not
- * an object literal) per the codebase's data-structure convention.
+ * The two facts that differ between the wp-start (advisory) and wp-finish (authoritative) build gates.
+ * Everything else is identical, so it lives once in runBuildGate and the difference is passed in here.
+ * A class (not an object literal) per the codebase's data-structure convention.
  */
 class BuildGateOptions {
     label; // section header shown above the gate
@@ -62,23 +30,49 @@ class BuildGateOptions {
     }
 }
 exports.BuildGateOptions = BuildGateOptions;
-/**
- * Run the configured build gate with consistent framing, throwing CliExitError(buildCode) on failure
- * so the bin's main()/runMain owns the exit. Single source of truth: wp-start-upsert-pr and
- * wp-finish-upsert-pr both call THIS (only the BuildGateOptions differ) so a fix to the gate flow is
- * made once, not in two near-identical copies.
- */
-function runBuildGate(repoRoot, opts) {
-    const buildCommand = resolveBuildCommand(repoRoot);
-    process.stdout.write('\n' + SEP + opts.label + '\n' + SEP + '\n');
-    process.stdout.write(`Running the build gate. To get it passing, run the SAME command yourself and fix everything it reports:\n\n` +
-        `    ${buildCommand}\n\n`);
-    const buildCode = runConfiguredBuildGate(repoRoot);
-    if (buildCode !== 0) {
-        throw new rules_config_1.CliExitError(buildCode, `\n❌ ${opts.failureHeadline}\n\n` +
-            `Run THIS exact command to reproduce and fix all errors, then re-run ${opts.rerunCommand}:\n\n` +
-            `    ${buildCommand}\n`);
+/** Runs the nx-affected build gate the same way for the advisory and authoritative PR gates. */
+let BuildAffected = class BuildAffected {
+    /**
+     * Resolve the exact build command this gate will run: the project's configured
+     * PrGateConfig.buildCommand, or the default affected-ci command when none is set.
+     */
+    resolveBuildCommand(repoRoot) {
+        const configured = (0, rules_config_1.loadAndValidate)(repoRoot).prGate.buildCommand;
+        return configured !== undefined && configured.trim() !== '' ? configured : exports.DEFAULT_BUILD_COMMAND;
     }
-    process.stdout.write('\n✅ Build passed.\n');
-}
+    /** Run the build gate. Returns the process exit code (0 = pass). */
+    runBuildAffected(repoRoot, buildCommand) {
+        const cmd = buildCommand !== undefined && buildCommand.trim() !== '' ? buildCommand : exports.DEFAULT_BUILD_COMMAND;
+        process.stdout.write(`\n▶ Build gate: ${cmd}\n\n`);
+        const result = (0, child_process_1.spawnSync)(cmd, { stdio: 'inherit', cwd: repoRoot, shell: true });
+        return result.status ?? 1;
+    }
+    /** Run the build gate using the project's configured command (PrGateConfig.buildCommand). */
+    runConfiguredBuildGate(repoRoot) {
+        return this.runBuildAffected(repoRoot, (0, rules_config_1.loadAndValidate)(repoRoot).prGate.buildCommand);
+    }
+    /**
+     * Run the configured build gate with consistent framing, throwing CliExitError(buildCode) on
+     * failure so the bin's main()/runMain owns the exit. Single source of truth: wp-start-upsert-pr and
+     * wp-finish-upsert-pr both call THIS (only the BuildGateOptions differ).
+     */
+    runBuildGate(repoRoot, opts) {
+        const buildCommand = this.resolveBuildCommand(repoRoot);
+        process.stdout.write('\n' + SEP + opts.label + '\n' + SEP + '\n');
+        process.stdout.write(`Running the build gate. To get it passing, run the SAME command yourself and fix everything it reports:\n\n` +
+            `    ${buildCommand}\n\n`);
+        const buildCode = this.runConfiguredBuildGate(repoRoot);
+        if (buildCode !== 0) {
+            throw new rules_config_1.CliExitError(buildCode, `\n❌ ${opts.failureHeadline}\n\n` +
+                `Run THIS exact command to reproduce and fix all errors, then re-run ${opts.rerunCommand}:\n\n` +
+                `    ${buildCommand}\n`);
+        }
+        process.stdout.write('\n✅ Build passed.\n');
+    }
+};
+exports.BuildAffected = BuildAffected;
+exports.BuildAffected = BuildAffected = tslib_1.__decorate([
+    (0, core_context_1.provideSingleton)(),
+    (0, inversify_1.injectable)()
+], BuildAffected);
 //# sourceMappingURL=build-affected.js.map

package/src/scripts/workflow/build-affected.js.map

@@ -1 +1 @@
-{"version":3,"file":"build-affected.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/pr-gate/src/scripts/workflow/build-affected.ts"],"names":[],"mappings":";;;AAiBA,kDAGC;AAOD,4CAKC;AAQD,wDAEC;AA4BD,oCAgBC;AAtFD,iDAA0C;AAC1C,0DAAwE;AAExE,gGAAgG;AAChG,kGAAkG;AAClG,wFAAwF;AACxF,gGAAgG;AAChG,kGAAkG;AAClG,kGAAkG;AAClG,0FAA0F;AAC7E,QAAA,qBAAqB,GAAG,wEAAwE,CAAC;AAE9G;;;;GAIG;AACH,SAAgB,mBAAmB,CAAC,QAAgB;IAChD,MAAM,UAAU,GAAG,IAAA,8BAAe,EAAC,QAAQ,CAAC,CAAC,MAAM,CAAC,YAAY,CAAC;IACjE,OAAO,UAAU,KAAK,SAAS,IAAI,UAAU,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,6BAAqB,CAAC;AACrG,CAAC;AAED;;;;GAIG;AACH,SAAgB,gBAAgB,CAAC,QAAgB,EAAE,YAAqB;IACpE,MAAM,GAAG,GAAG,YAAY,KAAK,SAAS,IAAI,YAAY,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,6BAAqB,CAAC;IAC5G,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,mBAAmB,GAAG,MAAM,CAAC,CAAC;IACnD,MAAM,MAAM,GAAG,IAAA,yBAAS,EAAC,GAAG,EAAE,EAAE,KAAK,EAAE,SAAS,EAAE,GAAG,EAAE,QAAQ,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IAChF,OAAO,MAAM,CAAC,MAAM,IAAI,CAAC,CAAC;AAC9B,CAAC;AAED;;;;;GAKG;AACH,SAAgB,sBAAsB,CAAC,QAAgB;IACnD,OAAO,gBAAgB,CAAC,QAAQ,EAAE,IAAA,8BAAe,EAAC,QAAQ,CAAC,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC;AACrF,CAAC;AAED,MAAM,GAAG,GAAG,0DAA0D,CAAC;AAEvE;;;;;GAKG;AACH,MAAa,gBAAgB;IACzB,KAAK,CAAS,CAAY,sCAAsC;IAChE,YAAY,CAAS,CAAK,gDAAgD;IAC1E,eAAe,CAAS,CAAE,gCAAgC;IAE1D,YAAY,KAAa,EAAE,YAAoB,EAAE,eAAuB;QACpE,IAAI,CAAC,KAAK,GAAG,KAAK,CAAC;QACnB,IAAI,CAAC,YAAY,GAAG,YAAY,CAAC;QACjC,IAAI,CAAC,eAAe,GAAG,eAAe,CAAC;IAC3C,CAAC;CACJ;AAVD,4CAUC;AAED;;;;;GAKG;AACH,SAAgB,YAAY,CAAC,QAAgB,EAAE,IAAsB;IACjE,MAAM,YAAY,GAAG,mBAAmB,CAAC,QAAQ,CAAC,CAAC;IACnD,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,GAAG,GAAG,GAAG,IAAI,CAAC,KAAK,GAAG,IAAI,GAAG,GAAG,GAAG,IAAI,CAAC,CAAC;IAClE,OAAO,CAAC,MAAM,CAAC,KAAK,CAChB,6GAA6G;QAC7G,OAAO,YAAY,MAAM,CAC5B,CAAC;IACF,MAAM,SAAS,GAAG,sBAAsB,CAAC,QAAQ,CAAC,CAAC;IACnD,IAAI,SAAS,KAAK,CAAC,EAAE,CAAC;QAClB,MAAM,IAAI,2BAAY,CAAC,SAAS,EAC5B,OAAO,IAAI,CAAC,eAAe,MAAM;YACjC,uEAAuE,IAAI,CAAC,YAAY,OAAO;YAC/F,OAAO,YAAY,IAAI,CAC1B,CAAC;IACN,CAAC;IACD,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC;AAChD,CAAC","sourcesContent":["import { spawnSync } from 'child_process';\nimport { loadAndValidate, CliExitError } from '@webpieces/rules-config';\n\n// Single source of truth for the build gate. The PR flow's advisory gate (`wp-start-upsert-pr`)\n// and authoritative merge gate (`wp-finish-upsert-pr`) both run THIS, so the two can never drift.\n// nx `affected` only rebuilds changed projects, so this stays fast on a large monorepo.\n// `--base=$(git merge-base origin/main HEAD)` (the fork point) instead of `--base=origin/main`:\n// origin/main rebuilds projects touched by OTHER people's merged PRs (your branch still holds the\n// pre-merge versions, so nx sees them as \"affected\"). The fork point scopes affected to only YOUR\n// branch's changes. The `$(...)` resolves because runBuildAffected runs with shell: true.\nexport const DEFAULT_BUILD_COMMAND = 'pnpm nx affected --target=ci --base=$(git merge-base origin/main HEAD)';\n\n/**\n * Resolve the exact build command this gate will run for a repo: the project's configured\n * PrGateConfig.buildCommand, or the default affected-ci command when none is set. Callers\n * print this so the AI knows precisely which command to run locally to get the gate passing.\n */\nexport function resolveBuildCommand(repoRoot: string): string {\n    const configured = loadAndValidate(repoRoot).prGate.buildCommand;\n    return configured !== undefined && configured.trim() !== '' ? configured : DEFAULT_BUILD_COMMAND;\n}\n\n/**\n * Run the build gate. Returns the process exit code (0 = pass). `buildCommand` overrides\n * the default (sourced from PrGateConfig.buildCommand by callers); empty/undefined uses the\n * default affected-ci command.\n */\nexport function runBuildAffected(repoRoot: string, buildCommand?: string): number {\n    const cmd = buildCommand !== undefined && buildCommand.trim() !== '' ? buildCommand : DEFAULT_BUILD_COMMAND;\n    process.stdout.write(`\\n▶ Build gate: ${cmd}\\n\\n`);\n    const result = spawnSync(cmd, { stdio: 'inherit', cwd: repoRoot, shell: true });\n    return result.status ?? 1;\n}\n\n/**\n * Run the build gate using the project's configured command (PrGateConfig.buildCommand). The\n * finish-upsert-pr gate and wp-start-upsert-pr both call THIS so they provably build with the same\n * command — the resolution the AI validates is built exactly as the PR command builds it.\n * Returns the exit code; callers print their own re-run hint.\n */\nexport function runConfiguredBuildGate(repoRoot: string): number {\n    return runBuildAffected(repoRoot, loadAndValidate(repoRoot).prGate.buildCommand);\n}\n\nconst SEP = '━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\\n';\n\n/**\n * The two facts that differ between the wp-start (advisory) and wp-finish (authoritative) build\n * gates. Everything else — resolve command, run it, print the re-run hint, exit non-zero on failure\n * — is identical, so it lives once in runBuildGate and the difference is passed in here. A class (not\n * an object literal) per the codebase's data-structure convention.\n */\nexport class BuildGateOptions {\n    label: string;            // section header shown above the gate\n    rerunCommand: string;     // command the AI re-runs after fixing the build\n    failureHeadline: string;  // first line printed on failure\n\n    constructor(label: string, rerunCommand: string, failureHeadline: string) {\n        this.label = label;\n        this.rerunCommand = rerunCommand;\n        this.failureHeadline = failureHeadline;\n    }\n}\n\n/**\n * Run the configured build gate with consistent framing, throwing CliExitError(buildCode) on failure\n * so the bin's main()/runMain owns the exit. Single source of truth: wp-start-upsert-pr and\n * wp-finish-upsert-pr both call THIS (only the BuildGateOptions differ) so a fix to the gate flow is\n * made once, not in two near-identical copies.\n */\nexport function runBuildGate(repoRoot: string, opts: BuildGateOptions): void {\n    const buildCommand = resolveBuildCommand(repoRoot);\n    process.stdout.write('\\n' + SEP + opts.label + '\\n' + SEP + '\\n');\n    process.stdout.write(\n        `Running the build gate. To get it passing, run the SAME command yourself and fix everything it reports:\\n\\n` +\n        `    ${buildCommand}\\n\\n`,\n    );\n    const buildCode = runConfiguredBuildGate(repoRoot);\n    if (buildCode !== 0) {\n        throw new CliExitError(buildCode,\n            `\\n❌ ${opts.failureHeadline}\\n\\n` +\n            `Run THIS exact command to reproduce and fix all errors, then re-run ${opts.rerunCommand}:\\n\\n` +\n            `    ${buildCommand}\\n`,\n        );\n    }\n    process.stdout.write('\\n✅ Build passed.\\n');\n}\n"]}
+{"version":3,"file":"build-affected.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/pr-gate/src/scripts/workflow/build-affected.ts"],"names":[],"mappings":";;;;AAAA,iDAA0C;AAC1C,0DAAwE;AACxE,0DAA2D;AAC3D,yCAAuC;AAEvC,oGAAoG;AACpG,iGAAiG;AACjG,mGAAmG;AACnG,kGAAkG;AAClG,kGAAkG;AAClG,kDAAkD;AACrC,QAAA,qBAAqB,GAAG,wEAAwE,CAAC;AAE9G,MAAM,GAAG,GAAG,0DAA0D,CAAC;AAEvE;;;;GAIG;AACH,MAAa,gBAAgB;IACzB,KAAK,CAAS,CAAY,sCAAsC;IAChE,YAAY,CAAS,CAAK,gDAAgD;IAC1E,eAAe,CAAS,CAAE,gCAAgC;IAE1D,YAAY,KAAa,EAAE,YAAoB,EAAE,eAAuB;QACpE,IAAI,CAAC,KAAK,GAAG,KAAK,CAAC;QACnB,IAAI,CAAC,YAAY,GAAG,YAAY,CAAC;QACjC,IAAI,CAAC,eAAe,GAAG,eAAe,CAAC;IAC3C,CAAC;CACJ;AAVD,4CAUC;AAED,gGAAgG;AAGzF,IAAM,aAAa,GAAnB,MAAM,aAAa;IACtB;;;OAGG;IACH,mBAAmB,CAAC,QAAgB;QAChC,MAAM,UAAU,GAAG,IAAA,8BAAe,EAAC,QAAQ,CAAC,CAAC,MAAM,CAAC,YAAY,CAAC;QACjE,OAAO,UAAU,KAAK,SAAS,IAAI,UAAU,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,6BAAqB,CAAC;IACrG,CAAC;IAED,oEAAoE;IACpE,gBAAgB,CAAC,QAAgB,EAAE,YAAqB;QACpD,MAAM,GAAG,GAAG,YAAY,KAAK,SAAS,IAAI,YAAY,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,6BAAqB,CAAC;QAC5G,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,mBAAmB,GAAG,MAAM,CAAC,CAAC;QACnD,MAAM,MAAM,GAAG,IAAA,yBAAS,EAAC,GAAG,EAAE,EAAE,KAAK,EAAE,SAAS,EAAE,GAAG,EAAE,QAAQ,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAChF,OAAO,MAAM,CAAC,MAAM,IAAI,CAAC,CAAC;IAC9B,CAAC;IAED,6FAA6F;IAC7F,sBAAsB,CAAC,QAAgB;QACnC,OAAO,IAAI,CAAC,gBAAgB,CAAC,QAAQ,EAAE,IAAA,8BAAe,EAAC,QAAQ,CAAC,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC;IAC1F,CAAC;IAED;;;;OAIG;IACH,YAAY,CAAC,QAAgB,EAAE,IAAsB;QACjD,MAAM,YAAY,GAAG,IAAI,CAAC,mBAAmB,CAAC,QAAQ,CAAC,CAAC;QACxD,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,GAAG,GAAG,GAAG,IAAI,CAAC,KAAK,GAAG,IAAI,GAAG,GAAG,GAAG,IAAI,CAAC,CAAC;QAClE,OAAO,CAAC,MAAM,CAAC,KAAK,CAChB,6GAA6G;YAC7G,OAAO,YAAY,MAAM,CAC5B,CAAC;QACF,MAAM,SAAS,GAAG,IAAI,CAAC,sBAAsB,CAAC,QAAQ,CAAC,CAAC;QACxD,IAAI,SAAS,KAAK,CAAC,EAAE,CAAC;YAClB,MAAM,IAAI,2BAAY,CAAC,SAAS,EAC5B,OAAO,IAAI,CAAC,eAAe,MAAM;gBACjC,uEAAuE,IAAI,CAAC,YAAY,OAAO;gBAC/F,OAAO,YAAY,IAAI,CAC1B,CAAC;QACN,CAAC;QACD,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,qBAAqB,CAAC,CAAC;IAChD,CAAC;CACJ,CAAA;AA7CY,sCAAa;wBAAb,aAAa;IAFzB,IAAA,+BAAgB,GAAE;IAClB,IAAA,sBAAU,GAAE;GACA,aAAa,CA6CzB","sourcesContent":["import { spawnSync } from 'child_process';\nimport { loadAndValidate, CliExitError } from '@webpieces/rules-config';\nimport { provideSingleton } from '@webpieces/core-context';\nimport { injectable } from 'inversify';\n\n// Single source of truth for the build gate. The PR flow's advisory gate (`wp-start-upsert-pr`) and\n// authoritative merge gate (`wp-finish-upsert-pr`) both run THIS, so the two can never drift. nx\n// `affected` only rebuilds changed projects. `--base=$(git merge-base origin/main HEAD)` (the fork\n// point) instead of `--base=origin/main`: origin/main rebuilds projects touched by OTHER people's\n// merged PRs. The fork point scopes affected to only YOUR branch's changes. The `$(...)` resolves\n// because runBuildAffected runs with shell: true.\nexport const DEFAULT_BUILD_COMMAND = 'pnpm nx affected --target=ci --base=$(git merge-base origin/main HEAD)';\n\nconst SEP = '━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\\n';\n\n/**\n * The two facts that differ between the wp-start (advisory) and wp-finish (authoritative) build gates.\n * Everything else is identical, so it lives once in runBuildGate and the difference is passed in here.\n * A class (not an object literal) per the codebase's data-structure convention.\n */\nexport class BuildGateOptions {\n    label: string;            // section header shown above the gate\n    rerunCommand: string;     // command the AI re-runs after fixing the build\n    failureHeadline: string;  // first line printed on failure\n\n    constructor(label: string, rerunCommand: string, failureHeadline: string) {\n        this.label = label;\n        this.rerunCommand = rerunCommand;\n        this.failureHeadline = failureHeadline;\n    }\n}\n\n/** Runs the nx-affected build gate the same way for the advisory and authoritative PR gates. */\n@provideSingleton()\n@injectable()\nexport class BuildAffected {\n    /**\n     * Resolve the exact build command this gate will run: the project's configured\n     * PrGateConfig.buildCommand, or the default affected-ci command when none is set.\n     */\n    resolveBuildCommand(repoRoot: string): string {\n        const configured = loadAndValidate(repoRoot).prGate.buildCommand;\n        return configured !== undefined && configured.trim() !== '' ? configured : DEFAULT_BUILD_COMMAND;\n    }\n\n    /** Run the build gate. Returns the process exit code (0 = pass). */\n    runBuildAffected(repoRoot: string, buildCommand?: string): number {\n        const cmd = buildCommand !== undefined && buildCommand.trim() !== '' ? buildCommand : DEFAULT_BUILD_COMMAND;\n        process.stdout.write(`\\n▶ Build gate: ${cmd}\\n\\n`);\n        const result = spawnSync(cmd, { stdio: 'inherit', cwd: repoRoot, shell: true });\n        return result.status ?? 1;\n    }\n\n    /** Run the build gate using the project's configured command (PrGateConfig.buildCommand). */\n    runConfiguredBuildGate(repoRoot: string): number {\n        return this.runBuildAffected(repoRoot, loadAndValidate(repoRoot).prGate.buildCommand);\n    }\n\n    /**\n     * Run the configured build gate with consistent framing, throwing CliExitError(buildCode) on\n     * failure so the bin's main()/runMain owns the exit. Single source of truth: wp-start-upsert-pr and\n     * wp-finish-upsert-pr both call THIS (only the BuildGateOptions differ).\n     */\n    runBuildGate(repoRoot: string, opts: BuildGateOptions): void {\n        const buildCommand = this.resolveBuildCommand(repoRoot);\n        process.stdout.write('\\n' + SEP + opts.label + '\\n' + SEP + '\\n');\n        process.stdout.write(\n            `Running the build gate. To get it passing, run the SAME command yourself and fix everything it reports:\\n\\n` +\n            `    ${buildCommand}\\n\\n`,\n        );\n        const buildCode = this.runConfiguredBuildGate(repoRoot);\n        if (buildCode !== 0) {\n            throw new CliExitError(buildCode,\n                `\\n❌ ${opts.failureHeadline}\\n\\n` +\n                `Run THIS exact command to reproduce and fix all errors, then re-run ${opts.rerunCommand}:\\n\\n` +\n                `    ${buildCommand}\\n`,\n            );\n        }\n        process.stdout.write('\\n✅ Build passed.\\n');\n    }\n}\n"]}

package/src/scripts/workflow/cleanTmp.d.ts

@@ -1 +1,6 @@
-export declare function cleanTmp(): Promise<void>;
+/** 30-day cleanup of stale per-feature merge/pr-review dirs (and the legacy flat layout). */
+export declare class CleanTmp {
+    cleanTmp(): Promise<void>;
+    private cleanStaleSubdirs;
+    private cleanLegacyTopLevel;
+}

package/src/scripts/workflow/cleanTmp.js

@@ -1,93 +1,98 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.cleanTmp = cleanTmp;
+exports.CleanTmp = void 0;
 const tslib_1 = require("tslib");
 const child_process_1 = require("child_process");
 const fs = tslib_1.__importStar(require("fs"));
 const path = tslib_1.__importStar(require("path"));
 const rules_config_1 = require("@webpieces/rules-config");
+const core_context_1 = require("@webpieces/core-context");
+const inversify_1 = require("inversify");
 const CUTOFF_DAYS = 30;
 const SEP = '━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n';
-// Per-feature workflow dirs now live under `.webpieces/merge-info/<feature>` and
-// `.webpieces/pr-review/<feature>`; those two homes are permanent (like hooks/ and instruct-ai/) and
-// only their stale subdirs are cleaned. `LEGACY_PREFIXES` sweeps the old flat top-level layout
-// (`merge-<feature>`/`review-<feature>`/`pr-<feature>`) so it self-clears after the move — which also
-// reaps the retired `pr-info/` home (it matches the `pr-` prefix and is no longer the active PR home).
+// Per-feature workflow dirs live under `.webpieces/merge-info/<feature>` and `.webpieces/pr-review/
+// <feature>`; those two homes are permanent (like hooks/ and instruct-ai/) and only their stale
+// subdirs are cleaned. `LEGACY_PREFIXES` sweeps the old flat top-level layout so it self-clears — which
+// also reaps the retired `pr-info/` home (it matches `pr-` and is no longer the active PR home).
 const LEGACY_PREFIXES = ['merge-', 'review-', 'pr-'];
-async function cleanTmp() {
-    const repoRoot = (0, child_process_1.execSync)('git rev-parse --show-toplevel', { encoding: 'utf8' }).trim();
-    const tmpBase = path.join(repoRoot, rules_config_1.WEBPIECES_TMP_DIR);
-    if (!fs.existsSync(tmpBase)) {
-        return;
-    }
-    process.stdout.write('\n');
-    process.stdout.write(SEP);
-    process.stdout.write('🧹 Cleaning Old Temporary Directories\n');
-    process.stdout.write(SEP);
-    process.stdout.write('\n');
-    process.stdout.write(`Location: ${tmpBase}\n`);
-    process.stdout.write(`Retention: ${CUTOFF_DAYS} days\n`);
-    process.stdout.write('\n');
-    const cutoffMs = CUTOFF_DAYS * 24 * 60 * 60 * 1000;
-    const now = Date.now();
-    let deletedCount = 0;
-    // Stale per-feature subdirs under each permanent home.
-    deletedCount += cleanStaleSubdirs(path.join(tmpBase, rules_config_1.MERGE_INFO_DIR), rules_config_1.MERGE_INFO_DIR, now, cutoffMs);
-    deletedCount += cleanStaleSubdirs(path.join(tmpBase, rules_config_1.PR_REVIEW_DIR), rules_config_1.PR_REVIEW_DIR, now, cutoffMs);
-    // Legacy flat top-level dirs from before the merge-info/pr-review nesting (also reaps old pr-info/).
-    deletedCount += cleanLegacyTopLevel(tmpBase, now, cutoffMs);
-    if (deletedCount === 0) {
-        process.stdout.write(`  ✅ No directories older than ${CUTOFF_DAYS} days found\n`);
-    }
-    else {
+/** 30-day cleanup of stale per-feature merge/pr-review dirs (and the legacy flat layout). */
+let CleanTmp = class CleanTmp {
+    async cleanTmp() {
+        const repoRoot = (0, child_process_1.execSync)('git rev-parse --show-toplevel', { encoding: 'utf8' }).trim();
+        const tmpBase = path.join(repoRoot, rules_config_1.WEBPIECES_TMP_DIR);
+        if (!fs.existsSync(tmpBase)) {
+            return;
+        }
+        process.stdout.write('\n');
+        process.stdout.write(SEP);
+        process.stdout.write('🧹 Cleaning Old Temporary Directories\n');
+        process.stdout.write(SEP);
+        process.stdout.write('\n');
+        process.stdout.write(`Location: ${tmpBase}\n`);
+        process.stdout.write(`Retention: ${CUTOFF_DAYS} days\n`);
+        process.stdout.write('\n');
+        const cutoffMs = CUTOFF_DAYS * 24 * 60 * 60 * 1000;
+        const now = Date.now();
+        let deletedCount = 0;
+        // Stale per-feature subdirs under each permanent home.
+        deletedCount += this.cleanStaleSubdirs(path.join(tmpBase, rules_config_1.MERGE_INFO_DIR), rules_config_1.MERGE_INFO_DIR, now, cutoffMs);
+        deletedCount += this.cleanStaleSubdirs(path.join(tmpBase, rules_config_1.PR_REVIEW_DIR), rules_config_1.PR_REVIEW_DIR, now, cutoffMs);
+        // Legacy flat top-level dirs from before the merge-info/pr-review nesting (also reaps old pr-info/).
+        deletedCount += this.cleanLegacyTopLevel(tmpBase, now, cutoffMs);
+        if (deletedCount === 0) {
+            process.stdout.write(`  ✅ No directories older than ${CUTOFF_DAYS} days found\n`);
+        }
+        else {
+            process.stdout.write('\n');
+            process.stdout.write(`  ✅ Deleted ${deletedCount} old director${deletedCount === 1 ? 'y' : 'ies'}\n`);
+        }
+        process.stdout.write('\n');
+        process.stdout.write(SEP);
         process.stdout.write('\n');
-        process.stdout.write(`  ✅ Deleted ${deletedCount} old director${deletedCount === 1 ? 'y' : 'ies'}\n`);
     }
-    process.stdout.write('\n');
-    process.stdout.write(SEP);
-    process.stdout.write('\n');
-}
-// Delete immediate subdirs of `parentDir` whose mtime is older than the cutoff. `label` prefixes
-// the printed path so output reads e.g. `merge-info/dean-foo`. No-op if the home doesn't exist.
-function cleanStaleSubdirs(parentDir, label, now, cutoffMs) {
-    if (!fs.existsSync(parentDir))
-        return 0;
-    let count = 0;
-    for (const entry of fs.readdirSync(parentDir)) {
-        const fullPath = path.join(parentDir, entry);
-        const stat = fs.statSync(fullPath);
-        if (!stat.isDirectory())
-            continue;
-        if (now - stat.mtimeMs < cutoffMs)
-            continue;
-        process.stdout.write(`  🗑️  Deleting: ${label}/${entry}\n`);
-        fs.rmSync(fullPath, { recursive: true, force: true });
-        count += 1;
+    // Delete immediate subdirs of `parentDir` whose mtime is older than the cutoff. No-op if absent.
+    cleanStaleSubdirs(parentDir, label, now, cutoffMs) {
+        if (!fs.existsSync(parentDir))
+            return 0;
+        let count = 0;
+        for (const entry of fs.readdirSync(parentDir)) {
+            const fullPath = path.join(parentDir, entry);
+            const stat = fs.statSync(fullPath);
+            if (!stat.isDirectory())
+                continue;
+            if (now - stat.mtimeMs < cutoffMs)
+                continue;
+            process.stdout.write(`  🗑️  Deleting: ${label}/${entry}\n`);
+            fs.rmSync(fullPath, { recursive: true, force: true });
+            count += 1;
+        }
+        return count;
     }
-    return count;
-}
-// Sweep the pre-nesting flat layout (`merge-<feature>` etc.), skipping the current homes (whose
-// names also start with `merge-`/`pr-`) so this never deletes merge-info/ or pr-review/ themselves.
-// The retired `pr-info/` home is deliberately NOT skipped — it matches `pr-` and self-clears here.
-function cleanLegacyTopLevel(tmpBase, now, cutoffMs) {
-    let count = 0;
-    for (const entry of fs.readdirSync(tmpBase)) {
-        if (entry === rules_config_1.MERGE_INFO_DIR || entry === rules_config_1.PR_REVIEW_DIR)
-            continue;
-        if (!LEGACY_PREFIXES.some((prefix) => entry.startsWith(prefix)))
-            continue;
-        const fullPath = path.join(tmpBase, entry);
-        const stat = fs.statSync(fullPath);
-        if (!stat.isDirectory())
-            continue;
-        if (now - stat.mtimeMs < cutoffMs)
-            continue;
-        process.stdout.write(`  🗑️  Deleting: ${entry}\n`);
-        fs.rmSync(fullPath, { recursive: true, force: true });
-        count += 1;
+    // Sweep the pre-nesting flat layout (`merge-<feature>` etc.), skipping the current homes so this
+    // never deletes merge-info/ or pr-review/ themselves. Retired `pr-info/` IS swept (matches `pr-`).
+    cleanLegacyTopLevel(tmpBase, now, cutoffMs) {
+        let count = 0;
+        for (const entry of fs.readdirSync(tmpBase)) {
+            if (entry === rules_config_1.MERGE_INFO_DIR || entry === rules_config_1.PR_REVIEW_DIR)
+                continue;
+            if (!LEGACY_PREFIXES.some((prefix) => entry.startsWith(prefix)))
+                continue;
+            const fullPath = path.join(tmpBase, entry);
+            const stat = fs.statSync(fullPath);
+            if (!stat.isDirectory())
+                continue;
+            if (now - stat.mtimeMs < cutoffMs)
+                continue;
+            process.stdout.write(`  🗑️  Deleting: ${entry}\n`);
+            fs.rmSync(fullPath, { recursive: true, force: true });
+            count += 1;
+        }
+        return count;
     }
-    return count;
-}
-if (require.main === module)
-    (0, rules_config_1.runMain)(cleanTmp);
+};
+exports.CleanTmp = CleanTmp;
+exports.CleanTmp = CleanTmp = tslib_1.__decorate([
+    (0, core_context_1.provideSingleton)(),
+    (0, inversify_1.injectable)()
+], CleanTmp);
 //# sourceMappingURL=cleanTmp.js.map

package/src/scripts/workflow/cleanTmp.js.map

@@ -1 +1 @@
-{"version":3,"file":"cleanTmp.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/pr-gate/src/scripts/workflow/cleanTmp.ts"],"names":[],"mappings":";;AAeA,4BAqCC;;AApDD,iDAAyC;AACzC,+CAAyB;AACzB,mDAA6B;AAC7B,0DAAoG;AAEpG,MAAM,WAAW,GAAG,EAAE,CAAC;AACvB,MAAM,GAAG,GAAG,0DAA0D,CAAC;AAEvE,iFAAiF;AACjF,qGAAqG;AACrG,+FAA+F;AAC/F,sGAAsG;AACtG,uGAAuG;AACvG,MAAM,eAAe,GAAG,CAAC,QAAQ,EAAE,SAAS,EAAE,KAAK,CAAC,CAAC;AAE9C,KAAK,UAAU,QAAQ;IAC1B,MAAM,QAAQ,GAAG,IAAA,wBAAQ,EAAC,+BAA+B,EAAE,EAAE,QAAQ,EAAE,MAAM,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;IACxF,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,gCAAiB,CAAC,CAAC;IAEvD,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;QAC1B,OAAO;IACX,CAAC;IAED,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC1B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,yCAAyC,CAAC,CAAC;IAChE,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC1B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,aAAa,OAAO,IAAI,CAAC,CAAC;IAC/C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,cAAc,WAAW,SAAS,CAAC,CAAC;IACzD,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAE3B,MAAM,QAAQ,GAAG,WAAW,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC;IACnD,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;IACvB,IAAI,YAAY,GAAG,CAAC,CAAC;IAErB,uDAAuD;IACvD,YAAY,IAAI,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,6BAAc,CAAC,EAAE,6BAAc,EAAE,GAAG,EAAE,QAAQ,CAAC,CAAC;IACrG,YAAY,IAAI,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,4BAAa,CAAC,EAAE,4BAAa,EAAE,GAAG,EAAE,QAAQ,CAAC,CAAC;IACnG,qGAAqG;IACrG,YAAY,IAAI,mBAAmB,CAAC,OAAO,EAAE,GAAG,EAAE,QAAQ,CAAC,CAAC;IAE5D,IAAI,YAAY,KAAK,CAAC,EAAE,CAAC;QACrB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,iCAAiC,WAAW,eAAe,CAAC,CAAC;IACtF,CAAC;SAAM,CAAC;QACJ,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAC3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,eAAe,YAAY,gBAAgB,YAAY,KAAK,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,CAAC;IAC1G,CAAC;IAED,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;IAC1B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;AAC/B,CAAC;AAED,iGAAiG;AACjG,gGAAgG;AAChG,SAAS,iBAAiB,CAAC,SAAiB,EAAE,KAAa,EAAE,GAAW,EAAE,QAAgB;IACtF,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC;QAAE,OAAO,CAAC,CAAC;IACxC,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,KAAK,MAAM,KAAK,IAAI,EAAE,CAAC,WAAW,CAAC,SAAS,CAAC,EAAE,CAAC;QAC5C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,KAAK,CAAC,CAAC;QAC7C,MAAM,IAAI,GAAG,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;QACnC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE;YAAE,SAAS;QAClC,IAAI,GAAG,GAAG,IAAI,CAAC,OAAO,GAAG,QAAQ;YAAE,SAAS;QAC5C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,oBAAoB,KAAK,IAAI,KAAK,IAAI,CAAC,CAAC;QAC7D,EAAE,CAAC,MAAM,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACtD,KAAK,IAAI,CAAC,CAAC;IACf,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC;AAED,gGAAgG;AAChG,oGAAoG;AACpG,mGAAmG;AACnG,SAAS,mBAAmB,CAAC,OAAe,EAAE,GAAW,EAAE,QAAgB;IACvE,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,KAAK,MAAM,KAAK,IAAI,EAAE,CAAC,WAAW,CAAC,OAAO,CAAC,EAAE,CAAC;QAC1C,IAAI,KAAK,KAAK,6BAAc,IAAI,KAAK,KAAK,4BAAa;YAAE,SAAS;QAClE,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC,MAAc,EAAW,EAAE,CAAC,KAAK,CAAC,UAAU,CAAC,MAAM,CAAC,CAAC;YAAE,SAAS;QAC3F,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;QAC3C,MAAM,IAAI,GAAG,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;QACnC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE;YAAE,SAAS;QAClC,IAAI,GAAG,GAAG,IAAI,CAAC,OAAO,GAAG,QAAQ;YAAE,SAAS;QAC5C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,oBAAoB,KAAK,IAAI,CAAC,CAAC;QACpD,EAAE,CAAC,MAAM,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACtD,KAAK,IAAI,CAAC,CAAC;IACf,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC;AAED,IAAI,OAAO,CAAC,IAAI,KAAK,MAAM;IAAE,IAAA,sBAAO,EAAC,QAAQ,CAAC,CAAC","sourcesContent":["import { execSync } from 'child_process';\nimport * as fs from 'fs';\nimport * as path from 'path';\nimport { WEBPIECES_TMP_DIR, MERGE_INFO_DIR, PR_REVIEW_DIR, runMain } from '@webpieces/rules-config';\n\nconst CUTOFF_DAYS = 30;\nconst SEP = '━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\\n';\n\n// Per-feature workflow dirs now live under `.webpieces/merge-info/<feature>` and\n// `.webpieces/pr-review/<feature>`; those two homes are permanent (like hooks/ and instruct-ai/) and\n// only their stale subdirs are cleaned. `LEGACY_PREFIXES` sweeps the old flat top-level layout\n// (`merge-<feature>`/`review-<feature>`/`pr-<feature>`) so it self-clears after the move — which also\n// reaps the retired `pr-info/` home (it matches the `pr-` prefix and is no longer the active PR home).\nconst LEGACY_PREFIXES = ['merge-', 'review-', 'pr-'];\n\nexport async function cleanTmp(): Promise<void> {\n    const repoRoot = execSync('git rev-parse --show-toplevel', { encoding: 'utf8' }).trim();\n    const tmpBase = path.join(repoRoot, WEBPIECES_TMP_DIR);\n\n    if (!fs.existsSync(tmpBase)) {\n        return;\n    }\n\n    process.stdout.write('\\n');\n    process.stdout.write(SEP);\n    process.stdout.write('🧹 Cleaning Old Temporary Directories\\n');\n    process.stdout.write(SEP);\n    process.stdout.write('\\n');\n    process.stdout.write(`Location: ${tmpBase}\\n`);\n    process.stdout.write(`Retention: ${CUTOFF_DAYS} days\\n`);\n    process.stdout.write('\\n');\n\n    const cutoffMs = CUTOFF_DAYS * 24 * 60 * 60 * 1000;\n    const now = Date.now();\n    let deletedCount = 0;\n\n    // Stale per-feature subdirs under each permanent home.\n    deletedCount += cleanStaleSubdirs(path.join(tmpBase, MERGE_INFO_DIR), MERGE_INFO_DIR, now, cutoffMs);\n    deletedCount += cleanStaleSubdirs(path.join(tmpBase, PR_REVIEW_DIR), PR_REVIEW_DIR, now, cutoffMs);\n    // Legacy flat top-level dirs from before the merge-info/pr-review nesting (also reaps old pr-info/).\n    deletedCount += cleanLegacyTopLevel(tmpBase, now, cutoffMs);\n\n    if (deletedCount === 0) {\n        process.stdout.write(`  ✅ No directories older than ${CUTOFF_DAYS} days found\\n`);\n    } else {\n        process.stdout.write('\\n');\n        process.stdout.write(`  ✅ Deleted ${deletedCount} old director${deletedCount === 1 ? 'y' : 'ies'}\\n`);\n    }\n\n    process.stdout.write('\\n');\n    process.stdout.write(SEP);\n    process.stdout.write('\\n');\n}\n\n// Delete immediate subdirs of `parentDir` whose mtime is older than the cutoff. `label` prefixes\n// the printed path so output reads e.g. `merge-info/dean-foo`. No-op if the home doesn't exist.\nfunction cleanStaleSubdirs(parentDir: string, label: string, now: number, cutoffMs: number): number {\n    if (!fs.existsSync(parentDir)) return 0;\n    let count = 0;\n    for (const entry of fs.readdirSync(parentDir)) {\n        const fullPath = path.join(parentDir, entry);\n        const stat = fs.statSync(fullPath);\n        if (!stat.isDirectory()) continue;\n        if (now - stat.mtimeMs < cutoffMs) continue;\n        process.stdout.write(`  🗑️  Deleting: ${label}/${entry}\\n`);\n        fs.rmSync(fullPath, { recursive: true, force: true });\n        count += 1;\n    }\n    return count;\n}\n\n// Sweep the pre-nesting flat layout (`merge-<feature>` etc.), skipping the current homes (whose\n// names also start with `merge-`/`pr-`) so this never deletes merge-info/ or pr-review/ themselves.\n// The retired `pr-info/` home is deliberately NOT skipped — it matches `pr-` and self-clears here.\nfunction cleanLegacyTopLevel(tmpBase: string, now: number, cutoffMs: number): number {\n    let count = 0;\n    for (const entry of fs.readdirSync(tmpBase)) {\n        if (entry === MERGE_INFO_DIR || entry === PR_REVIEW_DIR) continue;\n        if (!LEGACY_PREFIXES.some((prefix: string): boolean => entry.startsWith(prefix))) continue;\n        const fullPath = path.join(tmpBase, entry);\n        const stat = fs.statSync(fullPath);\n        if (!stat.isDirectory()) continue;\n        if (now - stat.mtimeMs < cutoffMs) continue;\n        process.stdout.write(`  🗑️  Deleting: ${entry}\\n`);\n        fs.rmSync(fullPath, { recursive: true, force: true });\n        count += 1;\n    }\n    return count;\n}\n\nif (require.main === module) runMain(cleanTmp);\n"]}
+{"version":3,"file":"cleanTmp.js","sourceRoot":"","sources":["../../../../../../../packages/tooling/pr-gate/src/scripts/workflow/cleanTmp.ts"],"names":[],"mappings":";;;;AAAA,iDAAyC;AACzC,+CAAyB;AACzB,mDAA6B;AAC7B,0DAA2F;AAC3F,0DAA2D;AAC3D,yCAAuC;AAEvC,MAAM,WAAW,GAAG,EAAE,CAAC;AACvB,MAAM,GAAG,GAAG,0DAA0D,CAAC;AAEvE,oGAAoG;AACpG,gGAAgG;AAChG,wGAAwG;AACxG,iGAAiG;AACjG,MAAM,eAAe,GAAG,CAAC,QAAQ,EAAE,SAAS,EAAE,KAAK,CAAC,CAAC;AAErD,6FAA6F;AAGtF,IAAM,QAAQ,GAAd,MAAM,QAAQ;IACjB,KAAK,CAAC,QAAQ;QACV,MAAM,QAAQ,GAAG,IAAA,wBAAQ,EAAC,+BAA+B,EAAE,EAAE,QAAQ,EAAE,MAAM,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;QACxF,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,gCAAiB,CAAC,CAAC;QAEvD,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,OAAO,CAAC,EAAE,CAAC;YAC1B,OAAO;QACX,CAAC;QAED,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAC3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC1B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,yCAAyC,CAAC,CAAC;QAChE,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC1B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAC3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,aAAa,OAAO,IAAI,CAAC,CAAC;QAC/C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,cAAc,WAAW,SAAS,CAAC,CAAC;QACzD,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAE3B,MAAM,QAAQ,GAAG,WAAW,GAAG,EAAE,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAC;QACnD,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAC;QACvB,IAAI,YAAY,GAAG,CAAC,CAAC;QAErB,uDAAuD;QACvD,YAAY,IAAI,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,6BAAc,CAAC,EAAE,6BAAc,EAAE,GAAG,EAAE,QAAQ,CAAC,CAAC;QAC1G,YAAY,IAAI,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,4BAAa,CAAC,EAAE,4BAAa,EAAE,GAAG,EAAE,QAAQ,CAAC,CAAC;QACxG,qGAAqG;QACrG,YAAY,IAAI,IAAI,CAAC,mBAAmB,CAAC,OAAO,EAAE,GAAG,EAAE,QAAQ,CAAC,CAAC;QAEjE,IAAI,YAAY,KAAK,CAAC,EAAE,CAAC;YACrB,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,iCAAiC,WAAW,eAAe,CAAC,CAAC;QACtF,CAAC;aAAM,CAAC;YACJ,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;YAC3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,eAAe,YAAY,gBAAgB,YAAY,KAAK,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,CAAC;QAC1G,CAAC;QAED,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAC3B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC1B,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC/B,CAAC;IAED,iGAAiG;IACzF,iBAAiB,CAAC,SAAiB,EAAE,KAAa,EAAE,GAAW,EAAE,QAAgB;QACrF,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC;YAAE,OAAO,CAAC,CAAC;QACxC,IAAI,KAAK,GAAG,CAAC,CAAC;QACd,KAAK,MAAM,KAAK,IAAI,EAAE,CAAC,WAAW,CAAC,SAAS,CAAC,EAAE,CAAC;YAC5C,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,KAAK,CAAC,CAAC;YAC7C,MAAM,IAAI,GAAG,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;YACnC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE;gBAAE,SAAS;YAClC,IAAI,GAAG,GAAG,IAAI,CAAC,OAAO,GAAG,QAAQ;gBAAE,SAAS;YAC5C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,oBAAoB,KAAK,IAAI,KAAK,IAAI,CAAC,CAAC;YAC7D,EAAE,CAAC,MAAM,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;YACtD,KAAK,IAAI,CAAC,CAAC;QACf,CAAC;QACD,OAAO,KAAK,CAAC;IACjB,CAAC;IAED,iGAAiG;IACjG,mGAAmG;IAC3F,mBAAmB,CAAC,OAAe,EAAE,GAAW,EAAE,QAAgB;QACtE,IAAI,KAAK,GAAG,CAAC,CAAC;QACd,KAAK,MAAM,KAAK,IAAI,EAAE,CAAC,WAAW,CAAC,OAAO,CAAC,EAAE,CAAC;YAC1C,IAAI,KAAK,KAAK,6BAAc,IAAI,KAAK,KAAK,4BAAa;gBAAE,SAAS;YAClE,IAAI,CAAC,eAAe,CAAC,IAAI,CAAC,CAAC,MAAc,EAAW,EAAE,CAAC,KAAK,CAAC,UAAU,CAAC,MAAM,CAAC,CAAC;gBAAE,SAAS;YAC3F,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;YAC3C,MAAM,IAAI,GAAG,EAAE,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC;YACnC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE;gBAAE,SAAS;YAClC,IAAI,GAAG,GAAG,IAAI,CAAC,OAAO,GAAG,QAAQ;gBAAE,SAAS;YAC5C,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,oBAAoB,KAAK,IAAI,CAAC,CAAC;YACpD,EAAE,CAAC,MAAM,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;YACtD,KAAK,IAAI,CAAC,CAAC;QACf,CAAC;QACD,OAAO,KAAK,CAAC;IACjB,CAAC;CACJ,CAAA;AAzEY,4BAAQ;mBAAR,QAAQ;IAFpB,IAAA,+BAAgB,GAAE;IAClB,IAAA,sBAAU,GAAE;GACA,QAAQ,CAyEpB","sourcesContent":["import { execSync } from 'child_process';\nimport * as fs from 'fs';\nimport * as path from 'path';\nimport { WEBPIECES_TMP_DIR, MERGE_INFO_DIR, PR_REVIEW_DIR } from '@webpieces/rules-config';\nimport { provideSingleton } from '@webpieces/core-context';\nimport { injectable } from 'inversify';\n\nconst CUTOFF_DAYS = 30;\nconst SEP = '━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\\n';\n\n// Per-feature workflow dirs live under `.webpieces/merge-info/<feature>` and `.webpieces/pr-review/\n// <feature>`; those two homes are permanent (like hooks/ and instruct-ai/) and only their stale\n// subdirs are cleaned. `LEGACY_PREFIXES` sweeps the old flat top-level layout so it self-clears — which\n// also reaps the retired `pr-info/` home (it matches `pr-` and is no longer the active PR home).\nconst LEGACY_PREFIXES = ['merge-', 'review-', 'pr-'];\n\n/** 30-day cleanup of stale per-feature merge/pr-review dirs (and the legacy flat layout). */\n@provideSingleton()\n@injectable()\nexport class CleanTmp {\n    async cleanTmp(): Promise<void> {\n        const repoRoot = execSync('git rev-parse --show-toplevel', { encoding: 'utf8' }).trim();\n        const tmpBase = path.join(repoRoot, WEBPIECES_TMP_DIR);\n\n        if (!fs.existsSync(tmpBase)) {\n            return;\n        }\n\n        process.stdout.write('\\n');\n        process.stdout.write(SEP);\n        process.stdout.write('🧹 Cleaning Old Temporary Directories\\n');\n        process.stdout.write(SEP);\n        process.stdout.write('\\n');\n        process.stdout.write(`Location: ${tmpBase}\\n`);\n        process.stdout.write(`Retention: ${CUTOFF_DAYS} days\\n`);\n        process.stdout.write('\\n');\n\n        const cutoffMs = CUTOFF_DAYS * 24 * 60 * 60 * 1000;\n        const now = Date.now();\n        let deletedCount = 0;\n\n        // Stale per-feature subdirs under each permanent home.\n        deletedCount += this.cleanStaleSubdirs(path.join(tmpBase, MERGE_INFO_DIR), MERGE_INFO_DIR, now, cutoffMs);\n        deletedCount += this.cleanStaleSubdirs(path.join(tmpBase, PR_REVIEW_DIR), PR_REVIEW_DIR, now, cutoffMs);\n        // Legacy flat top-level dirs from before the merge-info/pr-review nesting (also reaps old pr-info/).\n        deletedCount += this.cleanLegacyTopLevel(tmpBase, now, cutoffMs);\n\n        if (deletedCount === 0) {\n            process.stdout.write(`  ✅ No directories older than ${CUTOFF_DAYS} days found\\n`);\n        } else {\n            process.stdout.write('\\n');\n            process.stdout.write(`  ✅ Deleted ${deletedCount} old director${deletedCount === 1 ? 'y' : 'ies'}\\n`);\n        }\n\n        process.stdout.write('\\n');\n        process.stdout.write(SEP);\n        process.stdout.write('\\n');\n    }\n\n    // Delete immediate subdirs of `parentDir` whose mtime is older than the cutoff. No-op if absent.\n    private cleanStaleSubdirs(parentDir: string, label: string, now: number, cutoffMs: number): number {\n        if (!fs.existsSync(parentDir)) return 0;\n        let count = 0;\n        for (const entry of fs.readdirSync(parentDir)) {\n            const fullPath = path.join(parentDir, entry);\n            const stat = fs.statSync(fullPath);\n            if (!stat.isDirectory()) continue;\n            if (now - stat.mtimeMs < cutoffMs) continue;\n            process.stdout.write(`  🗑️  Deleting: ${label}/${entry}\\n`);\n            fs.rmSync(fullPath, { recursive: true, force: true });\n            count += 1;\n        }\n        return count;\n    }\n\n    // Sweep the pre-nesting flat layout (`merge-<feature>` etc.), skipping the current homes so this\n    // never deletes merge-info/ or pr-review/ themselves. Retired `pr-info/` IS swept (matches `pr-`).\n    private cleanLegacyTopLevel(tmpBase: string, now: number, cutoffMs: number): number {\n        let count = 0;\n        for (const entry of fs.readdirSync(tmpBase)) {\n            if (entry === MERGE_INFO_DIR || entry === PR_REVIEW_DIR) continue;\n            if (!LEGACY_PREFIXES.some((prefix: string): boolean => entry.startsWith(prefix))) continue;\n            const fullPath = path.join(tmpBase, entry);\n            const stat = fs.statSync(fullPath);\n            if (!stat.isDirectory()) continue;\n            if (now - stat.mtimeMs < cutoffMs) continue;\n            process.stdout.write(`  🗑️  Deleting: ${entry}\\n`);\n            fs.rmSync(fullPath, { recursive: true, force: true });\n            count += 1;\n        }\n        return count;\n    }\n}\n"]}

package/src/scripts/workflow/git-exec.d.ts

@@ -1,28 +1,32 @@
-export declare function uncommittedFiles(cwd: string): string;
-export declare function untrackedFiles(cwd: string): string;
-/**
- * Precondition for every PR/merge entry point: the working tree must be fully committed. The webpieces
- * tooling deliberately does NOT `git add`/`commit` the developer's work for them — a blanket `git add
- * -A` once swept a stale untracked directory into a squash commit, and building the working tree while
- * pushing HEAD let an uncommitted fix pass the build gate yet push a stale commit. Aborts with
- * instructions if anything is uncommitted (tracked or untracked, excluding gitignored).
- */
-export declare function assertCleanTree(cwd: string): void;
-/**
- * Narrower guard for the merge-resolve commit point, where tracked resolutions are legitimately in the
- * tree but untracked files must NOT be swept into the squash commit (replaces the old blanket `git add
- * -A`). Lists any untracked files and aborts so the AI commits or deletes them explicitly.
- */
-export declare function assertNoUntracked(cwd: string): void;
-/**
- * Run a git command that is expected to succeed; abort the process with a clear message
- * if it fails. Used for fetch/pull/checkout/commit where silently continuing on failure
- * would operate on stale or wrong state.
- */
-export declare function runGitChecked(args: string[], errMsg: string): void;
-/**
- * Push HEAD to origin/<currentBranch>. Single source of truth shared by wp-start-upsert-pr and
- * wp-finish-upsert-pr (they pushed with identical copy-pasted logic). Uses --force-with-lease for an
- * existing remote branch (the 3-point squash rewrites history) and -u for a brand-new branch.
- */
-export declare function ensurePushed(currentBranch: string): void;
+import { RepoRootFinder } from '@webpieces/rules-config';
+/** Shared git precondition checks + push logic for the PR/merge workflow. */
+export declare class GitExec {
+    private readonly repoRootFinder;
+    constructor(repoRootFinder: RepoRootFinder);
+    uncommittedFiles(cwd: string): string;
+    untrackedFiles(cwd: string): string;
+    /**
+     * Precondition for every PR/merge entry point: the working tree must be fully committed. The
+     * webpieces tooling deliberately does NOT `git add`/`commit` the developer's work for them. Aborts
+     * with instructions if anything is uncommitted (tracked or untracked, excluding gitignored).
+     */
+    assertCleanTree(cwd: string): void;
+    /**
+     * Narrower guard for the merge-resolve commit point, where tracked resolutions are legitimately in
+     * the tree but untracked files must NOT be swept into the squash commit. Lists any untracked files
+     * and aborts so the AI commits or deletes them explicitly.
+     */
+    assertNoUntracked(cwd: string): void;
+    /**
+     * Run a git command that is expected to succeed; abort the process with a clear message if it
+     * fails. Used for fetch/pull/checkout/commit where silently continuing on failure would operate on
+     * stale or wrong state.
+     */
+    runGitChecked(args: string[], errMsg: string): void;
+    /**
+     * Push HEAD to origin/<currentBranch>. Uses --force-with-lease for an existing remote branch (the
+     * 3-point squash rewrites history) and -u for a brand-new branch.
+     */
+    ensurePushed(currentBranch: string): void;
+    private gitQuery;
+}