PR #66727: fix: guard against undefined path in formatDocsLink

• Repository: openclaw/openclaw
• Author: nightq
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/66727

Description (problem / solution / changelog)

Summary

Fix TypeError crash in openclaw configure and onboarding when channel docs path is undefined.

Root Cause

formatChannelSelectionLine passes meta.docsPath to formatDocsLink, but docsPath can be undefined for some channels. formatDocsLink calls path.trim() on the undefined value, throwing TypeError: Cannot read properties of undefined (reading 'trim').

Fix

Added nullish coalescing guard (path ?? "") and early return with fallback when trimmed result is empty.

Test Plan

• openclaw configure wizard no longer crashes on channels without docsPath
• Onboarding channel selection no longer crashes
• Existing formatDocsLink callers continue to work

Closes openclaw#66718 Closes openclaw#66693

Changed files

• src/terminal/links.ts (modified, +5/-3)

PR #66882: fix(wizard): guard text-input prompts against undefined clack results

• Repository: openclaw/openclaw
• Author: nikilster
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/66882

Description (problem / solution / changelog)

Summary

Fixes the widespread onboarding crash:

TypeError: Cannot read properties of undefined (reading 'trim')

that kills openclaw onboard / openclaw configure during channel selection and channel-specific text-input steps (Telegram, Slack, Discord, etc.).

Root cause

@clack/prompts text() can resolve to undefined in some edge cases (e.g. submitting an empty value when neither initialValue nor placeholder is set), even though WizardPrompter.text is declared as Promise<string>. The wrapper in src/wizard/clack-prompter.ts passes the value through guardCancel but does not coerce the non-cancel path, so undefined leaks out to callers. buildChannelSetupWizardAdapterFromSetupWizard in src/channels/plugins/setup-wizard.ts then calls rawValue.trim() directly and crashes.

Fix

• src/wizard/clack-prompter.ts — enforce the declared Promise<string> contract by coercing nullish results to "" after guardCancel. This covers every prompter.text caller in one place.
• src/channels/plugins/setup-wizard.ts — belt-and-suspenders rawValue ?? "" at the existing call site so the same symptom can't be reintroduced by a future prompter swap.
• src/wizard/clack-prompter.test.ts — unit tests for the wrapper covering both the undefined → "" coercion and the string passthrough.

Related issues

• #66693 [Bug]: While onboarding: TypeError: Cannot read properties of undefined (reading 'trim')
• #66718 [Bug] openclaw configure TypeError: Cannot read properties of undefined (reading 'trim')
• #66848 [Bug]: TypeError: Cannot read properties of undefined (reading 'trim')
• #66677 [Bug]: Select channel (QuickStart)
• #66624 (closed) same symptom, v2026.4.14
• #66728 (closed) Skip for now variant
• #66641 (closed) installer crashes after Select channel (QuickStart)
• #66619 (closed) Telegram setup variant

Test plan

• New unit tests in src/wizard/clack-prompter.test.ts pass locally against the patched wrapper.
• CI: pnpm test:unit / channel plugin tests.
• Manual: openclaw onboard, pick Telegram, step through the text-input steps without hitting the TypeError.

Changed files

• src/channels/plugins/setup-wizard-helpers.ts (modified, +7/-7)
• src/channels/plugins/setup-wizard.ts (modified, +1/-1)
• src/wizard/clack-prompter.test.ts (modified, +40/-2)
• src/wizard/clack-prompter.ts (modified, +7/-1)