Skip to content

chore(ci): pin pr-agent to skynergroup/.github@main - #5

Merged
yashiels merged 1 commit into
mainfrom
chore/pr-agent-pin-main
Jul 12, 2026
Merged

chore(ci): pin pr-agent to skynergroup/.github@main#5
yashiels merged 1 commit into
mainfrom
chore/pr-agent-pin-main

Conversation

@yashiels

@yashiels yashiels commented Jul 12, 2026

Copy link
Copy Markdown
Owner

User description

Repoint pr-agent caller from pinned SHA (e5d65d0, broken secret syntax) to skynergroup/.github@main. Fixes the broken pr-agent AND bakes in ai_timeout=600 + max_model_tokens=200000 via the shared workflow. Key stays repo-level.


PR Type

Other


Description

  • Repin pr-agent caller to skynergroup/.github@main

  • Replace broken pinned SHA workflow reference

  • Keep repo-level KIMI_API_KEY secret override


Diagram Walkthrough

flowchart LR
  A[".github/workflows/pr-agent.yml"] -- "uses @e5d65d0" --> B["Broken SHA reference"]
  A -- "uses @main" --> C["Shared workflow on main"]
Loading

File Walkthrough

Relevant files
Configuration changes
pr-agent.yml
Pin pr-agent workflow to shared repo main branch                 

.github/workflows/pr-agent.yml

  • Switch uses reference from pinned SHA to @main
  • Retain KIMI_API_KEY secret injection unchanged
  • Fix reusable workflow reference syntax
+1/-1     

@yashiels
yashiels merged commit 0816022 into main Jul 12, 2026
2 checks passed
@yashiels
yashiels deleted the chore/pr-agent-pin-main branch July 12, 2026 21:03
@github-actions github-actions Bot added the Other label Jul 12, 2026
@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🏅 Score: 70
🧪 No relevant tests
🔒 Security concerns

Supply-chain risk:
Using @main for a reusable workflow means the calling workflow inherits whatever the shared workflow becomes, including any malicious changes, and passes the KIMI_API_KEY secret to it. Pin to a trusted SHA to keep immutable supply-chain guarantees.

✅ No TODO sections
🔀 No multiple PR themes
⚡ Recommended focus areas for review

Supply-chain risk

The reusable workflow is now referenced at @main, which is mutable. Any future change to skynergroup/.github@main will execute with the repo-level KIMI_API_KEY secret, enabling secret exfiltration or unintended code execution if that shared workflow is compromised. Pin to a verified SHA and verify the secret syntax fix in that commit instead.

uses: skynergroup/.github/.github/workflows/pr-agent.yml@main

Comment on lines 11 to +13
jobs:
review:
uses: skynergroup/.github/.github/workflows/pr-agent.yml@e5d65d05cdd00cdec39dff9fd3a7c32d5a41931b
uses: skynergroup/.github/.github/workflows/pr-agent.yml@main

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: Pin the reusable workflow to an immutable commit SHA rather than a mutable branch reference. Referencing main means any upstream change or compromise in skynergroup/.github will execute immediately in your CI, creating a supply-chain attack vector. Revert to the previously pinned SHA and update it intentionally after reviewing upstream changes. [security, importance: 9]

Suggested change
jobs:
review:
uses: skynergroup/.github/.github/workflows/pr-agent.yml@e5d65d05cdd00cdec39dff9fd3a7c32d5a41931b
uses: skynergroup/.github/.github/workflows/pr-agent.yml@main
jobs:
review:
uses: skynergroup/.github/.github/workflows/pr-agent.yml@e5d65d05cdd00cdec39dff9fd3a7c32d5a41931b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant