sequant 1.20.3 → 2.0.1

package/dist/src/lib/workflow/phase-mapper.d.ts

@@ -36,7 +36,7 @@ export declare const COMPLEX_LABELS: string[];
  */
 export declare const SECURITY_LABELS: string[];
 /**
- * Detect phases based on issue labels (like /solve logic)
+ * Detect phases based on issue labels (like /assess logic)
  */
 export declare function detectPhasesFromLabels(labels: string[]): {
     phases: Phase[];

package/dist/src/lib/workflow/phase-mapper.js

@@ -34,20 +34,20 @@ export const SECURITY_LABELS = [
     "admin",
 ];
 /**
- * Detect phases based on issue labels (like /solve logic)
+ * Detect phases based on issue labels (like /assess logic)
  */
 export function detectPhasesFromLabels(labels) {
     const lowerLabels = labels.map((l) => l.toLowerCase());
     // Check for bug/fix labels → exec → qa (skip spec)
-    const isBugFix = lowerLabels.some((label) => BUG_LABELS.some((bugLabel) => label.includes(bugLabel)));
+    const isBugFix = lowerLabels.some((label) => BUG_LABELS.some((bugLabel) => label === bugLabel));
     // Check for docs labels → exec → qa (skip spec)
-    const isDocs = lowerLabels.some((label) => DOCS_LABELS.some((docsLabel) => label.includes(docsLabel)));
+    const isDocs = lowerLabels.some((label) => DOCS_LABELS.some((docsLabel) => label === docsLabel));
     // Check for UI labels → add test phase
-    const isUI = lowerLabels.some((label) => UI_LABELS.some((uiLabel) => label.includes(uiLabel)));
+    const isUI = lowerLabels.some((label) => UI_LABELS.some((uiLabel) => label === uiLabel));
     // Check for complex labels → enable quality loop
-    const isComplex = lowerLabels.some((label) => COMPLEX_LABELS.some((complexLabel) => label.includes(complexLabel)));
+    const isComplex = lowerLabels.some((label) => COMPLEX_LABELS.some((complexLabel) => label === complexLabel));
     // Check for security labels → add security-review phase
-    const isSecurity = lowerLabels.some((label) => SECURITY_LABELS.some((secLabel) => label.includes(secLabel)));
+    const isSecurity = lowerLabels.some((label) => SECURITY_LABELS.some((secLabel) => label === secLabel));
     // Build phase list
     let phases;
     if (isBugFix || isDocs) {
@@ -118,7 +118,7 @@ export function parseRecommendedWorkflow(output) {
  * Check if an issue has UI-related labels
  */
 export function hasUILabels(labels) {
-    return labels.some((label) => UI_LABELS.some((uiLabel) => label.toLowerCase().includes(uiLabel)));
+    return labels.some((label) => UI_LABELS.some((uiLabel) => label.toLowerCase() === uiLabel));
 }
 /**
  * Determine phases to run based on options and issue labels

package/dist/src/lib/workflow/platforms/github.d.ts

@@ -0,0 +1,157 @@
+/**
+ * GitHubProvider — PlatformProvider implementation wrapping the `gh` CLI.
+ *
+ * Owns all `gh` CLI calls for the orchestration layer. Skills continue
+ * to call `gh` directly for v1 (see Non-Goals in #368).
+ *
+ * Sync methods are provided for callers that are currently synchronous
+ * (phase-detection, pr-status, system, doctor, worktree-manager).
+ * Async interface methods delegate to the sync implementations.
+ */
+import type { PlatformProvider, Issue, CreatePROptions, PRInfo, PRStatus, Comment } from "./platform-provider.js";
+/**
+ * PR merge status values (matches the casing returned by `gh pr view`).
+ */
+export type PRMergeStatus = "MERGED" | "CLOSED" | "OPEN" | null;
+/**
+ * Closed issue shape returned by `gh issue list --state closed`.
+ */
+export interface ClosedIssueRaw {
+    number: number;
+    title: string;
+    closedAt: string;
+    labels: {
+        name: string;
+    }[];
+}
+/**
+ * Result of a raw `gh pr create` CLI call.
+ */
+export interface CreatePRCliResult {
+    stdout: string;
+    stderr: string;
+    exitCode: number | null;
+}
+/**
+ * Info returned for an issue in a batch query.
+ */
+export interface BatchIssueInfo {
+    number: number;
+    title: string;
+    state: "OPEN" | "CLOSED";
+}
+/**
+ * Info returned for a PR in a batch query.
+ */
+export interface BatchPRInfo {
+    number: number;
+    state: "OPEN" | "MERGED" | "CLOSED";
+}
+/**
+ * Result of a batch GitHub query.
+ */
+export interface BatchGitHubResult {
+    issues: Record<number, BatchIssueInfo>;
+    pullRequests: Record<number, BatchPRInfo>;
+    error?: string;
+}
+export declare class GitHubProvider implements PlatformProvider {
+    name: string;
+    /**
+     * Fetch issue comment bodies as a string array.
+     * Used by phase-detection.ts for phase marker parsing.
+     */
+    fetchIssueCommentBodiesSync(issueId: string): string[];
+    /**
+     * Check if `gh` CLI is authenticated.
+     * Used by system.ts and doctor.ts.
+     */
+    checkAuthSync(): boolean;
+    /**
+     * Get PR merge status by PR number.
+     * Used by pr-status.ts and state-cleanup.ts.
+     */
+    getPRMergeStatusSync(prNumber: number): PRMergeStatus;
+    /**
+     * List recently closed issues.
+     * Used by doctor.ts for closed-issue verification.
+     */
+    listClosedIssuesSync(limit?: number): ClosedIssueRaw[];
+    /**
+     * View a PR by branch name, returning number and URL.
+     * Used by worktree-manager.ts to check for existing PRs.
+     */
+    viewPRByBranchSync(branch: string, cwd?: string): {
+        number: number;
+        url: string;
+    } | null;
+    /**
+     * Get the head branch name for a PR by number.
+     * Used by hooks/pre-tool.sh for pre-merge worktree cleanup.
+     */
+    getPRHeadBranchSync(prNumber: number): string | null;
+    /**
+     * Create a PR via `gh pr create` CLI, returning raw result.
+     * Used by worktree-manager.ts which needs access to stdout for URL extraction.
+     */
+    createPRCliSync(title: string, body: string, head: string, cwd?: string): CreatePRCliResult;
+    /**
+     * Batch fetch issue and PR status in a single GraphQL call.
+     * Returns a map keyed by issue/PR number.
+     */
+    batchFetchIssueAndPRStatus(issueNumbers: number[], prNumbers: number[]): BatchGitHubResult;
+    /**
+     * Fetch just the title of an issue by number.
+     * Used by merge-check and worktree-discovery.
+     */
+    fetchIssueTitleSync(issueId: string): string | null;
+    /**
+     * Check if the `gh` CLI binary is installed (not auth, just available).
+     * Used by upstream/assessment.ts for pre-flight checks.
+     */
+    checkGhInstalledSync(): boolean;
+    /**
+     * Fetch a single release from a specific repo.
+     * If `version` is omitted, fetches the latest release.
+     * Used by upstream/assessment.ts.
+     */
+    fetchReleaseSync(repo: string, version?: string): Record<string, unknown> | null;
+    /**
+     * List releases from a specific repo.
+     * Used by upstream/assessment.ts.
+     */
+    listReleasesSync(repo: string, limit?: number): Array<{
+        tagName: string;
+        publishedAt: string;
+    }>;
+    /**
+     * Search issues in a specific repo by labels and search query.
+     * Used by upstream/issues.ts for duplicate detection.
+     */
+    searchIssuesSync(repo: string, labels: string[], search: string, limit?: number): Array<{
+        number: number;
+        title: string;
+    }>;
+    /**
+     * Create an issue in a specific repo using a body file.
+     * Used by upstream/issues.ts.
+     */
+    createIssueWithBodyFileSync(repo: string, title: string, bodyFile: string, labels: string[]): {
+        number: number;
+        url: string;
+    } | null;
+    /**
+     * Add a comment to an issue in a specific repo using a body file.
+     * Used by upstream/issues.ts.
+     */
+    commentOnIssueWithBodyFileSync(repo: string, issueNumber: number, bodyFile: string): boolean;
+    fetchIssue(id: string): Promise<Issue>;
+    postComment(issueId: string, body: string): Promise<void>;
+    addLabel(issueId: string, label: string): Promise<void>;
+    removeLabel(issueId: string, label: string): Promise<void>;
+    createPR(opts: CreatePROptions): Promise<PRInfo>;
+    getPRStatus(prId: string): Promise<PRStatus>;
+    postPRComment(prId: string, body: string): Promise<void>;
+    checkAuth(): Promise<boolean>;
+    getIssueComments(issueId: string): Promise<Comment[]>;
+}

package/dist/src/lib/workflow/platforms/github.js

@@ -0,0 +1,466 @@
+/**
+ * GitHubProvider — PlatformProvider implementation wrapping the `gh` CLI.
+ *
+ * Owns all `gh` CLI calls for the orchestration layer. Skills continue
+ * to call `gh` directly for v1 (see Non-Goals in #368).
+ *
+ * Sync methods are provided for callers that are currently synchronous
+ * (phase-detection, pr-status, system, doctor, worktree-manager).
+ * Async interface methods delegate to the sync implementations.
+ */
+import { execSync, spawnSync } from "child_process";
+export class GitHubProvider {
+    name = "github";
+    // ─── Sync helpers (for synchronous callers) ────────────────────────
+    /**
+     * Fetch issue comment bodies as a string array.
+     * Used by phase-detection.ts for phase marker parsing.
+     */
+    fetchIssueCommentBodiesSync(issueId) {
+        try {
+            const result = spawnSync("gh", [
+                "issue",
+                "view",
+                issueId,
+                "--json",
+                "comments",
+                "--jq",
+                "[.comments[].body]",
+            ], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 15000 });
+            if (result.status !== 0 || !result.stdout)
+                return [];
+            return JSON.parse(result.stdout);
+        }
+        catch {
+            return [];
+        }
+    }
+    /**
+     * Check if `gh` CLI is authenticated.
+     * Used by system.ts and doctor.ts.
+     */
+    checkAuthSync() {
+        try {
+            execSync("gh auth status", { stdio: "ignore" });
+            return true;
+        }
+        catch {
+            return false;
+        }
+    }
+    /**
+     * Get PR merge status by PR number.
+     * Used by pr-status.ts and state-cleanup.ts.
+     */
+    getPRMergeStatusSync(prNumber) {
+        try {
+            const result = spawnSync("gh", ["pr", "view", String(prNumber), "--json", "state", "-q", ".state"], { stdio: "pipe", timeout: 10000 });
+            if (result.status === 0 && result.stdout) {
+                const state = result.stdout.toString().trim().toUpperCase();
+                if (state === "MERGED")
+                    return "MERGED";
+                if (state === "CLOSED")
+                    return "CLOSED";
+                if (state === "OPEN")
+                    return "OPEN";
+            }
+        }
+        catch {
+            // gh not available or error
+        }
+        return null;
+    }
+    /**
+     * List recently closed issues.
+     * Used by doctor.ts for closed-issue verification.
+     */
+    listClosedIssuesSync(limit = 100) {
+        try {
+            const result = spawnSync("gh", [
+                "issue",
+                "list",
+                "--state",
+                "closed",
+                "--json",
+                "number,title,closedAt,labels",
+                "--limit",
+                String(limit),
+            ], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 15000 });
+            if (result.status !== 0 || !result.stdout)
+                return [];
+            return JSON.parse(result.stdout);
+        }
+        catch {
+            return [];
+        }
+    }
+    /**
+     * View a PR by branch name, returning number and URL.
+     * Used by worktree-manager.ts to check for existing PRs.
+     */
+    viewPRByBranchSync(branch, cwd) {
+        const result = spawnSync("gh", ["pr", "view", branch, "--json", "number,url"], { stdio: "pipe", cwd, timeout: 15000 });
+        if (result.status === 0 && result.stdout) {
+            try {
+                const info = JSON.parse(result.stdout.toString());
+                if (info.number && info.url) {
+                    return { number: info.number, url: info.url };
+                }
+            }
+            catch {
+                // JSON parse failed
+            }
+        }
+        return null;
+    }
+    /**
+     * Get the head branch name for a PR by number.
+     * Used by hooks/pre-tool.sh for pre-merge worktree cleanup.
+     */
+    getPRHeadBranchSync(prNumber) {
+        const result = spawnSync("gh", [
+            "pr",
+            "view",
+            String(prNumber),
+            "--json",
+            "headRefName",
+            "--jq",
+            ".headRefName",
+        ], { stdio: "pipe", timeout: 10000 });
+        if (result.status === 0 && result.stdout) {
+            const branch = result.stdout.toString().trim();
+            return branch || null;
+        }
+        return null;
+    }
+    /**
+     * Create a PR via `gh pr create` CLI, returning raw result.
+     * Used by worktree-manager.ts which needs access to stdout for URL extraction.
+     */
+    createPRCliSync(title, body, head, cwd) {
+        const result = spawnSync("gh", ["pr", "create", "--title", title, "--body", body, "--head", head], { stdio: "pipe", cwd, timeout: 30000 });
+        return {
+            stdout: result.stdout?.toString() ?? "",
+            stderr: result.stderr?.toString() ?? "",
+            exitCode: result.status,
+        };
+    }
+    /**
+     * Batch fetch issue and PR status in a single GraphQL call.
+     * Returns a map keyed by issue/PR number.
+     */
+    batchFetchIssueAndPRStatus(issueNumbers, prNumbers) {
+        if (issueNumbers.length === 0 && prNumbers.length === 0) {
+            return { issues: {}, pullRequests: {}, error: undefined };
+        }
+        try {
+            // Build GraphQL query with aliases for each issue and PR
+            const issueFields = issueNumbers
+                .map((n) => `issue_${n}: issue(number: ${n}) { number title state }`)
+                .join("\n    ");
+            const prFields = prNumbers
+                .map((n) => `pr_${n}: pullRequest(number: ${n}) { number state }`)
+                .join("\n    ");
+            const query = `query {
+  repository(owner: "{owner}", name: "{repo}") {
+    ${issueFields}
+    ${prFields}
+  }
+}`;
+            // Get repo owner/name
+            const repoResult = spawnSync("gh", ["repo", "view", "--json", "owner,name"], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 10000 });
+            if (repoResult.status !== 0 || !repoResult.stdout) {
+                return {
+                    issues: {},
+                    pullRequests: {},
+                    error: "Failed to determine repository",
+                };
+            }
+            const repo = JSON.parse(repoResult.stdout);
+            const filledQuery = query
+                .replace("{owner}", repo.owner.login)
+                .replace("{repo}", repo.name);
+            const result = spawnSync("gh", ["api", "graphql", "-f", `query=${filledQuery}`], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 15000 });
+            if (result.status !== 0 || !result.stdout) {
+                const stderr = result.stderr?.trim() ?? "Unknown error";
+                return { issues: {}, pullRequests: {}, error: stderr };
+            }
+            const data = JSON.parse(result.stdout);
+            const issues = {};
+            const pullRequests = {};
+            const repoData = data.data?.repository ?? {};
+            for (const [key, value] of Object.entries(repoData)) {
+                if (!value)
+                    continue;
+                if (key.startsWith("issue_")) {
+                    issues[value.number] = {
+                        number: value.number,
+                        title: value.title ?? "",
+                        state: value.state,
+                    };
+                }
+                else if (key.startsWith("pr_")) {
+                    pullRequests[value.number] = {
+                        number: value.number,
+                        state: value.state,
+                    };
+                }
+            }
+            return { issues, pullRequests, error: undefined };
+        }
+        catch (err) {
+            return {
+                issues: {},
+                pullRequests: {},
+                error: err instanceof Error ? err.message : String(err),
+            };
+        }
+    }
+    // ─── Repo-aware sync helpers (for upstream / utility callers) ──────
+    /**
+     * Fetch just the title of an issue by number.
+     * Used by merge-check and worktree-discovery.
+     */
+    fetchIssueTitleSync(issueId) {
+        try {
+            const result = spawnSync("gh", ["issue", "view", issueId, "--json", "title", "--jq", ".title"], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 10000 });
+            if (result.status !== 0 || !result.stdout?.trim())
+                return null;
+            return result.stdout.trim();
+        }
+        catch {
+            return null;
+        }
+    }
+    /**
+     * Check if the `gh` CLI binary is installed (not auth, just available).
+     * Used by upstream/assessment.ts for pre-flight checks.
+     */
+    checkGhInstalledSync() {
+        try {
+            const result = spawnSync("gh", ["--version"], {
+                stdio: ["pipe", "pipe", "pipe"],
+                timeout: 5000,
+            });
+            return result.status === 0;
+        }
+        catch {
+            return false;
+        }
+    }
+    /**
+     * Fetch a single release from a specific repo.
+     * If `version` is omitted, fetches the latest release.
+     * Used by upstream/assessment.ts.
+     */
+    fetchReleaseSync(repo, version) {
+        try {
+            const args = ["release", "view"];
+            if (version)
+                args.push(version);
+            args.push("--repo", repo, "--json", "tagName,name,body,publishedAt");
+            const result = spawnSync("gh", args, {
+                encoding: "utf-8",
+                stdio: ["pipe", "pipe", "pipe"],
+                timeout: 15000,
+            });
+            if (result.status !== 0 || !result.stdout)
+                return null;
+            return JSON.parse(result.stdout);
+        }
+        catch {
+            return null;
+        }
+    }
+    /**
+     * List releases from a specific repo.
+     * Used by upstream/assessment.ts.
+     */
+    listReleasesSync(repo, limit = 50) {
+        try {
+            const result = spawnSync("gh", [
+                "release",
+                "list",
+                "--repo",
+                repo,
+                "--limit",
+                String(limit),
+                "--json",
+                "tagName,publishedAt",
+            ], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 15000 });
+            if (result.status !== 0 || !result.stdout)
+                return [];
+            return JSON.parse(result.stdout);
+        }
+        catch {
+            return [];
+        }
+    }
+    /**
+     * Search issues in a specific repo by labels and search query.
+     * Used by upstream/issues.ts for duplicate detection.
+     */
+    searchIssuesSync(repo, labels, search, limit = 10) {
+        try {
+            const args = ["issue", "list", "--repo", repo];
+            for (const label of labels) {
+                args.push("--label", label);
+            }
+            args.push("--search", search, "--json", "number,title", "--limit", String(limit));
+            const result = spawnSync("gh", args, {
+                encoding: "utf-8",
+                stdio: ["pipe", "pipe", "pipe"],
+                timeout: 15000,
+            });
+            if (result.status !== 0 || !result.stdout)
+                return [];
+            return JSON.parse(result.stdout);
+        }
+        catch {
+            return [];
+        }
+    }
+    /**
+     * Create an issue in a specific repo using a body file.
+     * Used by upstream/issues.ts.
+     */
+    createIssueWithBodyFileSync(repo, title, bodyFile, labels) {
+        try {
+            const args = [
+                "issue",
+                "create",
+                "--repo",
+                repo,
+                "--title",
+                title,
+                "--body-file",
+                bodyFile,
+            ];
+            for (const label of labels) {
+                args.push("--label", label);
+            }
+            const result = spawnSync("gh", args, {
+                encoding: "utf-8",
+                stdio: ["pipe", "pipe", "pipe"],
+                timeout: 30000,
+            });
+            if (result.status !== 0 || !result.stdout)
+                return null;
+            const url = result.stdout.trim();
+            const numberMatch = url.match(/\/issues\/(\d+)$/);
+            const number = numberMatch ? parseInt(numberMatch[1], 10) : 0;
+            return { number, url };
+        }
+        catch {
+            return null;
+        }
+    }
+    /**
+     * Add a comment to an issue in a specific repo using a body file.
+     * Used by upstream/issues.ts.
+     */
+    commentOnIssueWithBodyFileSync(repo, issueNumber, bodyFile) {
+        try {
+            const result = spawnSync("gh", [
+                "issue",
+                "comment",
+                String(issueNumber),
+                "--repo",
+                repo,
+                "--body-file",
+                bodyFile,
+            ], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 15000 });
+            return result.status === 0;
+        }
+        catch {
+            return false;
+        }
+    }
+    // ─── Async interface methods (PlatformProvider) ────────────────────
+    async fetchIssue(id) {
+        const result = spawnSync("gh", ["issue", "view", id, "--json", "number,title,body,labels,state"], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 15000 });
+        if (result.status !== 0 || !result.stdout) {
+            throw new Error(`Failed to fetch issue ${id}`);
+        }
+        const data = JSON.parse(result.stdout);
+        return {
+            id: String(data.number),
+            number: data.number,
+            title: data.title,
+            body: data.body,
+            labels: (data.labels ?? []).map((l) => l.name),
+            state: data.state.toLowerCase(),
+        };
+    }
+    async postComment(issueId, body) {
+        spawnSync("gh", ["issue", "comment", issueId, "--body", body], {
+            stdio: "pipe",
+            timeout: 15000,
+        });
+    }
+    async addLabel(issueId, label) {
+        spawnSync("gh", ["issue", "edit", issueId, "--add-label", label], {
+            stdio: "pipe",
+            timeout: 15000,
+        });
+    }
+    async removeLabel(issueId, label) {
+        spawnSync("gh", ["issue", "edit", issueId, "--remove-label", label], {
+            stdio: "pipe",
+            timeout: 15000,
+        });
+    }
+    async createPR(opts) {
+        const result = this.createPRCliSync(opts.title, opts.body, opts.head);
+        if (result.exitCode !== 0) {
+            const error = result.stderr.trim() || "Unknown error";
+            throw new Error(`gh pr create failed: ${error}`);
+        }
+        const urlMatch = result.stdout
+            .trim()
+            .match(/https:\/\/github\.com\/[^\s]+\/pull\/(\d+)/);
+        if (urlMatch) {
+            return {
+                number: parseInt(urlMatch[1], 10),
+                url: urlMatch[0],
+            };
+        }
+        throw new Error(`PR created but could not extract URL from output: ${result.stdout.trim()}`);
+    }
+    async getPRStatus(prId) {
+        const status = this.getPRMergeStatusSync(parseInt(prId, 10));
+        if (status) {
+            return { state: status.toLowerCase() };
+        }
+        throw new Error(`Could not determine PR status for ${prId}`);
+    }
+    async postPRComment(prId, body) {
+        spawnSync("gh", ["pr", "comment", prId, "--body", body], {
+            stdio: "pipe",
+            timeout: 15000,
+        });
+    }
+    async checkAuth() {
+        return this.checkAuthSync();
+    }
+    async getIssueComments(issueId) {
+        try {
+            const result = spawnSync("gh", [
+                "issue",
+                "view",
+                issueId,
+                "--json",
+                "comments",
+                "--jq",
+                "[.comments[] | {body: .body, createdAt: .createdAt}]",
+            ], { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"], timeout: 15000 });
+            if (result.status !== 0 || !result.stdout)
+                return [];
+            const data = JSON.parse(result.stdout);
+            return data;
+        }
+        catch {
+            return [];
+        }
+    }
+}

package/dist/src/lib/workflow/platforms/index.d.ts

@@ -0,0 +1,17 @@
+/**
+ * Platform provider registry.
+ *
+ * Simple map-based registry — not a plugin system.
+ * New providers are registered by adding entries to PLATFORMS.
+ */
+import type { PlatformProvider } from "./platform-provider.js";
+export type { PlatformProvider, Issue, CreatePROptions, PRInfo, PRStatus, Comment, } from "./platform-provider.js";
+export { GitHubProvider } from "./github.js";
+export type { PRMergeStatus, ClosedIssueRaw, CreatePRCliResult, } from "./github.js";
+/**
+ * Get a platform provider by name.
+ *
+ * @param name - Provider name (default: "github")
+ * @throws Error if provider name is unknown
+ */
+export declare function getPlatform(name?: string): PlatformProvider;