@lorekit/cli 1.62.0 → 1.63.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lorekit/cli",
-  "version": "1.62.0",
+  "version": "1.63.0",
   "description": "Install the LoreKit shared-memory skill and run health checks for the LoreKit MCP server.",
   "license": "MIT",
   "repository": {

package/src/core/lessons.mjs

@@ -193,6 +193,20 @@ export function scopeReadLimit(maxLessons) {
   return Math.min(Math.max(ceiling, SCOPE_READ_LIMIT), MAX_STORE_LIST_LIMIT);
 }
 
+// Is this row fit for the generic SessionStart digest? `kind: 'bus'` /
+// `'signal'` rows (a reviewer's PR-state record, a comment-relevance filter)
+// are loop-owned state with their own dedicated reader — an exact scope+key
+// read or a tag-filtered list, both bypassing this scope-wide, untagged read
+// entirely — not advice for whichever agent next opens a session in that
+// scope. Left in, one renders as a raw JSON blob dressed up as a lesson line
+// (`lessonHook` has no way to know it isn't prose) and the other misstates a
+// state record as considered guidance. `kind` absent or `'lesson'` passes: an
+// explicit kind is opt-in, so a row that never set one — the common case for
+// hand-written and pre-`kind` lessons — must not be silently dropped. Pure.
+function isGeneralLesson(kind) {
+  return kind == null || kind === 'lesson';
+}
+
 // `scope` may be injected instead of derived from `cwd` — a seam for callers
 // that already hold a resolved scope and for tests that need a deterministic
 // branch (deriveScope shells out to git, so the ambient branch — often a
@@ -238,7 +252,7 @@ export async function fetchLessons(
     const raw = Array.isArray(res.entries) ? res.entries : [];
     if (raw.length >= readLimit) truncatedScopes.add(s);
     const entries = raw
-      .filter((e) => e && e.key)
+      .filter((e) => e && e.key && isGeneralLesson(e.kind))
       .map((e) => ({ ...e, scope: s }));
     groups.push({ scope: s, error: null, entries });
   }

package/src/shared/obligations-map.mjs

@@ -179,6 +179,18 @@ export const SURFACE_PARTNER_MAP = [
     guard: null,
     note: 'The flag table and per-command help in bin/lorekit.mjs is the origin of every flag claim; docs/cli.md, the package README (prose AND its flag table) and CLAUDE.md each restate it. Found by running this command against its own changed-set while changing the meaning of --strict: nothing fired, because the map covered every generated surface and no hand-written one. A path proxy, not a content predicate — bin/lorekit.mjs changes for reasons unrelated to flags — so advisory by construction, like error-code-doc.',
   },
+  {
+    id: 'digest-contract-doc',
+    state: 'advisory',
+    owner: '@mthines',
+    added: '2026-09-01',
+    reviewBy: '2026-12-01',
+    match: 'packages/cli/src/core/lessons.mjs',
+    obliges: ['packages/cli/skill/lorekit-setup/rules/ci-state-records.md'],
+    cluster: 'copies-a-claim',
+    guard: null,
+    note: 'lessons.mjs owns the SessionStart digest contract (fetchLessons\' isGeneralLesson kind filter); ci-state-records.md documents that contract for authors of non-lesson records. This is the second instance of the same class in two consecutive changes — the first was cli-flag-doc (bin/lorekit.mjs → the docs copying its flag claims) — which is the recurrence the whole compile pipeline exists to notice. Advisory, no guard: lessons.mjs changes for many reasons unrelated to the digest contract, so this is a path proxy, not a content predicate.',
+  },
   {
     id: 'perf-index',
     state: 'advisory',