@mcoda/core 0.1.23 → 0.1.26

package/dist/services/planning/RefineTasksService.js

@@ -194,6 +194,22 @@ const formatTaskSummary = (task) => {
         .filter(Boolean)
         .join("\n");
 };
+const splitChildReferenceFields = ["taskKey", "key", "localId", "id", "slug", "alias", "ref"];
+const normalizeSplitDependencyRef = (value) => value.trim().toLowerCase();
+const collectSplitChildReferences = (child) => {
+    const references = [];
+    const childRecord = child;
+    for (const field of splitChildReferenceFields) {
+        const candidate = childRecord[field];
+        if (typeof candidate === "string" && candidate.trim().length > 0) {
+            references.push(candidate.trim());
+        }
+    }
+    if (typeof child.title === "string" && child.title.trim().length > 0) {
+        references.push(child.title.trim());
+    }
+    return Array.from(new Set(references));
+};
 export class RefineTasksService {
     constructor(workspace, deps) {
         this.workspace = workspace;
@@ -880,10 +896,39 @@ export class RefineTasksService {
         }
         if (op.op === "split_task") {
             const split = op;
-            const invalidDep = split.children.some((child) => child.dependsOn?.some((dep) => !keySet.has(dep)));
+            const taskKeysByNormalized = new Map();
+            for (const key of keySet) {
+                taskKeysByNormalized.set(normalizeSplitDependencyRef(key), key);
+            }
+            const siblingReferences = new Set();
+            const selfReferencesByChild = split.children.map((child) => {
+                const selfReferences = new Set();
+                for (const reference of collectSplitChildReferences(child)) {
+                    const normalized = normalizeSplitDependencyRef(reference);
+                    if (!normalized)
+                        continue;
+                    selfReferences.add(normalized);
+                    siblingReferences.add(normalized);
+                }
+                return selfReferences;
+            });
+            const invalidDep = split.children.some((child) => child.dependsOn?.some((dep) => {
+                const normalized = normalizeSplitDependencyRef(dep);
+                if (!normalized)
+                    return false;
+                if (taskKeysByNormalized.has(normalized))
+                    return false;
+                if (siblingReferences.has(normalized))
+                    return false;
+                return true;
+            }));
             if (invalidDep) {
                 return { valid: false, reason: "Split child references unknown dependency" };
             }
+            const selfDependency = split.children.some((child, index) => child.dependsOn?.some((dep) => selfReferencesByChild[index]?.has(normalizeSplitDependencyRef(dep))));
+            if (selfDependency) {
+                return { valid: false, reason: "Split child cannot depend on itself" };
+            }
             if (split.children.some((child) => child.storyPoints !== undefined && child.storyPoints !== null && (child.storyPoints < 0 || child.storyPoints > 13))) {
                 return { valid: false, reason: "Child story points out of bounds" };
             }
@@ -1049,8 +1094,25 @@ export class RefineTasksService {
                                 createdAt: new Date().toISOString(),
                             });
                         }
-                        for (const child of op.children) {
-                            const childKey = keyGen();
+                        const existingTaskKeyByNormalized = new Map();
+                        for (const key of taskByKey.keys()) {
+                            existingTaskKeyByNormalized.set(normalizeSplitDependencyRef(key), key);
+                        }
+                        const childKeys = op.children.map(() => keyGen());
+                        const childRefToKey = new Map();
+                        op.children.forEach((child, index) => {
+                            const childKey = childKeys[index];
+                            childRefToKey.set(normalizeSplitDependencyRef(childKey), childKey);
+                            for (const reference of collectSplitChildReferences(child)) {
+                                const normalized = normalizeSplitDependencyRef(reference);
+                                if (!normalized || childRefToKey.has(normalized))
+                                    continue;
+                                childRefToKey.set(normalized, childKey);
+                            }
+                        });
+                        for (let index = 0; index < op.children.length; index += 1) {
+                            const child = op.children[index];
+                            const childKey = childKeys[index];
                             const childSp = child.storyPoints ?? null;
                             if (childSp) {
                                 storyPointsDelta += childSp;
@@ -1082,17 +1144,24 @@ export class RefineTasksService {
                                 openapiVersionAtCreation: target.openapiVersionAtCreation ?? null,
                             };
                             newTasks.push(childInsert);
-                            const dependsOn = child.dependsOn ?? [];
-                            for (const depKey of dependsOn) {
-                                const depTask = taskByKey.get(depKey);
-                                if (depTask) {
-                                    pendingDeps.push({
-                                        childKey,
-                                        dependsOnId: depTask.id,
-                                        dependsOnKey: depTask.key,
-                                        relationType: "blocks",
-                                    });
+                            for (const dependencyReference of child.dependsOn ?? []) {
+                                const normalizedReference = normalizeSplitDependencyRef(dependencyReference);
+                                if (!normalizedReference)
+                                    continue;
+                                const dependencyKey = existingTaskKeyByNormalized.get(normalizedReference) ?? childRefToKey.get(normalizedReference);
+                                if (!dependencyKey) {
+                                    warnings.push(`Skipped split dependency ${childKey}->${dependencyReference}: unresolved sibling or task reference.`);
+                                    continue;
+                                }
+                                if (dependencyKey === childKey) {
+                                    warnings.push(`Skipped split dependency ${childKey}->${dependencyReference}: self dependency.`);
+                                    continue;
                                 }
+                                pendingDeps.push({
+                                    childKey,
+                                    dependsOnKey: dependencyKey,
+                                    relationType: "blocks",
+                                });
                             }
                             taskByKey.set(childKey, {
                                 ...childInsert,
@@ -1247,9 +1316,14 @@ export class RefineTasksService {
                     }
                     for (const dep of allowedDeps) {
                         const childId = idByKey.get(dep.childKey);
-                        if (childId) {
-                            deps.push({ taskId: childId, dependsOnTaskId: dep.dependsOnId, relationType: dep.relationType });
+                        if (!childId)
+                            continue;
+                        const dependsOnId = idByKey.get(dep.dependsOnKey) ?? taskByKey.get(dep.dependsOnKey)?.id;
+                        if (!dependsOnId) {
+                            warnings.push(`Skipped refine dependency ${dep.childKey}->${dep.dependsOnKey}: dependency task not found.`);
+                            continue;
                         }
+                        deps.push({ taskId: childId, dependsOnTaskId: dependsOnId, relationType: dep.relationType });
                     }
                     if (deps.length > 0) {
                         stage = "insert:deps";

package/dist/services/planning/TaskSufficiencyService.d.ts

@@ -0,0 +1,73 @@
+import { WorkspaceRepository } from "@mcoda/db";
+import { WorkspaceResolution } from "../../workspace/WorkspaceManager.js";
+import { JobService } from "../jobs/JobService.js";
+export interface TaskSufficiencyAuditRequest {
+    workspace: WorkspaceResolution;
+    projectKey: string;
+    dryRun?: boolean;
+    maxIterations?: number;
+    maxTasksPerIteration?: number;
+    minCoverageRatio?: number;
+    sourceCommand?: string;
+}
+export interface TaskSufficiencyAuditIteration {
+    iteration: number;
+    coverageRatio: number;
+    totalSignals: number;
+    missingSectionCount: number;
+    missingFolderCount: number;
+    createdTaskKeys: string[];
+}
+export interface TaskSufficiencyAuditResult {
+    jobId: string;
+    commandRunId: string;
+    projectKey: string;
+    sourceCommand?: string;
+    satisfied: boolean;
+    dryRun: boolean;
+    totalTasksAdded: number;
+    totalTasksUpdated: number;
+    maxIterations: number;
+    minCoverageRatio: number;
+    finalCoverageRatio: number;
+    remainingSectionHeadings: string[];
+    remainingFolderEntries: string[];
+    remainingGaps: {
+        sections: number;
+        folders: number;
+        total: number;
+    };
+    iterations: TaskSufficiencyAuditIteration[];
+    reportPath: string;
+    reportHistoryPath?: string;
+    warnings: string[];
+}
+type TaskSufficiencyDeps = {
+    workspaceRepo: WorkspaceRepository;
+    jobService: JobService;
+};
+export declare class TaskSufficiencyService {
+    private readonly workspaceRepo;
+    private readonly jobService;
+    private readonly ownsWorkspaceRepo;
+    private readonly ownsJobService;
+    private readonly workspace;
+    constructor(workspace: WorkspaceResolution, deps: TaskSufficiencyDeps, ownership?: {
+        ownsWorkspaceRepo?: boolean;
+        ownsJobService?: boolean;
+    });
+    static create(workspace: WorkspaceResolution): Promise<TaskSufficiencyService>;
+    close(): Promise<void>;
+    private discoverSdsPaths;
+    private walkSdsCandidates;
+    private loadSdsSources;
+    private loadProjectSnapshot;
+    private evaluateCoverage;
+    private buildGapItems;
+    private ensureTargetStory;
+    private insertGapTasks;
+    private writeReportArtifacts;
+    runAudit(request: TaskSufficiencyAuditRequest): Promise<TaskSufficiencyAuditResult>;
+}
+export {};
+//# sourceMappingURL=TaskSufficiencyService.d.ts.map

package/dist/services/planning/TaskSufficiencyService.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"TaskSufficiencyService.d.ts","sourceRoot":"","sources":["../../../src/services/planning/TaskSufficiencyService.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,mBAAmB,EAAiC,MAAM,WAAW,CAAC;AAE/E,OAAO,EAAE,mBAAmB,EAAE,MAAM,qCAAqC,CAAC;AAC1E,OAAO,EAAE,UAAU,EAAE,MAAM,uBAAuB,CAAC;AAsCnD,MAAM,WAAW,2BAA2B;IAC1C,SAAS,EAAE,mBAAmB,CAAC;IAC/B,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,6BAA6B;IAC5C,SAAS,EAAE,MAAM,CAAC;IAClB,aAAa,EAAE,MAAM,CAAC;IACtB,YAAY,EAAE,MAAM,CAAC;IACrB,mBAAmB,EAAE,MAAM,CAAC;IAC5B,kBAAkB,EAAE,MAAM,CAAC;IAC3B,eAAe,EAAE,MAAM,EAAE,CAAC;CAC3B;AAED,MAAM,WAAW,0BAA0B;IACzC,KAAK,EAAE,MAAM,CAAC;IACd,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,SAAS,EAAE,OAAO,CAAC;IACnB,MAAM,EAAE,OAAO,CAAC;IAChB,eAAe,EAAE,MAAM,CAAC;IACxB,iBAAiB,EAAE,MAAM,CAAC;IAC1B,aAAa,EAAE,MAAM,CAAC;IACtB,gBAAgB,EAAE,MAAM,CAAC;IACzB,kBAAkB,EAAE,MAAM,CAAC;IAC3B,wBAAwB,EAAE,MAAM,EAAE,CAAC;IACnC,sBAAsB,EAAE,MAAM,EAAE,CAAC;IACjC,aAAa,EAAE;QACb,QAAQ,EAAE,MAAM,CAAC;QACjB,OAAO,EAAE,MAAM,CAAC;QAChB,KAAK,EAAE,MAAM,CAAC;KACf,CAAC;IACF,UAAU,EAAE,6BAA6B,EAAE,CAAC;IAC5C,UAAU,EAAE,MAAM,CAAC;IACnB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,QAAQ,EAAE,MAAM,EAAE,CAAC;CACpB;AAED,KAAK,mBAAmB,GAAG;IACzB,aAAa,EAAE,mBAAmB,CAAC;IACnC,UAAU,EAAE,UAAU,CAAC;CACxB,CAAC;AAqFF,qBAAa,sBAAsB;IACjC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAsB;IACpD,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAa;IACxC,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAAU;IAC5C,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAU;IACzC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAsB;gBAG9C,SAAS,EAAE,mBAAmB,EAC9B,IAAI,EAAE,mBAAmB,EACzB,SAAS,GAAE;QAAE,iBAAiB,CAAC,EAAE,OAAO,CAAC;QAAC,cAAc,CAAC,EAAE,OAAO,CAAA;KAAO;WAS9D,MAAM,CAAC,SAAS,EAAE,mBAAmB,GAAG,OAAO,CAAC,sBAAsB,CAAC;IAU9E,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;YASd,gBAAgB;YA4BhB,iBAAiB;YAmCjB,cAAc;YAcd,mBAAmB;IAgEjC,OAAO,CAAC,gBAAgB;IA2BxB,OAAO,CAAC,aAAa;YAqBP,iBAAiB;YAyFjB,cAAc;YAkGd,oBAAoB;IAkB5B,QAAQ,CAAC,OAAO,EAAE,2BAA2B,GAAG,OAAO,CAAC,0BAA0B,CAAC;CA4R1F"}