moflo 4.13.3 → 4.13.5

package/dist/src/cli/services/team-artifact-sync.js

@@ -72,6 +72,7 @@ import { isDurableNamespace } from './cherry-pick-learnings.js';
 import { planReconcile, reconcileId, splitReconcileId, isPrunableTombstone, recordStamp, TOMBSTONE_TTL_MS, } from './durable-reconcile.js';
 import { readDurableSnapshot, applyDurableActions, } from './durable-store-io.js';
 import { detectToolCallMarkup } from '../memory/tool-call-markup.js';
+import { isMisfiledDurable } from './durable-key-rules.js';
 /** Allowed `type` values — the schema CHECK set. An out-of-set value would make
  *  INSERT OR IGNORE silently drop a hand-edited artifact row, so we coerce. */
 const VALID_TYPES = new Set([
@@ -208,8 +209,9 @@ function readArtifact(artifactPath) {
     const lines = new Map();
     const records = new Map();
     let malformed = 0;
+    let misfiled = 0;
     if (!fs.existsSync(artifactPath))
-        return { lines, records, malformed, existed: false };
+        return { lines, records, malformed, misfiled, existed: false };
     const raw = fs.readFileSync(artifactPath, 'utf-8');
     for (const line of raw.split(/\r?\n/)) {
         const trimmed = line.trim();
@@ -222,6 +224,13 @@ function readArtifact(artifactPath) {
         }
         const id = lineId(parsed);
         const record = lineToRecord(parsed);
+        // Dropped at the read both directions share, so import never applies one
+        // and export rewrites the file without it — the artifact shrinks instead of
+        // re-seeding every teammate's `learnings` on their next session (#1495).
+        if (isMisfiledDurable(record.namespace, record.key)) {
+            misfiled++;
+            continue;
+        }
         const existing = records.get(id);
         // Newest wins, ties keep the first. Built on the merge rule's own
         // comparison basis so the two cannot disagree about which of a live line
@@ -231,7 +240,7 @@ function readArtifact(artifactPath) {
         lines.set(id, parsed);
         records.set(id, record);
     }
-    return { lines, records, malformed, existed: true };
+    return { lines, records, malformed, misfiled, existed: true };
 }
 /**
  * Serialise lines to JSONL, sorted by the (namespace, key) each line is about —
@@ -296,8 +305,15 @@ function payloadFromLine(id, line) {
         tags: line.tags ? JSON.stringify(line.tags) : null,
         metadata: JSON.stringify({ provenance: line.provenance, sharedFrom: 'team-artifact' }),
         ownerId: line.provenance?.author || null,
-        // created_at/updated_at are INTEGER NOT NULL — never bind null.
-        createdAt: typeof line.created_at === 'number' ? line.created_at : Date.now(),
+        // created_at/updated_at are INTEGER NOT NULL — never bind null. A line with
+        // no created_at falls back to its edit time, never the import time: stamping
+        // now makes a re-imported old entry read as a new learning, which is what
+        // inflated growth metrics after every re-seed (#1495).
+        createdAt: typeof line.created_at === 'number'
+            ? line.created_at
+            : typeof line.updated_at === 'number'
+                ? line.updated_at
+                : Date.now(),
         // The row gets the best timestamp available even when the RECORD compared
         // as 0 (a pre-#1463 line): 0 governs only who wins the merge, while the row
         // itself should carry the most accurate time we have.
@@ -326,7 +342,7 @@ export function exportTeamArtifact(opts) {
     const parsedSharedAt = Date.parse(opts.sharedAt);
     const now = opts.now ?? (Number.isNaN(parsedSharedAt) ? Date.now() : parsedSharedAt);
     const ttlMs = opts.tombstoneTtlMs ?? TOMBSTONE_TTL_MS;
-    const { lines, records: target, malformed, existed } = readArtifact(opts.artifactPath);
+    const { lines, records: target, malformed, misfiled, existed } = readArtifact(opts.artifactPath);
     const provenance = {
         author: resolveAuthor(projectRoot),
         source: resolveSource(),
@@ -418,7 +434,7 @@ export function exportTeamArtifact(opts) {
     }
     // Skip the write when the merge changed nothing AND the file already exists:
     // a git-tracked artifact should not show up as modified after a no-op run.
-    const changed = actions.length > 0 || prunedTombstones > 0 || backfilled > 0;
+    const changed = actions.length > 0 || prunedTombstones > 0 || backfilled > 0 || misfiled > 0;
     const wrote = changed || !existed;
     if (wrote) {
         fs.mkdirSync(path.dirname(opts.artifactPath), { recursive: true });
@@ -436,6 +452,7 @@ export function exportTeamArtifact(opts) {
         backfilled,
         skippedMalformed: malformed,
         skippedCorrupt,
+        droppedMisfiled: misfiled,
         total: live,
         tombstones,
         wrote,
@@ -467,9 +484,11 @@ export function importTeamArtifact(opts) {
         skippedMalformed: 0,
         skippedNonDurable: 0,
         skippedCorrupt: 0,
+        skippedMisfiled: 0,
     };
-    const { lines, records: parsed, malformed } = readArtifact(opts.artifactPath);
+    const { lines, records: parsed, malformed, misfiled } = readArtifact(opts.artifactPath);
     report.skippedMalformed = malformed;
+    report.skippedMisfiled = misfiled;
     if (lines.size === 0)
         return report;
     const source = new Map();

package/dist/src/cli/version.js

@@ -2,5 +2,5 @@
  * Auto-generated by build. Do not edit manually.
  * Source of truth: root package.json → scripts/sync-version.mjs
  */
-export const VERSION = '4.13.3';
+export const VERSION = '4.13.5';
 //# sourceMappingURL=version.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "moflo",
-  "version": "4.13.3",
+  "version": "4.13.5",
   "description": "MoFlo — AI agent orchestration for Claude Code. A standalone, opinionated toolkit with semantic memory, learned routing, gates, spells, and the /flo issue-execution skill.",
   "main": "dist/src/cli/index.js",
   "type": "module",
@@ -99,7 +99,7 @@
     "@typescript-eslint/parser": "^8.65.0",
     "eslint": "^10.8.0",
     "glob": "^11.1.0",
-    "moflo": "^4.13.2",
+    "moflo": "^4.13.4",
     "tsx": "^4.21.0",
     "typescript": "^5.9.3",
     "vitest": "^4.0.0"