dsh-ssh-tui 0.8.4-rc.1 → 0.8.4

package/lib/types/doctor.d.ts

@@ -30,7 +30,11 @@ export interface DoctorCheck {
 export interface DoctorRouting {
     provider?: string;
     model?: string;
-    /** Every provider id the llm-pi-ai section configures. */
+    /**
+     * Every route exposed by the live adapter directory or configured under
+     * `llm-pi-ai.providers`. Presence is registration/configuration evidence,
+     * not a network probe.
+     */
     routes: readonly string[];
     /** Model ids the selected provider's route lists. */
     routeModels: readonly string[];
@@ -50,6 +54,14 @@ export interface DoctorFacts {
         roster: boolean;
         /** `codeRuntime` — what the PTC preset mounts. */
         codeRuntime: boolean;
+        /**
+         * `tools.get('ask_user_question')` — the actual registration, not the row
+         * text. Absent on snapshots taken before this field existed, where the
+         * textual check remains the fallback.
+         */
+        askUserTool?: boolean;
+        /** `tools.get('present')` — the other half of the 0.1.7 agent plane. */
+        presentTool?: boolean;
     };
     /** The profile patch. `readable` is false when it does not exist yet. */
     patch: {

package/lib/types/preset-rows.d.ts

@@ -133,7 +133,14 @@ export declare function analyzePatch(text: string): PatchAnalysis;
  * @returns the later occurrences, in file order.
  */
 export declare function duplicatePatchRows(rows: readonly PatchRowRef[]): PatchRowRef[];
-/** Whether a patch already declares one roster row, by id or by module name. */
+/**
+ * Whether a patch already addresses one roster row, by id or by module name.
+ *
+ * Every kind counts: an override or a disable targets the row just as an insert
+ * does. A repair must leave an intentional `- id: present` / `disabled: true`
+ * alone instead of mounting the same id a second time; the later entry would
+ * either collide with it or silently undo the user's choice.
+ */
 export declare function patchNamesRow(rows: readonly PatchRowRef[], row: RosterRow): boolean;
 /** The roster rows a patch does not declare yet. */
 export declare function missingRosterRows(rows: readonly PatchRowRef[], candidates?: readonly RosterRow[]): RosterRow[];
@@ -159,6 +166,8 @@ export interface PatchRepair {
  * @param missing - the roster rows to mount; defaults to the 0.1.5 set.
  * @param generation - which host line the profile boots on.
  * @returns the new text, or `undefined` when nothing has to change.
+ * @throws when `existing` is not valid YAML; appending to a broken patch would
+ * make it longer without making it bootable.
  */
 export declare function planRosterRepair(existing: string, missing?: readonly RosterRow[], generation?: HostGeneration): PatchRepair | undefined;
 /**
@@ -175,26 +184,32 @@ export declare function planDuplicateRepair(existing: string): PatchRepair | und
 /**
  * Mount the roster in one profile, unless it is already composed.
  *
- * Idempotent: a profile whose patch already names the roster row (one bundling
- * `@deepseek-ai/dsh-web-app`, for example) is left untouched. The write is a
- * plain whole-file replace because the file is small and read once at boot; a
+ * Idempotent: a profile whose patch already addresses the roster row (one
+ * bundling `@deepseek-ai/dsh-web-app`, for example) is left untouched. An
+ * existing file is backed up before the small whole-file replace; a
  * half-written patch would only be seen by the next launch.
  * @param home - the harness home carrying `profiles/`.
  * @param profile - the profile to patch.
  * @param missing - the rows to mount; defaults to the 0.1.5 set.
  * @param generation - which host line the profile boots on.
+ * @param stamp - filename-safe backups stamp, injectable for tests.
  * @returns `present` when the row already exists, else `written`.
+ * @throws when the existing patch cannot be read, is not valid YAML, or cannot
+ * be backed up; a file that cannot be preserved is not overwritten.
  */
-export declare function ensureRosterRows(home: string, profile: string, missing?: readonly RosterRow[], generation?: HostGeneration): Promise<'present' | 'written'>;
+export declare function ensureRosterRows(home: string, profile: string, missing?: readonly RosterRow[], generation?: HostGeneration, stamp?: string): Promise<'present' | 'written'>;
 /**
  * Write a repaired patch next to a copy of the previous content.
  *
- * `/doctor` changes a file the user owns, so the old text is kept beside it
- * before anything is replaced. A file that does not exist yet needs no backup.
+ * `/doctor` and the automatic repair change a file the user owns, so the old
+ * text is kept beside it before anything is replaced. A source that vanished
+ * between the read and the copy needs no backup; any other backup failure
+ * aborts before the write, leaving the original untouched.
  * @param path - the patch file to replace.
  * @param text - the new content.
  * @param stamp - filename-safe timestamp for the backup, injectable for tests.
  * @returns the backup path, or `undefined` when there was nothing to back up.
+ * @throws when the backup cannot be written; the destination is not touched.
  */
 export declare function writePatchWithBackup(path: string, text: string, stamp?: string): Promise<string | undefined>;
 /** The patch text the roster block would produce on its own, for callers that report it. */

package/lib/types/tui.d.ts

@@ -2096,12 +2096,32 @@ export declare class SshTui {
     /** /mode: pick an agent preset (standard / minimal / ptc / cordis / routing-suite / ...). */
     private runModeCommand;
     /**
-     * `/mode fix`: write the roster row into this profile's user patch layer.
+     * The two tools a 0.1.7 terminal profile mounts itself, read from the running
+     * composition. The profile patch is only the instruction; this is the answer
+     * to whether the instruction actually produced a tool.
+     */
+    private agentPlaneTools;
+    /** Whether this process's agent plane has both tools. */
+    private agentPlaneReady;
+    /**
+     * Repair an already-configured profile at boot and say a restart is needed.
      *
-     * The write only takes effect on the next launch — the loader composes the
-     * patch tree once at boot — so the report says so instead of pretending the
-     * running session gained a roster.
+     * A first run deliberately does not call this: its wizard save is the point
+     * where settings and credentials exist, and `onboardDone` repairs immediately
+     * after that write. This method serves the old/partial profile upgrade path,
+     * where there is no wizard to wait for.
+     */
+    private repairAgentPlaneAtBoot;
+    /**
+     * Add only the profile rows whose tools are missing from this composition.
+     *
+     * A row that is already present in the patch is never written again, even if
+     * its service failed to activate: the planner sees overrides and disables by
+     * id or module name, and `ensureRosterRows` creates a backup first. This is
+     * the first-run/upgrade repair, not an install-hook claim: it changes what
+     * the next boot composes, not what this process has already composed.
      */
+    private ensureProfileAgentPlane;
     /** Re-read the roster fact this process was composed with (cheap, no IO). */
     private refreshRosterHealth;
     /** Which settings protocol this host speaks; decides what "missing" means. */
@@ -2114,6 +2134,14 @@ export declare class SshTui {
      * all — the repair there is the agent-plane rows, not a roster.
      */
     private missingPresetService;
+    /**
+     * `/mode fix`: write the missing profile rows into the user patch layer.
+     *
+     * The write only takes effect on the next launch — the loader composes the
+     * patch tree once at boot — so the report says so instead of pretending the
+     * running session gained a roster. The helper consults the actual tools, so
+     * a row a bundle already supplies is not mounted a second time.
+     */
     private repairRoster;
     /** Current provider route selected for the running agent. */
     private currentProvider;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-ssh-tui",
-  "version": "0.8.4-rc.1",
+  "version": "0.8.4",
   "description": "Remote-first terminal UI for DeepSeek Harness with SSH reconnect, Windows/ConPTY and incremental ANSI rendering.",
   "keywords": [
     "deepseek-harness",