Skip to content

Commit a36c01b

Browse files
committed
docs(skills): require a GA target before adding sunset replacedBy
Review finding. The add-integration cutover steps said to add `sunset.replacedBy` without stating the precondition, so an author following them while v2 was still preview-gated would get a `check-block-registry` failure — the exact constraint this cutover hit. The sibling add-block and add-block-preview skills already spell this out; this brings the third in line.
1 parent 966d3ac commit a36c01b

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

‎.agents/skills/add-integration/SKILL.md‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -601,7 +601,12 @@ If creating V2 versions (API-aligned outputs):
601601
3. **V1 Block** - Add `(Legacy)` to name, set `hideFromToolbar: true`, and add
602602
`sunset: { status: 'legacy', replacedBy: '{service}_v2' }` — `check-block-registry`
603603
fails a legacy block with no `replacedBy`, and the amber legacy badge plus its
604-
click-to-upgrade action read from that field
604+
click-to-upgrade action read from that field.
605+
606+
**Only add `replacedBy` once the target is GA.** The same check also fails when
607+
the target is unregistered, itself sunset, or still `preview: true`. If v2 is
608+
preview-gated, leave v1 alone until GA and drop `preview` in the *same commit*
609+
that adds the sunset — splitting them breaks the build in between.
605610
4. **Registry** - Register both versions
606611

607612
```typescript

0 commit comments

Comments
 (0)