fix(ci): fail post.sh when the model stages a path outside the conflict set
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 74/100
調査の方向性
Read .github/scripts/merge-base/post.sh and its existing staged-path checks, then inspect .github/scripts/merge-base/post.test.sh to follow the test patterns. Compare the final index against the stage-0 snapshot and the snapshot’s unmerged paths; add a test for an unexpected staged path. Done when that case fails with a distinct reason and the existing checks still pass.
索引モデルが issue の本文から書いたものです。
説明
Problem
.github/scripts/merge-base/post.sh escalates on an unmerged entry, a
conflict marker in any entry staged since the snapshot, and any change
under .github/workflows/. It does not limit which paths the model may
stage. If resolve-conflicts in CI mode ever runs git add -A, git add . or a directory (a model mistake, or steered by text in the conflicted
files), any other modified or untracked file in the checkout is staged,
passes the marker scan, and reaches git write-tree and the commit.
The skill's allowed-tools cannot close this: Bash(git add:*) is a
prefix rule, and the rule syntax has no way to exclude -A, --all,
. or :/. The skill forbids bulk staging in prose, and the
no-wildcard-staging eval grader measures compliance, but nothing on
the runner enforces it.
Proposed fix
In post.sh, after the existing checks, fail closed (new reason, for
example unexpected-path-staged) when the final index holds an entry
that is neither identical to a stage-0 snapshot entry nor a path the
snapshot held unmerged. Mirror the workflow-staged check and add the
case to post.test.sh.
Related
Raised in review of #7178:
https://github.com/melodic-software/claude-code-plugins/pull/7178#discussion_r4244581612
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 8時間 24分
- マージ済み PR（30日）
- 947
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
needs-human needs-triage priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
melodic-software/claude-code-plugins#7455 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#7351 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
agent-ready priority: high work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
melodic-software/claude-code-plugins#7312 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
melodic-software/claude-code-plugins#7288 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
knowledge: docpage-digest dual verification defines no verdict vocabulary or pass/fail thresholdオープンneeds-human
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
melodic-software/claude-code-plugins#7286 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
Netcracker/qubership-test-pipelines#307 ·
メンテナーはふだん 2 日以内に返信
-
Webcam overlay freezes during voxtype dictation while screen recording (pause_media pauses the overlay mpv)対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
TYPO3BestPractices/tea#2409 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
msys2/MSYS2-packages#6825 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
fw876/helloworld#2067 ·
メンテナーはふだん 1 日以内に返信