@rmartz/pr-policy 0.8.0 → 0.9.1

package/dist/bin/pr-policy.js

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

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

@@ -892,10 +892,17 @@ function substantiveWorkflowChanges(changes) {
   return changes.filter((change) => !isShippedReusableWorkflow(change) && !isPinBumpOnly(change)).map((change) => change.path);
 }
 
+// src/checks/test-paths.ts
+var TEST_DIRS = /* @__PURE__ */ new Set(["test", "tests", "__tests__", "__snapshots__", "__mocks__", "e2e"]);
+var TEST_FILE = /\.(test|spec)\.[cm]?[jt]sx?$|^test_.*\.py$|_test\.(py|go)$|^conftest\.py$/;
+function isTestPath(path) {
+  const parts = path.split("/");
+  const name = parts.at(-1) ?? "";
+  return parts.slice(0, -1).some((dir) => TEST_DIRS.has(dir)) || TEST_FILE.test(name);
+}
+
 // src/checks/uat/trivial.ts
 var TRIVIAL_CATEGORIES = ["docs", "tests", "ci", "dependencies", "metadata"];
-var TEST_DIRS = /* @__PURE__ */ new Set(["test", "tests", "__tests__", "__snapshots__", "__mocks__", "e2e"]);
-var TEST_FILE = /\.(test|spec)\.[cm]?[jt]sx?$|^test_.*\.py$|_test\.py$|^conftest\.py$/;
 var LOCKFILES = /* @__PURE__ */ new Set([
   "pnpm-lock.yaml",
   "package-lock.json",
@@ -930,9 +937,8 @@ function isDependencyOnlyManifest(change) {
 function categoryOf(path, pr) {
   const parts = path.split("/");
   const name = parts[parts.length - 1] ?? "";
-  const dirs = parts.slice(0, -1);
   if (name.endsWith(".md") || parts[0] === "docs") return "docs";
-  if (dirs.some((dir) => TEST_DIRS.has(dir)) || TEST_FILE.test(name)) return "tests";
+  if (isTestPath(path)) return "tests";
   if (path.startsWith(".github/workflows/") || path.startsWith(".github/actions/")) {
     const change = pr.workflowChanges.find((file) => file.path === path);
     return change !== void 0 && isShippedReusableWorkflow(change) ? void 0 : "ci";
@@ -1149,6 +1155,35 @@ function isReleasePleaseTitle(title) {
   return RELEASE_PLEASE_TITLE.test(title);
 }
 
+// src/checks/title/type-paths.ts
+function isMarkdownPath(path) {
+  return path.toLowerCase().endsWith(".md");
+}
+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 +1237,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-TFZMH2KF.js";
 export {
   AMBIGUOUS_INDICATORS,
   BLOCKED_LABEL,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rmartz/pr-policy",
-  "version": "0.8.0",
+  "version": "0.9.1",
   "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",