@dotdrelle/wiki-manager 0.14.7 → 0.14.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dotdrelle/wiki-manager",
-  "version": "0.14.7",
+  "version": "0.14.9",
   "description": "Agentic shell and orchestration cockpit for llm-wiki workspaces.",
   "license": "PolyForm-Noncommercial-1.0.0",
   "author": "dotrelle",

package/src/agent/graph.js

@@ -403,6 +403,23 @@ function summarizeToolArguments(rawArguments) {
   }
 }
 
+function googleOAuthUrlFromMessages(messages) {
+  for (const message of [...(messages ?? [])].reverse()) {
+    if (message?.role !== 'tool') continue;
+    const match = String(message.content ?? '').match(/https:\/\/accounts\.google\.com\/[^\s<>"')\]]+/i);
+    if (match) return match[0];
+  }
+  return null;
+}
+
+function preserveRequiredOAuthUrl(content, messages) {
+  const text = String(content ?? '');
+  const url = googleOAuthUrlFromMessages(messages);
+  if (!url || text.includes(url)) return text;
+  const prefix = text.trimEnd();
+  return `${prefix}${prefix ? '\n\n' : ''}Lien d’autorisation Google : ${url}`;
+}
+
 function buildQueuedResult(session, item, activeJobId = null) {
   const message = activeJobId != null
     ? `Production job queued as ${item.id}; waiting for ${activeJobId}.`
@@ -709,6 +726,13 @@ async function handleRuntimeControlTool(session, tool, args = {}) {
     if (tool === 'delegate') {
       const objective = String(args.objective ?? '').trim();
       if (!objective) return 'Delegation rejected: missing objective.';
+      const connectorConfig = connectorConfigurationTarget(session, objective);
+      if (connectorConfig?.setupTool) {
+        return `Delegation rejected: configuring or authenticating ${connectorConfig.serverName} is not an orchestrated export. Call the offered ${connectorConfig.serverName}__${connectorConfig.setupTool} tool directly and present its authorization instructions or URL to the user.`;
+      }
+      if (connectorConfig) {
+        return `Delegation rejected: ${connectorConfig.serverName} advertises no setup or authentication tool. Do not call an unrelated data tool and do not delegate to export. Explain conversationally that authentication must be completed outside MCP, using only configuration instructions already available in the current context.`;
+      }
       const result = await postRuntimeDelegate(objective, { url, workspace });
       return result?.runId
         ? JSON.stringify({
@@ -746,6 +770,24 @@ async function handleRuntimeControlTool(session, tool, args = {}) {
   }
 }
 
+function connectorConfigurationTarget(session, objective) {
+  const text = String(objective ?? '').toLowerCase();
+  if (!/(?:configur|connect|authent|oauth|setup|sign[ -]?in)/i.test(text)) return null;
+  for (const [serverName, server] of Object.entries(session?.mcp ?? {})) {
+    if (server?.status !== 'connected' || !Array.isArray(server.tools) || server.tools.length === 0) continue;
+    const aliases = String(serverName).toLowerCase().split(/[^a-z0-9]+/).filter((part) => part.length >= 3);
+    if (!aliases.some((alias) => text.includes(alias))) continue;
+    const setupTool = server.tools.find((tool) => {
+      const name = String(tool?.name ?? '').toLowerCase();
+      const description = String(tool?.description ?? '').toLowerCase();
+      return /(?:^|_)(?:setup|config|configure|auth|authenticate|oauth|connect)(?:_|$)/.test(name)
+        || /(?:initiat|start|configure|authenticate).{0,30}(?:oauth|authentication)/.test(description);
+    })?.name ?? null;
+    return { serverName, setupTool };
+  }
+  return null;
+}
+
 function handleWikiTool(session, tool, args) {
   if (tool === 'plan_set') {
     const steps = Array.isArray(args.steps) ? args.steps : [];
@@ -888,10 +930,10 @@ export function buildAgentSystemPrompt(state) {
     'For service actions, recommend only available service primitives from Available primitives, with the exact service name when the primitive supports one.',
     'Scope discipline: execute ONLY the action(s) the user explicitly requested. Never chain additional mutating operations (ingest, build, export, polish, delete, send…) that the user did not ask for — even when diagnostics or recommendations suggest them. Finish with the requested result and stop. Example: "applique les recommandations de config" means apply the config; it does NOT authorize launching the ingest those recommendations mention.',
     state.session.runtime?.url
-      ? 'The runtime is connected and runtime__delegate is bound and available to you right now — it is a tool you call directly, not a slash command or a missing primitive. It is the ONLY way to execute an action (ingest, build, export, configure, send…). Never tell the user that delegation or the runtime is unavailable while it is connected; call runtime__delegate instead.'
+      ? 'The runtime is connected and runtime__delegate is bound and available for heavy orchestrated operations (ingest, build, export, polish, pipeline). Single-step connector actions such as configure, authenticate, add a source, convert, search, or send use the connected MCP tool directly. Never delegate connector configuration to an export capability.'
       : 'No runtime is connected, so you cannot execute actions. State that plainly and name the runtime connection as the missing capability — do not invent a workaround.',
     'If the connector or service needed for a requested read or action is absent from the Connected MCP tools above (its service is not running — e.g. CME, documents, or production), say plainly that this service is not connected and name it as the missing capability. Never redirect a simple read (e.g. "give me the CME config") to an "agent action", never invent its result, and never propose a workaround. Only requests you can actually serve with a listed tool are answered with data.',
-    'For any requested action, call runtime__delegate with the user objective only. Never choose a capability, operation, agent, plan, or implementation yourself. The runtime resolves the registry and validates the provider plan before accepting. Never call <provider>__agent_plan, <provider>__agent_execute, legacy production__production_start_job, wiki__plan_set, or wiki__plan_done from interactive chat.',
+    'For heavy orchestrated operations only (ingest, build, export, polish, pipeline), call runtime__delegate with the user objective only. For a single-step connector action, call the offered connector tool directly. Never choose a capability, operation, agent, plan, or implementation yourself. Never call <provider>__agent_plan, <provider>__agent_execute, legacy production__production_start_job, wiki__plan_set, or wiki__plan_done from interactive chat.',
     'Do not ask the user which sources, files, connectors, or templates to use for an ingest, build, or export: the specialized agent discovers them from the workspace. When the objective is clear (e.g. "lance une ingestion"), delegate it as stated, without a clarifying question.',
     'Promise only what the resolved capability actually exposes in its declared contract (the input schema the specialized agent publishes for that capability). When the user requests an execution parameter — a batch or chunk size, a count "N at a time", concurrency, ordering, priority, or any tuning knob — apply it only if that parameter exists in the target capability\'s published input schema. Otherwise do not confirm or promise it: delegate the objective, and if the user explicitly asked for that parameter, say plainly in one line that you started the work but do not control that aspect (the runtime and the specialized agent decide it). Never state or imply a parameter was applied when the agent contract cannot enforce it.',
     'If runtime__delegate returns a blocker or no specialized provider is available, report only that concrete blocker concisely. Never replace the missing execution path with a suggested slash command, skill, MCP tool name, manual file move, administrator escalation, or alternative workflow unless the user explicitly asks for alternatives.',
@@ -963,15 +1005,18 @@ function toolsForClassification(classification, writeTools, session = null) {
   return [SHELL_READ_COMMAND_TOOL, ...controlTools, ...capabilityRunTools, ...writeTools];
 }
 
+const DONNA_READ_VERBS = new Set(['status', 'list', 'search', 'read', 'get', 'fetch']);
+
 export function isDonnaReadTool(item) {
   const name = String(item?.function?.name ?? '');
   if (!name || name.startsWith('shell__') || name === 'wiki__plan_set' || name === 'wiki__plan_done') return false;
   if (item?.readOnly === true) return true;
   const tool = name.includes('__') ? name.slice(name.indexOf('__') + 2) : name;
-  return tool === 'wiki_workspace_status'
-    || tool === 'agent_describe'
-    || tool === 'agent_status'
-    || /(?:^|_)(?:status|list|search|read|get)$/.test(tool);
+  if (tool === 'wiki_workspace_status' || tool === 'agent_describe' || tool === 'agent_status') return true;
+  // Match a read verb anywhere in the underscore-tokenized name, not just as
+  // a trailing suffix — third-party MCPs don't all name tools verb-last
+  // (e.g. exa's "web_search_exa"/"web_fetch_exa" put the verb in the middle).
+  return tool.split('_').some((segment) => DONNA_READ_VERBS.has(segment));
 }
 
 // Two-tier tool policy. Donna may call any connected MCP tool directly
@@ -1192,6 +1237,15 @@ export function createAgentGraph(options = {}) {
         };
       }
 
+      // Authentication URLs are execution outputs, not optional prose. Small
+      // models sometimes summarize an OAuth tool result as "open the supplied
+      // link" while dropping the link itself, leaving ShellUI unusable even
+      // though the MCP call succeeded. Preserve that exact URL deterministically.
+      const finalContent = preserveRequiredOAuthUrl(result.content, conversationMessages);
+      if (finalContent !== String(result.content ?? '')) {
+        result.content = finalContent;
+        result.message = { ...(result.message ?? { role: 'assistant' }), content: finalContent };
+      }
       const invalidCommands = invalidSuggestedSlashCommands(result.content, state.session);
       const leakedTools = invalidUserFacingToolNames(result.content, state.session);
       if (invalidCommands.length > 0 || leakedTools.length > 0) {
@@ -1246,7 +1300,7 @@ export function createAgentGraph(options = {}) {
 
       // Fallback path (streamWithTools unavailable): hand off to runLine for streaming.
       state.session._onStep?.('Agent: streaming final answer…');
-      if (typeof llm.stream === 'function') {
+      if (typeof llm.stream === 'function' && !googleOAuthUrlFromMessages(conversationMessages)) {
         return {
           response: null,
           pendingToolCalls: null,

package/src/agent/graph.test.js

@@ -395,6 +395,110 @@ test('Donna delegates the objective without choosing technical identifiers', asy
   }
 });
 
+test('Donna refuses to delegate connector authentication to an export capability', async () => {
+  const originalFetch = globalThis.fetch;
+  const fetchedUrls = [];
+  globalThis.fetch = async (url, options = {}) => {
+    fetchedUrls.push(String(url));
+    const body = JSON.parse(String(options.body ?? '{}'));
+    assert.equal(body.params?.name, 'start_google_auth');
+    return {
+      ok: true,
+      status: 200,
+      headers: { get: () => null },
+      text: async () => JSON.stringify({ result: { content: [{ type: 'text', text: 'ACTION REQUIRED: authorize at https://accounts.google.com/o/oauth2/auth?client_id=test&state=abc' }] } }),
+    };
+  };
+  let turn = 0;
+  const session = sessionBase({
+    runtime: { url: 'http://runtime.test' },
+    mcp: {
+      'google-workspace': {
+        status: 'connected',
+        url: 'http://google.test/mcp',
+        tools: [
+          {
+            name: 'start_google_auth',
+            description: 'Manually initiate Google OAuth authentication flow.',
+            inputSchema: { type: 'object', additionalProperties: true },
+          },
+          { name: 'search_gmail_messages', inputSchema: { type: 'object', additionalProperties: true } },
+        ],
+      },
+    },
+    llm: {
+      async completeWithTools() {
+        turn += 1;
+        if (turn === 1) return {
+          content: null,
+          message: { role: 'assistant', content: null },
+          tool_calls: [{ id: 'wrong-delegate', type: 'function', function: { name: 'runtime__delegate', arguments: '{"objective":"je veux configurer google"}' } }],
+        };
+        if (turn === 2) return {
+          content: null,
+          message: { role: 'assistant', content: null },
+          tool_calls: [{ id: 'google-auth', type: 'function', function: { name: 'google-workspace__start_google_auth', arguments: '{}' } }],
+        };
+        return {
+          content: 'J’ai lancé l’authentification. Ouvre le lien fourni.',
+          message: { role: 'assistant', content: 'J’ai lancé l’authentification. Ouvre le lien fourni.' },
+          tool_calls: null,
+        };
+      },
+    },
+  });
+
+  try {
+    const result = await createAgentGraph().invoke({ input: 'je veux configurer google', session });
+    assert.match(result.response, /J’ai lancé l’authentification/);
+    assert.match(result.response, /https:\/\/accounts\.google\.com\/o\/oauth2\/auth\?client_id=test&state=abc/);
+    assert.equal(fetchedUrls.some((url) => url.includes('runtime.test')), false);
+    assert.equal(fetchedUrls.some((url) => url.includes('google.test')), true);
+  } finally {
+    globalThis.fetch = originalFetch;
+  }
+});
+
+test('Donna does not invent a setup tool when a connector advertises data tools only', async () => {
+  const originalFetch = globalThis.fetch;
+  globalThis.fetch = async () => assert.fail('neither runtime delegation nor an unrelated data tool should be called');
+  let turn = 0;
+  const session = sessionBase({
+    runtime: { url: 'http://runtime.test' },
+    mcp: {
+      acme: {
+        status: 'connected',
+        url: 'http://acme.test/mcp',
+        tools: [{ name: 'list_records', description: 'List records.', inputSchema: { type: 'object' } }],
+      },
+    },
+    llm: {
+      async completeWithTools({ messages }) {
+        turn += 1;
+        if (turn === 1) return {
+          content: null,
+          message: { role: 'assistant', content: null },
+          tool_calls: [{ id: 'wrong-delegate', type: 'function', function: { name: 'runtime__delegate', arguments: '{"objective":"configure acme"}' } }],
+        };
+        const refusal = (messages ?? []).filter((message) => message.role === 'tool').at(-1)?.content;
+        assert.match(String(refusal), /advertises no setup or authentication tool/);
+        return {
+          content: 'ACME doit être authentifié hors de cette interface.',
+          message: { role: 'assistant', content: 'ACME doit être authentifié hors de cette interface.' },
+          tool_calls: null,
+        };
+      },
+    },
+  });
+
+  try {
+    const result = await createAgentGraph().invoke({ input: 'configure acme', session });
+    assert.equal(result.response, 'ACME doit être authentifié hors de cette interface.');
+  } finally {
+    globalThis.fetch = originalFetch;
+  }
+});
+
 test('runtime status does not manufacture a plan', async () => {
   const originalFetch = globalThis.fetch;
   globalThis.fetch = async () => ({
@@ -697,16 +801,10 @@ test('buildAgentSystemPrompt forbids inventing slash commands or arguments', ()
   assert.doesNotMatch(prompt, /executor:"/);
 });
 
-test('workspace package manifest is not exposed as an executable skill', () => {
+test('workspace skills are discovered from the fixed .wiki/skills directory', () => {
   const root = mkdtempSync(join(tmpdir(), 'wiki-manager-skills-'));
   try {
     mkdirSync(join(root, '.wiki', 'skills'), { recursive: true });
-    writeFileSync(join(root, 'skill.yaml'), [
-      'name: basic',
-      'description: Demo workspace package',
-      'entrypoints:',
-      '  uiSkillDir: .wiki/skills',
-    ].join('\n'));
     writeFileSync(join(root, '.wiki', 'skills', 'ingest.md'), [
       '---',
       'name: ingest',
@@ -716,7 +814,6 @@ test('workspace package manifest is not exposed as an executable skill', () => {
     ].join('\n'));
 
     const prompt = buildAgentSystemPrompt({ session: sessionBase({ workspacePath: root }) });
-    assert.doesNotMatch(prompt, /\/basic:/);
     assert.match(prompt, /\/ingest: Ingest pending sources/);
   } finally {
     rmSync(root, { recursive: true, force: true });

package/src/cli/wiki-manager.js

@@ -6,11 +6,11 @@ import { ensureManagerScaffold, loadManagerEnv } from '../core/env.js';
 loadManagerEnv();
 import { createAgentGraph } from '../agent/graph.js';
 import { handleSlashCommand, printHelp, printVersion, refreshMcpRuntimeStatus } from '../commands/slash.js';
-import { runShell } from '../shell/repl.js';
+import { runShell, runHeadlessChatTurn } from '../shell/repl.js';
 import { runChecks } from '../core/startupCheck.js';
 import { applySessionWikircProfile } from '../core/sessionConfig.js';
 import { listWikircProfiles } from '../core/wikirc.js';
-import { callMcpTool, formatMcpToolResult } from '../core/mcp.js';
+import { callMcpTool, formatMcpToolResult, readChatAccessConfig } from '../core/mcp.js';
 import { extractActivity, parseJsonText, sessionActivities, terminalFailures } from '../core/activity.js';
 import { syncActivitiesToPlan, formatPlanStatus } from '../core/plan.js';
 import { createAgentEvent, dispatchAgentEvent, reduceAgentEvents } from '../core/agentEvents.js';
@@ -478,7 +478,7 @@ async function runRuntime(argv, agent) {
       'Defaults:',
       '  --host 127.0.0.1',
       '  --port 7788',
-      '  --state-dir .wiki-manager',
+      '  --state-dir .wiki/runtime',
     ].join('\n'));
     return;
   }
@@ -1039,7 +1039,25 @@ async function runRuntime(argv, agent) {
       workspace: context.workspace ?? null,
       payload: { content: input },
     }));
-    const response = await runAgentTurn(agent, ephemeral, input, { messages, signal });
+    // Read-only chat turn: same chatAccess policy as the Shell UI's /chat, now
+    // reachable over HTTP so `wiki serve` chat mode gets read tools without
+    // duplicating the loop. Anything other than mode === 'chat' stays the full
+    // unrestricted agent turn.
+    const chatMode = String(body.mode ?? '').toLowerCase() === 'chat';
+    let response;
+    if (chatMode) {
+      ephemeral.chatMode = true;
+      ephemeral.chatAccess = readChatAccessConfig();
+      const history = messages.length && messages[messages.length - 1]?.role === 'user'
+        ? messages.slice(0, -1)
+        : messages;
+      response = await runHeadlessChatTurn(ephemeral, input, {
+        history,
+        onStep: ephemeral._onStep,
+      });
+    } else {
+      response = await runAgentTurn(agent, ephemeral, input, { messages, signal });
+    }
     ensureInteractiveAssistantMessage(ephemeral, response, {
       turnId,
       workspace: context.workspace ?? null,

package/src/commands/slash.js

@@ -232,8 +232,8 @@ function statLine(label, stat) {
   return `${label}: ${stat.count} (${formatBytes(stat.totalBytes)})`;
 }
 
-function workspaceStatsText(stats) {
-  if (!stats) return 'No workspace loaded.';
+function workspaceStatsColumns(stats) {
+  if (!stats) return { left: 'No workspace loaded.', right: '' };
   const hints = [];
   if (stats.untracked.count > 0) {
     hints.push(`${stats.untracked.count} raw/untracked document(s) are waiting for ingest.`);
@@ -280,11 +280,10 @@ function workspaceStatsText(stats) {
   ]);
   const hintsColumn = sectionBlock('Hints', hints.length > 0 ? hints : ['No immediate content action detected.']);
 
-  return [
-    twoColumns(wikiColumn, rawColumn),
-    '',
-    twoColumns(deliveryColumn, `${internalColumn}\n\n${hintsColumn}`),
-  ].join('\n');
+  return {
+    left: [wikiColumn, deliveryColumn].join('\n\n'),
+    right: [rawColumn, internalColumn, hintsColumn].join('\n\n'),
+  };
 }
 
 function workspaceLoadedText(workspace, summary, session) {
@@ -530,16 +529,18 @@ async function statusText(session) {
   const runtimeColumn = sectionBlock('Runtime', (states ? serviceStatesText(states) : 'Docker runtime not available or no workspace loaded.').split('\n'));
   const mcpColumn = sectionBlock('MCP', formatMcpStatus(session.mcp).split('\n'));
   const mcpToolsColumn = sectionBlock('MCP tool summary', formatMcpToolSummary(session.mcp).split('\n'));
-
-  return [
-    twoColumns(workspaceColumn, configColumn),
-    '',
-    workspaceStatsText(workspaceStats),
-    '',
-    runtimeColumn,
-    '',
-    twoColumns(mcpColumn, mcpToolsColumn),
-  ].join('\n');
+  const stats = workspaceStatsColumns(workspaceStats);
+
+  const leftColumn = [workspaceColumn, stats.left, runtimeColumn, mcpColumn].filter(Boolean).join('\n\n');
+  const rightColumn = [configColumn, stats.right, mcpToolsColumn].filter(Boolean).join('\n\n');
+
+  // Leading/trailing padding row on *both* columns so the boxed pair doesn't
+  // butt directly against the pane border when the view is scrolled to show
+  // the tail. A single space on each side (not '') keeps the row tab-joined,
+  // so both the left and right box render — an empty string on either side
+  // of the tab makes twoColumns drop the pairing and only the left box shows.
+  const pad = ' \t ';
+  return [pad, twoColumns(leftColumn, rightColumn), pad].join('\n');
 }
 
 function loadWorkspaceSystemPrompt(workspacePath) {

package/src/core/buildInfo.json

@@ -1,4 +1,4 @@
 {
-  "version": "0.14.7",
-  "commit": "f17765a"
+  "version": "0.14.9",
+  "commit": "768fb25"
 }

package/src/core/env.js

@@ -1,4 +1,4 @@
-import { copyFileSync, existsSync, readFileSync, writeFileSync } from 'node:fs';
+import { copyFileSync, existsSync, mkdirSync, readFileSync, renameSync, writeFileSync } from 'node:fs';
 import { dirname, isAbsolute, join, resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';
 
@@ -13,7 +13,14 @@ export function managerStateDir() {
 }
 
 export function managerRuntimeDir() {
-  return join(managerStateDir(), '.wiki-manager');
+  const root = managerStateDir();
+  const runtimeDir = join(root, '.wiki', 'runtime');
+  const legacyDir = join(root, '.wiki-manager');
+  if (!existsSync(runtimeDir) && existsSync(legacyDir)) {
+    mkdirSync(join(root, '.wiki'), { recursive: true });
+    renameSync(legacyDir, runtimeDir);
+  }
+  return runtimeDir;
 }
 
 export function defaultRuntimeStateDir() {

package/src/core/mcp.js

@@ -1,7 +1,7 @@
 import { existsSync, readFileSync } from 'node:fs';
 import { managerEnvFile, managerMcpEndpointsFile, readEnvFile } from './env.js';
 
-const WIKI_MANAGER_VERSION = '0.14.7';
+const WIKI_MANAGER_VERSION = '0.14.9';
 
 function envValue(key) {
   const filePath = managerEnvFile();
@@ -29,6 +29,27 @@ function normalizeHeaders(headers) {
   );
 }
 
+// An endpoint's url/headers may reference `${VAR}` placeholders with no
+// `:-default`. If that env var is unset, the placeholder interpolates to ''
+// (see interpolateEnv) and the endpoint would otherwise look "configured"
+// with a blank credential — then discoverMcpTools happily probes the live
+// endpoint and can report it "connected" even though it has no real auth.
+function hasMissingRequiredEnv(value) {
+  if (typeof value !== 'string') return false;
+  let missing = false;
+  value.replace(/\$\{([^}]+)\}/g, (_, expr) => {
+    const sep = expr.indexOf(':-');
+    if (sep === -1 && !envValue(expr)) missing = true;
+    return '';
+  });
+  return missing;
+}
+
+function endpointHasMissingCredentials(endpoint) {
+  const headerValues = Object.values(endpoint?.headers ?? {}).filter((value) => typeof value === 'string');
+  return [String(endpoint?.url ?? ''), ...headerValues].some(hasMissingRequiredEnv);
+}
+
 function normalizeExternalUrlForRuntime(url) {
   if (process.env.WIKI_MANAGER_KEEP_DOCKER_HOST === '1') return url;
   try {
@@ -57,8 +78,14 @@ export function readChatAccessConfig() {
   if (!chatAccess || typeof chatAccess !== 'object' || Array.isArray(chatAccess)) return null;
   const servers = {};
   for (const [name, entry] of Object.entries(chatAccess.servers ?? {})) {
-    if (entry?.allow === '*') servers[name] = { allow: '*' };
-    else if (Array.isArray(entry?.allow)) servers[name] = { allow: entry.allow.map(String).filter(Boolean) };
+    // "*" is also commonly written as a one-element array (["*"]) since every
+    // other "allow" example in this config is an array of tool names — treat
+    // both forms as the same wildcard rather than silently allowing nothing.
+    if (entry?.allow === '*' || (Array.isArray(entry?.allow) && entry.allow.length === 1 && entry.allow[0] === '*')) {
+      servers[name] = { allow: '*' };
+    } else if (Array.isArray(entry?.allow)) {
+      servers[name] = { allow: entry.allow.map(String).filter(Boolean) };
+    }
   }
   const maxToolIterations = Number.isFinite(Number(chatAccess.maxToolIterations)) && Number(chatAccess.maxToolIterations) > 0
     ? Math.floor(Number(chatAccess.maxToolIterations))
@@ -75,24 +102,27 @@ function readExternalMcpEndpoints() {
   return Object.fromEntries(
     Object.entries(servers)
       .filter(([, endpoint]) => endpoint?.url)
-      .map(([name, endpoint]) => [
-        name,
-        {
-          ...endpointStatus(true),
-          url: normalizeExternalUrlForRuntime(interpolateEnv(String(endpoint.url))),
-          configuredUrl: interpolateEnv(String(endpoint.url)),
-          headers: normalizeHeaders(endpoint.headers),
-          // Tools the endpoint marks approval-gated: Donna may still call them
-          // directly (they are single-step tools), but toolRequiresApproval
-          // makes the call wait for the user's confirmation first (e.g. a
-          // destructive cme_export_run). Agent/operator owned — no hard-coded
-          // business name in the manager.
-          requireApproval: Array.isArray(endpoint.requireApproval)
-            ? endpoint.requireApproval.map(String).filter(Boolean)
-            : undefined,
-          external: true,
-        },
-      ]),
+      .map(([name, endpoint]) => {
+        const missingCredentials = endpointHasMissingCredentials(endpoint);
+        return [
+          name,
+          {
+            ...endpointStatus(!missingCredentials, missingCredentials ? 'credential not set' : ''),
+            url: normalizeExternalUrlForRuntime(interpolateEnv(String(endpoint.url))),
+            configuredUrl: interpolateEnv(String(endpoint.url)),
+            headers: normalizeHeaders(endpoint.headers),
+            // Tools the endpoint marks approval-gated: Donna may still call them
+            // directly (they are single-step tools), but toolRequiresApproval
+            // makes the call wait for the user's confirmation first (e.g. a
+            // destructive cme_export_run). Agent/operator owned — no hard-coded
+            // business name in the manager.
+            requireApproval: Array.isArray(endpoint.requireApproval)
+              ? endpoint.requireApproval.map(String).filter(Boolean)
+              : undefined,
+            external: true,
+          },
+        ];
+      }),
   );
 }
 

package/src/core/skills.js

@@ -1,6 +1,5 @@
 import { existsSync, readdirSync, readFileSync } from 'node:fs';
 import { basename, join } from 'node:path';
-import { parse as parseYaml } from 'yaml';
 
 const SKILL_NAME_RE = /^[a-zA-Z0-9_-]{1,80}$/;
 const DEFAULT_UI_SKILL_DIR = '.wiki/skills';
@@ -51,33 +50,6 @@ export function readOptionalText(filePath) {
   return readFileSync(filePath, 'utf8').trim();
 }
 
-function safeRelativeEntry(value, fallback) {
-  const text = String(value || fallback).trim();
-  if (!text || text.startsWith('/') || text.includes('..')) return fallback;
-  return text;
-}
-
-function readWorkspaceManifest(workspacePath) {
-  const manifestPath = join(workspacePath, 'skill.yaml');
-  if (!existsSync(manifestPath)) return null;
-  try {
-    const manifest = parseYaml(readFileSync(manifestPath, 'utf8'));
-    if (!manifest || typeof manifest !== 'object' || Array.isArray(manifest)) return null;
-    return { manifest, manifestPath };
-  } catch {
-    return null;
-  }
-}
-
-function workspaceUiSkillDir(loadedManifest = null) {
-  if (!loadedManifest) return DEFAULT_UI_SKILL_DIR;
-  const { manifest } = loadedManifest;
-  const entrypoints = manifest?.entrypoints && typeof manifest.entrypoints === 'object'
-    ? manifest.entrypoints
-    : {};
-  return safeRelativeEntry(entrypoints.uiSkillDir, DEFAULT_UI_SKILL_DIR);
-}
-
 function collectDirectorySkills(dir, scope) {
   if (!existsSync(dir)) return [];
   return readdirSync(dir, { withFileTypes: true })
@@ -92,8 +64,7 @@ function collectDirectorySkills(dir, scope) {
 export function listSkills(session = {}) {
   const skills = [];
   if (session.workspacePath) {
-    const loadedManifest = readWorkspaceManifest(session.workspacePath);
-    skills.push(...collectDirectorySkills(join(session.workspacePath, workspaceUiSkillDir(loadedManifest)), 'workspace'));
+    skills.push(...collectDirectorySkills(join(session.workspacePath, DEFAULT_UI_SKILL_DIR), 'workspace'));
   }
 
   const byName = new Map();

package/src/runtime/recoveryManager.test.js

@@ -81,7 +81,7 @@ test('recoveryManager requeues an active job by idempotencyKey when status is st
 
 function storeWithActiveTask() {
   const root = mkdtempSync(join(tmpdir(), 'wiki-manager-recovery-'));
-  const stateDir = join(root, '.wiki-manager');
+  const stateDir = join(root, '.wiki', 'runtime');
   const store = openRuntimeStore({ stateDir });
   const runId = 'run-recovery';
   const taskId = 'run-recovery:build-a';

package/src/runtime/server.js

@@ -1,6 +1,6 @@
 import { createServer } from 'node:http';
 import { randomUUID, timingSafeEqual } from 'node:crypto';
-import { createAgentEvent, dispatchAgentEvent, resetSessionProjection } from '../core/agentEvents.js';
+import { createAgentEvent, dispatchAgentEvent, resetSessionProjection, reduceAgentEvents } from '../core/agentEvents.js';
 import { activeCacertPath } from '../core/cacert.js';
 import { normalizePlanPatch, rebasePlanPatch } from '../core/planPatch.js';
 import { validateContractInDev } from '../contracts/schemas.js';
@@ -284,7 +284,11 @@ export function startRuntimeServer({
           sendJson(response, 400, { error: 'Missing input.' });
           return;
         }
-        if (context.running) {
+        // Read-only chat turns intentionally remain available while an agent
+        // run is active. Other interactive turns still become control
+        // messages so they cannot start a competing agent decision.
+        const readOnlyChat = String(body.mode ?? '').toLowerCase() === 'chat';
+        if (context.running && !readOnlyChat) {
           const result = await handleControlMessage(context, store, input, {
             intent: body.intent,
             startNextControlRequest,
@@ -305,7 +309,7 @@ export function startRuntimeServer({
           // A preceding serialized turn may have delegated and started a run
           // after this request was accepted. Reclassify against the fresh
           // state instead of starting another interactive decision in parallel.
-          if (context.running) {
+          if (context.running && !readOnlyChat) {
             const result = await handleControlMessage(context, store, input, {
               intent: body.intent,
               startNextControlRequest,
@@ -595,10 +599,18 @@ function controlStatus(context, store) {
   };
 }
 
-function runtimeState(context, store, { workspace = null, session = null } = {}) {
+export function runtimeState(context, store, { workspace = null, session = null } = {}) {
   const state = store.getState(context?.session ?? session ?? null, { workspace });
   return {
     ...state,
+    // Interactive (runtime_turn) replies are persisted as events but never
+    // merged into the canonical in-memory projection, so a state built from
+    // that projection omits them — chat mode and conversational agent turns
+    // then show no reply at all. Rebuild the conversation from the full
+    // persisted event log (the same source interactive turns use to seed their
+    // own history) so those replies surface. The log is a superset of the
+    // canonical run conversation, so run rendering is unaffected.
+    conversation: reduceAgentEvents(store.listEvents({ workspace })).conversation,
     status: context?.running ? 'running' : state.status ?? 'idle',
     running: Boolean(context?.running),
     runId: context?.currentRunId ?? state.runId ?? null,

package/src/runtime/server.test.js

@@ -2,7 +2,14 @@ import assert from 'node:assert/strict';
 import test from 'node:test';
 import { createAgentEvent, dispatchAgentEvent } from '../core/agentEvents.js';
 import { createInteractiveSession, ensureInteractiveAssistantMessage } from '../cli/wiki-manager.js';
-import { startRuntimeServer } from './server.js';
+import { runtimeState, startRuntimeServer as startRuntimeServerImpl } from './server.js';
+
+// Most server tests exercise endpoint behavior rather than authentication. Keep
+// them independent from a developer's WIKI_MANAGER_RUNTIME_TOKEN environment;
+// auth-specific tests can still override this default explicitly.
+function startRuntimeServer(options) {
+  return startRuntimeServerImpl({ token: '', ...options });
+}
 
 test('interactive runtime sessions isolate canonical run state', () => {
   const mcp = { wiki: { status: 'connected' } };
@@ -40,6 +47,35 @@ test('interactive turns publish a fallback assistant message exactly once', () =
   assert.equal(published[0].payload.content, 'Réponse concise.');
 });
 
+test('runtime state rebuilds interactive conversation from persisted events', () => {
+  const user = {
+    ...createAgentEvent('user_message', { origin: 'runtime_turn', turnId: 'turn-1', workspace: 'demo', payload: { content: 'Bonjour' } }),
+    origin: 'runtime_turn', turnId: 'turn-1', workspace: 'demo',
+  };
+  const assistant = {
+    ...createAgentEvent('assistant_message', { origin: 'runtime_turn', turnId: 'turn-1', workspace: 'demo', payload: { content: 'Salut !' } }),
+    origin: 'runtime_turn', turnId: 'turn-1', workspace: 'demo',
+  };
+  const session = { agentProjection: { conversation: [] } };
+  const store = {
+    getState: (receivedSession) => {
+      assert.equal(receivedSession, session);
+      return { status: 'idle', conversation: [] };
+    },
+    listEvents: ({ workspace }) => {
+      assert.equal(workspace, 'demo');
+      return [user, assistant];
+    },
+  };
+
+  const state = runtimeState({ workspace: 'demo', session, running: false }, store, { workspace: 'demo' });
+
+  assert.deepEqual(state.conversation.map(({ role, content }) => ({ role, content })), [
+    { role: 'user', content: 'Bonjour' },
+    { role: 'assistant', content: 'Salut !' },
+  ]);
+});
+
 test('runtime server checks bearer and x-runtime-token credentials', async (t) => {
   let handle;
   try {
@@ -1478,6 +1514,50 @@ test('runtime server accepts an interactive turn without starting a run', async
   }
 });
 
+test('runtime server keeps read-only chat turns available during an active run', async (t) => {
+  const context = { workspace: 'demo', session: {}, running: true };
+  let received = null;
+  let handle;
+  try {
+    handle = await startRuntimeServer({
+      host: '127.0.0.1',
+      port: 0,
+      token: 'runtime-secret',
+      store: {
+        dbPath: ':memory:',
+        getState: () => ({ status: 'running' }),
+        listEvents: () => [],
+      },
+      getContext: async () => context,
+      run: async () => assert.fail('/turn must not start another runtime run'),
+      turn: async (_context, body, meta) => { received = { body, meta }; },
+    });
+  } catch (err) {
+    if (err?.code === 'EPERM') {
+      t.skip('network listen is not permitted in this sandbox');
+      return;
+    }
+    throw err;
+  }
+  try {
+    const response = await fetch(`http://127.0.0.1:${handle.port}/turn`, {
+      method: 'POST',
+      headers: { authorization: 'Bearer runtime-secret', 'content-type': 'application/json' },
+      body: JSON.stringify({ input: 'Quel est le statut CME ?', mode: 'chat', workspace: 'demo' }),
+    });
+    assert.equal(response.status, 202);
+    const body = await response.json();
+    assert.equal(body.kind, 'turn');
+    await context.interactiveTurn;
+    assert.equal(received.body.mode, 'chat');
+    assert.equal(received.body.input, 'Quel est le statut CME ?');
+    assert.equal(received.meta.turnId, body.turnId);
+    assert.equal(context.running, true);
+  } finally {
+    await handle.close();
+  }
+});
+
 test('runtime health reports active runs across workspaces', async (t) => {
   let handle;
   try {

package/src/runtime/store.js

@@ -1261,7 +1261,10 @@ function ensureRuntimeMeta(metaPath) {
 
 function migrateLegacyRuntimeMeta(metaPath) {
   if (existsSync(metaPath)) return;
-  const legacyPath = join(dirname(metaPath), '..', '.wiki', 'meta.json');
+  const stateDir = dirname(metaPath);
+  const legacyPath = stateDir.endsWith(join('.wiki', 'runtime'))
+    ? join(stateDir, '..', 'meta.json')
+    : join(stateDir, '..', '.wiki', 'meta.json');
   if (!existsSync(legacyPath)) return;
   mkdirSync(dirname(metaPath), { recursive: true });
   renameSync(legacyPath, metaPath);

package/src/runtime/store.test.js

@@ -8,7 +8,7 @@ import { createAgentEvent, dispatchAgentEvent } from '../core/agentEvents.js';
 import { openRuntimeStore } from './store.js';
 
 function runtimeStateDir() {
-  return join(mkdtempSync(join(tmpdir(), 'wiki-manager-runtime-')), '.wiki-manager');
+  return join(mkdtempSync(join(tmpdir(), 'wiki-manager-runtime-')), '.wiki', 'runtime');
 }
 
 function plannedTask(id) {
@@ -223,10 +223,10 @@ test('runtime store persists task assignments attempts and results from events',
 
 test('runtime store writes schema versions to sqlite and meta file', () => {
   const root = mkdtempSync(join(tmpdir(), 'wiki-manager-runtime-'));
-  const stateDir = join(root, '.wiki-manager');
+  const stateDir = join(root, '.wiki', 'runtime');
   const store = openRuntimeStore({ stateDir });
   const version = store.db.prepare('PRAGMA user_version').get().user_version;
-  const meta = JSON.parse(readFileSync(join(root, '.wiki-manager', 'meta.json'), 'utf8'));
+  const meta = JSON.parse(readFileSync(join(root, '.wiki', 'runtime', 'meta.json'), 'utf8'));
 
   assert.equal(version, 1);
   assert.equal(meta.schemaVersion, 1);
@@ -249,7 +249,7 @@ test('runtime store refuses unknown sqlite schema versions', () => {
 
 test('runtime store refuses unknown runtime metadata versions', () => {
   const root = mkdtempSync(join(tmpdir(), 'wiki-manager-runtime-'));
-  const stateDir = join(root, '.wiki-manager');
+  const stateDir = join(root, '.wiki', 'runtime');
   mkdirSync(stateDir, { recursive: true });
   writeFileSync(join(stateDir, 'meta.json'), '{"schemaVersion":99}\n', 'utf8');
 
@@ -259,9 +259,9 @@ test('runtime store refuses unknown runtime metadata versions', () => {
   );
 });
 
-test('runtime store migrates legacy .wiki metadata to .wiki-manager', () => {
+test('runtime store migrates legacy .wiki metadata to .wiki/runtime', () => {
   const root = mkdtempSync(join(tmpdir(), 'wiki-manager-runtime-'));
-  const stateDir = join(root, '.wiki-manager');
+  const stateDir = join(root, '.wiki', 'runtime');
   mkdirSync(join(root, '.wiki'), { recursive: true });
   writeFileSync(join(root, '.wiki', 'meta.json'), '{"schemaVersion":1,"legacy":true}\n', 'utf8');
 

package/src/shell/LeftPane.tsx

@@ -312,7 +312,7 @@ function renderMarkdownLines(lines: Array<{ text: string; isCode: boolean }>, ro
 function isStatusOutput(message: { role: string; content: string }) {
   const content = String(message.content ?? '');
   return message.role === 'command'
-    && content.startsWith('Workspace')
+    && content.trimStart().startsWith('Workspace')
     && content.includes('Config')
     && content.includes('MCP');
 }

package/src/shell/repl.js

@@ -1279,6 +1279,35 @@ async function runDirectChatTurn(input, { session, onUpdate, onStep }) {
   return { exit: false };
 }
 
+// Headless equivalent of runDirectChatTurn for HTTP callers (the runtime /turn
+// in chat mode). Reuses the exact same read-only policy — chatReadTools +
+// runChatReadToolLoop + buildDirectChatSystemPrompt — so there is no second
+// implementation of chat access; it just returns the final text instead of
+// driving a live repl bubble. The caller must have seeded session.chatAccess
+// (and session.mcp) so chatReadTools can resolve the allow-listed read tools.
+export async function runHeadlessChatTurn(session, input, { history = [], onStep } = {}) {
+  const donnaMessage = { role: 'donna', content: '' };
+  const readTools = chatReadTools(session);
+  const canUseReadTools = readTools.length > 0 && typeof session.llm?.completeWithTools === 'function';
+  if (canUseReadTools) {
+    await runChatReadToolLoop({ input, session, history, donnaMessage, onStep, readTools });
+    return donnaMessage.content;
+  }
+  if (typeof session.llm?.stream === 'function') {
+    let content = '';
+    for await (const delta of session.llm.stream({
+      system: buildDirectChatSystemPrompt(session),
+      messages: [...history, { role: 'user', content: input }],
+      signal: session._abortSignal,
+    })) {
+      const clean = stripDsmlArtifacts(delta);
+      if (clean) content += clean;
+    }
+    return stripDsmlArtifacts(content).trimEnd() || formatLlmUnavailableMessage('flux vide');
+  }
+  return directChatUnavailableText(session);
+}
+
 function directChatUnavailableText(session) {
   if (!session.workspacePath) {
     return 'Direct chat unavailable: no workspace loaded. Use /use <workspace>.';

package/src/shell/repl.test.js

@@ -7,6 +7,7 @@ import {
   applyRuntimeStateToShellSession,
   chatReadTools,
   createSession,
+  runHeadlessChatTurn,
   conversationMessages,
   recordRuntimeUnavailableAgentInput,
   runLine,
@@ -443,3 +444,36 @@ test('/chat falls back to the plain stream when no read tools are declared', asy
   assert.match(last.content, /PLAIN_STREAM/);
   assert.doesNotMatch(last.content, /SHOULD_NOT_APPEAR/);
 });
+
+test('runHeadlessChatTurn (HTTP /chat) uses the read-tool path and returns text', async () => {
+  const session = createSession();
+  session.chatMode = true;
+  session.chatAccess = { maxToolIterations: 4, servers: { cme: { allow: ['cme_status'] } } };
+  session.mcp = { cme: { status: 'connected', tools: [{ name: 'cme_status', inputSchema: { type: 'object', properties: {} } }] } };
+  let usedComplete = false;
+  session.llm = {
+    async *stream() { yield 'STREAM_FALLBACK'; },
+    async completeWithTools() {
+      usedComplete = true;
+      return { tool_calls: [], content: 'CME est configuré.', message: { role: 'assistant', content: 'CME est configuré.' } };
+    },
+  };
+  const reply = await runHeadlessChatTurn(session, 'le cme est-il configuré', { history: [] });
+  assert.ok(usedComplete, 'completeWithTools path was taken');
+  assert.match(reply, /CME est configuré/);
+  assert.doesNotMatch(reply, /STREAM_FALLBACK/);
+});
+
+test('runHeadlessChatTurn falls back to the plain stream without read tools', async () => {
+  const session = createSession();
+  session.chatMode = true;
+  session.chatAccess = null;
+  session.mcp = {};
+  session.llm = {
+    async *stream() { yield 'PLAIN_STREAM'; },
+    async completeWithTools() { return { tool_calls: [], content: 'SHOULD_NOT_APPEAR' }; },
+  };
+  const reply = await runHeadlessChatTurn(session, 'bonjour', { history: [] });
+  assert.match(reply, /PLAIN_STREAM/);
+  assert.doesNotMatch(reply, /SHOULD_NOT_APPEAR/);
+});

package/wiki-workspace

@@ -17,7 +17,12 @@ WORKSPACES_DIR="${WIKI_WORKSPACES_DIR:-$DEFAULT_MANAGER_DIR/workspaces}"
 MANAGER_ENV_FILE="${WIKI_MANAGER_ENV_FILE:-$DEFAULT_MANAGER_DIR/.env}"
 MANAGER_ENDPOINTS_FILE="${WIKI_MANAGER_ENDPOINTS_FILE:-$DEFAULT_MANAGER_DIR/mcp.endpoints.json}"
 MANAGER_STATE_DIR="$(cd "$(dirname "$MANAGER_ENV_FILE")" && pwd)"
-MANAGER_RUNTIME_DIR="$MANAGER_STATE_DIR/.wiki-manager"
+MANAGER_RUNTIME_DIR="$MANAGER_STATE_DIR/.wiki/runtime"
+LEGACY_MANAGER_RUNTIME_DIR="$MANAGER_STATE_DIR/.wiki-manager"
+if [[ ! -e "$MANAGER_RUNTIME_DIR" && -d "$LEGACY_MANAGER_RUNTIME_DIR" ]]; then
+  mkdir -p "$MANAGER_STATE_DIR/.wiki"
+  mv "$LEGACY_MANAGER_RUNTIME_DIR" "$MANAGER_RUNTIME_DIR"
+fi
 CACERT_PATH="${WIKI_MANAGER_CACERT_PATH:-}"
 
 normalize_path() {