[BUG]

[c-sunc6]

Problem (one or two sentences)

子任务结束崩溃

Context (who is affected and when)

子任务结束崩溃

Reproduction steps

子任务结束崩溃

Expected result

返回主任务

Actual result

子任务结束崩溃

Variations tried (optional)

No response

App Version

v3.51.0

API Provider (optional)

OpenAI Compatible

Model Used (optional)

No response

Roo Code Task Links (optional)

No response

Relevant logs or errors (optional)

出现了错误 (v3.51.0)
请通过在以下位置报告此错误来帮助我们改进 我们的 GitHub Issues 页面

复制并粘贴以下错误信息，将其作为提交内容的一部分：

错误堆栈：
TypeError: n.replace is not a function
    at countMarkdownHeadings (../../../../webview-ui/src/utils/markdown.ts:10:32)
    at hasComplexMarkdown (../../../../webview-ui/src/utils/markdown.ts:22:8)
    at hasComplexMarkdown (../../../../webview-ui/src/components/chat/OpenMarkdownPreviewButton.tsx:14:6)
    at Nh (../../../../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js:160:136)
    at gj (../../../../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js:196:257)
    at bj (../../../../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js:194:203)
    at Fk (../../../../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js:296:99)
    at tj (../../../../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js:279:388)
    at tj (../../../../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js:279:319)
    at tj (../../../../node_modules/.pnpm/react-dom@18.3.1_react@18.3.1/node_modules/react-dom/cjs/react-dom.production.min.js:279:179)
组件堆栈：

    at https://vscode-remote+attached-002dcontainer-002b7b22636f6e7461696e65724e616d65223a222f73696d5f7632227d.vscode-resource.vscode-cdn.net/root/.vscode-server/extensions/rooveterinaryinc.roo-cline-3.51.0/webview-ui/build/assets/index.js:2750:9414
    at div
    at div
at Cao (../../../../webview-ui/src/components/chat/ChatRow.tsx:169:1)
    at div
    at https://vscode-remote+attached-002dcontainer-002b7b22636f6e7461696e65724e616d65223a222f73696d5f7632227d.vscode-resource.vscode-cdn.net/root/.vscode-server/extensions/rooveterinaryinc.roo-cline-3.51.0/webview-ui/build/assets/index.js:2750:10334
    at div
    at div
    at https://vscode-remote+attached-002dcontainer-002b7b22636f6e7461696e65724e616d65223a222f73696d5f7632227d.vscode-resource.vscode-cdn.net/root/.vscode-server/extensions/rooveterinaryinc.roo-cline-3.51.0/webview-ui/build/assets/index.js:261:39510
    at div
at dYt (../../../../node_modules/.pnpm/react-virtuoso@4.12.7_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/react-virtuoso/dist/index.mjs:2470:24)
    at div
    at https://vscode-remote+attached-002dcontainer-002b7b22636f6e7461696e65724e616d65223a222f73696d5f7632227d.vscode-resource.vscode-cdn.net/root/.vscode-server/extensions/rooveterinaryinc.roo-cline-3.51.0/webview-ui/build/assets/index.js:261:42379
    at https://vscode-remote+attached-002dcontainer-002b7b22636f6e7461696e65724e616d65223a222f73696d5f7632227d.vscode-resource.vscode-cdn.net/root/.vscode-server/extensions/rooveterinaryinc.roo-cline-3.51.0/webview-ui/build/assets/index.js:261:44530
    at https://vscode-remote+attached-002dcontainer-002b7b22636f6e7461696e65724e616d65223a222f73696d5f7632227d.vscode-resource.vscode-cdn.net/root/.vscode-server/extensions/rooveterinaryinc.roo-cline-3.51.0/webview-ui/build/assets/index.js:261:35648
    at div
    at div
at Alo (../../../../webview-ui/src/components/chat/ChatView.tsx:69:3)
at Bbo (../../../../webview-ui/src/App.tsx:69:5)
at d (../../../../node_modules/.pnpm/@radix-ui+react-context@1.1.2_@types+react@18.3.23_react@18.3.1/node_modules/@radix-ui/react-context/dist/index.mjs:27:14)
at X3t (../../../../node_modules/.pnpm/@radix-ui+react-tooltip@1.2.6_@types+react-dom@18.3.7_@types+react@18.3.23__@types+reac_99504aa956c623e0884a4e3cb215669a/node_modules/@radix-ui/react-tooltip/dist/index.mjs:29:4)
at A3 (../../../../webview-ui/src/components/ui/tooltip.tsx:8:47704)
at G6t (../../../../node_modules/.pnpm/@tanstack+react-query@5.76.1_react@18.3.1/node_modules/@tanstack/react-query/build/modern/QueryClientProvider.js:20:2)
at SHt (../../../../webview-ui/src/i18n/TranslationContext.tsx:16:70)
at xHt (../../../../webview-ui/src/context/ExtensionStateContext.tsx:192:86)
at jbo (../../../../webview-ui/src/components/ErrorBoundary.tsx:18:2)
at i (../../../../node_modules/.pnpm/react-i18next@15.5.1_i18next@25.2.1_typescript@5.8.3__react-dom@18.3.1_react@18.3.1__react@18.3.1_typescript@5.8.3/node_modules/react-i18next/dist/es/withTranslation.js:6:4)
at Wbo (../../../../webview-ui/src/App.tsx:319:25)

[roomote-v0]

I've analyzed this issue and here's my proposed implementation plan:

The crash is caused by TypeError: n.replace is not a function in countMarkdownHeadings() (webview-ui/src/utils/markdown.ts:10). The function accepts string | undefined and guards against falsy values, but at runtime message.text (passed from ChatRow.tsx via OpenMarkdownPreviewButton) can be a non-string truthy value (e.g., an array). When .replace() is called on a non-string, it throws.

Proposed fix:

1. In countMarkdownHeadings() (webview-ui/src/utils/markdown.ts), add a typeof text !== "string" guard that returns 0 early, so any non-string value is handled safely.
2. Add a corresponding test case in the existing test file to cover this scenario.

This is a minimal, defensive fix that prevents the crash without changing the component hierarchy or prop types.

Please mention @roomote in a comment saying that you approve this plan, and I'll proceed with the implementation.

[c-sunc6]

@roomote 同意该方案

[c-sunc6]

@roomote approve this plan

[roomote-v0]

I've seen your approval and will proceed with the implementation now. I'll add a typeof guard in countMarkdownHeadings() to handle non-string values safely, along with test coverage. I'll open a PR shortly.

[roomote-v0]

Opened PR #11880 with my attempt at addressing this. Feedback welcome! #11880