fix: return a result instead of throwing when the host passes no context

[isamu]

Summary

host が execute() に null の context を渡すためクラッシュしていたのを修正した。
receptron/mulmoclaude#2709 と同じ原因で、調査の結果 14 プラグインが同じ問題を抱えていた
(receptron/mulmoclaude#2758 に横断報告)。

TypeError: Cannot read properties of null (reading 'app')

Items to Confirm / Review

1. 型が実態と食い違っているのが根本原因 — gui-chat-protocol の
   execute: (context: ToolContext, args: A) => ... は context を non-nullable と宣言しているが、
   MulmoClaude の server bridge は実際には null を渡す。型が嘘をついているのでガードを書く動機が無かった。
   プロトコル側の型修正を #2758 で提案している。
2. 修正は最小限に留めた — context を ToolContext | null | undefined に広げ、
   src/core/plugin.ts:28 の参照を ?. でガードしただけ。ガード後の呼び出し
   (context.app.xxx(...) 等)はそのまま残してある。TypeScript が optional chaining 経由で
   narrowing してくれるうえ、メソッド呼び出しの形を保つことで this 束縛を壊さないため。
3. 回帰テストを追加した — tests/plugin.test.ts。修正を戻すと実際に fail することを確認済み
   (テストが常に通るだけの飾りになっていない)。

変更内容

• src/core/plugin.ts — context を nullable にし、src/core/plugin.ts:28 をガード
• tests/plugin.test.ts — null / undefined / 空オブジェクトの 3 ケース
• package.json — test script と tsx(MindMap の修正と同じ構成)、patch version bump
• .github/workflows/pull_request.yaml — CI で yarn test を実行
• eslint.config.js — tests/** を lint 対象外に(ガイド §3: 型プログラムを小さく保つ)

検証

node_modules を消してからのクリーン install で確認済み
(warm な node_modules は lockfile 変更後の CI を再現しないため):

rm -rf node_modules && yarn install --frozen-lockfile
yarn typecheck  → pass
yarn lint       → pass
yarn build      → pass
yarn test       → pass

さらに execute(null, args) を直接呼んで、修正前は TypeError、修正後は正常な
ToolResult が返ることを実機で確認した。

🤖 Generated with Claude Code

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	tsx@​4.23.4

View full report