Skip to content

機能: MCPツールにisErrorの処理を追加 - #8

Merged
soukouki merged 3 commits into
mainfrom
add-is-error-v2
May 16, 2026
Merged

soukouki merged 3 commits into
mainfrom
add-is-error-v2

Conversation

@soukouki

Copy link
Copy Markdown
Owner

概要

MCPツールに isError の処理を追加しました。ノートが見つからない場合に isError を true にする変更です。

変更内容

  • Vault の tool_read / tool_list の戻り値を Result (text + error) に構造化
  • ReadTool / ListTool で error 引数を Response に追加
  • isError のテストケース (成功/失敗) を追加

Closes #4

Comment thread lib/obsidian_fetch.rb Outdated
def tool_list name
name = Vault.normalize_note_name(name)
split_name = name.split(/[\s ]+/)
split_name = name.split(/[\s]+/)

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この変更は戻す

@soukouki

Copy link
Copy Markdown
Owner Author

コメントありがとうございます。

2つの問題を修正しました:

  1. split_name = name.split(/[\s ]+/) の変更を戻しました
  2. Result.new の呼び出しに text: キーワード引数を追加しました -- CI失敗の原因でした

テストは全てパスしています。

@soukouki
soukouki merged commit 33a87de into main May 16, 2026
1 check passed
@soukouki
soukouki deleted the add-is-error-v2 branch May 16, 2026 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

isErrorを付ける

1 participant