You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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"]
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.
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.
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]
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
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.
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-agentcaller toskynergroup/.github@mainReplace broken pinned SHA workflow reference
Keep repo-level
KIMI_API_KEYsecret overrideDiagram Walkthrough
File Walkthrough
pr-agent.yml
Pin pr-agent workflow to shared repo main branch.github/workflows/pr-agent.yml
usesreference from pinned SHA to@mainKIMI_API_KEYsecret injection unchanged