PR #67663: fix(wizard): guard undefined prompt return values in setup helpers

• Repository: openclaw/openclaw
• Author: neo1027144-creator
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/67663

Description (problem / solution / changelog)

Summary

Fixes #67649 — QuickStart wizard crashes with TypeError: Cannot read properties of undefined (reading 'trim') after showing the DM access warning.

Root Cause

Commit e1a350d08e ("refactor: remove redundant setup helper conversions") removed String() wrappers around prompter.text() return values. However, @clack/prompts text() can return undefined in edge cases rather than an empty string. The removed String() calls had been silently converting undefined to the string "undefined", preventing the crash.

Changes

• src/channels/plugins/setup-wizard-helpers.ts — Restore runtime safety at 5 call sites where prompter.text() results are consumed. Instead of reverting to String() (which turns undefined into "undefined"), use typeof guards that fall back to empty string. Same pattern used in #67086.
• src/channels/plugins/setup-wizard.ts — Guard rawValue.trim() in the text input step against non-string return values.
• src/channels/plugins/setup-wizard-helpers.test.ts — Add regression test: verify promptSingleChannelToken does not throw TypeError when prompter.text() returns undefined at runtime.

Testing

• pnpm test src/channels/plugins/setup-wizard-helpers.test.ts — 89 passed (including the new regression test), 1 pre-existing timeout (unrelated patchChannelConfigForAccount test).

Changed files

• src/channels/plugins/setup-wizard-helpers.test.ts (modified, +55/-0)
• src/channels/plugins/setup-wizard-helpers.ts (modified, +12/-7)
• src/channels/plugins/setup-wizard.ts (modified, +2/-0)

---

PR #67694: fix(ui): ensure markdown-it-task-lists is bundled by Vite [AI-assisted]

• Repository: openclaw/openclaw
• Author: neo1027144-creator
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/67694

Description (problem / solution / changelog)

Summary

Fixes #67680 — Control UI renders a blank page because the Vite production bundle contains an unresolved bare module specifier for markdown-it-task-lists.

Root Cause

The markdown-it-task-lists package is declared in ui/package.json but the build pre-flight check in scripts/ui.js only validates vite and dompurify. If markdown-it-task-lists is not installed (e.g., incomplete pnpm install in CI), Vite cannot resolve the import and emits a bare specifier in the output bundle. Browsers cannot load bare specifiers, causing the entire UI to crash.

Fix

1. scripts/ui.js: Add markdown-it-task-lists to the depsInstalled() pre-flight check. If the package is missing, pnpm install runs automatically before vite build.
2. ui/vite.config.ts: Add markdown-it-task-lists to optimizeDeps.include for consistent pre-bundling.

Changed Files

• scripts/ui.js — 1 line added (require.resolve('markdown-it-task-lists'))
• ui/vite.config.ts — optimizeDeps.include extended

Testing

• Verified require.resolve('markdown-it-task-lists') resolves correctly from ui/package.json
• Existing ui/src/ui/markdown.test.ts covers task list rendering

Closes #67680

[!NOTE] This PR also contains the wizard guard fix from #67663 (shared base branch). GitHub will auto-deduplicate when either is merged first.

---

[!NOTE] This contribution was made with the assistance of AI tools (Antigravity).

Changed files

• scripts/ui.js (modified, +1/-0)
• ui/vite.config.ts (modified, +1/-1)

---

PR #67802: Fix: [Bug]: TypeError: Cannot read properties of undefi

• Repository: openclaw/openclaw
• Author: ztexydt-cqh
• State: closed | merged: False
• Link: https://github.com/openclaw/openclaw/pull/67802

Description (problem / solution / changelog)

This PR fixes the issue described in #67649.

Closes #67649

Changed files

• CONTRIBUTORS_AUTO.md (added, +1/-0)