@goodtek/vibeops 0.7.22 → 1.0.0

package/dist/commands/task-done.js

@@ -1,98 +1,141 @@
-import { relative, resolve } from "node:path";
+import { resolve } from "node:path";
+import { askYesNo } from "../lib/inquirer-helpers.js";
 import { readText } from "../lib/filesystem.js";
 import { bold, cyan, dim, green, log, yellow } from "../lib/logger.js";
 import { projectPaths } from "../lib/paths.js";
-import { findTaskFile, hasNonEmptySection, parseTaskFilename, pickNextTask, readGitContext, readTaskFile, scanTasks, statusDisplay, updateInlineStatus, } from "../lib/task.js";
-function relOrAbs(root, p) {
-    const r = relative(root, p);
-    return r === "" ? "." : r.startsWith("..") ? p : r;
+import { resolveTask, taskNotFoundMessage } from "../lib/resolve-task.js";
+import { mergeTaskBranch } from "../lib/task-merge.js";
+import { applyTaskDoneMemory, fallbackResultSections, llmCompleteTaskDone, writeTaskResultSections, } from "../lib/task-done-llm.js";
+import { relPath } from "../lib/task-context.js";
+import { gitAddPaths, listWorkingTreeRelPaths, partitionPathsForAutoCommit, readGitInfo, runGit, } from "../lib/git.js";
+import { hasNonEmptySection, markGitContextDone, parseTaskFilename, readGitContext, readTaskFile, updateInlineStatus, } from "../lib/task.js";
+function commitMessageFor(taskId, title) {
+    const slug = title.replace(/^TASK-\d+\s*[:\-]\s*/i, "").trim() || taskId;
+    return `feat(${taskId.toLowerCase()}): ${slug}`;
 }
-function commitMessageFor(taskId, title, mvpPhase) {
-    const trimmed = (title || taskId).trim();
-    const slug = trimmed.replace(/^TASK-\d+\s*[·:\-]\s*/i, "").trim() || trimmed;
-    const scope = taskId.toLowerCase();
-    const typePrefix = typeof mvpPhase === "string" && /rollback/i.test(mvpPhase) ? "chore" : "feat";
-    return `${typePrefix}(${scope}): ${slug}`;
+async function tryCommitDirty(cwd, message, dryRun) {
+    const git = await readGitInfo(cwd);
+    if (!git.isRepo || git.dirty !== true)
+        return false;
+    const { committable, excluded } = partitionPathsForAutoCommit(await listWorkingTreeRelPaths(cwd), { unmerged: [] });
+    if (committable.length === 0)
+        return false;
+    if (excluded.length > 0) {
+        log.warn(`Skipping ${excluded.length} build artifact path(s) from auto-commit.`);
+    }
+    if (dryRun) {
+        log.info(dim(`  would commit ${committable.length} path(s)`));
+        return true;
+    }
+    await gitAddPaths(cwd, committable);
+    await runGit(cwd, ["commit", "-q", "-m", message]);
+    log.ok(`Committed: ${message}`);
+    return true;
 }
-export async function taskDoneCommand(taskId, options = {}) {
+export async function taskDoneCommand(taskRef, options = {}) {
     const cwd = resolve(options.cwd ?? process.cwd());
     const paths = projectPaths(cwd);
-    const taskFile = await findTaskFile(paths.docsTasks, taskId);
-    if (!taskFile) {
-        log.error(`TASK not found: ${taskId} (looked in ${relOrAbs(cwd, paths.docsTasks)})`);
+    const dryRun = options.dryRun === true;
+    const interactive = !dryRun && process.stdin.isTTY === true && options.yes !== true;
+    const resolved = await resolveTask(paths, cwd, taskRef);
+    if (resolved === null) {
+        log.error(taskNotFoundMessage(taskRef));
         process.exitCode = 1;
         return;
     }
+    const { taskId, taskFile } = resolved;
     const meta = await readTaskFile(taskFile);
     const body = await readText(taskFile);
-    log.info(bold(`vibeops task done ${meta.id}`));
-    log.info(`  ${dim("file")}    ${relOrAbs(cwd, taskFile)}`);
-    log.info(`  ${dim("title")}   ${meta.title || dim("(no title)")}`);
-    log.info(`  ${dim("status")}  ${statusDisplay(meta.status)}`);
+    const relFile = relPath(cwd, taskFile);
+    log.info(bold(`vibeops task done ${taskId}`));
+    log.info(`  ${dim("file")}  ${relFile}`);
     log.blank();
-    const resultOk = hasNonEmptySection(body, "Result");
-    const testResultOk = hasNonEmptySection(body, "Test Result");
-    const docCheck = [
-        ["Result section", resultOk],
-        ["Test Result section", testResultOk],
-    ];
-    log.info(bold("Required sections"));
-    let missing = 0;
-    for (const [label, ok] of docCheck) {
-        if (!ok)
-            missing++;
-        log.info(`  ${ok ? green("✓") : yellow("·")} ${label}${ok ? "" : dim(" (empty or placeholder)")}`);
+    let resultOk = hasNonEmptySection(body, "Result");
+    let testOk = hasNonEmptySection(body, "Test Result");
+    if (!resultOk || !testOk) {
+        log.step("LLM — Result / Test Result + project memory");
+        if (dryRun) {
+            log.info(dim("  would call LLM and patch docs/project/*"));
+        }
+        else {
+            const patch = await llmCompleteTaskDone({
+                cwd,
+                taskId,
+                taskTitle: meta.title,
+                taskBody: body,
+                taskFileRel: relFile,
+            });
+            if (patch !== null) {
+                await writeTaskResultSections(taskFile, patch.result, patch.testResult);
+                log.ok(`Result / Test Result updated (${patch.provider})`);
+                const memory = await applyTaskDoneMemory(cwd, patch);
+                for (const p of memory.updated)
+                    log.ok(`Updated ${p}`);
+                for (const p of memory.skipped)
+                    log.skip(`Skipped ${p}`);
+            }
+            else {
+                log.warn("LLM unavailable — using git diff fallback for Result sections.");
+                const fb = await fallbackResultSections(cwd, taskId, taskFile);
+                await writeTaskResultSections(taskFile, fb.result, fb.testResult);
+            }
+        }
+        const body2 = dryRun ? body : await readText(taskFile);
+        resultOk = hasNonEmptySection(body2, "Result");
+        testOk = hasNonEmptySection(body2, "Test Result");
     }
+    log.info(bold("Sections"));
+    log.info(`  ${resultOk ? green("✓") : yellow("·")} Result`);
+    log.info(`  ${testOk ? green("✓") : yellow("·")} Test Result`);
     log.blank();
-    if (missing > 0) {
-        log.error(`${missing} required section(s) still empty. Fill Result / Test Result first, then re-run \`vibeops task done\`.`);
+    if (!dryRun && (!resultOk || !testOk)) {
+        log.error("Fill Result and Test Result in the TASK file, then rerun.");
         process.exitCode = 1;
         return;
     }
-    const target = options.finalize === true ? "done" : "review";
-    const targetDisplay = statusDisplay(target);
-    if (options.dryRun === true) {
-        log.info(bold("dry-run — would perform:"));
-        log.info(`  · update Status → ${targetDisplay} in ${relOrAbs(cwd, taskFile)}`);
-        log.info(`  · (no git commit, no Notion call)`);
-        log.blank();
-    }
-    else {
-        await updateInlineStatus(taskFile, target);
-        log.ok(`Status → ${targetDisplay}  (${relOrAbs(cwd, taskFile)})`);
-        log.blank();
+    if (dryRun) {
+        log.info(bold("dry-run — would:"));
+        if (!resultOk || !testOk) {
+            log.info("  · LLM fill Result / Test Result + patch docs/project/*");
+        }
+        log.info("  · Status → Done");
+        log.info("  · commit if dirty, merge task branch (unless --no-merge)");
+        return;
     }
-    const ctx = await readGitContext(taskFile);
+    await updateInlineStatus(taskFile, "done");
+    await markGitContextDone(taskFile);
+    log.ok("Status → Done");
     const parts = parseTaskFilename(taskFile);
-    const commitMsg = commitMessageFor(parts.id, meta.title, meta.mvpPhase);
-    log.info(bold("Suggested commit"));
-    log.info(`  ${cyan(`git add -A && git commit -m "${commitMsg}"`)}`);
-    if (ctx !== null) {
-        log.info(`  ${dim(`after commit on ${ctx.taskBranch}: open PR or run \`git switch ${ctx.baseBranch} && git merge --no-ff ${ctx.taskBranch}\``)}`);
-    }
-    else {
-        log.info(`  ${dim("(no Git Context section — skipping merge guidance)")}`);
+    const commitMsg = commitMessageFor(parts.id, meta.title);
+    await tryCommitDirty(cwd, commitMsg, false);
+    const gitCtx = await readGitContext(taskFile);
+    if (options.noMerge !== true && gitCtx !== null) {
+        let doMerge = options.yes === true || !interactive;
+        if (interactive) {
+            doMerge = await askYesNo({
+                message: `Merge into ${gitCtx.baseBranch} and delete ${gitCtx.taskBranch}?`,
+                nonInteractive: false,
+                defaultValue: true,
+            });
+        }
+        if (doMerge) {
+            const mergeResult = await mergeTaskBranch({
+                cwd,
+                taskId: meta.id,
+                taskTitle: meta.title,
+                gitCtx,
+                allowDirty: options.allowDirty,
+            });
+            if (!mergeResult.ok)
+                process.exitCode = 1;
+        }
+        else {
+            log.info(dim("Merge skipped."));
+        }
     }
-    log.blank();
-    log.info(bold("Notion"));
-    log.info(`  ${dim("TODO: run `vibeops notion sync` to push this TASK metadata.")}`);
-    log.blank();
-    const all = await scanTasks(paths.docsTasks);
-    const next = pickNextTask(all.filter((t) => t.id.toUpperCase() !== meta.id.toUpperCase()));
-    log.info(bold("Next TASK candidate"));
-    if (next) {
-        log.info(`  → ${cyan(next.id)} — ${next.title || dim("(no title)")}  ${dim(`[${statusDisplay(next.status)}]`)}`);
-        log.info(`    file: ${dim(relOrAbs(cwd, next.filePath))}`);
-        log.info(`    start: ${dim(`vibeops task start ${next.id}`)}`);
-    }
-    else {
-        log.info(`  ${dim("(no remaining planned / in-progress / review TASK)")}`);
+    else if (gitCtx === null) {
+        log.info(dim("No Git Context — merge skipped."));
     }
     log.blank();
-    if (target === "review") {
-        log.info(`Status moved to ${cyan("Review")}. After human or Reviewer Agent review, finalize with ${cyan(`vibeops task done ${meta.id} --finalize`)}.`);
-    }
-    else {
-        log.ok(`${meta.id} finalized → Done.`);
-    }
+    log.info(`Next: ${cyan("vibeops task add")} or ${cyan("vibeops status")}`);
 }

package/dist/lib/config.js

@@ -2,39 +2,7 @@ import { join } from "node:path";
 import { readTextOrNull, writeText } from "./filesystem.js";
 import { VIBEOPS_CONFIG_FILE } from "./paths.js";
 import { VERSION } from "../version.js";
-import { DEFAULT_GITHUB_CONFIG, VIBEOPS_CONFIG_SCHEMA_VERSION, } from "../types/config.js";
-function parseNotionSection(raw) {
-    if (raw === null || typeof raw !== "object")
-        return undefined;
-    const r = raw;
-    const out = {
-        enabled: typeof r.enabled === "boolean" ? r.enabled : false,
-        projectsTargetId: typeof r.projectsTargetId === "string" ? r.projectsTargetId : "",
-        tasksTargetId: typeof r.tasksTargetId === "string" ? r.tasksTargetId : "",
-        projectsDatabaseId: typeof r.projectsDatabaseId === "string" ? r.projectsDatabaseId : "",
-        tasksDatabaseId: typeof r.tasksDatabaseId === "string" ? r.tasksDatabaseId : "",
-    };
-    return out;
-}
-function parseGithubVisibility(raw) {
-    if (raw === "public" || raw === "private")
-        return raw;
-    return "";
-}
-function parseGithubSection(raw) {
-    if (raw === null || typeof raw !== "object")
-        return undefined;
-    const r = raw;
-    return {
-        enabled: typeof r.enabled === "boolean" ? r.enabled : false,
-        mode: r.mode === "gh-cli" ? "gh-cli" : "gh-cli",
-        owner: typeof r.owner === "string" ? r.owner : "",
-        repo: typeof r.repo === "string" ? r.repo : "",
-        remote: typeof r.remote === "string" && r.remote.length > 0 ? r.remote : "origin",
-        visibility: parseGithubVisibility(r.visibility),
-        url: typeof r.url === "string" ? r.url : "",
-    };
-}
+import { VIBEOPS_CONFIG_SCHEMA_VERSION } from "../types/config.js";
 export async function readConfig(root) {
     const text = await readTextOrNull(join(root, VIBEOPS_CONFIG_FILE));
     if (text === null)
@@ -45,19 +13,12 @@ export async function readConfig(root) {
             typeof parsed.vibeopsVersion === "string" &&
             typeof parsed.createdAt === "string" &&
             parsed.schemaVersion === VIBEOPS_CONFIG_SCHEMA_VERSION) {
-            const notion = parseNotionSection(parsed.notion);
-            const github = parseGithubSection(parsed.github);
-            const config = {
+            return {
                 name: parsed.name,
                 vibeopsVersion: parsed.vibeopsVersion,
                 schemaVersion: VIBEOPS_CONFIG_SCHEMA_VERSION,
                 createdAt: parsed.createdAt,
             };
-            if (notion)
-                config.notion = notion;
-            if (github)
-                config.github = github;
-            return config;
         }
         return null;
     }
@@ -79,80 +40,3 @@ export function configToJson(config) {
 export async function writeConfig(root, config) {
     await writeText(join(root, VIBEOPS_CONFIG_FILE), configToJson(config));
 }
-/** Merge a partial notion section into an existing config without touching other fields. */
-export function mergeNotionConfig(base, patch) {
-    const current = base.notion ?? {
-        enabled: false,
-        projectsTargetId: "",
-        tasksTargetId: "",
-        projectsDatabaseId: "",
-        tasksDatabaseId: "",
-    };
-    const next = {
-        enabled: patch.enabled ?? current.enabled,
-        projectsTargetId: patch.projectsTargetId !== undefined && patch.projectsTargetId.length > 0
-            ? patch.projectsTargetId
-            : current.projectsTargetId,
-        tasksTargetId: patch.tasksTargetId !== undefined && patch.tasksTargetId.length > 0
-            ? patch.tasksTargetId
-            : current.tasksTargetId,
-        projectsDatabaseId: patch.projectsDatabaseId !== undefined && patch.projectsDatabaseId.length > 0
-            ? patch.projectsDatabaseId
-            : current.projectsDatabaseId,
-        tasksDatabaseId: patch.tasksDatabaseId !== undefined && patch.tasksDatabaseId.length > 0
-            ? patch.tasksDatabaseId
-            : current.tasksDatabaseId,
-    };
-    const changed = base.notion === undefined ||
-        next.enabled !== current.enabled ||
-        next.projectsTargetId !== current.projectsTargetId ||
-        next.tasksTargetId !== current.tasksTargetId ||
-        next.projectsDatabaseId !== current.projectsDatabaseId ||
-        next.tasksDatabaseId !== current.tasksDatabaseId;
-    return { merged: { ...base, notion: next }, changed };
-}
-/** Preferred Projects target id for API calls: data_source target first, legacy DB fallback. */
-export function notionProjectsTargetId(notion) {
-    return notion.projectsTargetId.length > 0
-        ? notion.projectsTargetId
-        : notion.projectsDatabaseId;
-}
-/** Preferred Tasks target id for API calls: data_source target first, legacy DB fallback. */
-export function notionTasksTargetId(notion) {
-    return notion.tasksTargetId.length > 0 ? notion.tasksTargetId : notion.tasksDatabaseId;
-}
-/**
- * Merge a partial github section into an existing config without touching
- * other fields (notion, name, etc.). Empty strings in the patch are treated
- * as "no value provided" — the existing value is kept. Pass through
- * `enabled` / `mode` always (booleans / enums always overwrite).
- */
-export function mergeGithubConfig(base, patch) {
-    const current = base.github ?? DEFAULT_GITHUB_CONFIG;
-    const next = {
-        enabled: patch.enabled ?? current.enabled,
-        mode: patch.mode ?? current.mode,
-        owner: patch.owner !== undefined && patch.owner.length > 0
-            ? patch.owner
-            : current.owner,
-        repo: patch.repo !== undefined && patch.repo.length > 0
-            ? patch.repo
-            : current.repo,
-        remote: patch.remote !== undefined && patch.remote.length > 0
-            ? patch.remote
-            : current.remote,
-        visibility: patch.visibility !== undefined && patch.visibility.length > 0
-            ? patch.visibility
-            : current.visibility,
-        url: patch.url !== undefined && patch.url.length > 0 ? patch.url : current.url,
-    };
-    const changed = base.github === undefined ||
-        next.enabled !== current.enabled ||
-        next.mode !== current.mode ||
-        next.owner !== current.owner ||
-        next.repo !== current.repo ||
-        next.remote !== current.remote ||
-        next.visibility !== current.visibility ||
-        next.url !== current.url;
-    return { merged: { ...base, github: next }, changed };
-}

package/dist/lib/git.js

@@ -370,7 +370,7 @@ export async function gitAddPaths(cwd, paths) {
 /**
  * When the index/worktree is dirty, classify whether every changed path is
  * limited to governance docs and VibeOps metadata (`.vibeops/**`, docs/tasks, etc.).
- * Used by `vibeops task start` so `task done --finalize` on main does not
+ * Used when switching task branches so governance paths are not left off-disk after switch.
  * block the next TASK unless application code is also dirty.
  */
 export async function gitGovernanceOnlyDirty(cwd) {

package/dist/lib/paths.js

@@ -14,13 +14,17 @@ export function projectPaths(root) {
         envExample: join(abs, VIBEOPS_ENV_EXAMPLE_FILE),
         agentsMd: join(abs, "AGENTS.md"),
         cursorRules: join(abs, ".cursor", "rules"),
+        cursorSkills: join(abs, ".cursor", "skills"),
         docsProject: join(abs, "docs", "project"),
         docsTasks: join(abs, "docs", "tasks"),
         docsLogs: join(abs, "docs", "logs"),
         vibeopsDir: join(abs, ".vibeops"),
-        vibeopsAgents: join(abs, ".vibeops", "agents"),
-        vibeopsPrompts: join(abs, ".vibeops", "prompts"),
-        vibeopsWorkflows: join(abs, ".vibeops", "workflows"),
         gitignore: join(abs, ".gitignore"),
     };
 }
+/** Project memory files updated by `vibeops task done`. */
+export const PROJECT_MEMORY_FILES = {
+    currentState: "docs/project/05-current-state.md",
+    architecture: "docs/project/03-architecture.md",
+    decisions: "docs/project/06-decisions.md",
+};

package/dist/lib/plan-llm-openai.js

@@ -29,7 +29,6 @@ export async function openAiChatCompletionJson(params) {
     }
     return content.trim();
 }
-/** Plain assistant text (no `response_format`) — used for planner markdown apply. */
 export async function openAiChatCompletionText(params) {
     const key = process.env.OPENAI_API_KEY?.trim();
     if (!key) {
@@ -70,65 +69,3 @@ export function extractJsonObject(text) {
         return trimmed.slice(start, end + 1);
     return trimmed;
 }
-export function parsePlanLlmTurn(jsonText) {
-    const raw = JSON.parse(jsonText);
-    const turn = raw.turn;
-    if (turn === "done") {
-        const pb = raw.projectBrief;
-        if (!pb || typeof pb !== "object" || Array.isArray(pb)) {
-            throw new Error('Invalid JSON: "done" turn requires object "projectBrief".');
-        }
-        const plannerAssumptions = raw.plannerAssumptions;
-        const pa = Array.isArray(plannerAssumptions) && plannerAssumptions.every((x) => typeof x === "string")
-            ? plannerAssumptions
-            : undefined;
-        return {
-            turn: "done",
-            projectBrief: pb,
-            plannerAssumptions: pa,
-        };
-    }
-    if (turn === "confirm") {
-        const readableSummary = typeof raw.readableSummary === "string" ? raw.readableSummary.trim() : "";
-        if (readableSummary.length === 0) {
-            throw new Error('Invalid JSON: "confirm" turn needs non-empty "readableSummary" (markdown).');
-        }
-        const plannerNote = typeof raw.plannerNote === "string" ? raw.plannerNote.trim() : undefined;
-        return {
-            turn: "confirm",
-            readableSummary,
-            plannerNote: plannerNote && plannerNote.length > 0 ? plannerNote : undefined,
-        };
-    }
-    if (turn === "question") {
-        const message = typeof raw.message === "string" ? raw.message : "";
-        if (message.length === 0) {
-            throw new Error('Invalid JSON: "question" turn needs non-empty "message".');
-        }
-        const options = Array.isArray(raw.options)
-            ? raw.options.filter((x) => typeof x === "string" && x.trim().length > 0)
-            : [];
-        const rawQt = raw.questionType;
-        let questionType;
-        if (rawQt === "single" || rawQt === "multi" || rawQt === "text") {
-            questionType = rawQt;
-        }
-        else if (options.length >= 2) {
-            /** Model often omits or mistypes questionType — multi is the safer default when lists are offered. */
-            questionType = "multi";
-        }
-        else if (options.length === 1) {
-            questionType = "single";
-        }
-        else {
-            questionType = "text";
-        }
-        return {
-            turn: "question",
-            message,
-            questionType,
-            options: options.length > 0 ? options : undefined,
-        };
-    }
-    throw new Error('Invalid JSON: "turn" must be "question", "confirm", or "done".');
-}

package/dist/lib/resolve-task.js

@@ -0,0 +1,34 @@
+import { findTaskFile, loadActionableTasks, pickActiveTask, pickInProgressTask, } from "./task.js";
+export function normalizeTaskRef(ref) {
+    const t = ref.trim();
+    if (/^task-/i.test(t) && !/^TASK-/i.test(t)) {
+        return `TASK-${t.replace(/^task-/i, "")}`;
+    }
+    return t.toUpperCase().startsWith("TASK-") ? t.toUpperCase() : `TASK-${t}`;
+}
+export async function resolveTask(paths, cwd, taskRef) {
+    const trimmed = taskRef?.trim();
+    if (trimmed !== undefined && trimmed.length > 0) {
+        const taskId = normalizeTaskRef(trimmed);
+        const taskFile = await findTaskFile(paths.docsTasks, taskId);
+        if (taskFile === null)
+            return null;
+        return { taskId, taskFile };
+    }
+    const tasks = await loadActionableTasks(paths.docsTasks);
+    const inProgress = pickInProgressTask(tasks);
+    if (inProgress !== null) {
+        return { taskId: inProgress.id, taskFile: inProgress.filePath };
+    }
+    const onBranch = await pickActiveTask(paths.docsTasks, cwd);
+    if (onBranch !== null && onBranch.status !== "done") {
+        return { taskId: onBranch.id, taskFile: onBranch.filePath };
+    }
+    return null;
+}
+export function taskNotFoundMessage(taskRef) {
+    if (taskRef?.trim()) {
+        return `TASK not found: ${normalizeTaskRef(taskRef)}. Check docs/tasks/.`;
+    }
+    return "No open TASK found. Run `vibeops task add` or pass `vibeops task done TASK-NNN`.";
+}