dev-loops 0.2.6 → 0.3.0

package/scripts/github/build-adjacent-bundle.mjs

@@ -0,0 +1,448 @@
+#!/usr/bin/env node
+/**
+ * build-adjacent-bundle.mjs — deterministic, neutral adjacent-code bundle builder.
+ *
+ * Part of issue #895 (build-once neutral context bundle for the gate fan-out).
+ * The gate context-builder runs ONCE and emits a generous, NEUTRAL bundle that
+ * every independent reviewer is seeded with verbatim — instead of every reviewer
+ * re-deriving the diff + adjacent code from scratch (the actual N× waste this
+ * fixes). The primary win is WORK-DEDUP (build once vs. N× re-derivation);
+ * prompt-cache of the shared prefix is an opportunistic bonus.
+ *
+ * Neutrality + determinism are guaranteed because this is a SCRIPT, not an
+ * agent: it cannot editorialize, and identical (head + changed files) input
+ * always produces an identical bundle (sorted file list, stable JSON shape).
+ *
+ * Bundle contents: for each changed JS/.mjs/.cjs file we include
+ *   - the files it imports (1-hop out-edges), and
+ *   - the files that import it (1-hop in-edges, via a repo-wide import scan)
+ * resolved against the repo (deterministic 1-hop import/require graph).
+ *
+ * Size guards (record, do not silently drop):
+ *   - skip lockfiles (package-lock.json / *-lock.yaml), generated trees
+ *     (.claude/, dist/, lib/, node_modules/, coverage/), binary, and minified
+ *     files (*.min.*) — recorded in `stripped` with a reason.
+ *   - cap per-file bytes (default ~48KB) and truncate the long tail —
+ *     recorded in `truncated` with original/included byte counts.
+ *
+ * The output is purely structural adjacency; it does NOT carry any opinion,
+ * verdict, or main-agent state, so seeding a reviewer with it is the INTENDED
+ * neutral seed (RFC-2), not contamination.
+ */
+import { open, readFile, readdir, stat } from "node:fs/promises";
+import path from "node:path";
+
+/** Default per-file byte cap before truncation. */
+export const DEFAULT_MAX_FILE_BYTES = 48 * 1024;
+
+/**
+ * Binary-sniff window: the NUL-byte heuristic inspects this many leading bytes.
+ * The guarded read must cover at least this window even when the per-file cap is
+ * configured smaller, so a small cap can't cause a binary (whose first NUL is
+ * past the cap) to be misclassified as text.
+ */
+const BINARY_SNIFF_BYTES = 8 * 1024;
+
+/** Source extensions we resolve import edges for (JS family, this repo). */
+const SOURCE_EXTENSIONS = [".mjs", ".js", ".cjs"];
+
+/** Directory prefixes treated as generated / vendored and skipped. */
+const GENERATED_DIR_PREFIXES = [
+  ".claude/",
+  "dist/",
+  "lib/",
+  "node_modules/",
+  "coverage/",
+  ".git/",
+];
+
+/**
+ * Classify why a file is skipped from the bundle, or null when it is includable.
+ * Deterministic, path-shape based. Exported for tests.
+ * @param {string} relPath — repo-relative POSIX path
+ * @returns {"lockfile"|"generated"|"binary"|"minified"|null}
+ */
+export function classifyStripReason(relPath) {
+  const posix = String(relPath).replace(/\\/g, "/");
+  const base = posix.split("/").pop() ?? posix;
+
+  if (base === "package-lock.json" || base === "npm-shrinkwrap.json" || /-lock\.ya?ml$/.test(base) || base === "yarn.lock" || base === "pnpm-lock.yaml") {
+    return "lockfile";
+  }
+  for (const prefix of GENERATED_DIR_PREFIXES) {
+    if (posix === prefix.slice(0, -1) || posix.startsWith(prefix) || posix.includes(`/${prefix}`)) {
+      return "generated";
+    }
+  }
+  if (/\.min\.[a-z0-9]+$/i.test(base)) return "minified";
+  if (isBinaryPath(base)) return "binary";
+  return null;
+}
+
+const BINARY_EXTENSIONS = new Set([
+  ".png", ".jpg", ".jpeg", ".gif", ".webp", ".ico", ".bmp", ".pdf",
+  ".zip", ".gz", ".tgz", ".tar", ".bz2", ".7z", ".rar",
+  ".woff", ".woff2", ".ttf", ".otf", ".eot",
+  ".mp3", ".mp4", ".mov", ".avi", ".wav", ".ogg", ".webm",
+  ".so", ".dylib", ".dll", ".exe", ".bin", ".wasm", ".class", ".o", ".a",
+  ".node", ".lockb",
+]);
+
+function isBinaryPath(base) {
+  const dot = base.lastIndexOf(".");
+  if (dot < 0) return false;
+  return BINARY_EXTENSIONS.has(base.slice(dot).toLowerCase());
+}
+
+/**
+ * Heuristic binary-content sniff: a NUL byte within the first 8KB marks the
+ * file as binary regardless of extension (covers extension-less binaries).
+ * @param {Buffer} buf
+ * @returns {boolean}
+ */
+function looksBinaryContent(buf) {
+  const limit = Math.min(buf.length, BINARY_SNIFF_BYTES);
+  for (let i = 0; i < limit; i++) {
+    if (buf[i] === 0) return true;
+  }
+  return false;
+}
+
+/**
+ * Extract the static import/require specifiers from JS source text.
+ * Deterministic regex scan (no execution). Captures:
+ *   - import ... from "x"        / export ... from "x"
+ *   - import("x")                (dynamic import)
+ *   - require("x")
+ * Returns specifiers in a deterministic, grouped-by-kind order — each kind is
+ * scanned in its own pass (export/import-from → bare side-effect import →
+ * dynamic import() → require()), so the output is grouped by form rather than
+ * by position in the source. De-duplicated (first occurrence wins).
+ * @param {string} source
+ * @returns {string[]}
+ */
+export function extractImportSpecifiers(source) {
+  if (typeof source !== "string" || source.length === 0) return [];
+  const specs = [];
+  const seen = new Set();
+  const push = (s) => {
+    if (typeof s === "string" && s.length > 0 && !seen.has(s)) {
+      seen.add(s);
+      specs.push(s);
+    }
+  };
+  // import ... from "x" | export ... from "x"
+  const fromRe = /\b(?:import|export)\b[^;'"]*?\bfrom\s*['"]([^'"]+)['"]/g;
+  // bare side-effect import "x"
+  const bareRe = /\bimport\s+['"]([^'"]+)['"]/g;
+  // dynamic import("x")
+  const dynRe = /\bimport\s*\(\s*['"]([^'"]+)['"]\s*\)/g;
+  // require("x")
+  const reqRe = /\brequire\s*\(\s*['"]([^'"]+)['"]\s*\)/g;
+  for (const re of [fromRe, bareRe, dynRe, reqRe]) {
+    let m;
+    while ((m = re.exec(source)) !== null) push(m[1]);
+  }
+  return specs;
+}
+
+/**
+ * Resolve a RELATIVE import specifier (./ or ../) from a source file to a
+ * repo-relative POSIX path, trying common extensions and index files.
+ * Bare/package specifiers (no leading dot) are NOT resolved (out of repo).
+ * Pure path math against the provided existing-files set — deterministic.
+ * @param {string} fromRelPath — repo-relative path of the importing file
+ * @param {string} specifier
+ * @param {Set<string>} existing — set of repo-relative POSIX paths that exist
+ * @returns {string|null}
+ */
+export function resolveRelativeImport(fromRelPath, specifier, existing) {
+  if (typeof specifier !== "string" || !specifier.startsWith(".")) return null;
+  const fromDir = path.posix.dirname(String(fromRelPath).replace(/\\/g, "/"));
+  const base = path.posix.normalize(path.posix.join(fromDir, specifier));
+  if (base.startsWith("..")) return null; // escaped the repo root
+  const candidates = [base];
+  for (const ext of SOURCE_EXTENSIONS) candidates.push(base + ext);
+  for (const ext of SOURCE_EXTENSIONS) candidates.push(path.posix.join(base, "index" + ext));
+  for (const c of candidates) {
+    if (existing.has(c)) return c;
+  }
+  return null;
+}
+
+function isSourceFile(relPath) {
+  const ext = path.posix.extname(String(relPath).replace(/\\/g, "/")).toLowerCase();
+  return SOURCE_EXTENSIONS.includes(ext);
+}
+
+/**
+ * Recursively list repo-relative POSIX paths under repoRoot, skipping
+ * generated/vendored dirs and the tmp/ scratch tree for determinism+speed.
+ * @param {string} repoRoot
+ * @returns {Promise<string[]>} sorted repo-relative POSIX paths
+ */
+export async function listRepoFiles(repoRoot) {
+  const out = [];
+  const skipDir = new Set([".git", "node_modules", "dist", "lib", ".claude", "coverage", "tmp"]);
+  async function walk(absDir, relDir) {
+    let entries;
+    try {
+      entries = await readdir(absDir, { withFileTypes: true });
+    } catch {
+      return;
+    }
+    for (const ent of entries) {
+      if (ent.isSymbolicLink()) continue;
+      const relPath = relDir ? `${relDir}/${ent.name}` : ent.name;
+      if (ent.isDirectory()) {
+        if (skipDir.has(ent.name)) continue;
+        await walk(path.join(absDir, ent.name), relPath);
+      } else if (ent.isFile()) {
+        out.push(relPath);
+      }
+    }
+  }
+  await walk(repoRoot, "");
+  out.sort();
+  return out;
+}
+
+/**
+ * Build the reverse-import (importers) index: for each source file, which other
+ * source files import it (1-hop in-edges). Deterministic given the repo files.
+ * @param {string[]} repoFiles — repo-relative POSIX paths
+ * @param {string} repoRoot
+ * @returns {Promise<Map<string, string[]>>} target → sorted importers
+ */
+export async function buildImporterIndex(repoFiles, repoRoot) {
+  const existing = new Set(repoFiles);
+  const importers = new Map();
+  for (const file of repoFiles) {
+    if (!isSourceFile(file)) continue;
+    // Skip minified artifacts: they are stripped from the final bundle anyway and
+    // are not meaningful import sources, so reading them here is wasted I/O/memory
+    // on repos with large minified files.
+    if (/\.min\.[a-z0-9]+$/i.test(file)) continue;
+    let source;
+    try {
+      source = await readFile(path.resolve(repoRoot, file), "utf8");
+    } catch {
+      continue;
+    }
+    for (const spec of extractImportSpecifiers(source)) {
+      const target = resolveRelativeImport(file, spec, existing);
+      if (!target) continue;
+      if (!importers.has(target)) importers.set(target, []);
+      const list = importers.get(target);
+      if (!list.includes(file)) list.push(file);
+    }
+  }
+  for (const list of importers.values()) list.sort();
+  return importers;
+}
+
+/**
+ * Decide whether a repo-relative path is safe to resolve+read under repoRoot.
+ * Fails closed: rejects absolute paths, any `..` segment, and any resolved
+ * absolute path that escapes repoRoot. Deterministic, pure path math.
+ * @param {string} repoRoot
+ * @param {string} relPath — repo-relative POSIX path
+ * @returns {{ ok: true, abs: string } | { ok: false }}
+ */
+export function resolveSafeRepoPath(repoRoot, relPath) {
+  const posix = String(relPath).replace(/\\/g, "/");
+  if (posix.length === 0) return { ok: false };
+  if (path.posix.isAbsolute(posix) || path.win32.isAbsolute(posix)) return { ok: false };
+  if (posix.split("/").includes("..")) return { ok: false };
+  const root = path.resolve(repoRoot);
+  const abs = path.resolve(root, posix);
+  if (abs !== root && !abs.startsWith(root + path.sep)) return { ok: false };
+  return { ok: true, abs };
+}
+
+/**
+ * Read a file with the size guards applied. Returns the (possibly truncated)
+ * content plus guard metadata. Never throws on a missing/unreadable file.
+ *
+ * Path safety: unsafe relPaths (absolute, `..` segments, or escaping repoRoot)
+ * are NOT read — they are recorded with strip reason "unsafe-path".
+ *
+ * Bounded read: when the file is over the cap it is truncated, so we read at
+ * most maxFileBytes+1 bytes (the +1 lets us also sniff binary content past the
+ * cap without loading the whole file). The full original size is read from
+ * stat() for the manifest.
+ * @param {string} repoRoot
+ * @param {string} relPath
+ * @param {number} maxFileBytes
+ * @returns {Promise<{ relPath: string, content: string|null, bytes: number, includedBytes: number, truncated: boolean, missing: boolean, strip: string|null }>}
+ */
+async function readGuardedFile(repoRoot, relPath, maxFileBytes) {
+  const strip = classifyStripReason(relPath);
+  if (strip) {
+    return { relPath, content: null, bytes: 0, includedBytes: 0, truncated: false, missing: false, strip };
+  }
+  const safe = resolveSafeRepoPath(repoRoot, relPath);
+  if (!safe.ok) {
+    return { relPath, content: null, bytes: 0, includedBytes: 0, truncated: false, missing: false, strip: "unsafe-path" };
+  }
+  const abs = safe.abs;
+  let handle;
+  try {
+    const info = await stat(abs);
+    if (!info.isFile()) {
+      return { relPath, content: null, bytes: 0, includedBytes: 0, truncated: false, missing: true, strip: null };
+    }
+    const bytes = info.size;
+    // Read enough to (a) detect over-cap and truncate, and (b) always cover the
+    // binary-sniff window — so a cap configured below BINARY_SNIFF_BYTES can't let
+    // a binary slip through as text. Still bounded (never the whole large file).
+    const readCap = Math.min(bytes, Math.max(maxFileBytes + 1, BINARY_SNIFF_BYTES));
+    const buf = Buffer.allocUnsafe(readCap);
+    handle = await open(abs, "r");
+    const { bytesRead } = await handle.read(buf, 0, readCap, 0);
+    const data = buf.subarray(0, bytesRead);
+    if (looksBinaryContent(data)) {
+      return { relPath, content: null, bytes, includedBytes: 0, truncated: false, missing: false, strip: "binary" };
+    }
+    let truncated = false;
+    let slice = data;
+    if (bytes > maxFileBytes) {
+      slice = data.subarray(0, maxFileBytes);
+      truncated = true;
+    }
+    return {
+      relPath,
+      content: slice.toString("utf8"),
+      bytes,
+      includedBytes: slice.length,
+      truncated,
+      missing: false,
+      strip: null,
+    };
+  } catch {
+    return { relPath, content: null, bytes: 0, includedBytes: 0, truncated: false, missing: true, strip: null };
+  } finally {
+    if (handle) await handle.close().catch(() => {});
+  }
+}
+
+/**
+ * Build the deterministic, neutral adjacent-code bundle for a set of changed
+ * files. For each changed source file, collect its 1-hop import out-edges
+ * (resolved relative imports) and in-edges (files that import it) and include
+ * each adjacent file's (guarded) content.
+ *
+ * Output shape (added to the gate-context artifact under `adjacentCode`):
+ *   {
+ *     maxFileBytes,
+ *     files: [{ path, role: "changed"|"imports"|"importedBy", relatedTo: [..],
+ *               bytes, includedBytes, truncated, content }],  // sorted by path
+ *     stripped: [{ path, reason, role: "changed"|"imports"|"importedBy",
+ *                 relatedTo: [..] }],                          // sorted by path
+ *     truncated: [{ path, bytes, includedBytes }],            // sorted by path
+ *     missing: [string],                                      // sorted
+ *   }
+ *
+ * Deterministic: same (repoRoot contents + changedFiles) → identical bundle.
+ *
+ * @param {object} input
+ * @param {string[]} input.changedFiles — repo-relative paths from the diff
+ * @param {string} [input.repoRoot]
+ * @param {number} [input.maxFileBytes]
+ * @returns {Promise<object>}
+ */
+export async function buildAdjacentBundle({ changedFiles, repoRoot = process.cwd(), maxFileBytes = DEFAULT_MAX_FILE_BYTES }) {
+  const changed = Array.from(
+    new Set((Array.isArray(changedFiles) ? changedFiles : []).map((f) => String(f).replace(/\\/g, "/")).filter((f) => f.length > 0)),
+  ).sort();
+
+  // No changed files → empty bundle. Return early so the empty case is O(1) and
+  // skips the whole-repo scan + importer-index build entirely.
+  if (changed.length === 0) {
+    return { maxFileBytes, files: [], stripped: [], truncated: [], missing: [] };
+  }
+
+  const repoFiles = await listRepoFiles(repoRoot);
+  const existing = new Set(repoFiles);
+  const importerIndex = await buildImporterIndex(repoFiles, repoRoot);
+
+  // role per path: changed > imports/importedBy. relatedTo accumulates the
+  // changed files an adjacent file is adjacent to (sorted, deduped).
+  /** @type {Map<string, { role: string, relatedTo: Set<string> }>} */
+  const selected = new Map();
+  const note = (relPath, role, relatedTo) => {
+    if (!relPath) return;
+    let entry = selected.get(relPath);
+    if (!entry) {
+      entry = { role, relatedTo: new Set() };
+      selected.set(relPath, entry);
+    } else if (entry.role !== "changed" && role === "changed") {
+      entry.role = "changed";
+    }
+    if (relatedTo) entry.relatedTo.add(relatedTo);
+  };
+
+  for (const file of changed) {
+    note(file, "changed", null);
+  }
+
+  for (const file of changed) {
+    // Out-edges: imports declared by the changed file (source files only).
+    if (isSourceFile(file) && existing.has(file)) {
+      try {
+        const source = await readFile(path.resolve(repoRoot, file), "utf8");
+        for (const spec of extractImportSpecifiers(source)) {
+          const target = resolveRelativeImport(file, spec, existing);
+          if (target && !changed.includes(target)) note(target, "imports", file);
+        }
+      } catch {
+        // unreadable changed file — diff still carries it; skip out-edges
+      }
+    }
+    // In-edges: files that import the changed file.
+    const importers = importerIndex.get(file) ?? [];
+    for (const imp of importers) {
+      if (!changed.includes(imp)) note(imp, "importedBy", file);
+    }
+  }
+
+  const files = [];
+  const stripped = [];
+  const truncatedList = [];
+  const missing = [];
+
+  for (const relPath of Array.from(selected.keys()).sort()) {
+    const entry = selected.get(relPath);
+    const relatedTo = Array.from(entry.relatedTo).sort();
+    const guarded = await readGuardedFile(repoRoot, relPath, maxFileBytes);
+    if (guarded.strip) {
+      stripped.push({ path: relPath, reason: guarded.strip, role: entry.role, relatedTo });
+      continue;
+    }
+    if (guarded.missing) {
+      // A changed file deleted by the diff legitimately won't exist on disk.
+      missing.push(relPath);
+      continue;
+    }
+    if (guarded.truncated) {
+      truncatedList.push({ path: relPath, bytes: guarded.bytes, includedBytes: guarded.includedBytes });
+    }
+    files.push({
+      path: relPath,
+      role: entry.role,
+      relatedTo,
+      bytes: guarded.bytes,
+      includedBytes: guarded.includedBytes,
+      truncated: guarded.truncated,
+      content: guarded.content,
+    });
+  }
+
+  return {
+    maxFileBytes,
+    files,
+    stripped,
+    truncated: truncatedList,
+    missing: missing.sort(),
+  };
+}

package/scripts/github/capture-review-threads.mjs

@@ -8,7 +8,8 @@ import {
   parseReviewThreads,
   readInput,
 } from "../_core-helpers.mjs";
-import { parsePrNumber, requireOptionValue, runChild } from "../_cli-primitives.mjs";
+import { parseArgs } from "node:util";
+import { parsePrNumber, requireTokenValue, runChild } from "../_cli-primitives.mjs";
 import { parseRepoSlug } from "@dev-loops/core/github/repo-slug";
 export const REVIEW_THREADS_QUERY = [
   "query($owner: String!, $name: String!, $pr: Int!) {",
@@ -49,7 +50,19 @@ Exit codes:
   1   Error
 `;
 export function parseCaptureCliArgs(argv) {
-  const args = [...argv];
+  const { tokens } = parseArgs({
+    args: [...argv],
+    options: {
+      help: { type: "boolean", short: "h" },
+      input: { type: "string" },
+      output: { type: "string" },
+      repo: { type: "string" },
+      pr: { type: "string" },
+    },
+    allowPositionals: true,
+    strict: false,
+    tokens: true,
+  });
   const options = {
     inputPath: undefined,
     outputPath: undefined,
@@ -57,29 +70,34 @@ export function parseCaptureCliArgs(argv) {
     pr: undefined,
     help: false,
   };
-  while (args.length > 0) {
-    const token = args.shift();
-    if (token === "--help" || token === "-h") {
+  for (const token of tokens) {
+    if (token.kind === "positional") {
+      throw new Error(`Unknown argument: ${token.value}`);
+    }
+    if (token.kind !== "option") {
+      continue;
+    }
+    if (token.name === "help") {
       options.help = true;
       return options;
     }
-    if (token === "--input") {
-      options.inputPath = requireOptionValue(args, "--input");
+    if (token.name === "input") {
+      options.inputPath = requireTokenValue(token);
       continue;
     }
-    if (token === "--output") {
-      options.outputPath = requireOptionValue(args, "--output");
+    if (token.name === "output") {
+      options.outputPath = requireTokenValue(token);
       continue;
     }
-    if (token === "--repo") {
-      options.repo = requireOptionValue(args, "--repo");
+    if (token.name === "repo") {
+      options.repo = requireTokenValue(token);
       continue;
     }
-    if (token === "--pr") {
-      options.pr = parsePrNumber(requireOptionValue(args, "--pr"));
+    if (token.name === "pr") {
+      options.pr = parsePrNumber(requireTokenValue(token));
       continue;
     }
-    throw new Error(`Unknown argument: ${token}`);
+    throw new Error(`Unknown argument: ${token.rawName}`);
   }
   const hasLiveArgs = options.repo !== undefined || options.pr !== undefined;
   const hasCompleteLiveArgs = options.repo !== undefined && options.pr !== undefined;

package/scripts/github/{create-draft-pr.mjs → create-pr.mjs}

@@ -2,17 +2,20 @@
 import { readFile } from "node:fs/promises";
 import { spawn } from "node:child_process";
 import { buildParseError, formatCliError, isDirectCliRun } from "../_core-helpers.mjs";
-const USAGE = `Usage: create-draft-pr.mjs [gh pr create args...]
-Thin wrapper around \`gh pr create\` that enforces draft-first PR creation.
+const USAGE = `Usage: create-pr.mjs [gh pr create args...]
+Canonical PR-creation wrapper around \`gh pr create\`. Every PR opened through this
+tool is ALWAYS a draft and is self-assigned by default. Never call raw \`gh pr create\`.
 Behavior:
-  - injects exactly one \`--draft\` when absent
+  - injects exactly one \`--draft\` when absent (draft is the only mode)
+  - defaults \`--assignee @me\` when no assignee is given (self-assigned by default)
+  - honors an explicit \`--assignee <login>\` / \`-a <login>\` when supplied (no default injected)
   - rejects \`--ready\` before invoking \`gh\`
   - detects missing \`Closes #N\` / \`Fixes #N\` in \`--body\` or \`--body-file\` content (non-fatal stderr warning)
   - forwards every other argument to \`gh pr create\` unchanged
   - preserves the underlying \`gh pr create\` stdout, stderr, and exit code
 Examples:
-  node scripts/github/create-draft-pr.mjs --repo owner/repo --assignee @me --base main --head feature --title "..." --body-file pr.md
-  node <resolved-skill-scripts>/github/create-draft-pr.mjs --repo owner/repo --assignee @me --base main --head feature --title "..." --body-file pr.md
+  node scripts/github/create-pr.mjs --repo owner/repo --base main --head feature --title "..." --body-file pr.md
+  node <resolved-skill-scripts>/github/create-pr.mjs --repo owner/repo --base main --head feature --title "..." --body-file pr.md
 Notes:
   - Use \`gh pr ready\` later to leave draft state; this wrapper never opens a ready PR.
   - Wrapper-owned validation is limited to \`--ready\`; all other argument validation is left to \`gh pr create\`.
@@ -25,6 +28,12 @@ const parseError = buildParseError(USAGE);
 const READY_FLAG_PATTERN = /^--ready(?:$|=)/u;
 const DRAFT_FLAG_PATTERN = /^--draft(?:=(.*))?$/iu;
 const DRAFT_TRUE_VALUE_PATTERN = /^(?:true|1)$/iu;
+// Detect both the long `--assignee`/`--assignee=<login>` forms and the `-a`
+// short flag that `gh pr create` documents, so an explicit assignee in either
+// form suppresses the `--assignee @me` default (otherwise a caller passing
+// `-a <login>` would get a conflicting `--assignee @me` injected). (#894)
+const ASSIGNEE_FLAG_PATTERN = /^(?:--assignee(?:$|=)|-a$)/u;
+const DEFAULT_ASSIGNEE = "@me";
 const CLOSING_KEYWORD_PATTERN = /Closes\s+#\d+|Fixes\s+#\d+/i;
 const MAX_BODY_SCAN_BYTES = 16 * 1024;
 export function detectClosingKeyword(body) {
@@ -52,12 +61,12 @@ async function warnMissingClosingKeyword(args) {
   if (body === null) return; // no --body or --body-file, skip
   if (!detectClosingKeyword(body)) {
     process.stderr.write(
-      "[create-draft-pr] Warning: PR body missing `Closes #N` or `Fixes #N`. " +
+      "[create-pr] Warning: PR body missing `Closes #N` or `Fixes #N`. " +
         "GitHub will not auto-close the linked issue on merge.\n",
     );
   }
 }
-export function buildCreateDraftPrArgs(argv) {
+export function buildCreatePrArgs(argv) {
   const args = [...argv];
   if (args.includes("--help") || args.includes("-h")) {
     return {
@@ -66,17 +75,24 @@ export function buildCreateDraftPrArgs(argv) {
     };
   }
   if (args.some((token) => READY_FLAG_PATTERN.test(token))) {
-    throw parseError("create-draft-pr rejects --ready; open the PR as draft first, then run `gh pr ready` after the draft gate is satisfied");
+    throw parseError("create-pr rejects --ready; open the PR as draft first, then run `gh pr ready` after the draft gate is satisfied");
   }
   const draftTokens = args.filter((token) => DRAFT_FLAG_PATTERN.test(token));
   const lastDraftToken = draftTokens.length > 0 ? draftTokens.at(-1) : null;
   const lastDraftSuppliesDraft = lastDraftToken === "--draft" || (typeof lastDraftToken === "string" && DRAFT_TRUE_VALUE_PATTERN.test(lastDraftToken.slice("--draft=".length)));
+  const hasAssignee = args.some((token) => ASSIGNEE_FLAG_PATTERN.test(token));
   return {
     help: false,
-    ghArgs: ["pr", "create", ...args, ...(lastDraftSuppliesDraft ? [] : ["--draft"])],
+    ghArgs: [
+      "pr",
+      "create",
+      ...args,
+      ...(hasAssignee ? [] : ["--assignee", DEFAULT_ASSIGNEE]),
+      ...(lastDraftSuppliesDraft ? [] : ["--draft"]),
+    ],
   };
 }
-export function spawnCreateDraftPr(ghArgs, { ghCommand = "gh", env = process.env } = {}) {
+export function spawnCreatePr(ghArgs, { ghCommand = "gh", env = process.env } = {}) {
   return new Promise((resolve, reject) => {
     const child = spawn(ghCommand, ghArgs, {
       env,
@@ -89,13 +105,13 @@ export function spawnCreateDraftPr(ghArgs, { ghCommand = "gh", env = process.env
   });
 }
 export async function main(argv = process.argv.slice(2), runtime = {}) {
-  const { help, ghArgs } = buildCreateDraftPrArgs(argv);
+  const { help, ghArgs } = buildCreatePrArgs(argv);
   if (help) {
     process.stdout.write(`${USAGE}\n`);
     return 0;
   }
   await warnMissingClosingKeyword(argv);
-  return spawnCreateDraftPr(ghArgs, runtime);
+  return spawnCreatePr(ghArgs, runtime);
 }
 if (isDirectCliRun(import.meta.url)) {
   try {