Skip to content

Commit e180296

Browse files
build: regenerate the Action bundle for the contract hardening
0c2eefb changed packages/core/src/verify.ts, and the Action bundle inlines core, so check:action-bundle failed on the pushed branch: the committed dist/index.mjs still carried the pre-fix no-test-changed anchoring. Targeted tests and typecheck do not catch this — only the full `npm run ci` does, because the check diffs the rebuilt bundle against the committed one. No source change; regenerated output only. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent 0c2eefb commit e180296

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

packages/action/dist/index.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2127,10 +2127,11 @@ function verifyPlan(report, repo) {
21272127
const changedTests = changed.filter(isTest);
21282128
if (changedSource.length > 0 && changedTests.length === 0) {
21292129
const suggested = [...new Set(report.testRoutes.flatMap((route) => route.relatedFiles))].filter(isTest);
2130+
const anchors = suggested.length > 0 ? suggested : changedSource;
21302131
findings.push({
21312132
code: "no-test-changed",
21322133
severity: "warning",
2133-
paths: suggested,
2134+
paths: anchors,
21342135
message: suggested.length > 0 ? `Code changed but no test did. The plan routed ${suggested.length === 1 ? "this test" : "these tests"} as most related.` : report.testRoutes.length > 0 ? `Code changed but no test did. Run the routed ${report.testRoutes.length === 1 ? "command" : "commands"}: ${report.testRoutes.map((route) => route.command).join(", ")}.` : "Code changed but no test did, and the plan found no related test to point at."
21352136
});
21362137
}

0 commit comments

Comments
 (0)