@prowi/deskcheck 0.1.0

package/build/core/context-extractor.js

@@ -0,0 +1,69 @@
+import fs from "node:fs";
+import path from "node:path";
+import { execFileSync } from "node:child_process";
+/**
+ * Extract review context based on the source type.
+ *
+ * - diff: runs `git diff` for the given files against the source target
+ * - file: reads each file's content from disk
+ * - symbol: reads the file containing the symbol (symbol identification
+ *   is left to the executor agent which has Read/Grep tools)
+ */
+export function extractContext(sourceType, sourceTarget, files, projectRoot, symbol) {
+    let content;
+    switch (sourceType) {
+        case "diff":
+            content = extractDiffContext(sourceTarget, files, projectRoot);
+            break;
+        case "file":
+            content = extractFileContext(files, projectRoot);
+            break;
+        case "symbol":
+            content = extractFileContext(files, projectRoot);
+            break;
+        default:
+            throw new Error(`Unknown source type: ${sourceType}`);
+    }
+    return {
+        contextType: sourceType,
+        content,
+        symbol: symbol ?? null,
+    };
+}
+/** Run git diff for each file and concatenate results. */
+function extractDiffContext(target, files, projectRoot) {
+    const parts = [];
+    for (const file of files) {
+        try {
+            const diff = execFileSync("git", ["diff", target, "--", file], {
+                cwd: projectRoot,
+                encoding: "utf-8",
+                maxBuffer: 10 * 1024 * 1024,
+                timeout: 30_000,
+            });
+            if (diff.trim().length > 0) {
+                parts.push(diff);
+            }
+        }
+        catch (error) {
+            console.error(`Warning: git diff failed for ${file}: ${error instanceof Error ? error.message : String(error)}`);
+        }
+    }
+    return parts.join("\n");
+}
+/** Read each file's content and concatenate with file path headers. */
+function extractFileContext(files, projectRoot) {
+    const parts = [];
+    for (const file of files) {
+        try {
+            const absolutePath = path.resolve(projectRoot, file);
+            const content = fs.readFileSync(absolutePath, "utf-8");
+            parts.push(`--- ${file} ---\n${content}`);
+        }
+        catch (error) {
+            console.error(`Warning: could not read ${file}: ${error instanceof Error ? error.message : String(error)}`);
+        }
+    }
+    return parts.join("\n\n");
+}
+//# sourceMappingURL=context-extractor.js.map

package/build/core/context-extractor.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"context-extractor.js","sourceRoot":"","sources":["../../src/core/context-extractor.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAUlD;;;;;;;GAOG;AACH,MAAM,UAAU,cAAc,CAC5B,UAAuB,EACvB,YAAoB,EACpB,KAAe,EACf,WAAmB,EACnB,MAAe;IAEf,IAAI,OAAe,CAAC;IAEpB,QAAQ,UAAU,EAAE,CAAC;QACnB,KAAK,MAAM;YACT,OAAO,GAAG,kBAAkB,CAAC,YAAY,EAAE,KAAK,EAAE,WAAW,CAAC,CAAC;YAC/D,MAAM;QACR,KAAK,MAAM;YACT,OAAO,GAAG,kBAAkB,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC;YACjD,MAAM;QACR,KAAK,QAAQ;YACX,OAAO,GAAG,kBAAkB,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC;YACjD,MAAM;QACR;YACE,MAAM,IAAI,KAAK,CAAC,wBAAwB,UAAU,EAAE,CAAC,CAAC;IAC1D,CAAC;IAED,OAAO;QACL,WAAW,EAAE,UAAU;QACvB,OAAO;QACP,MAAM,EAAE,MAAM,IAAI,IAAI;KACvB,CAAC;AACJ,CAAC;AAED,0DAA0D;AAC1D,SAAS,kBAAkB,CACzB,MAAc,EACd,KAAe,EACf,WAAmB;IAEnB,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC;YACH,MAAM,IAAI,GAAG,YAAY,CAAC,KAAK,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,CAAC,EAAE;gBAC7D,GAAG,EAAE,WAAW;gBAChB,QAAQ,EAAE,OAAO;gBACjB,SAAS,EAAE,EAAE,GAAG,IAAI,GAAG,IAAI;gBAC3B,OAAO,EAAE,MAAM;aAChB,CAAC,CAAC;YAEH,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;gBAC3B,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACnB,CAAC;QACH,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,OAAO,CAAC,KAAK,CACX,gCAAgC,IAAI,KAAK,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAClG,CAAC;QACJ,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC;AAED,uEAAuE;AACvE,SAAS,kBAAkB,CACzB,KAAe,EACf,WAAmB;IAEnB,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC;YACH,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,IAAI,CAAC,CAAC;YACrD,MAAM,OAAO,GAAG,EAAE,CAAC,YAAY,CAAC,YAAY,EAAE,OAAO,CAAC,CAAC;YACvD,KAAK,CAAC,IAAI,CAAC,OAAO,IAAI,SAAS,OAAO,EAAE,CAAC,CAAC;QAC5C,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,OAAO,CAAC,KAAK,CACX,2BAA2B,IAAI,KAAK,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAC7F,CAAC;QACJ,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;AAC5B,CAAC"}

package/build/core/glob-matcher.d.ts

@@ -0,0 +1,32 @@
+import type { ReviewModule } from "./types.js";
+/** The result of matching changed files against a single criterion. */
+export interface MatchResult {
+    /** The criterion that matched. */
+    module: ReviewModule;
+    /** Files from the changed set that matched the criterion's globs. */
+    matchedFiles: string[];
+}
+/**
+ * Check whether a single file path matches a set of glob patterns.
+ *
+ * Globs prefixed with `!` act as exclusion patterns: if a file matches any
+ * exclusion glob, it is NOT considered a match even if it matches a positive glob.
+ * Positive globs are checked first; then exclusions are applied to remove matches.
+ *
+ * @param filePath - The file path to test (e.g. "app/Http/Controllers/OrderController.php").
+ * @param globs - Array of glob patterns. Prefix with `!` to exclude.
+ * @returns True if the file matches at least one positive glob and no exclusion globs.
+ */
+export declare function fileMatchesGlobs(filePath: string, globs: string[]): boolean;
+/**
+ * Find which criteria match a set of changed files.
+ *
+ * For each criterion, checks every changed file against the criterion's glob patterns.
+ * Criteria with at least one matching file are included in the results.
+ *
+ * @param changedFiles - Array of file paths that were changed.
+ * @param modules - Array of criteria to match against.
+ * @returns Array of match results, one per criterion that matched at least one file.
+ */
+export declare function findMatchingModules(changedFiles: string[], modules: ReviewModule[]): MatchResult[];
+//# sourceMappingURL=glob-matcher.d.ts.map

package/build/core/glob-matcher.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"glob-matcher.d.ts","sourceRoot":"","sources":["../../src/core/glob-matcher.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,YAAY,CAAC;AAE/C,uEAAuE;AACvE,MAAM,WAAW,WAAW;IAC1B,kCAAkC;IAClC,MAAM,EAAE,YAAY,CAAC;IACrB,qEAAqE;IACrE,YAAY,EAAE,MAAM,EAAE,CAAC;CACxB;AAED;;;;;;;;;;GAUG;AACH,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,GAAG,OAAO,CAyB3E;AAED;;;;;;;;;GASG;AACH,wBAAgB,mBAAmB,CACjC,YAAY,EAAE,MAAM,EAAE,EACtB,OAAO,EAAE,YAAY,EAAE,GACtB,WAAW,EAAE,CAcf"}

package/build/core/glob-matcher.js

@@ -0,0 +1,51 @@
+import { minimatch } from "minimatch";
+/**
+ * Check whether a single file path matches a set of glob patterns.
+ *
+ * Globs prefixed with `!` act as exclusion patterns: if a file matches any
+ * exclusion glob, it is NOT considered a match even if it matches a positive glob.
+ * Positive globs are checked first; then exclusions are applied to remove matches.
+ *
+ * @param filePath - The file path to test (e.g. "app/Http/Controllers/OrderController.php").
+ * @param globs - Array of glob patterns. Prefix with `!` to exclude.
+ * @returns True if the file matches at least one positive glob and no exclusion globs.
+ */
+export function fileMatchesGlobs(filePath, globs) {
+    const positiveGlobs = [];
+    const exclusionGlobs = [];
+    for (const glob of globs) {
+        if (glob.startsWith("!")) {
+            exclusionGlobs.push(glob.slice(1));
+        }
+        else {
+            positiveGlobs.push(glob);
+        }
+    }
+    const matchesPositive = positiveGlobs.some((pattern) => minimatch(filePath, pattern));
+    if (!matchesPositive) {
+        return false;
+    }
+    const matchesExclusion = exclusionGlobs.some((pattern) => minimatch(filePath, pattern));
+    return !matchesExclusion;
+}
+/**
+ * Find which criteria match a set of changed files.
+ *
+ * For each criterion, checks every changed file against the criterion's glob patterns.
+ * Criteria with at least one matching file are included in the results.
+ *
+ * @param changedFiles - Array of file paths that were changed.
+ * @param modules - Array of criteria to match against.
+ * @returns Array of match results, one per criterion that matched at least one file.
+ */
+export function findMatchingModules(changedFiles, modules) {
+    const results = [];
+    for (const module of modules) {
+        const matchedFiles = changedFiles.filter((file) => fileMatchesGlobs(file, module.globs));
+        if (matchedFiles.length > 0) {
+            results.push({ module, matchedFiles });
+        }
+    }
+    return results;
+}
+//# sourceMappingURL=glob-matcher.js.map

package/build/core/glob-matcher.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"glob-matcher.js","sourceRoot":"","sources":["../../src/core/glob-matcher.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AAWtC;;;;;;;;;;GAUG;AACH,MAAM,UAAU,gBAAgB,CAAC,QAAgB,EAAE,KAAe;IAChE,MAAM,aAAa,GAAa,EAAE,CAAC;IACnC,MAAM,cAAc,GAAa,EAAE,CAAC;IAEpC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YACzB,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACrC,CAAC;aAAM,CAAC;YACN,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC3B,CAAC;IACH,CAAC;IAED,MAAM,eAAe,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CACrD,SAAS,CAAC,QAAQ,EAAE,OAAO,CAAC,CAC7B,CAAC;IAEF,IAAI,CAAC,eAAe,EAAE,CAAC;QACrB,OAAO,KAAK,CAAC;IACf,CAAC;IAED,MAAM,gBAAgB,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,EAAE,CACvD,SAAS,CAAC,QAAQ,EAAE,OAAO,CAAC,CAC7B,CAAC;IAEF,OAAO,CAAC,gBAAgB,CAAC;AAC3B,CAAC;AAED;;;;;;;;;GASG;AACH,MAAM,UAAU,mBAAmB,CACjC,YAAsB,EACtB,OAAuB;IAEvB,MAAM,OAAO,GAAkB,EAAE,CAAC;IAElC,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,MAAM,YAAY,GAAG,YAAY,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAChD,gBAAgB,CAAC,IAAI,EAAE,MAAM,CAAC,KAAK,CAAC,CACrC,CAAC;QAEF,IAAI,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC5B,OAAO,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,YAAY,EAAE,CAAC,CAAC;QACzC,CAAC;IACH,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC"}

package/build/core/module-parser.d.ts

@@ -0,0 +1,26 @@
+import type { ReviewModule } from "./types.js";
+/**
+ * Parse a single criterion markdown file into a ReviewModule object.
+ *
+ * Reads the file, extracts YAML frontmatter with `gray-matter`, validates
+ * required fields, applies defaults for optional fields, and returns the
+ * structured criterion representation.
+ *
+ * @param filePath - Absolute or relative path to the markdown file.
+ * @param basePath - Base directory used to compute relative paths and criterion IDs.
+ * @returns Parsed ReviewModule.
+ * @throws {Error} If the file cannot be read or frontmatter is invalid.
+ */
+export declare function parseModule(filePath: string, basePath: string): ReviewModule;
+/**
+ * Discover all criteria by recursively scanning a directory for `.md` files.
+ *
+ * Each markdown file is parsed for YAML frontmatter containing criterion metadata
+ * (description, severity, globs, etc.) and the markdown body becomes the detective prompt.
+ *
+ * @param modulesDir - Path to the directory containing criterion markdown files.
+ * @returns Array of parsed ReviewModule objects, sorted by ID for deterministic ordering.
+ * @throws {Error} If the directory does not exist or any criterion file has invalid frontmatter.
+ */
+export declare function discoverModules(modulesDir: string): ReviewModule[];
+//# sourceMappingURL=module-parser.d.ts.map

package/build/core/module-parser.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"module-parser.d.ts","sourceRoot":"","sources":["../../src/core/module-parser.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAA8B,YAAY,EAAE,MAAM,YAAY,CAAC;AAkB3E;;;;;;;;;;;GAWG;AACH,wBAAgB,WAAW,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,YAAY,CAkE5E;AAED;;;;;;;;;GASG;AACH,wBAAgB,eAAe,CAAC,UAAU,EAAE,MAAM,GAAG,YAAY,EAAE,CAqBlE"}

package/build/core/module-parser.js

@@ -0,0 +1,98 @@
+import fs from "node:fs";
+import path from "node:path";
+import matter from "gray-matter";
+const VALID_SEVERITIES = new Set([
+    "critical",
+    "high",
+    "medium",
+    "low",
+]);
+const VALID_MODELS = new Set([
+    "haiku",
+    "sonnet",
+    "opus",
+]);
+const DEFAULT_MODE = "Create one review per changed file";
+const DEFAULT_MODEL = "haiku";
+/**
+ * Parse a single criterion markdown file into a ReviewModule object.
+ *
+ * Reads the file, extracts YAML frontmatter with `gray-matter`, validates
+ * required fields, applies defaults for optional fields, and returns the
+ * structured criterion representation.
+ *
+ * @param filePath - Absolute or relative path to the markdown file.
+ * @param basePath - Base directory used to compute relative paths and criterion IDs.
+ * @returns Parsed ReviewModule.
+ * @throws {Error} If the file cannot be read or frontmatter is invalid.
+ */
+export function parseModule(filePath, basePath) {
+    const absolutePath = path.resolve(filePath);
+    const absoluteBase = path.resolve(basePath);
+    const raw = fs.readFileSync(absolutePath, "utf-8");
+    const { data: frontmatter, content } = matter(raw);
+    const relativePath = path.relative(absoluteBase, absolutePath);
+    const relativeFile = path.relative(path.dirname(absoluteBase), absolutePath);
+    // --- Validate required fields ---
+    if (typeof frontmatter.description !== "string" || frontmatter.description.trim() === "") {
+        throw new Error(`Invalid criterion ${relativePath}: "description" is required and must be a non-empty string`);
+    }
+    if (!VALID_SEVERITIES.has(frontmatter.severity)) {
+        throw new Error(`Invalid criterion ${relativePath}: "severity" must be one of: ${[...VALID_SEVERITIES].join(", ")}. Got: ${JSON.stringify(frontmatter.severity)}`);
+    }
+    if (!Array.isArray(frontmatter.globs) || frontmatter.globs.length === 0) {
+        throw new Error(`Invalid criterion ${relativePath}: "globs" is required and must be a non-empty array of strings`);
+    }
+    for (const glob of frontmatter.globs) {
+        if (typeof glob !== "string" || glob.trim() === "") {
+            throw new Error(`Invalid criterion ${relativePath}: each entry in "globs" must be a non-empty string. Got: ${JSON.stringify(glob)}`);
+        }
+    }
+    // --- Apply defaults for optional fields ---
+    const mode = typeof frontmatter.mode === "string" && frontmatter.mode.trim() !== ""
+        ? frontmatter.mode
+        : DEFAULT_MODE;
+    const model = frontmatter.model ?? DEFAULT_MODEL;
+    if (!VALID_MODELS.has(model)) {
+        throw new Error(`Invalid criterion ${relativePath}: "model" must be one of: ${[...VALID_MODELS].join(", ")}. Got: ${JSON.stringify(model)}`);
+    }
+    // --- Build the criterion ID from relative path without extension ---
+    const id = relativePath.replace(/\.md$/, "").split(path.sep).join("/");
+    return {
+        id,
+        file: relativeFile.split(path.sep).join("/"),
+        description: frontmatter.description,
+        severity: frontmatter.severity,
+        globs: frontmatter.globs,
+        mode,
+        model: model,
+        prompt: content.trim(),
+    };
+}
+/**
+ * Discover all criteria by recursively scanning a directory for `.md` files.
+ *
+ * Each markdown file is parsed for YAML frontmatter containing criterion metadata
+ * (description, severity, globs, etc.) and the markdown body becomes the detective prompt.
+ *
+ * @param modulesDir - Path to the directory containing criterion markdown files.
+ * @returns Array of parsed ReviewModule objects, sorted by ID for deterministic ordering.
+ * @throws {Error} If the directory does not exist or any criterion file has invalid frontmatter.
+ */
+export function discoverModules(modulesDir) {
+    const absoluteDir = path.resolve(modulesDir);
+    if (!fs.existsSync(absoluteDir)) {
+        throw new Error(`Criteria directory does not exist: ${absoluteDir}`);
+    }
+    const entries = fs.readdirSync(absoluteDir, { recursive: true });
+    const mdFiles = entries
+        .filter((entry) => entry.endsWith(".md"))
+        .sort();
+    const modules = [];
+    for (const relativeFile of mdFiles) {
+        const absoluteFile = path.join(absoluteDir, relativeFile);
+        modules.push(parseModule(absoluteFile, absoluteDir));
+    }
+    return modules.sort((a, b) => a.id.localeCompare(b.id));
+}
+//# sourceMappingURL=module-parser.js.map

package/build/core/module-parser.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"module-parser.js","sourceRoot":"","sources":["../../src/core/module-parser.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,SAAS,CAAC;AACzB,OAAO,IAAI,MAAM,WAAW,CAAC;AAC7B,OAAO,MAAM,MAAM,aAAa,CAAC;AAGjC,MAAM,gBAAgB,GAAwB,IAAI,GAAG,CAAiB;IACpE,UAAU;IACV,MAAM;IACN,QAAQ;IACR,KAAK;CACN,CAAC,CAAC;AAEH,MAAM,YAAY,GAAwB,IAAI,GAAG,CAAa;IAC5D,OAAO;IACP,QAAQ;IACR,MAAM;CACP,CAAC,CAAC;AAEH,MAAM,YAAY,GAAG,oCAAoC,CAAC;AAC1D,MAAM,aAAa,GAAe,OAAO,CAAC;AAE1C;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,WAAW,CAAC,QAAgB,EAAE,QAAgB;IAC5D,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;IAC5C,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;IAE5C,MAAM,GAAG,GAAG,EAAE,CAAC,YAAY,CAAC,YAAY,EAAE,OAAO,CAAC,CAAC;IACnD,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,OAAO,EAAE,GAAG,MAAM,CAAC,GAAG,CAAC,CAAC;IAEnD,MAAM,YAAY,GAAG,IAAI,CAAC,QAAQ,CAAC,YAAY,EAAE,YAAY,CAAC,CAAC;IAC/D,MAAM,YAAY,GAAG,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,YAAY,CAAC,EAAE,YAAY,CAAC,CAAC;IAE7E,mCAAmC;IAEnC,IAAI,OAAO,WAAW,CAAC,WAAW,KAAK,QAAQ,IAAI,WAAW,CAAC,WAAW,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;QACzF,MAAM,IAAI,KAAK,CACb,qBAAqB,YAAY,4DAA4D,CAC9F,CAAC;IACJ,CAAC;IAED,IAAI,CAAC,gBAAgB,CAAC,GAAG,CAAC,WAAW,CAAC,QAAQ,CAAC,EAAE,CAAC;QAChD,MAAM,IAAI,KAAK,CACb,qBAAqB,YAAY,gCAAgC,CAAC,GAAG,gBAAgB,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,IAAI,CAAC,SAAS,CAAC,WAAW,CAAC,QAAQ,CAAC,EAAE,CAClJ,CAAC;IACJ,CAAC;IAED,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,WAAW,CAAC,KAAK,CAAC,IAAI,WAAW,CAAC,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACxE,MAAM,IAAI,KAAK,CACb,qBAAqB,YAAY,gEAAgE,CAClG,CAAC;IACJ,CAAC;IAED,KAAK,MAAM,IAAI,IAAI,WAAW,CAAC,KAAK,EAAE,CAAC;QACrC,IAAI,OAAO,IAAI,KAAK,QAAQ,IAAI,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;YACnD,MAAM,IAAI,KAAK,CACb,qBAAqB,YAAY,4DAA4D,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,EAAE,CACpH,CAAC;QACJ,CAAC;IACH,CAAC;IAED,6CAA6C;IAE7C,MAAM,IAAI,GACR,OAAO,WAAW,CAAC,IAAI,KAAK,QAAQ,IAAI,WAAW,CAAC,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE;QACpE,CAAC,CAAC,WAAW,CAAC,IAAI;QAClB,CAAC,CAAC,YAAY,CAAC;IAEnB,MAAM,KAAK,GAAG,WAAW,CAAC,KAAK,IAAI,aAAa,CAAC;IACjD,IAAI,CAAC,YAAY,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;QAC7B,MAAM,IAAI,KAAK,CACb,qBAAqB,YAAY,6BAA6B,CAAC,GAAG,YAAY,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,EAAE,CAC5H,CAAC;IACJ,CAAC;IAED,sEAAsE;IAEtE,MAAM,EAAE,GAAG,YAAY,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IAEvE,OAAO;QACL,EAAE;QACF,IAAI,EAAE,YAAY,CAAC,KAAK,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC;QAC5C,WAAW,EAAE,WAAW,CAAC,WAAW;QACpC,QAAQ,EAAE,WAAW,CAAC,QAA0B;QAChD,KAAK,EAAE,WAAW,CAAC,KAAiB;QACpC,IAAI;QACJ,KAAK,EAAE,KAAmB;QAC1B,MAAM,EAAE,OAAO,CAAC,IAAI,EAAE;KACvB,CAAC;AACJ,CAAC;AAED;;;;;;;;;GASG;AACH,MAAM,UAAU,eAAe,CAAC,UAAkB;IAChD,MAAM,WAAW,GAAG,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;IAE7C,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,WAAW,CAAC,EAAE,CAAC;QAChC,MAAM,IAAI,KAAK,CAAC,sCAAsC,WAAW,EAAE,CAAC,CAAC;IACvE,CAAC;IAED,MAAM,OAAO,GAAG,EAAE,CAAC,WAAW,CAAC,WAAW,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAa,CAAC;IAE7E,MAAM,OAAO,GAAG,OAAO;SACpB,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;SACxC,IAAI,EAAE,CAAC;IAEV,MAAM,OAAO,GAAmB,EAAE,CAAC;IAEnC,KAAK,MAAM,YAAY,IAAI,OAAO,EAAE,CAAC;QACnC,MAAM,YAAY,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC;QAC1D,OAAO,CAAC,IAAI,CAAC,WAAW,CAAC,YAAY,EAAE,WAAW,CAAC,CAAC,CAAC;IACvD,CAAC;IAED,OAAO,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;AAC1D,CAAC"}

package/build/core/plan-builder.d.ts

@@ -0,0 +1,12 @@
+import { ReviewStorage } from "./storage.js";
+import type { ReviewModule, ReviewPlan, ReviewSource } from "./types.js";
+/**
+ * Build a complete review plan: create the plan, match files to modules,
+ * generate tasks, and finalize.
+ *
+ * This is the single authoritative implementation of plan+task creation,
+ * shared by both the planner agent's in-process MCP tool and any other
+ * entry point that needs to create a plan from a file list.
+ */
+export declare function buildPlanWithTasks(storage: ReviewStorage, name: string, source: ReviewSource, files: string[], modules: ReviewModule[]): ReviewPlan;
+//# sourceMappingURL=plan-builder.d.ts.map

package/build/core/plan-builder.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"plan-builder.d.ts","sourceRoot":"","sources":["../../src/core/plan-builder.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC7C,OAAO,KAAK,EAEV,YAAY,EACZ,UAAU,EACV,YAAY,EACb,MAAM,YAAY,CAAC;AAEpB;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAChC,OAAO,EAAE,aAAa,EACtB,IAAI,EAAE,MAAM,EACZ,MAAM,EAAE,YAAY,EACpB,KAAK,EAAE,MAAM,EAAE,EACf,OAAO,EAAE,YAAY,EAAE,GACtB,UAAU,CA6DZ"}

package/build/core/plan-builder.js

@@ -0,0 +1,66 @@
+import { findMatchingModules } from "./glob-matcher.js";
+/**
+ * Build a complete review plan: create the plan, match files to modules,
+ * generate tasks, and finalize.
+ *
+ * This is the single authoritative implementation of plan+task creation,
+ * shared by both the planner agent's in-process MCP tool and any other
+ * entry point that needs to create a plan from a file list.
+ */
+export function buildPlanWithTasks(storage, name, source, files, modules) {
+    // Create the plan shell
+    const plan = storage.createPlan(name, source);
+    // Match files against criteria
+    const matches = findMatchingModules(files, modules);
+    // Track coverage
+    const matchedFileSet = new Set();
+    for (const match of matches) {
+        for (const file of match.matchedFiles) {
+            matchedFileSet.add(file);
+        }
+    }
+    const matchedFiles = [...matchedFileSet].sort();
+    const unmatchedFiles = files.filter((f) => !matchedFileSet.has(f)).sort();
+    storage.setMatchedFiles(plan.plan_id, matchedFiles, unmatchedFiles);
+    // Set module summaries
+    const moduleSummaries = {};
+    for (const match of matches) {
+        moduleSummaries[match.module.id] = {
+            review_id: match.module.id,
+            description: match.module.description,
+            severity: match.module.severity,
+            model: match.module.model,
+            task_count: 0,
+            matched_files: match.matchedFiles,
+        };
+    }
+    storage.setModules(plan.plan_id, moduleSummaries);
+    // Create tasks based on module mode
+    for (const match of matches) {
+        const isGrouped = match.module.mode.toLowerCase().includes("grouped") ||
+            match.module.mode.toLowerCase().includes("all files");
+        if (isGrouped) {
+            storage.addTask(plan.plan_id, {
+                review_id: match.module.id,
+                review_file: match.module.file,
+                files: match.matchedFiles,
+                hint: null,
+                model: match.module.model,
+            });
+        }
+        else {
+            for (const file of match.matchedFiles) {
+                storage.addTask(plan.plan_id, {
+                    review_id: match.module.id,
+                    review_file: match.module.file,
+                    files: [file],
+                    hint: null,
+                    model: match.module.model,
+                });
+            }
+        }
+    }
+    // Finalize the plan (sets status to "ready", recounts tasks per module)
+    return storage.finalizePlan(plan.plan_id);
+}
+//# sourceMappingURL=plan-builder.js.map

package/build/core/plan-builder.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"plan-builder.js","sourceRoot":"","sources":["../../src/core/plan-builder.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,mBAAmB,EAAE,MAAM,mBAAmB,CAAC;AASxD;;;;;;;GAOG;AACH,MAAM,UAAU,kBAAkB,CAChC,OAAsB,EACtB,IAAY,EACZ,MAAoB,EACpB,KAAe,EACf,OAAuB;IAEvB,wBAAwB;IACxB,MAAM,IAAI,GAAG,OAAO,CAAC,UAAU,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;IAE9C,+BAA+B;IAC/B,MAAM,OAAO,GAAG,mBAAmB,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC;IAEpD,iBAAiB;IACjB,MAAM,cAAc,GAAG,IAAI,GAAG,EAAU,CAAC;IACzC,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC5B,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,YAAY,EAAE,CAAC;YACtC,cAAc,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;QAC3B,CAAC;IACH,CAAC;IACD,MAAM,YAAY,GAAG,CAAC,GAAG,cAAc,CAAC,CAAC,IAAI,EAAE,CAAC;IAChD,MAAM,cAAc,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,cAAc,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;IAC1E,OAAO,CAAC,eAAe,CAAC,IAAI,CAAC,OAAO,EAAE,YAAY,EAAE,cAAc,CAAC,CAAC;IAEpE,uBAAuB;IACvB,MAAM,eAAe,GAAkC,EAAE,CAAC;IAC1D,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC5B,eAAe,CAAC,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,GAAG;YACjC,SAAS,EAAE,KAAK,CAAC,MAAM,CAAC,EAAE;YAC1B,WAAW,EAAE,KAAK,CAAC,MAAM,CAAC,WAAW;YACrC,QAAQ,EAAE,KAAK,CAAC,MAAM,CAAC,QAAQ;YAC/B,KAAK,EAAE,KAAK,CAAC,MAAM,CAAC,KAAK;YACzB,UAAU,EAAE,CAAC;YACb,aAAa,EAAE,KAAK,CAAC,YAAY;SAClC,CAAC;IACJ,CAAC;IACD,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,OAAO,EAAE,eAAe,CAAC,CAAC;IAElD,oCAAoC;IACpC,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC5B,MAAM,SAAS,GACb,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,SAAS,CAAC;YACnD,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC;QAExD,IAAI,SAAS,EAAE,CAAC;YACd,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,OAAO,EAAE;gBAC5B,SAAS,EAAE,KAAK,CAAC,MAAM,CAAC,EAAE;gBAC1B,WAAW,EAAE,KAAK,CAAC,MAAM,CAAC,IAAI;gBAC9B,KAAK,EAAE,KAAK,CAAC,YAAY;gBACzB,IAAI,EAAE,IAAI;gBACV,KAAK,EAAE,KAAK,CAAC,MAAM,CAAC,KAAK;aAC1B,CAAC,CAAC;QACL,CAAC;aAAM,CAAC;YACN,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,YAAY,EAAE,CAAC;gBACtC,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,OAAO,EAAE;oBAC5B,SAAS,EAAE,KAAK,CAAC,MAAM,CAAC,EAAE;oBAC1B,WAAW,EAAE,KAAK,CAAC,MAAM,CAAC,IAAI;oBAC9B,KAAK,EAAE,CAAC,IAAI,CAAC;oBACb,IAAI,EAAE,IAAI;oBACV,KAAK,EAAE,KAAK,CAAC,MAAM,CAAC,KAAK;iBAC1B,CAAC,CAAC;YACL,CAAC;QACH,CAAC;IACH,CAAC;IAED,wEAAwE;IACxE,OAAO,OAAO,CAAC,YAAY,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;AAC5C,CAAC"}

package/build/core/storage.d.ts

@@ -0,0 +1,118 @@
+import type { ContextType, Finding, ModuleSummary, ReviewPlan, ReviewResults, ReviewSource, ReviewTask, TaskUsage } from "./types.js";
+/**
+ * Manages the two-file storage format (plan.json + results.json) for deskcheck runs.
+ *
+ * Each deskcheck run lives in a timestamped directory under the configured storage
+ * directory. The plan file tracks tasks and coverage; the results file tracks
+ * findings and aggregations.
+ */
+export declare class ReviewStorage {
+    private readonly storageDir;
+    /** Tracks active lock file paths so they can be cleaned up on process exit. */
+    private static activeLocks;
+    constructor(storageDir: string);
+    /**
+     * Execute a function while holding a file-based lock for the given plan.
+     *
+     * Uses `{ flag: "wx" }` for atomic exclusive create — either the file is
+     * created (lock acquired) or it already exists (another holder). This
+     * eliminates the TOCTOU race of existsSync + writeFileSync.
+     *
+     * Stale locks are detected via a timestamp in the lock file body.
+     */
+    private withLock;
+    /** Get the directory path for a plan. */
+    private planDir;
+    /**
+     * Create a new review plan with an empty task set.
+     *
+     * Creates the timestamped directory and writes an initial plan.json with
+     * status "planning" and empty collections.
+     */
+    createPlan(name: string, source: ReviewSource): ReviewPlan;
+    /** Read the plan.json for a given plan ID. */
+    getPlan(planId: string): ReviewPlan;
+    /**
+     * Return the most recent plan ID (latest timestamped directory), or null if
+     * no plans exist.
+     */
+    getLatestPlanId(): string | null;
+    /** List all plans with basic metadata. */
+    listPlans(): Array<{
+        planId: string;
+        name: string;
+        status: string;
+        createdAt: string;
+    }>;
+    /**
+     * Finalize a plan by setting its status to "ready" and updating task counts
+     * in module summaries.
+     */
+    finalizePlan(planId: string): ReviewPlan;
+    /**
+     * Add a task to an existing plan.
+     *
+     * Auto-generates the task_id by flattening slashes in the review_id and
+     * appending a zero-padded incrementing suffix (e.g., "-001", "-002").
+     * The task is created with status "pending" and null context fields.
+     */
+    addTask(planId: string, task: Omit<ReviewTask, "task_id" | "status" | "created_at" | "started_at" | "completed_at" | "context" | "context_type" | "symbol" | "prompt">): ReviewTask;
+    /**
+     * Claim a pending task for execution.
+     *
+     * Sets the task status to "in_progress", fills in the context fields
+     * (context type, content, symbol, prompt), and records the start time.
+     */
+    claimTask(planId: string, taskId: string, context: {
+        contextType: ContextType;
+        content: string;
+        symbol?: string;
+        prompt: string;
+    }): ReviewTask;
+    /**
+     * Return tasks eligible for execution: those with status "pending" or
+     * "in_progress" tasks that are stale (older than 5 minutes).
+     */
+    getPendingTasks(planId: string): ReviewTask[];
+    /**
+     * Mark a task as complete and record its findings.
+     *
+     * Updates the task status in plan.json, adds a TaskResult entry to
+     * results.json, and recomputes all aggregations (by_file, by_module,
+     * summary, completion).
+     */
+    completeTask(planId: string, taskId: string, findings: Finding[], usage?: TaskUsage | null): void;
+    /**
+     * Mark a task as errored.
+     *
+     * Sets the task status to "error" without recording any findings.
+     * Updates plan.json and recomputes result aggregations so the error
+     * is reflected in completion counts.
+     */
+    errorTask(planId: string, taskId: string, errorMessage: string, usage?: TaskUsage | null): void;
+    /** Read the results.json for a given plan ID. */
+    getResults(planId: string): ReviewResults;
+    /** Set the matched and unmatched file lists in the plan. */
+    setMatchedFiles(planId: string, matched: string[], unmatched: string[]): void;
+    /** Set the per-module summaries in the plan. */
+    setModules(planId: string, modules: Record<string, ModuleSummary>): void;
+    private planPath;
+    private resultsPath;
+    private writePlan;
+    private writeResults;
+    private createEmptyResults;
+    /**
+     * Load existing results.json or create a new empty structure.
+     */
+    private loadOrCreateResults;
+    /**
+     * Recompute all derived aggregations in results from the task_results
+     * and the current plan state.
+     *
+     * This rebuilds by_file, by_module, summary, and completion from scratch
+     * on every completeTask call. Since there is a single orchestrator process,
+     * this is safe and keeps the logic simple.
+     */
+    private recomputeAggregations;
+}
+//# sourceMappingURL=storage.d.ts.map

package/build/core/storage.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"storage.d.ts","sourceRoot":"","sources":["../../src/core/storage.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EACV,WAAW,EACX,OAAO,EAGP,aAAa,EACb,UAAU,EACV,aAAa,EACb,YAAY,EACZ,UAAU,EAEV,SAAS,EAEV,MAAM,YAAY,CAAC;AAsCpB;;;;;;GAMG;AACH,qBAAa,aAAa;IACxB,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAS;IAEpC,+EAA+E;IAC/E,OAAO,CAAC,MAAM,CAAC,WAAW,CAAqB;gBAanC,UAAU,EAAE,MAAM;IAI9B;;;;;;;;OAQG;IACH,OAAO,CAAC,QAAQ;IAoDhB,yCAAyC;IACzC,OAAO,CAAC,OAAO;IAQf;;;;;OAKG;IACH,UAAU,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,GAAG,UAAU;IA0B1D,8CAA8C;IAC9C,OAAO,CAAC,MAAM,EAAE,MAAM,GAAG,UAAU;IAMnC;;;OAGG;IACH,eAAe,IAAI,MAAM,GAAG,IAAI;IAsBhC,0CAA0C;IAC1C,SAAS,IAAI,KAAK,CAAC;QACjB,MAAM,EAAE,MAAM,CAAC;QACf,IAAI,EAAE,MAAM,CAAC;QACb,MAAM,EAAE,MAAM,CAAC;QACf,SAAS,EAAE,MAAM,CAAC;KACnB,CAAC;IAqCF;;;OAGG;IACH,YAAY,CAAC,MAAM,EAAE,MAAM,GAAG,UAAU;IAsBxC;;;;;;OAMG;IACH,OAAO,CACL,MAAM,EAAE,MAAM,EACd,IAAI,EAAE,IAAI,CACR,UAAU,EACR,SAAS,GACT,QAAQ,GACR,YAAY,GACZ,YAAY,GACZ,cAAc,GACd,SAAS,GACT,cAAc,GACd,QAAQ,GACR,QAAQ,CACX,GACA,UAAU;IAkCb;;;;;OAKG;IACH,SAAS,CACP,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,EACd,OAAO,EAAE;QACP,WAAW,EAAE,WAAW,CAAC;QACzB,OAAO,EAAE,MAAM,CAAC;QAChB,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,MAAM,EAAE,MAAM,CAAC;KAChB,GACA,UAAU;IA6Bb;;;OAGG;IACH,eAAe,CAAC,MAAM,EAAE,MAAM,GAAG,UAAU,EAAE;IAkB7C;;;;;;OAMG;IACH,YAAY,CACV,MAAM,EAAE,MAAM,EACd,MAAM,EAAE,MAAM,EACd,QAAQ,EAAE,OAAO,EAAE,EACnB,KAAK,CAAC,EAAE,SAAS,GAAG,IAAI,GACvB,IAAI;IAiDP;;;;;;OAMG;IACH,SAAS,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,SAAS,GAAG,IAAI,GAAG,IAAI;IA8C/F,iDAAiD;IACjD,UAAU,CAAC,MAAM,EAAE,MAAM,GAAG,aAAa;IAgBzC,4DAA4D;IAC5D,eAAe,CACb,MAAM,EAAE,MAAM,EACd,OAAO,EAAE,MAAM,EAAE,EACjB,SAAS,EAAE,MAAM,EAAE,GAClB,IAAI;IASP,gDAAgD;IAChD,UAAU,CACR,MAAM,EAAE,MAAM,EACd,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,aAAa,CAAC,GACrC,IAAI;IAYP,OAAO,CAAC,QAAQ;IAIhB,OAAO,CAAC,WAAW;IAInB,OAAO,CAAC,SAAS;IAOjB,OAAO,CAAC,YAAY;IAOpB,OAAO,CAAC,kBAAkB;IAkC1B;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAW3B;;;;;;;OAOG;IACH,OAAO,CAAC,qBAAqB;CAkH9B"}