@agent-plan/core 0.2.19-next.6 → 0.2.19-next.8
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/plan-store.d.ts +22 -0
- package/dist/plan-store.d.ts.map +1 -1
- package/dist/plan-store.js +65 -12
- package/package.json +1 -1
package/dist/plan-store.d.ts
CHANGED
|
@@ -9,12 +9,17 @@ export declare function withFeatureLock<T>(featureId: string, fn: () => Promise<
|
|
|
9
9
|
/** Summary of a phase that has a non-empty handoff, for the listHandoffs() API. */
|
|
10
10
|
export interface PhaseHandoffSummary {
|
|
11
11
|
phaseId: string;
|
|
12
|
+
/** Owning feature id (for deep-linking to the phase-detail route). */
|
|
13
|
+
featureId?: string | undefined;
|
|
12
14
|
/** Human-readable composite ref, e.g. `P003` or `P003(F002)`. */
|
|
13
15
|
compositeRef: string;
|
|
14
16
|
/** handoffUpdatedAt (falls back to phase.updatedAt if unset). */
|
|
15
17
|
updatedAt: string;
|
|
16
18
|
/** First non-empty line of the handoff, leading markdown headers stripped, ~80 chars. */
|
|
17
19
|
firstLine: string;
|
|
20
|
+
/** Full handoff content (phase.handoff) — lets the /handoff viewer render
|
|
21
|
+
* inline without a per-phase fetch. */
|
|
22
|
+
content: string;
|
|
18
23
|
}
|
|
19
24
|
export declare function migrateToUuids(store: PlanStore): Promise<void>;
|
|
20
25
|
/**
|
|
@@ -66,6 +71,18 @@ export declare class PlanStore {
|
|
|
66
71
|
ensureStructureOrdering(): Promise<{
|
|
67
72
|
changed: boolean;
|
|
68
73
|
}>;
|
|
74
|
+
/** Rebuild each phase's `tasks` + `taskIds` from the task's OWN `phaseId`
|
|
75
|
+
* (source of truth). Heals plans where tasks got filed into the wrong phase
|
|
76
|
+
* file (e.g. the migrateToGlobalSequence index-mismatch bug, @agent-plan/core
|
|
77
|
+
* <0.2.19-next.7). Deterministic, lossless, idempotent: groups every task by
|
|
78
|
+
* its phaseId, preserves each phase's existing taskIds order, appends orphan
|
|
79
|
+
* tasks (whose phaseId dangles or is empty) by number. Writes a phase file
|
|
80
|
+
* only when its task set actually changed. */
|
|
81
|
+
rebuildContainment(): Promise<{
|
|
82
|
+
changed: number;
|
|
83
|
+
tasks: number;
|
|
84
|
+
orphan: number;
|
|
85
|
+
}>;
|
|
69
86
|
private manifestPath;
|
|
70
87
|
private projectPath;
|
|
71
88
|
private requirementsPath;
|
|
@@ -171,6 +188,11 @@ export declare class PlanStore {
|
|
|
171
188
|
prioritiesAssigned: number;
|
|
172
189
|
duplicateShortIds: string[];
|
|
173
190
|
};
|
|
191
|
+
containment: {
|
|
192
|
+
changed: number;
|
|
193
|
+
tasks: number;
|
|
194
|
+
orphan: number;
|
|
195
|
+
};
|
|
174
196
|
integrity: {
|
|
175
197
|
duplicatePhaseIds: string[];
|
|
176
198
|
danglingPhaseIds: string[];
|
package/dist/plan-store.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"plan-store.d.ts","sourceRoot":"","sources":["../src/plan-store.ts"],"names":[],"mappings":"AAEA,OAAO,EAGL,KAAK,OAAO,EACZ,KAAK,gBAAgB,EAGrB,KAAK,QAAQ,EAEb,KAAK,KAAK,EAEV,KAAK,aAAa,EAElB,KAAK,OAAO,EAEZ,KAAK,oBAAoB,EAIzB,KAAK,aAAa,EAClB,KAAK,WAAW,EAChB,KAAK,eAAe,EACpB,KAAK,WAAW,EACjB,MAAM,aAAa,CAAC;AAOrB,qBAAa,cAAe,SAAQ,KAAK;aAGrB,KAAK,CAAC,EAAE,OAAO;gBAD/B,OAAO,EAAE,MAAM,EACC,KAAK,CAAC,EAAE,OAAO,YAAA;CAKlC;AAeD,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAC,GAAG,SAAS,GAAG,IAAI,CAElF;AAKD,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,CAAC,MAAM,IAAI,CAAC,GAAG,SAAS,GAAG,IAAI,CAEvE;AAaD,wBAAgB,eAAe,CAAC,CAAC,EAAE,SAAS,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC,CAStF;AAmDD,mFAAmF;AACnF,MAAM,WAAW,mBAAmB;IAClC,OAAO,EAAE,MAAM,CAAC;IAChB,iEAAiE;IACjE,YAAY,EAAE,MAAM,CAAC;IACrB,iEAAiE;IACjE,SAAS,EAAE,MAAM,CAAC;IAClB,yFAAyF;IACzF,SAAS,EAAE,MAAM,CAAC;
|
|
1
|
+
{"version":3,"file":"plan-store.d.ts","sourceRoot":"","sources":["../src/plan-store.ts"],"names":[],"mappings":"AAEA,OAAO,EAGL,KAAK,OAAO,EACZ,KAAK,gBAAgB,EAGrB,KAAK,QAAQ,EAEb,KAAK,KAAK,EAEV,KAAK,aAAa,EAElB,KAAK,OAAO,EAEZ,KAAK,oBAAoB,EAIzB,KAAK,aAAa,EAClB,KAAK,WAAW,EAChB,KAAK,eAAe,EACpB,KAAK,WAAW,EACjB,MAAM,aAAa,CAAC;AAOrB,qBAAa,cAAe,SAAQ,KAAK;aAGrB,KAAK,CAAC,EAAE,OAAO;gBAD/B,OAAO,EAAE,MAAM,EACC,KAAK,CAAC,EAAE,OAAO,YAAA;CAKlC;AAeD,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAC,GAAG,SAAS,GAAG,IAAI,CAElF;AAKD,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,CAAC,MAAM,IAAI,CAAC,GAAG,SAAS,GAAG,IAAI,CAEvE;AAaD,wBAAgB,eAAe,CAAC,CAAC,EAAE,SAAS,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC,CAStF;AAmDD,mFAAmF;AACnF,MAAM,WAAW,mBAAmB;IAClC,OAAO,EAAE,MAAM,CAAC;IAChB,sEAAsE;IACtE,SAAS,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC/B,iEAAiE;IACjE,YAAY,EAAE,MAAM,CAAC;IACrB,iEAAiE;IACjE,SAAS,EAAE,MAAM,CAAC;IAClB,yFAAyF;IACzF,SAAS,EAAE,MAAM,CAAC;IAClB;4CACwC;IACxC,OAAO,EAAE,MAAM,CAAC;CACjB;AASD,wBAAsB,cAAc,CAAC,KAAK,EAAE,SAAS,GAAG,OAAO,CAAC,IAAI,CAAC,CAsEpE;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAsB,uBAAuB,CAAC,KAAK,EAAE,SAAS,GAAG,OAAO,CAAC;IAAE,QAAQ,EAAE,OAAO,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,KAAK,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,CAAC,CAyE/I;AA4BD,qBAAa,SAAS;IACpB,SAAgB,IAAI,EAAE,MAAM,CAAC;IAC7B,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,SAAS,CAAS;IAO1B,OAAO,CAAC,eAAe,CAAS;gBAEpB,IAAI,EAAE,MAAM;IAIxB;;4EAEwE;IACxE,cAAc,CAAC,KAAK,EAAE,OAAO,GAAG,IAAI;IAEpC;;;2BAGuB;YACT,UAAU;IAUxB,2EAA2E;IACrE,oBAAoB,CAAC,CAAC,EAAE,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC;IAI/D;;oFAEgF;IAC1E,QAAQ,CAAC,CAAC,EAAE,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC;YAIrC,aAAa;IAK3B,OAAO,CAAC,cAAc;IAMtB,OAAO,CAAC,yBAAyB;IAKjC,OAAO,CAAC,sBAAsB;IAc9B,OAAO,CAAC,0BAA0B;IAyD5B,uBAAuB,IAAI,OAAO,CAAC;QAAE,OAAO,EAAE,OAAO,CAAA;KAAE,CAAC;IAkB9D;;;;;;mDAM+C;IACzC,kBAAkB,IAAI,OAAO,CAAC;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;IAgCvF,OAAO,CAAC,YAAY;IAGpB,OAAO,CAAC,WAAW;IAGnB,OAAO,CAAC,gBAAgB;IAGxB,OAAO,CAAC,YAAY;IAGpB,OAAO,CAAC,WAAW;IAGnB,OAAO,CAAC,WAAW;IAGnB,OAAO,CAAC,gBAAgB;IAMxB;;gFAE4E;YAC9D,aAAa;IAa3B,OAAO,CAAC,SAAS;IAGjB,OAAO,CAAC,SAAS;IAGjB,OAAO,CAAC,YAAY;IAGpB,OAAO,CAAC,YAAY;IAGpB,OAAO,CAAC,UAAU;IAGlB,OAAO,CAAC,YAAY;IAKd,IAAI,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAkGxC,MAAM,IAAI,OAAO,CAAC,OAAO,CAAC;IAW1B,YAAY,IAAI,OAAO,CAAC,QAAQ,CAAC;IAIjC,WAAW,IAAI,OAAO,CAAC,OAAO,CAAC;IAIrC;;;;;;;OAOG;IACG,kBAAkB,IAAI,OAAO,CAAC,MAAM,CAAC;IACrC,gBAAgB,IAAI,OAAO,CAAC,MAAM,CAAC;IACnC,eAAe,IAAI,OAAO,CAAC,MAAM,CAAC;YAC1B,cAAc;IAUtB,SAAS,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,KAAK,CAAC;IAMhD;;0BAEsB;YACR,eAAe;IAqCvB,YAAY,IAAI,OAAO,CAAC,gBAAgB,CAAC;IAOzC,mBAAmB,IAAI,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC;IAQtD,mBAAmB,CAAC,OAAO,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC;IAM5D,UAAU,IAAI,OAAO,CAAC,WAAW,GAAG,IAAI,CAAC;IAQzC,UAAU,CAAC,MAAM,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC;IAiBpD;;;;;OAKG;IACG,oBAAoB,CAAC,eAAe,SAAK,GAAG,OAAO,CAAC,MAAM,CAAC;IAmBjE,qCAAqC;IAC/B,gBAAgB,IAAI,OAAO,CAAC,IAAI,CAAC;IAQvC,kEAAkE;IAC5D,eAAe,IAAI,OAAO,CAAC,OAAO,CAAC;IAQnC,eAAe,IAAI,OAAO,CAAC,WAAW,CAAC;IAQvC,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,aAAa,CAAC;IAaxF,0EAA0E;IACpE,aAAa,CAAC,KAAK,CAAC,EAAE,MAAM,EAAE,kBAAkB,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,CAAC;IAqBhF,gBAAgB,IAAI,OAAO,CAAC,oBAAoB,CAAC;IAQjD,aAAa,IAAI,OAAO,CAAC,KAAK,EAAE,CAAC;IA0BjC,OAAO,IAAI,OAAO,CAAC,aAAa,CAAC;IAYvC;4DACwD;IAClD,eAAe,IAAI,OAAO,CAAC;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,MAAM,CAAA;KAAE,CAAC;IAmEzF;;;;;OAKG;IACG,oBAAoB,IAAI,OAAO,CAAC;QAAE,OAAO,EAAE,MAAM,CAAA;KAAE,CAAC;IA0B1D;mFAC+E;IACzE,gBAAgB,IAAI,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;IAY9C;;;qEAGiE;IAC3D,YAAY,CAAC,IAAI,EAAE,SAAS,GAAG,OAAO,GAAG,MAAM,EAAE,QAAQ,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAmB1F;;sFAEkF;IAC5E,yBAAyB,IAAI,OAAO,CAAC;QACzC,gBAAgB,EAAE,MAAM,CAAC;QACzB,kBAAkB,EAAE,MAAM,CAAC;QAC3B,iBAAiB,EAAE,MAAM,EAAE,CAAC;KAC7B,CAAC;IAuFF,gFAAgF;IAC1E,MAAM,IAAI,OAAO,CAAC;QACtB,QAAQ,EAAE;YAAE,OAAO,EAAE,MAAM,CAAC;YAAC,QAAQ,EAAE,MAAM,CAAC;YAAC,QAAQ,EAAE,MAAM,CAAA;SAAE,CAAC;QAClE,QAAQ,EAAE;YAAE,gBAAgB,EAAE,MAAM,CAAC;YAAC,kBAAkB,EAAE,MAAM,CAAC;YAAC,iBAAiB,EAAE,MAAM,EAAE,CAAA;SAAE,CAAC;QAChG,WAAW,EAAE;YAAE,OAAO,EAAE,MAAM,CAAC;YAAC,KAAK,EAAE,MAAM,CAAC;YAAC,MAAM,EAAE,MAAM,CAAA;SAAE,CAAC;QAChE,SAAS,EAAE;YAAE,iBAAiB,EAAE,MAAM,EAAE,CAAC;YAAC,gBAAgB,EAAE,MAAM,EAAE,CAAC;YAAC,iBAAiB,EAAE,MAAM,EAAE,CAAA;SAAE,CAAC;KACrG,CAAC;IAcF,0FAA0F;IACpF,iBAAiB,IAAI,OAAO,CAAC;QAAE,iBAAiB,EAAE,MAAM,EAAE,CAAC;QAAC,gBAAgB,EAAE,MAAM,EAAE,CAAC;QAAC,iBAAiB,EAAE,MAAM,EAAE,CAAA;KAAE,CAAC;IA4B5H,OAAO,CAAC,iBAAiB;IAoBzB,OAAO,CAAC,mBAAmB;IAoBrB,YAAY,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;IAOvC;;;wFAGoF;IAC9E,oBAAoB,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;IAe7D,aAAa,CAAC,OAAO,EAAE,CAAC,CAAC,EAAE,OAAO,KAAK,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC;IAMjE,cAAc,CAAC,OAAO,EAAE,CAAC,CAAC,EAAE,gBAAgB,KAAK,gBAAgB,GAAG,OAAO,CAAC,gBAAgB,CAAC;IAY7F,kBAAkB,CAAC,OAAO,EAAE,CAAC,CAAC,EAAE,oBAAoB,KAAK,oBAAoB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAM7G,WAAW,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IAO5C,YAAY,CAAC,QAAQ,EAAE,gBAAgB,GAAG,OAAO,CAAC,IAAI,CAAC;IAS7D,kGAAkG;YACpF,eAAe;IAsB7B,oFAAoF;IAC9E,WAAW,CAAC,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IAW5C,gBAAgB,CAAC,IAAI,EAAE,oBAAoB,GAAG,OAAO,CAAC,IAAI,CAAC;IAMzD,SAAS,CAAC,KAAK,EAAE,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC;IAQ9C;;6DAEyD;IACnD,WAAW,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,KAAK,KAAK,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC;IAgBpF,8EAA8E;IACxE,eAAe,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC;IAIvD;8EAC0E;IACpE,eAAe,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAKnE;;wCAEoC;IACpC,OAAO,CAAC,iBAAiB;IAIzB;;kFAE8E;IACxE,eAAe,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAIrD;;;;;;yFAMqF;IAC/E,uBAAuB,IAAI,OAAO,CAAC;QAAE,QAAQ,EAAE,OAAO,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IAyBlF;;;;;wFAKoF;IAC9E,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,MAAM,SAAW,GAAG,OAAO,CAAC,IAAI,CAAC;IAoB1E;sFACkF;IAC5E,YAAY,IAAI,OAAO,CAAC,mBAAmB,EAAE,CAAC;IAsB9C,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAWjD,sFAAsF;IAChF,aAAa,IAAI,OAAO,CAAC,aAAa,CAAC;IAW7C,iEAAiE;IAC3D,cAAc,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;IA0BzC,qDAAqD;YACvC,aAAa;CAS5B"}
|
package/dist/plan-store.js
CHANGED
|
@@ -233,12 +233,21 @@ export async function migrateToGlobalSequence(store) {
|
|
|
233
233
|
const newFeatures = renumber(features);
|
|
234
234
|
const newPhases = renumber(phases);
|
|
235
235
|
const numberedTasks = renumber(allTasks.map((x) => x.task));
|
|
236
|
-
// Reassemble renumbered tasks into their phases,
|
|
236
|
+
// Reassemble renumbered tasks into their phases, keyed by the task's OWN
|
|
237
|
+
// phaseId (source of truth). NOTE: do NOT pair `numberedTasks[i]` with
|
|
238
|
+
// `allTasks[i]!.phase.id` — the two arrays are in DIFFERENT orders
|
|
239
|
+
// (numberedTasks is sorted by createdAt, allTasks is in phase-iteration
|
|
240
|
+
// order), so a positional index would file each task under the wrong phase.
|
|
241
|
+
const phaseIdByTaskId = new Map(allTasks.map((x) => [x.task.id, x.phase.id]));
|
|
242
|
+
const newPhaseIds = new Set(newPhases.map((p) => p.id));
|
|
237
243
|
const tasksByPhase = new Map();
|
|
238
|
-
for (
|
|
239
|
-
|
|
244
|
+
for (const t of numberedTasks) {
|
|
245
|
+
// Prefer the task's own phaseId when it points to a real phase; otherwise
|
|
246
|
+
// fall back to the phase the task was loaded from (handles legacy tasks
|
|
247
|
+
// with empty/stale phaseId without losing them).
|
|
248
|
+
const pid = (t.phaseId && newPhaseIds.has(t.phaseId)) ? t.phaseId : (phaseIdByTaskId.get(t.id) ?? "");
|
|
240
249
|
const bucket = tasksByPhase.get(pid) ?? [];
|
|
241
|
-
bucket.push(
|
|
250
|
+
bucket.push(t);
|
|
242
251
|
tasksByPhase.set(pid, bucket);
|
|
243
252
|
}
|
|
244
253
|
const finalPhases = newPhases.map((p) => {
|
|
@@ -415,6 +424,48 @@ export class PlanStore {
|
|
|
415
424
|
await this.importLegacyHandoffFile().catch(() => { });
|
|
416
425
|
return result;
|
|
417
426
|
}
|
|
427
|
+
/** Rebuild each phase's `tasks` + `taskIds` from the task's OWN `phaseId`
|
|
428
|
+
* (source of truth). Heals plans where tasks got filed into the wrong phase
|
|
429
|
+
* file (e.g. the migrateToGlobalSequence index-mismatch bug, @agent-plan/core
|
|
430
|
+
* <0.2.19-next.7). Deterministic, lossless, idempotent: groups every task by
|
|
431
|
+
* its phaseId, preserves each phase's existing taskIds order, appends orphan
|
|
432
|
+
* tasks (whose phaseId dangles or is empty) by number. Writes a phase file
|
|
433
|
+
* only when its task set actually changed. */
|
|
434
|
+
async rebuildContainment() {
|
|
435
|
+
return this.runAsBatch(async () => {
|
|
436
|
+
const phases = await this.loadAllPhases();
|
|
437
|
+
const phaseById = new Map(phases.map((p) => [p.id, p]));
|
|
438
|
+
const allTasks = [];
|
|
439
|
+
for (const p of phases)
|
|
440
|
+
for (const t of p.tasks)
|
|
441
|
+
allTasks.push({ task: t, fromPhaseId: p.id });
|
|
442
|
+
const grouped = new Map();
|
|
443
|
+
let orphan = 0;
|
|
444
|
+
for (const { task, fromPhaseId } of allTasks) {
|
|
445
|
+
const pid = (task.phaseId && phaseById.has(task.phaseId)) ? task.phaseId : fromPhaseId;
|
|
446
|
+
if (!phaseById.has(pid))
|
|
447
|
+
orphan++;
|
|
448
|
+
const bucket = grouped.get(pid) ?? [];
|
|
449
|
+
bucket.push(task);
|
|
450
|
+
grouped.set(pid, bucket);
|
|
451
|
+
}
|
|
452
|
+
let changed = 0;
|
|
453
|
+
for (const p of phases) {
|
|
454
|
+
const tasks = grouped.get(p.id) ?? [];
|
|
455
|
+
const byId = new Map(tasks.map((t) => [t.id, t]));
|
|
456
|
+
const ordered = p.taskIds.map((id) => byId.get(id)).filter((t) => Boolean(t));
|
|
457
|
+
for (const t of tasks.slice().sort((a, b) => a.number - b.number))
|
|
458
|
+
if (!ordered.some((o) => o.id === t.id))
|
|
459
|
+
ordered.push(t);
|
|
460
|
+
const same = ordered.length === p.tasks.length && ordered.every((t, i) => t.id === p.tasks[i]?.id);
|
|
461
|
+
if (same)
|
|
462
|
+
continue;
|
|
463
|
+
await this.savePhase({ ...p, tasks: ordered, taskIds: ordered.map((t) => t.id) });
|
|
464
|
+
changed++;
|
|
465
|
+
}
|
|
466
|
+
return { changed, tasks: allTasks.length, orphan };
|
|
467
|
+
});
|
|
468
|
+
}
|
|
418
469
|
// ── Path helpers ─────────────────────────────────────────────────────
|
|
419
470
|
manifestPath() {
|
|
420
471
|
return join(this.root, "manifest.json");
|
|
@@ -997,13 +1048,9 @@ export class PlanStore {
|
|
|
997
1048
|
let shortIdsAssigned = 0;
|
|
998
1049
|
let prioritiesAssigned = 0;
|
|
999
1050
|
let featuresDirty = false;
|
|
1000
|
-
|
|
1001
|
-
|
|
1002
|
-
|
|
1003
|
-
return index + 1;
|
|
1004
|
-
}
|
|
1005
|
-
return current;
|
|
1006
|
-
};
|
|
1051
|
+
// Priority is left to reorder (midpoint-insert); ensureShortIds only
|
|
1052
|
+
// backfills shortIds. New items keep priority 0 until first drag reindex.
|
|
1053
|
+
const assignPriority = (current, _index) => current;
|
|
1007
1054
|
// Features: shortId + priority (project scope)
|
|
1008
1055
|
const sortedFeatures = [...featuresDoc.features].sort((a, b) => a.number - b.number || a.createdAt.localeCompare(b.createdAt));
|
|
1009
1056
|
sortedFeatures.forEach((f, index) => {
|
|
@@ -1083,9 +1130,13 @@ export class PlanStore {
|
|
|
1083
1130
|
return this.runAsBatch(async () => {
|
|
1084
1131
|
const migrated = await this.migratePhaseIds();
|
|
1085
1132
|
const backfill = await this.ensureShortIdsAndPriority();
|
|
1133
|
+
// Rebuild phase containment from each task's own phaseId. Heals plans
|
|
1134
|
+
// corrupted by the migrateToGlobalSequence index-mismatch bug (core
|
|
1135
|
+
// <0.2.19-next.7). Lossless + idempotent — safe to run every repair.
|
|
1136
|
+
const containment = await this.rebuildContainment();
|
|
1086
1137
|
const integrity = await this.validateIntegrity();
|
|
1087
1138
|
await this.writeGenerated();
|
|
1088
|
-
return { migrated, backfill, integrity };
|
|
1139
|
+
return { migrated, backfill, containment, integrity };
|
|
1089
1140
|
});
|
|
1090
1141
|
}
|
|
1091
1142
|
/** Validate plan integrity: globally unique phase ids and resolvable feature.phaseIds. */
|
|
@@ -1391,9 +1442,11 @@ export class PlanStore {
|
|
|
1391
1442
|
const fnum = p.featureId ? featureNumber.get(p.featureId) : undefined;
|
|
1392
1443
|
out.push({
|
|
1393
1444
|
phaseId: p.id,
|
|
1445
|
+
featureId: p.featureId,
|
|
1394
1446
|
compositeRef: formatPhaseRef(p.number, fnum),
|
|
1395
1447
|
updatedAt: p.handoffUpdatedAt || p.updatedAt,
|
|
1396
1448
|
firstLine: handoffFirstLine(p.handoff),
|
|
1449
|
+
content: p.handoff,
|
|
1397
1450
|
});
|
|
1398
1451
|
}
|
|
1399
1452
|
out.sort((a, b) => b.updatedAt.localeCompare(a.updatedAt));
|
package/package.json
CHANGED