fix(knowledge): list-promotion-candidates prints a stack trace on a malformed claim inventory
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 82/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript, nodejs
- 領域
- cli
調査の方向性
plugins/knowledge/skills/video-digest/extraction/watch/ の list-promotion-candidates.js から始めてください。セッション数ゼロの場合の throw は 45〜48 行目付近にあり、CLI のブロック(114〜121 行目付近)には catch がありません。同じディレクトリの vision-gated-promote.js(93〜107 行目)からエラー処理をコピーしてください。そちらはメッセージを stderr に書き出し、終了コード 1 で終了します。不正な inventory を渡したときに、形式のヒントを含む 1 行のメッセージだけが出力され、0 以外の終了コードで終了すること、そしてその両方を確認する CLI テストがあれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Summary
Run as a CLI, list-promotion-candidates.js calls listPromotionCandidates with no error handling, so its documented failure (no session parsed from research/claim-inventory.md) and any missing-file error escape as an uncaught exception with a Node stack trace. The sibling vision-gated-promote.js catches, prints the message to stderr and exits 1.
Failure scenario
Expected: a malformed claim inventory prints the one-line message with the format hint to stderr and exits with a nonzero code, with no stack trace.
Actual: the function throws at the zero-sessions check, and the CLI block does not catch it, so Node prints the stack trace. The useful part (SESSION_FORMAT_HINT) is buried under it.
Evidence
- The throw: https://github.com/melodic-software/claude-code-plugins/blob/7787be41e98e30cd6e3b82e99494886d77e30895/plugins/knowledge/skills/video-digest/extraction/watch/list-promotion-candidates.js#L45-L48
- CLI block with no catch: https://github.com/melodic-software/claude-code-plugins/blob/7787be41e98e30cd6e3b82e99494886d77e30895/plugins/knowledge/skills/video-digest/extraction/watch/list-promotion-candidates.js#L114-L121
- The sibling's pattern to match: https://github.com/melodic-software/claude-code-plugins/blob/7787be41e98e30cd6e3b82e99494886d77e30895/plugins/knowledge/skills/video-digest/extraction/watch/vision-gated-promote.js#L93-L107
Suggested fix
Wrap the call in try/catch, write error.message with writeStderr, and process.exit(1), as vision-gated-promote.js does. Add a CLI test with a claim inventory that parses to zero sessions asserting a one-line stderr and a nonzero exit.
Related
- #6813 (session-coverage gates and a zero-session inventory)
🤖 Generated with Claude Code
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 6時間 42分
- マージ済み PR(30日)
- 902
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
needs-triage priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
melodic-software/claude-code-plugins#7288 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdオープンneeds-human needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
melodic-software/claude-code-plugins#7286 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs-triage priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
melodic-software/claude-code-plugins#7282 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs-triage priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
melodic-software/claude-code-plugins#7273 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs-human needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
melodic-software/claude-code-plugins#7232 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
Install software-properties-common before adding the Suricata PPA対応中かも @himanishpuri が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
Ergane-Foundation/vajra#55 · コメント 2 件 ·
メンテナーはふだん 2 日以内に返信
-
[cifuzz] "Minimizing crashes reduces fuzzing time" is logged for every batch run, even with minimize-crashes off対応中かも @martinfrancois が今日担当しました。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
alunduil/alunduil-chezmoi#914 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
obra/superpowers#2505 ·
メンテナーはふだん 3 日以内に返信
-
cao-evolution cao-evolution:compiler-security
難易度 1/5 1時間未満 初心者へのやさしさ 86/100
githubnext/gh-aw-cao#17586 ·
メンテナーはふだん 1 日以内に返信