dsh-gitbash-shell 0.38.5 → 0.39.0

package/src/inline-tools.js

@@ -1,6 +1,6 @@
 /**
  * dsh-gitbash-shell/inline-tools — runtime shell-tool patching for the OFFICIAL
- * presets on Windows (v0.35.0).
+ * presets on Windows (v0.35.0; route B since v0.39.0).
  *
  * WHY THIS EXISTS. The official preset rows compose their tool set at boot
  * (`tool-bash` disabled on win32, `tool-pwsh` enabled), and the patch layer
@@ -8,9 +8,8 @@
  * carry `group: true` (vendor/include/src/index.ts buildMap), while a preset
  * declaration is a plain row whose `config.plugins` is opaque data. Rewriting
  * the declaration's whole `config` would freeze the official row list at our
- * restatement instant — every future official row addition would be invisible
- * until we hand-synced. So the fix runs at the tool layer instead, where the
- * official contract explicitly invites per-agent variants:
+ * restatement instant. So the fix runs at the tool layer, where the official
+ * contract explicitly invites per-agent variants:
  *
  *   - the tools registry REJECTS same-scope duplicate names but points at this
  *     exact path for variants ("for a per-agent variant, register through that
@@ -18,36 +17,32 @@
  *   - `agent/created` fires once per agent with the composed tool face
  *     reachable through `ctx.get('tools').schemas(agent)`.
  *
- * One reconcile per agent reads the face and acts on three facts:
+ * ROUTE B (v0.39.0, the DSH-Store-compliant architecture). The plugin no
+ * longer replaces the host `ctx.shell` executor, so the OFFICIAL tool-bash
+ * module — whose every execution goes through `ctx.shell`, i.e. PowerShell
+ * here — is OFF the table. Instead one reconcile per agent reads the face and
+ * acts on three facts:
  *
- *   1. `pwsh` present + `bash` absent  -> activate the OFFICIAL
- *      @deepseek-ai/dsh-tool-bash module inside an agent-owned scope (full
- *      official behaviour: jobs, approval escalation, rendering — zero
- *      behaviour is re-implemented here) and deny the fake `pwsh` (its
- *      executor is ctx.shell, which on this host IS Git Bash — a pwsh tool
- *      that runs bash is a lie to the model);
- *   2. `ops_*` tools present (dsh-ops installed) -> our FastCtx bridge and
- *      pwsh7 stands down; the official bash activation still runs, the
- *      two command surfaces are complements, not duplicates;
- *   3. `bash` already present (variant presets on, or a future official
- *      change) -> touch nothing, so the variantPresets switch and this path
- *      are mutually exclusive BY OBSERVATION, with no explicit lock.
+ *   1. `pwsh` present + `bash` absent  -> register the plugin's OWN `bash`
+ *      tool (src/bash-tool.js) in an agent-owned scope. It executes through a
+ *      PRIVATE GitBashSandboxExecutor instance (constructed once per process
+ *      in a private cordis scope, never registered as a service), and mirrors
+ *      the official tool's schema, background jobs, escalation and rendering
+ *      call-for-call.
+ *   2. `ops_*` tools present (dsh-ops installed) -> our FastCtx bridge stands
+ *      down, the pwsh hiding stands down (dsh-ops owns the host-pwsh story),
+ *      and a bash tool they already publish means ours is not registered.
+ *   3. `pwshTool` switch OFF (the default) -> the REAL host `pwsh` tool is
+ *      hidden per agent (`tools.restrict`) and denied at the call layer. With
+ *      route B the host pwsh is REAL PowerShell again — hiding it is a pure
+ *      product choice (bash-first host), not a safety denial, and flipping
+ *      the switch surfaces it unchanged.
  *
  * Everything here is Windows-only by caller gate; on other hosts the module
  * loads but mounts nothing.
  */
 
-/**
- * The tool-result contract the HOST registry requires from every registered
- * tool (issue #18, root cause 2): `register()` throws
- * `must declare output { schema, render, presentationMeta? }` unless the
- * definition carries an `output` whose `schema` passes
- * `assertSupportedJsonSchema` and whose `render` is a function. Both of this
- * plugin's tool families return `{ content: [{ type: 'text', text }] }` from
- * `execute`, so the schema and the text renderer below are shared verbatim
- * with the FastCtx bridge — the same frozen shape dsh-ops ships for its own
- * MCP-backed tools (proven on real Windows hosts).
- */
+/** FastCtx-bridge shared output contract (unchanged since v0.35.0). */
 export const TOOL_OUTPUT_SCHEMA = Object.freeze({
   type: 'object',
   properties: {
@@ -63,8 +58,6 @@ export const TOOL_OUTPUT_SCHEMA = Object.freeze({
  * blocks and non-string text fields are skipped; empty content renders as one
  * empty text block rather than an empty array, so the card never loses its
  * placeholder.
- * @param {unknown} value - the execute return value.
- * @returns {Array<{ type: string, text: string }>} content blocks for the host renderer.
  */
 export function renderTextContent(value) {
   const blocks = Array.isArray(value?.content) ? value.content : []
@@ -75,18 +68,25 @@ export function renderTextContent(value) {
   return [{ type: 'text', text }]
 }
 
-/** Answer for one observed tool face. Pure. */
-export function inlineNeeds(names) {
+/**
+ * Answer for one observed tool face. Pure.
+ * @param {string[]|undefined} names - the agent's composed tool names.
+ * @param {{ pwshVisible?: boolean }} [options] - whether the row switch shows
+ *   the (real) host pwsh tool; defaults to hidden.
+ */
+export function inlineNeeds(names, { pwshVisible } = {}) {
   const set = new Set(Array.isArray(names) ? names : [])
   const has = (name) => set.has(name)
   return {
     // A pwsh tool without a bash tool is the official win32 composition on
-    // this host: bash must be patched in, and the pwsh tool would execute
-    // Git Bash through ctx.shell (a lying name) — deny it.
+    // this host: our own bash tool must be patched in.
     needsBash: has('pwsh') && !has('bash'),
-    denyPwsh: has('pwsh') && !has('bash'),
+    // The real pwsh tool is hidden while the row switch is off (the default):
+    // the product answer is bash-first. dsh-ops co-install hands the whole
+    // host-shell story over to dsh-ops (checked by the caller).
+    hidePwsh: has('pwsh') && pwshVisible !== true,
     // dsh-ops publishes file/search tools under the ops_ prefix. When they are
-    // here, our FastCtx bridge and pwsh replacement would be duplicates.
+    // here, our FastCtx bridge would be a duplicate.
     opsPresent: has('ops_inspect_local_file') || has('ops_grep') || has('ops_bash'),
   }
 }
@@ -94,14 +94,14 @@ export function inlineNeeds(names) {
 /**
  * The routing addendum appended to the `bash` tool's description through the
  * system-prompt assemble waterfall. TWO variants (v0.37.0, user decision):
- * the pwsh tool is OFF by default, and a routing sentence that names a tool
- * the agent cannot see only teaches the model to call a denied name. The ON
- * variant names `pwsh` (the real PowerShell 7 tool that shadows the fake
- * global one); the OFF variant states bash-only routing. Both hold regardless
- * of the path dialect a delegate sees.
+ * the pwsh tool is hidden by default, and a routing sentence that names a tool
+ * the agent cannot see only teaches the model to call a hidden name. The ON
+ * variant names `pwsh` (the REAL PowerShell tool, real again since route B —
+ * the official executor stays in place); the OFF variant states bash-only
+ * routing. Both hold regardless of the path dialect a delegate sees.
  */
 export const GITBASH_ROUTE_ADDENDUM = [
-  'Prefer this bash tool for ALL commands; use the pwsh7 tool ONLY when a Windows-native capability (cmdlets, registry, services, WMI) is truly required.',
+  'Prefer this bash tool for ALL commands; use the pwsh tool ONLY when a Windows-native capability (cmdlets, registry, services, WMI) is truly required.',
   'Fix failed bash commands in bash — never switch shells mid-task, never embed `pwsh -Command` inside a bash command (variable expansion and output encoding break there).',
 ].join('\n')
 
@@ -121,217 +121,45 @@ export const GITBASH_MSYS_ADDENDUM = 'This bash is Git for Windows (MSYS): paths
 /** @deprecated composed form kept for tests: route + MSYS sentences. */
 export const GITBASH_BASH_ADDENDUM = GITBASH_ROUTE_ADDENDUM + '\n' + GITBASH_MSYS_ADDENDUM
 
-/** The pwsh-replacement tool's model-facing description. */
-export function gitbashPwshDescription() {
-  return 'Run a command in Windows PowerShell 7 (pwsh). Use ONLY when a Windows-native capability '
-    + '(cmdlets, registry, services, WMI, scheduled tasks) is required — for everything else prefer the bash tool. '
-    + 'Each call uses a fresh shell; set workdir explicitly. Paths inside command must be Windows form (C:/... or C:\\\\...). '
-    + 'For scripts longer than about 1 KB, or containing quotes/backslashes/non-ASCII text, write the script to a file first '
-    + 'and pass script_path instead. Output is UTF-8; long output is truncated with the full text saved to a reported file. '
-    + 'Fix failed pwsh commands here; do not switch shells or mix PowerShell syntax into bash commands.'
-}
-
-/**
- * Resolve the host's own PowerShell 7 — never a bundled payload, never
- * Windows PowerShell 5.1. Mirrors the probe order the official pwsh resolver
- * documents (well-known install locations, then PATH), WITHOUT its 5.1
- * fallback: if no pwsh 7 exists the caller simply does not publish the tool.
- * Pure I/O via the injected `exists` so the candidate matrix is testable.
- * @returns {string} an absolute pwsh.exe path, or '' when absent.
- */
-export function resolveHostPwsh7({ env = process.env, exists, platform = process.platform } = {}) {
-  if (platform !== 'win32') return ''
-  const programFiles = env.ProgramFiles ?? env.PROGRAMFILES ?? 'C:\\Program Files'
-  const programFilesX86 = env['ProgramFiles(x86)'] ?? 'C:\\Program Files (x86)'
-  const localAppData = env.LOCALAPPDATA ?? env.LocalAppData ?? ''
-  const userProfile = env.USERPROFILE ?? ''
-  const candidates = [
-    `${programFiles}\\PowerShell\\7\\pwsh.exe`,
-    `${programFiles}\\PowerShell\\7-preview\\pwsh.exe`,
-    `${programFilesX86}\\PowerShell\\7\\pwsh.exe`,
-    localAppData === '' ? '' : `${localAppData}\\Microsoft\\WindowsApps\\pwsh.exe`,
-    userProfile === '' ? '' : `${userProfile}\\scoop\\shims\\pwsh.exe`,
-    'C:\\ProgramData\\chocolatey\\bin\\pwsh.exe',
-  ].filter((entry) => entry !== '')
-  for (const candidate of candidates) {
-    if (exists(candidate)) return candidate
-  }
-  // PATH last: a PATH hit may be a store alias or shim, so it ranks below the
-  // well-known installs but still beats "no tool at all".
-  const pathValue = env.PATH ?? env.Path ?? ''
-  for (const dir of pathValue.split(';')) {
-    const clean = dir.trim().replace(/^"(.*)"$/, '$1')
-    if (clean === '') continue
-    const candidate = `${clean.replace(/[\\/]+$/, '')}\\pwsh.exe`
-    if (exists(candidate)) return candidate
-  }
-  return ''
-}
-
-/** Arguments `pwsh7` accepts — strict: anything else is rejected. */
-export const PWSH_ARGUMENTS = Object.freeze(['command', 'script_path', 'workdir', 'timeoutMs'])
-
-const PWSH_DEFAULT_TIMEOUT_MS = 120_000
-const PWSH_MAX_TIMEOUT_MS = 600_000
-
-/**
- * Build the argv for one pwsh invocation. `command` and `script_path` are
- * exclusive; a script path keeps the script's bytes OUT of the tool-call JSON
- * entirely (the whole point of the dsh-ops issue #7 lesson: long heredoc-heavy
- * commands inside one JSON string raise the malformed-tool-call rate and burn
- * a whole model turn). Pure.
- */
-export function pwshArgv(pwsh, args, { msysToWin } = {}) {
-  const hasCommand = typeof args.command === 'string' && args.command.trim() !== ''
-  const hasScript = typeof args.script_path === 'string' && args.script_path.trim() !== ''
-  if (hasCommand === hasScript) return { error: 'provide exactly one of command or script_path' }
-  if (hasScript) {
-    const raw = /** @type {string} */ (args.script_path)
-    // The model speaks MSYS (our prompt dialect); pwsh needs a Windows path.
-    const script = msysToWin ? msysToWin(raw) : raw
-    return { argv: [pwsh, '-NoLogo', '-NoProfile', '-NonInteractive', '-File', script] }
-  }
-  return { argv: [pwsh, '-NoLogo', '-NoProfile', '-NonInteractive', '-Command', /** @type {string} */ (args.command)] }
-}
-
 /**
- * Validate one pwsh7 call's arguments. Strict like ops_bash: an
- * undeclared key is a hard error (the host bash/pwsh tools accept
- * `run_in_background`; this tool does not, and a silent ignore would leave a
- * server occupying the call until timeout). Pure.
- * @returns {string|undefined} an error message, or undefined when valid.
+ * The deny reason for a hidden pwsh call (route B wording): the tool is REAL
+ * PowerShell on this host, just hidden by the row switch — the sentence says
+ * so and points at the switch instead of calling the tool a lie.
  */
-export function validatePwshArgs(args) {
-  if (args === undefined || args === null || typeof args !== 'object' || Array.isArray(args)) {
-    return 'invalid arguments: expected an object'
-  }
-  const unknown = Object.keys(args).filter((key) => !PWSH_ARGUMENTS.includes(key))
-  if (unknown.length > 0) {
-    return `invalid arguments: unknown key(s) ${unknown.map((key) => JSON.stringify(key)).join(', ')}; this tool accepts ${PWSH_ARGUMENTS.join(', ')}`
-  }
-  const hasCommand = typeof args.command === 'string' && args.command.trim() !== ''
-  const hasScript = typeof args.script_path === 'string' && args.script_path.trim() !== ''
-  if (hasCommand === hasScript) return 'invalid pwsh input: provide exactly one of command or script_path'
-  if (args.timeoutMs !== undefined && (!Number.isFinite(args.timeoutMs) || args.timeoutMs <= 0)) {
-    return 'invalid timeoutMs: expected a positive number'
-  }
-  if (args.timeoutMs !== undefined && /** @type {number} */ (args.timeoutMs) > PWSH_MAX_TIMEOUT_MS) {
-    return `invalid timeoutMs: must be no greater than ${PWSH_MAX_TIMEOUT_MS}`
-  }
-  return undefined
-}
-
-/** The official bash-tool package the inline patch activates. */
-const TOOL_BASH_PACKAGE = '@deepseek-ai/dsh-tool-bash'
-
-/**
- * Resolve the OFFICIAL @deepseek-ai/dsh-tool-bash module. TWO channels, in
- * order (v0.38.1, the desktop-host field report):
- *
- *   ① THE HOST'S OWN LOADER — `ctx.loader.internal.import(name, baseUrl, {})`,
- *      the exact call the row mount itself uses
- *      (vendor/loader/src/config/tree.ts:122). It runs the host's resolve
- *      hooks, so it reaches packages inside the desktop app.asar that a bare
- *      Node resolver cannot see from the plugin's profile layout.
- *   ② A plain createRequire from the host base — the v0.35.0 channel, kept
- *      for layouts where the internal loader is absent.
- *
- * @returns {Promise<{ load: async () => object }|undefined>} a lazy loader for
- *   the module, or undefined when neither channel resolves it.
- */
-export async function resolveToolBashEntry(ctx) {
-  try {
-    const internal = ctx?.loader?.internal
-    if (internal && typeof internal.import === 'function') {
-      const mod = await internal.import(TOOL_BASH_PACKAGE, ctx?.baseUrl, {})
-      if (mod && typeof mod.apply === 'function') return { load: async () => mod }
-    }
-  } catch { /* fall through to the plain resolution */ }
-  try {
-    const { createRequire } = await import('node:module')
-    const { pathToFileURL } = await import('node:url')
-    const req = createRequire(ctx?.baseUrl ?? import.meta.url)
-    const href = pathToFileURL(req.resolve(TOOL_BASH_PACKAGE)).href
-    return { load: async () => import(href) }
-  } catch {
-    return undefined
-  }
+export function hiddenPwshDenyReason() {
+  return 'the pwsh tool is hidden on this host by the dsh-gitbash-shell settings (pwsh tool switch, off by default). Use the bash tool for commands. If you need Windows-native work (registry, services, WMI), ask the user to enable the pwsh tool on the dsh-gitbash-shell settings card.'
 }
 
-/**
- * Activate the OFFICIAL tool-bash inside one agent-owned scope: a child scope
- * of the agent's context keeps every registration (tool, prompt section,
- * effect) revertible at agent disposal, and the scoped-layer tool shadows the
- * (absent) global one exactly the way the registry's own error message
- * prescribes for per-agent variants. The official module is loaded ONCE per
- * process; each agent gets a fresh scope and a fresh apply().
- */
-export async function activateOfficialBash(agent, resolved, log) {
-  let mod
-  try {
-    mod = await resolved.load()
-  } catch (error) {
-    log(`official tool-bash module import failed: ${error?.message ?? error}`)
-    return undefined
-  }
-  if (!mod || typeof mod.apply !== 'function') {
-    log('official tool-bash module has no apply(); activation skipped')
-    return undefined
-  }
-  // Mount through cordis's OWN plugin path. The official module object
-  // DECLARES its service needs (`inject: ['tools','shell','systemPrompt',
-  // 'shellEnv','workingDirectory']`); a hand-called mod.apply(ctx) has no
-  // such declaration, so cordis refuses every service read with
-  // `cannot get property "shell" without inject` (v0.38.2 field report).
-  // ctx.plugin(mod) registers a REAL fiber with the declaration honored,
-  // runs apply itself, and the wrapped fiber is a thenable — awaiting it
-  // waits for the activation (and its async fiber start) to finish.
-  let fiber
-  try {
-    fiber = agent.ctx.plugin(mod)
-  } catch (error) {
-    log(`official tool-bash mount failed: ${error?.message ?? error}`)
-    return undefined
-  }
-  try {
-    await fiber
-  } catch (error) {
-    log(`official tool-bash fiber failed: ${error?.message ?? error}`)
-    return undefined
-  }
-  return {
-    fiber,
-    dispose: async () => { try { await fiber?.dispose?.() } catch { /* already gone */ } },
-  }
-}
+/** The host packages the own bash tool consumes, resolved once per process. */
+const HOST_MODULES = [
+  ['toolsModule', '@deepseek-ai/dsh-tools'],
+  ['sandboxModule', '@deepseek-ai/dsh-sandbox'],
+  ['llmModule', '@deepseek-ai/dsh-llm'],
+]
 
 /**
  * Mount the per-agent reconcile. `deps`:
  *   - tools(ctx)       -> the host tool registry (or undefined)
  *   - agents(ctx)      -> the host agents service (or undefined)
  *   - createScope(fn)  -> cordis createScope
- *   - allowed(session) -> danger-full-access verdict for the pwsh channel
- *   - echo(text)       -> result dialect echo (Windows -> MSYS), or undefined
- *   - spawn(spec)      -> the host subprocess seam, or undefined
- *   - pwshPath         -> resolved host pwsh 7, or '' (tool not published)
- *   - pwshEnabled()    -> the row switch for publishing the pwsh tool
+ *   - pwshEnabled()    -> the row switch; TRUE shows the real host pwsh tool,
+ *                         false/undefined (default) hides it
  *   - log(message)     -> the plugin's tagged logger
  *
  * Registration decisions re-run on EVERY agent creation AND once at mount
- * time for every ALREADY-LIVE agent (the dsh-ops-proven shape,
- * `installHostShellDenial`): this row can activate long after agents exist —
- * a plugin hot-reload, or a host whose sessions resumed before this bundle
- * row finished mounting — and an agent/created-only listener leaves every
- * such agent with the fake pwsh and no bash, which is exactly the Windows
- * field report behind this block. A second, INDEPENDENT layer guards the
+ * time for every ALREADY-LIVE agent (the dsh-ops-proven shape): this row can
+ * activate long after agents exist. A second, INDEPENDENT layer guards the
  * call itself through `tools/pre-execute`: visibility restriction can lose a
- * race, and a lying pwsh that reaches its executor still runs Git Bash.
+ * race.
  */
 export function mountInlineTools(ctx, deps) {
-  const { tools, agents, createScope, allowed, spawn, pwshPath, echo, log } = deps
-  /** agent -> { timer, scope, denyPwsh, pwshScope }; named `tracked` because `agents` above is the host service */
+  const { tools, agents, createScope, pwshEnabled, log } = deps
+  /** agent -> { timer, scope, denyPwsh }; named `tracked` because `agents` above is the host service */
   const tracked = new Map()
-  let toolBashResolvePromise
+  /** Resolved once per process: { executor, executorScope, module }. */
+  let bashOnce
+  /** Set as soon as any face shows ops_* tools (the call-layer guard reads it). */
+  let opsSeen = false
 
   const faceNames = (agent) => {
     try {
@@ -343,30 +171,72 @@ export function mountInlineTools(ctx, deps) {
   }
 
   /**
-   * Provide the additive `pwsh7` tool inside one agent-owned scope (v0.37.0).
-   * The name never collides with the host's fake global `pwsh`, so this is a
-   * pure ADDITION — it shadows nothing and requires no deny juggling. The
-   * scope keeps the registration revertible at agent disposal.
+   * Construct the private executor and the own bash-tool module ONCE per
+   * process. The executor mounts through cordis's own class-plugin path in a
+   * PRIVATE scope (`ctx.plugin(class)` honours the static inject/Config
+   * declarations — the §4k schemastery semantics come for free), so it never
+   * touches the root `ctx.shell` service slot the official pwsh executor
+   * owns. The executor scope lives for the plugin's lifetime and is disposed
+   * by the effect registered at the bottom of this function.
    */
-  const providePwsh = async (agent, state) => {
-    if (state.pwshScope) return
-    if (typeof spawn !== 'function' || !pwshPath) return
-    if (!createScope) return
+  const bashOncePromise = async () => {
+    if (bashOnce) return bashOnce
+    bashOnce = (async () => {
+      const { GitBashSandboxExecutor } = await import('./shell.js')
+      if (!GitBashSandboxExecutor) throw new Error('executor class unavailable')
+      if (!GitBashSandboxExecutor.Config) {
+        throw new Error('executor Config unavailable (schemastery unresolved; see the schemastery log lines above)')
+      }
+      if (typeof createScope !== 'function') throw new Error('no createScope seam')
+      const executorScope = createScope(ctx)
+      if (executorScope.fiber) { try { await executorScope.fiber } catch { /* disposed */ } }
+      const mountTimeout = new Promise((_, reject) => {
+        const timer = setTimeout(() => reject(new Error('executor scope mount timed out (a required service may be absent in this composition)')), 8000)
+        timer.unref?.()
+      })
+      const fiber = executorScope.ctx.plugin(GitBashSandboxExecutor)
+      const executor = await Promise.race([
+        (async () => { await fiber; return executorScope.ctx.shell })(),
+        mountTimeout,
+      ])
+      if (!executor || typeof executor.resolve !== 'function' || typeof executor.execute !== 'function') {
+        throw new Error('executor instance did not expose resolve()/execute()')
+      }
+      const hostModules = {}
+      for (const [key, name] of HOST_MODULES) {
+        try {
+          hostModules[key] = await resolveHostModule(ctx, name)
+        } catch { hostModules[key] = undefined }
+      }
+      const { createBashToolModule } = await import('./bash-tool.js')
+      const module = createBashToolModule({ executor, log, ...hostModules })
+      return { executor, executorScope, module }
+    })()
+    bashOnce.catch(() => { bashOnce = undefined })
+    return bashOnce
+  }
+
+  /** Register the own bash tool inside one agent-owned scope. */
+  const provideBash = async (agent, state) => {
+    if (state.scope) return
     try {
+      const { executor, module } = await bashOncePromise()
+      log(`own bash tool ready (private Git Bash executor at ${executor.bashPath || '(unresolved)'}; official pwsh executor untouched)`)
       const scope = createScope(agent.ctx)
-      // Same asynchronous-fiber shape as activateOfficialBash: wait for the
-      // apply callback to capture scope.ctx before touching its tools.
+      // Same asynchronous-fiber shape the official module needed: wait for
+      // the apply callback to capture scope.ctx before touching its tools.
       if (scope.fiber) { try { await scope.fiber } catch { /* disposed */ } }
       const registry = scope.ctx?.tools
-      if (!registry || typeof registry.register !== 'function') {
+      if (!registry || typeof scope.ctx.plugin !== 'function') {
         void scope.dispose().catch(() => {})
         return
       }
-      registry.register(gitbashPwshDefinition({ pwsh: pwshPath, spawn, allowed, msysToWin: deps.msysToWin, echo }))
-      state.pwshScope = scope
-      log(`pwsh7 tool provided for this agent (real PowerShell 7 at ${pwshPath})`)
+      const fiber = scope.ctx.plugin(module)
+      await fiber
+      state.scope = scope
+      log('own bash tool activated for this agent (Git Bash execution; official pwsh tool real but hidden by default)')
     } catch (error) {
-      log(`pwsh7 tool registration failed for this agent: ${error?.message ?? error}`)
+      log(`own bash tool activation failed for this agent: ${error?.message ?? error}`)
     }
   }
 
@@ -374,9 +244,8 @@ export function mountInlineTools(ctx, deps) {
     if (!agent?.ctx) return
     const names = faceNames(agent)
     // The composed face may not be wired yet at created-time. Retry bounded:
-    // a face with NEITHER tool carries no verdict, and guessing would deny a
-    // real pwsh or duplicate a real bash. Five tries at 250ms, then give up
-    // quietly — a slower composition simply gets no inline patch this boot.
+    // a face with NEITHER tool carries no verdict. Five tries at 250ms, then
+    // give up quietly — a slower composition simply gets no inline patch.
     if (attempt < 5 && !names.includes('bash') && !names.includes('pwsh')) {
       const timer = setTimeout(() => { try { reconcile(agent, { attempt: attempt + 1 }) } catch { /* never crash on a timer */ } }, 250)
       timer.unref?.()
@@ -385,56 +254,36 @@ export function mountInlineTools(ctx, deps) {
       else tracked.set(agent, { timer, scope: undefined, denyPwsh: undefined })
       return
     }
-    const needs = inlineNeeds(names)
+    const pwshVisible = deps.pwshEnabled?.() === true
+    const needs = inlineNeeds(names, { pwshVisible })
+    if (needs.opsPresent) opsSeen = true
     let state = tracked.get(agent)
     if (state?.timer) { clearTimeout(state.timer); state.timer = undefined }
     if (!state) {
-      state = { timer: undefined, scope: undefined, denyPwsh: undefined, pwshScope: undefined }
+      state = { timer: undefined, scope: undefined, denyPwsh: undefined }
       tracked.set(agent, state)
     }
-    // The pwsh face has three facts (v0.37.0, user decision — the pwsh7 tool
-    // is OFF by default, and its switch lives ONLY on this plugin's row):
-    //   dsh-ops present  -> hands off entirely (their host-pwsh story stands;
-    //                       our pwsh7 and our deny would both be duplicates).
-    //   the fake global `pwsh` (executor = ctx.shell = Git Bash) is DENIED
-    //                       per agent in EVERY case — the lying name is never
-    //                       callable, and `pwsh7` cannot be caught by it.
-    //   switch ON        -> the REAL PowerShell 7 tool is ADDED as `pwsh7` in
-    //                       an agent-owned scope (pure addition, no shadowing).
-    // The row switch is re-read per reconcile, so a flip lands at the next
-    // agent (or on the next event after a hot toggle).
-    if (needs.denyPwsh) {
-      if (needs.opsPresent) {
-        // hands off
-      } else {
-        if (!state.denyPwsh) {
-          // The deny MUST go through the AGENT's scoped context
-          // (`agent.ctx.tools`, the dsh-ops-proven shape): on the ROOT context
-          // the host throws "requires a scoped context" and the deny silently
-          // never exists (issue #18, root cause 3).
-          try {
-            const deny = agent.ctx?.tools?.restrict?.({ deny: ['pwsh'] })
-            if (typeof deny === 'function') state.denyPwsh = deny
-          } catch (error) {
-            log(`fake-pwsh deny failed: ${error?.message ?? error}`)
-          }
+    // The pwsh face (route B): the host tool is REAL PowerShell now, so the
+    // default is a product-level HIDE, not a safety denial.
+    //   dsh-ops present  -> hands off entirely (their host-pwsh story stands).
+    //   switch OFF       -> hide the tool for this agent (tools.restrict).
+    //   switch ON        -> leave the real tool visible, untouched.
+    if (needs.hidePwsh && !needs.opsPresent) {
+      if (!state.denyPwsh) {
+        // Through the AGENT's scoped context (`agent.ctx.tools`, the dsh-ops
+        // -proven shape): on the ROOT context the host throws "requires a
+        // scoped context" and the restriction silently never exists (issue
+        // #18, root cause 3).
+        try {
+          const deny = agent.ctx?.tools?.restrict?.({ deny: ['pwsh'] })
+          if (typeof deny === 'function') state.denyPwsh = deny
+        } catch (error) {
+          log(`pwsh hide failed: ${error?.message ?? error}`)
         }
-        if (deps.pwshEnabled && deps.pwshEnabled() === true) void providePwsh(agent, state)
       }
     }
     if (needs.needsBash && !state.scope) {
-      if (toolBashResolvePromise === undefined) toolBashResolvePromise = resolveToolBashEntry(ctx)
-      void toolBashResolvePromise.then(async (resolved) => {
-        if (!resolved) {
-          log('official @deepseek-ai/dsh-tool-bash is not resolvable (neither the host loader nor a plain resolution found it); bash stays absent for this agent')
-          return
-        }
-        const scope = await activateOfficialBash(agent, resolved, log)
-        if (scope && tracked.get(agent) === state) {
-          state.scope = scope
-          log('official bash tool activated for this agent (Git Bash host; fake pwsh denied)')
-        }
-      }).catch(() => { /* logged inside activateOfficialBash */ })
+      void provideBash(agent, state)
     }
   }
 
@@ -444,30 +293,26 @@ export function mountInlineTools(ctx, deps) {
     tracked.delete(agent)
     if (state.timer) clearTimeout(state.timer)
     if (state.denyPwsh) { try { state.denyPwsh() } catch { /* already lifted */ } }
-    if (state.pwshScope) { void state.pwshScope.dispose().catch(() => {}) }
     if (state.scope) { void state.scope.dispose().catch(() => {}) }
   }
 
   try {
     ctx.on('agent/created', ({ agent } = {}) => { try { reconcile(agent) } catch { /* never break agent creation */ } })
     ctx.on('agent/disposed', ({ agent } = {}) => { try { teardown(agent) } catch { /* never break disposal */ } })
-    // MOUNT-TIME SWEEP (v0.35.2, the Windows field report): agents that
-    // already exist when this row mounts never fire agent/created — a plugin
-    // hot-reload or a host that resumed sessions before this bundle row
-    // finished mounting would otherwise keep the fake pwsh and no bash on
-    // every live agent. Same shape the proven dsh-ops denial uses.
+    // MOUNT-TIME SWEEP (v0.35.2): agents that already exist when this row
+    // mounts never fire agent/created. Same shape the proven dsh-ops denial
+    // uses.
     for (const agent of agents?.(ctx)?.list?.() ?? []) {
       try { reconcile(agent) } catch { /* one broken agent never blocks the rest */ }
     }
-    // CALL-LAYER GUARD (v0.35.2, second independent layer; v0.37.0 wording):
-    // a `pwsh` call ALWAYS reaches the fake global tool (executor = ctx.shell
-    // = Git Bash on this host) — our real tool is `pwsh7`, a different name —
-    // so the fake name is denied at dispatch unconditionally, with the
-    // replacement named.
+    // CALL-LAYER GUARD (v0.35.2, second independent layer; route B wording):
+    // while the row switch hides the real pwsh tool, a `pwsh` call that slips
+    // past a visibility race is denied with the switch named. With the switch
+    // ON the REAL tool answers for itself.
     ctx.on('tools/pre-execute', (exec, next) => {
       try {
-        if (exec?.name === 'pwsh') {
-          return { kind: 'deny', reason: 'the host pwsh tool would run Git Bash on this host (its executor was replaced). Use the bash tool for commands. If you need Windows-native work (registry, services, WMI), the real PowerShell 7 tool is pwsh7 — enable it on the dsh-gitbash-shell settings card if it is not present.' }
+        if (exec?.name === 'pwsh' && !opsSeen && deps.pwshEnabled?.() !== true) {
+          return { kind: 'deny', reason: hiddenPwshDenyReason() }
         }
       } catch { /* never block the waterfall */ }
       return next()
@@ -480,91 +325,17 @@ export function mountInlineTools(ctx, deps) {
   ctx.effect(() => () => {
     for (const agent of [...tracked.keys()]) teardown(agent)
     tracked.clear()
+    if (bashOnce) {
+      void Promise.resolve(bashOnce).then(({ executorScope } = {}) => {
+        try { executorScope?.dispose?.() } catch { /* already gone */ }
+      }).catch(() => {})
+      bashOnce = undefined
+    }
   }, 'dsh-gitbash-shell: inline tool patching')
 }
 
-/**
- * The pwsh-replacement tool definition, bound to a resolved pwsh 7.
- *
- * It spawns through the HOST's `ctx.subprocess` service (credential scrub +
- * process governance + output spill are the service's own guarantees; the
- * bash executors consume the same seam). There is deliberately NO file
- * sandbox on this channel — the tool is published only to danger-full-access
- * sessions and says so in the error it raises elsewhere.
- * @param {object} options
- * @param {string} options.pwsh - resolved pwsh.exe path.
- * @param {(spec: object) => { done: Promise<object>, collected?: object }} options.spawn - the host subprocess seam.
- * @param {(session: object|undefined) => boolean} options.allowed - per-session permission verdict.
- * @param {(path: string) => string} [options.msysToWin] - argument dialect translation (MSYS -> Windows), the same mount table the file tools use.
- * @param {(text: string) => string} [options.echo] - result dialect echo (Windows -> MSYS), gated by the caller.
- */
-export function gitbashPwshDefinition({ pwsh, spawn, allowed, msysToWin, echo }) {
-  return {
-    // v0.37.0 (user decision): the tool is named `pwsh7` — NOT `pwsh` (that
-    // name belongs to the host's fake global tool, whose executor is
-    // ctx.shell = Git Bash on this host and stays denied) and NOT
-    // `gitbash_pwsh` (which read like a bash/pwsh hybrid). `pwsh7` is purely
-    // additive: it never shadows anything, and it exists only on hosts where
-    // a real PowerShell 7 is installed.
-    name: 'pwsh7',
-    description: gitbashPwshDescription(),
-    parameters: {
-      type: 'object',
-      properties: {
-        command: { type: 'string', description: 'Short PowerShell command; for long or multi-line scripts use script_path.' },
-        script_path: { type: 'string', description: 'Existing script path; use instead of command for long or multi-line scripts.' },
-        workdir: { type: 'string', description: 'Working directory; defaults to the session workspace when omitted.' },
-        timeoutMs: { type: 'number', description: 'Timeout in milliseconds; the command is killed on expiry. Defaults to 120000.' },
-      },
-      required: ['command'],
-      additionalProperties: false,
-    },
-    output: {
-      schema: TOOL_OUTPUT_SCHEMA,
-      render: (_args, value) => renderTextContent(value),
-    },
-    async execute(args, exec) {
-      const invalid = validatePwshArgs(args)
-      if (invalid) throw new Error(invalid)
-      if (!allowed(exec?.agent?.session)) {
-        throw new Error('pwsh7 requires danger-full-access for this session; this channel has no file sandbox.')
-      }
-      const built = pwshArgv(pwsh, args, { msysToWin })
-      if (built.error) throw new Error(built.error)
-      const timeoutMs = args.timeoutMs === undefined ? PWSH_DEFAULT_TIMEOUT_MS : args.timeoutMs
-      const controller = new AbortController()
-      const timer = setTimeout(() => controller.abort(new Error(`command timed out after ${timeoutMs}ms`)), timeoutMs)
-      timer.unref?.()
-      try {
-        const handle = spawn({
-          argv: built.argv,
-          cwd: typeof args.workdir === 'string' && args.workdir !== '' ? args.workdir : undefined,
-          stdio: {
-            stdin: 'ignore',
-            stdout: { maxBytes: 64_000, spill: { maxBytes: 64 * 1024 * 1024 } },
-            stderr: { maxBytes: 64_000, spill: { maxBytes: 64 * 1024 * 1024 } },
-          },
-          signal: controller.signal,
-        })
-        const outcome = await handle.done
-        const streamText = (reader) => {
-          if (reader === undefined || typeof reader.readFrom !== 'function') return ''
-          const read = reader.readFrom(0)
-          const text = read.text ?? ''
-          return echo ? echo(text) : text
-        }
-        const stdout = streamText(handle.collected?.stdout)
-        const stderr = streamText(handle.collected?.stderr)
-        let body = stdout
-        if (stderr !== '') body += (body !== '' && !body.endsWith('\n') ? '\n' : '') + '[stderr]\n' + stderr
-        if (body === '') body = '(no output)'
-        const markers = []
-        if (outcome?.signal !== undefined && outcome?.signal !== null) markers.push(`[signal: ${outcome.signal}]`)
-        if (outcome?.exitCode !== undefined && outcome?.exitCode !== null && outcome.exitCode !== 0) markers.push(`[exit code: ${outcome.exitCode}]`)
-        return { content: [{ type: 'text', text: [body, ...markers].join('\n') }] }
-      } finally {
-        clearTimeout(timer)
-      }
-    },
-  }
-}
+// The host-package resolution channel lives in bash-tool.js; import it for
+// local use AND re-export it so the host half (and the tests) see ONE
+// implementation.
+import { resolveHostModule } from './bash-tool.js'
+export { resolveHostModule }