@agentplaneorg/core 0.2.5 → 0.2.6

package/dist/commit/commit-policy.d.ts

@@ -6,7 +6,7 @@ export declare function extractTaskSuffix(taskId: string): string;
 export declare function isGenericSubject(subject: string, genericTokens: string[]): boolean;
 export declare function validateCommitSubject(opts: {
     subject: string;
-    taskId: string;
+    taskId?: string;
     genericTokens: string[];
 }): CommitPolicyResult;
 //# sourceMappingURL=commit-policy.d.ts.map

package/dist/commit/commit-policy.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"commit-policy.d.ts","sourceRoot":"","sources":["../../src/commit/commit-policy.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,kBAAkB,GAAG;IAC/B,EAAE,EAAE,OAAO,CAAC;IACZ,MAAM,EAAE,MAAM,EAAE,CAAC;CAClB,CAAC;AAMF,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAGxD;AAED,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,EAAE,GAAG,OAAO,CAOlF;AA4BD,wBAAgB,qBAAqB,CAAC,IAAI,EAAE;IAC1C,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,MAAM,EAAE,CAAC;CACzB,GAAG,kBAAkB,CAgCrB"}
+{"version":3,"file":"commit-policy.d.ts","sourceRoot":"","sources":["../../src/commit/commit-policy.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,kBAAkB,GAAG;IAC/B,EAAE,EAAE,OAAO,CAAC;IACZ,MAAM,EAAE,MAAM,EAAE,CAAC;CAClB,CAAC;AAQF,wBAAgB,iBAAiB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAGxD;AAED,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,EAAE,GAAG,OAAO,CAOlF;AA8CD,wBAAgB,qBAAqB,CAAC,IAAI,EAAE;IAC1C,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,MAAM,EAAE,CAAC;CACzB,GAAG,kBAAkB,CAwErB"}

package/dist/commit/commit-policy.js

@@ -1,3 +1,4 @@
+const NON_TASK_SUFFIX = "DEV";
 function stripPunctuation(input) {
     return input.replaceAll(/[^\p{L}\p{N}\s-]/gu, " ");
 }
@@ -15,7 +16,7 @@ export function isGenericSubject(subject, genericTokens) {
     const tokenSet = new Set(genericTokens.map((t) => t.toLowerCase()));
     return words.length <= 3 && words.every((w) => tokenSet.has(w));
 }
-function parseSubjectTemplate(subject) {
+function parseTaskSubjectTemplate(subject) {
     const trimmed = subject.trim();
     if (!trimmed)
         return null;
@@ -36,24 +37,64 @@ function parseSubjectTemplate(subject) {
         return null;
     return { emoji, suffix, scope, summary };
 }
+function parseNonTaskSubjectTemplate(subject) {
+    const trimmed = subject.trim();
+    if (!trimmed)
+        return null;
+    // Non-task: `<emoji> <scope>: <summary>`
+    const match = /^(\S+)\s+([a-z][a-z0-9_-]*):\s+(.+)$/.exec(trimmed);
+    if (!match)
+        return null;
+    const emoji = match[1] ?? "";
+    const scope = match[2] ?? "";
+    const summary = (match[3] ?? "").trim();
+    if (!emoji || !scope || !summary)
+        return null;
+    return { emoji, scope, summary };
+}
 export function validateCommitSubject(opts) {
     const errors = [];
     const subject = opts.subject.trim();
     if (!subject)
         errors.push("commit subject must be non-empty");
-    const suffix = extractTaskSuffix(opts.taskId);
-    const template = parseSubjectTemplate(subject);
-    if (!template) {
-        errors.push("commit subject must match: <emoji> <suffix> <scope>: <summary>");
-        return { ok: false, errors };
-    }
-    if (!suffix) {
-        errors.push("task id has no suffix");
+    const taskId = (opts.taskId ?? "").trim();
+    const taskSuffix = taskId ? extractTaskSuffix(taskId) : "";
+    if (taskSuffix) {
+        const template = parseTaskSubjectTemplate(subject);
+        if (!template) {
+            errors.push("commit subject must match: <emoji> <suffix> <scope>: <summary>", `example: ✅ ${taskSuffix} close: <summary>`, `example: 🚧 ${taskSuffix} task: <summary>`);
+            return { ok: false, errors };
+        }
+        if (template.suffix.toLowerCase() !== taskSuffix.toLowerCase()) {
+            errors.push("commit subject must include the task suffix as the second token");
+        }
     }
-    else if (template.suffix.toLowerCase() !== suffix.toLowerCase()) {
-        errors.push("commit subject must include task suffix as the second token");
+    else {
+        // Non-task commits: `<emoji> <scope>: <summary>`.
+        // We also support the explicit legacy form: `<emoji> DEV <scope>: <summary>`.
+        const nonTask = parseNonTaskSubjectTemplate(subject);
+        if (!nonTask) {
+            const taskLike = parseTaskSubjectTemplate(subject);
+            if (taskLike?.suffix?.toLowerCase() === NON_TASK_SUFFIX.toLowerCase()) {
+                // Explicit non-task form is allowed.
+            }
+            else {
+                if (taskLike?.suffix?.toLowerCase() !== NON_TASK_SUFFIX.toLowerCase()) {
+                    errors.push("task-like commit subject found, but task context is missing (AGENTPLANE_TASK_ID is unset)", "Fix:", "  1) Use the non-task format: <emoji> <scope>: <summary>", "  2) Or run the commit via agentplane so task context is set", "Examples:", "  ✨ ci: enforce full tests before push", "  🚧 ABCDEF task: implement upgrade allowlist (via agentplane)");
+                    return { ok: false, errors };
+                }
+                errors.push("non-task commit subject must match: <emoji> <scope>: <summary>", "example: ✨ ci: enforce full tests before push", `example (legacy explicit): ✨ ${NON_TASK_SUFFIX} ci: enforce full tests before push`);
+                return { ok: false, errors };
+            }
+        }
     }
-    const normalizedSummary = stripPunctuation(template.summary).toLowerCase().trim();
+    const parsedForSummary = parseNonTaskSubjectTemplate(subject) ??
+        (() => {
+            const t = parseTaskSubjectTemplate(subject);
+            return t ? { summary: t.summary } : null;
+        })();
+    const summary = parsedForSummary?.summary ?? "";
+    const normalizedSummary = stripPunctuation(summary).toLowerCase().trim();
     if (!normalizedSummary) {
         errors.push("commit subject is too generic");
         return { ok: errors.length === 0, errors };

package/dist/git/git-utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"git-utils.d.ts","sourceRoot":"","sources":["../../src/git/git-utils.ts"],"names":[],"mappings":"AAoBA,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAGpB;AAED,wBAAsB,gBAAgB,CAAC,IAAI,EAAE;IAC3C,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAOpB;AAID,wBAAsB,uBAAuB,CAAC,IAAI,EAAE;IAClD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAGpB"}
+{"version":3,"file":"git-utils.d.ts","sourceRoot":"","sources":["../../src/git/git-utils.ts"],"names":[],"mappings":"AA2CA,wBAAsB,cAAc,CAAC,IAAI,EAAE;IACzC,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAGpB;AAED,wBAAsB,gBAAgB,CAAC,IAAI,EAAE;IAC3C,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAOpB;AAID,wBAAsB,uBAAuB,CAAC,IAAI,EAAE;IAClD,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAGpB"}

package/dist/git/git-utils.js

@@ -14,9 +14,32 @@ async function gitNullSeparatedPaths(cwd, args) {
         .map((entry) => entry.trim())
         .filter((entry) => entry.length > 0);
 }
+async function gitStagedPathsIncludingRenames(cwd) {
+    // `--name-only` collapses renames to the destination path only, which allows
+    // protected-path deletions via `git mv`. `--name-status -z` includes both
+    // sides of renames/copies.
+    const parts = await gitNullSeparatedPaths(cwd, ["diff", "--name-status", "--cached", "-z"]);
+    const out = [];
+    for (let i = 0; i < parts.length;) {
+        const status = parts[i] ?? "";
+        const pathA = parts[i + 1] ?? "";
+        if (!status || !pathA)
+            break;
+        out.push(pathA);
+        i += 2;
+        const code = status[0] ?? "";
+        if (code === "R" || code === "C") {
+            const pathB = parts[i] ?? "";
+            if (pathB)
+                out.push(pathB);
+            i += 1;
+        }
+    }
+    return [...new Set(out)].toSorted((a, b) => a.localeCompare(b));
+}
 export async function getStagedFiles(opts) {
     const resolved = await resolveProject({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null });
-    return await gitNullSeparatedPaths(resolved.gitRoot, ["diff", "--name-only", "--cached", "-z"]);
+    return await gitStagedPathsIncludingRenames(resolved.gitRoot);
 }
 export async function getUnstagedFiles(opts) {
     const resolved = await resolveProject({ cwd: opts.cwd, rootOverride: opts.rootOverride ?? null });

package/dist/project/project-root.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"project-root.d.ts","sourceRoot":"","sources":["../../src/project/project-root.ts"],"names":[],"mappings":"AAiBA,wBAAsB,WAAW,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAU1E;AAED,MAAM,MAAM,qBAAqB,GAAG;IAClC,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,CAAC;AAEF,MAAM,MAAM,eAAe,GAAG;IAC5B,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,MAAM,CAAC;CACvB,CAAC;AAEF,wBAAsB,cAAc,CAAC,IAAI,EAAE,qBAAqB,GAAG,OAAO,CAAC,eAAe,CAAC,CAU1F"}
+{"version":3,"file":"project-root.d.ts","sourceRoot":"","sources":["../../src/project/project-root.ts"],"names":[],"mappings":"AAiBA,wBAAsB,WAAW,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAU1E;AAED,MAAM,MAAM,qBAAqB,GAAG;IAClC,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;CAC9B,CAAC;AAEF,MAAM,MAAM,eAAe,GAAG;IAC5B,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,MAAM,CAAC;CACvB,CAAC;AAEF,wBAAsB,cAAc,CAAC,IAAI,EAAE,qBAAqB,GAAG,OAAO,CAAC,eAAe,CAAC,CAW1F"}

package/dist/project/project-root.js

@@ -27,12 +27,13 @@ export async function findGitRoot(startDir) {
 }
 export async function resolveProject(opts) {
     const start = opts.rootOverride ? path.resolve(opts.rootOverride) : path.resolve(opts.cwd);
-    const gitRoot = await findGitRoot(start);
-    if (!gitRoot) {
+    // Intentionally do not search parent directories. agentplane is scoped to the
+    // explicit rootOverride, or to the current working directory.
+    if (!(await isGitRoot(start))) {
         throw new Error(`Not a git repository (start: ${start})`);
     }
     return {
-        gitRoot,
-        agentplaneDir: path.join(gitRoot, ".agentplane"),
+        gitRoot: start,
+        agentplaneDir: path.join(start, ".agentplane"),
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentplaneorg/core",
-  "version": "0.2.5",
+  "version": "0.2.6",
   "description": "Core utilities and models for the Agent Plane CLI.",
   "keywords": [
     "agentplane",