[codex] LINE: avoid runtime lookup during onboarding#49960
Conversation
Greptile SummaryThis PR fixes a crash during Key changes:
Confidence Score: 4/5
Last reviewed commit: "LINE: avoid runtime ..." |
Greptile SummaryThis PR fixes a crash during Key changes:
Confidence Score: 4/5
Last reviewed commit: "LINE: avoid runtime ..." |
Summary
LINE runtime not initialized - plugin not registeredwhen rendering the channel picker.pnpm openclaw onboard --install-daemoncould fail before the operator selected any channel, blocking a common first-run path.Change Type (select all)
Scope (select all touched areas)
Linked Issue/PR
User-visible / Behavior Changes
QuickStart onboarding no longer requires the LINE plugin runtime to be initialized just to render
Select channel (QuickStart). Users hittingpnpm openclaw onboard --install-daemonshould now reach the channel picker instead of failing immediately after the channel primer.Security Impact (required)
NoNoNoNoNoYes, explain risk + mitigation:Repro + Verification
Environment
Steps
pnpm openclaw onboard --install-daemon.How channels worknote.Expected
Select channel (QuickStart)without crashing.Actual
LINE runtime not initialized - plugin not registeredwhile building the QuickStart picker.Evidence
Human Verification (required)
What I personally verified (not just CI), and how:
pnpm test -- src/commands/onboard-channels.e2e.test.tsand confirmed the full file passes, including the new regression that renders QuickStart and skips from the picker.Review Conversations
Compatibility / Migration
YesNoNoFailure Recovery (if this breaks)
013ecde2b1401b6da19cf77047ac43e5dda7e483.extensions/line/src/config-adapter.ts,src/commands/onboard-channels.e2e.test.ts.Risks and Mitigations