dev-loops 0.7.1 → 0.7.2

package/scripts/projects/_resolve-project.mjs

@@ -1,148 +1 @@
-import { readFileSync } from "node:fs";
-import path from "node:path";
-import { parse as parseYaml } from "yaml";
-
-// Read .devloops (and extension variants) queue settings, mirroring the
-// resolution used by ensure-queue-board.mjs. Returns { project }, { title },
-// and/or { olderThanDays } when configured; never throws on a missing/bad file.
-function resolveSettings(cwd) {
-  const basePath = path.join(cwd, ".devloops");
-  const extensions = ["", ".yaml", ".yml", ".json"];
-  for (const ext of extensions) {
-    try {
-      const raw = readFileSync(basePath + ext, "utf-8");
-      const settings = ext === ".json" ? JSON.parse(raw) : parseYaml(raw);
-      const queue = settings?.queue;
-      if (!queue) return null;
-      const out = {};
-      if (typeof queue.projectNumber === "number" && Number.isInteger(queue.projectNumber) && queue.projectNumber > 0) {
-        out.project = queue.projectNumber;
-      } else if (typeof queue.boardTitle === "string" && queue.boardTitle.trim().length > 0) {
-        out.title = queue.boardTitle.trim();
-      }
-      if (typeof queue.archiveOlderThanDays === "number" && Number.isInteger(queue.archiveOlderThanDays) && queue.archiveOlderThanDays > 0) {
-        out.olderThanDays = queue.archiveOlderThanDays;
-      }
-      return out;
-    } catch {
-      // extension not present or unparseable — try next
-    }
-  }
-  return null;
-}
-
-// Parse a --project value into { kind:"id"|"number"|"uri", ... }. Throws
-// INVALID_PROJECT on empty/malformed input (bare "0" is rejected too).
-//
-// Supported forms:
-//   <n>           positive integer  → { kind:"number", value:<n> }
-//   <NODE_ID>     alphanumeric/_ ID → { kind:"id", value:<NODE_ID> }
-//   https://github.com/users/<login>/projects/<n>
-//   https://github.com/orgs/<login>/projects/<n>
-//                 board URI        → { kind:"uri", number:<n>, owner:<login>, ownerKind:"user"|"org" }
-const GLOBAL_NODE_ID_RE = /^[A-Za-z0-9_]+$/;
-
-// GitHub Projects V2 board URI pattern (user- or org-scoped boards).
-const BOARD_URI_RE = /^https:\/\/github\.com\/(users|orgs)\/([A-Za-z0-9](?:[A-Za-z0-9-]*[A-Za-z0-9])?)\/projects\/(\d+)$/;
-
-function parseProjectRef(raw) {
-  if (!raw || typeof raw !== "string" || raw.trim().length === 0) {
-    throw Object.assign(new Error("--project is required"), { code: "INVALID_PROJECT" });
-  }
-  const trimmed = raw.trim();
-
-  // Board URI: https://github.com/users/<login>/projects/<n>
-  //            https://github.com/orgs/<login>/projects/<n>
-  const uriMatch = BOARD_URI_RE.exec(trimmed);
-  if (uriMatch) {
-    const ownerKind = uriMatch[1] === "users" ? "user" : "org";
-    const owner = uriMatch[2];
-    const number = Number(uriMatch[3]);
-    if (number < 1) {
-      throw Object.assign(
-        new Error(`--project board URI must reference a positive project number, got "${raw}"`),
-        { code: "INVALID_PROJECT" },
-      );
-    }
-    return { kind: "uri", number, owner, ownerKind };
-  }
-
-  const asNum = Number(trimmed);
-  if (Number.isInteger(asNum) && asNum > 0 && String(asNum) === trimmed) {
-    return { kind: "number", value: asNum };
-  }
-  // Reject bare "0" — valid node ID character but not a meaningful project reference
-  if (trimmed === "0") {
-    throw Object.assign(
-      new Error(`--project must be a positive integer, a node ID, or a board URI, got "${raw}"`),
-      { code: "INVALID_PROJECT" },
-    );
-  }
-  if (GLOBAL_NODE_ID_RE.test(trimmed)) {
-    return { kind: "id", value: trimmed };
-  }
-  throw Object.assign(
-    new Error(`--project must be a positive integer, a node ID, or a board URI, got "${raw}"`),
-    { code: "INVALID_PROJECT" },
-  );
-}
-
-// Selector precedence: explicit --project ref wins; else resolve by board title
-// from .devloops (passed as args.projectTitle by runCli). Fail closed if neither.
-function resolveProjectSelector(args) {
-  const hasProjectRef = typeof args.project === "string" && args.project.trim().length > 0;
-  const projectRef = hasProjectRef ? parseProjectRef(args.project) : null;
-  const projectTitle = !hasProjectRef && typeof args.projectTitle === "string" && args.projectTitle.trim().length > 0
-    ? args.projectTitle.trim()
-    : null;
-  if (!projectRef && !projectTitle) {
-    throw Object.assign(
-      new Error("--project is required (or set queue.projectNumber / queue.boardTitle in .devloops)"),
-      { code: "INVALID_PROJECT" },
-    );
-  }
-  return { projectRef, projectTitle };
-}
-
-// Find the project in `projects` matching the resolved selector; throws
-// PROJECT_NOT_FOUND (desc: "<id>" / number N / title "T" / URI number N under "<owner>").
-function findProject(projects, { projectRef, projectTitle }, owner) {
-  let project;
-  if (projectRef) {
-    if (projectRef.kind === "id") {
-      project = projects.find((p) => p.id === projectRef.value);
-    } else if (projectRef.kind === "uri") {
-      project = projects.find((p) => p.number === projectRef.number);
-    } else {
-      project = projects.find((p) => p.number === projectRef.value);
-    }
-  } else {
-    project = projects.find((p) => p.title === projectTitle);
-  }
-  if (!project) {
-    const desc = projectRef
-      ? (projectRef.kind === "id"
-          ? `"${projectRef.value}"`
-          : projectRef.kind === "uri"
-            ? `URI number ${projectRef.number} under "${projectRef.owner}"`
-            : `number ${projectRef.value}`)
-      : `title "${projectTitle}"`;
-    throw Object.assign(
-      new Error(`Project ${desc} not found under owner "${owner}"`),
-      { code: "PROJECT_NOT_FOUND" },
-    );
-  }
-  return project;
-}
-
-// Apply .devloops board settings when --project was not passed. Precedence:
-// explicit --project flag > queue.projectNumber/boardTitle. Mutates args.
-function applyDevloopsBoard(args, cwd) {
-  if (args.project === undefined) {
-    const settings = resolveSettings(cwd);
-    if (settings?.project) args.project = String(settings.project);
-    else if (settings?.title) args.projectTitle = settings.title;
-  }
-}
-
-export { resolveSettings, parseProjectRef, resolveProjectSelector, findProject, applyDevloopsBoard };
+export { resolveSettings, parseProjectRef, parseItemRef, resolveProjectSelector, findProject, applyDevloopsBoard } from "@dev-loops/core/projects/resolve-project";

package/scripts/projects/list-queue-items.mjs

@@ -1,9 +1,9 @@
 #!/usr/bin/env node
-import { formatCliError, isDirectCliRun, parseJsonText } from "../_core-helpers.mjs";
-import { runChild as _runChild } from "../_cli-primitives.mjs";
-import { resolveProjectSelector, findProject, applyDevloopsBoard } from "./_resolve-project.mjs";
+import { formatCliError, isDirectCliRun } from "../_core-helpers.mjs";
+import { applyDevloopsBoard } from "./_resolve-project.mjs";
 import { parseArgs } from "node:util";
 import { JQ_OUTPUT_PARSE_OPTIONS, JQ_OUTPUT_USAGE, emitResult, matchJqOutputToken } from "../lib/jq-output.mjs";
+import { main, classifyExitCode } from "@dev-loops/core/projects/list-queue-items";
 
 const USAGE = `Usage: dev-loops queue list --repo <owner/name> [--project <number|id>] [--column <name>] [--limit <n>]
        dev-loops queue list --repo <owner/name> [--project <number|id>] --summary [--done-limit <n>]
@@ -139,380 +139,6 @@ function parseCliArgs(argv) {
   }
   return args;
 }
-// ── Validation ───────────────────────────────────────────────────────────
-
-const OWNER_RE = /^[a-zA-Z0-9](?:[a-zA-Z0-9-]*[a-zA-Z0-9])?$/;
-const REPO_NAME_RE = /^[a-zA-Z0-9](?:[a-zA-Z0-9_.-]*[a-zA-Z0-9])?$/;
-
-function validateRepo(repo) {
-  if (!repo || typeof repo !== "string") {
-    throw Object.assign(new Error("--repo is required"), { code: "INVALID_REPO" });
-  }
-  const trimmed = repo.trim();
-  if (trimmed !== repo) {
-    throw Object.assign(
-      new Error(`--repo must not have leading/trailing whitespace, got "${repo}"`),
-      { code: "INVALID_REPO" },
-    );
-  }
-  const slashIdx = repo.indexOf("/");
-  if (slashIdx === -1) {
-    throw Object.assign(new Error(`--repo must be exactly owner/name, got "${repo}"`), { code: "INVALID_REPO" });
-  }
-  const owner = repo.slice(0, slashIdx);
-  const name = repo.slice(slashIdx + 1);
-  if (!owner || !name || !OWNER_RE.test(owner) || !REPO_NAME_RE.test(name)) {
-    throw Object.assign(new Error(`--repo must be exactly owner/name, got "${repo}"`), { code: "INVALID_REPO" });
-  }
-  return repo;
-}
-
-// ── API helpers ──────────────────────────────────────────────────────────
-
-async function ghGraphql(query, vars, env, runChild = _runChild) {
-  const fieldArgs = [];
-  for (const [key, value] of Object.entries(vars)) {
-    fieldArgs.push("--field", `${key}=${value}`);
-  }
-  const result = await runChild(
-    "gh",
-    ["api", "graphql", "--field", `query=${query}`, ...fieldArgs],
-    env,
-  );
-  if (result.code !== 0) {
-    const detail = result.stderr.trim() || `exit code ${result.code}`;
-    throw Object.assign(new Error(`gh api graphql failed: ${detail}`), { code: "GH_API_ERROR" });
-  }
-  const payload = parseJsonText(result.stdout);
-  if (payload.errors && payload.errors.length > 0) {
-    throw Object.assign(
-      new Error(`GraphQL errors: ${payload.errors.map((e) => e.message).join("; ")}`),
-      { code: "GRAPHQL_ERROR" },
-    );
-  }
-  return payload;
-}
-
-// ── GraphQL fragments ────────────────────────────────────────────────────
-
-const GET_USER_ID = [
-  "query($login:String!) {",
-  "  user(login:$login) { id }",
-  "}"
-].join("\n");
-
-const GET_ORG_ID = [
-  "query($login:String!) {",
-  "  organization(login:$login) { id }",
-  "}"
-].join("\n");
-
-const LIST_USER_PROJECTS = [
-  "query($login:String!, $after:String) {",
-  "  user(login:$login) {",
-  "    projectsV2(first:50, after:$after) {",
-  "      pageInfo { hasNextPage endCursor }",
-  "      nodes { id number title url }",
-  "    }",
-  "  }",
-  "}"
-].join("\n");
-
-const LIST_ORG_PROJECTS = [
-  "query($login:String!, $after:String) {",
-  "  organization(login:$login) {",
-  "    projectsV2(first:50, after:$after) {",
-  "      pageInfo { hasNextPage endCursor }",
-  "      nodes { id number title url }",
-  "    }",
-  "  }",
-  "}"
-].join("\n");
-
-const GET_PROJECT_FIELDS = [
-  "query($projectId:ID!, $after:String) {",
-  "  node(id:$projectId) {",
-  "    ... on ProjectV2 {",
-  "      fields(first:50, after:$after) {",
-  "        pageInfo { hasNextPage endCursor }",
-  "        nodes {",
-  "          ... on ProjectV2SingleSelectField {",
-  "            id name",
-  "            options { id name }",
-  "          }",
-  "        }",
-  "      }",
-  "    }",
-  "  }",
-  "}"
-].join("\n");
-
-const GET_PROJECT_ITEMS = [
-  "query($projectId:ID!, $after:String) {",
-  "  node(id:$projectId) {",
-  "    ... on ProjectV2 {",
-  "      items(first:100, after:$after) {",
-  "        pageInfo { hasNextPage endCursor }",
-  "        nodes {",
-  "          id",
-  "          fieldValues(first:20) {",
-  "            nodes {",
-  "              ... on ProjectV2ItemFieldSingleSelectValue {",
-  "                field { ... on ProjectV2SingleSelectField { id name } }",
-  "                name",
-  "              }",
-  "            }",
-  "          }",
-  "          content {",
-  "            ... on Issue { number title url id }",
-  "            ... on PullRequest { number title url id }",
-  "          }",
-  "        }",
-  "      }",
-  "    }",
-  "  }",
-  "}"
-].join("\n");
-
-// ── Owner resolution ────────────────────────────────────────────────────
-
-async function resolveOwner(login, env, runChild) {
-  const userPayload = await ghGraphql(GET_USER_ID, { login }, env, runChild);
-  if (userPayload?.data?.user?.id) {
-    return { id: userPayload.data.user.id, kind: "user" };
-  }
-  const orgPayload = await ghGraphql(GET_ORG_ID, { login }, env, runChild);
-  if (orgPayload?.data?.organization?.id) {
-    return { id: orgPayload.data.organization.id, kind: "org" };
-  }
-  throw Object.assign(
-    new Error(`Could not resolve owner ID for "${login}"`),
-    { code: "NO_USER_ID" },
-  );
-}
-
-// ── Paginated project listing ────────────────────────────────────────────
-
-async function listAllProjects(login, kind, env, runChild) {
-  const query = kind === "org" ? LIST_ORG_PROJECTS : LIST_USER_PROJECTS;
-  const projects = [];
-  let after = null;
-  while (true) {
-    const vars = { login };
-    if (after) vars.after = after;
-    const payload = await ghGraphql(query, vars, env, runChild);
-    const connection = kind === "org"
-      ? payload?.data?.organization?.projectsV2
-      : payload?.data?.user?.projectsV2;
-    const nodes = connection?.nodes ?? [];
-    projects.push(...nodes);
-    const pageInfo = connection?.pageInfo ?? {};
-    if (!pageInfo.hasNextPage) break;
-    if (!pageInfo.endCursor) {
-      throw Object.assign(
-        new Error("Invalid projects list payload: hasNextPage is true but endCursor is missing"),
-        { code: "GH_API_ERROR" },
-      );
-    }
-    after = pageInfo.endCursor;
-  }
-  return projects;
-}
-
-// ── Paginated field listing ──────────────────────────────────────────────
-
-async function listAllFields(projectId, env, runChild) {
-  const fields = [];
-  let after = null;
-  while (true) {
-    const vars = { projectId };
-    if (after) vars.after = after;
-    const payload = await ghGraphql(GET_PROJECT_FIELDS, vars, env, runChild);
-    const connection = payload?.data?.node?.fields;
-    const nodes = connection?.nodes ?? [];
-    fields.push(...nodes);
-    const pageInfo = connection?.pageInfo ?? {};
-    if (!pageInfo.hasNextPage) break;
-    if (!pageInfo.endCursor) {
-      throw Object.assign(
-        new Error("Invalid fields payload: hasNextPage is true but endCursor is missing"),
-        { code: "GH_API_ERROR" },
-      );
-    }
-    after = pageInfo.endCursor;
-  }
-  return fields;
-}
-
-// ── Paginated item listing ───────────────────────────────────────────────
-
-async function listAllItems(projectId, env, runChild) {
-  const items = [];
-  let after = null;
-  while (true) {
-    const vars = { projectId };
-    if (after) vars.after = after;
-    const payload = await ghGraphql(GET_PROJECT_ITEMS, vars, env, runChild);
-    const connection = payload?.data?.node?.items;
-    const nodes = connection?.nodes ?? [];
-    items.push(...nodes);
-    const pageInfo = connection?.pageInfo ?? {};
-    if (!pageInfo.hasNextPage) break;
-    if (!pageInfo.endCursor) {
-      throw Object.assign(
-        new Error("Invalid items payload: hasNextPage is true but endCursor is missing"),
-        { code: "GH_API_ERROR" },
-      );
-    }
-    after = pageInfo.endCursor;
-  }
-  return items;
-}
-
-// ── Exit code classification ────────────────────────────────────────────
-
-function classifyExitCode(err) {
-  if (err.code === "INVALID_REPO" || err.code === "INVALID_PROJECT" || err.code === "INVALID_ARGS") return 1;
-  if (err.code === "PROJECT_NOT_FOUND" || err.code === "FIELD_NOT_FOUND" || err.code === "COLUMN_NOT_FOUND") return 3;
-  return 2;
-}
-
-// ── Main logic ──────────────────────────────────────────────────────────
-
-async function main(args, { env = process.env, runChild } = {}) {
-  const child = runChild ?? _runChild;
-  const repo = validateRepo(args.repo);
-  const [owner] = repo.split("/");
-  const selector = resolveProjectSelector(args);
-
-  // Mutual exclusion: --summary is the whole-board grouped view; --column/--limit
-  // are flat-mode knobs. Combining them is ambiguous.
-  if (args.summary && args.column) {
-    throw Object.assign(
-      new Error("--summary and --column are mutually exclusive (--column filters to one status; --summary groups the whole board)"),
-      { code: "INVALID_ARGS" },
-    );
-  }
-  if (args.summary && args.limit) {
-    throw Object.assign(
-      new Error("--summary and --limit are mutually exclusive; use --done-limit to cap the Done group (or terminal column if no Done column exists)"),
-      { code: "INVALID_ARGS" },
-    );
-  }
-  if (args.doneLimit !== undefined && !args.summary) {
-    throw Object.assign(
-      new Error("--done-limit only applies with --summary"),
-      { code: "INVALID_ARGS" },
-    );
-  }
-
-  // 1. Resolve owner (user or org).
-  // URI refs encode owner+kind directly; skip the API round-trip for owner resolution.
-  const projectOwner = selector.projectRef?.kind === "uri" ? selector.projectRef.owner : owner;
-  const ownerKind = selector.projectRef?.kind === "uri"
-    ? selector.projectRef.ownerKind
-    : (await resolveOwner(owner, env, child)).kind;
-
-  // 2. Resolve project
-  const projects = await listAllProjects(projectOwner, ownerKind, env, child);
-  const project = findProject(projects, selector, projectOwner);
-
-  // 3. Resolve Status field and target column
-  const fieldNodes = await listAllFields(project.id, env, child);
-  const statusField = fieldNodes.find((f) => f.name === "Status" && f.options);
-  if (!statusField) {
-    throw Object.assign(
-      new Error(`Status field not found in project "${project.title}" (number ${project.number})`),
-      { code: "FIELD_NOT_FOUND" },
-    );
-  }
-
-  let targetOption = null;
-  if (args.column) {
-    targetOption = statusField.options.find(
-      (o) => o.name === args.column,
-    );
-    if (!targetOption) {
-      const available = statusField.options.map((o) => o.name).join(", ");
-      throw Object.assign(
-        new Error(
-          `Column "${args.column}" not found in Status field. Available: ${available}`,
-        ),
-        { code: "COLUMN_NOT_FOUND" },
-      );
-    }
-  }
-
-  // 4. List and filter items (ordered by position ascending, GraphQL default)
-  const rawItems = await listAllItems(project.id, env, child);
-
-  const results = [];
-  for (const item of rawItems) {
-    const content = item.content;
-    if (!content) continue;
-
-    // Determine status from field values
-    let status = null;
-    const fieldValues = item.fieldValues?.nodes ?? [];
-    for (const fv of fieldValues) {
-      if (fv && fv.field && fv.field.name === "Status") {
-        status = fv.name;
-        break;
-      }
-    }
-
-    // Filter by column
-    if (args.column && status !== args.column) continue;
-
-    const isPr = content.__typename === "PullRequest";
-
-    results.push({
-      issueNumber: isPr ? null : content.number,
-      prNumber: isPr ? content.number : null,
-      title: content.title ?? null,
-      url: content.url ?? null,
-      itemId: item.id,
-      contentId: content.id ?? null,
-      status: status ?? null,
-    });
-  }
-
-  // 5a. Summary mode: group by Status column in board option order.
-  if (args.summary) {
-    // Object.create(null): board option names are free text, so a column named
-    // "__proto__"/"constructor" must be an own key, not touch Object.prototype.
-    const groups = Object.create(null);
-    for (const option of statusField.options) {
-      groups[option.name] = { count: 0, items: [] };
-    }
-    for (const r of results) {
-      // Items with null status belong to no Status option, so they are excluded here — matches --column filtering behavior.
-      if (r.status === null) continue;
-      const group = groups[r.status];
-      if (!group) continue; // status value not among current board options
-      group.count += 1;
-      group.items.push(r);
-    }
-    if (args.doneLimit !== undefined) {
-      // Cap "Done" per the issue AC; if no column is literally named "Done",
-      // fall back to the last board option (conventionally the terminal column)
-      // so --done-limit is honest instead of a silent no-op.
-      const doneGroup = groups.Done ?? groups[statusField.options.at(-1)?.name];
-      if (doneGroup) {
-        doneGroup.items = doneGroup.items.slice(0, args.doneLimit);
-      }
-    }
-    return { ok: true, groups };
-  }
-
-  // 5b. Flat mode: items are returned in position order from GraphQL. Apply limit.
-  const limited = args.limit ? results.slice(0, args.limit) : results;
-
-  return {
-    ok: true,
-    items: limited,
-  };
-}
 
 // ── CLI entrypoint ──────────────────────────────────────────────────────