@rmartz/pr-policy 0.8.0 → 0.9.0

package/dist/bin/pr-policy.js

@@ -7,7 +7,7 @@ import {
   postVerdict,
   resolveRepoTarget,
   selectChecks
-} from "../chunk-5VYRZAUT.js";
+} from "../chunk-PVXDVMME.js";
 
 // src/bin/pr-policy.ts
 import { readFile } from "fs/promises";

package/dist/{chunk-5VYRZAUT.js → chunk-PVXDVMME.js}

@@ -1149,6 +1149,42 @@ function isReleasePleaseTitle(title) {
   return RELEASE_PLEASE_TITLE.test(title);
 }
 
+// src/checks/title/type-paths.ts
+var TEST_DIRS2 = /* @__PURE__ */ new Set(["test", "tests", "__tests__"]);
+var TEST_FILE2 = /\.(test|spec)\.[^/]+$|^test_[^/]*\.py$|_test\.(py|go)$/;
+function isMarkdownPath(path) {
+  return path.toLowerCase().endsWith(".md");
+}
+function isTestPath(path) {
+  const parts = path.split("/");
+  const name = parts.at(-1) ?? "";
+  return parts.slice(0, -1).some((dir) => TEST_DIRS2.has(dir)) || TEST_FILE2.test(name);
+}
+var quote = (paths) => {
+  const shown = paths.slice(0, 5).map((path) => `\`${path}\``);
+  if (paths.length > 5) shown.push(`and ${paths.length - 5} more`);
+  return shown.join(", ");
+};
+function typePathViolation(type, changedFiles) {
+  const paths = [...new Set(changedFiles)];
+  if (type === "docs") {
+    const offending = paths.filter((path) => !isMarkdownPath(path));
+    if (offending.length === 0) return void 0;
+    return `A \`docs\` PR may change only Markdown files, but this one changes ${quote(offending)}. \`docs\` doesn't release, so that change would never ship. Retitle to the type of the non-docs change, or split it into its own PR.`;
+  }
+  if (type === "refactor") {
+    const offending = paths.filter((path) => isTestPath(path) && !isMarkdownPath(path));
+    if (offending.length === 0) return void 0;
+    return `A \`refactor\` PR must leave the tests untouched, so they can confirm behaviour is unchanged, but this one changes ${quote(offending)}. Land the test changes first as a \`test:\` PR, then refactor against them.`;
+  }
+  if (type === "test") {
+    const offending = paths.filter((path) => !isTestPath(path) && !isMarkdownPath(path));
+    if (offending.length === 0) return void 0;
+    return `A \`test\` PR may change only test files and docs, so it locks in current behaviour without changing it, but this one changes ${quote(offending)}. \`test\` doesn't release, so that change would never ship. Retitle to the type of the non-test change, or split it into its own PR.`;
+  }
+  return void 0;
+}
+
 // src/checks/title/index.ts
 var TITLE_CHECK = "title";
 var FUNCTIONAL = FUNCTIONAL_TYPES.map((type) => `\`${type}\``).join("/");
@@ -1202,6 +1238,8 @@ function decideTitle(pr) {
       )
     );
   }
+  const pathViolation = typePathViolation(title.type, pr.changedFiles);
+  if (pathViolation !== void 0) findings.push(block2(pathViolation));
   const mayRetitle = !functional && title.type !== "ci";
   const workflows = substantiveWorkflowChanges(pr.workflowChanges);
   if (workflows.length > 0 && mayRetitle) {

package/dist/index.js

@@ -48,7 +48,7 @@ import {
   sensitiveBumps,
   statusBody,
   titleCheck
-} from "./chunk-5VYRZAUT.js";
+} from "./chunk-PVXDVMME.js";
 export {
   AMBIGUOUS_INDICATORS,
   BLOCKED_LABEL,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rmartz/pr-policy",
-  "version": "0.8.0",
+  "version": "0.9.0",
   "description": "Read-only PR content classifiers — CI-change classification, title-type rules, and more — reported as one blocking pr-policy check-run. Distributed as a pinned Action kept current by Dependabot.",
   "keywords": [
     "pr-policy",