split-by-codeowners 1.0.0 → 1.0.3

package/action.yml

@@ -91,6 +91,16 @@ inputs:
     required: false
     default: "Automated changes bucketed by CODEOWNERS.\n\nOwners: {owners}\nBucket key: {bucket_key}\n\nFiles:\n{files}\n"
 
+  pr_body_mode:
+    description: "PR body mode for PR creation: custom | template | template_with_bucket | none"
+    required: false
+    default: "custom"
+
+  pr_template_path:
+    description: "Path to pull request template file (used when pr_body_mode=template*)."
+    required: false
+    default: ".github/pull_request_template.md"
+
   draft:
     description: "Create PRs as drafts."
     required: false

package/dist/index.js

@@ -27470,6 +27470,7 @@ const codeowners_1 = __nccwpck_require__(3586);
 const git_1 = __nccwpck_require__(1243);
 const github_1 = __nccwpck_require__(9248);
 const ghcli_1 = __nccwpck_require__(8064);
+const pr_template_1 = __nccwpck_require__(2456);
 function formatTemplate(tpl, vars) {
     let out = tpl;
     for (const [k, v] of Object.entries(vars))
@@ -27589,8 +27590,22 @@ async function runSplit(config, logger) {
                 (0, git_1.pushBranch)(config.remoteName, branch, worktreeDir);
                 const ownersStr = b.owners.length ? b.owners.join(", ") : "(unowned)";
                 const filesStr = b.files.map(f => `- ${f.file}`).join("\n");
+                const bucketInfo = formatTemplate("Automated changes bucketed by CODEOWNERS.\n\nOwners: {owners}\nBucket key: {bucket_key}\n\nFiles:\n{files}\n", { owners: ownersStr, bucket_key: b.key, files: filesStr });
                 const title = formatTemplate(config.prTitle, { owners: ownersStr, bucket_key: b.key });
-                const body = formatTemplate(config.prBody, { owners: ownersStr, bucket_key: b.key, files: filesStr });
+                let body;
+                if (config.prBodyMode === "none") {
+                    body = undefined;
+                }
+                else if (config.prBodyMode === "custom") {
+                    body = formatTemplate(config.prBody, { owners: ownersStr, bucket_key: b.key, files: filesStr });
+                }
+                else {
+                    const template = (0, pr_template_1.readPrTemplate)(worktreeDir, config.prTemplatePath) ?? "";
+                    body =
+                        config.prBodyMode === "template_with_bucket"
+                            ? (template ? template.trimEnd() + "\n\n---\n\n" + bucketInfo : bucketInfo)
+                            : (template || bucketInfo);
+                }
                 const pr = useGhCli
                     ? (() => {
                         return (0, ghcli_1.upsertPullRequestViaGh)({
@@ -27598,7 +27613,7 @@ async function runSplit(config, logger) {
                             base: baseBranch,
                             head: branch,
                             title,
-                            body,
+                            body: body ?? "",
                             draft: config.draft,
                             bucketKey: b.key
                         });
@@ -27609,7 +27624,7 @@ async function runSplit(config, logger) {
                         base: baseBranch,
                         head: branch,
                         title,
-                        body,
+                        body: body ?? "",
                         draft: config.draft,
                         bucketKey: b.key
                     });
@@ -27714,6 +27729,35 @@ exports.parseCodeowners = parseCodeowners;
 exports.ownersForFile = ownersForFile;
 const node_fs_1 = __importDefault(__nccwpck_require__(3024));
 const minimatch_1 = __nccwpck_require__(6507);
+function splitPatternOwners(line) {
+    // CODEOWNERS lines are: <pattern><whitespace><owner>...
+    // Patterns may contain spaces if escaped as `\ `.
+    // We parse by finding the first whitespace not escaped by a backslash.
+    let i = 0;
+    let escaped = false;
+    for (; i < line.length; i++) {
+        const ch = line[i];
+        if (escaped) {
+            escaped = false;
+            continue;
+        }
+        if (ch === "\\") {
+            escaped = true;
+            continue;
+        }
+        if (/\s/.test(ch))
+            break;
+    }
+    const rawPattern = line.slice(0, i).trim();
+    const rest = line.slice(i).trim();
+    if (!rawPattern || !rest)
+        return null;
+    const pattern = rawPattern.replaceAll("\\ ", " ");
+    const owners = rest.split(/\s+/).filter(Boolean);
+    if (!owners.length)
+        return null;
+    return { pattern, owners };
+}
 function parseCodeowners(path) {
     const text = node_fs_1.default.readFileSync(path, "utf8");
     const rules = [];
@@ -27721,18 +27765,60 @@ function parseCodeowners(path) {
         const line = raw.trim();
         if (!line || line.startsWith("#"))
             continue;
-        const parts = line.split(/\s+/).filter(Boolean);
-        if (parts.length < 2)
+        const parsed = splitPatternOwners(line);
+        if (!parsed)
             continue;
-        rules.push({ pattern: parts[0], owners: parts.slice(1) });
+        rules.push({ pattern: parsed.pattern, owners: parsed.owners });
     }
     return rules;
 }
+function hasGlobMagic(p) {
+    // Good-enough heuristic for gitignore-style globs.
+    return /[*?\[]/.test(p);
+}
 function matches(file, pattern) {
-    const f = file.replace(/\\/g, "/");
-    // crude but works for most repos:
-    const pat = pattern.startsWith("/") ? pattern.slice(1) : pattern;
-    return (0, minimatch_1.minimatch)(f, pat, { dot: true, matchBase: !pattern.startsWith("/") });
+    // GitHub CODEOWNERS patterns are gitignore-like:
+    // - Leading `/` anchors to repo root
+    // - Pattern without `/` matches a basename anywhere
+    // - A directory pattern matches everything under it (e.g. `/frontend/admin` should match `/frontend/admin/**`)
+    const f = file.replace(/\\/g, "/").replace(/^\.?\//, "");
+    let pat = pattern.replace(/\\/g, "/").trim();
+    if (!pat)
+        return false;
+    const anchored = pat.startsWith("/");
+    if (anchored)
+        pat = pat.slice(1);
+    // Directory patterns: trailing `/` means directory; also treat non-glob literal paths as directory-or-file match.
+    const trailingSlash = pat.endsWith("/");
+    if (trailingSlash)
+        pat = pat.slice(0, -1);
+    const isLiteral = !hasGlobMagic(pat);
+    const hasSlash = pat.includes("/");
+    // Literal path: match exact file OR directory subtree.
+    if (isLiteral && (anchored || hasSlash)) {
+        return f === pat || f.startsWith(pat + "/");
+    }
+    // Literal basename: match basename OR directory subtree anywhere.
+    if (isLiteral && !hasSlash) {
+        const base = f.split("/").pop() ?? f;
+        if (base === pat)
+            return true;
+        return f.includes("/" + pat + "/");
+    }
+    // Glob patterns: use minimatch.
+    // - If anchored: match from repo root.
+    // - If not anchored and no slash: match basename anywhere.
+    // - If not anchored but includes a slash: match relative to repo root (like a root CODEOWNERS / .gitignore).
+    const matchBase = !anchored && !hasSlash;
+    const mm = (0, minimatch_1.minimatch)(f, pat, { dot: true, matchBase });
+    if (mm)
+        return true;
+    // If the pattern explicitly denotes a directory (trailing slash), also match directory subtree.
+    if (trailingSlash) {
+        // Convert `dir/` to subtree match.
+        return (0, minimatch_1.minimatch)(f, pat + "/**", { dot: true, matchBase: false });
+    }
+    return false;
 }
 function ownersForFile(file, rules) {
     let hit;
@@ -28039,7 +28125,7 @@ async function upsertPullRequest(params) {
         head,
         base,
         title,
-        body,
+        body: body || "",
         draft
     });
     return { bucket_key: bucketKey, branch: head, number: created.data.number, url: created.data.html_url };
@@ -28111,6 +28197,8 @@ async function run() {
             prTitle: core.getInput("pr_title") || "chore: automated changes ({owners})",
             prBody: core.getInput("pr_body") ||
                 "Automated changes bucketed by CODEOWNERS.\n\nOwners: {owners}\nBucket key: {bucket_key}\n\nFiles:\n{files}\n",
+            prBodyMode: (core.getInput("pr_body_mode") || "custom"),
+            prTemplatePath: core.getInput("pr_template_path") || ".github/pull_request_template.md",
             draft: (0, buckets_1.parseBool)(core.getInput("draft")),
             remoteName: "origin",
         };
@@ -28127,6 +28215,40 @@ async function run() {
 run();
 
 
+/***/ }),
+
+/***/ 2456:
+/***/ (function(__unused_webpack_module, exports, __nccwpck_require__) {
+
+"use strict";
+
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.readPrTemplate = readPrTemplate;
+const node_fs_1 = __importDefault(__nccwpck_require__(3024));
+const node_path_1 = __importDefault(__nccwpck_require__(6760));
+function readPrTemplate(cwd, templatePath) {
+    const candidates = [];
+    if (templatePath && templatePath.trim()) {
+        candidates.push(templatePath.trim());
+    }
+    else {
+        candidates.push(".github/pull_request_template.md");
+        candidates.push(".github/PULL_REQUEST_TEMPLATE.md");
+        candidates.push("pull_request_template.md");
+    }
+    for (const rel of candidates) {
+        const abs = node_path_1.default.resolve(cwd, rel);
+        if (node_fs_1.default.existsSync(abs) && node_fs_1.default.statSync(abs).isFile()) {
+            return node_fs_1.default.readFileSync(abs, "utf8");
+        }
+    }
+    return null;
+}
+
+
 /***/ }),
 
 /***/ 2613: