@pcircle/memesh 4.8.0 → 4.8.2

package/scripts/hooks/session-summary.js

@@ -87,6 +87,30 @@ async function runAutoUpdateAtStop() {
 // Handles the current Claude Code transcript format where tool_use/tool_result
 // are nested inside assistant/user message entries, not top-level entries.
 // Defensive: never throws — malformed lines are silently skipped.
+/**
+ * File paths a shell command writes in place. Recognises the shapes an
+ * agent actually uses to edit without Write/Edit: heredoc redirection
+ * (`> path <<'EOF'`, `cat > path`), `sed -i`, `tee`, and `pathlib.Path('x')
+ * .write_text(` / `fs.writeFileSync('x'` inside an inline script. Returns
+ * basenames' sources; the caller keeps basename() as the stored form.
+ */
+function bashEditedPaths(cmd) {
+  if (typeof cmd !== 'string') return [];
+  const found = new Set();
+  const add = (m) => { if (m && m[1] && !m[1].startsWith('/dev/') && !m[1].startsWith('/tmp/')) found.add(m[1]); };
+  for (const re of [
+    /(?:^|[^<])>\s*"?([^\s"'>|&;]+)"?\s*<<\s*['"]?\w+['"]?/g,   // > file <<'EOF'
+    /\bcat\s*>\s*"?([^\s"'>|&;]+)"?/g,                            // cat > file
+    /\btee\s+(?:-a\s+)?"?([^\s"'>|&;]+)"?/g,                      // tee file
+    /\bsed\s+-i(?:\s+'')?\s+(?:'[^']*'|"[^"]*")\s+"?([^\s"'>|&;]+)"?/g, // sed -i '...' file
+    /Path\(\s*['"]([^'"]+)['"]\s*\)\s*\.write_text\(/g,           // pathlib write_text
+    /writeFileSync\(\s*['"]([^'"]+)['"]/g,                        // fs.writeFileSync
+  ]) {
+    let m; while ((m = re.exec(cmd)) !== null) add(m);
+  }
+  return [...found];
+}
+
 function parseTranscript(transcriptPath) {
   const filesEdited = new Set();
   const bashCommands = [];
@@ -118,6 +142,15 @@ function parseTranscript(transcriptPath) {
             }
             if (block.name === 'Bash') {
               const cmd = block.input?.command ?? '';
+              // A session that edits through Bash — heredocs, sed -i, a
+              // short python script — never produces a Write/Edit block, so
+              // `filesEdited` stayed empty and the summary asserted "0 files
+              // edited" for a session that edited a dozen. The count also
+              // drives the re-capture guard below, so the same gap made the
+              // -summary entity re-append on every Stop (#240). Recognise the
+              // common in-place write shapes; anything not matched is simply
+              // uncounted, which is honest — it is not asserted as zero.
+              for (const fp of bashEditedPaths(cmd)) filesEdited.add(basename(fp));
               if (typeof cmd === 'string' && cmd.length > 10 && !cmd.startsWith('ls') && !cmd.startsWith('cd')) {
                 // Redact BEFORE truncating. A bash command line is the single
                 // most likely place a credential appears in a transcript
@@ -347,7 +380,13 @@ process.stdin.on('end', async () => {
       // A dedup bail is a SUCCESSFUL run — the loop executed and correctly
       // decided there was nothing to do — so it stamps the heartbeat like
       // the capture path below does. Only a throw leaves no stamp.
-      const alreadyCaptured = db.prepare("SELECT id FROM entities WHERE name = ?").get(`session-${sessionId}-files`);
+      // Guard on ANY of this session's three entities, not only `-files`.
+      // A Bash-only session created no `-files` row, so the guard never
+      // tripped and `-summary` was re-appended on every Stop — measured: 56
+      // observations, 16 unique, three commands stored fourteen times each.
+      const alreadyCaptured = db.prepare(
+        "SELECT id FROM entities WHERE name IN (?, ?, ?) LIMIT 1",
+      ).get(`session-${sessionId}-files`, `session-${sessionId}-fixes`, `session-${sessionId}-summary`);
       if (alreadyCaptured) {
         recordHookRun(db, 'session-summary');
         return exit0();

package/skills/memesh/SKILL.md

@@ -23,6 +23,22 @@ Persistent memory for AI agents. The point is continuity: the next session start
 
 All examples below use CLI. MCP tools accept the same parameters as JSON objects.
 
+## All 11 MCP tools
+
+| Tool | Purpose |
+|---|---|
+| `remember` | Store knowledge as an entity with observations, tags, and relations |
+| `recall` | Search stored knowledge; empty query lists recent memories |
+| `forget` | Archive an entity or remove one exact observation |
+| `export` | Export memories as portable JSON |
+| `import` | Import a JSON export with the required skip, append, or overwrite strategy |
+| `learn` | Record a structured lesson with error, fix, root cause, and prevention |
+| `task_state` | Read or update user-stated goal, next step, blocker, and finished work |
+| `briefing` | Assemble the current project's work topology |
+| `user_patterns` | Analyze work schedule, tool preferences, and focus areas |
+| `improvement` | Propose an evidence-linked product improvement or read its status; only a human may accept or reject it |
+| `message` | Contact another local agent (hand off, ask, report back) — send here first; the durable inbox is the record. Fetching does not acknowledge |
+
 ## The Loop
 
 Four moments. Everything else in this file is detail.
@@ -61,7 +77,7 @@ Use the routing and identity fields returned by `fetch`. A reply has this shape
 }
 ```
 
-An active compatible managed host can receive a native push, which removes polling for that live delivery. One-time provider enablement and a MeMesh-managed Codex app-server, Claude Channel, or Gemini ACP session may be required; ordinary unattached sessions are presence-only/inbound-unavailable. Adapter imports and a live router socket do not prove host registration or `host_accept`. Do not promise that a stopped, missing, or replaced session will wake up: it is not resumed or silently rerouted. Use the stable principal for logical routing, and an exact session/generation only when delivery must not move to a replacement connection. Local owns durable storage and host-native delivery; Cloud relay, A2A, SSE, discovery, or fetch is not host delivery.
+An active compatible managed host can receive a native push, which removes polling for that live delivery. One-time provider enablement and a MeMesh-managed Codex app-server or Claude Channel may be required; ordinary unattached sessions are presence-only/inbound-unavailable. The bundled Gemini ACP adapter is experimental protocol-development code, not a release-gated native-wakeup provider. Adapter imports and a live router socket do not prove host registration or `host_accept`. Do not promise that a stopped, missing, or replaced session will wake up: it is not resumed or silently rerouted. Use the stable principal for logical routing, and an exact session/generation only when delivery must not move to a replacement connection. Local owns durable storage and host-native delivery; Cloud relay, A2A, SSE, discovery, or fetch is not host delivery.
 
 Durable audit does not mean unbounded silent growth. Owners can inspect it with `memesh message storage report --cutoff <ISO timestamp>`, preview bounded terminal-payload tombstones with `memesh message storage prune --cutoff <ISO timestamp>`, and explicitly add `--apply`. Never prune unresolved/offline-pending work. `MEMESH_AGENT_MESSAGE_STORAGE_QUOTA_BYTES` is an optional owner policy; there is no default quota or automatic pruning.