Onboarding: support plugin-owned interactive channel flows by gumadeiras · Pull Request #27191 · openclaw/openclaw

gumadeiras · 2026-02-26T04:54:45Z

Summary

add optional plugin onboarding hooks for interactive setup and configured-state flows
wire onboarding command to hand off to plugin-owned interactive handlers when provided
preserve existing generic flow as fallback when no plugin hook is defined

Validation

pnpm check

greptile-apps · 2026-02-26T04:57:21Z

Greptile Summary

Adds optional plugin-owned interactive onboarding hooks to the channel plugin system, allowing plugins to customize or completely replace the default onboarding flows.

Key changes:

Added configureInteractive hook: lets plugins handle the entire interactive onboarding flow (both configured and unconfigured states)
Added configureWhenConfigured hook: lets plugins customize behavior specifically when a channel is already configured
Onboarding command now checks for these hooks and delegates to plugin implementations when provided
Generic flow preserved as fallback when hooks are not defined
Return type ChannelOnboardingConfiguredResult allows plugins to return "skip" to opt out of configuration

The implementation properly integrates the new hooks as early returns, maintains backward compatibility through optional properties, and follows existing code patterns in the codebase.

Confidence Score: 5/5

This PR is safe to merge with minimal risk
The changes are well-structured, type-safe, and backward compatible. New functionality is added through optional methods that don't affect existing implementations. The code follows established patterns in the codebase and maintains proper type narrowing. No logical errors or security concerns identified.
No files require special attention

_{Last reviewed commit: ce0d9ec}

Copilot

Pull request overview

Adds plugin-extensible onboarding behavior for channel setup, allowing channel plugins to fully own interactive configuration flows while preserving the existing generic onboarding behavior as a fallback.

Changes:

Introduces optional onboarding adapter hooks for interactive setup (configureInteractive) and already-configured flows (configureWhenConfigured).
Updates the channel onboarding command flow to delegate to plugin-owned hooks when provided, with early-return handling for "skip".
Extends onboarding type definitions to support the new hooks and interactive context.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
src/commands/onboard-channels.ts	Wires `setupChannels` to call adapter-provided interactive/configured handlers and apply their results.
src/channels/plugins/onboarding-types.ts	Adds new adapter hook types and context/result unions to support plugin-owned onboarding flows.

gumadeiras · 2026-02-26T06:15:01Z

Merged via squash.

Prepared head SHA: 53872cf
Merge commit: f08fe02

Thanks @gumadeiras!

Syncs 29 upstream commits including: - Account-scoped bind and routing commands (openclaw#27195) - Single-account config migration to accounts.default (openclaw#27334) - Preserve agent-level apiKey/baseUrl during models.json merge (openclaw#27293) - Plugin-owned interactive channel onboarding flows (openclaw#27191) - Android notifications.list, invoke refactoring - Daemon launchd restart hardening - SSRF dispatcher fix, doctor improvements Conflict resolution: 4 import-order conflicts in onboarding/channel commands resolved by taking upstream (no custom code in those files).

@gumadeiras

Merged via /review-pr -> /prepare-pr -> /merge-pr. Prepared head SHA: 53872cf Co-authored-by: gumadeiras <[email protected]> Co-authored-by: gumadeiras <[email protected]> Reviewed-by: @gumadeiras

@gumadeiras