everything: misleading names in get-resource-links (isOdd) and copy-pasted JSDoc tool names
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 82/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- typescript
調査の方向性
src/everything/tools/get-resource-links.ts、src/everything/tools/trigger-long-running-operation.ts、docs/features.mdを読みます。以前のPR #4503と#4769は、リファクタリング後のコードのプロトタイプとして説明されています。誤解を招くフラグの名前を変更し、偶数と奇数のIDの対応関係を文書化し、各JSDocのツール名を修正します。フラグがその動作に合致し、対応関係が文書化され、両方のヘッダーが各ファイルで登録されるツールと一致したら完了です。
索引モデルが issue の本文から書いたものです。
説明
Problem
Two small naming slips in the everything server, which people read to learn the protocol:
- In
src/everything/tools/get-resource-links.ts, the flagisOddis computed asresourceId % 2 === 0, so it is true for even IDs (which get Text resources). The behavior is right; the name says the opposite, anddocs/features.mddoesn't say which IDs map to which resource type. - The JSDoc header in
src/everything/tools/get-resource-links.tsnames theget-resource-referencetool (copied from its neighbor), and the one insrc/everything/tools/trigger-long-running-operation.tsmisspells its tool's name.
Expected
Rename the flag (for example to isText), document the even/odd mapping, and have each JSDoc header name the tool its file registers.
Prior work
- #4503 by @Joosboy
- #4769 by @simpleqt
The PR(s) above targeted the pre-refactor code and are closed as part of the backlog triage in #4875; this issue is where the work continues, with the PR(s) as a prototype. Thank you to their authors.
- 主要言語
- TypeScript
- スター
- 91.4k
- フォーク
- 11.8k
- 平均マージ
- 7時間 36分
- マージ済み PR（30日）
- 126
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
modelcontextprotocol/servers のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
modelcontextprotocol/servers#5216 ·
メンテナーはふだん 1 日以内に返信
-
bug v2
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
modelcontextprotocol/servers#5215 ·
メンテナーはふだん 1 日以内に返信
-
documentation server-git server-time v2
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
modelcontextprotocol/servers#5213 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
documentation v2
難易度 1/5 1〜3時間 初心者へのやさしさ 78/100
modelcontextprotocol/servers#5212 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
documentation v2
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
modelcontextprotocol/servers#5211 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
modelcontextprotocol/servers の issue をすべて見る
似ている issue
-
agent/scanner help wanted hive/claimed-by-scanner kind/bug stage/020-claimed
難易度 2/5 1〜3時間 初心者へのやさしさ 90/100
hivecommons/rationguard#315 ·
メンテナーはふだん 1 日以内に返信
-
agent-ready area: ui-sdk priority: p3 type: chore
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
schlessera/brain-kit#2534 ·
メンテナーはふだん 1 日以内に返信
-
bug lane:gh-5
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
LLM-4-People/bridlewise-testing#122 ·
メンテナーはふだん 1 日以内に返信
-
bug priority: P2
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
enhancement priority:low ready-for-dev
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
メンテナーはふだん 1 日以内に返信