@gamaze/hicortex 0.17.5 → 0.17.6

package/dist/type-classify.d.ts

@@ -15,16 +15,20 @@
  * ------------
  * Walks memories ordered by rowid in batches (default 200). Default scope =
  * episodes only (`memory_type = 'episode'`); `--all` reclassifies every memory
- * regardless of current type. For each memory, ONE constrained LLM call asks
- * the model to classify the content as episode / fact / decision. The reply is
- * parsed + validated, and `UPDATE memories SET memory_type = ? WHERE id = ?`
- * runs inside a per-batch transaction. The cursor (`typeCursor` in state.json)
- * advances to the last committed rowid after each batch — crash-safe and
- * infra-abort-safe (same discipline as classify-domains).
+ * regardless of current type **except lessons** (see below). For each memory,
+ * ONE constrained LLM call asks the model to classify the content as
+ * episode / fact / decision. The reply is parsed + validated, and
+ * `UPDATE memories SET memory_type = ? WHERE id = ?` runs inside a per-batch
+ * transaction. The cursor (`typeCursor` in state.json) advances to the last
+ * committed rowid after each batch — crash-safe and infra-abort-safe (same
+ * discipline as classify-domains).
  *
- * Lessons are NEVER produced here: the reflection stage owns them. A model that
- * replies "lesson" is treated as unparseable (the memory keeps its current type
- * and is retried next run via the cursor).
+ * Lessons are NEVER touched here: the reflection stage owns them. The `--all`
+ * scope explicitly excludes `memory_type = 'lesson'` — this is the primary
+ * defence. (The prompt asks only for episode/fact/decision, so a lesson that
+ * DID enter scope would be overwritten to E/F/D — the model never replies
+ * "lesson". `parseTypeReply`'s rejection of a "lesson" reply is a backstop, not
+ * the main guard.)
  *
  * This command does NOT use the consolidation budget — it is a standalone CLI,
  * not a nightly stage.

package/dist/type-classify.js

@@ -16,16 +16,20 @@
  * ------------
  * Walks memories ordered by rowid in batches (default 200). Default scope =
  * episodes only (`memory_type = 'episode'`); `--all` reclassifies every memory
- * regardless of current type. For each memory, ONE constrained LLM call asks
- * the model to classify the content as episode / fact / decision. The reply is
- * parsed + validated, and `UPDATE memories SET memory_type = ? WHERE id = ?`
- * runs inside a per-batch transaction. The cursor (`typeCursor` in state.json)
- * advances to the last committed rowid after each batch — crash-safe and
- * infra-abort-safe (same discipline as classify-domains).
+ * regardless of current type **except lessons** (see below). For each memory,
+ * ONE constrained LLM call asks the model to classify the content as
+ * episode / fact / decision. The reply is parsed + validated, and
+ * `UPDATE memories SET memory_type = ? WHERE id = ?` runs inside a per-batch
+ * transaction. The cursor (`typeCursor` in state.json) advances to the last
+ * committed rowid after each batch — crash-safe and infra-abort-safe (same
+ * discipline as classify-domains).
  *
- * Lessons are NEVER produced here: the reflection stage owns them. A model that
- * replies "lesson" is treated as unparseable (the memory keeps its current type
- * and is retried next run via the cursor).
+ * Lessons are NEVER touched here: the reflection stage owns them. The `--all`
+ * scope explicitly excludes `memory_type = 'lesson'` — this is the primary
+ * defence. (The prompt asks only for episode/fact/decision, so a lesson that
+ * DID enter scope would be overwritten to E/F/D — the model never replies
+ * "lesson". `parseTypeReply`'s rejection of a "lesson" reply is a backstop, not
+ * the main guard.)
  *
  * This command does NOT use the consolidation budget — it is a standalone CLI,
  * not a nightly stage.
@@ -205,8 +209,15 @@ async function runClassifyTypes(options = {}) {
         report.cursor = cursor;
         console.log(`[hicortex] classify-types starting: scope ${all ? "ALL" : "episodes only"}, ` +
             `batch ${batchSize}, cursor ${cursor}${options.reset ? " (reset)" : ""}`);
-        // Scope filter: default = episodes only; --all = everything.
-        const scopeSql = all ? "rowid > ?" : "rowid > ? AND memory_type = 'episode'";
+        // Scope filter: default = episodes only; --all = everything EXCEPT lessons.
+        // Lessons are owned by the reflection stage — they must never be
+        // reclassified here. (The prompt asks only for episode/fact/decision, so a
+        // lesson row in scope gets overwritten: the model never replies "lesson".
+        // The scope exclusion is therefore the real guard; parseTypeReply's
+        // "lesson" rejection is a backstop, not the primary defence.)
+        const scopeSql = all
+            ? "rowid > ? AND (memory_type IS NULL OR memory_type != 'lesson')"
+            : "rowid > ? AND memory_type = 'episode'";
         const batchStmt = db.prepare(`SELECT rowid AS __rowid, id, content, memory_type FROM memories
        WHERE ${scopeSql} ORDER BY rowid ASC LIMIT ?`);
         // Set true when the classifier returns null (infra error): finish the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gamaze/hicortex",
-  "version": "0.17.5",
+  "version": "0.17.6",
   "description": "Self-learning memory for AI agents — experience captured automatically, distilled into lessons overnight, shared across your whole fleet. Works with Hermes, OpenClaw, Claude Code, and Pi.",
   "main": "dist/index.js",
   "bin": {