Skip to content

Comments

feat: Add Line plugin#1630

Merged
steipete merged 6 commits intoopenclaw:mainfrom
plum-dawg:line-plugin
Jan 25, 2026
Merged

feat: Add Line plugin#1630
steipete merged 6 commits intoopenclaw:mainfrom
plum-dawg:line-plugin

Conversation

@plum-dawg
Copy link
Contributor

@plum-dawg plum-dawg commented Jan 24, 2026

Adds Line messaging support. they're big in japan?

This was coded by AI, not me, and it works for me, but i haven't thoroughly looked at it. I just thought people might want line support. Bring Clawd to the East, etc etc

@plum-dawg plum-dawg changed the title Line plugin feat: Add Line plugin Jan 24, 2026
@steipete
Copy link
Contributor

steipete commented Jan 24, 2026

Thank you! Was hoping for that!

Code wise... this is not really a plugin - you put 90% of the code in core! I'll unfurl and take over.

@steipete steipete self-assigned this Jan 24, 2026
@steipete
Copy link
Contributor

steipete commented Jan 24, 2026

  • BLOCKER: plugin‑sdk export removals break extension imports (MarkdownConfigSchema/ToolPolicySchema/createTypingCallbacks/ createReplyPrefixContext/resolveControlCommandGate/logAckFailure/ ack‑reaction helpers). Examples: extensions/matrix/src/config-schema.ts:1, extensions/mattermost/src/mattermost/monitor.ts:10, extensions/bluebubbles/src/monitor.ts:5, extensions/nextcloud-talk/src/inbound.ts:3. Source: src/plugin-sdk/index.ts:94. Re‑add exports or update all extensions.
  • BLOCKER: plugin runtime surface shrunk; extensions call removed APIs (channel.text.resolveMarkdownTableMode, channel.text.convertMarkdownTables, channel.session.recordInboundSession, channel.reactions.*, channel.mentions.matchesMentionWithExplicit). Examples: extensions/zalo/src/monitor.ts:573, extensions/bluebubbles/src/ monitor.ts:1535, extensions/matrix/src/matrix/send.ts:53. Runtime now omits them in src/plugins/runtime/index.ts:165 and src/ plugins/runtime/types.ts:182.
  • HIGH: tools invoke endpoint dropped from HTTP chain; handleToolsInvokeHttpRequest no longer called, likely breaking / tools/invoke. src/gateway/server-http.ts:232.
  • HIGH: LINE webhook signature verification uses JSON.stringify(req.body) (parsed body), not raw bytes. Signature likely fails on whitespace/order differences. Need raw body capture or SDK middleware. src/line/webhook.ts:31.
  • MEDIUM: target normalization order leaves group:/room: prefix when input is line:group:… or line:room:…. Outbound ID malformed. extensions/line/src/channel.ts:185.
  • MEDIUM: extension package depends on clawdbot via workspace:* in dependencies; npm install --omit=dev in plugin dir breaks outside monorepo. Move to peerDependencies or devDependencies. extensions/ line/package.json:12.

@plum-dawg
Copy link
Contributor Author

plum-dawg commented Jan 24, 2026

sorry I literally gave it no instructions haha. I just had it go at it for personal reasons. but glad it'll get untangled

steipete added a commit that referenced this pull request Jan 25, 2026
@steipete
feat: add LINE plugin (#1630) (thanks @plum-dawg)
9a6964b
steipete added a commit that referenced this pull request Jan 25, 2026
@steipete
feat: complete LINE plugin (#1630) (thanks @plum-dawg)
ca627dd
steipete added a commit that referenced this pull request Jan 25, 2026
@steipete
chore: drop line plugin node_modules (#1630) (thanks @plum-dawg)
bb82851
steipete added a commit that referenced this pull request Jan 25, 2026
@steipete
test: align BlueBubbles send target fallback (#1630) (thanks @plum-dawg)
43c2c9b
@steipete steipete merged commit c96ffa7 into openclaw:main Jan 25, 2026
21 of 22 checks passed
@steipete
Copy link
Contributor

steipete commented Jan 25, 2026

Landed via squash onto main (admin override to avoid waiting on macOS CI).

  • Gate: pnpm lint && pnpm build && pnpm test
  • Land commit: b7c0d7a
  • Merge commit: c96ffa7

Thanks @plum-dawg!

@Glucksberg Glucksberg mentioned this pull request Jan 25, 2026
Closed
rodrigouroz pushed a commit to rodrigouroz/openclaw that referenced this pull request Jan 26, 2026
@plum-dawg @steipete @rodrigouroz
feat: Add Line plugin (openclaw#1630)
da59035 
* feat: add LINE plugin (openclaw#1630) (thanks @plum-dawg)

* feat: complete LINE plugin (openclaw#1630) (thanks @plum-dawg)

* chore: drop line plugin node_modules (openclaw#1630) (thanks @plum-dawg)

* test: mock /context report in commands test (openclaw#1630) (thanks @plum-dawg)

* test: limit macOS CI workers to avoid OOM (openclaw#1630) (thanks @plum-dawg)

* test: reduce macOS CI vitest workers (openclaw#1630) (thanks @plum-dawg)

---------

Co-authored-by: Peter Steinberger <[email protected]>
Glucksberg added a commit to Glucksberg/OpenClaw that referenced this pull request Jan 28, 2026
@Glucksberg @claude
fix(tts): restore inline menu args removed by LINE plugin PR
35f1c6b 
PR openclaw#1630 (LINE plugin) accidentally removed the detailed TTS command
definition with args and argsMenu, replacing it with a simplified
version. This restores the original functionality from PR openclaw#1559.

Fixes regression where /tts no longer showed the inline action menu
on Telegram and other native command surfaces.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
dgarson pushed a commit to dgarson/clawdbot that referenced this pull request Feb 2, 2026
@plum-dawg @steipete @dgarson
feat: Add Line plugin (openclaw#1630)
26977d8 
* feat: add LINE plugin (openclaw#1630) (thanks @plum-dawg)

* feat: complete LINE plugin (openclaw#1630) (thanks @plum-dawg)

* chore: drop line plugin node_modules (openclaw#1630) (thanks @plum-dawg)

* test: mock /context report in commands test (openclaw#1630) (thanks @plum-dawg)

* test: limit macOS CI workers to avoid OOM (openclaw#1630) (thanks @plum-dawg)

* test: reduce macOS CI vitest workers (openclaw#1630) (thanks @plum-dawg)

---------

Co-authored-by: Peter Steinberger <[email protected]>
@adolago adolago mentioned this pull request Feb 7, 2026
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@steipete steipete

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@plum-dawg @steipete