npm - @bakapiano/ccsm - Versions diffs - 0.15.0 → 0.15.2 - Mend

@bakapiano/ccsm 0.15.0 → 0.15.2

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.

Files changed (10) hide show

package/CLAUDE.md +60 -0
package/lib/cliActivity.js +24 -3
package/lib/config.js +13 -19
package/package.json +1 -1
package/public/css/sidebar.css +42 -1
package/public/js/pages/AboutPage.js +9 -1
package/public/js/pages/ConfigurePage.js +9 -1
package/scripts/restart-helper.js +6 -1
package/scripts/upgrade-helper.js +6 -1
package/server.js +11 -4

package/CLAUDE.md CHANGED Viewed

@@ -443,6 +443,66 @@ fresh server. So `npm i -g @bakapiano/ccsm@latest && ccsm` is one
 seamless step. From the frontend, the About page's Upgrade button
 achieves the same thing without leaving the browser.
+### Release process
+Three artifacts ship per release: a git tag, a GitHub Release, and an
+npm publish. The whole thing is CI-driven — you never `npm publish`
+locally — but it requires you to drive three steps in order:
+1. **Commit + bump + push (local).** Stage everything, write a release
+   commit, then bump + tag + push:
+   ```powershell
+   git add -A
+   git commit -m "vX.Y.Z: <one-line summary>
+   <body>
+   Co-Authored-By: Claude ..."
+   npm --prefix . version <patch|minor|major> -m "v%s"
+   git push origin main
+   git push origin vX.Y.Z
+   ```
+   `npm version` writes the new version into `package.json` +
+   `package-lock.json`, creates its OWN commit, and tags it. The
+   `--prefix .` is needed on Windows where bare `npm version` errors on
+   the global `%APPDATA%\npm\package.json`. Push BOTH `main` and the
+   tag — pushing only main skips the tag-triggered draft-release
+   workflow.
+2. **Tag-push fires two workflows automatically:**
+   - `Deploy frontend to GitHub Pages` → publishes `pages-root/` → `/`
+     and `public/` → `/<X.Y.Z>/` on `gh-pages`. Old `/<X.Y.Z>/`
+     subdirs stay forever (`keep_files: true`).
+   - `Draft GitHub Release on tag push` → creates a **draft** release
+     for `vX.Y.Z`.
+3. **Publish the draft (manual one-liner):**
+   ```powershell
+   gh release edit vX.Y.Z --draft=false
+   ```
+   This flips the draft to "published", which fires the third workflow
+   — `Publish to npm` — using the `NPM_TOKEN` repo secret with
+   provenance. The runner needs ~30s; verify with
+   `gh run watch <run-id> --exit-status` or just refresh npmjs.com.
+The reason for the draft step instead of auto-publishing on tag push:
+gives you a chance to abort a half-baked tag (delete the draft +
+`git push --delete origin vX.Y.Z`) before it lands on the public
+registry.
+### Why we don't publish from the local box
+`npm publish` from a dev machine works in principle but skips
+provenance attestation (the sigstore + GitHub OIDC binding that npm
+displays as a "Provenance" badge on the package page). CI has the OIDC
+token; you don't. Local publish also wouldn't have the consistent
+runner state, so reproducible-build claims fall apart. The pipeline
+exists; use it.
 ## Cross-platform
 Today: Windows-first.

package/lib/cliActivity.js CHANGED Viewed

@@ -90,9 +90,20 @@ async function resolveTranscript(record, cliCfg) {
 async function probeActivity(record, cliCfg) {
   let s = state.get(record.id);
   if (!s) {
-    s = { resolvedPath: null, lastMtimeMs: 0, lastChangedAt: 0 };
+    s = { resolvedPath: null, lastMtimeMs: 0, lastChangedAt: 0, lastOutputAt: 0 };
     state.set(record.id, s);
   }
+  // PTY output (CLI is streaming text — thinking spinners, token output,
+  // status lines) is the strongest signal that the CLI is working. It's
+  // ALSO the only signal we have when the transcript file isn't being
+  // updated — claude/codex buffer reasoning + tool results for tens of
+  // seconds before flushing a turn, so mtime alone reports "idle"
+  // through long thinking phases. Check PTY first; short-circuit if the
+  // CLI is clearly active, skipping the fs.stat below.
+  const now = Date.now();
+  if (s.lastOutputAt && (now - s.lastOutputAt) < WORKING_WINDOW_MS) {
+    return 'working';
+  }
   if (!s.resolvedPath) {
     s.resolvedPath = await resolveTranscript(record, cliCfg);
     if (!s.resolvedPath) return 'unknown';
@@ -105,7 +116,6 @@ async function probeActivity(record, cliCfg) {
     s.resolvedPath = null;
     return 'unknown';
   }
-  const now = Date.now();
   if (mtimeMs !== s.lastMtimeMs) {
     s.lastMtimeMs = mtimeMs;
     s.lastChangedAt = now;
@@ -113,6 +123,17 @@ async function probeActivity(record, cliCfg) {
   return (now - s.lastChangedAt) < WORKING_WINDOW_MS ? 'working' : 'idle';
 }
+// Called from server.js's spawnCliSession onData hook. Cheap (timestamp
+// write); bound by how often the PTY emits, which is fine.
+function noteOutput(sessionId) {
+  let s = state.get(sessionId);
+  if (!s) {
+    s = { resolvedPath: null, lastMtimeMs: 0, lastChangedAt: 0, lastOutputAt: 0 };
+    state.set(sessionId, s);
+  }
+  s.lastOutputAt = Date.now();
+}
 function releaseSession(sessionId) { state.delete(sessionId); }
-module.exports = { probeActivity, releaseSession };
+module.exports = { probeActivity, noteOutput, releaseSession };

package/lib/config.js CHANGED Viewed

@@ -130,26 +130,20 @@ function mergeWithDefaults(partial) {
     if (existing) {
       existing.builtin = true;
       // Backfill defaults from the built-in template for any field the
-      // user's saved copy is missing — keeps old configs aligned with new
-      // schema additions (resumeIdArgs, type, etc.) without clobbering
-      // the user's customisations.
-      if (existing.resumeIdArgs == null) existing.resumeIdArgs = def.resumeIdArgs;
-      if (existing.newSessionIdArgs == null) existing.newSessionIdArgs = def.newSessionIdArgs;
-      // Drop the v0.x `resumeArgs` fallback — all builtins now have
-      // pre-assigned UUIDs (claude/copilot via flag, codex via seed) so
-      // resumeIdArgs always applies on resume.
+      // user's saved copy is missing OR has as an empty array. Empty
+      // arrays matter because users upgrading from a pre-0.15 config
+      // never wrote `newSessionIdArgs` (didn't exist), AND a partial
+      // 0.14→0.15 dev iteration shipped codex with `[]`. Treat both
+      // the same: a builtin with no template means "use the canonical
+      // one ccsm now knows about", since these fields are the
+      // integration contract with the upstream CLI — not user knobs.
+      const needsBackfill = (v) => v == null || (Array.isArray(v) && v.length === 0);
+      if (needsBackfill(existing.resumeIdArgs)) existing.resumeIdArgs = def.resumeIdArgs;
+      if (needsBackfill(existing.newSessionIdArgs)) existing.newSessionIdArgs = def.newSessionIdArgs;
+      // Drop the v0.x `resumeArgs` fallback — every builtin now has a
+      // pre-assigned UUID (claude/copilot via flag, codex via seed) so
+      // resumeIdArgs always applies on resume; the field is dead weight.
       delete existing.resumeArgs;
-      // Special-case codex: an unreleased earlier iteration of this
-      // schema shipped `newSessionIdArgs: []` for codex. The seeded-file
-      // trick (lib/codexSeed) now lets us pre-assign, so backfill the
-      // ['resume','<id>'] template over an empty array too.
-      if (existing.id === 'codex'
-          && Array.isArray(existing.newSessionIdArgs)
-          && existing.newSessionIdArgs.length === 0
-          && Array.isArray(def.newSessionIdArgs)
-          && def.newSessionIdArgs.length > 0) {
-        existing.newSessionIdArgs = def.newSessionIdArgs;
-      }
       if (!existing.type) existing.type = def.type;
     } else {
       out.clis.unshift({ ...def });

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "@bakapiano/ccsm",
-  "version": "0.15.0",
+  "version": "0.15.2",
   "description": "Claude Code Session Manager — Windows web UI to manage many concurrent claude sessions: live list, snapshot/restore, focus existing window, new session in an isolated workspace with repo clones",
   "license": "MIT",
   "main": "server.js",

package/public/css/sidebar.css CHANGED Viewed

@@ -568,11 +568,52 @@ body.is-resizing-sidebar * {
 }
 .tree-session.is-running .tree-dot::after {
   background: var(--green);
+  /* Soft halo so the dot reads as "alive" even from across the sidebar.
+     Box-shadow uses currentColor isn't ideal here (the dot itself uses
+     background, not color), so we hardcode each state's halo color
+     below. */
+  box-shadow: 0 0 0 0 rgba(74, 138, 74, 0.55);
+  animation: tree-dot-breathe-idle 2.8s ease-in-out infinite;
 }
 /* Working = CLI is actively writing to its transcript (i.e. thinking
-   or printing tokens). Idle stays green; working flips to blue. */
+   or printing tokens). Idle stays green + slow breathe; working flips
+   to blue + faster, more obvious breathe. */
 .tree-session.is-running.is-working .tree-dot::after {
   background: var(--blue, #4a73a5);
+  box-shadow: 0 0 0 0 rgba(74, 115, 165, 0.65);
+  animation: tree-dot-breathe-working 1.4s ease-in-out infinite;
+}
+@keyframes tree-dot-breathe-idle {
+  0%, 100% {
+    box-shadow: 0 0 0 0 rgba(74, 138, 74, 0.45);
+    opacity: 0.85;
+  }
+  50% {
+    box-shadow: 0 0 0 4px rgba(74, 138, 74, 0);
+    opacity: 1;
+  }
+}
+@keyframes tree-dot-breathe-working {
+  0%, 100% {
+    box-shadow: 0 0 0 0 rgba(74, 115, 165, 0.65);
+    opacity: 0.9;
+    transform: scale(1);
+  }
+  50% {
+    box-shadow: 0 0 0 5px rgba(74, 115, 165, 0);
+    opacity: 1;
+    transform: scale(1.15);
+  }
+}
+/* Respect users who've asked for less motion — keep the color signal
+   but drop the pulse. */
+@media (prefers-reduced-motion: reduce) {
+  .tree-session.is-running .tree-dot::after,
+  .tree-session.is-running.is-working .tree-dot::after {
+    animation: none;
+    box-shadow: none;
+  }
 }
 .tree-label {
   flex: 1;

package/public/js/pages/AboutPage.js CHANGED Viewed

@@ -65,8 +65,16 @@ function UpgradeCard() {
     if (!info?.updateAvailable) return;
     setUpgrading(true);
     try {
-      await api('POST', '/api/upgrade', { target: 'latest' });
+      const r = await api('POST', '/api/upgrade', { target: 'latest' });
       setToast(`upgrading to v${info.latest} · backend will restart`);
+      if (r?.closeFrontend) {
+        // Backend will respawn with a fresh browser window — close this
+        // one so the user isn't stuck on the OfflineBanner during the
+        // upgrade window. window.close() only works when the window was
+        // script-opened (Edge --app=, our spawned browser); regular tabs
+        // ignore it silently, which is fine (OfflineBanner takes over).
+        setTimeout(() => { try { window.close(); } catch {} }, 400);
+      }
     } catch (e) {
       setUpgrading(false);
       setToast(e.message, 'error');

package/public/js/pages/ConfigurePage.js CHANGED Viewed

@@ -431,8 +431,16 @@ function RestartButton() {
     if (!ok) return;
     setBusy(true);
     try {
-      await restartBackend();
+      const r = await restartBackend();
       setToast('restarting backend…');
+      if (r?.closeFrontend) {
+        // Backend respawn will pop a fresh browser window — close this
+        // one so the user isn't stuck on the OfflineBanner during the
+        // ~3s downtime. window.close() only fires in script-opened
+        // windows (Edge --app=); regular tabs ignore it and stay open,
+        // which is the right behavior for them.
+        setTimeout(() => { try { window.close(); } catch {} }, 400);
+      }
     } catch (e) {
       setBusy(false);
       setToast(e.message, 'error');

package/scripts/restart-helper.js CHANGED Viewed

@@ -62,7 +62,12 @@ function pidAlive(pid) {
   const isWin = process.platform === 'win32';
   const ccsmCmd = isWin ? 'ccsm.cmd' : 'ccsm';
-  const childEnv = { ...process.env, CCSM_NO_BROWSER: '1' };
+  // Inherit env but DROP CCSM_NO_BROWSER so the respawned server pops a
+  // fresh browser window — the frontend that triggered the restart
+  // called window.close() in parallel, and the new window takes its
+  // place without the OfflineBanner gap.
+  const childEnv = { ...process.env };
+  delete childEnv.CCSM_NO_BROWSER;
   let exe, exeArgs;
   if (isWin) {
     exe = process.env.ComSpec || 'cmd.exe';

package/scripts/upgrade-helper.js CHANGED Viewed

@@ -126,7 +126,12 @@ function pidAlive(pid) {
   const ccsmCmd = installPrefix
     ? (isWin ? path.join(installPrefix, 'ccsm.cmd') : path.join(installPrefix, 'bin', 'ccsm'))
     : (isWin ? 'ccsm.cmd' : 'ccsm');
-  const childEnv = { ...process.env, CCSM_NO_BROWSER: '1' };
+  // Drop CCSM_NO_BROWSER so the post-upgrade server pops a fresh window
+  // — the frontend that triggered the upgrade window.close()'d in
+  // parallel, and the new window takes its place. Skips the
+  // OfflineBanner gap that used to bridge the upgrade.
+  const childEnv = { ...process.env };
+  delete childEnv.CCSM_NO_BROWSER;
   let exe, exeArgs;
   if (isWin) {
     exe = process.env.ComSpec || 'cmd.exe';

package/server.js CHANGED Viewed

@@ -221,7 +221,10 @@ function spawnCliSession({ cli, cwd, sessionId, meta, extraArgs = [] }) {
     cwd,
     env,
     meta: { ...meta, cliId: cli.id, cliName: cli.name },
-    onData: () => { persistedSessions.touch(sessionId).catch(() => {}); },
+    onData: () => {
+      persistedSessions.touch(sessionId).catch(() => {});
+      try { require('./lib/cliActivity').noteOutput(sessionId); } catch {}
+    },
     onExit: ({ exitCode }) => {
       persistedSessions.markExited(sessionId, exitCode).catch(() => {});
     },
@@ -926,7 +929,7 @@ app.post('/api/restart', asyncH(async (_req, res) => {
   restartInFlight = true;
   if (process.env.CCSM_DEV === '1') {
-    res.json({ ok: true, started: true, mode: 'dev' });
+    res.json({ ok: true, started: true, mode: 'dev', closeFrontend: false });
     setImmediate(() => gracefulShutdown('restart (dev)'));
     return;
   }
@@ -941,7 +944,11 @@ app.post('/api/restart', asyncH(async (_req, res) => {
     return res.status(500).json({ error: `helper copy failed: ${e.message}` });
   }
   const args = [helperTmp, String(currentPort), String(process.pid)];
-  res.json({ ok: true, started: true, helper: helperTmp });
+  // closeFrontend asks the calling tab to window.close() itself — the
+  // helper will respawn ccsm WITHOUT CCSM_NO_BROWSER, so a fresh window
+  // pops up once the new backend is listening. Net effect: the user
+  // never sees the OfflineBanner during a restart.
+  res.json({ ok: true, started: true, helper: helperTmp, closeFrontend: true });
   setImmediate(() => {
     const { spawn } = require('node:child_process');
@@ -1081,7 +1088,7 @@ app.post('/api/upgrade', asyncH(async (req, res) => {
   }
   const args = [helperTmp, target, String(currentPort), String(process.pid), installPrefix, respawn];
-  res.json({ ok: true, started: true, target, helper: helperTmp });
+  res.json({ ok: true, started: true, target, helper: helperTmp, closeFrontend: true });
   // Flush response, then spawn helper detached and gracefulShutdown so
   // the helper's npm install isn't fighting our open file handles.