chore: update fullsend shim workflow - #570
fullsend-ai-fullsend[bot] wants to merge 1 commit into
Conversation
Update the shim workflow to match the current template in the .fullsend config repo.
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
🚀 Preview is available at https://f20630e0.enterprise-contract.pages.dev |
|
Risk Assessment: moderate (2/5) DetailsBot-authored XS shim workflow update (1 file, 2 lines) with no security-sensitive paths and no dependency changes, but CI_WORKFLOW_CHANGED and a protected-path hit elevate Tier 1 above 1, yielding a moderate composite of 1.55. |
ReviewFindingsHigh
Next steps:
|
| github.event_name != 'issue_comment' | ||
| || github.event.comment.user.type != 'Bot' | ||
| uses: conforma/.fullsend/.github/workflows/dispatch.yml@e8f0a9f37ce121873e8e320c2758eabee6752df8 # main | ||
| uses: conforma/.fullsend/.github/workflows/dispatch.yml@main |
There was a problem hiding this comment.
[high] protected-path
PR modifies a governance/infrastructure file under a protected path (.github/). No linked issue explicitly authorizes changes to protected paths, so human approval is required. Additionally, the specific change swaps a 40-char commit-SHA pin (@e8f0a9f37ce121873e8e320c2758eabee6752df8 # main) for a mutable branch reference (@main) on a reusable workflow called under pull_request_target with actions: write + id-token: write. This reduces supply-chain defense-in-depth (subsequent commits to .fullsend/main execute immediately with those permissions and a repo-scoped OIDC token, with no per-consumer audit). Mitigating context: .fullsend is same-org and its upstream template itself uses @main, so this really is a shim resync — but confirm org policy on same-org reusable workflows and the branch-protection posture on .fullsend/main before approving.
Suggested fix: Confirm explicit human approval given org policy and .fullsend/main branch-protection; OR re-pin to a full-length commit SHA with a # main trailing comment and let the existing digest-bump automation keep it up to date.
This PR updates the fullsend shim workflow to match the current template in the
.fullsendconfig repo.The shim content has drifted from the template — this brings it back in sync.
Getting started
Once this PR is merged, interact with fullsend by commenting one of these slash commands. The supported target (issue and/or pull request) is shown for each:
/fs-triage(issue or PR) — Invoke the triage agent to categorize, label, and assess an issue./fs-code(issue only) — Invoke the code agent to implement a fix for an issue and open a PR./fs-review(PR only) — Invoke the review agent to review a pull request./fs-fix(PR only) — Invoke the fix agent to address review feedback on a pull request./fs-retro(issue or PR) — Invoke the retro agent to analyze completed work and propose improvements./fs-prioritize(issue or PR) — Invoke the prioritize agent to score an issue for project board ranking.