Skip to content

chore: update fullsend shim workflow - #47

Open
fullsend-ai-fullsend[bot] wants to merge 1 commit into
mainfrom
fullsend/onboard
Open

chore: update fullsend shim workflow#47
fullsend-ai-fullsend[bot] wants to merge 1 commit into
mainfrom
fullsend/onboard

Conversation

@fullsend-ai-fullsend

Copy link
Copy Markdown
Contributor

This PR updates the fullsend shim workflow to match the current template in the .fullsend config repo.

The shim content has drifted from the template — this brings it back in sync.

@fullsend-ai-fullsend
fullsend-ai-fullsend Bot requested a review from a team as a code owner July 31, 2026 21:34
@fullsend-ai-review

fullsend-ai-review Bot commented Jul 31, 2026

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 9:35 PM UTC · Completed 9:50 PM UTC
Commit: b439095 · View workflow run →

@fullsend-ai-review

fullsend-ai-review Bot commented Jul 31, 2026

Copy link
Copy Markdown

Review

Findings

Info

  • [protected-path] .github/workflows/fullsend.yaml — This PR modifies a file under the protected path .github/. Note: intent-coherence finding cites the managed-file header (This file is managed by fullsend) and bot authorship (fullsend-ai-fullsend[bot]) as authorization for this change. Human approval is still required for protected-path changes.
Previous run

Review

Findings

Low

  • [missing-authorization] .github/workflows/fullsend.yaml — No linked issue for a non-trivial change with three distinct behavioral modifications: (1) permissions scoping from workflow-level to job-level, (2) concurrency group isolation for labeled events (#2452), and (3) event filtering restricting labeled triggers to ready-* labels. The managed-file model (file header + bot author) provides implicit authorization through upstream template conformance, but referencing #2452 or the upstream PR in the body would improve traceability.

Info

  • [protected-path] .github/workflows/fullsend.yaml — This PR modifies a file under the protected path .github/. Note: intent-coherence finding cites the managed-file header (This file is managed by fullsend) and bot authorship (fullsend-ai-fullsend[bot]) as authorization for this change. Human approval is still required for protected-path changes.

@fullsend-ai-review fullsend-ai-review Bot added the requires-manual-review Review requires human judgment label Jul 31, 2026
Update the shim workflow to match the current template
in the .fullsend config repo.
@fullsend-ai-review

fullsend-ai-review Bot commented Aug 1, 2026

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 3:39 PM UTC · Completed 3:52 PM UTC
Commit: d290aab · View workflow run →

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

Labels

requires-manual-review Review requires human judgment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants