Repository navigation
CP-6852: bump snok/install-poetry to v1.4.2 so CI can install poetry - #28
Merged
Merged
Conversation
install.python-poetry.org now returns a 403 to the Python urllib user agent that v1.4.1 downloads with. v1.4.2 fetches a pinned installer from raw.githubusercontent.com instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> AI-Tool: Claude Code CLI AI-Model: claude-opus-5-5 AI-Reasoning: medium AI-Session: 44e374b1-da7f-4486-9f70-524b793b5c67 AI-Host: stephenbrannen-MacBook-Pro AI-Workspace: regenerate-client AI-Subagent-Models: claude-sonnet-5-5
stephen-kintsugi
requested review from
kin-review-bot and
perry-trinh
and removed request for
a team
November 9, 2026 19:09
Collaborator
Author
|
/kin-review |
kin-review-bot
approved these changes
Nov 9, 2026
kin-review-bot
left a comment
There was a problem hiding this comment.
Approving: one-line SHA-pinned CI action bump, Low risk, no findings.
- Blockers: 0
- Nits: 0
Decision recordreview_event: APPROVE
would_have: null
blockers: 0
nits: 0
nits_dropped: 0
open_kept: 0
held_back: 0
head_sha: 4730298d2b2a15b50417f2fdbbbe772fbdfc95ff
base_sha: 9ceb299c02f44e83bb2f52dd440e1e0090c87efa
policy_ref: f968c5b1710a0c511787d63558668610616622a8
run_nonce: d6435ad0-e7f7-4115-9ffd-b2618b20f3ab
record_verdict: APPROVE
risk_level: Low
inline_comments: 0
reanchored: 0
pr_level: 0
audits_required: []
audits_missing: []
action_taken: approved
gates: []
evidence: 1 files, +1 / -1, reconciliation OK, removed names 0
ledger_mode: first_review
ledger_trust: dedicated
last_reviewed_commit: null
cleared_orphans: []
rationale: "One-line SHA-pin bump of snok/install-poetry in .github/actions/setup-python-poetry/action.yml. CI-only, fail-closed, no shared-resource load; both SHAs verified against upstream tags v1.4.1 and v1.4.2 via git ls-remote."Agent review summary
Why
What to do
Failed gatesEvery gate passed. Reviewed |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CI in this repo cannot install Poetry. Since Poetry moved its installer hosting on Nov 8,
install.python-poetry.orgreturns a Cloudflare 403 (error 1010) to Python'surllibuser agent, andsnok/install-poetryv1.4.1 downloads the installer that way. This pins the action to v1.4.2, which downloads a pinned installer commit fromraw.githubusercontent.cominstead and never contacts that host..github/actions/setup-python-poetry/action.yml:snok/install-poetryv1.4.1 to v1.4.2, pinned by commit SHA (a783c322, the v1.4.2 tag) with a# v1.4.2comment.Approve from: @kintsugi-tax/core-product
The same fix is green on kintsugi-tax/platform-client#65. The longer-term fix is one shared setup action for every Python repo (CP-6855).
Deferred findings (pre-existing, not changed by this PR):
/, which does not include.github/actions/*. Tracked under CP-6855.ci.ymlhas no top-levelpermissions:, so jobs onmainget the repo's default write token. Worth its own ticket.Jira
CP-6852
Tests
lint-and-testand all fivetest-matrixlegs inci.yml. A green run there is the proof./pr-review(all five phases, plusaudit-code-standardsandaudit-ci-pinning): no blockers, no findings on the changed line.Document Checklist
🤖 Generated with Claude Code
Note
Low Risk
Single composite-action pin change with no app or secret-handling impact; only affects how CI installs Poetry.
Overview
Fixes CI Poetry installation by updating the composite
setup-python-poetrystep to usesnok/install-poetryv1.4.2 (commit-pinned) instead of v1.4.1.v1.4.1 pulls the installer from
install.python-poetry.org, which now blocks Python’s defaulturllibuser agent (Cloudflare 403). v1.4.2 uses a pinned installer fetched fromraw.githubusercontent.com, so jobs that call./.github/actions/setup-python-poetrycan install Poetry again. Poetry 2.5.1 and the action inputs are unchanged.Reviewed by Cursor Bugbot for commit 4730298. Bugbot is set up for automated code reviews on this repo. Configure here.