@ericrisco/rsc 2.0.13 → 2.0.15

package/manifest.json

@@ -1,5 +1,5 @@
 {
-  "version": "2.0.13",
+  "version": "2.0.15",
   "counts": {
     "skills": 273,
     "agents": 33,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ericrisco/rsc",
-  "version": "2.0.13",
+  "version": "2.0.15",
   "description": "A guided harness builder that turns your goal into a minimal, verified AI-agent setup using strict best practices.",
   "type": "module",
   "bin": {
@@ -85,6 +85,10 @@
     {
       "fixedIn": "2.0.12",
       "symptom": "Security: session continuation context could come from a different git worktree"
+    },
+    {
+      "fixedIn": "2.0.15",
+      "symptom": "The 'consider compacting' notice never appears in sessions that edit files through the terminal instead of the Edit tool"
     }
   ]
 }

package/scripts/rsc.js

@@ -501,7 +501,18 @@ function printContextBudget(b) {
   say('════════════════════════════════════════════════\n');
 }
 
-async function wizard(flagTargets) {
+async function syncDeclared(targets, dry = false) {
+  for (const t of targets) {
+    const result = await syncInstalled({ target: t, dryRun: dry });
+    const verb = dry ? 'Would sync' : 'Synced';
+    say(`${verb} ${t}: ${result.synced.length ? result.synced.join(', ') : '(nothing to sync)'}`);
+    if (dry && result.paths?.length) {
+      for (const p of result.paths) say(`  ${p}`);
+    }
+  }
+}
+
+async function wizard(flagTargets, declaredTargets = []) {
   const m = loadManifest();
   await banner(m.counts.skills);
   say('  the skill catalog for your assistant (Claude Code · Codex · Cursor · Gemini · Antigravity)\n');
@@ -511,12 +522,16 @@ async function wizard(flagTargets) {
   const baseIds = skillsForProfile(m, 'minimal');
   // Navigable loop: esc / "← Back" / "no" all return here instead of quitting.
   for (;;) {
+    const declared = readManifest();
     const choice = await select('What do you want to do?', [
+      // First, because it is what someone who already has a harness came here for nine times in ten.
+      ...(declared ? [{ key: 'update', label: `Update this project — keeps its ${declared.skills.length} skills, refreshes hooks and content` }] : []),
       { key: 'base', label: `Base install — the essentials (${baseIds.length} skills)` },
       { key: 'sdd', label: 'Base + Spec-Driven Development — the specify → plan → implement → ship flow' },
       { key: 'manual', label: 'Pick skills by hand, by area' },
     ]);
     if (choice === null) { say('\nOK — nothing installed. Anytime: npx @ericrisco/rsc'); return; }
+    if (choice === 'update') return syncDeclared(declaredTargets);
 
     let ids;
     if (choice === 'base') ids = baseIds;
@@ -621,7 +636,14 @@ async function main() {
   const target = targets[0];
   switch (cmd) {
     case undefined:
-      return hasDeclaredHarness() ? wizard(f ? targets : null) : runOnboarding(f ? targets : []);
+      if (!hasDeclaredHarness()) return runOnboarding(f ? targets : []);
+      // A project that already has a harness: the bare command means "bring it up to date". It is
+      // what every update notice, the README and our own replies to users tell people to run, with
+      // the promise "that reinstalls and refreshes". It used to open the install wizard instead, and
+      // with no terminal — which is how an agent runs it when it obeys the notice — it did nothing.
+      // Restoring what the project already declared is not a decision, so there is nothing to ask.
+      if (!isInteractive()) return syncDeclared(targets);
+      return wizard(f ? targets : null, targets);
     case 'onboard':
       return runOnboarding(f ? targets : []);
     case 'reassess':
@@ -773,18 +795,8 @@ async function main() {
       process.exitCode = 1;
       return;
     }
-    case 'sync': {
-      const dry = argv.includes('--dry-run');
-      for (const t of targets) {
-        const result = await syncInstalled({ target: t, dryRun: dry });
-        const verb = dry ? 'Would sync' : 'Synced';
-        say(`${verb} ${t}: ${result.synced.length ? result.synced.join(', ') : '(nothing to sync)'}`);
-        if (dry && result.paths?.length) {
-          for (const p of result.paths) say(`  ${p}`);
-        }
-      }
-      return;
-    }
+    case 'sync':
+      return syncDeclared(targets, argv.includes('--dry-run'));
     case 'backups': {
       const backups = listBackups();
       if (!backups.length) return void say('(no backups)');

package/targets/session-memory-core.mjs

@@ -319,7 +319,20 @@ export function capture(input = {}) {
   if (input.event === 'start' && !existing) return { record: null, path: null, notice: null, compactionHint: false };
 
   const repo = snapshot(here, anchor.baselineHead);
-  const editCount = Math.max(0, (existing?.editCount || 0) + (finiteOrNull(input.editDelta, true) || 0));
+  // Count what CHANGED, not which tool changed it. `editDelta` is +1 per Edit/Write tool event, and
+  // on its own it was blind to everything done through the shell — a heredoc, `sed -i`, a script, a
+  // formatter — which is how automated sessions work. The session that found this had touched dozens
+  // of files and its record said editCount=1, so the compaction hint never reached the long sessions
+  // it exists for.
+  //
+  // `snapshot` already fingerprints every modified file on every event. Comparing those fingerprints
+  // with the previous event's counts a change from any source. `max`, not `+`: an Edit that changed a
+  // file shows up in BOTH signals, and must count once.
+  const previousFingerprints = anchor.lastFingerprints || anchor.baselineFingerprints || {};
+  const observed = Object.entries(repo.fingerprints || {})
+    .filter(([path, fingerprint]) => fingerprint && previousFingerprints[path] !== fingerprint).length;
+  const editDelta = Math.max(finiteOrNull(input.editDelta, true) || 0, observed);
+  const editCount = Math.max(0, (existing?.editCount || 0) + editDelta);
   const baselineFiles = new Set(anchor.baselineFiles || []);
   const newDirtyPath = repo.files.some((path) => !baselineFiles.has(path));
   const baselineFingerprints = anchor.baselineFingerprints || {};
@@ -368,7 +381,17 @@ export function capture(input = {}) {
   // already written and `readRecords` would quietly drop the project's whole history.
   let compactionHint = Boolean(config.compactionHint && editCount >= config.editThreshold);
   if (compactionHint && anchor.compactionHintedAt) compactionHint = false;
-  else if (compactionHint) atomicJson(anchorPath, { ...anchor, compactionHintedAt: now });
+  // One anchor write per event at most, carrying both pieces of per-session state: the fingerprints
+  // the next event compares against, and the once-per-session hint mark. Written only when either
+  // actually changed, so a read-only shell command costs no extra disk.
+  const fingerprintsMoved = observed > 0 || !anchor.lastFingerprints;
+  if (compactionHint || fingerprintsMoved) {
+    atomicJson(anchorPath, {
+      ...anchor,
+      lastFingerprints: repo.fingerprints || {},
+      ...(compactionHint ? { compactionHintedAt: now } : {}),
+    });
+  }
   return {
     record,
     path: recordPath,