@devory/github 0.1.0 → 0.3.1

package/dist/index.js

@@ -28,8 +28,8 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 
 // src/index.ts
-var index_exports = {};
-__export(index_exports, {
+var src_exports = {};
+__export(src_exports, {
   appendStepSummary: () => appendStepSummary,
   branchPrefix: () => branchPrefix,
   buildBranchName: () => buildBranchName,
@@ -50,7 +50,7 @@ __export(index_exports, {
   slugify: () => slugify,
   taskScope: () => taskScope
 });
-module.exports = __toCommonJS(index_exports);
+module.exports = __toCommonJS(src_exports);
 
 // src/lib/branch-helpers.ts
 var BRANCH_PREFIX_MAP = {

package/package.json

@@ -1,7 +1,11 @@
 {
   "name": "@devory/github",
-  "version": "0.1.0",
-  "description": "Devory GitHub integration — branch naming, PR helpers, and GitHub Actions support",
+  "version": "0.3.1",
+  "description": "Devory GitHub integration \u2014 branch naming, PR helpers, and GitHub Actions support",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/devoryai/devory"
+  },
   "main": "./dist/index.js",
   "types": "./src/index.ts",
   "exports": {
@@ -9,7 +13,7 @@
   },
   "files": [
     "dist/",
-    "src/",
+    "src/index.ts",
     "README.md"
   ],
   "scripts": {
@@ -18,8 +22,8 @@
     "test": "tsx --test src/test/branch-helpers.test.ts src/test/pr-helpers.test.ts src/test/action-helpers.test.ts"
   },
   "dependencies": {
-    "@devory/cli": "0.1.1",
-    "@devory/core": "0.1.0"
+    "@devory/cli": "0.3.1",
+    "@devory/core": "0.3.1"
   },
   "devDependencies": {
     "esbuild": "^0.20.0",

package/src/lib/action-helpers.ts

@@ -1,99 +0,0 @@
-/**
- * packages/github/src/lib/action-helpers.ts
- *
- * Helpers for emitting outputs and environment variables in a GitHub Actions
- * step context.
- *
- * Two modes:
- *  - Live (default): writes to GITHUB_OUTPUT / GITHUB_ENV files per the
- *    Actions protocol.
- *  - Dry-run / test: returns the lines that *would* be written without any
- *    filesystem side effects.
- *
- * Reference:
- *   https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions
- */
-
-import * as fs from "fs";
-
-// ---------------------------------------------------------------------------
-// Output writing (GITHUB_OUTPUT)
-// ---------------------------------------------------------------------------
-
-/**
- * Set a step output value.  The `name=value` line is appended to the file
- * pointed to by the GITHUB_OUTPUT environment variable.
- *
- * @returns The line that was (or would be) written.
- */
-export function setOutput(name: string, value: string): string {
-  const line = `${name}=${value}`;
-  const outputFile = process.env.GITHUB_OUTPUT;
-  if (outputFile) {
-    fs.appendFileSync(outputFile, line + "\n", "utf-8");
-  }
-  return line;
-}
-
-/**
- * Set multiple step outputs at once.
- * @returns Array of `name=value` lines written.
- */
-export function setOutputs(pairs: Record<string, string>): string[] {
-  return Object.entries(pairs).map(([name, value]) => setOutput(name, value));
-}
-
-// ---------------------------------------------------------------------------
-// Environment variable writing (GITHUB_ENV)
-// ---------------------------------------------------------------------------
-
-/**
- * Export an environment variable for subsequent steps.
- * Appends `NAME=VALUE` to the GITHUB_ENV file.
- *
- * @returns The line that was (or would be) written.
- */
-export function setEnv(name: string, value: string): string {
-  const line = `${name}=${value}`;
-  const envFile = process.env.GITHUB_ENV;
-  if (envFile) {
-    fs.appendFileSync(envFile, line + "\n", "utf-8");
-  }
-  return line;
-}
-
-// ---------------------------------------------------------------------------
-// Step summary (GITHUB_STEP_SUMMARY)
-// ---------------------------------------------------------------------------
-
-/**
- * Append markdown content to the GitHub Actions step summary page.
- *
- * @returns The content that was (or would be) written.
- */
-export function appendStepSummary(markdown: string): string {
-  const summaryFile = process.env.GITHUB_STEP_SUMMARY;
-  if (summaryFile) {
-    fs.appendFileSync(summaryFile, markdown + "\n", "utf-8");
-  }
-  return markdown;
-}
-
-// ---------------------------------------------------------------------------
-// Detection
-// ---------------------------------------------------------------------------
-
-/** Returns true when running inside a GitHub Actions runner. */
-export function isGitHubActions(): boolean {
-  return process.env.GITHUB_ACTIONS === "true";
-}
-
-/** Returns the current GitHub Actions run ID, or null when not in Actions. */
-export function getRunId(): string | null {
-  return process.env.GITHUB_RUN_ID ?? null;
-}
-
-/** Returns the repository slug (`owner/repo`), or null when not in Actions. */
-export function getRepoSlug(): string | null {
-  return process.env.GITHUB_REPOSITORY ?? null;
-}

package/src/lib/branch-helpers.ts

@@ -1,117 +0,0 @@
-/**
- * packages/github/src/lib/branch-helpers.ts
- *
- * Pure helpers for deriving branch names from factory task metadata.
- * No filesystem access. Depends only on @devory/core types.
- *
- * Design rule: if the task already declares a `branch` field, use it.
- * Otherwise derive deterministically from id + title so names are
- * stable across invocations.
- */
-
-import type { TaskMeta } from "@devory/core";
-
-// ---------------------------------------------------------------------------
-// Types
-// ---------------------------------------------------------------------------
-
-export interface BranchResult {
-  /** The derived or confirmed branch name */
-  branch: string;
-  /**
-   * "task-meta" — the task's `branch` field was used as-is.
-   * "derived"   — branch was constructed from id + title slug.
-   */
-  source: "task-meta" | "derived";
-  /** Advisory messages, e.g. unusual characters that were stripped */
-  warnings: string[];
-}
-
-// ---------------------------------------------------------------------------
-// Branch-prefix table
-// Maps task `type` to the branch prefix segment.
-// ---------------------------------------------------------------------------
-
-const BRANCH_PREFIX_MAP: Record<string, string> = {
-  feature: "feat",
-  feat: "feat",
-  bugfix: "fix",
-  bug: "fix",
-  fix: "fix",
-  refactor: "refactor",
-  chore: "chore",
-  documentation: "docs",
-  docs: "docs",
-  test: "test",
-  tests: "test",
-  perf: "perf",
-  performance: "perf",
-};
-
-/** Map a task type to its branch prefix. Unknown types default to "task". */
-export function branchPrefix(taskType: string | undefined): string {
-  const t = (taskType ?? "").toLowerCase().trim();
-  return BRANCH_PREFIX_MAP[t] ?? "task";
-}
-
-// ---------------------------------------------------------------------------
-// Slug helpers
-// ---------------------------------------------------------------------------
-
-/**
- * Convert an arbitrary string to a branch-safe slug.
- * - Lowercased
- * - Non-alphanumeric runs replaced with single hyphens
- * - Leading/trailing hyphens stripped
- * - Truncated to maxLen characters
- */
-export function slugify(s: string, maxLen = 50): string {
-  return s
-    .toLowerCase()
-    .replace(/[^a-z0-9]+/g, "-")
-    .replace(/^-+|-+$/g, "")
-    .slice(0, maxLen);
-}
-
-// ---------------------------------------------------------------------------
-// Main export
-// ---------------------------------------------------------------------------
-
-/**
- * Derive a branch name from task metadata.
- *
- * Resolution order:
- *  1. `meta.branch` — used as-is if present and non-empty
- *  2. `task/<prefix>/<id>-<title-slug>` derived from id + title
- *  3. `task/<id>` if title is empty
- */
-export function buildBranchName(meta: Partial<TaskMeta>): BranchResult {
-  const warnings: string[] = [];
-
-  // Case 1: task declares its own branch name
-  if (typeof meta.branch === "string" && meta.branch.trim()) {
-    return { branch: meta.branch.trim(), source: "task-meta", warnings };
-  }
-
-  const id = typeof meta.id === "string" ? meta.id.trim() : "";
-  const title = typeof meta.title === "string" ? meta.title.trim() : "";
-  const prefix = branchPrefix(typeof meta.type === "string" ? meta.type : undefined);
-
-  const titleSlug = slugify(title, 40);
-
-  // Case 2: id + title slug
-  if (id && titleSlug) {
-    const branch = `${prefix}/${id}-${titleSlug}`;
-    return { branch, source: "derived", warnings };
-  }
-
-  // Case 3: id only (unusual — warn)
-  if (id) {
-    warnings.push("task title is empty; branch slug uses id only");
-    return { branch: `${prefix}/${id}`, source: "derived", warnings };
-  }
-
-  // Fallback: should not happen in a valid task
-  warnings.push("task has no id or title; using fallback branch name");
-  return { branch: "task/unnamed", source: "derived", warnings };
-}

package/src/lib/pr-create.ts

@@ -1,167 +0,0 @@
-/**
- * packages/github/src/lib/pr-create.ts
- *
- * factory-071: Gated PR creation helper.
- *
- * Creates a GitHub PR via the `gh` CLI (https://cli.github.com/).
- * The factory is read-only by default — this module only executes when
- * the caller explicitly passes `options.confirm: true` AND GITHUB_TOKEN
- * is present in the environment.
- *
- * Design constraints:
- *  - No side effects unless `options.confirm === true`
- *  - `GITHUB_TOKEN` must be present; absent token → PrCreateResult.ok false
- *  - `options.branch` must be supplied; the caller is responsible for
- *    ensuring the branch exists before calling createPr
- *  - Uses `gh pr create` — no direct GitHub API calls
- *  - All pure helper functions (canCreatePr, buildGhCreateArgs) are
- *    independently testable with no process spawning
- */
-
-import { spawnSync } from "child_process";
-import type { TaskMeta } from "@devory/core";
-import { buildPrTitle, buildPrBody } from "./pr-helpers.js";
-
-// ---------------------------------------------------------------------------
-// Types
-// ---------------------------------------------------------------------------
-
-export interface PrCreateOptions {
-  /**
-   * Must be explicitly true to create the PR.
-   * When false or absent, createPr() is a no-op and returns ok: false.
-   */
-  confirm: boolean;
-  /** Branch name to create the PR from. Required. */
-  branch: string;
-  /** Base branch. Defaults to "main". */
-  base?: string;
-  /** Override environment (injected for testing; defaults to process.env). */
-  env?: NodeJS.ProcessEnv;
-}
-
-export interface PrCreateResult {
-  ok: boolean;
-  /** URL of the created PR, if successful. */
-  prUrl?: string;
-  /** Human-readable reason when ok is false. */
-  error?: string;
-  /** True when createPr was called without confirm: true (safe no-op). */
-  skipped?: boolean;
-}
-
-// ---------------------------------------------------------------------------
-// Guards
-// ---------------------------------------------------------------------------
-
-/**
- * Returns true when GITHUB_TOKEN is present in the given environment.
- * Does not verify the token is valid — only that it is non-empty.
- */
-export function canCreatePr(env: NodeJS.ProcessEnv = process.env): boolean {
-  const token = env["GITHUB_TOKEN"];
-  return typeof token === "string" && token.trim().length > 0;
-}
-
-/**
- * Returns a human-readable reason why PR creation is not possible,
- * or null if creation should be allowed.
- */
-export function prCreateBlockedReason(
-  options: PrCreateOptions,
-  env: NodeJS.ProcessEnv = process.env
-): string | null {
-  if (!options.confirm) {
-    return "PR creation requires --confirm flag";
-  }
-  if (!options.branch || options.branch.trim().length === 0) {
-    return "PR creation requires a branch name (--branch)";
-  }
-  if (!canCreatePr(env)) {
-    return "GITHUB_TOKEN is not set — cannot create PR";
-  }
-  return null;
-}
-
-// ---------------------------------------------------------------------------
-// Invocation building
-// ---------------------------------------------------------------------------
-
-/**
- * Build the argument list for `gh pr create`.
- * Pure function — no side effects.
- */
-export function buildGhCreateArgs(
-  meta: Partial<TaskMeta>,
-  taskBody: string,
-  options: Pick<PrCreateOptions, "branch" | "base">
-): string[] {
-  const title = buildPrTitle(meta);
-  const body = buildPrBody(meta, taskBody);
-  const base = options.base?.trim() || "main";
-
-  return [
-    "pr",
-    "create",
-    "--title", title,
-    "--body", body,
-    "--head", options.branch,
-    "--base", base,
-  ];
-}
-
-// ---------------------------------------------------------------------------
-// Execution
-// ---------------------------------------------------------------------------
-
-/**
- * Create a GitHub PR via `gh pr create`.
- *
- * Only executes when:
- *   1. `options.confirm === true`
- *   2. GITHUB_TOKEN is present in the environment
- *   3. `options.branch` is non-empty
- *
- * Returns `{ ok: false, skipped: true }` when any guard fails.
- * Returns `{ ok: true, prUrl }` on success.
- * Returns `{ ok: false, error }` on gh execution failure.
- */
-export function createPr(
-  meta: Partial<TaskMeta>,
-  taskBody: string,
-  options: PrCreateOptions
-): PrCreateResult {
-  const env = options.env ?? process.env;
-
-  const blockedReason = prCreateBlockedReason(options, env);
-  if (blockedReason) {
-    return { ok: false, skipped: true, error: blockedReason };
-  }
-
-  const args = buildGhCreateArgs(meta, taskBody, options);
-
-  const result = spawnSync("gh", args, {
-    encoding: "utf-8",
-    env,
-    timeout: 30_000,
-  });
-
-  if (result.error) {
-    return {
-      ok: false,
-      error: `Failed to spawn gh: ${result.error.message}. Is the gh CLI installed?`,
-    };
-  }
-
-  if (result.status !== 0) {
-    const stderr = (result.stderr ?? "").trim();
-    const stdout = (result.stdout ?? "").trim();
-    return {
-      ok: false,
-      error: (stderr || stdout || `gh pr create exited with code ${result.status}`).slice(0, 500),
-    };
-  }
-
-  const prUrl = (result.stdout ?? "").trim();
-  return { ok: true, prUrl: prUrl || undefined };
-}

package/src/lib/pr-helpers.ts

@@ -1,141 +0,0 @@
-/**
- * packages/github/src/lib/pr-helpers.ts
- *
- * Pure helpers for generating GitHub PR titles and body text from factory
- * task metadata and body content.
- *
- * No filesystem access. Depends only on @devory/core types.
- *
- * All functions are deterministic and side-effect free so they can be tested
- * in isolation and composed into GitHub Actions, CLI commands, or UI flows.
- */
-
-import type { TaskMeta } from "@devory/core";
-
-// ---------------------------------------------------------------------------
-// Conventional-commit type table (reused for PR titles)
-// ---------------------------------------------------------------------------
-
-const COMMIT_TYPE_MAP: Record<string, string> = {
-  feature: "feat",
-  feat: "feat",
-  bugfix: "fix",
-  bug: "fix",
-  fix: "fix",
-  refactor: "refactor",
-  chore: "chore",
-  documentation: "docs",
-  docs: "docs",
-  test: "test",
-  tests: "test",
-  perf: "perf",
-  performance: "perf",
-  subtask: "feat",
-};
-
-/** Map task type to a conventional-commit prefix. Defaults to "feat". */
-export function commitType(taskType: string | undefined): string {
-  const t = (taskType ?? "").toLowerCase().trim();
-  return COMMIT_TYPE_MAP[t] ?? "feat";
-}
-
-/** Derive a short scope label. Priority: repo_area → lane → project → "core". */
-export function taskScope(meta: Partial<TaskMeta>): string {
-  const repoArea = typeof meta.repo_area === "string" ? meta.repo_area.trim() : "";
-  const lane = typeof meta.lane === "string" ? meta.lane.trim() : "";
-  const project = typeof meta.project === "string" ? meta.project.trim() : "";
-  return repoArea || lane || project || "core";
-}
-
-// ---------------------------------------------------------------------------
-// PR title
-// ---------------------------------------------------------------------------
-
-/**
- * Build a PR title in conventional-commit format:
- *   `<type>(<scope>): <task title>`
- *
- * Truncated to 72 characters to fit GitHub's recommended title length.
- */
-export function buildPrTitle(meta: Partial<TaskMeta>): string {
-  const type = commitType(typeof meta.type === "string" ? meta.type : undefined);
-  const scope = taskScope(meta);
-  const title = typeof meta.title === "string" ? meta.title.trim() : "(untitled)";
-  const line = `${type}(${scope}): ${title}`;
-  return line.length > 72 ? line.slice(0, 71) + "…" : line;
-}
-
-// ---------------------------------------------------------------------------
-// PR body
-// ---------------------------------------------------------------------------
-
-export interface PrMetadata {
-  title: string;
-  body: string;
-}
-
-/**
- * Build a structured PR description from task metadata and task body text.
- *
- * The body follows GitHub's recommended PR template structure:
- *  - Summary section (task fields)
- *  - Context section (task body if present)
- *  - Verification checklist (from task's verification field)
- *  - Factory metadata footer
- */
-export function buildPrBody(meta: Partial<TaskMeta>, taskBody: string): string {
-  const lines: string[] = [];
-
-  // ── Summary ─────────────────────────────────────────────────────────────
-  lines.push("## Summary", "");
-  lines.push(`- **Task:** ${meta.id ?? "(unknown)"} — ${meta.title ?? "(untitled)"}`);
-  lines.push(`- **Project:** ${meta.project ?? "(unknown)"}`);
-  lines.push(`- **Type:** ${meta.type ?? "feature"}`);
-  lines.push(`- **Priority:** ${meta.priority ?? "medium"}`);
-  lines.push(`- **Agent:** ${meta.agent ?? "(none)"}`);
-
-  const deps = Array.isArray(meta.depends_on) ? meta.depends_on : [];
-  if (deps.length > 0) {
-    lines.push(`- **Depends on:** ${deps.join(", ")}`);
-  }
-  lines.push("");
-
-  // ── Context ──────────────────────────────────────────────────────────────
-  const trimmedBody = (taskBody ?? "").trim();
-  if (trimmedBody) {
-    lines.push("## Context", "");
-    // Include up to ~30 lines of the task body to keep the PR description focused
-    const bodyLines = trimmedBody.split("\n").slice(0, 30);
-    lines.push(...bodyLines);
-    lines.push("");
-  }
-
-  // ── Verification ──────────────────────────────────────────────────────────
-  const verification = Array.isArray(meta.verification) ? meta.verification : [];
-  if (verification.length > 0) {
-    lines.push("## Verification", "");
-    for (const cmd of verification) {
-      lines.push(`- [ ] \`${cmd}\``);
-    }
-    lines.push("");
-  }
-
-  // ── Footer ───────────────────────────────────────────────────────────────
-  lines.push("---");
-  lines.push("*Generated by Devory · ai-dev-factory — human review required before merge.*");
-
-  return lines.join("\n");
-}
-
-/**
- * Build a complete PR metadata object (title + body) from task metadata.
- */
-export function buildPrMetadata(
-  meta: Partial<TaskMeta>,
-  taskBody: string
-): PrMetadata {
-  return {
-    title: buildPrTitle(meta),
-    body: buildPrBody(meta, taskBody),
-  };
-}

package/src/test/action-helpers.test.ts

@@ -1,158 +0,0 @@
-/**
- * packages/github/src/test/action-helpers.test.ts
- *
- * Tests for src/lib/action-helpers.ts.
- * Verifies the correct output format without actually writing to env files.
- *
- * Run: tsx --test packages/github/src/test/action-helpers.test.ts
- */
-
-import { test, describe, before, after } from "node:test";
-import assert from "node:assert/strict";
-import * as fs from "fs";
-import * as os from "os";
-import * as path from "path";
-
-import {
-  setOutput,
-  setOutputs,
-  setEnv,
-  appendStepSummary,
-  isGitHubActions,
-  getRunId,
-  getRepoSlug,
-} from "../lib/action-helpers.js";
-
-// ── Return value tests (no env files) ─────────────────────────────────────
-
-describe("setOutput return value", () => {
-  test("returns name=value string", () => {
-    const line = setOutput("branch", "feat/factory-066");
-    assert.equal(line, "branch=feat/factory-066");
-  });
-
-  test("handles values with spaces", () => {
-    const line = setOutput("title", "feat(core): My Title");
-    assert.equal(line, "title=feat(core): My Title");
-  });
-});
-
-describe("setOutputs return value", () => {
-  test("returns array of name=value strings", () => {
-    const lines = setOutputs({ branch: "feat/x", title: "feat(p): T" });
-    assert.equal(lines.length, 2);
-    assert.ok(lines.includes("branch=feat/x"));
-    assert.ok(lines.includes("title=feat(p): T"));
-  });
-
-  test("returns empty array for empty input", () => {
-    const lines = setOutputs({});
-    assert.deepEqual(lines, []);
-  });
-});
-
-describe("setEnv return value", () => {
-  test("returns NAME=value string", () => {
-    const line = setEnv("DEVORY_BRANCH", "feat/my-branch");
-    assert.equal(line, "DEVORY_BRANCH=feat/my-branch");
-  });
-});
-
-describe("appendStepSummary return value", () => {
-  test("returns the markdown passed in", () => {
-    const md = "# My Summary\nSome content.";
-    const result = appendStepSummary(md);
-    assert.equal(result, md);
-  });
-});
-
-// ── Detection helpers ──────────────────────────────────────────────────────
-
-describe("isGitHubActions", () => {
-  test("returns false when GITHUB_ACTIONS is not set", () => {
-    const saved = process.env.GITHUB_ACTIONS;
-    delete process.env.GITHUB_ACTIONS;
-    assert.equal(isGitHubActions(), false);
-    if (saved !== undefined) process.env.GITHUB_ACTIONS = saved;
-  });
-
-  test("returns true when GITHUB_ACTIONS=true", () => {
-    const saved = process.env.GITHUB_ACTIONS;
-    process.env.GITHUB_ACTIONS = "true";
-    assert.equal(isGitHubActions(), true);
-    if (saved !== undefined) process.env.GITHUB_ACTIONS = saved;
-    else delete process.env.GITHUB_ACTIONS;
-  });
-});
-
-describe("getRunId", () => {
-  test("returns null when GITHUB_RUN_ID not set", () => {
-    const saved = process.env.GITHUB_RUN_ID;
-    delete process.env.GITHUB_RUN_ID;
-    assert.equal(getRunId(), null);
-    if (saved !== undefined) process.env.GITHUB_RUN_ID = saved;
-  });
-
-  test("returns run ID string when set", () => {
-    const saved = process.env.GITHUB_RUN_ID;
-    process.env.GITHUB_RUN_ID = "12345";
-    assert.equal(getRunId(), "12345");
-    if (saved !== undefined) process.env.GITHUB_RUN_ID = saved;
-    else delete process.env.GITHUB_RUN_ID;
-  });
-});
-
-describe("getRepoSlug", () => {
-  test("returns null when GITHUB_REPOSITORY not set", () => {
-    const saved = process.env.GITHUB_REPOSITORY;
-    delete process.env.GITHUB_REPOSITORY;
-    assert.equal(getRepoSlug(), null);
-    if (saved !== undefined) process.env.GITHUB_REPOSITORY = saved;
-  });
-
-  test("returns owner/repo when set", () => {
-    const saved = process.env.GITHUB_REPOSITORY;
-    process.env.GITHUB_REPOSITORY = "devory/ai-dev-factory";
-    assert.equal(getRepoSlug(), "devory/ai-dev-factory");
-    if (saved !== undefined) process.env.GITHUB_REPOSITORY = saved;
-    else delete process.env.GITHUB_REPOSITORY;
-  });
-});
-
-// ── Live file writing (with GITHUB_OUTPUT set) ─────────────────────────────
-
-describe("setOutput file writing", () => {
-  let tmpDir: string;
-  let outputFile: string;
-  const originalEnv = process.env.GITHUB_OUTPUT;
-
-  before(() => {
-    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "devory-gh-test-"));
-    outputFile = path.join(tmpDir, "output");
-    fs.writeFileSync(outputFile, "", "utf-8");
-    process.env.GITHUB_OUTPUT = outputFile;
-  });
-
-  after(() => {
-    fs.rmSync(tmpDir, { recursive: true, force: true });
-    if (originalEnv !== undefined) {
-      process.env.GITHUB_OUTPUT = originalEnv;
-    } else {
-      delete process.env.GITHUB_OUTPUT;
-    }
-  });
-
-  test("appends name=value to GITHUB_OUTPUT file", () => {
-    setOutput("my-key", "my-value");
-    const content = fs.readFileSync(outputFile, "utf-8");
-    assert.ok(content.includes("my-key=my-value"));
-  });
-
-  test("multiple outputs are each on their own line", () => {
-    fs.writeFileSync(outputFile, "", "utf-8"); // reset
-    setOutput("a", "1");
-    setOutput("b", "2");
-    const lines = fs.readFileSync(outputFile, "utf-8").trim().split("\n");
-    assert.equal(lines.length, 2);
-  });
-});

package/src/test/branch-helpers.test.ts

@@ -1,168 +0,0 @@
-/**
- * packages/github/src/test/branch-helpers.test.ts
- *
- * Tests for src/lib/branch-helpers.ts.
- * Run: tsx --test packages/github/src/test/branch-helpers.test.ts
- */
-
-import { test, describe } from "node:test";
-import assert from "node:assert/strict";
-
-import {
-  slugify,
-  branchPrefix,
-  buildBranchName,
-} from "../lib/branch-helpers.js";
-
-// ── slugify ────────────────────────────────────────────────────────────────
-
-describe("slugify", () => {
-  test("lowercases input", () => {
-    assert.equal(slugify("Hello World"), "hello-world");
-  });
-
-  test("replaces spaces with hyphens", () => {
-    assert.equal(slugify("foo bar baz"), "foo-bar-baz");
-  });
-
-  test("collapses multiple non-alphanumeric chars to one hyphen", () => {
-    assert.equal(slugify("foo  --  bar"), "foo-bar");
-  });
-
-  test("strips leading and trailing hyphens", () => {
-    assert.equal(slugify("  hello  "), "hello");
-  });
-
-  test("handles numbers", () => {
-    assert.equal(slugify("Factory 066 MVP"), "factory-066-mvp");
-  });
-
-  test("truncates at maxLen", () => {
-    const long = "a".repeat(100);
-    assert.equal(slugify(long, 10).length, 10);
-  });
-
-  test("empty string returns empty string", () => {
-    assert.equal(slugify(""), "");
-  });
-
-  test("strips special characters", () => {
-    assert.equal(slugify("feat: add @devory/core"), "feat-add-devory-core");
-  });
-});
-
-// ── branchPrefix ──────────────────────────────────────────────────────────
-
-describe("branchPrefix", () => {
-  test("feature → feat", () => assert.equal(branchPrefix("feature"), "feat"));
-  test("feat → feat", () => assert.equal(branchPrefix("feat"), "feat"));
-  test("bugfix → fix", () => assert.equal(branchPrefix("bugfix"), "fix"));
-  test("bug → fix", () => assert.equal(branchPrefix("bug"), "fix"));
-  test("refactor → refactor", () => assert.equal(branchPrefix("refactor"), "refactor"));
-  test("chore → chore", () => assert.equal(branchPrefix("chore"), "chore"));
-  test("docs → docs", () => assert.equal(branchPrefix("docs"), "docs"));
-  test("documentation → docs", () => assert.equal(branchPrefix("documentation"), "docs"));
-  test("unknown type defaults to task", () => assert.equal(branchPrefix("unknown"), "task"));
-  test("undefined defaults to task", () => assert.equal(branchPrefix(undefined), "task"));
-  test("case-insensitive", () => assert.equal(branchPrefix("FEATURE"), "feat"));
-});
-
-// ── buildBranchName ────────────────────────────────────────────────────────
-
-describe("buildBranchName — task-meta source", () => {
-  test("uses meta.branch when present", () => {
-    const r = buildBranchName({ branch: "task/factory-066-my-feature" });
-    assert.equal(r.branch, "task/factory-066-my-feature");
-    assert.equal(r.source, "task-meta");
-    assert.equal(r.warnings.length, 0);
-  });
-
-  test("uses meta.branch even if type/id/title are present", () => {
-    const r = buildBranchName({
-      branch: "custom/my-branch",
-      id: "factory-001",
-      title: "Something else",
-      type: "feature",
-    });
-    assert.equal(r.branch, "custom/my-branch");
-    assert.equal(r.source, "task-meta");
-  });
-
-  test("trims whitespace from meta.branch", () => {
-    const r = buildBranchName({ branch: "  feat/some-branch  " });
-    assert.equal(r.branch, "feat/some-branch");
-  });
-});
-
-describe("buildBranchName — derived source", () => {
-  test("derives from id and title when no branch field", () => {
-    const r = buildBranchName({
-      id: "factory-066",
-      title: "GitHub Integration MVP",
-      type: "feature",
-    });
-    assert.equal(r.source, "derived");
-    assert.ok(r.branch.includes("factory-066"));
-    assert.ok(r.branch.includes("github-integration-mvp"));
-  });
-
-  test("branch starts with type-based prefix", () => {
-    const r = buildBranchName({ id: "x", title: "y", type: "bugfix" });
-    assert.ok(r.branch.startsWith("fix/"));
-  });
-
-  test("branch starts with 'task/' for unknown type", () => {
-    const r = buildBranchName({ id: "x", title: "y", type: "unknown" });
-    assert.ok(r.branch.startsWith("task/"));
-  });
-
-  test("branch starts with 'task/' when type is missing", () => {
-    const r = buildBranchName({ id: "x", title: "y" });
-    assert.ok(r.branch.startsWith("task/"));
-  });
-
-  test("branch contains id", () => {
-    const r = buildBranchName({ id: "factory-099", title: "My Task" });
-    assert.ok(r.branch.includes("factory-099"));
-  });
-
-  test("branch slug is lowercase", () => {
-    const r = buildBranchName({ id: "x", title: "Hello World" });
-    assert.equal(r.branch, r.branch.toLowerCase());
-  });
-
-  test("handles title with special chars", () => {
-    const r = buildBranchName({ id: "f-001", title: "Add @devory/core to repo" });
-    assert.ok(!r.branch.includes("@"));
-    assert.ok(!r.branch.includes("/devory"));
-  });
-
-  test("falls back to id-only when title is empty", () => {
-    const r = buildBranchName({ id: "factory-001", title: "" });
-    assert.equal(r.source, "derived");
-    assert.ok(r.branch.includes("factory-001"));
-    assert.equal(r.warnings.length, 1);
-  });
-
-  test("returns fallback branch when id and title both empty", () => {
-    const r = buildBranchName({});
-    assert.equal(r.branch, "task/unnamed");
-    assert.ok(r.warnings.length > 0);
-  });
-});
-
-describe("buildBranchName — branch name format", () => {
-  test("branch contains no whitespace", () => {
-    const r = buildBranchName({ id: "factory-066", title: "Multi Word Title Here" });
-    assert.ok(!/\s/.test(r.branch));
-  });
-
-  test("branch length is reasonable (under 80 chars)", () => {
-    const r = buildBranchName({
-      id: "factory-066",
-      title: "A very long task title that might overflow the branch name limit",
-      type: "feature",
-    });
-    assert.ok(r.branch.length <= 80);
-  });
-});

package/src/test/pr-create.test.ts

@@ -1,223 +0,0 @@
-/**
- * packages/github/src/test/pr-create.test.ts
- *
- * Tests for src/lib/pr-create.ts — gated PR creation helper.
- * All tests use pure functions; no process spawning.
- *
- * Run: tsx --test packages/github/src/test/pr-create.test.ts
- */
-
-import { test, describe } from "node:test";
-import assert from "node:assert/strict";
-
-import {
-  canCreatePr,
-  prCreateBlockedReason,
-  buildGhCreateArgs,
-  createPr,
-} from "../lib/pr-create.js";
-
-const META = {
-  id: "factory-071",
-  title: "GitHub PR creation (gated automation)",
-  project: "ai-dev-factory",
-  type: "feature" as const,
-  priority: "high" as const,
-  status: "review" as const,
-  repo: ".",
-  branch: "task/factory-071-github-pr-creation",
-};
-
-const BODY = "## Goal\n\nAdd gated PR creation to the factory.";
-
-// ---------------------------------------------------------------------------
-// canCreatePr
-// ---------------------------------------------------------------------------
-
-describe("canCreatePr", () => {
-  test("returns true when GITHUB_TOKEN is set", () => {
-    assert.equal(canCreatePr({ GITHUB_TOKEN: "ghp_test_token" }), true);
-  });
-
-  test("returns false when GITHUB_TOKEN is absent", () => {
-    assert.equal(canCreatePr({}), false);
-  });
-
-  test("returns false when GITHUB_TOKEN is empty string", () => {
-    assert.equal(canCreatePr({ GITHUB_TOKEN: "" }), false);
-  });
-
-  test("returns false when GITHUB_TOKEN is whitespace only", () => {
-    assert.equal(canCreatePr({ GITHUB_TOKEN: "   " }), false);
-  });
-
-  test("returns true for non-empty token with whitespace padding", () => {
-    // Token itself has content — only the full value is checked
-    assert.equal(canCreatePr({ GITHUB_TOKEN: " ghp_abc " }), true);
-  });
-});
-
-// ---------------------------------------------------------------------------
-// prCreateBlockedReason
-// ---------------------------------------------------------------------------
-
-describe("prCreateBlockedReason", () => {
-  const validOptions = {
-    confirm: true,
-    branch: "task/factory-071",
-    env: { GITHUB_TOKEN: "ghp_test" },
-  };
-
-  test("returns null when all guards pass", () => {
-    assert.equal(
-      prCreateBlockedReason(validOptions, { GITHUB_TOKEN: "ghp_test" }),
-      null
-    );
-  });
-
-  test("blocks when confirm is false", () => {
-    const reason = prCreateBlockedReason(
-      { ...validOptions, confirm: false },
-      { GITHUB_TOKEN: "ghp_test" }
-    );
-    assert.ok(reason !== null);
-    assert.ok(reason!.includes("--confirm"));
-  });
-
-  test("blocks when branch is empty string", () => {
-    const reason = prCreateBlockedReason(
-      { ...validOptions, branch: "" },
-      { GITHUB_TOKEN: "ghp_test" }
-    );
-    assert.ok(reason !== null);
-    assert.ok(reason!.includes("branch"));
-  });
-
-  test("blocks when branch is whitespace only", () => {
-    const reason = prCreateBlockedReason(
-      { ...validOptions, branch: "   " },
-      { GITHUB_TOKEN: "ghp_test" }
-    );
-    assert.ok(reason !== null);
-    assert.ok(reason!.includes("branch"));
-  });
-
-  test("blocks when GITHUB_TOKEN is absent", () => {
-    const reason = prCreateBlockedReason(validOptions, {});
-    assert.ok(reason !== null);
-    assert.ok(reason!.includes("GITHUB_TOKEN"));
-  });
-
-  test("confirm guard takes priority over token guard", () => {
-    const reason = prCreateBlockedReason(
-      { ...validOptions, confirm: false },
-      {} // no token
-    );
-    assert.ok(reason !== null);
-    assert.ok(reason!.includes("--confirm"));
-  });
-});
-
-// ---------------------------------------------------------------------------
-// buildGhCreateArgs
-// ---------------------------------------------------------------------------
-
-describe("buildGhCreateArgs", () => {
-  test("includes 'pr create' subcommand", () => {
-    const args = buildGhCreateArgs(META, BODY, { branch: "task/factory-071" });
-    assert.equal(args[0], "pr");
-    assert.equal(args[1], "create");
-  });
-
-  test("includes --title derived from meta", () => {
-    const args = buildGhCreateArgs(META, BODY, { branch: "task/factory-071" });
-    const i = args.indexOf("--title");
-    assert.ok(i >= 0);
-    assert.ok(args[i + 1].includes("GitHub PR creation"));
-  });
-
-  test("includes --body derived from meta + task body", () => {
-    const args = buildGhCreateArgs(META, BODY, { branch: "task/factory-071" });
-    const i = args.indexOf("--body");
-    assert.ok(i >= 0);
-    assert.ok(args[i + 1].includes("factory-071"));
-  });
-
-  test("includes --head with branch name", () => {
-    const args = buildGhCreateArgs(META, BODY, { branch: "task/factory-071" });
-    const i = args.indexOf("--head");
-    assert.ok(i >= 0 && args[i + 1] === "task/factory-071");
-  });
-
-  test("defaults --base to main", () => {
-    const args = buildGhCreateArgs(META, BODY, { branch: "task/factory-071" });
-    const i = args.indexOf("--base");
-    assert.ok(i >= 0 && args[i + 1] === "main");
-  });
-
-  test("uses custom --base when provided", () => {
-    const args = buildGhCreateArgs(META, BODY, { branch: "task/factory-071", base: "develop" });
-    const i = args.indexOf("--base");
-    assert.ok(i >= 0 && args[i + 1] === "develop");
-  });
-
-  test("title follows conventional-commit format", () => {
-    const args = buildGhCreateArgs(META, BODY, { branch: "task/factory-071" });
-    const i = args.indexOf("--title");
-    assert.ok(args[i + 1].startsWith("feat("));
-  });
-
-  test("body contains summary section", () => {
-    const args = buildGhCreateArgs(META, BODY, { branch: "task/factory-071" });
-    const i = args.indexOf("--body");
-    assert.ok(args[i + 1].includes("## Summary"));
-  });
-});
-
-// ---------------------------------------------------------------------------
-// createPr — guard paths (no process spawning)
-// ---------------------------------------------------------------------------
-
-describe("createPr — guard paths", () => {
-  test("returns skipped when confirm is false", () => {
-    const result = createPr(META, BODY, {
-      confirm: false,
-      branch: "task/factory-071",
-      env: { GITHUB_TOKEN: "ghp_test" },
-    });
-    assert.equal(result.ok, false);
-    assert.equal(result.skipped, true);
-    assert.ok(result.error?.includes("--confirm"));
-  });
-
-  test("returns skipped when GITHUB_TOKEN is absent", () => {
-    const result = createPr(META, BODY, {
-      confirm: true,
-      branch: "task/factory-071",
-      env: {},
-    });
-    assert.equal(result.ok, false);
-    assert.equal(result.skipped, true);
-    assert.ok(result.error?.includes("GITHUB_TOKEN"));
-  });
-
-  test("returns skipped when branch is empty", () => {
-    const result = createPr(META, BODY, {
-      confirm: true,
-      branch: "",
-      env: { GITHUB_TOKEN: "ghp_test" },
-    });
-    assert.equal(result.ok, false);
-    assert.equal(result.skipped, true);
-    assert.ok(result.error?.includes("branch"));
-  });
-
-  test("skipped result has no prUrl", () => {
-    const result = createPr(META, BODY, {
-      confirm: false,
-      branch: "task/factory-071",
-      env: { GITHUB_TOKEN: "ghp_test" },
-    });
-    assert.equal(result.prUrl, undefined);
-  });
-});

package/src/test/pr-helpers.test.ts

@@ -1,187 +0,0 @@
-/**
- * packages/github/src/test/pr-helpers.test.ts
- *
- * Tests for src/lib/pr-helpers.ts.
- * Run: tsx --test packages/github/src/test/pr-helpers.test.ts
- */
-
-import { test, describe } from "node:test";
-import assert from "node:assert/strict";
-
-import {
-  commitType,
-  taskScope,
-  buildPrTitle,
-  buildPrBody,
-  buildPrMetadata,
-} from "../lib/pr-helpers.js";
-
-// ── commitType ─────────────────────────────────────────────────────────────
-
-describe("commitType", () => {
-  test("feature → feat", () => assert.equal(commitType("feature"), "feat"));
-  test("feat → feat", () => assert.equal(commitType("feat"), "feat"));
-  test("bugfix → fix", () => assert.equal(commitType("bugfix"), "fix"));
-  test("bug → fix", () => assert.equal(commitType("bug"), "fix"));
-  test("refactor → refactor", () => assert.equal(commitType("refactor"), "refactor"));
-  test("chore → chore", () => assert.equal(commitType("chore"), "chore"));
-  test("docs → docs", () => assert.equal(commitType("docs"), "docs"));
-  test("documentation → docs", () => assert.equal(commitType("documentation"), "docs"));
-  test("test → test", () => assert.equal(commitType("test"), "test"));
-  test("perf → perf", () => assert.equal(commitType("perf"), "perf"));
-  test("subtask → feat", () => assert.equal(commitType("subtask"), "feat"));
-  test("unknown → feat", () => assert.equal(commitType("unknown"), "feat"));
-  test("undefined → feat", () => assert.equal(commitType(undefined), "feat"));
-  test("case-insensitive", () => assert.equal(commitType("FEATURE"), "feat"));
-});
-
-// ── taskScope ──────────────────────────────────────────────────────────────
-
-describe("taskScope", () => {
-  test("uses repo_area first", () => {
-    assert.equal(taskScope({ repo_area: "api", lane: "infra", project: "proj" }), "api");
-  });
-
-  test("falls back to lane when no repo_area", () => {
-    assert.equal(taskScope({ lane: "infra", project: "proj" }), "infra");
-  });
-
-  test("falls back to project when no repo_area or lane", () => {
-    assert.equal(taskScope({ project: "ai-dev-factory" }), "ai-dev-factory");
-  });
-
-  test("defaults to 'core' when all empty", () => {
-    assert.equal(taskScope({}), "core");
-  });
-
-  test("trims whitespace", () => {
-    assert.equal(taskScope({ repo_area: "  api  " }), "api");
-  });
-});
-
-// ── buildPrTitle ───────────────────────────────────────────────────────────
-
-describe("buildPrTitle", () => {
-  test("builds conventional-commit title", () => {
-    const title = buildPrTitle({
-      type: "feature",
-      project: "ai-dev-factory",
-      title: "GitHub Integration MVP",
-    });
-    assert.equal(title, "feat(ai-dev-factory): GitHub Integration MVP");
-  });
-
-  test("uses repo_area as scope when present", () => {
-    const title = buildPrTitle({
-      type: "bugfix",
-      repo_area: "api",
-      title: "Fix null pointer",
-    });
-    assert.ok(title.startsWith("fix(api):"));
-  });
-
-  test("truncates to 72 chars with ellipsis", () => {
-    const longTitle = "A".repeat(100);
-    const title = buildPrTitle({ type: "feature", project: "p", title: longTitle });
-    assert.ok(title.length <= 72);
-    assert.ok(title.endsWith("…"));
-  });
-
-  test("does not truncate short titles", () => {
-    const title = buildPrTitle({ type: "chore", project: "proj", title: "short" });
-    assert.ok(title.length < 72);
-    assert.ok(!title.endsWith("…"));
-  });
-
-  test("handles missing type (defaults to feat)", () => {
-    const title = buildPrTitle({ project: "p", title: "My task" });
-    assert.ok(title.startsWith("feat("));
-  });
-
-  test("handles missing title", () => {
-    const title = buildPrTitle({ type: "feature", project: "p" });
-    assert.ok(title.includes("(untitled)"));
-  });
-});
-
-// ── buildPrBody ────────────────────────────────────────────────────────────
-
-describe("buildPrBody", () => {
-  const meta = {
-    id: "factory-066",
-    title: "GitHub Integration MVP",
-    project: "ai-dev-factory",
-    type: "feature",
-    priority: "medium",
-    agent: "fullstack-builder",
-    depends_on: ["factory-054", "factory-063"],
-    verification: ["npm run test", "npm run validate:task -- tasks/backlog/factory-066.md"],
-  };
-
-  test("includes task id in body", () => {
-    const body = buildPrBody(meta, "");
-    assert.ok(body.includes("factory-066"));
-  });
-
-  test("includes task title in body", () => {
-    const body = buildPrBody(meta, "");
-    assert.ok(body.includes("GitHub Integration MVP"));
-  });
-
-  test("includes project in body", () => {
-    const body = buildPrBody(meta, "");
-    assert.ok(body.includes("ai-dev-factory"));
-  });
-
-  test("includes depends_on when present", () => {
-    const body = buildPrBody(meta, "");
-    assert.ok(body.includes("factory-054"));
-    assert.ok(body.includes("factory-063"));
-  });
-
-  test("includes verification commands as checklist items", () => {
-    const body = buildPrBody(meta, "");
-    assert.ok(body.includes("- [ ] `npm run test`"));
-  });
-
-  test("includes task body content when provided", () => {
-    const body = buildPrBody(meta, "## Goal\nDo the thing.");
-    assert.ok(body.includes("Do the thing."));
-  });
-
-  test("omits context section when body is empty", () => {
-    const body = buildPrBody(meta, "");
-    assert.ok(!body.includes("## Context"));
-  });
-
-  test("includes safety footer", () => {
-    const body = buildPrBody(meta, "");
-    assert.ok(body.includes("human review required before merge"));
-  });
-
-  test("includes Summary section header", () => {
-    const body = buildPrBody(meta, "");
-    assert.ok(body.includes("## Summary"));
-  });
-
-  test("omits verification section when verification is empty", () => {
-    const body = buildPrBody({ ...meta, verification: [] }, "");
-    assert.ok(!body.includes("## Verification"));
-  });
-});
-
-// ── buildPrMetadata ────────────────────────────────────────────────────────
-
-describe("buildPrMetadata", () => {
-  test("returns object with title and body", () => {
-    const result = buildPrMetadata({ type: "feature", project: "p", title: "T" }, "");
-    assert.ok(typeof result.title === "string" && result.title.length > 0);
-    assert.ok(typeof result.body === "string" && result.body.length > 0);
-  });
-
-  test("title matches buildPrTitle output", () => {
-    const meta = { type: "bugfix", project: "proj", title: "Fix bug" };
-    const result = buildPrMetadata(meta, "");
-    assert.equal(result.title, buildPrTitle(meta));
-  });
-});