Skip to content

Commit b1fdcec

Browse files
authored
Merge pull request #1968 from fullsend-ai/chore/update-cutting-releases-fullsend-ai-main
chore(skills): update cutting-releases for fullsend-ai @main switch
2 parents 5faa79f + 9e42931 commit b1fdcec

2 files changed

Lines changed: 60 additions & 32 deletions

File tree

skills/cutting-releases/post-flight.md

Lines changed: 33 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ Part of the [cutting-releases](SKILL.md) skill.
44

55
Run after the version tag is pushed, the `v0` tag is moved, and the
66
CI workflows complete. Focus on the areas identified during pre-flight
7-
step E.
7+
step F.
88

99
## A. Wait for CI workflows
1010

@@ -29,59 +29,65 @@ gh release view <tag>
2929
Check that the title, changelog, and binary assets look correct.
3030
Verify the release is not marked as a draft.
3131

32-
## C. Check fullsend-ai repos
32+
## C. Skip fullsend-ai repos
3333

34-
The skill user is a fullsend repo admin, so fullsend-ai org repos
35-
are always accessible. Check recent workflow runs in the org's repos
36-
that consume `@v0` reusable workflows:
34+
The `fullsend-ai/.fullsend` repo references reusable workflows via
35+
`@main`, not `@v0`. Its runs do **not** exercise the `v0` tag and
36+
cannot confirm that the tag move worked. (Those runs are checked
37+
during pre-flight instead, as a signal that `main` is healthy.)
38+
39+
Skip fullsend-ai for post-flight `v0` verification. Focus on other
40+
downstream consumers in step D.
41+
42+
## D. Check additional downstream repos (optional)
43+
44+
Use `AskUserQuestion` to ask if the user has access to additional
45+
downstream orgs:
46+
47+
> Do you have access to any other downstream orgs/repos to verify?
48+
> (e.g. "konflux-ci, redhat-developer/rhdh-agentic")
49+
> Leave blank to skip.
50+
51+
For each repo provided, check recent workflow runs that started
52+
**after** the `v0` tag move:
3753

3854
```
39-
gh run list --repo fullsend-ai/fullsend --limit=3
40-
gh run list --repo fullsend-ai/.fullsend --limit=3
55+
gh run list --repo <org/repo> --limit=5
4156
```
4257

43-
Look for runs that started **after** the `v0` tag move. Confirm they
44-
completed without workflow-resolution errors (e.g. "could not find
45-
reusable workflow"). If no runs occurred naturally, check for any
46-
recent failed or cancelled runs that can be retriggered:
58+
Confirm they completed without workflow-resolution errors (e.g.
59+
"could not find reusable workflow"). If no runs occurred naturally,
60+
check for recent failed runs that can be retriggered:
4761

4862
```
49-
gh run list --repo fullsend-ai/.fullsend --status=failure --limit=3
63+
gh run list --repo <org/repo> --status=failure --limit=3
5064
```
5165

5266
Present any candidate to the user for confirmation before retriggering:
5367

54-
> I found run `<run-id>` (failed) in `fullsend-ai/.fullsend`.
68+
> I found run `<run-id>` (failed) in `<org/repo>`.
5569
> Retrigger it to verify `@v0` resolves?
5670
5771
Once confirmed:
5872

5973
```
60-
gh run rerun <run-id> --failed --repo fullsend-ai/.fullsend
74+
gh run rerun <run-id> --failed --repo <org/repo>
6175
```
6276

63-
## D. Check additional downstream repos (optional)
64-
65-
Use `AskUserQuestion` to ask if the user has access to additional
66-
downstream orgs:
67-
68-
> Do you have access to any other downstream orgs/repos to verify?
69-
> (e.g. "konflux-ci, redhat-developer/rhdh-agentic")
70-
> Leave blank to skip.
71-
72-
If the user provides repos, repeat the same checks from step C for
73-
each one. If blank, skip this step — not all admins have access to
74-
every enrolled org.
77+
If blank, skip this step — not all admins have access to every
78+
enrolled org.
7579

7680
## E. Present post-flight summary
7781

7882
Summarize results to the user:
7983

8084
| Org/Repo | `@v0` Refs | Status |
8185
|----------|-----------|--------|
82-
| fullsend-ai/.fullsend | Confirmed | Passing |
8386
| ... | ... | ... |
8487

88+
Note: `fullsend-ai` repos are excluded from this table — they use
89+
`@main` and were checked during pre-flight.
90+
8591
Distinguish between:
8692
- **Release-related failures** — workflow resolution errors, missing
8793
secrets, or permission failures caused by the tag move.

skills/cutting-releases/pre-flight.md

Lines changed: 27 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -102,21 +102,43 @@ gh run list --branch=main --limit=5
102102
All recent runs should be passing. If E2E tests are failing, investigate
103103
before releasing.
104104

105-
## E. Identify post-flight check areas
105+
## E. Check fullsend-ai runs as a main-branch signal
106+
107+
The `fullsend-ai/.fullsend` repo references reusable workflows via
108+
`@main` (not `@v0`). This means its recent runs already exercise the
109+
exact commit we are about to tag. Use this as a pre-flight signal:
110+
111+
```
112+
gh run list --repo fullsend-ai/.fullsend --limit=5
113+
gh run list --repo fullsend-ai/fullsend --limit=5
114+
```
115+
116+
If recent runs are passing, that is evidence the reusable workflows on
117+
`main` work correctly and moving `v0` to this commit is low-risk.
118+
119+
If recent runs are failing, investigate whether the failures are related
120+
to reusable workflow changes. Workflow-resolution errors or permission
121+
failures are blockers. Unrelated agent runtime errors are not.
122+
123+
## F. Identify post-flight check areas
106124

107125
Based on the changes found in steps A–C, determine what needs
108126
post-flight verification after the `v0` tag moves:
109127

110-
- **Reusable workflow changes** → verify workflow runs in fullsend-ai
111-
repos resolve `@v0` correctly and pass.
128+
- **Reusable workflow changes** → verify workflow runs in downstream
129+
`@v0` consumers (outside fullsend-ai) resolve correctly and pass.
112130
- **New secrets or permissions** → verify affected workflows don't
113131
fail on missing secrets.
114132
- **CLI default changes** → note migration steps for existing
115133
installs in the release summary.
116134
- **No reusable workflow changes** → post-flight can be limited to
117135
confirming the release artifacts built correctly.
118136

119-
## F. Present summary
137+
Note: `fullsend-ai/.fullsend` uses `@main`, not `@v0`, so its runs
138+
are not useful for post-flight `v0` verification. Post-flight checks
139+
should focus on other downstream consumers that reference `@v0`.
140+
141+
## G. Present summary
120142

121143
Summarize findings to the user in a table:
122144

@@ -126,6 +148,6 @@ Summarize findings to the user in a table:
126148
| Scaffold templates | ... | No/Yes |
127149
| CLI / internal | ... | No/Yes |
128150

129-
List the post-flight check areas identified in step E.
151+
List the post-flight check areas identified in step F.
130152

131153
Give a **GO / NO-GO** verdict. Do not proceed until the user confirms.

0 commit comments

Comments
 (0)