chore(skills): update cutting-releases for fullsend-ai @main switch#1968
Conversation
fullsend-ai/.fullsend now references reusable workflows via @main instead of @v0. This means: - Pre-flight: fullsend-ai runs are now a signal that main is healthy before moving the v0 tag (new step E) - Post-flight: fullsend-ai runs no longer tell us anything about v0 resolution, so step C is replaced with an explanatory note and downstream verification is moved to step D with inline commands Assisted-by: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Ralph Bean <rbean@redhat.com>
Site previewPreview: https://4bb8db19-site.fullsend-ai.workers.dev Commit: |
ReviewFindingsHigh
Low
Info
Previous runReviewFindingsHigh
Medium
Low
|
![fullsend-ai-review[bot]](https://avatars.githubusercontent.com/in/3478501?s=60&v=4){kind=small}
|
|
||
| Use `AskUserQuestion` to ask if the user has access to additional | ||
| downstream orgs: | ||
| ``` |
There was a problem hiding this comment.
[medium] logic-error
Step D instructs the agent to present a candidate for retriggering but omits the gh run rerun command template and user-facing prompt format that were in the old step C. The procedural doc should include the command pattern for self-containment.
Suggested fix: Add the rerun command block (gh run rerun --failed --repo <org/repo>) and prompt template to step D.
There was a problem hiding this comment.
Fixed in 9e42931. Added the rerun command block and prompt template.
| Verify the release is not marked as a draft. | ||
|
|
||
| ## C. Check fullsend-ai repos | ||
| ## C. Note on fullsend-ai repos |
There was a problem hiding this comment.
[low] documentation-structure
Heading '## C. Note on fullsend-ai repos' uses informational 'Note on' prefix breaking the action-verb pattern of all other lettered steps. Consider removing the letter or retitling to 'Skip fullsend-ai repos'.
There was a problem hiding this comment.
Fixed in 9e42931. Renamed to "Skip fullsend-ai repos".
- Rename heading to "Skip fullsend-ai repos" to match action-verb pattern - Add rerun command template and prompt to step D for self-containment Assisted-by: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Ralph Bean <rbean@redhat.com>
ralphbean
changed the title
| exact commit we are about to tag. Use this as a pre-flight signal: | ||
|
|
||
| ``` | ||
| gh run list --repo fullsend-ai/.fullsend --limit=5 |
There was a problem hiding this comment.
[low] logic-error
New step E lists 'gh run list --repo fullsend-ai/fullsend --limit=5', but step D already runs 'gh run list --branch=main --limit=5' against the same repo. Only the .fullsend repo check adds new information.
Suggested fix: Remove the redundant fullsend-ai/fullsend check from step E, keeping only the .fullsend repo check.
|
|
||
| ## F. Present summary | ||
| Note: `fullsend-ai/.fullsend` uses `@main`, not `@v0`, so its runs | ||
| are not useful for post-flight `v0` verification. Post-flight checks |
There was a problem hiding this comment.
[low] internal-consistency
The note after step F duplicates the rationale already stated in step E about fullsend-ai/.fullsend using @main. Creates maintenance risk if one copy is updated without the other.
Suggested fix: Remove the redundant note or replace with a cross-reference to step E.
| ``` | ||
| gh run list --repo fullsend-ai/fullsend --limit=3 | ||
| gh run list --repo fullsend-ai/.fullsend --limit=3 | ||
| gh run list --repo <org/repo> --limit=5 |
There was a problem hiding this comment.
[info] prior-finding-resolved
Prior finding about step D missing rerun command template is resolved. The new step D is now self-contained with full command blocks and prompt templates.
|
🤖 Review · Started 3:36 PM UTC |
|
🤖 Retro · Started 3:38 PM UTC |
Retro: PR #1968 —
|
|
🤖 Finished Retro · ✅ Success · Started 3:38 PM UTC · Completed 3:44 PM UTC |
|
Review skipped — this PR is already merged. The Posted by fullsend post-review check |
|
🤖 Finished Review · ✅ Success · Started 3:36 PM UTC · Completed 3:45 PM UTC |
2 participants
Summary
@maininstead of@v0for reusable workflowsv0v0anymore) and made downstream checks self-contained🤖 Generated with Claude Code