@aragon-agent/cli 0.6.34 → 0.6.36

package/dist/ui/overlays/SettingsScreen.js

@@ -24,7 +24,10 @@ import { ModelProfileEditor } from './ModelProfileEditor.js';
 import { ModelDiscoveryList } from './ModelDiscoveryList.js';
 import { useModelProbe } from './use-model-probe.js';
 import { canProbe, editorProbeTarget, visibleProbeViews } from './model-probe-view.js';
-import { createModelProfileState, createCustomProfile, modelProfileReducer, profileUsers, draftTeam, draftSubagentProfileId, subagentModelSummary, subagentSelectedId } from './model-profile-state.js';
+import { createModelProfileState, createCustomProfile, modelProfileReducer, profileUsers, draftTeam, draftSubagentProfileId, subagentModelSummary, subagentSelectedId, SUBAGENT_FAST_ROW_ID } from './model-profile-state.js';
+import { KitEditor } from './KitEditor.js';
+import { KitManager } from './KitManager.js';
+import { heldSlots, kitManagerRows, kitPackRow, kitReducer } from './kit-settings-state.js';
 /**
  * THE ROOT PAGE'S ROW INDEXES, AS NAMES (subagent-model V-5).
  *
@@ -32,12 +35,18 @@ import { createModelProfileState, createCustomProfile, modelProfileReducer, prof
  * model row moves every one of them. Each site below now cites its constant,
  * so adding a row later is a one-line change instead of a silent regression
  * that focuses an error message on the wrong field.
+ *
+ * DERIVED FROM ONE LIST (agent-model-kits §4.5), because the kit row is the
+ * fifth of these and the "five offsets agreed by accident" defect does not need
+ * a sequel. Order here IS the render order.
  */
-const ROOT_MAIN = 0;
-const ROOT_FAST = 1;
-const ROOT_SUBAGENT = 2;
-const ROOT_MANAGE = 3;
-const ROOT_FIRST_FIELD = 4;
+const ROOT_ROWS = ['main', 'fast', 'subagent', 'kit', 'manage'];
+const ROOT_MAIN = ROOT_ROWS.indexOf('main');
+const ROOT_FAST = ROOT_ROWS.indexOf('fast');
+const ROOT_SUBAGENT = ROOT_ROWS.indexOf('subagent');
+const ROOT_KIT = ROOT_ROWS.indexOf('kit');
+const ROOT_MANAGE = ROOT_ROWS.indexOf('manage');
+const ROOT_FIRST_FIELD = ROOT_ROWS.length;
 /** Stand-in target while no editor exists; no probe row is rendered then. */
 const NO_PROFILE_TARGET = { provider: '', model: '', baseUrl: null,
     apiKey: null };
@@ -523,8 +532,23 @@ function saveError(result) {
     };
     return errors[result.code ?? ''] ?? 'Could not save. Review settings and retry.';
 }
+function isKitAction(action) {
+    return action.type.startsWith('kit-');
+}
+/**
+ * ONE REDUCER FOR THE WHOLE SCREEN, TWO OWNERS INSIDE IT (agent-model-kits §4.5).
+ *
+ * The kit pages need their own data mutations (create, duplicate, delete, apply)
+ * but must not introduce a second state object: `Save`, `Esc` and the dirty
+ * badge are shared, and a second store is how a settings screen starts
+ * disagreeing with itself. `kitReducer` returns the SAME state shape, so the
+ * split is invisible from the outside.
+ */
+function settingsReducer(state, action) {
+    return isKitAction(action) ? kitReducer(state, action) : modelProfileReducer(state, action);
+}
 function useProfileSettingsState(props) {
-    const [state, dispatch] = useReducer(modelProfileReducer, props.draft, createModelProfileState);
+    const [state, dispatch] = useReducer(settingsReducer, props.draft, createModelProfileState);
     const [initial, setInitial] = useState(() => profileSettingsValues(props.initial, props.draft));
     const [values, setValues] = useState(initial);
     const [index, setIndex] = useState(0);
@@ -559,12 +583,16 @@ function useProfileSettingsState(props) {
             && ['provider', 'model', 'baseUrl', 'contextWindow', 'apiKey'].includes(item.key)),
         ...FIELDS.filter((item) => library.fastId === null
             && ['fastModel', 'fastProvider'].includes(item.key)), ...PROFILE_FIELDS];
-    // SIX FIXED ROWS now (Main, Fast, SUBAGENT, Manage, Save, Reload) around
-    // the `fields` block - see the ROOT_* constants for the five offset sites.
-    const count = fields.length + 6;
+    // SEVEN FIXED ROWS now (Main, Fast, SUBAGENT, Kit pack, Manage, Save, Reload)
+    // around the `fields` block - derived from ROOT_ROWS so the count and the
+    // offsets cannot drift apart (see those constants).
+    const count = fields.length + ROOT_ROWS.length + 2;
     const field = fields[index - ROOT_FIRST_FIELD];
     const readError = state.draft.readError;
-    const confirming = state.page === 'delete-confirm' || state.page === 'discard-confirm';
+    // THREE CONFIRMATIONS SHARE ONE PAGE, which is why the page name - not a
+    // boolean - is what the renderer and the input hook both branch on.
+    const confirming = state.page === 'delete-confirm' || state.page === 'discard-confirm'
+        || state.page === 'kit-delete';
     return { state, dispatch, initial, setInitial, values, setValues, index, setIndex,
         error, setError, persisted, setPersisted, confirmIndex, setConfirmIndex,
         reloadPending, setReloadPending, offset, library, glyphs, dirty, fields, count,
@@ -651,6 +679,11 @@ function activateRootSetting(props, view) {
         // `draft.patch.team` rather than a `${role}Id` library key.
         dispatch({ type: 'page', page: 'picker', role: 'subagent' });
     }
+    else if (index === ROOT_KIT) {
+        // The fourth root row opens the manager, not a picker: kits are data the user
+        // owns here, and `ctrl+K` is where they are CONSUMED (agent-model-kits §4.5).
+        dispatch({ type: 'page', page: 'kits' });
+    }
     else if (index === ROOT_MANAGE)
         dispatch({ type: 'page', page: 'manager' });
     else
@@ -717,6 +750,11 @@ function useSettingsConfirmationInput(props, view) {
         }
         if (!key.return)
             return;
+        if (state.page === 'kit-delete') {
+            dispatch(confirmIndex === 1 ? { type: 'kit-confirm-delete' }
+                : { type: 'page', page: 'kits' });
+            return;
+        }
         if (state.page === 'delete-confirm') {
             dispatch(confirmIndex === 1 ? { type: 'confirm-delete' }
                 : { type: 'page', page: 'manager' });
@@ -729,8 +767,26 @@ function useSettingsConfirmationInput(props, view) {
             props.onClose?.();
     }, { isActive: confirming });
 }
+/**
+ * The kit deletion confirmation (agent-model-kits D-9 / AC-30).
+ *
+ * `Cancel` IS THE DEFAULT FOCUS, like the profile confirmation next door: the
+ * `Enter` key that opened this page must not be the one that deletes. The second
+ * line names what the user loses - a kit, and with it the default pointer and any
+ * session binding that named it.
+ */
+function renderKitDeleteConfirmation(props, view) {
+    const { state, confirmIndex, glyphs } = view;
+    const kit = (state.draft.profiles?.kits ?? []).find((item) => item.id === state.kitDraft?.id);
+    const rows = ['Cancel', 'Delete'].map((label, at) => (_jsxs(Text, { wrap: "truncate", color: props.theme.primary, children: [confirmIndex === at ? `${glyphs.caret} ` : '  ', label] }, label)));
+    rows.push(_jsx(Text, { wrap: "truncate", color: props.theme.muted, children: `Deletes kit "${kit?.name ?? ''}". Its three slots stay as they are; only the preset goes.` }, "consequence"));
+    return _jsx(OverlayFrame, { ...props, title: `Delete kit ${kit?.name ?? ''}?`, hint: "Enter choose | Esc continue", rows: rows, scrollOffset: confirmIndex });
+}
 function renderSettingsConfirmation(props, view) {
     const { state, library, confirmIndex, glyphs } = view;
+    const deletingKit = state.page === 'kit-delete';
+    if (deletingKit)
+        return renderKitDeleteConfirmation(props, view);
     const deleting = state.page === 'delete-confirm';
     const name = library.entries.find((entry) => entry.id === state.deleteId)?.name ?? '';
     // UNBIND-AND-DELETE (D-6): a referenced entry stays deletable in ONE flow,
@@ -780,6 +836,16 @@ function fastTierUnavailable(props, view) {
 }
 function renderProfileSubpage(props, view) {
     const { state, library, dispatch, setConfirmIndex } = view;
+    // THE KIT PAGES (agent-model-kits §4.5). They are dispatched BEFORE the profile
+    // pages because `'kit-slot'` RENDERS a profile picker while belonging to the
+    // kit editor - the page name, not the component, decides where the user is.
+    if (state.page === 'kits')
+        return _jsx(KitManager, { ...props, rows: kitManagerRows(state), error: state.error, dispatch: dispatch, onBack: () => dispatch({ type: 'page', page: 'root' }) });
+    if (state.page === 'kit-edit' && state.kitDraft)
+        return _jsx(KitEditor, { ...props, draft: state.kitDraft, state: state, dispatch: dispatch });
+    if (state.page === 'kit-slot' && state.kitDraft) {
+        return renderKitSlotPicker(props, view, state.kitDraft);
+    }
     if (state.page === 'editor' && state.editor)
         return _jsx(ModelProfileEditor, { ...props, editor: state.editor, users: profileUsers(library, state.editor.original.id, draftSubagentProfileId(state.draft)), error: state.error, dispatch: dispatch, probeView: view.probeViews.test, probeBusy: view.probe.busy, probeTarget: view.probeTarget, onTest: () => startTest(view), onDiscover: () => startDiscovery(view, dispatch), onCancelProbe: view.probe.cancel });
     // A DISCOVERY IS A PAGE, NOT A POPUP: the list is where the answer belongs, and
@@ -788,13 +854,50 @@ function renderProfileSubpage(props, view) {
         return _jsx(ModelDiscoveryList, { ...props, provider: state.editor.provider, currentModel: state.editor.model, view: view.probeViews.discovery, busy: view.probe.discovery.phase === 'discovering', onChoose: (model) => dispatch({ type: 'choose-model', model }), onRetry: () => startDiscovery(view, dispatch), onBack: () => dispatch({ type: 'page', page: 'editor' }), onCancelProbe: view.probe.cancel });
     // A NEW PROFILE SEEDED FROM THE MAIN CONNECTION while the subagent picker is
     // open: 'subagent' is picker-only (6.2), so the seed role falls back to
-    // 'main' while `state.role` keeps driving the select that binds it.
-    const createRole = state.role === 'subagent' ? 'main' : state.role;
+    // 'main' while `state.role` keeps driving the select that binds it. 'binding'
+    // is the kit slot's chooser and seeds the same way.
+    const createRole = state.role === 'subagent' || state.role === 'binding'
+        ? 'main' : state.role;
     if (state.page === 'picker' || state.page === 'manager')
         return _jsx(ModelProfilePicker, { ...props, profiles: library, savedProfiles: state.draft.baseline.modelProfiles, role: state.role, customProfile: createCustomProfile(state.draft, createRole), manager: state.page === 'manager', error: state.error, selectedIdOverride: state.role === 'subagent' ? subagentSelectedId(state.draft) : undefined, subagentProfileId: draftSubagentProfileId(state.draft), fastTierOff: state.role === 'subagent' && fastTierUnavailable(props, view), onSelect: (id) => dispatch({ type: 'select', role: state.role, id }), onCreate: () => dispatch({ type: 'edit', profile: createCustomProfile(state.draft, createRole),
                 bindNew: state.page === 'picker' }), onEdit: (profile) => dispatch({ type: 'edit', profile }), onDuplicate: (id) => dispatch({ type: 'duplicate', id }), onDelete: (id) => { setConfirmIndex(0); dispatch({ type: 'delete', id }); }, onCancel: () => dispatch({ type: 'page', page: 'root' }) });
     return null;
 }
+/** Which slot value the kit picker should mark `[Selected]`. */
+function kitSlotSelection(kitDraft) {
+    if (kitDraft.slot === 'main')
+        return kitDraft.mainProfileId || null;
+    if (kitDraft.slot === 'fast')
+        return kitDraft.fastProfileId;
+    if (kitDraft.subagent.model === 'profile')
+        return kitDraft.subagent.modelProfileId;
+    return kitDraft.subagent.model === 'fast' ? SUBAGENT_FAST_ROW_ID : null;
+}
+/**
+ * The slot chooser: the SAME profile picker, in its `binding` role.
+ *
+ * A kit slot can only name a profile, so there is no `Current custom` row and no
+ * `Same as main` (those are the subagent slot's own three-state vocabulary). The
+ * subagent slot keeps the `subagent` role, which is why `Fast tier (off - falls
+ * back to main)` needs no new wording anywhere (RV-P2-6).
+ */
+function renderKitSlotPicker(props, view, kitDraft) {
+    const { state, library, dispatch } = view;
+    const role = kitDraft.slot === 'subagent' ? 'subagent' : 'binding';
+    return _jsx(ModelProfilePicker, { ...props, profiles: library, savedProfiles: state.draft.baseline.modelProfiles, role: role, customProfile: createCustomProfile(state.draft, 'main'), error: state.error, selectedIdOverride: kitSlotSelection(kitDraft), subagentProfileId: draftSubagentProfileId(state.draft), fastTierOff: kitDraft.slot === 'subagent' && fastTierUnavailable(props, view), onSelect: (id) => {
+            if (kitDraft.slot === 'main') {
+                dispatch({ type: 'kit-main', id });
+                return;
+            }
+            if (kitDraft.slot === 'fast') {
+                dispatch({ type: 'kit-fast', id });
+                return;
+            }
+            dispatch({ type: 'kit-subagent',
+                model: id === null ? 'main' : id === SUBAGENT_FAST_ROW_ID ? 'fast' : 'profile',
+                modelProfileId: id === null || id === SUBAGENT_FAST_ROW_ID ? null : id });
+        }, onCreate: () => dispatch({ type: 'edit', profile: createCustomProfile(state.draft, 'main') }), onEdit: (profile) => dispatch({ type: 'edit', profile }), onDuplicate: (id) => dispatch({ type: 'duplicate', id }), onDelete: (id) => dispatch({ type: 'delete', id }), onCancel: () => dispatch({ type: 'kit-cancel' }) });
+}
 function buildRootSettingsRows(props, view) {
     const { library, fields, values, persisted, readError, index, glyphs, state } = view;
     const name = (id) => {
@@ -823,9 +926,29 @@ function buildRootSettingsRows(props, view) {
         modelProfileId: team.modelProfileId, mainConnection, fastConnection,
         profiles: state.draft.profiles,
         hasKey: (id) => (props.apiKeys[id] ?? '').length > 0 });
-    const lines = [`Main profile      ${name(library.mainId)}`,
-        `Fast profile      ${name(library.fastId)}`,
-        `Subagent model    ${subagentLine}`,
+    // HELD SLOTS SHOW THE KIT'S EFFECTIVE VALUES (RV-P0-2 / AC-38). Three rows on
+    // this page read the FILE's bindings while the header reads the live ones, so
+    // without this the screen says `Main profile  Opus` over a header that says
+    // `haiku` - and the only available conclusion is that the app is broken.
+    // `Enter` on any of these rows writes an explicit choice, which releases that
+    // slot (§4.3), so what the user sees is what they are about to change.
+    const held = heldSlots(state);
+    const kitTag = held.kitName ? `  (kit ${held.kitName})` : '';
+    const mainValue = `${name(held.mainId ?? library.mainId)}${held.held.includes('main')
+        ? kitTag : ''}`;
+    const fastValue = held.held.includes('fast')
+        ? `${held.fastId ? name(held.fastId) : 'Off (this kit turns the tier off)'}${kitTag}`
+        : name(library.fastId);
+    const subagentValue = held.subagent
+        ? `${subagentModelSummary({ mode: held.subagent.model,
+            modelProfileId: held.subagent.modelProfileId, mainConnection, fastConnection,
+            profiles: state.draft.profiles,
+            hasKey: (id) => (props.apiKeys[id] ?? '').length > 0 })}${kitTag}`
+        : subagentLine;
+    const lines = [`Main profile      ${mainValue}`,
+        `Fast profile      ${fastValue}`,
+        `Subagent model    ${subagentValue}`,
+        `Kit pack          ${kitPackRow(state)}`,
         `Manage profiles   ${library.entries.length} saved`,
         ...fields.map((item) => `${item.label.padEnd(18)}${item.key === 'apiKey'
             ? values.apiKey ? 'Configured (replacement)' : props.apiKeys[values.provider]

package/dist/ui/overlays/kit-picker-state.d.ts

@@ -0,0 +1,87 @@
+/**
+ * The `ctrl+K` list, as a pure view model (agent-model-kits §4.6 / D-10).
+ *
+ * ASCII ONLY - `src/ui/**` is inside `glyphs.test.ts::inScope`.
+ *
+ * WHY THE ROWS ARE COMPUTED HERE AND NOT IN THE COMPONENT: three surfaces print
+ * the same three connections (this list, the settings manager, `/kit list`).
+ * One function produces them, so the wording of "the fast tier is off" cannot
+ * differ between the key the user pressed and the command they typed.
+ */
+import type { CliConfig, TeamConfig } from '../../config/schema.js';
+import type { ModelProfilesConfig } from '../../config/model-profiles.js';
+import { type KitConnection, type KitSlot } from '../../config/model-kits.js';
+import { type ActiveKit } from '../../config/kit-resolution.js';
+export interface KitPickerRow {
+    /** `null` = `Current setup` (the release action). */
+    id: string | null;
+    /** The one-line label, markers included. */
+    label: string;
+    /** Machine-checkable markers, so tests never grep a rendered string. */
+    markers: {
+        inUse: boolean;
+        isDefault: boolean;
+        modified: boolean;
+    };
+    /** Non-empty when a referenced profile is gone. */
+    missing: KitSlot[];
+}
+export interface KitPickerInput {
+    library?: ModelProfilesConfig | undefined;
+    active: ActiveKit;
+}
+/** `provider:model` plus the endpoint clause shared by every surface. */
+export declare function connectionText(connection: KitConnection): string;
+/**
+ * Rows in a FIXED order: `Current setup` first, then the library's own order.
+ *
+ * The first row never moves (AC-24) - it is the only row that can always be
+ * chosen, and a list whose "off" entry wanders with the data is a list a user
+ * has to read every single time.
+ */
+export declare function kitRows(input: KitPickerInput): KitPickerRow[];
+/**
+ * The three bindings `Current setup` describes.
+ *
+ * STRUCTURAL, NOT `CliConfig`: the caller hands in the PERSISTED config for this
+ * block (§4.6), because the live config no longer holds the file's bindings once
+ * a session kit is applied - it mirrors the kit (I-6). Both shapes satisfy this
+ * interface, so neither layer has to convert anything.
+ */
+export interface KitConnectionsSource {
+    provider: string;
+    model: string;
+    team?: {
+        model: TeamConfig['model'];
+        modelProfileId?: string | null;
+    } | undefined;
+    fast: {
+        enabled: boolean;
+        provider: string;
+        model: string;
+    };
+    modelProfiles?: ModelProfilesConfig | undefined;
+}
+/** The three lines under the focused row (§4.6). */
+export declare function kitDetail(input: {
+    row: KitPickerRow;
+    library?: ModelProfilesConfig | undefined;
+    config: CliConfig;
+    /** The config file's copy of the bindings; see the `Current setup` note below. */
+    diskConfig?: KitConnectionsSource | undefined;
+}): string[];
+/**
+ * The footer. `busy` is a warning, never a reason to hide the list (§4.7).
+ *
+ * THE LAST CLAUSE IS D-7, VERBATIM: this overlay is a session-only surface, so it
+ * says where the persistence actually lives. Without it the natural reading of
+ * `Enter` is "make this my default", and D-7 exists to keep that action one
+ * deliberate trip to `/settings` instead of one keystroke next to a browse key.
+ * It is LAST so that a 40-column terminal clips the pointer, never the action.
+ */
+export declare function kitPickHint(input: {
+    busy: boolean;
+    hasKits: boolean;
+}): string;
+/** The one guidance row an empty library renders (AC-26). */
+export declare const NO_KITS_HINT = "No kits yet. Create one in /settings (Settings > Kit pack).";

package/dist/ui/overlays/kit-picker-state.js

@@ -0,0 +1,136 @@
+/**
+ * The `ctrl+K` list, as a pure view model (agent-model-kits §4.6 / D-10).
+ *
+ * ASCII ONLY - `src/ui/**` is inside `glyphs.test.ts::inScope`.
+ *
+ * WHY THE ROWS ARE COMPUTED HERE AND NOT IN THE COMPONENT: three surfaces print
+ * the same three connections (this list, the settings manager, `/kit list`).
+ * One function produces them, so the wording of "the fast tier is off" cannot
+ * differ between the key the user pressed and the command they typed.
+ */
+import { kitSlotConnections } from '../../config/model-kits.js';
+import { CURRENT_SETUP_NAME } from '../../config/kit-resolution.js';
+function duplicateNames(kits) {
+    const seen = new Set();
+    const repeated = new Set();
+    for (const kit of kits) {
+        if (seen.has(kit.name))
+            repeated.add(kit.name);
+        seen.add(kit.name);
+    }
+    return repeated;
+}
+/** `provider:model` plus the endpoint clause shared by every surface. */
+export function connectionText(connection) {
+    const endpoint = connection.baseUrl ? hostOf(connection.baseUrl) : 'default URL';
+    return `${connection.provider}:${connection.model}  ${endpoint}`;
+}
+function hostOf(baseUrl) {
+    try {
+        const url = new URL(baseUrl);
+        return url.username || url.password || url.search || url.hash ? 'Invalid URL' : url.host;
+    }
+    catch {
+        return 'Invalid URL';
+    }
+}
+/**
+ * Rows in a FIXED order: `Current setup` first, then the library's own order.
+ *
+ * The first row never moves (AC-24) - it is the only row that can always be
+ * chosen, and a list whose "off" entry wanders with the data is a list a user
+ * has to read every single time.
+ */
+export function kitRows(input) {
+    const kits = input.library?.kits ?? [];
+    const repeated = duplicateNames(kits);
+    const rows = [{
+            id: null,
+            label: `${input.active.source === 'none' ? '[In use] ' : ''}${CURRENT_SETUP_NAME}`,
+            markers: { inUse: input.active.source === 'none', isDefault: false, modified: false },
+            missing: [],
+        }];
+    for (const kit of kits) {
+        const inUse = input.active.source !== 'none' && input.active.id === kit.id;
+        const isDefault = (input.library?.defaultKitId ?? null) === kit.id;
+        const modified = inUse && input.active.modified;
+        const missing = inUse ? input.active.missingProfiles : [];
+        // A DUPLICATE NAME gets its id prefix, exactly as the profile picker does:
+        // two rows reading `Cheap` would make Enter a coin flip.
+        const identity = repeated.has(kit.name) ? `#${kit.id.slice(0, 8)} ` : '';
+        const markers = [inUse ? '[In use]' : '', isDefault ? '[Default]' : '']
+            .filter(Boolean).join(' ');
+        rows.push({
+            id: kit.id,
+            label: `${identity}${markers ? `${markers} ` : ''}${kit.name}${modified ? '~' : ''}`,
+            markers: { inUse, isDefault, modified },
+            missing,
+        });
+    }
+    return rows;
+}
+function subagentText(config) {
+    const mode = config.team?.model ?? 'main';
+    if (mode === 'main')
+        return 'same as main';
+    if (mode === 'fast')
+        return 'fast tier model';
+    const id = config.team?.modelProfileId ?? null;
+    const entry = id ? config.modelProfiles?.entries.find((item) => item.id === id) : undefined;
+    return entry ? `profile ${entry.name}` : 'Profile missing';
+}
+/** The three lines under the focused row (§4.6). */
+export function kitDetail(input) {
+    const { row, library, config } = input;
+    const kit = row.id ? (library?.kits ?? []).find((item) => item.id === row.id) : undefined;
+    if (!kit) {
+        // `Current setup` describes the FILE's bindings - what the next process
+        // would run - because that is exactly what choosing it restores.
+        //
+        // THE LIVE CONFIG IS THE WRONG SOURCE WHILE A SESSION KIT IS HELD: it mirrors
+        // the kit (I-6), so this block would print the current values and thereby
+        // promise that releasing the kit changes nothing. The first row is the only
+        // row whose whole meaning is "what the file says", so it is the only one that
+        // has to be read from the file.
+        const file = input.diskConfig ?? config;
+        const fast = file.fast.enabled
+            ? `${file.fast.provider || file.provider}:${file.fast.model || '(not set)'}`
+            : 'off';
+        return [`main      ${file.provider}:${file.model}`,
+            `subagent  ${subagentText(file)}`,
+            `fast      ${fast}`];
+    }
+    const connections = kitSlotConnections(library, kit);
+    const mainLine = connections.main ? connectionText(connections.main) : 'Profile missing';
+    const fastLine = kit.fastProfileId === null ? 'off (this kit turns the tier off)'
+        : connections.fast ? `${connectionText(connections.fast)}  tier on` : 'Profile missing';
+    // THE SUBAGENT LINE REUSES THE SLOT'S OWN THREE-STATE VOCABULARY (`same as main`
+    // / `fast tier` / a profile), so this block and the settings screen's subagent
+    // row cannot say different things about the same kit.
+    const subagentLine = kit.subagent.model === 'main' ? 'same as main'
+        : kit.subagent.model === 'fast'
+            ? (connections.fast ? 'fast tier' : 'fast tier (off - falls back to main)')
+            : connections.subagent.name
+                ? `profile ${connections.subagent.name}  ${connections.subagent.provider}:${connections.subagent.model}` : 'Profile missing';
+    return [`main      ${mainLine}`, `subagent  ${subagentLine}`, `fast      ${fastLine}`];
+}
+/**
+ * The footer. `busy` is a warning, never a reason to hide the list (§4.7).
+ *
+ * THE LAST CLAUSE IS D-7, VERBATIM: this overlay is a session-only surface, so it
+ * says where the persistence actually lives. Without it the natural reading of
+ * `Enter` is "make this my default", and D-7 exists to keep that action one
+ * deliberate trip to `/settings` instead of one keystroke next to a browse key.
+ * It is LAST so that a 40-column terminal clips the pointer, never the action.
+ */
+export function kitPickHint(input) {
+    if (!input.hasKits)
+        return 'Esc close | Create a kit in /settings';
+    if (input.busy) {
+        return 'Enter use for this session (blocked while running) | Esc close'
+            + ' | Set a default in /settings';
+    }
+    return 'Enter use for this session | Esc close | Set a default in /settings';
+}
+/** The one guidance row an empty library renders (AC-26). */
+export const NO_KITS_HINT = 'No kits yet. Create one in /settings (Settings > Kit pack).';

package/dist/ui/overlays/kit-settings-state.d.ts

@@ -0,0 +1,129 @@
+/**
+ * The settings screen's kit pages: manager rows, the editor draft, and the ONE
+ * reducer that turns intents into draft mutations (agent-model-kits §4.5).
+ *
+ * ASCII ONLY - `src/ui/**` is inside `glyphs.test.ts::inScope`.
+ *
+ * TWO REDUCERS, ONE PAGE AXIS. `modelProfileReducer` owns the profile pages and
+ * `kitReducer` owns kit data; both return the same `ModelProfileState`, so the
+ * screen still has exactly one state object and one Save button. The alternative
+ * - a second state machine inside `SettingsScreen.tsx`, which is already over the
+ * 1000-line cap - is what this module exists to avoid.
+ *
+ * NOTHING HERE WRITES ANYTHING: every action mutates the transaction draft and
+ * sets `dirty`. `Save` stays the only writer (F-7/F-10).
+ */
+import type { KitSlot, AgentKit } from '../../config/model-kits.js';
+import type { ModelProfilesConfig } from '../../config/model-profiles.js';
+import type { SubagentModelMode } from '../../team/model.js';
+import type { ModelProfileState } from './model-profile-state.js';
+export interface KitDraft {
+    /** `null` while creating; the delete confirmation reuses this field as its id. */
+    id: string | null;
+    name: string;
+    mainProfileId: string;
+    /** `null` = this kit turns the fast tier off (D-8). */
+    fastProfileId: string | null;
+    subagent: {
+        model: SubagentModelMode;
+        modelProfileId: string | null;
+    };
+    /** Which slot the profile picker is choosing for; `null` on the editor page. */
+    slot: KitSlot | null;
+    error?: string;
+    /** The row the cursor jumps to when `error` is set (focus-lands-on-error). */
+    field?: KitSlot | 'name';
+}
+export type KitAction = {
+    type: 'kit-open';
+    id: string | null;
+} | {
+    type: 'kit-name';
+    value: string;
+} | {
+    type: 'kit-slot';
+    slot: KitSlot;
+} | {
+    type: 'kit-main';
+    id: string | null;
+} | {
+    type: 'kit-fast';
+    id: string | null;
+} | {
+    type: 'kit-subagent';
+    model: SubagentModelMode;
+    modelProfileId: string | null;
+} | {
+    type: 'kit-save';
+} | {
+    type: 'kit-cancel';
+} | {
+    type: 'kit-duplicate';
+    id: string;
+} | {
+    type: 'kit-delete';
+    id: string;
+} | {
+    type: 'kit-confirm-delete';
+} | {
+    type: 'kit-apply';
+    id: string;
+} | {
+    type: 'kit-default';
+    id: string;
+};
+/** The library the transaction would save, with a read error falling back to empty. */
+export declare function draftLibrary(state: ModelProfileState): ModelProfilesConfig;
+export declare function findKitInState(state: ModelProfileState, id: string | null): AgentKit | undefined;
+/**
+ * Open the editor on an existing kit, or on a PREFILLED new one (§4.10).
+ *
+ * THE FIRST KIT IS NEVER A BLANK FORM (D-13): it starts from the three bindings
+ * this machine already runs, so "create a kit" is one rename and one Save. When
+ * no Main profile is bound the kit starts with an empty main slot and Save
+ * refuses - the picker's own `Create profile` action is where a profile is born,
+ * which keeps this module free of the profile-creating code path.
+ */
+export declare function openKitDraft(state: ModelProfileState, id: string | null): KitDraft;
+/** Every kit-page intent, applied to the ONE draft. */
+export declare function kitReducer(state: ModelProfileState, action: KitAction): ModelProfileState;
+/** One manager row: the label with its markers, plus the three-slot summary. */
+export interface KitManagerRow {
+    id: string;
+    label: string;
+    summary: string;
+    isDefault: boolean;
+    inUse: boolean;
+}
+/** Rows in library order, each with the markers the picker also uses (§4.5). */
+export declare function kitManagerRows(state: ModelProfileState): KitManagerRow[];
+/**
+ * The `Kit pack` root row's five states (§4.5 table, AC-27).
+ *
+ * READ FROM THE DRAFT, not from the live config: the row sits inside a
+ * transaction, and showing the saved state under a pending selection is how a
+ * settings screen lies to the user about what `Save` is going to do.
+ */
+export declare function kitPackRow(state: ModelProfileState): string;
+/** Which of the three root rows must show the KIT's value, and under which name. */
+export interface HeldSlots {
+    kitName: string | null;
+    /** The slots the session kit still owns; `mainId`/`fastId` are only meaningful
+     *  for a slot in this list (`null` means "this kit turns the tier off"). */
+    held: KitSlot[];
+    mainId: string | null;
+    fastId: string | null;
+    subagent: {
+        model: SubagentModelMode;
+        modelProfileId: string | null;
+    } | null;
+}
+/**
+ * The held slots' EFFECTIVE values (RV-P0-2).
+ *
+ * WITHOUT THIS the root page shows the file's bindings while the header shows the
+ * kit's, and the only conclusion available to the user is that one of them is
+ * broken. `Enter` on any of these rows writes an explicit choice, which releases
+ * that slot - so "what you see is what you change" stays true.
+ */
+export declare function heldSlots(state: ModelProfileState): HeldSlots;