From c07e23d36a2cd250d404a74d0279ff1f4e7762bb Mon Sep 17 00:00:00 2001 From: Marten Klitzke Date: Wed, 7 Oct 2026 12:02:32 +0200 Subject: [PATCH] fix(skills): read StatusContext results in deps CI gate --- .claude/skills/deps/SKILL.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.claude/skills/deps/SKILL.md b/.claude/skills/deps/SKILL.md index 4caf5b8..b3672ba 100644 --- a/.claude/skills/deps/SKILL.md +++ b/.claude/skills/deps/SKILL.md @@ -71,10 +71,10 @@ Provider majors always go to the report. Terraform providers use the major versi ```bash gh pr view --repo fleetyards/infrastructure \ --json statusCheckRollup \ - --jq '[.statusCheckRollup[] | select(.conclusion != "SUCCESS" and .conclusion != "SKIPPED" and .conclusion != "NEUTRAL")] | map("\(.name): \(.conclusion // .status)") | .[]' + --jq '[.statusCheckRollup[] | select((.conclusion // .state) as $r | $r != "SUCCESS" and $r != "SKIPPED" and $r != "NEUTRAL")] | map("\(.name // .context): \(.conclusion // .state // .status)") | .[]' ``` -Empty output means green. Remember what `terraform_test` does and does not cover — see the warning above. +Empty output means green. The rollup mixes `CheckRun` entries (result in `conclusion`) with commit `StatusContext` entries such as CodeRabbit and Devin Review (result in `state`, name in `context`), so the filter has to read both. Remember what `terraform_test` does and does not cover — see the warning above. #### Gate C — the plan is clean