skill wording: settings-token fallback in evals
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 92/100
- issue の種類
- ドキュメント
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- bash, markdown
調査の方向性
plugins/evals/skills/plugin-eval/SKILL.md の 306 行目付近から始め、plugins/code-tidying/skills/tidy/SKILL.md の hard_exclusions の表現を、示された先例として使用します。設定の表現を更新し、plugins/evals/ のベースライン行を削除し、プラグインのモードで必要な changelog またはバージョン bump のエントリを追加します。完了を確認するため、bash plugins/skill-quality/scripts/check-skill.sh plugins/evals/skills と提供された grep コマンドを実行します。
索引モデルが issue の本文から書いたものです。
説明
No related issue: follow-up from the Check 31 rollout (baseline row in plugins/skill-quality/scripts/settings-token-baseline.txt).
Summary
plugins/evals/skills/plugin-eval/SKILL.md:306 uses ${user_config.grader_run_twice} as the subject of a condition ("unless ... is false") with no inline fallback, so an unexpanded token reads as not false.
Fix
State the token once with the existing fallback form, e.g. "Grader run twice: ${user_config.grader_run_twice} (unexpanded or empty means the default)", refer to "the grader-run-twice setting" afterwards, and remove the plugins/evals/ row from the baseline file. The form is the one plugins/code-tidying/skills/tidy/SKILL.md uses for hard_exclusions. One PR, plus an .changes/evals/ or version-bump entry per the plugin's changelog mode.
Verification
bash plugins/skill-quality/scripts/check-skill.sh plugins/evals/skills prints no Check 31 line and grep -c '^plugins/evals/' plugins/skill-quality/scripts/settings-token-baseline.txt returns 0.
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 8時間 41分
- マージ済み PR(30日)
- 892
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
needs-human work-class: mechanical
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
melodic-software/claude-code-plugins#8021 ·
メンテナーはふだん 1 日以内に返信
-
needs-human work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
melodic-software/claude-code-plugins#8018 ·
メンテナーはふだん 1 日以内に返信
-
docs(planning): point the interview surface context at the surface README instead of restating itオープンneeds-human work-class: mechanical
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
melodic-software/claude-code-plugins#8015 ·
メンテナーはふだん 1 日以内に返信
-
needs-human work-class: mechanical
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
melodic-software/claude-code-plugins#8004 ·
メンテナーはふだん 1 日以内に返信
-
needs-human work-class: mechanical
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
melodic-software/claude-code-plugins#8003 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
-
good first issue packaging
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
-
help wanted let's document new command
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
tldr-pages/tldr#24773 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 63/100
メンテナーはふだん 1 日以内に返信