@kolisachint/hoocode-agent 0.5.16 → 0.5.18
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/CHANGELOG.md +79 -0
- package/dist/core/learn/cache.d.ts +58 -0
- package/dist/core/learn/cache.d.ts.map +1 -0
- package/dist/core/learn/cache.js +120 -0
- package/dist/core/learn/cache.js.map +1 -0
- package/dist/core/learn/coverage.d.ts +58 -0
- package/dist/core/learn/coverage.d.ts.map +1 -0
- package/dist/core/learn/coverage.js +142 -0
- package/dist/core/learn/coverage.js.map +1 -0
- package/dist/core/learn/digest.d.ts +1 -0
- package/dist/core/learn/digest.d.ts.map +1 -1
- package/dist/core/learn/digest.js +31 -4
- package/dist/core/learn/digest.js.map +1 -1
- package/dist/core/learn/extract.d.ts +71 -103
- package/dist/core/learn/extract.d.ts.map +1 -1
- package/dist/core/learn/extract.js +162 -437
- package/dist/core/learn/extract.js.map +1 -1
- package/dist/core/learn/mine.d.ts +123 -0
- package/dist/core/learn/mine.d.ts.map +1 -0
- package/dist/core/learn/mine.js +285 -0
- package/dist/core/learn/mine.js.map +1 -0
- package/dist/core/learn/reduce.d.ts +78 -0
- package/dist/core/learn/reduce.d.ts.map +1 -0
- package/dist/core/learn/reduce.js +123 -0
- package/dist/core/learn/reduce.js.map +1 -0
- package/dist/core/learn/state.d.ts +8 -0
- package/dist/core/learn/state.d.ts.map +1 -1
- package/dist/core/learn/state.js +18 -3
- package/dist/core/learn/state.js.map +1 -1
- package/dist/core/settings-manager.d.ts +2 -0
- package/dist/core/settings-manager.d.ts.map +1 -1
- package/dist/core/settings-manager.js +4 -0
- package/dist/core/settings-manager.js.map +1 -1
- package/dist/core/startup-progress.d.ts +12 -7
- package/dist/core/startup-progress.d.ts.map +1 -1
- package/dist/core/startup-progress.js +12 -7
- package/dist/core/startup-progress.js.map +1 -1
- package/dist/extensions/core/learn.d.ts +8 -4
- package/dist/extensions/core/learn.d.ts.map +1 -1
- package/dist/extensions/core/learn.js +208 -27
- package/dist/extensions/core/learn.js.map +1 -1
- package/dist/modes/interactive/components/footer.d.ts.map +1 -1
- package/dist/modes/interactive/components/footer.js +7 -25
- package/dist/modes/interactive/components/footer.js.map +1 -1
- package/dist/modes/interactive/components/progress-bar.d.ts +50 -0
- package/dist/modes/interactive/components/progress-bar.d.ts.map +1 -0
- package/dist/modes/interactive/components/progress-bar.js +77 -0
- package/dist/modes/interactive/components/progress-bar.js.map +1 -0
- package/dist/modes/interactive/voice/voice-panel.d.ts +6 -1
- package/dist/modes/interactive/voice/voice-panel.d.ts.map +1 -1
- package/dist/modes/interactive/voice/voice-panel.js +18 -14
- package/dist/modes/interactive/voice/voice-panel.js.map +1 -1
- package/examples/extensions/custom-provider-anthropic/package.json +1 -1
- package/examples/extensions/custom-provider-gitlab-duo/package.json +1 -1
- package/examples/extensions/sandbox/package.json +1 -1
- package/examples/extensions/with-deps/package.json +1 -1
- package/package.json +4 -4
- package/dist/core/learn/normalize.d.ts +0 -65
- package/dist/core/learn/normalize.d.ts.map +0 -1
- package/dist/core/learn/normalize.js +0 -245
- package/dist/core/learn/normalize.js.map +0 -1
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"learn.d.ts","sourceRoot":"","sources":["../../../src/extensions/core/learn.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAMH,OAAO,KAAK,EAAE,YAAY,EAA2B,MAAM,gCAAgC,CAAC;AAiO5F,wBAAgB,UAAU,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CAoGjD","sourcesContent":["/**\n * `/learn` — promote what recent sessions actually taught into durable rules\n * and skills.\n *\n * The command is a thin shell on purpose. It runs the deterministic extractor\n * over session transcripts on disk, renders the ranked result, and injects it\n * as a follow-up message; every judgement after that belongs to the model,\n * which can read the repo and phrase a rule far better than a heuristic can.\n *\n * Reading transcripts from disk rather than the live context is what makes this\n * work: the on-disk history survives compaction, and it spans past sessions, so\n * \"you have said this in five separate sessions\" is available as a number\n * instead of a guess. That number is the whole reason the command exists.\n *\n * Follows /grill in modes.ts: no session switch, no mode change, no config\n * write — just a follow-up message. Writes to AGENTS.md happen through ordinary\n * edit tools, so the existing permission prompt is the approval step and no\n * separate picker is needed.\n */\n\nimport { homedir } from \"node:os\";\nimport { join } from \"node:path\";\nimport { CONFIG_DIR_NAME, getHooCodeDir } from \"../../config.js\";\nimport { loadProjectContextFiles } from \"../../core/context-files.js\";\nimport type { ExtensionAPI, ExtensionCommandContext } from \"../../core/extensions/types.js\";\nimport { isEmptyDigest, renderLearnDigest } from \"../../core/learn/digest.js\";\nimport {\n\tbuildCoverageIndex,\n\textractLearnDigest,\n\ttype LearnDigest,\n\tmatchCoverage,\n\ttype SessionScanReport,\n\tscanSessions,\n} from \"../../core/learn/extract.js\";\nimport {\n\tgetLearnStatePath,\n\treadLearnState,\n\trecordSurfaced,\n\tsummarizeLearnState,\n\twriteLearnState,\n} from \"../../core/learn/state.js\";\nimport { getSessionDirPath } from \"../../core/session-manager.js\";\nimport { SettingsManager } from \"../../core/settings-manager.js\";\n\n/** Guards against double-registration when default extensions load more than once. */\nconst REGISTERED = Symbol.for(\"hoocode.learn.registered\");\n\n/** User-scope destination offered for personal rules that travel across repos. */\nconst USER_SCOPE_PATH = join(homedir(), \".agents\", \"AGENTS.md\");\n\n/** Render a home-relative path the way the user would type it. */\nfunction displayPath(path: string): string {\n\tconst home = homedir();\n\treturn path.startsWith(home) ? `~${path.slice(home.length)}` : path;\n}\n\nfunction shortDate(iso: string | undefined): string {\n\tif (!iso) return \"unknown\";\n\tconst date = new Date(iso);\n\treturn Number.isNaN(date.getTime()) ? \"unknown\" : date.toISOString().slice(0, 10);\n}\n\n/** The settings keys `/learn` reads, paired with the values in force right now. */\ntype LearnWindow = ReturnType<SettingsManager[\"getLearnSettings\"]>;\n\nconst SETTING_KEYS: Array<{ key: keyof LearnWindow; setting: string; note: string }> = [\n\t{ key: \"maxSessions\", setting: \"learnMaxSessions\", note: \"recent sessions scanned\" },\n\t{ key: \"maxAgeDays\", setting: \"learnMaxAgeDays\", note: \"ignore sessions older than this, in days\" },\n\t{ key: \"minRepeats\", setting: \"learnMinRepeats\", note: \"times a directive must recur to be proposed\" },\n\t{\n\t\tkey: \"minWorkflowRepeats\",\n\t\tsetting: \"learnMinWorkflowRepeats\",\n\t\tnote: \"repeats before a tool sequence is proposed\",\n\t},\n\t{ key: \"maxProposals\", setting: \"learnMaxProposals\", note: \"cap on each list in the digest\" },\n];\n\n/**\n * Where the knobs live, and what they are set to.\n *\n * `/learn` has five settings and no UI, so until this existed the only way to\n * find them was to already know they were in `settings.json`. Every message that\n * reports a disappointing result names a threshold, so every one of them ends\n * with these lines.\n */\nfunction settingsPathLines(ctx: ExtensionCommandContext, agentDir: string): string[] {\n\treturn [\n\t\t\"Settings — edit either file, no restart needed\",\n\t\t` user ${displayPath(join(agentDir, \"settings.json\"))}`,\n\t\t` project ${displayPath(join(ctx.cwd, CONFIG_DIR_NAME, \"settings.json\"))} (wins where both set a key)`,\n\t];\n}\n\nfunction settingsLines(ctx: ExtensionCommandContext, agentDir: string, window: LearnWindow): string[] {\n\tconst lines = settingsPathLines(ctx, agentDir);\n\tfor (const { key, setting, note } of SETTING_KEYS) {\n\t\tlines.push(` ${setting.padEnd(24)} ${String(window[key]).padStart(3)} ${note}`);\n\t}\n\treturn lines;\n}\n\n/**\n * The directory whose name keys this cwd's bookmark.\n *\n * Derived from the cwd, never from the live session manager. An in-memory\n * session (`--no-session`) reports an empty session directory, which used to key\n * every such run to the same nameless state file, and a shared custom\n * `sessionDir` used to make two unrelated projects share one bookmark. The cwd\n * is what \"per directory\" means here, so the cwd is what it is keyed on.\n */\nfunction stateKeyDir(ctx: ExtensionCommandContext, agentDir: string): string {\n\treturn getSessionDirPath(ctx.cwd, agentDir);\n}\n\n/** Run the directory scan without ranking anything, for the reports that only need counts. */\nfunction sessionScanPreview(ctx: ExtensionCommandContext, agentDir: string, window: LearnWindow): SessionScanReport {\n\treturn scanSessions({\n\t\tcwd: ctx.cwd,\n\t\tagentDir,\n\t\tsessionDir: ctx.sessionManager.getSessionDir(),\n\t\tmaxSessions: window.maxSessions,\n\t\tmaxAgeDays: window.maxAgeDays,\n\t});\n}\n\n/** Where sessions were looked for, and what was passed over — the \"why nothing?\" answer. */\nfunction scanLines(scan: SessionScanReport, window: LearnWindow): string[] {\n\tconst lines: string[] = [\"Looked in\"];\n\tfor (const dir of scan.dirs) {\n\t\tconst missing = scan.missingDirs.includes(dir) ? \" (does not exist)\" : \"\";\n\t\tlines.push(` ${displayPath(dir)}${missing}`);\n\t}\n\tlines.push(`Found ${scan.files} session file(s)`);\n\n\tconst skips: string[] = [];\n\tif (scan.tooOld > 0) skips.push(`${scan.tooOld} older than ${window.maxAgeDays} days (learnMaxAgeDays)`);\n\tif (scan.otherCwd > 0) skips.push(`${scan.otherCwd} recorded a different working directory`);\n\tif (scan.overLimit > 0) skips.push(`${scan.overLimit} beyond the newest ${window.maxSessions} (learnMaxSessions)`);\n\tif (scan.unreadable > 0) skips.push(`${scan.unreadable} empty or unreadable`);\n\tfor (const skip of skips) lines.push(` skipped: ${skip}`);\n\treturn lines;\n}\n\n/**\n * Explain an empty scan rather than asserting there is no history.\n *\n * The old single sentence was wrong as often as it was right: sessions existed,\n * they were simply all outside the window or recorded under another path. Naming\n * the directory searched and the reason each file was passed over turns a dead\n * end into something the reader can fix.\n */\nfunction reportNoSessions(ctx: ExtensionCommandContext, agentDir: string, digest: LearnDigest, window: LearnWindow) {\n\tconst lines: string[] = [];\n\tlines.push(\n\t\tdigest.scan.files === 0\n\t\t\t? \"/learn found no session transcripts for this directory.\"\n\t\t\t: \"/learn found session transcripts, but none inside the current window.\",\n\t);\n\tlines.push(\"\");\n\tlines.push(...scanLines(digest.scan, window));\n\tlines.push(\"\");\n\tlines.push(...settingsLines(ctx, agentDir, window));\n\tctx.ui.notify(lines.join(\"\\n\"), \"warning\");\n}\n\n/**\n * `/learn stats` — what became of past proposals.\n *\n * Reads the state file and recomputes coverage; it does not re-mine sessions,\n * so it is instant and answers a different question than a normal run: not\n * \"what should I write down\" but \"is this command earning its place\".\n */\nfunction reportStats(ctx: ExtensionCommandContext): void {\n\tconst agentDir = getHooCodeDir();\n\tconst window = SettingsManager.create(ctx.cwd, agentDir).getLearnSettings();\n\tconst statePath = getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir));\n\tconst state = readLearnState(statePath);\n\n\tif (Object.keys(state.surfaced).length === 0) {\n\t\t// Nothing on record means `/learn` has never proposed anything here — which\n\t\t// is as likely to be \"it never found any sessions\" as \"you never ran it\", so\n\t\t// point at both the sessions it can see and the knobs that gate them.\n\t\tconst lines = [\"No /learn history for this directory yet — nothing has been proposed here.\"];\n\t\tlines.push(` State file ${displayPath(statePath)} (not created yet)`);\n\t\tlines.push(\"\");\n\t\tlines.push(...scanLines(sessionScanPreview(ctx, agentDir, window), window));\n\t\tlines.push(\"\");\n\t\tlines.push(...settingsPathLines(ctx, agentDir));\n\t\tlines.push(\" Run /learn settings for the thresholds in force.\");\n\t\tctx.ui.notify(lines.join(\"\\n\"), \"info\");\n\t\treturn;\n\t}\n\n\tconst coverage = buildCoverageIndex({ cwd: ctx.cwd, agentDir });\n\tconst stats = summarizeLearnState(state, (normalized) => {\n\t\tconst match = matchCoverage(normalized, coverage);\n\t\treturn !!(match.rule || match.skill);\n\t});\n\n\tconst contextTokens = loadProjectContextFiles({ cwd: ctx.cwd, agentDir }).agentsFiles.reduce(\n\t\t(sum, file) => sum + (file.tokens ?? 0),\n\t\t0,\n\t);\n\n\tconst lines: string[] = [];\n\tlines.push(`/learn history for this directory — ${shortDate(stats.earliest)} to ${shortDate(stats.latest)}`);\n\tlines.push(\n\t\t` Proposals shown ${stats.total} (${stats.directives} directive, ${stats.fixes} fix, ${stats.workflows} workflow)`,\n\t);\n\tif (stats.lastRun) lines.push(` Last run ${shortDate(stats.lastRun)}`);\n\tlines.push(\"\");\n\n\tif (stats.open === 0) {\n\t\tlines.push(\"No directive proposals yet, so there is nothing to measure adoption against.\");\n\t} else {\n\t\tconst rate = Math.round((stats.adopted / stats.open) * 100);\n\t\tlines.push(\"Directive adoption — the only category with a coverage signal\");\n\t\tlines.push(` Written down ${stats.adopted} of ${stats.open} (${rate}%)`);\n\t\tlines.push(` Passed over ${stats.declined}`);\n\t\tlines.push(\"\");\n\t\t// Without this the number invites the wrong conclusion. Adoption is a proxy\n\t\t// for usefulness, and a proposal correctly rejected as not durable counts\n\t\t// against it exactly like a junk one — so near-100% means the bar is too\n\t\t// low, not that the extractor is perfect.\n\t\tlines.push(\" A very high rate means the bar is too low, not that every proposal was good.\");\n\t\tlines.push(\" Near zero means the extractor is proposing the wrong things.\");\n\t}\n\n\tlines.push(\"\");\n\tlines.push(`Context files ~${contextTokens} tokens, re-sent every request`);\n\tlines.push(`State file ${displayPath(statePath)}`);\n\tlines.push(\"\");\n\tlines.push(...settingsPathLines(ctx, agentDir));\n\tlines.push(\" Run /learn settings for the thresholds in force.\");\n\n\tctx.ui.notify(lines.join(\"\\n\"), \"info\");\n}\n\n/** `/learn settings` — the knobs, their current values, and the files to set them in. */\nfunction reportSettings(ctx: ExtensionCommandContext): void {\n\tconst agentDir = getHooCodeDir();\n\tconst window = SettingsManager.create(ctx.cwd, agentDir).getLearnSettings();\n\tconst lines = settingsLines(ctx, agentDir, window);\n\tlines.push(\"\");\n\tlines.push(...scanLines(sessionScanPreview(ctx, agentDir, window), window));\n\tlines.push(`State file ${displayPath(getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir)))}`);\n\tctx.ui.notify(lines.join(\"\\n\"), \"info\");\n}\n\nexport function setupLearn(pi: ExtensionAPI): void {\n\tconst guarded = pi as unknown as Record<symbol, boolean>;\n\tif (guarded[REGISTERED]) return;\n\tguarded[REGISTERED] = true;\n\n\tpi.registerCommand(\"learn\", {\n\t\tdescription: \"Mine recent sessions for durable rules and skills. Usage: /learn [all|stats|settings]\",\n\t\tgetArgumentCompletions: (prefix: string) =>\n\t\t\t(\n\t\t\t\t[\n\t\t\t\t\t{ value: \"all\", label: \"re-propose everything\" },\n\t\t\t\t\t{ value: \"stats\", label: \"what happened to past proposals\" },\n\t\t\t\t\t{ value: \"settings\", label: \"where sessions are read from, and the knobs\" },\n\t\t\t\t] as const\n\t\t\t)\n\t\t\t\t.filter((option) => option.value.startsWith(prefix))\n\t\t\t\t.map((option) => ({ value: option.value, label: option.label })),\n\t\thandler: async (args: string, ctx: ExtensionCommandContext): Promise<void> => {\n\t\t\tconst argument = args.trim().toLowerCase();\n\t\t\tif (argument && argument !== \"all\" && argument !== \"stats\" && argument !== \"settings\") {\n\t\t\t\tctx.ui.notify(\"Usage: /learn [all|stats|settings]\", \"warning\");\n\t\t\t\treturn;\n\t\t\t}\n\t\t\tif (argument === \"stats\") {\n\t\t\t\treportStats(ctx);\n\t\t\t\treturn;\n\t\t\t}\n\t\t\tif (argument === \"settings\") {\n\t\t\t\treportSettings(ctx);\n\t\t\t\treturn;\n\t\t\t}\n\t\t\tconst ignoreState = argument === \"all\";\n\n\t\t\t// Read per-invocation so a settings edit takes effect without a reload,\n\t\t\t// and so a project settings.json can narrow the window for one repo.\n\t\t\tconst agentDir = getHooCodeDir();\n\t\t\tconst window = SettingsManager.create(ctx.cwd, agentDir).getLearnSettings();\n\t\t\tconst statePath = getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir));\n\n\t\t\tlet digest: LearnDigest;\n\t\t\ttry {\n\t\t\t\tdigest = extractLearnDigest({\n\t\t\t\t\tcwd: ctx.cwd,\n\t\t\t\t\tagentDir,\n\t\t\t\t\t// Searched in addition to the per-cwd default directory, so a session\n\t\t\t\t\t// manager pointing elsewhere (`--session`, a custom `sessionDir`, or\n\t\t\t\t\t// an in-memory session reporting none at all) cannot hide the history.\n\t\t\t\t\tsessionDir: ctx.sessionManager.getSessionDir(),\n\t\t\t\t\tmaxSessions: window.maxSessions,\n\t\t\t\t\tmaxAgeDays: window.maxAgeDays,\n\t\t\t\t\tminRepeats: window.minRepeats,\n\t\t\t\t\tminWorkflowRepeats: window.minWorkflowRepeats,\n\t\t\t\t\tmaxProposals: window.maxProposals,\n\t\t\t\t\tstate: readLearnState(statePath),\n\t\t\t\t\tignoreState,\n\t\t\t\t});\n\t\t\t} catch (error) {\n\t\t\t\tctx.ui.notify(`/learn could not read session history: ${error}`, \"error\");\n\t\t\t\treturn;\n\t\t\t}\n\n\t\t\tif (digest.scannedSessions === 0) {\n\t\t\t\treportNoSessions(ctx, agentDir, digest, window);\n\t\t\t\treturn;\n\t\t\t}\n\n\t\t\tif (isEmptyDigest(digest)) {\n\t\t\t\tconst lines: string[] = [];\n\t\t\t\tlines.push(\n\t\t\t\t\tdigest.suppressed > 0\n\t\t\t\t\t\t? `Scanned ${digest.scannedSessions} session(s) — nothing new since last time (${digest.suppressed} already shown). Run /learn all to see them again.`\n\t\t\t\t\t\t: `Scanned ${digest.scannedSessions} session(s) — nothing repeated often enough to be worth a rule yet.`,\n\t\t\t\t);\n\t\t\t\tif (digest.suppressed === 0) {\n\t\t\t\t\t// The thresholds are the reason a scan with real sessions in it came\n\t\t\t\t\t// back empty, so this is the moment they are worth knowing about.\n\t\t\t\t\tlines.push(\"\");\n\t\t\t\t\tlines.push(...settingsLines(ctx, agentDir, window));\n\t\t\t\t}\n\t\t\t\tctx.ui.notify(lines.join(\"\\n\"), \"info\");\n\t\t\t\treturn;\n\t\t\t}\n\n\t\t\tconst counts = [\n\t\t\t\tdigest.directives.length > 0 ? `${digest.directives.length} directive(s)` : undefined,\n\t\t\t\tdigest.fixes.length > 0 ? `${digest.fixes.length} fix(es)` : undefined,\n\t\t\t\tdigest.workflows.length > 0 ? `${digest.workflows.length} workflow(s)` : undefined,\n\t\t\t].filter((part): part is string => !!part);\n\t\t\tconst held = digest.suppressed > 0 ? `, ${digest.suppressed} held back` : \"\";\n\t\t\tctx.ui.notify(`Mined ${digest.scannedSessions} session(s): ${counts.join(\", \")}${held}.`, \"info\");\n\n\t\t\t// Record before delivering: what matters is that these were put in front\n\t\t\t// of the user, which is true whether or not they act on the digest.\n\t\t\twriteLearnState(statePath, recordSurfaced(readLearnState(statePath), digest.surfaced));\n\n\t\t\tpi.sendUserMessage(renderLearnDigest(digest, { userScopePath: displayPath(USER_SCOPE_PATH) }), {\n\t\t\t\tdeliverAs: \"followUp\",\n\t\t\t});\n\t\t},\n\t});\n}\n"]}
|
|
1
|
+
{"version":3,"file":"learn.d.ts","sourceRoot":"","sources":["../../../src/extensions/core/learn.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAOH,OAAO,KAAK,EAAE,YAAY,EAA2B,MAAM,gCAAgC,CAAC;AA8W5F,wBAAgB,UAAU,CAAC,EAAE,EAAE,YAAY,GAAG,IAAI,CAiLjD","sourcesContent":["/**\n * `/learn` — promote what recent sessions actually taught into durable rules\n * and skills.\n *\n * The command is a thin shell on purpose. It runs the mining pipeline over\n * session transcripts on disk, renders the ranked result, and injects it as a\n * follow-up message; every judgement after that belongs to the model, which can\n * read the repo and phrase a rule far better than a heuristic can.\n *\n * Reading transcripts from disk rather than the live context is what makes this\n * work: the on-disk history survives compaction, and it spans past sessions, so\n * \"you have said this in five separate sessions\" is available as a number\n * instead of a guess. That number is the whole reason the command exists.\n *\n * The pipeline reads every transcript with a model rather than pre-filtering\n * with regexes, which costs real tokens on a cold cache. That price is stated\n * before it is paid, never inferred: a run with sessions to read asks first.\n *\n * Follows /grill in modes.ts: no session switch, no mode change, no config\n * write — just a follow-up message. Writes to AGENTS.md happen through ordinary\n * edit tools, so the existing permission prompt is the approval step and no\n * separate picker is needed.\n */\n\nimport { homedir } from \"node:os\";\nimport { join } from \"node:path\";\nimport type { Api, Model } from \"@kolisachint/hoocode-ai\";\nimport { CONFIG_DIR_NAME, getHooCodeDir } from \"../../config.js\";\nimport { loadProjectContextFiles } from \"../../core/context-files.js\";\nimport type { ExtensionAPI, ExtensionCommandContext } from \"../../core/extensions/types.js\";\nimport type { CoverageJudge } from \"../../core/learn/coverage.js\";\nimport { createLlmCoverageJudge } from \"../../core/learn/coverage.js\";\nimport { isEmptyDigest, renderLearnDigest } from \"../../core/learn/digest.js\";\nimport {\n\tbuildCoverageIndex,\n\ttype LearnDigest,\n\tmineLearnDigest,\n\tplanMining,\n\ttype SessionScanReport,\n\tscanSessions,\n} from \"../../core/learn/extract.js\";\nimport type { Miner } from \"../../core/learn/mine.js\";\nimport { chunkCharsForModel, createLlmMiner } from \"../../core/learn/mine.js\";\nimport {\n\tgetLearnStatePath,\n\treadLearnState,\n\trecordSurfaced,\n\tsummarizeLearnState,\n\twriteLearnState,\n} from \"../../core/learn/state.js\";\nimport { resolveModelCategory } from \"../../core/model-categories.js\";\nimport { getSessionDirPath } from \"../../core/session-manager.js\";\nimport { SettingsManager } from \"../../core/settings-manager.js\";\nimport { startupProgress } from \"../../core/startup-progress.js\";\n\n/** Guards against double-registration when default extensions load more than once. */\nconst REGISTERED = Symbol.for(\"hoocode.learn.registered\");\n\n/** User-scope destination offered for personal rules that travel across repos. */\nconst USER_SCOPE_PATH = join(homedir(), \".agents\", \"AGENTS.md\");\n\n/** Footer key for the mining progress bar. */\nconst PROGRESS_KEY = \"learn-mining\";\n\n/** Escape, the way a raw terminal delivers it. */\nconst ESCAPE = \"\\x1b\";\n\n/**\n * Sessions that can be read without asking first.\n *\n * A run that has one or two new transcripts to read is the normal daily case\n * and interrupting it to confirm a trivial cost is noise. Beyond this the run\n * is a backfill — onboarding to an existing repo, or a first run — and the\n * reader should get to decide before it starts.\n */\nconst CONFIRM_ABOVE_PENDING = 3;\n\n/** Render a home-relative path the way the user would type it. */\nfunction displayPath(path: string): string {\n\tconst home = homedir();\n\treturn path.startsWith(home) ? `~${path.slice(home.length)}` : path;\n}\n\nfunction shortDate(iso: string | undefined): string {\n\tif (!iso) return \"unknown\";\n\tconst date = new Date(iso);\n\treturn Number.isNaN(date.getTime()) ? \"unknown\" : date.toISOString().slice(0, 10);\n}\n\n/** The settings keys `/learn` reads, paired with the values in force right now. */\ntype LearnWindow = ReturnType<SettingsManager[\"getLearnSettings\"]>;\n\nconst SETTING_KEYS: Array<{ key: keyof LearnWindow; setting: string; note: string }> = [\n\t{ key: \"maxSessions\", setting: \"learnMaxSessions\", note: \"recent sessions scanned\" },\n\t{ key: \"maxAgeDays\", setting: \"learnMaxAgeDays\", note: \"ignore sessions older than this, in days\" },\n\t{ key: \"minRepeats\", setting: \"learnMinRepeats\", note: \"times a directive must recur to be proposed\" },\n\t{\n\t\tkey: \"minWorkflowRepeats\",\n\t\tsetting: \"learnMinWorkflowRepeats\",\n\t\tnote: \"repeats before a tool sequence is proposed\",\n\t},\n\t{ key: \"maxProposals\", setting: \"learnMaxProposals\", note: \"cap on each list in the digest\" },\n];\n\n/**\n * Where the knobs live, and what they are set to.\n *\n * `/learn` has its settings and no UI, so until this existed the only way to\n * find them was to already know they were in `settings.json`. Every message that\n * reports a disappointing result names a threshold, so every one of them ends\n * with these lines.\n */\nfunction settingsPathLines(ctx: ExtensionCommandContext, agentDir: string): string[] {\n\treturn [\n\t\t\"Settings — edit either file, no restart needed\",\n\t\t` user ${displayPath(join(agentDir, \"settings.json\"))}`,\n\t\t` project ${displayPath(join(ctx.cwd, CONFIG_DIR_NAME, \"settings.json\"))} (wins where both set a key)`,\n\t];\n}\n\nfunction settingsLines(ctx: ExtensionCommandContext, agentDir: string, window: LearnWindow): string[] {\n\tconst lines = settingsPathLines(ctx, agentDir);\n\tfor (const { key, setting, note } of SETTING_KEYS) {\n\t\tlines.push(` ${setting.padEnd(24)} ${String(window[key] ?? \"—\").padStart(3)} ${note}`);\n\t}\n\treturn lines;\n}\n\n/**\n * The directory whose name keys this cwd's bookmark.\n *\n * Derived from the cwd, never from the live session manager. An in-memory\n * session (`--no-session`) reports an empty session directory, which used to key\n * every such run to the same nameless state file, and a shared custom\n * `sessionDir` used to make two unrelated projects share one bookmark. The cwd\n * is what \"per directory\" means here, so the cwd is what it is keyed on.\n */\nfunction stateKeyDir(ctx: ExtensionCommandContext, agentDir: string): string {\n\treturn getSessionDirPath(ctx.cwd, agentDir);\n}\n\n/** Run the directory scan without mining anything, for the reports that only need counts. */\nfunction sessionScanPreview(ctx: ExtensionCommandContext, agentDir: string, window: LearnWindow): SessionScanReport {\n\treturn scanSessions({\n\t\tcwd: ctx.cwd,\n\t\tagentDir,\n\t\tsessionDir: ctx.sessionManager.getSessionDir(),\n\t\tmaxSessions: window.maxSessions,\n\t\tmaxAgeDays: window.maxAgeDays,\n\t});\n}\n\n/** Where sessions were looked for, and what was passed over — the \"why nothing?\" answer. */\nfunction scanLines(scan: SessionScanReport, window: LearnWindow): string[] {\n\tconst lines: string[] = [\"Looked in\"];\n\tfor (const dir of scan.dirs) {\n\t\tconst missing = scan.missingDirs.includes(dir) ? \" (does not exist)\" : \"\";\n\t\tlines.push(` ${displayPath(dir)}${missing}`);\n\t}\n\tlines.push(`Found ${scan.files} session file(s)`);\n\n\tconst skips: string[] = [];\n\tif (scan.tooOld > 0) skips.push(`${scan.tooOld} older than ${window.maxAgeDays} days (learnMaxAgeDays)`);\n\tif (scan.otherCwd > 0) skips.push(`${scan.otherCwd} recorded a different working directory`);\n\tif (scan.overLimit > 0) skips.push(`${scan.overLimit} beyond the newest ${window.maxSessions} (learnMaxSessions)`);\n\tif (scan.unreadable > 0) skips.push(`${scan.unreadable} empty or unreadable`);\n\tfor (const skip of skips) lines.push(` skipped: ${skip}`);\n\treturn lines;\n}\n\n/**\n * Explain an empty scan rather than asserting there is no history.\n *\n * The old single sentence was wrong as often as it was right: sessions existed,\n * they were simply all outside the window or recorded under another path. Naming\n * the directory searched and the reason each file was passed over turns a dead\n * end into something the reader can fix.\n */\nfunction reportNoSessions(ctx: ExtensionCommandContext, agentDir: string, digest: LearnDigest, window: LearnWindow) {\n\tconst lines: string[] = [];\n\tlines.push(\n\t\tdigest.scan.files === 0\n\t\t\t? \"/learn found no session transcripts for this directory.\"\n\t\t\t: \"/learn found session transcripts, but none inside the current window.\",\n\t);\n\tlines.push(\"\");\n\tlines.push(...scanLines(digest.scan, window));\n\tlines.push(\"\");\n\tlines.push(...settingsLines(ctx, agentDir, window));\n\tctx.ui.notify(lines.join(\"\\n\"), \"warning\");\n}\n\n/**\n * The model that reads transcripts.\n *\n * This is the one call in the pipeline that reads *everything*, so it wants the\n * cheapest capable model rather than the session's. That question already has an\n * answer in this codebase — the `fast` model category, which subagents use for\n * exactly this kind of bulk read — so it is reused rather than reinvented.\n * `settings.modelCategories.fast` wins when set; otherwise the tier is derived\n * from the user's available models, and nothing here is provider-specific.\n *\n * Falls back to the session model when the tier resolves to nothing or to a\n * model the registry cannot find, since a mis-set tier should not take the\n * command out entirely.\n */\nfunction resolveMinerModel(ctx: ExtensionCommandContext, settings: SettingsManager): Model<Api> | undefined {\n\tconst ref = resolveModelCategory(\n\t\t\"fast\",\n\t\t{\n\t\t\tmodelCategories: settings.getModelCategories(),\n\t\t\tdefaultProvider: settings.getDefaultProvider(),\n\t\t\tdefaultModel: settings.getDefaultModel(),\n\t\t},\n\t\tctx.modelRegistry.getAvailable(),\n\t);\n\tif (!ref) return ctx.model;\n\n\tconst slash = ref.indexOf(\"/\");\n\tconst found = slash > 0 ? ctx.modelRegistry.find(ref.slice(0, slash), ref.slice(slash + 1)) : undefined;\n\treturn found ?? ctx.model;\n}\n\n/** Build the two model-backed stages, or report why they cannot be built. */\nasync function buildPipeline(\n\tctx: ExtensionCommandContext,\n\tsettings: SettingsManager,\n): Promise<{ miner: Miner; coverageJudge: CoverageJudge; model: Model<Api> } | { error: string }> {\n\tconst model = resolveMinerModel(ctx, settings);\n\tif (!model) {\n\t\treturn {\n\t\t\terror: \"/learn reads session transcripts with a model, and no model is selected. Pick one with /model, then run /learn again.\",\n\t\t};\n\t}\n\n\tconst auth = await ctx.modelRegistry.getApiKeyAndHeaders(model);\n\tif (!auth.ok) {\n\t\treturn { error: `/learn could not authenticate ${model.provider}/${model.id}: ${auth.error}` };\n\t}\n\n\tconst deps = { model, apiKey: auth.apiKey, headers: auth.headers };\n\treturn { miner: createLlmMiner(deps), coverageJudge: createLlmCoverageJudge(deps), model };\n}\n\n/**\n * What this run still owes the model.\n *\n * Delegated to `planMining` so the number quoted by the confirmation prompt\n * comes from the same session selection the run will use — same window, same\n * cwd check, same de-duplication.\n */\nfunction pendingWork(ctx: ExtensionCommandContext, agentDir: string, window: LearnWindow) {\n\treturn planMining({\n\t\tcwd: ctx.cwd,\n\t\tagentDir,\n\t\tsessionDir: ctx.sessionManager.getSessionDir(),\n\t\tmaxSessions: window.maxSessions,\n\t\tmaxAgeDays: window.maxAgeDays,\n\t});\n}\n\n/**\n * `/learn stats` — what became of past proposals.\n *\n * Reads the state file and recomputes coverage; it does not re-mine sessions,\n * so it costs one small model call and answers a different question than a\n * normal run: not \"what should I write down\" but \"is this command earning its\n * place\".\n */\nasync function reportStats(ctx: ExtensionCommandContext): Promise<void> {\n\tconst agentDir = getHooCodeDir();\n\tconst settings = SettingsManager.create(ctx.cwd, agentDir);\n\tconst window = settings.getLearnSettings();\n\tconst statePath = getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir));\n\tconst state = readLearnState(statePath);\n\n\tif (Object.keys(state.surfaced).length === 0) {\n\t\t// Nothing on record means `/learn` has never proposed anything here — which\n\t\t// is as likely to be \"it never found any sessions\" as \"you never ran it\", so\n\t\t// point at both the sessions it can see and the knobs that gate them.\n\t\tconst lines = [\"No /learn history for this directory yet — nothing has been proposed here.\"];\n\t\tlines.push(` State file ${displayPath(statePath)} (not created yet)`);\n\t\tlines.push(\"\");\n\t\tlines.push(...scanLines(sessionScanPreview(ctx, agentDir, window), window));\n\t\tlines.push(\"\");\n\t\tlines.push(...settingsPathLines(ctx, agentDir));\n\t\tlines.push(\" Run /learn settings for the thresholds in force.\");\n\t\tctx.ui.notify(lines.join(\"\\n\"), \"info\");\n\t\treturn;\n\t}\n\n\tconst coverage = buildCoverageIndex({ cwd: ctx.cwd, agentDir });\n\n\t// Coverage is a model judgement now, so it is resolved once for every stored\n\t// label in a single batch and then answered from the resulting set. Asking\n\t// per item would be one call per proposal ever made in this directory.\n\tconst labels = Object.entries(state.surfaced)\n\t\t.filter(([key]) => key.startsWith(\"directive:\"))\n\t\t// Entries recorded before the wording was kept fall back to the slug, which\n\t\t// is a weaker question but still a answerable one.\n\t\t.map(([key, item]) => ({\n\t\t\tlabel: key.slice(\"directive:\".length),\n\t\t\ttext: item.text || key.slice(\"directive:\".length),\n\t\t}));\n\n\tconst covered = new Set<string>();\n\tif (labels.length > 0) {\n\t\tconst pipeline = await buildPipeline(ctx, settings);\n\t\tif (!(\"error\" in pipeline)) {\n\t\t\ttry {\n\t\t\t\tconst verdicts = await pipeline.coverageJudge(labels, coverage);\n\t\t\t\tfor (const [label, match] of verdicts) {\n\t\t\t\t\tif (match.rule || match.skill) covered.add(label);\n\t\t\t\t}\n\t\t\t} catch {\n\t\t\t\t// Adoption is reported as zero rather than failing the report; the\n\t\t\t\t// caveat below already tells the reader not to over-read the number.\n\t\t\t}\n\t\t}\n\t}\n\n\tconst stats = summarizeLearnState(state, (label) => covered.has(label));\n\n\tconst contextTokens = loadProjectContextFiles({ cwd: ctx.cwd, agentDir }).agentsFiles.reduce(\n\t\t(sum, file) => sum + (file.tokens ?? 0),\n\t\t0,\n\t);\n\n\tconst lines: string[] = [];\n\tlines.push(`/learn history for this directory — ${shortDate(stats.earliest)} to ${shortDate(stats.latest)}`);\n\tlines.push(\n\t\t` Proposals shown ${stats.total} (${stats.directives} directive, ${stats.fixes} fix, ${stats.workflows} workflow)`,\n\t);\n\tif (stats.lastRun) lines.push(` Last run ${shortDate(stats.lastRun)}`);\n\tlines.push(\"\");\n\n\tif (stats.open === 0) {\n\t\tlines.push(\"No directive proposals yet, so there is nothing to measure adoption against.\");\n\t} else {\n\t\tconst rate = Math.round((stats.adopted / stats.open) * 100);\n\t\tlines.push(\"Directive adoption — the only category with a coverage signal\");\n\t\tlines.push(` Written down ${stats.adopted} of ${stats.open} (${rate}%)`);\n\t\tlines.push(` Passed over ${stats.declined}`);\n\t\tlines.push(\"\");\n\t\t// Without this the number invites the wrong conclusion. Adoption is a proxy\n\t\t// for usefulness, and a proposal correctly rejected as not durable counts\n\t\t// against it exactly like a junk one — so near-100% means the bar is too\n\t\t// low, not that the extractor is perfect.\n\t\tlines.push(\" A very high rate means the bar is too low, not that every proposal was good.\");\n\t\tlines.push(\" Near zero means the miner is proposing the wrong things.\");\n\t}\n\n\tlines.push(\"\");\n\tlines.push(`Context files ~${contextTokens} tokens, re-sent every request`);\n\tlines.push(`State file ${displayPath(statePath)}`);\n\tlines.push(\"\");\n\tlines.push(...settingsPathLines(ctx, agentDir));\n\tlines.push(\" Run /learn settings for the thresholds in force.\");\n\n\tctx.ui.notify(lines.join(\"\\n\"), \"info\");\n}\n\n/** `/learn settings` — the knobs, their current values, and the files to set them in. */\nfunction reportSettings(ctx: ExtensionCommandContext): void {\n\tconst agentDir = getHooCodeDir();\n\tconst settings = SettingsManager.create(ctx.cwd, agentDir);\n\tconst window = settings.getLearnSettings();\n\tconst lines = settingsLines(ctx, agentDir, window);\n\n\t// The reading model is not a `/learn` setting — it is the shared `fast` tier,\n\t// so name it here rather than leaving the reader to guess which model is\n\t// about to read their history, and point at the setting that changes it.\n\tconst model = resolveMinerModel(ctx, settings);\n\tlines.push(\n\t\t` reads transcripts with ${model ? `${model.provider}/${model.id}` : \"no model selected\"}` +\n\t\t\t` (the \\`fast\\` tier — set modelCategories.fast to change it)`,\n\t);\n\tif (model) {\n\t\tlines.push(\n\t\t\t` ${Math.round(chunkCharsForModel(model) / 1000)}k characters per call, from its ${model.contextWindow} token window`,\n\t\t);\n\t}\n\n\tlines.push(\"\");\n\tlines.push(...scanLines(sessionScanPreview(ctx, agentDir, window), window));\n\tlines.push(`State file ${displayPath(getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir)))}`);\n\tconst { pending } = pendingWork(ctx, agentDir, window);\n\tlines.push(\n\t\tpending === 0\n\t\t\t? \"All sessions in the window are already mined; the next /learn costs one small coverage call.\"\n\t\t\t: `${pending} session(s) in the window still need reading, roughly one call each.`,\n\t);\n\tctx.ui.notify(lines.join(\"\\n\"), \"info\");\n}\n\nexport function setupLearn(pi: ExtensionAPI): void {\n\tconst guarded = pi as unknown as Record<symbol, boolean>;\n\tif (guarded[REGISTERED]) return;\n\tguarded[REGISTERED] = true;\n\n\tpi.registerCommand(\"learn\", {\n\t\tdescription: \"Mine recent sessions for durable rules and skills. Usage: /learn [all|stats|settings]\",\n\t\tgetArgumentCompletions: (prefix: string) =>\n\t\t\t(\n\t\t\t\t[\n\t\t\t\t\t{ value: \"all\", label: \"re-propose everything\" },\n\t\t\t\t\t{ value: \"stats\", label: \"what happened to past proposals\" },\n\t\t\t\t\t{ value: \"settings\", label: \"where sessions are read from, and the knobs\" },\n\t\t\t\t] as const\n\t\t\t)\n\t\t\t\t.filter((option) => option.value.startsWith(prefix))\n\t\t\t\t.map((option) => ({ value: option.value, label: option.label })),\n\t\thandler: async (args: string, ctx: ExtensionCommandContext): Promise<void> => {\n\t\t\tconst argument = args.trim().toLowerCase();\n\t\t\tif (argument && argument !== \"all\" && argument !== \"stats\" && argument !== \"settings\") {\n\t\t\t\tctx.ui.notify(\"Usage: /learn [all|stats|settings]\", \"warning\");\n\t\t\t\treturn;\n\t\t\t}\n\t\t\tif (argument === \"stats\") {\n\t\t\t\tawait reportStats(ctx);\n\t\t\t\treturn;\n\t\t\t}\n\t\t\tif (argument === \"settings\") {\n\t\t\t\treportSettings(ctx);\n\t\t\t\treturn;\n\t\t\t}\n\t\t\tconst ignoreState = argument === \"all\";\n\n\t\t\t// Read per-invocation so a settings edit takes effect without a reload,\n\t\t\t// and so a project settings.json can narrow the window for one repo.\n\t\t\tconst agentDir = getHooCodeDir();\n\t\t\tconst settings = SettingsManager.create(ctx.cwd, agentDir);\n\t\t\tconst window = settings.getLearnSettings();\n\t\t\tconst statePath = getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir));\n\n\t\t\tconst pipeline = await buildPipeline(ctx, settings);\n\t\t\tif (\"error\" in pipeline) {\n\t\t\t\tctx.ui.notify(pipeline.error, \"error\");\n\t\t\t\treturn;\n\t\t\t}\n\n\t\t\t// State the price before charging it. A first run in a busy repo reads\n\t\t\t// every transcript in the window, which is the expensive path by design\n\t\t\t// — but it should never be a surprise, and the cache means it is paid\n\t\t\t// once rather than on every run.\n\t\t\tconst { pending } = pendingWork(ctx, agentDir, window);\n\t\t\tif (pending > CONFIRM_ABOVE_PENDING) {\n\t\t\t\tconst proceed = await ctx.ui.confirm(\n\t\t\t\t\t\"Read session transcripts?\",\n\t\t\t\t\t`${pending} session(s) have not been read yet. /learn reads each one with a model ` +\n\t\t\t\t\t\t`(${pipeline.model.provider}/${pipeline.model.id}) and caches the result, so this cost is paid once ` +\n\t\t\t\t\t\t`per session. Later runs reuse it.`,\n\t\t\t\t);\n\t\t\t\tif (!proceed) {\n\t\t\t\t\tctx.ui.notify(\"/learn cancelled — nothing was read.\", \"info\");\n\t\t\t\t\treturn;\n\t\t\t\t}\n\t\t\t}\n\n\t\t\t// A backfill can run for minutes across dozens of transcripts, and the\n\t\t\t// agent is idle throughout — so `ctx.signal` is undefined and there is no\n\t\t\t// ambient way out. Escape gets one.\n\t\t\tconst controller = new AbortController();\n\t\t\tconst unsubscribe = ctx.ui.onTerminalInput((data) => {\n\t\t\t\tif (data !== ESCAPE) return undefined;\n\t\t\t\tcontroller.abort();\n\t\t\t\treturn { consume: true };\n\t\t\t});\n\n\t\t\tlet digest: LearnDigest;\n\t\t\ttry {\n\t\t\t\tdigest = await mineLearnDigest({\n\t\t\t\t\tcwd: ctx.cwd,\n\t\t\t\t\tagentDir,\n\t\t\t\t\t// Searched in addition to the per-cwd default directory, so a session\n\t\t\t\t\t// manager pointing elsewhere (`--session`, a custom `sessionDir`, or\n\t\t\t\t\t// an in-memory session reporting none at all) cannot hide the history.\n\t\t\t\t\tsessionDir: ctx.sessionManager.getSessionDir(),\n\t\t\t\t\tmaxSessions: window.maxSessions,\n\t\t\t\t\tmaxAgeDays: window.maxAgeDays,\n\t\t\t\t\tminRepeats: window.minRepeats,\n\t\t\t\t\tminWorkflowRepeats: window.minWorkflowRepeats,\n\t\t\t\t\tmaxProposals: window.maxProposals,\n\t\t\t\t\tstate: readLearnState(statePath),\n\t\t\t\t\tignoreState,\n\t\t\t\t\tminer: pipeline.miner,\n\t\t\t\t\tcoverageJudge: pipeline.coverageJudge,\n\t\t\t\t\tsignal: controller.signal,\n\t\t\t\t\tonProgress: ({ done, total, cached }) => {\n\t\t\t\t\t\t// The same footer bar the semantic index uses. Cached sessions are\n\t\t\t\t\t\t// counted as done because they are: the bar measures progress\n\t\t\t\t\t\t// through the window, not money spent, and a run that is mostly\n\t\t\t\t\t\t// cache should look nearly finished from the start.\n\t\t\t\t\t\tstartupProgress.set({\n\t\t\t\t\t\t\tkey: PROGRESS_KEY,\n\t\t\t\t\t\t\tkind: \"work\",\n\t\t\t\t\t\t\tlabel:\n\t\t\t\t\t\t\t\tcached > 0\n\t\t\t\t\t\t\t\t\t? `Reading sessions (${cached} cached) — esc to stop`\n\t\t\t\t\t\t\t\t\t: \"Reading sessions — esc to stop\",\n\t\t\t\t\t\t\tdone,\n\t\t\t\t\t\t\ttotal,\n\t\t\t\t\t\t\tunit: \"sessions\",\n\t\t\t\t\t\t});\n\t\t\t\t\t},\n\t\t\t\t});\n\t\t\t} catch (error) {\n\t\t\t\tctx.ui.notify(`/learn could not read session history: ${error}`, \"error\");\n\t\t\t\treturn;\n\t\t\t} finally {\n\t\t\t\tunsubscribe();\n\t\t\t\tstartupProgress.remove(PROGRESS_KEY);\n\t\t\t}\n\n\t\t\t// A cancelled run counted only part of the window, so its numbers are not\n\t\t\t// merely incomplete — they are low. Showing them would be misleading and\n\t\t\t// bookmarking them would hide those items on the next, complete run.\n\t\t\t// Everything read so far is cached, so stopping costs nothing but time.\n\t\t\tif (digest.aborted) {\n\t\t\t\tctx.ui.notify(\n\t\t\t\t\t`/learn stopped — ${digest.mining.mined} session(s) were read and cached, so resuming picks up where this left off.`,\n\t\t\t\t\t\"info\",\n\t\t\t\t);\n\t\t\t\treturn;\n\t\t\t}\n\n\t\t\tif (digest.scannedSessions === 0) {\n\t\t\t\treportNoSessions(ctx, agentDir, digest, window);\n\t\t\t\treturn;\n\t\t\t}\n\n\t\t\tif (isEmptyDigest(digest)) {\n\t\t\t\tconst lines: string[] = [];\n\t\t\t\tlines.push(\n\t\t\t\t\tdigest.suppressed > 0\n\t\t\t\t\t\t? `Read ${digest.scannedSessions} session(s) — nothing new since last time (${digest.suppressed} already shown). Run /learn all to see them again.`\n\t\t\t\t\t\t: `Read ${digest.scannedSessions} session(s) — nothing repeated often enough to be worth a rule yet.`,\n\t\t\t\t);\n\t\t\t\tif (digest.suppressed === 0) {\n\t\t\t\t\t// The thresholds are the reason a scan with real sessions in it came\n\t\t\t\t\t// back empty, so this is the moment they are worth knowing about.\n\t\t\t\t\tlines.push(\"\");\n\t\t\t\t\tlines.push(...settingsLines(ctx, agentDir, window));\n\t\t\t\t}\n\t\t\t\tctx.ui.notify(lines.join(\"\\n\"), \"info\");\n\t\t\t\treturn;\n\t\t\t}\n\n\t\t\tconst counts = [\n\t\t\t\tdigest.directives.length > 0 ? `${digest.directives.length} directive(s)` : undefined,\n\t\t\t\tdigest.fixes.length > 0 ? `${digest.fixes.length} fix(es)` : undefined,\n\t\t\t\tdigest.workflows.length > 0 ? `${digest.workflows.length} workflow(s)` : undefined,\n\t\t\t].filter((part): part is string => !!part);\n\t\t\tconst held = digest.suppressed > 0 ? `, ${digest.suppressed} held back` : \"\";\n\t\t\tctx.ui.notify(\n\t\t\t\t`Mined ${digest.scannedSessions} session(s) (${digest.mining.mined} read, ${digest.mining.cached} cached): ${counts.join(\", \")}${held}.`,\n\t\t\t\t\"info\",\n\t\t\t);\n\n\t\t\t// Record before delivering: what matters is that these were put in front\n\t\t\t// of the user, which is true whether or not they act on the digest.\n\t\t\twriteLearnState(statePath, recordSurfaced(readLearnState(statePath), digest.surfaced));\n\n\t\t\tpi.sendUserMessage(\n\t\t\t\trenderLearnDigest(digest, {\n\t\t\t\t\tuserScopePath: displayPath(USER_SCOPE_PATH),\n\t\t\t\t\tmode: ignoreState ? \"all\" : \"incremental\",\n\t\t\t\t}),\n\t\t\t\t{ deliverAs: \"followUp\" },\n\t\t\t);\n\t\t},\n\t});\n}\n"]}
|
|
@@ -2,16 +2,20 @@
|
|
|
2
2
|
* `/learn` — promote what recent sessions actually taught into durable rules
|
|
3
3
|
* and skills.
|
|
4
4
|
*
|
|
5
|
-
* The command is a thin shell on purpose. It runs the
|
|
6
|
-
*
|
|
7
|
-
*
|
|
8
|
-
*
|
|
5
|
+
* The command is a thin shell on purpose. It runs the mining pipeline over
|
|
6
|
+
* session transcripts on disk, renders the ranked result, and injects it as a
|
|
7
|
+
* follow-up message; every judgement after that belongs to the model, which can
|
|
8
|
+
* read the repo and phrase a rule far better than a heuristic can.
|
|
9
9
|
*
|
|
10
10
|
* Reading transcripts from disk rather than the live context is what makes this
|
|
11
11
|
* work: the on-disk history survives compaction, and it spans past sessions, so
|
|
12
12
|
* "you have said this in five separate sessions" is available as a number
|
|
13
13
|
* instead of a guess. That number is the whole reason the command exists.
|
|
14
14
|
*
|
|
15
|
+
* The pipeline reads every transcript with a model rather than pre-filtering
|
|
16
|
+
* with regexes, which costs real tokens on a cold cache. That price is stated
|
|
17
|
+
* before it is paid, never inferred: a run with sessions to read asks first.
|
|
18
|
+
*
|
|
15
19
|
* Follows /grill in modes.ts: no session switch, no mode change, no config
|
|
16
20
|
* write — just a follow-up message. Writes to AGENTS.md happen through ordinary
|
|
17
21
|
* edit tools, so the existing permission prompt is the approval step and no
|
|
@@ -21,15 +25,32 @@ import { homedir } from "node:os";
|
|
|
21
25
|
import { join } from "node:path";
|
|
22
26
|
import { CONFIG_DIR_NAME, getHooCodeDir } from "../../config.js";
|
|
23
27
|
import { loadProjectContextFiles } from "../../core/context-files.js";
|
|
28
|
+
import { createLlmCoverageJudge } from "../../core/learn/coverage.js";
|
|
24
29
|
import { isEmptyDigest, renderLearnDigest } from "../../core/learn/digest.js";
|
|
25
|
-
import { buildCoverageIndex,
|
|
30
|
+
import { buildCoverageIndex, mineLearnDigest, planMining, scanSessions, } from "../../core/learn/extract.js";
|
|
31
|
+
import { chunkCharsForModel, createLlmMiner } from "../../core/learn/mine.js";
|
|
26
32
|
import { getLearnStatePath, readLearnState, recordSurfaced, summarizeLearnState, writeLearnState, } from "../../core/learn/state.js";
|
|
33
|
+
import { resolveModelCategory } from "../../core/model-categories.js";
|
|
27
34
|
import { getSessionDirPath } from "../../core/session-manager.js";
|
|
28
35
|
import { SettingsManager } from "../../core/settings-manager.js";
|
|
36
|
+
import { startupProgress } from "../../core/startup-progress.js";
|
|
29
37
|
/** Guards against double-registration when default extensions load more than once. */
|
|
30
38
|
const REGISTERED = Symbol.for("hoocode.learn.registered");
|
|
31
39
|
/** User-scope destination offered for personal rules that travel across repos. */
|
|
32
40
|
const USER_SCOPE_PATH = join(homedir(), ".agents", "AGENTS.md");
|
|
41
|
+
/** Footer key for the mining progress bar. */
|
|
42
|
+
const PROGRESS_KEY = "learn-mining";
|
|
43
|
+
/** Escape, the way a raw terminal delivers it. */
|
|
44
|
+
const ESCAPE = "\x1b";
|
|
45
|
+
/**
|
|
46
|
+
* Sessions that can be read without asking first.
|
|
47
|
+
*
|
|
48
|
+
* A run that has one or two new transcripts to read is the normal daily case
|
|
49
|
+
* and interrupting it to confirm a trivial cost is noise. Beyond this the run
|
|
50
|
+
* is a backfill — onboarding to an existing repo, or a first run — and the
|
|
51
|
+
* reader should get to decide before it starts.
|
|
52
|
+
*/
|
|
53
|
+
const CONFIRM_ABOVE_PENDING = 3;
|
|
33
54
|
/** Render a home-relative path the way the user would type it. */
|
|
34
55
|
function displayPath(path) {
|
|
35
56
|
const home = homedir();
|
|
@@ -55,7 +76,7 @@ const SETTING_KEYS = [
|
|
|
55
76
|
/**
|
|
56
77
|
* Where the knobs live, and what they are set to.
|
|
57
78
|
*
|
|
58
|
-
* `/learn` has
|
|
79
|
+
* `/learn` has its settings and no UI, so until this existed the only way to
|
|
59
80
|
* find them was to already know they were in `settings.json`. Every message that
|
|
60
81
|
* reports a disappointing result names a threshold, so every one of them ends
|
|
61
82
|
* with these lines.
|
|
@@ -70,7 +91,7 @@ function settingsPathLines(ctx, agentDir) {
|
|
|
70
91
|
function settingsLines(ctx, agentDir, window) {
|
|
71
92
|
const lines = settingsPathLines(ctx, agentDir);
|
|
72
93
|
for (const { key, setting, note } of SETTING_KEYS) {
|
|
73
|
-
lines.push(` ${setting.padEnd(24)} ${String(window[key]).padStart(3)} ${note}`);
|
|
94
|
+
lines.push(` ${setting.padEnd(24)} ${String(window[key] ?? "—").padStart(3)} ${note}`);
|
|
74
95
|
}
|
|
75
96
|
return lines;
|
|
76
97
|
}
|
|
@@ -86,7 +107,7 @@ function settingsLines(ctx, agentDir, window) {
|
|
|
86
107
|
function stateKeyDir(ctx, agentDir) {
|
|
87
108
|
return getSessionDirPath(ctx.cwd, agentDir);
|
|
88
109
|
}
|
|
89
|
-
/** Run the directory scan without
|
|
110
|
+
/** Run the directory scan without mining anything, for the reports that only need counts. */
|
|
90
111
|
function sessionScanPreview(ctx, agentDir, window) {
|
|
91
112
|
return scanSessions({
|
|
92
113
|
cwd: ctx.cwd,
|
|
@@ -136,16 +157,75 @@ function reportNoSessions(ctx, agentDir, digest, window) {
|
|
|
136
157
|
lines.push(...settingsLines(ctx, agentDir, window));
|
|
137
158
|
ctx.ui.notify(lines.join("\n"), "warning");
|
|
138
159
|
}
|
|
160
|
+
/**
|
|
161
|
+
* The model that reads transcripts.
|
|
162
|
+
*
|
|
163
|
+
* This is the one call in the pipeline that reads *everything*, so it wants the
|
|
164
|
+
* cheapest capable model rather than the session's. That question already has an
|
|
165
|
+
* answer in this codebase — the `fast` model category, which subagents use for
|
|
166
|
+
* exactly this kind of bulk read — so it is reused rather than reinvented.
|
|
167
|
+
* `settings.modelCategories.fast` wins when set; otherwise the tier is derived
|
|
168
|
+
* from the user's available models, and nothing here is provider-specific.
|
|
169
|
+
*
|
|
170
|
+
* Falls back to the session model when the tier resolves to nothing or to a
|
|
171
|
+
* model the registry cannot find, since a mis-set tier should not take the
|
|
172
|
+
* command out entirely.
|
|
173
|
+
*/
|
|
174
|
+
function resolveMinerModel(ctx, settings) {
|
|
175
|
+
const ref = resolveModelCategory("fast", {
|
|
176
|
+
modelCategories: settings.getModelCategories(),
|
|
177
|
+
defaultProvider: settings.getDefaultProvider(),
|
|
178
|
+
defaultModel: settings.getDefaultModel(),
|
|
179
|
+
}, ctx.modelRegistry.getAvailable());
|
|
180
|
+
if (!ref)
|
|
181
|
+
return ctx.model;
|
|
182
|
+
const slash = ref.indexOf("/");
|
|
183
|
+
const found = slash > 0 ? ctx.modelRegistry.find(ref.slice(0, slash), ref.slice(slash + 1)) : undefined;
|
|
184
|
+
return found ?? ctx.model;
|
|
185
|
+
}
|
|
186
|
+
/** Build the two model-backed stages, or report why they cannot be built. */
|
|
187
|
+
async function buildPipeline(ctx, settings) {
|
|
188
|
+
const model = resolveMinerModel(ctx, settings);
|
|
189
|
+
if (!model) {
|
|
190
|
+
return {
|
|
191
|
+
error: "/learn reads session transcripts with a model, and no model is selected. Pick one with /model, then run /learn again.",
|
|
192
|
+
};
|
|
193
|
+
}
|
|
194
|
+
const auth = await ctx.modelRegistry.getApiKeyAndHeaders(model);
|
|
195
|
+
if (!auth.ok) {
|
|
196
|
+
return { error: `/learn could not authenticate ${model.provider}/${model.id}: ${auth.error}` };
|
|
197
|
+
}
|
|
198
|
+
const deps = { model, apiKey: auth.apiKey, headers: auth.headers };
|
|
199
|
+
return { miner: createLlmMiner(deps), coverageJudge: createLlmCoverageJudge(deps), model };
|
|
200
|
+
}
|
|
201
|
+
/**
|
|
202
|
+
* What this run still owes the model.
|
|
203
|
+
*
|
|
204
|
+
* Delegated to `planMining` so the number quoted by the confirmation prompt
|
|
205
|
+
* comes from the same session selection the run will use — same window, same
|
|
206
|
+
* cwd check, same de-duplication.
|
|
207
|
+
*/
|
|
208
|
+
function pendingWork(ctx, agentDir, window) {
|
|
209
|
+
return planMining({
|
|
210
|
+
cwd: ctx.cwd,
|
|
211
|
+
agentDir,
|
|
212
|
+
sessionDir: ctx.sessionManager.getSessionDir(),
|
|
213
|
+
maxSessions: window.maxSessions,
|
|
214
|
+
maxAgeDays: window.maxAgeDays,
|
|
215
|
+
});
|
|
216
|
+
}
|
|
139
217
|
/**
|
|
140
218
|
* `/learn stats` — what became of past proposals.
|
|
141
219
|
*
|
|
142
220
|
* Reads the state file and recomputes coverage; it does not re-mine sessions,
|
|
143
|
-
* so it
|
|
144
|
-
* "what should I write down" but "is this command earning its
|
|
221
|
+
* so it costs one small model call and answers a different question than a
|
|
222
|
+
* normal run: not "what should I write down" but "is this command earning its
|
|
223
|
+
* place".
|
|
145
224
|
*/
|
|
146
|
-
function reportStats(ctx) {
|
|
225
|
+
async function reportStats(ctx) {
|
|
147
226
|
const agentDir = getHooCodeDir();
|
|
148
|
-
const
|
|
227
|
+
const settings = SettingsManager.create(ctx.cwd, agentDir);
|
|
228
|
+
const window = settings.getLearnSettings();
|
|
149
229
|
const statePath = getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir));
|
|
150
230
|
const state = readLearnState(statePath);
|
|
151
231
|
if (Object.keys(state.surfaced).length === 0) {
|
|
@@ -163,10 +243,35 @@ function reportStats(ctx) {
|
|
|
163
243
|
return;
|
|
164
244
|
}
|
|
165
245
|
const coverage = buildCoverageIndex({ cwd: ctx.cwd, agentDir });
|
|
166
|
-
|
|
167
|
-
|
|
168
|
-
|
|
169
|
-
|
|
246
|
+
// Coverage is a model judgement now, so it is resolved once for every stored
|
|
247
|
+
// label in a single batch and then answered from the resulting set. Asking
|
|
248
|
+
// per item would be one call per proposal ever made in this directory.
|
|
249
|
+
const labels = Object.entries(state.surfaced)
|
|
250
|
+
.filter(([key]) => key.startsWith("directive:"))
|
|
251
|
+
// Entries recorded before the wording was kept fall back to the slug, which
|
|
252
|
+
// is a weaker question but still a answerable one.
|
|
253
|
+
.map(([key, item]) => ({
|
|
254
|
+
label: key.slice("directive:".length),
|
|
255
|
+
text: item.text || key.slice("directive:".length),
|
|
256
|
+
}));
|
|
257
|
+
const covered = new Set();
|
|
258
|
+
if (labels.length > 0) {
|
|
259
|
+
const pipeline = await buildPipeline(ctx, settings);
|
|
260
|
+
if (!("error" in pipeline)) {
|
|
261
|
+
try {
|
|
262
|
+
const verdicts = await pipeline.coverageJudge(labels, coverage);
|
|
263
|
+
for (const [label, match] of verdicts) {
|
|
264
|
+
if (match.rule || match.skill)
|
|
265
|
+
covered.add(label);
|
|
266
|
+
}
|
|
267
|
+
}
|
|
268
|
+
catch {
|
|
269
|
+
// Adoption is reported as zero rather than failing the report; the
|
|
270
|
+
// caveat below already tells the reader not to over-read the number.
|
|
271
|
+
}
|
|
272
|
+
}
|
|
273
|
+
}
|
|
274
|
+
const stats = summarizeLearnState(state, (label) => covered.has(label));
|
|
170
275
|
const contextTokens = loadProjectContextFiles({ cwd: ctx.cwd, agentDir }).agentsFiles.reduce((sum, file) => sum + (file.tokens ?? 0), 0);
|
|
171
276
|
const lines = [];
|
|
172
277
|
lines.push(`/learn history for this directory — ${shortDate(stats.earliest)} to ${shortDate(stats.latest)}`);
|
|
@@ -188,7 +293,7 @@ function reportStats(ctx) {
|
|
|
188
293
|
// against it exactly like a junk one — so near-100% means the bar is too
|
|
189
294
|
// low, not that the extractor is perfect.
|
|
190
295
|
lines.push(" A very high rate means the bar is too low, not that every proposal was good.");
|
|
191
|
-
lines.push(" Near zero means the
|
|
296
|
+
lines.push(" Near zero means the miner is proposing the wrong things.");
|
|
192
297
|
}
|
|
193
298
|
lines.push("");
|
|
194
299
|
lines.push(`Context files ~${contextTokens} tokens, re-sent every request`);
|
|
@@ -201,11 +306,25 @@ function reportStats(ctx) {
|
|
|
201
306
|
/** `/learn settings` — the knobs, their current values, and the files to set them in. */
|
|
202
307
|
function reportSettings(ctx) {
|
|
203
308
|
const agentDir = getHooCodeDir();
|
|
204
|
-
const
|
|
309
|
+
const settings = SettingsManager.create(ctx.cwd, agentDir);
|
|
310
|
+
const window = settings.getLearnSettings();
|
|
205
311
|
const lines = settingsLines(ctx, agentDir, window);
|
|
312
|
+
// The reading model is not a `/learn` setting — it is the shared `fast` tier,
|
|
313
|
+
// so name it here rather than leaving the reader to guess which model is
|
|
314
|
+
// about to read their history, and point at the setting that changes it.
|
|
315
|
+
const model = resolveMinerModel(ctx, settings);
|
|
316
|
+
lines.push(` reads transcripts with ${model ? `${model.provider}/${model.id}` : "no model selected"}` +
|
|
317
|
+
` (the \`fast\` tier — set modelCategories.fast to change it)`);
|
|
318
|
+
if (model) {
|
|
319
|
+
lines.push(` ${Math.round(chunkCharsForModel(model) / 1000)}k characters per call, from its ${model.contextWindow} token window`);
|
|
320
|
+
}
|
|
206
321
|
lines.push("");
|
|
207
322
|
lines.push(...scanLines(sessionScanPreview(ctx, agentDir, window), window));
|
|
208
323
|
lines.push(`State file ${displayPath(getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir)))}`);
|
|
324
|
+
const { pending } = pendingWork(ctx, agentDir, window);
|
|
325
|
+
lines.push(pending === 0
|
|
326
|
+
? "All sessions in the window are already mined; the next /learn costs one small coverage call."
|
|
327
|
+
: `${pending} session(s) in the window still need reading, roughly one call each.`);
|
|
209
328
|
ctx.ui.notify(lines.join("\n"), "info");
|
|
210
329
|
}
|
|
211
330
|
export function setupLearn(pi) {
|
|
@@ -229,7 +348,7 @@ export function setupLearn(pi) {
|
|
|
229
348
|
return;
|
|
230
349
|
}
|
|
231
350
|
if (argument === "stats") {
|
|
232
|
-
reportStats(ctx);
|
|
351
|
+
await reportStats(ctx);
|
|
233
352
|
return;
|
|
234
353
|
}
|
|
235
354
|
if (argument === "settings") {
|
|
@@ -240,11 +359,41 @@ export function setupLearn(pi) {
|
|
|
240
359
|
// Read per-invocation so a settings edit takes effect without a reload,
|
|
241
360
|
// and so a project settings.json can narrow the window for one repo.
|
|
242
361
|
const agentDir = getHooCodeDir();
|
|
243
|
-
const
|
|
362
|
+
const settings = SettingsManager.create(ctx.cwd, agentDir);
|
|
363
|
+
const window = settings.getLearnSettings();
|
|
244
364
|
const statePath = getLearnStatePath(agentDir, stateKeyDir(ctx, agentDir));
|
|
365
|
+
const pipeline = await buildPipeline(ctx, settings);
|
|
366
|
+
if ("error" in pipeline) {
|
|
367
|
+
ctx.ui.notify(pipeline.error, "error");
|
|
368
|
+
return;
|
|
369
|
+
}
|
|
370
|
+
// State the price before charging it. A first run in a busy repo reads
|
|
371
|
+
// every transcript in the window, which is the expensive path by design
|
|
372
|
+
// — but it should never be a surprise, and the cache means it is paid
|
|
373
|
+
// once rather than on every run.
|
|
374
|
+
const { pending } = pendingWork(ctx, agentDir, window);
|
|
375
|
+
if (pending > CONFIRM_ABOVE_PENDING) {
|
|
376
|
+
const proceed = await ctx.ui.confirm("Read session transcripts?", `${pending} session(s) have not been read yet. /learn reads each one with a model ` +
|
|
377
|
+
`(${pipeline.model.provider}/${pipeline.model.id}) and caches the result, so this cost is paid once ` +
|
|
378
|
+
`per session. Later runs reuse it.`);
|
|
379
|
+
if (!proceed) {
|
|
380
|
+
ctx.ui.notify("/learn cancelled — nothing was read.", "info");
|
|
381
|
+
return;
|
|
382
|
+
}
|
|
383
|
+
}
|
|
384
|
+
// A backfill can run for minutes across dozens of transcripts, and the
|
|
385
|
+
// agent is idle throughout — so `ctx.signal` is undefined and there is no
|
|
386
|
+
// ambient way out. Escape gets one.
|
|
387
|
+
const controller = new AbortController();
|
|
388
|
+
const unsubscribe = ctx.ui.onTerminalInput((data) => {
|
|
389
|
+
if (data !== ESCAPE)
|
|
390
|
+
return undefined;
|
|
391
|
+
controller.abort();
|
|
392
|
+
return { consume: true };
|
|
393
|
+
});
|
|
245
394
|
let digest;
|
|
246
395
|
try {
|
|
247
|
-
digest =
|
|
396
|
+
digest = await mineLearnDigest({
|
|
248
397
|
cwd: ctx.cwd,
|
|
249
398
|
agentDir,
|
|
250
399
|
// Searched in addition to the per-cwd default directory, so a session
|
|
@@ -258,12 +407,43 @@ export function setupLearn(pi) {
|
|
|
258
407
|
maxProposals: window.maxProposals,
|
|
259
408
|
state: readLearnState(statePath),
|
|
260
409
|
ignoreState,
|
|
410
|
+
miner: pipeline.miner,
|
|
411
|
+
coverageJudge: pipeline.coverageJudge,
|
|
412
|
+
signal: controller.signal,
|
|
413
|
+
onProgress: ({ done, total, cached }) => {
|
|
414
|
+
// The same footer bar the semantic index uses. Cached sessions are
|
|
415
|
+
// counted as done because they are: the bar measures progress
|
|
416
|
+
// through the window, not money spent, and a run that is mostly
|
|
417
|
+
// cache should look nearly finished from the start.
|
|
418
|
+
startupProgress.set({
|
|
419
|
+
key: PROGRESS_KEY,
|
|
420
|
+
kind: "work",
|
|
421
|
+
label: cached > 0
|
|
422
|
+
? `Reading sessions (${cached} cached) — esc to stop`
|
|
423
|
+
: "Reading sessions — esc to stop",
|
|
424
|
+
done,
|
|
425
|
+
total,
|
|
426
|
+
unit: "sessions",
|
|
427
|
+
});
|
|
428
|
+
},
|
|
261
429
|
});
|
|
262
430
|
}
|
|
263
431
|
catch (error) {
|
|
264
432
|
ctx.ui.notify(`/learn could not read session history: ${error}`, "error");
|
|
265
433
|
return;
|
|
266
434
|
}
|
|
435
|
+
finally {
|
|
436
|
+
unsubscribe();
|
|
437
|
+
startupProgress.remove(PROGRESS_KEY);
|
|
438
|
+
}
|
|
439
|
+
// A cancelled run counted only part of the window, so its numbers are not
|
|
440
|
+
// merely incomplete — they are low. Showing them would be misleading and
|
|
441
|
+
// bookmarking them would hide those items on the next, complete run.
|
|
442
|
+
// Everything read so far is cached, so stopping costs nothing but time.
|
|
443
|
+
if (digest.aborted) {
|
|
444
|
+
ctx.ui.notify(`/learn stopped — ${digest.mining.mined} session(s) were read and cached, so resuming picks up where this left off.`, "info");
|
|
445
|
+
return;
|
|
446
|
+
}
|
|
267
447
|
if (digest.scannedSessions === 0) {
|
|
268
448
|
reportNoSessions(ctx, agentDir, digest, window);
|
|
269
449
|
return;
|
|
@@ -271,8 +451,8 @@ export function setupLearn(pi) {
|
|
|
271
451
|
if (isEmptyDigest(digest)) {
|
|
272
452
|
const lines = [];
|
|
273
453
|
lines.push(digest.suppressed > 0
|
|
274
|
-
? `
|
|
275
|
-
: `
|
|
454
|
+
? `Read ${digest.scannedSessions} session(s) — nothing new since last time (${digest.suppressed} already shown). Run /learn all to see them again.`
|
|
455
|
+
: `Read ${digest.scannedSessions} session(s) — nothing repeated often enough to be worth a rule yet.`);
|
|
276
456
|
if (digest.suppressed === 0) {
|
|
277
457
|
// The thresholds are the reason a scan with real sessions in it came
|
|
278
458
|
// back empty, so this is the moment they are worth knowing about.
|
|
@@ -288,13 +468,14 @@ export function setupLearn(pi) {
|
|
|
288
468
|
digest.workflows.length > 0 ? `${digest.workflows.length} workflow(s)` : undefined,
|
|
289
469
|
].filter((part) => !!part);
|
|
290
470
|
const held = digest.suppressed > 0 ? `, ${digest.suppressed} held back` : "";
|
|
291
|
-
ctx.ui.notify(`Mined ${digest.scannedSessions} session(s): ${counts.join(", ")}${held}.`, "info");
|
|
471
|
+
ctx.ui.notify(`Mined ${digest.scannedSessions} session(s) (${digest.mining.mined} read, ${digest.mining.cached} cached): ${counts.join(", ")}${held}.`, "info");
|
|
292
472
|
// Record before delivering: what matters is that these were put in front
|
|
293
473
|
// of the user, which is true whether or not they act on the digest.
|
|
294
474
|
writeLearnState(statePath, recordSurfaced(readLearnState(statePath), digest.surfaced));
|
|
295
|
-
pi.sendUserMessage(renderLearnDigest(digest, {
|
|
296
|
-
|
|
297
|
-
|
|
475
|
+
pi.sendUserMessage(renderLearnDigest(digest, {
|
|
476
|
+
userScopePath: displayPath(USER_SCOPE_PATH),
|
|
477
|
+
mode: ignoreState ? "all" : "incremental",
|
|
478
|
+
}), { deliverAs: "followUp" });
|
|
298
479
|
},
|
|
299
480
|
});
|
|
300
481
|
}
|