@north-light/crouter 0.3.226 → 0.3.228

package/dist/commands/sys/settings-shell.js

@@ -12,6 +12,7 @@ import { KeybindingsPanel } from "./panels/keybindings-panel.js";
 import { ModelsPanel } from "./panels/models-panel.js";
 import { PAGE_CHROME_ROWS, columns, padAnsi, pageFrame, theme, } from "../../core/tui/panel.js";
 import { PrefsPanel } from "./panels/prefs-panel.js";
+import { ProfilesPanel } from "./panels/profiles-panel.js";
 import { ProviderPanel } from "./panels/provider-panel.js";
 /** Rail column width, including its badge gutter. */
 const RAIL_WIDTH = 22;
@@ -26,6 +27,7 @@ export class SettingsShell {
     models;
     keybindings;
     brokerLimits;
+    profiles;
     destinations;
     index = 0;
     /** The rail owns the arrow keys until you step into a panel. */
@@ -35,7 +37,7 @@ export class SettingsShell {
     cachedHeight;
     cachedLines;
     onCancel;
-    constructor(tui, agentDir, runtime, registry, bindings, rawKeybindings, terminalRows, rawTmuxPassthrough = undefined) {
+    constructor(tui, agentDir, runtime, registry, bindings, rawKeybindings, terminalRows, rawTmuxPassthrough = undefined, initial) {
         this.tui = tui;
         this.bindings = bindings;
         this.terminalRows = terminalRows;
@@ -46,6 +48,10 @@ export class SettingsShell {
         this.models.onChange = invalidate;
         this.keybindings = new KeybindingsPanel(bindings, rawKeybindings, rawTmuxPassthrough, terminalRows, invalidate);
         this.brokerLimits = new BrokerLimitsPanel(invalidate);
+        // Profile purview and pins read against the directory the page was opened
+        // in — the same directory the startup menu was deciding for when `v` brought
+        // the user here.
+        this.profiles = new ProfilesPanel(process.cwd(), invalidate);
         const prefs = (group, title, subtitle) => new PrefsPanel(group, title, subtitle, invalidate);
         this.destinations = [
             {
@@ -72,6 +78,12 @@ export class SettingsShell {
                 label: "Display",
                 panel: prefs("display", "Display", "How an attached viewer renders a conversation."),
             },
+            {
+                id: "profiles",
+                group: "Workspace",
+                label: "Profiles",
+                panel: this.profiles,
+            },
             {
                 id: "panes",
                 group: "Workspace",
@@ -91,6 +103,16 @@ export class SettingsShell {
                 panel: prefs("updates", "Updates", "How crouter checks for and applies new releases."),
             },
         ];
+        // A deep link lands ON its destination with the rail already stepped out of:
+        // the caller named where they wanted to be, not where to start navigating
+        // from.
+        if (initial !== undefined) {
+            const at = this.destinations.findIndex((destination) => destination.id === initial);
+            if (at >= 0) {
+                this.index = at;
+                this.railFocused = false;
+            }
+        }
     }
     // --- focus and navigation -------------------------------------------------
     set focused(value) {
@@ -155,6 +177,8 @@ export class SettingsShell {
                 return this.models.isPickerOpen();
             case "keybindings":
                 return this.keybindings.takesPriorityInput;
+            case "profiles":
+                return this.profiles.ownsInput(data);
             case "broker-limits":
                 // Threshold fields take digits; a remapped destination gesture must
                 // never swallow a number the user is typing into a limit.
@@ -280,9 +304,14 @@ export class SettingsShell {
     footer(width) {
         const right = `${this.label("crtr.setup.tab-next")}/${this.label("crtr.setup.tab-previous")} destination \u00b7 ${this.label("crtr.setup.cancel")} close`;
         const move = `${this.label("crtr.setup.list-down")}/${this.label("crtr.setup.list-up")} move \u00b7 ${this.label("crtr.setup.advance")} open`;
+        // A panel that has claimed Escape for its own "back" step must not also be
+        // advertised as the way out to the rail — one key, one meaning per screen.
+        const escapes = !this.panelOwnsKeyboard("\u001b");
         const candidates = this.railFocused
             ? [`${move} \u00b7 type a letter to jump`, move]
-            : [`${this.panelHint()} \u00b7 esc rail`, this.panelHint()];
+            : escapes
+                ? [`${this.panelHint()} \u00b7 esc rail`, this.panelHint()]
+                : [this.panelHint()];
         for (const left of candidates) {
             const gap = width - visibleWidth(left) - visibleWidth(right);
             if (gap >= 2)
@@ -305,6 +334,8 @@ export class SettingsShell {
                 return this.keybindings.footerHint();
             case "broker-limits":
                 return `${move} field \u00b7 type a value \u00b7 ${this.label("crtr.setup.advance")} save`;
+            case "profiles":
+                return this.profiles.footerHint();
             default:
                 return `${move} row \u00b7 ${this.label("crtr.setup.toggle")} or ${this.label("crtr.setup.cell-previous")}/${this.label("crtr.setup.cell-next")} change value`;
         }
@@ -349,14 +380,14 @@ export class SettingsShell {
         return lines;
     }
 }
-export async function runSettings() {
+export async function runSettings(initial) {
     await reconcileInboxFrontDoor();
     if (!process.stdout.isTTY) {
         process.stdout.write("crtr sys settings needs a TTY. Run it in a terminal, or use `crtr sys config get|set` for scriptable access.\n");
         return;
     }
     await runSetupSettingsProcess(({ tui, terminal, agentDir, runtime, registry, bindings, rawUserConfig }, control) => {
-        const shell = new SettingsShell(tui, agentDir, runtime, registry, bindings, rawUserConfig?.keybindings, () => terminal.rows, rawUserConfig?.tmux_passthrough);
+        const shell = new SettingsShell(tui, agentDir, runtime, registry, bindings, rawUserConfig?.keybindings, () => terminal.rows, rawUserConfig?.tmux_passthrough, initial);
         shell.onCancel = control.cancel;
         return shell;
     });

package/dist/commands/sys/settings.js

@@ -6,25 +6,41 @@
 // be reachable by a static import from here.
 import { defineLeaf } from '../../core/command.js';
 import { BROKER_LIMIT_OVERRIDE_PRECEDENCE } from './setup.js';
+/** Rail destinations, in rail order. Kept here rather than imported from the
+ *  shell so registering the `sys` subtree still costs no pi module graph. */
+const PAGES = [
+    'providers', 'models', 'keybindings', 'display', 'profiles', 'panes', 'broker-limits', 'updates',
+];
 export const sysSettingsLeaf = defineLeaf({
     name: 'settings',
-    description: 'open the settings page — providers, models, keybindings, display, panes, broker limits, updates',
+    description: 'open the settings page — providers, models, keybindings, display, profiles, panes, broker limits, updates',
     whenToUse: 'the user wants to change crouter configuration interactively. Agents should use sys config get/set, which is scriptable; sys setup installs a fresh machine.',
     help: {
         name: 'sys settings',
         summary: 'open the interactive settings page for an already-installed crouter',
-        inputNote: 'No input parameters.',
+        params: [
+            {
+                kind: 'flag',
+                name: 'page',
+                type: 'enum',
+                choices: [...PAGES],
+                required: false,
+                constraint: 'Open directly on one rail destination instead of the first. Omit to start on the rail.',
+            },
+        ],
         output: [],
         outputKind: 'object',
         effects: [
-            'Opens a pi-tui page whose left rail groups ongoing configuration: Intelligence (providers, models), Interface (keybindings, display), and Workspace (panes, broker limits, updates).',
+            'Opens a pi-tui page whose left rail groups ongoing configuration: Intelligence (providers, models), Interface (keybindings, display), and Workspace (profiles, panes, broker limits, updates).',
+            'Profiles edits are immediate: adding or removing a project directory, cycling its memory rung, setting a home, renaming, pausing and pinning a directory default all write the same profile manifests and pin file as the `crtr profile` leaves; deleting a profile runs the same cascade as `crtr profile delete --force`.',
             'Preference, model-ladder and keybinding edits save to the user config as you make them; saved keybinding changes also refresh crouter-owned tmux bindings without overwriting occupied user keys.',
             `Broker warning and automatic-revive hard-cap values save under user config brokerThresholds; crtrd reads them on its next supervision tick. ${BROKER_LIMIT_OVERRIDE_PRECEDENCE}`,
             'Requires a TTY. Without one it prints a pointer to sys config and exits.',
         ],
     },
-    run: async () => {
-        await (await import('./settings-shell.js')).runSettings();
+    run: async (input) => {
+        const page = input['page'];
+        await (await import('./settings-shell.js')).runSettings(page);
     },
     render: () => '',
 });

package/dist/commands/sys/sync-deps.d.ts

@@ -6,6 +6,8 @@ export declare const DEPS_GENERATED_MARKER = "crtr sys sync deps";
 interface DepsRow {
     source: string;
     target: string;
+    /** Full canonical name the vendored doc answers at (`<namespace>/deps/...`),
+     *  except on an `excluded` row, which names the excluded package selector. */
     name: string;
     scope: 'project';
     status: 'imported' | 'would-import' | 'skipped' | 'excluded';

package/dist/commands/sys/sync-deps.js

@@ -12,25 +12,31 @@
 //     deps/ dir is never destroyed. A self-ignoring `.gitignore` keeps the
 //     whole dir out of git.
 //   • Each SKILL.md becomes its bundle dir's INDEX doc
-//     (deps/<bundle>/INDEX.md): `crtr memory read deps/<bundle>` resolves it
-//     via the bare-dir → INDEX contract, AND the bundle's relative sidecar
+//     (deps/<bundle>/INDEX.md), the document attached to that directory: it
+//     answers at `<namespace>/deps/<bundle>`, AND the bundle's relative sidecar
 //     links (./references/x.md, ./docs/y.md) keep resolving on disk because
 //     the bundle's file layout is preserved. Sidecar .md files get minimal
 //     substrate frontmatter injected (lint-clean, system name/file none); other
 //     files copy verbatim.
+//   • Physical layout is store-local (`deps/**` under the project's memory
+//     dir); every name the lane reports, links, or tells an agent to read is
+//     the full canonical name, composed with the store's declared namespace.
+//     The store must already declare one — this lane vendors documents into a
+//     project store, never the declaration that gives them their identity.
 //   • Vendored bundle and sidecar docs sit at system name/file none, so the boot
 //     catalog exposes their resolvable identities without loading their bodies.
 //     The ONE generated top deps/INDEX.md sits at system preview/file none and
 //     carries the routing catalog. The catalog is one read away, each vendored
 //     doc another.
 import { copyFileSync, mkdirSync, readdirSync, rmSync } from 'node:fs';
-import { CRTR_DIR_NAME } from '../../types.js';
 import { basename, dirname, join, relative, resolve, sep } from 'node:path';
 import { defineLeaf } from '../../core/command.js';
 import { usage } from '../../core/errors.js';
 import { parseFrontmatterGeneric } from '../../core/frontmatter.js';
 import { pathExists, readJsonIfExists, readText, walkFiles, writeText } from '../../core/fs-utils.js';
-import { serializeMemoryDoc } from '../memory/shared.js';
+import { composeCanonicalName } from '../../core/memory/identity.js';
+import { openProjectMemoryStore } from '../../core/memory-resolver.js';
+import { requireMountedStore, serializeMemoryDoc } from '../memory/shared.js';
 import { routeFromDescription, scalarString } from './sync-shared.js';
 import { isRecord } from '../../shared/predicates.js';
 /** Marker value written to the generated top INDEX's `generated-by` field —
@@ -207,7 +213,11 @@ export function runDepsSync(opts) {
     if (!pathExists(nodeModules)) {
         throw usage(`sys sync deps: ${nodeModules} does not exist; install dependencies first (npm ci / npm install), then re-run`);
     }
-    const depsDir = join(projectRoot, CRTR_DIR_NAME, 'memory', 'deps');
+    // Identity before any planning: the store's declared namespace is what every
+    // reported row, catalog link, and read instruction composes under.
+    const store = requireMountedStore(openProjectMemoryStore(projectRoot));
+    const depsDir = join(store.storeRoot, 'deps');
+    const depsName = composeCanonicalName(store.namespace, 'deps');
     const exclusions = readDepsExclusions(projectRoot);
     const exclusionSet = new Set(exclusions.map((e) => exclusionKey(e.package, e.version, e.skill)));
     const selectorMatches = new Map(exclusions.map((e) => [exclusionKey(e.package, e.version, e.skill), 0]));
@@ -301,7 +311,7 @@ export function runDepsSync(opts) {
                     rows.push({
                         source: file,
                         target: join(depsDir, ...targetRel.split('/')),
-                        name: `deps/${dirname(targetRel)}`,
+                        name: `${depsName}/${dirname(targetRel)}`,
                         scope: 'project',
                         status: 'skipped',
                         reason: `collides with same-named bundle from ${existing.pkg}`,
@@ -330,7 +340,7 @@ export function runDepsSync(opts) {
         rows.push({
             source: skill.sourceAbs,
             target: join(depsDir, ...targetRel.split('/')),
-            name: `deps/${skill.bundleRel}`,
+            name: `${depsName}/${skill.bundleRel}`,
             scope: 'project',
             status,
             ...(n > 0 ? { reason: `+${n} sidecar file${n === 1 ? '' : 's'}` } : {}),
@@ -339,7 +349,7 @@ export function runDepsSync(opts) {
     rows.push({
         source: nodeModules,
         target: join(depsDir, 'INDEX.md'),
-        name: 'deps',
+        name: depsName,
         scope: 'project',
         status,
         reason: `generated catalog of ${skills.length} docs from ${catalogPackages.size} package${catalogPackages.size === 1 ? '' : 's'}`,
@@ -383,7 +393,7 @@ export function runDepsSync(opts) {
                 body = readText(entry.sourceAbs);
             }
             const parentBundle = bundleDirs.find((d) => targetRel.startsWith(`${d}/`));
-            const parentName = parentBundle ? `deps/${parentBundle}` : 'a deps/ document';
+            const parentName = parentBundle ? `${depsName}/${parentBundle}` : `a ${depsName} document`;
             const fm = {
                 kind: 'knowledge',
                 'when-and-why-to-read': `When ${parentName} links to this file, this knowledge should be read because it carries the linked supporting reference.`,
@@ -407,7 +417,7 @@ export function runDepsSync(opts) {
         const key = `${skill.pkg}@${skill.version}`;
         const bucket = byPackage.get(key) ?? { version: skill.version, lines: [] };
         const desc = skill.description !== '' ? ` — ${condenseDescription(skill.description, 180)}` : '';
-        bucket.lines.push(`- \`deps/${skill.bundleRel}\`${desc}`);
+        bucket.lines.push(`- \`${depsName}/${skill.bundleRel}\`${desc}`);
         byPackage.set(key, bucket);
     }
     const sections = [...byPackage.entries()]
@@ -420,7 +430,7 @@ export function runDepsSync(opts) {
             ? 'When deciding whether dependency-shipped agent docs are available in this project, this catalog should be read because no dependency guides are currently vendored.'
             : `When writing code against this project's dependencies (${shownLibs}), this catalog should be read ` +
                 'because those packages ship their own authoritative agent guides, vendored below and readable with ' +
-                '`crtr memory read deps/<name>`.',
+                `\`crtr memory read ${depsName}/<name>\`.`,
         'short-form': emptyCatalog
             ? 'No dependency-shipped agent docs are currently vendored.'
             : "Generated catalog of the agent docs shipped inside this project's npm dependencies.",
@@ -432,7 +442,7 @@ export function runDepsSync(opts) {
         : '# Dependency-shipped agent docs (generated)\n\n' +
             'Generated by `crtr sys sync deps` from the Agent Skill bundles inside this project\'s ' +
             'node_modules. Do not hand-edit: the whole `deps/` directory is wiped and regenerated on every run — ' +
-            're-run after dependencies change. Read any entry with `crtr memory read deps/<name>`.\n\n' +
+            `re-run after dependencies change. Read any entry with \`crtr memory read ${depsName}/<name>\`.\n\n` +
             sections;
     writeText(join(depsDir, 'INDEX.md'), serializeMemoryDoc(indexFm, indexBody));
     // Derived state never enters git: the dir ignores itself.
@@ -467,7 +477,7 @@ export const sysSyncDepsLeaf = defineLeaf({
         name: 'sys sync deps',
         summary: 'vendor npm-dependency Agent Skill bundles into <project>/.crouter/memory/deps/ (wholly derived, regenerated every run)',
         params: [
-            { kind: 'flag', name: 'source', type: 'string', required: false, constraint: 'The project directory whose dependencies to vendor. Default: the nearest ancestor of cwd with a package.json. node_modules must already be installed there.' },
+            { kind: 'flag', name: 'source', type: 'string', required: false, constraint: 'The project directory whose dependencies to vendor. Default: the nearest ancestor of cwd with a package.json. node_modules must already be installed there, and its repository-root `.crouter/memory/INDEX.md` must declare a `namespace:`.' },
             { kind: 'flag', name: 'dry-run', type: 'bool', required: false, constraint: 'Report what would be vendored without writing anything.' },
         ],
         output: [
@@ -475,11 +485,12 @@ export const sysSyncDepsLeaf = defineLeaf({
             { name: 'wouldImport', type: 'number', required: true, constraint: 'Count of memory docs that would be written under --dry-run.' },
             { name: 'skipped', type: 'number', required: true, constraint: 'Count of same-named bundle collisions skipped (first-sorted package wins).' },
             { name: 'excluded', type: 'number', required: true, constraint: 'Count of dependency skill bundles excluded by exact package.json crtr.sync.deps.exclude selectors.' },
-            { name: 'results', type: 'object[]', required: true, constraint: 'Each: {source,target,name,scope,status,reason?}; status is one of imported, would-import, skipped, excluded. An excluded row is a dependency bundle rejected before its files enter the vendoring planner.' },
+            { name: 'results', type: 'object[]', required: true, constraint: 'Each: {source,target,name,scope,status,reason?}; status is one of imported, would-import, skipped, excluded. `name` is the full canonical name the vendored doc answers at (`<namespace>/deps/<bundle>`), except on an excluded row, which names the `<package>@<version>:<skill>` selector. An excluded row is a dependency bundle rejected before its files enter the vendoring planner.' },
         ],
         outputKind: 'object',
         effects: [
-            "Wipes and regenerates <project>/.crouter/memory/deps/ from installed dependency skills, except bundles selected by exact {package,version,skill} entries in package.json crtr.sync.deps.exclude. Each included SKILL.md becomes a system-name/file-none deps/<bundle>/INDEX.md (relative sidecar links keep resolving), Markdown sidecar docs are vendored alongside at the same rungs, other sidecar assets are copied verbatim, and a system-preview/file-none deps/INDEX.md catalog plus a self-ignoring .gitignore are written. Excluded bundles and their sidecars do not enter the catalog.",
+            "Wipes and regenerates <project>/.crouter/memory/deps/ from installed dependency skills, except bundles selected by exact {package,version,skill} entries in package.json crtr.sync.deps.exclude. Each included SKILL.md becomes a system-name/file-none deps/<bundle>/INDEX.md answering at <namespace>/deps/<bundle> (relative sidecar links keep resolving), Markdown sidecar docs are vendored alongside at the same rungs, other sidecar assets are copied verbatim, and a system-preview/file-none deps/INDEX.md catalog answering at <namespace>/deps plus a self-ignoring .gitignore are written. Excluded bundles and their sidecars do not enter the catalog.",
+            'Refuses a project store whose repository root declares no namespace, naming `crtr sys migrate --dir <repositoryRoot>` as the remedy; it never declares one itself.',
             'With --dry-run: read-only; writes nothing.',
         ],
     },