agileflow 4.0.0-alpha.17 → 4.0.0-alpha.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agileflow",
-  "version": "4.0.0-alpha.17",
+  "version": "4.0.0-alpha.18",
   "description": "AI-driven agile development toolkit for Claude Code — skills-first architecture with opt-in plugins (v4)",
   "keywords": [
     "agile",

package/src/cli/commands/launch.js

@@ -1182,25 +1182,29 @@ async function maybeOfferAutoRestore(prefs) {
     if (ap !== bp) return bp - ap;
     return 0;
   });
-  // Synthetic toggles at the top of the picker for users with many
-  // saved sessions — checking one of them rewrites the selection in
-  // bulk after submit (clack's multiselect doesn't natively support
-  // a "select all" shortcut, but synthetic entries are the lightest
-  // path and stay accessible via keyboard).
-  const SELECT_ALL = "__select_all__";
-  const DESELECT_ALL = "__deselect_all__";
+  // Smart toggle at the top of the picker: one entry that flips the
+  // current selection state. If everything's selected, checking it
+  // deselects all; if anything's unselected, checking it selects all.
+  // Visually separated from session entries by a dash divider line
+  // so it doesn't blend in with real options.
+  const TOGGLE_ALL = "__toggle_all__";
+  const DIVIDER = "__divider__";
   const allNames = sorted.map((s) => s.name);
   const sessionOptions = sorted.map((s) => ({
     value: s.name,
-    label: `${s.pinned ? "★ " : "  "}${s.name}`,
+    label: `${s.pinned ? "* " : "  "}${s.name}`,
     hint: `${s.cli} — ${s.cwd}${s.worktree && s.worktree.branch ? ` [wt ${s.worktree.branch}]` : ""}`,
   }));
   const options = [
-    { value: SELECT_ALL, label: "✅ Select all", hint: "check every session" },
     {
-      value: DESELECT_ALL,
-      label: "⬜ Deselect all",
-      hint: "uncheck every session",
+      value: TOGGLE_ALL,
+      label: "[ select all / deselect all ]",
+      hint: "toggles every session below",
+    },
+    {
+      value: DIVIDER,
+      label: "─────────────────────────────",
+      hint: "",
     },
     ...sessionOptions,
   ];
@@ -1228,14 +1232,18 @@ async function maybeOfferAutoRestore(prefs) {
   }
   /** @type {string[]} */
   let chosen = Array.isArray(selection) ? selection : [];
-  // Resolve the synthetic select-all / deselect-all toggles. If both
-  // were checked, select-all wins (the safer "do more" interpretation);
-  // either way, strip the synthetic entries from the final list.
-  const wantSelectAll = chosen.includes(SELECT_ALL);
-  const wantDeselectAll = chosen.includes(DESELECT_ALL);
-  chosen = chosen.filter((v) => v !== SELECT_ALL && v !== DESELECT_ALL);
-  if (wantSelectAll) chosen = [...allNames];
-  else if (wantDeselectAll) chosen = [];
+  // Strip the synthetic divider always (it's never a real choice).
+  // Resolve the toggle: if the user checked it, flip the current
+  // selection state — everything selected goes to nothing, anything
+  // partial or empty goes to everything.
+  const toggled = chosen.includes(TOGGLE_ALL);
+  chosen = chosen.filter((v) => v !== TOGGLE_ALL && v !== DIVIDER);
+  if (toggled) {
+    const allSelected =
+      chosen.length === allNames.length &&
+      allNames.every((n) => chosen.includes(n));
+    chosen = allSelected ? [] : [...allNames];
+  }
   if (chosen.length === 0) {
     prompts.outro(
       "Skipped. Run `agileflow launch restore` later to bring them back.",

package/src/runtime/launch/tmux.js

@@ -241,24 +241,53 @@ function applyTabFormat(sessionName, runner, opts = {}) {
     `#[bg=${PILL_BG},fg=${ACCENT}] #h ` +
     `#[fg=${PILL_BG},bg=${BG}]${HALF_ROUND_CLOSE}`;
 
-  // Apply each option and collect failures. If ANYTHING fails we surface
-  // it on stderr so users debugging "why doesn't my tab strip look right"
-  // can see the tmux complaint instead of staring at default formatting.
+  // Apply each option with the correct tmux scope. Session-scope opts
+  // (status-style, status-left/right, status-justify) take
+  // `-t <session>`. Window-scope opts (window-status-format,
+  // window-status-current-format, window-status-separator) take
+  // `-wg` so every window in every session picks them up. Earlier
+  // code applied ALL options with `-t session`, which made tmux
+  // silently ignore the window-scope ones — the visible symptom
+  // was tmux's default green status bar surviving on every session.
   const ops = [
-    ["status-style", `bg=${BG},fg=${theme.inactiveFg}`],
-    ["status-justify", "centre"],
-    ["status-left", statusLeft],
-    ["status-left-length", "100"],
-    ["status-right", statusRight],
-    ["status-right-length", "100"],
-    ["window-status-separator", ""],
-    ["window-status-format", inactiveFormat],
-    ["window-status-current-format", activeFormat],
+    {
+      scope: "session",
+      option: "status-style",
+      value: `bg=${BG},fg=${theme.inactiveFg}`,
+    },
+    { scope: "session", option: "status-justify", value: "centre" },
+    { scope: "session", option: "status-left", value: statusLeft },
+    { scope: "session", option: "status-left-length", value: "100" },
+    { scope: "session", option: "status-right", value: statusRight },
+    { scope: "session", option: "status-right-length", value: "100" },
+    // Number windows from 1 so Alt+1 maps to the first tab (matches
+    // Chrome's Ctrl+1 mental model). Tmux default is base-index 0,
+    // which means Alt+1 with no other tabs open does nothing.
+    { scope: "session", option: "base-index", value: "1" },
+    // Keep tab indices contiguous after a close — without this,
+    // closing window 2 leaves indices 1, 3, 4 and Alt+2 becomes
+    // dead. tmux renumbers on close so Alt+1..N always works.
+    { scope: "session", option: "renumber-windows", value: "on" },
+    { scope: "window-global", option: "window-status-separator", value: "" },
+    {
+      scope: "window-global",
+      option: "window-status-format",
+      value: inactiveFormat,
+    },
+    {
+      scope: "window-global",
+      option: "window-status-current-format",
+      value: activeFormat,
+    },
   ];
   let lastResult = { status: 0, stderr: "" };
   const failures = [];
-  for (const [option, value] of ops) {
-    const r = runner.runSync(["set-option", "-t", sessionName, option, value]);
+  for (const { scope, option, value } of ops) {
+    const args =
+      scope === "session"
+        ? ["set-option", "-t", sessionName, option, value]
+        : ["set-option", "-wg", option, value];
+    const r = runner.runSync(args);
     lastResult = r;
     if (r.status !== 0) {
       failures.push({ option, stderr: (r.stderr || "").trim() });