diff --git a/.changeset/deprecate-review-gated-coding.md b/.changeset/deprecate-review-gated-coding.md new file mode 100644 index 0000000000..2228e410f5 --- /dev/null +++ b/.changeset/deprecate-review-gated-coding.md @@ -0,0 +1,7 @@ +--- +"@runfusion/fusion": patch +--- + +summary: Retire the Coding (review-gated) workflow, superseded by Coding (Ideas) V2. +category: internal +dev: Adds `builtin:review-gated-coding` to `DEPRECATED_BUILTIN_WORKFLOW_IDS`, the registry's official retirement mechanism: it disappears from new selection and from `toggleEligibleBuiltinWorkflowIds()` while `getBuiltinWorkflow` still resolves it, so tasks that already selected it keep working. It shipped with a success path that could never complete (`code-review -> documentation-delivery` places a write-capable node after a passed review, refused as `workspace-review-seal-required`). diff --git a/packages/core/src/types.ts b/packages/core/src/types.ts index e4eadbc27d..9c481da52a 100644 --- a/packages/core/src/types.ts +++ b/packages/core/src/types.ts @@ -60,8 +60,18 @@ export type { IngestedCheckState, IngestedCheckStateValue, MergeablePrCheck } fr * selection. FN-7970 and FN-7969 preserve direct resolution for pre-existing * Brainstorming and Coding (Ideas) task selections while hiding them elsewhere. */ +/* +FNXC:WorkflowDeprecation 2026-08-24-10:40: +builtin:review-gated-coding is retired in favour of builtin:coding-ideas-v2. It shipped with a +success path that could never complete: `code-review -> documentation-delivery` puts a write-capable +node after a passed review, which `execute-workflow-graph` refuses with +`workspace-review-seal-required`, and its plan node declared a seam `resolveSeamName` throws on. +Deprecated rather than deleted so any task that already selected it still resolves its definition; +it is simply no longer offered for new work. +*/ export const DEPRECATED_BUILTIN_WORKFLOW_IDS: ReadonlySet = new Set([ "builtin:brainstorming", + "builtin:review-gated-coding", ]);