dsh-gitbash-shell 0.17.0 → 0.17.1

package/dsh.plugin.json

@@ -1,6 +1,6 @@
 {
   "id": "dsh-external/dsh-gitbash-shell",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "main": "./src/index.js",
   "description": "Windows 全模式 Git Bash:用 Git for Windows bash 替换 PowerShell 执行器,并物化 standard/minimal/code/cordis 的 Git Bash 变体 preset。双 era 适配 dsh <= 0.1.1(内置 id 'code')与 >= 0.1.2(改名 'ptc'),启动探测自动选择,preset id 永不变。联动:dsh-ptc-cordis-preset(两者同装时 PTC 创造模式自动物化为 Git Bash 版)+ dsh-better-sidebar(终端 UI 标签与模型 terminal_* 工具自动切到 Git Bash)。",
   "engines": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-gitbash-shell",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "description": "Windows 上让 agent 的每条 shell 命令改经 Git for Windows bash 执行;物化四个内置预设的 Git Bash 变体,统一 MSYS 盘根路径方言,侧栏终端接管可开关。· Run every agent shell command through Git for Windows bash on Windows; materialize Git Bash variants of the built-in presets, unify the MSYS drive-root path dialect, and adopt the sidebar terminal behind a switch.",
   "type": "module",
   "repository": {

package/src/index.js

@@ -724,6 +724,38 @@ function posixSeparators(value) {
     : value
 }
 
+/**
+ * Rewrite Windows absolute paths inside an ERROR result's text blocks to
+ * the MSYS dialect (pure, v0.17.1). Error results cannot carry a
+ * replacement `value` (the registry rejects that), but replacing `content`
+ * is the official post-execute channel, so the message the model reads
+ * stays in one dialect — it must not relearn Windows forms from failure
+ * text. Non-drive-letter diagnostics (device paths, URLs) never match
+ * windowsToMsys and are left verbatim.
+ */
+/**
+ * The only tools whose ERROR content is dialect-translated (v0.17.1).
+ * These messages are harness-generated path diagnostics (cannot read
+ * "C:\..."), so rewriting them is safe. Every other tool — bash above all,
+ * whose failed runs carry raw command stdout/stderr — keeps its error
+ * content VERBATIM: output is data, never rewritten (the same red line as
+ * file content in successful results).
+ */
+const ERROR_CONTENT_TOOLS = new Set(['read', 'read_image', 'write', 'edit', 'glob', 'grep', 'present'])
+
+export function rewriteErrorContent(content) {
+  if (!Array.isArray(content)) return content
+  let changed = false
+  const next = content.map((block) => {
+    if (!block || typeof block !== 'object' || typeof block.text !== 'string') return block
+    const out = windowsToMsys(block.text)
+    if (out === block.text) return block
+    changed = true
+    return { ...block, text: out }
+  })
+  return changed ? next : content
+}
+
 export function rewriteResultPaths(name, value, env) {
   if (!value || typeof value !== 'object') return value
   // v0.17.0: paths under the user TEMP directory echo back in the bash
@@ -1369,17 +1401,26 @@ export async function apply(ctx, config = {}) {
     try {
       ctx.on('tools/post-execute', (exec, result, next) => {
         let patch
+        let contentPatch
         try {
-          if (readPosixPaths(ctx) && result && result.isError === false && result.value && typeof result.value === 'object') {
-            const value = rewriteResultPaths(exec && exec.name, result.value, buildTranslateEnv())
-            if (value !== result.value) patch = value
+          if (readPosixPaths(ctx) && result && typeof result === 'object') {
+            if (result.isError === false && result.value && typeof result.value === 'object') {
+              const value = rewriteResultPaths(exec && exec.name, result.value, buildTranslateEnv())
+              if (value !== result.value) patch = value
+            } else if (result.isError === true && exec && ERROR_CONTENT_TOOLS.has(exec.name) && Array.isArray(result.content)) {
+              const content = rewriteErrorContent(result.content)
+              if (content !== result.content) contentPatch = content
+            }
           }
         } catch { /* never block a result */ }
         const chain = next()
-        if (patch === undefined) return chain
+        if (patch === undefined && contentPatch === undefined) return chain
         return chain.then((decision) => {
           try {
-            if (decision && decision.kind === 'accept') return { ...decision, value: patch }
+            if (decision && decision.kind === 'accept') {
+              if (patch !== undefined) return { ...decision, value: patch }
+              return { ...decision, content: contentPatch }
+            }
           } catch { /* keep the downstream decision */ }
           return decision
         })
@@ -1510,4 +1551,4 @@ export async function apply(ctx, config = {}) {
 }
 
 // Test surface: pure helpers, no Cordis context required.
-export const _internal = { PRESET_IDS, translateMsysPath, translatePathArguments, translateGlobArguments, buildTranslateEnv, readPosixPaths, readAdoptSidebar, windowsToMsys, rewriteResultPaths, MARKER_FILE, classify, materialize, cleanupOnDispose, firstUserRoot, hashTree, skillsHashes, syncDecision, installRegisterShim, baseForRoster, detectBase, pickComposition, personaEraForText, detectPersonaEra, injectPresentRow, hostHasToolPresent, detectPresentSupport, injectPluginManagerRow, hostHasPluginManagerTools, rowFormOf, rowFormsOf, alignEngineRow, alignRalphRow, ROW_SOURCE }
+export const _internal = { PRESET_IDS, translateMsysPath, translatePathArguments, translateGlobArguments, buildTranslateEnv, rewriteErrorContent, ERROR_CONTENT_TOOLS, readPosixPaths, readAdoptSidebar, windowsToMsys, rewriteResultPaths, MARKER_FILE, classify, materialize, cleanupOnDispose, firstUserRoot, hashTree, skillsHashes, syncDecision, installRegisterShim, baseForRoster, detectBase, pickComposition, personaEraForText, detectPersonaEra, injectPresentRow, hostHasToolPresent, detectPresentSupport, injectPluginManagerRow, hostHasPluginManagerTools, rowFormOf, rowFormsOf, alignEngineRow, alignRalphRow, ROW_SOURCE }