@dotdrelle/wiki-manager 0.15.96 → 0.15.98

package/src/runtime/skillRun.js

@@ -165,7 +165,7 @@ export async function generateSkillAcknowledgment(session, { publicInput, object
       // slow provider must not block the skill-launch HTTP response forever.
       const reply = await llm.complete({
         system: 'You are Donna, the workspace assistant. You acknowledge a launched workflow in the user\'s language. Be concise: exactly one short sentence.',
-        input: `The user just launched the workspace skill ${publicInput}. It was compiled into ${count} step(s) and is now running.\n\nWrite ONE short sentence in ${language} that confirms the launch, echoes the skill and its arguments, and says progress will be reported. Do not ask a question, do not propose options, and do not offer to check, monitor or cancel anything: the runtime reports progress on its own and this acknowledgement is not a decision point. Return only that sentence, nothing else.`,
+        input: `The user just launched the workspace skill ${publicInput}. It was compiled into ${count} step(s) and has been queued.\n\nWrite ONE short sentence in ${language} that confirms the launch, echoes the skill and its arguments, and says progress will be reported. Do not claim the work is running, executing or done: a mutating step waits for the user's approval before it runs. Do not ask a question, do not propose options, and do not offer to check, monitor or cancel anything: the runtime reports progress on its own and this acknowledgement is not a decision point. Return only that sentence, nothing else.`,
         signal: AbortSignal.timeout(8_000),
       });
       const text = String(reply ?? '').trim();
@@ -179,7 +179,7 @@ export async function generateSkillAcknowledgment(session, { publicInput, object
       emitRuntimeLog(session, `skill-acknowledgment: LLM call failed, using the neutral fallback — ${err instanceof Error ? err.message : String(err)}`);
     }
   }
-  return `Started ${publicInput} — ${count} step(s) in progress.`;
+  return `Started ${publicInput} — ${count} step(s) queued.`;
 }
 
 function argumentError(message) {

package/src/runtime/skillRun.test.js

@@ -95,19 +95,21 @@ test('generateSkillAcknowledgment asks Donna in the session language and echoes
   assert.match(calls[0].input, /es/);
   assert.match(calls[0].input, /\/deliver deliverable="Informe"/);
   // The acknowledgement is not a decision point: it must not invite the user
-  // into a dialog the runtime cannot act on.
+  // into a dialog the runtime cannot act on, and it must not claim the work is
+  // already executing — a mutating step waits for approval.
   assert.match(calls[0].input, /Do not ask a question/);
+  assert.match(calls[0].input, /Do not claim the work is running/);
 });
 
 test('generateSkillAcknowledgment degrades to a neutral message without an LLM client', async () => {
   const reply = await generateSkillAcknowledgment({ language: 'fr' }, { publicInput: '/wiki-ingest docs', objectives: 2 });
-  assert.equal(reply, 'Started /wiki-ingest docs — 2 step(s) in progress.');
+  assert.equal(reply, 'Started /wiki-ingest docs — 2 step(s) queued.');
 });
 
 test('generateSkillAcknowledgment falls back when the LLM call fails', async () => {
   const session = { language: 'en', llm: { complete: async () => { throw new Error('down'); } } };
   const reply = await generateSkillAcknowledgment(session, { publicInput: '/deliver', objectives: 1 });
-  assert.equal(reply, 'Started /deliver — 1 step(s) in progress.');
+  assert.equal(reply, 'Started /deliver — 1 step(s) queued.');
 });
 
 test('generateSkillAcknowledgment announces an LLM failure instead of degrading silently', async () => {

package/src/shell/openExternal.js

@@ -51,3 +51,46 @@ export function openExternalUrl(value, { run = execFileSync } = {}) {
   }
   return null;
 }
+
+// A Chromium `--app=<url>` window is chromeless from the start — no tab
+// strip, no address bar — and, when the origin serves a manifest (llm-wiki's
+// `/manifest.webmanifest`), it picks up the site's name/icon/theme-color and
+// window-controls-overlay exactly like a formally "installed" PWA, with no
+// prior install step required. `/openui`'s whole point is a desktop-feeling
+// window, so it tries this before falling back to `openExternalUrl`'s plain
+// tab. Same candidate/fallback shape as `openerCandidates` above, and the same
+// contract: returns the opened URL, or null when no Chromium browser answered.
+function appModeCandidates() {
+  if (process.platform === 'darwin') {
+    return [
+      ['open', ['-na', 'Google Chrome', '--args']],
+      ['open', ['-na', 'Microsoft Edge', '--args']],
+    ];
+  }
+  if (process.platform === 'win32') {
+    return [
+      ['cmd', ['/c', 'start', '', 'chrome']],
+      ['cmd', ['/c', 'start', '', 'msedge']],
+    ];
+  }
+  return [
+    ['google-chrome', []],
+    ['chromium', []],
+    ['chromium-browser', []],
+    ['microsoft-edge', []],
+  ];
+}
+
+export function openAppWindowUrl(value, { run = execFileSync } = {}) {
+  const url = normalizeExternalUrl(value);
+  if (!url) return null;
+  for (const [command, args] of appModeCandidates()) {
+    try {
+      run(command, [...args, `--app=${url}`], { stdio: 'ignore', timeout: 5_000 });
+      return url;
+    } catch {
+      // Missing binary or non-zero exit: try the next candidate.
+    }
+  }
+  return null;
+}

package/src/shell/repl.js

@@ -94,7 +94,7 @@ const COMMAND_COMPLETION_DESCRIPTIONS = {
   '/clear': 'Clear the conversation screen.',
   '/chat': 'Switch free text to direct LLM chat without tools.',
   '/agent': 'Switch to agent mode, or run one agent request with /agent <question>.',
-  '/openui': 'Open the workspace web UI in the browser.',
+  '/openui': 'Open the workspace web UI as a chromeless desktop window (Chrome/Edge), or a browser tab as fallback.',
   '/run': 'Inspect, cancel, kill runtime runs, or start a capability run.',
   '/approve': 'Approve a pending runtime run or tool.',
 };