@pellux/goodvibes-tui 1.16.1 → 1.18.0

package/src/input/commands/recall-files-git.ts

@@ -1,57 +1,19 @@
 /**
- * recall-files-git.ts — the synchronous git seam for memory file projection.
+ * recall-files-git.ts — the git seam for memory file projection.
  *
- * `projectMemoryToFiles(records, dir, { git })` (SDK, platform/state) is a
- * pure synchronous function and its `MemoryProjectionGit` seam is declared
- * synchronous (`add(dir): void`, `commit(dir, message): void`) — it throws on
- * failure rather than returning a result. The TUI's existing git wrapper,
- * `GitService` (platform/git), is async (wraps simple-git), so it cannot
- * satisfy this seam directly. `GitService`'s own static helpers
- * (initRepo/isGitRepo/getRepoRoot) ARE synchronous, using `Bun.spawnSync`
- * directly (see service.ts) — this file follows the exact same convention
- * for `add`/`commit` so the seam matches the SDK's synchronous contract.
+ * The SDK owns the production seam now (`createMemoryProjectionGit`,
+ * platform/state): it shells out to the system git, commits with a neutral
+ * projection identity (never the operator's), tolerates an unchanged tree,
+ * and — through `projectMemoryToFiles`'s ownership gate — only ever commits
+ * to a repository whose toplevel IS the projection directory, initializing
+ * one there when needed. A projection directory nested inside some other
+ * checkout can never pollute that checkout. This module is a thin local
+ * seam factory so command code keeps a single import site.
  */
+import { createMemoryProjectionGit } from '@pellux/goodvibes-sdk/platform/state';
 import type { MemoryProjectionGit } from '@pellux/goodvibes-sdk/platform/state';
-import { GitService } from '@pellux/goodvibes-sdk/platform/git';
 
-function decode(bytes: Uint8Array | undefined): string {
-  return bytes ? new TextDecoder().decode(bytes).trim() : '';
-}
-
-/**
- * Build a synchronous git seam rooted at `repoRoot`. `commit()` treats a
- * "nothing to commit" exit as a benign no-op (not an error) so
- * `/recall files sync` stays safely re-runnable when the projection files
- * did not actually change since the last sync — any OTHER commit failure
- * (no git identity configured, hook rejection, etc.) still throws, since the
- * seam's contract has no way to report partial failure back to the caller
- * other than throwing.
- */
-export function createSyncGitSeam(repoRoot: string): MemoryProjectionGit {
-  return {
-    add(dir: string): void {
-      const result = Bun.spawnSync(['git', 'add', dir], { cwd: repoRoot });
-      if (result.exitCode !== 0) {
-        throw new Error(`git add failed: ${decode(result.stderr) || 'unknown error'}`);
-      }
-    },
-    commit(dir: string, message: string): void {
-      const result = Bun.spawnSync(['git', 'commit', '-m', message, '--', dir], { cwd: repoRoot });
-      if (result.exitCode === 0) return;
-      const output = `${decode(result.stdout)} ${decode(result.stderr)}`;
-      if (/nothing to commit/i.test(output)) return; // benign — no new/changed files this run
-      throw new Error(`git commit failed: ${output.trim() || 'unknown error'}`);
-    },
-  };
-}
-
-/**
- * Resolve the git seam for `dir`, or null when `dir` is not inside a git
- * repository. Never auto-initializes a repository — that is a much bigger
- * side effect than a memory sync command should take unasked.
- */
-export function resolveSyncGitSeam(dir: string): { seam: MemoryProjectionGit; repoRoot: string } | null {
-  const repoRoot = GitService.getRepoRoot(dir);
-  if (!repoRoot) return null;
-  return { seam: createSyncGitSeam(repoRoot), repoRoot };
+/** The projection git seam (see module note — SDK-owned behavior). */
+export function createSyncGitSeam(): MemoryProjectionGit {
+  return createMemoryProjectionGit();
 }

package/src/input/commands/recall-files.ts

@@ -39,7 +39,7 @@ import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { requireShellPaths } from './runtime-services.ts';
 import { getMemorySpine } from './recall-query.ts';
 import { readMemoryProjectionDir } from './recall-files-config.ts';
-import { resolveSyncGitSeam } from './recall-files-git.ts';
+import { createSyncGitSeam } from './recall-files-git.ts';
 
 const PROJECTION_LIST_LIMIT = 5000;
 
@@ -58,19 +58,16 @@ export async function handleRecallFilesSync(args: string[], context: CommandCont
 
   const dir = resolveProjectionDir(args, context);
   const records = await memory.list({ limit: PROJECTION_LIST_LIMIT });
-  // Resolve the git seam from the project working directory, not `dir` itself
-  // — `dir` may not exist yet on a first sync, and `git -C <dir>` requires the
-  // directory to already exist. The working directory is always real.
-  const gitSeam = resolveSyncGitSeam(requireShellPaths(context).workingDirectory);
 
   try {
-    const report = projectMemoryToFiles(records, dir, gitSeam ? { git: gitSeam.seam } : {});
+    // The SDK's ownership gate commits only to a repository whose toplevel IS
+    // the projection directory (initializing one there when needed) — never
+    // to a checkout the directory merely sits inside.
+    const report = projectMemoryToFiles(records, dir, { git: createSyncGitSeam() });
     context.print(`[recall] Projected ${report.written.length} record(s) to ${report.dir}`);
-    if (gitSeam) {
-      context.print(report.committed ? `  committed in ${gitSeam.repoRoot}` : '  not committed (git seam did not run)');
-    } else {
-      context.print(`  not committed: ${dir} is not inside a git repository`);
-    }
+    context.print(report.committed
+      ? `  committed in ${report.dir} (the projection's own repository)`
+      : '  not committed (git seam did not run)');
   } catch (error) {
     context.print(`[recall] Projection sync failed: ${summarizeError(error)}`);
   }

package/src/input/commands/recall-review.ts

@@ -112,9 +112,9 @@ export function handleRecallExplain(args: string[], context: CommandContext): vo
  * `exportState()` returns the same full `AgentRecord[]` (it's what
  * compactConversation() already uses for the same reason).
  *
- * With NO agent id (wo805 wiring), renders the MAIN interactive session's own
+ * With NO agent id (wiring), renders the MAIN interactive session's own
  * per-turn injection ring — `Orchestrator.getTurnInjections()`, threaded onto
- * the command context as `session.getMainSessionTurnInjections`. As of wo805 the
+ * the command context as `session.getMainSessionTurnInjections`. As of the
  * main session DOES route through the passive-injection engine, so this is the
  * honest main-session default (previously there was no record source, so the
  * no-id path could only print a usage hint). When the accessor isn't wired

package/src/input/commands/relay-runtime.ts

@@ -47,7 +47,7 @@ async function renderStatus(ctx: CommandContext): Promise<void> {
 async function renderPair(ctx: CommandContext): Promise<void> {
   const cfg = relayConfigured(ctx);
   if (!cfg.enabled || !cfg.flagOn) {
-    ctx.print('Relay is disabled — enable relay.enabled and the relay-connect feature flag first (see /config relay, /flags).');
+    ctx.print('Relay is disabled — turn on relay.enabled in /config relay first.');
     return;
   }
   const externalServices = ctx.platform.externalServices;

package/src/input/commands/remote-runtime.ts

@@ -48,7 +48,7 @@ export function registerRemoteRuntimeCommands(registry: CommandRegistry): void {
     async handler(args, ctx) {
       if (args.length === 0) {
         if (ctx.openModal) {
-          ctx.openModal('remote-modal'); // W6.1: remote panel -> config modal
+          ctx.openModal('remote-modal'); // remote panel -> config modal
           return;
         }
         ctx.print('Remote surface is not available in this runtime.');

package/src/input/commands/runtime-services.ts

@@ -104,8 +104,8 @@ export function openCommandPanel(
 }
 
 /**
- * W6.1 (group-B migration): open a config-modal surface by name via the
- * ctx.openModal seam (ui-openers wires it; the WO-A host replaces the interim
+ * (group-B migration): open a config-modal surface by name via the
+ * ctx.openModal seam (ui-openers wires it; the host replaces the interim
  * "not available yet" implementation with real dispatch). Front-doors for
  * panels that migrated to modals call this instead of openCommandPanel — the
  * modal is the surface's new home. Stubbed in tests by setting ctx.openModal.
@@ -256,7 +256,7 @@ export function requireProviderApi(context: CommandContext): ProviderApi {
 }
 
 /** Result of a manual /compact run: the SDK's CompactionEvent, plus an
- * out-of-band quality score (W5.4/B28) when one could be computed. */
+ * out-of-band quality score () when one could be computed. */
 export interface CompactionRunResult {
   readonly event: CompactionEvent;
   readonly qualityScore: CompactionQualityScore | null;
@@ -328,7 +328,7 @@ export async function compactConversation(context: CommandContext): Promise<Comp
   const eventAfter = getLastCompactionEvent();
   // Return the new event only if it differs from the one recorded before the call.
   if (eventAfter !== null && eventAfter !== eventBefore) {
-    // W5.4/B28: score this run out-of-band (see compaction-quality.ts for why
+    //: score this run out-of-band (see compaction-quality.ts for why
     // this can't just subscribe to the SDK's own CompactionManager pipeline)
     // and keep the score keyed by this event's timestamp for /compact-history.
     const qualityScore = scoreCompactionRun({

package/src/input/commands/schedule-runtime.ts

@@ -10,6 +10,7 @@ import type { AutomationManager } from '@pellux/goodvibes-sdk/platform/automatio
 import type { AutomationJob } from '@pellux/goodvibes-sdk/platform/automation';
 import type { AutomationScheduleDefinition } from '@pellux/goodvibes-sdk/platform/automation';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
+import { buildAutomationEmptyState } from '@pellux/goodvibes-sdk/platform/runtime/feature-announcements';
 import { parseNaturalLanguageSchedule } from './schedule-nl.ts';
 import type {
   AutomationExecutionPolicy,
@@ -106,13 +107,19 @@ export function registerScheduleRuntimeCommands(registry: CommandRegistry): void
       if (!sub || sub === 'list') {
         const jobs = manager.listJobs();
         if (jobs.length === 0) {
-          ctx.print(
-            'No automation jobs.\n'
-            + 'Use:\n'
-            + '  /schedule add cron "*/30 * * * *" "check build status"\n'
-            + '  /schedule add every 15m "summarize open PRs"\n'
-            + '  /schedule add at 2026-04-10T09:00:00 "send release reminder"'
-          );
+          // The SDK owns the presence + title of the how-to-create-your-first-
+          // routine empty state (shown while automation is on and idle). Its
+          // body points at a command name this registry doesn't have, so we
+          // render its title with THIS surface's real /schedule commands — a
+          // command pointer that actually exists in the registry.
+          const emptyState = buildAutomationEmptyState({ enabled: true, routineCount: jobs.length });
+          ctx.print([
+            emptyState ? emptyState.title : 'No automation jobs.',
+            'Automation is on and idle. Create your first routine with:',
+            '  /schedule add cron "*/30 * * * *" "check build status"',
+            '  /schedule add every 15m "summarize open PRs"',
+            '  /schedule add at 2026-04-10T09:00:00 "send release reminder"',
+          ].join('\n'));
           return;
         }
         const lines = ['Automation jobs:', ''];

package/src/input/commands/services-runtime.ts

@@ -16,7 +16,7 @@ export function registerServicesRuntimeCommands(registry: CommandRegistry): void
       const sub = args[0] ?? 'open';
       const shellPaths = requireShellPaths(ctx);
       if (sub === 'open' || sub === 'panel') {
-        ctx.openModal?.('services-modal'); // W6.1: services panel -> config modal
+        ctx.openModal?.('services-modal'); // services panel -> config modal
         return;
       }
       const svcRegistry = requireServiceRegistry(ctx);

package/src/input/commands/session-workflow.ts

@@ -89,7 +89,7 @@ export function reopenPanelsFromReturnContext(ctx: CommandContext, summary: Sess
   const reopened: string[] = [];
   const movedToModal: string[] = [];
   for (const panelId of summary.openPanels.slice(0, 4)) {
-    // W6.1 (the purge): a MIGRATE-TO-MODAL id has no panel to restore — a modal
+    // (the purge): a MIGRATE-TO-MODAL id has no panel to restore — a modal
     // is not part of the saved panel layout. Skip it (don't pop a modal
     // mid-resume) and note it once, rather than firing openModal + revealing an
     // empty workspace during resume.
@@ -283,7 +283,7 @@ export async function handleSessionWorkflowCommand(args: string[], ctx: CommandC
         },
       });
       // Hydrate the footer's token counters from the resumed (+ journal-replayed)
-      // history now, before ctx.renderRequest() below (W0.9).
+      // history now, before ctx.renderRequest() below.
       ctx.session.hydrateSessionUsage?.();
 
       if (meta.model) {

package/src/input/commands/settings-sync-runtime.ts

@@ -32,7 +32,7 @@ export function registerSettingsSyncRuntimeCommands(registry: CommandRegistry):
       const shellPaths = requireShellPaths(ctx);
       const controlPlaneConfigDir = ctx.platform.configManager.getControlPlaneConfigDir();
       const sub = (args[0] ?? '').toLowerCase();
-      // DEBT-5 item 3 (report-vs-modal front doors): settings-sync has a full
+      // item 3 (report-vs-modal front doors): settings-sync has a full
       // modal surface (settings-sync-modal), so the bare command now opens it
       // — the old bare/`review` transcript report moved to an explicit
       // `report` subcommand (scriptability preserved: /settings-sync report).
@@ -41,7 +41,7 @@ export function registerSettingsSyncRuntimeCommands(registry: CommandRegistry):
         return;
       }
       if (sub === 'panel' || sub === 'open') {
-        ctx.openModal?.('settings-sync-modal'); // W6.1: settings-sync panel -> config modal
+        ctx.openModal?.('settings-sync-modal'); // settings-sync panel -> config modal
         return;
       }
       if (sub === 'report' || sub === 'review') {

package/src/input/commands/share-runtime.ts

@@ -115,7 +115,7 @@ export function registerShareRuntimeCommands(registry: CommandRegistry): void {
       );
       // The exported document has no field for "this cost is a placeholder" —
       // omit `cost` entirely rather than embed a misleading 0 when the model
-      // never resolved to a real price (WO-315).
+      // never resolved to a real price.
       const costIsUnpriced = !isModelPriced(sessionModel);
       if (costIsUnpriced) {
         ctx.print(`Note: cost omitted from export — no pricing data for model "${sessionModel}".`);

package/src/input/commands/shell-core.ts

@@ -6,6 +6,7 @@ import { executeWriteQuit } from './quit-shared.ts';
 import { compactConversation, requireKeybindingsManager, requireProviderApi, requireSessionMemoryStore } from './runtime-services.ts';
 import { buildCompactionPreview, buildCompactionAfterNotice, buildPinUsageText, buildPinSuccessText } from '../../renderer/compaction-preview.ts';
 import { buildCompactionHistoryText } from '../../renderer/compaction-history-modal.ts';
+import { ensureProviderKeyThenSelect } from '../provider-key-intake.ts';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { logger } from '@pellux/goodvibes-sdk/platform/utils';
 
@@ -32,11 +33,15 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
         const modelId = args[0];
         try {
           const selected = await providerApi.selectModel(modelId);
-          ctx.session.runtime.model = selected.registryKey;
-          ctx.session.runtime.provider = selected.providerId;
-          ctx.platform.configManager.set('provider.model', selected.registryKey);
-          ctx.print(`Switched to model: ${selected.displayName} (${selected.providerId})`);
-          void providerApi.recordModelUsage(selected.registryKey).catch((err) => { logger.debug('model usage record failed', { err }); });
+          // Capture the key for an unconfigured provider inline before the
+          // selection is announced, so the switch lands ready to chat.
+          ensureProviderKeyThenSelect(ctx, selected.providerId, () => {
+            ctx.session.runtime.model = selected.registryKey;
+            ctx.session.runtime.provider = selected.providerId;
+            ctx.platform.configManager.set('provider.model', selected.registryKey);
+            ctx.print(`Switched to model: ${selected.displayName} (${selected.providerId})`);
+            void providerApi.recordModelUsage(selected.registryKey).catch((err) => { logger.debug('model usage record failed', { err }); });
+          });
         } catch (e) {
           ctx.print(`Error: ${summarizeError(e)}`);
         }
@@ -112,75 +117,48 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'help',
     aliases: ['h', '?'],
-    description: 'Show available commands and keyboard shortcuts',
-    argsHint: '[command]',
+    description: 'Browse every command with its description; picking one runs it',
     handler(_args, ctx) {
+      // Generated live from the command registry — never a hand-maintained
+      // list — so /help can never drift from the real command set. Category
+      // labels come from the same single source of truth as the generated
+      // reference and the palette (ctx.getCommandCategories; see its doc for
+      // why it is threaded through the context rather than imported here).
+      const commands = registry
+        .getAll()
+        .slice()
+        .sort((a, b) => a.name.localeCompare(b.name));
       if (ctx.openSelection) {
-        const items: SelectionItem[] = [
-          { id: '/model', label: '/model [id]', detail: 'Select LLM model', category: 'Model & Provider' },
-          { id: '/provider', label: '/provider [name]', detail: 'Switch provider', category: 'Model & Provider' },
-          { id: '/effort', label: '/effort [level]', detail: 'Reasoning effort (instant/low/medium/high)', category: 'Model & Provider' },
-          { id: '/context window', label: '/context window [<size>|clear]', detail: 'Show, set, or clear a custom context window for the current model', category: 'Model & Provider' },
-          { id: '/config', label: '/config [category|key]', detail: 'Open fullscreen configuration workspace', category: 'Config & Display' },
-          { id: '/debug', label: '/debug', detail: 'Toggle debug mode', category: 'Config & Display' },
-          { id: '/expand', label: '/expand [type]', detail: 'Expand blocks (all|thinking|tool|code)', category: 'Config & Display' },
-          { id: '/collapse', label: '/collapse [type]', detail: 'Collapse blocks', category: 'Config & Display' },
-          { id: '/bookmarks', label: '/bookmarks', detail: 'List bookmarked blocks', category: 'Config & Display' },
-          { id: '/clear', label: '/clear', detail: 'Clear display (keep context)', category: 'Conversation' },
-          { id: '/reset', label: '/reset', detail: 'Clear display + context', category: 'Conversation' },
-          { id: '/compact', label: '/compact', detail: 'Summarize to free context', category: 'Conversation' },
-          { id: '/export', label: '/export [file]', detail: 'Export as markdown', category: 'Conversation' },
-          { id: '/title', label: '/title [text]', detail: 'Show or set title', category: 'Conversation' },
-          { id: '/save', label: '/save [name]', detail: 'Save session', category: 'Conversation' },
-          { id: '/load', label: '/load <name>', detail: 'Load session', category: 'Conversation' },
-          { id: '/sessions', label: '/sessions', detail: 'List saved sessions', category: 'Conversation' },
-          { id: '/session', label: '/session', detail: 'Current session info', category: 'Conversation' },
-          { id: '/session list', label: '/session list', detail: 'List all sessions', category: 'Conversation' },
-          { id: '/session rename', label: '/session rename <name>', detail: 'Rename current session', category: 'Conversation' },
-          { id: '/resume', label: '/resume [id]', detail: 'Resume a previous session (picker when no id)', category: 'Conversation' },
-          { id: '/session resume', label: '/session resume <id>', detail: 'Load and resume a session', category: 'Conversation' },
-          { id: '/session fork', label: '/session fork [name]', detail: 'Fork current session to new ID', category: 'Conversation' },
-          { id: '/session save', label: '/session save [name]', detail: 'Force-save current session', category: 'Conversation' },
-          { id: '/session info', label: '/session info [id]', detail: 'Show session details', category: 'Conversation' },
-          { id: '/session export', label: '/session export <id> [format]', detail: 'Export session as markdown/text', category: 'Conversation' },
-          { id: '/session search', label: '/session search <query>', detail: 'Search across all sessions', category: 'Conversation' },
-          { id: '/session delete', label: '/session delete <id>', detail: 'Delete a session', category: 'Conversation' },
-          { id: '/undo', label: '/undo', detail: 'Remove last turn', category: 'Conversation' },
-          { id: '/redo', label: '/redo', detail: 'Restore undone turn', category: 'Conversation' },
-          { id: '/retry', label: '/retry [text]', detail: 'Re-send last message', category: 'Conversation' },
-          { id: '/fork', label: '/fork [name]', detail: 'Snapshot conversation as a named branch', category: 'Conversation' },
-          { id: '/branch', label: '/branch [name]', detail: 'List branches or switch to one', category: 'Conversation' },
-          { id: '/merge', label: '/merge <name>', detail: 'Append messages from a branch', category: 'Conversation' },
-          { id: '/template', label: '/template', detail: 'Browse templates', category: 'Templates' },
-          { id: '/template save', label: '/template save <name>', detail: 'Save prompt as template', category: 'Templates' },
-          { id: '/template use', label: '/template use <name>', detail: 'Execute template', category: 'Templates' },
-          { id: '/tools', label: '/tools', detail: 'List available tools', category: 'Tools & System' },
-          { id: '/search', label: '/search <query> [--limit <n>]', detail: 'Search the web and show ranked results', category: 'Tools & System' },
-          { id: '/imagine', label: '/imagine <prompt>', detail: 'Generate an image from a prompt', category: 'Tools & System' },
-          { id: '/paste', label: '/paste', detail: 'Insert clipboard text or image into the prompt', category: 'Tools & System' },
-          { id: '/shortcuts', label: '/shortcuts', detail: 'View keyboard shortcuts reference', category: 'Tools & System' },
-          { id: '/commands', label: '/commands', detail: 'Browse all commands in a scrollable list', category: 'Tools & System' },
-          { id: '/secrets', label: '/secrets set|link|get|test|list|delete', detail: 'Manage encrypted and provider-backed secrets', category: 'Tools & System' },
-          { id: '/help', label: '/help', detail: 'This help', category: 'Tools & System' },
-          { id: '/quit', label: '/quit', detail: 'Exit', category: 'Tools & System' },
-          { id: '/wq', label: '/wq', detail: 'Commit all git changes and then exit', category: 'Tools & System' },
-        ];
-        // UX-C: every item here is a command, and picking one RUNS it — label
+        const categoryByName = ctx.getCommandCategories?.() ?? new Map<string, string>();
+        const items: SelectionItem[] = commands.map((cmd) => {
+          const argsHint = cmd.argsHint ?? cmd.usage;
+          const aliases = cmd.aliases ?? [];
+          return {
+            id: cmd.name,
+            label: argsHint ? `/${cmd.name} ${argsHint}` : `/${cmd.name}`,
+            detail: aliases.length > 0 ? `${cmd.description} [aka ${aliases.map((a) => `/${a}`).join(', ')}]` : cmd.description,
+            category: categoryByName.get(cmd.name) ?? 'Other',
+          };
+        });
+        // every item here is a command, and picking one RUNS it — label
         // the verb "Run" (matching the slash-command palette) instead of the
         // generic "Select" default.
         ctx.openSelection('Help  —  Commands', items, { allowSearch: true, primaryVerbLabel: 'Run' }, (result) => {
           if (!result) return;
-          const command = result.item.id;
-          if (command.startsWith('/')) {
-            const parts = command.slice(1).trim().split(/\s+/);
-            const name = parts[0];
-            const cmdArgs = parts.slice(1);
-            void (ctx.executeCommand?.(name, cmdArgs) ?? registry.execute(name, cmdArgs, ctx));
-          }
+          const name = result.item.id;
+          void (ctx.executeCommand?.(name, []) ?? registry.execute(name, [], ctx));
         });
         return;
       }
-      ctx.print('Use /help to open the help modal. Commands: /model, /provider, /config, /template, /tools, /paste, /sessions, /bookmarks, /save, /load, /undo, /redo, /retry, /clear, /reset, /compact, /export, /title, /effort, /expand, /collapse, /debug, /quit, /wq');
+      ctx.print(
+        [
+          'Commands:',
+          ...commands.map((cmd) => {
+            const argsHint = cmd.argsHint ?? cmd.usage;
+            return `  /${cmd.name}${argsHint ? ` ${argsHint}` : ''} — ${cmd.description}`;
+          }),
+        ].join('\n'),
+      );
     },
   });
 
@@ -214,7 +192,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
     description: 'Summarize conversation to free context window',
     async handler(_args, ctx) {
       const messages = ctx.session.conversationManager.getMessagesForLLM();
-      // W5.4 (wo803): contextWindow IS reachable from CommandContext — the same
+      // contextWindow IS reachable from CommandContext — the same
       // call compactConversation() (runtime-services.ts) already makes to scale
       // its own behaviour by window size. Previously hardcoded to 0 here, which
       // silently suppressed buildCompactionPreview()'s capacity-% clause even
@@ -230,7 +208,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
       const result = await compactConversation(ctx);
       if (result) {
         // Post-compact notice: uses real CompactionEvent figures, plus an
-        // out-of-band quality-score grade line when one was computed (W5.4/B28).
+        // out-of-band quality-score grade line when one was computed ().
         ctx.print(buildCompactionAfterNotice({ event: result.event, pinnedMemoryCount, qualityScore: result.qualityScore }));
       } else {
         ctx.print('[Context] Compact complete.');
@@ -300,7 +278,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
       ctx.session.runtime.debugMode = !ctx.session.runtime.debugMode;
       ctx.print(`Debug mode: ${ctx.session.runtime.debugMode ? 'ON' : 'OFF'}`);
       // Honest quiet counter for direct terminal writes the guard intercepted
-      // (routed here instead of spamming the transcript). (UX-B item 1a.)
+      // (routed here instead of spamming the transcript). (item 1a.)
       const intercepted = ctx.session.runtime.terminalWritesIntercepted ?? 0;
       if (intercepted > 0) {
         ctx.print(`Terminal writes intercepted this session: ${intercepted} (details in the activity log)`);

package/src/input/commands/subscription-runtime.ts

@@ -80,7 +80,7 @@ export function registerSubscriptionRuntimeCommands(registry: CommandRegistry):
     async handler(args, ctx) {
       const shellPaths = requireShellPaths(ctx);
       if (args.length === 0 && ctx.openModal) {
-        ctx.openModal('subscription-modal'); // W6.1: subscription panel -> config modal
+        ctx.openModal('subscription-modal'); // subscription panel -> config modal
         return;
       }
       const sub = (args[0] ?? 'review').toLowerCase();