improvement(blocks): two-tier sunset schema (legacy amber / deprecated red)#5793
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Legacy (amber legacy badge) means superseded but still supported: requires a resolvable All existing superseded block configs move to Reviewed by Cursor Bugbot for commit 0240e68. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThis PR splits block sunset handling into legacy and deprecated tiers. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (2): Last reviewed commit: "fix(blocks): require replacedBy on legac..." | Re-trigger Greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit bbe942c. Configure here.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 0240e68. Configure here.
| tooltip: 'This block is no longer supported. Click to replace', | ||
| prompt: target | ||
| ? `The "${name}" block is no longer supported. ${migrationPrompt(name, target)}` | ||
| : `The "${name}" block is no longer supported and has no direct successor. Replace it with current blocks that achieve the same result and rewire the connections.`, |
There was a problem hiding this comment.
Missing model tip on deprecated path
Low Severity
The deprecated sunset branch with a resolvable successor drops the model migration tip that the legacy branch still appends when the block has a model subblock. Click-to-fix prompts for red-tier blocks that both name a successor and expose a model field will omit guidance to switch to a current model.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 0240e68. Configure here.


Summary
Follow-up to #5785. Splits block deprecation into two tiers.
deprecatedblock field tosunset: { status: 'legacy' | 'deprecated', replacedBy? }sunsetvision/vision_v2annotations (no successor → unbadged)Models still map to the legacy tier for now; a red model tier (API unavailable) is a separate follow-up.
Type of Change
Testing
Tested manually.
bun run lint:check,check:api-validation:strict, thesunsetregistry invariant, and renderer + apptscall pass.Checklist