diff --git a/.changeset/fn-7045-code-review-step-name.md b/.changeset/fn-7045-code-review-step-name.md new file mode 100644 index 0000000000..554a1a46e1 --- /dev/null +++ b/.changeset/fn-7045-code-review-step-name.md @@ -0,0 +1,7 @@ +--- +"@runfusion/fusion": patch +--- + +summary: Capitalize the built-in Code Review step name consistently. +category: fix +dev: Updates the compound-engineering built-in workflow node display name and regression coverage. diff --git a/packages/core/src/__tests__/builtin-workflows.test.ts b/packages/core/src/__tests__/builtin-workflows.test.ts index 87f3962dcc..7ecf0007a7 100644 --- a/packages/core/src/__tests__/builtin-workflows.test.ts +++ b/packages/core/src/__tests__/builtin-workflows.test.ts @@ -39,6 +39,14 @@ describe("built-in workflows", () => { } }); + it("does not expose lowercase Code review step names in built-in workflow nodes", () => { + for (const workflow of BUILTIN_WORKFLOWS) { + for (const node of workflow.ir.nodes) { + expect(node.config?.name, `${workflow.id}:${node.id}`).not.toBe("Code review"); + } + } + }); + it("includes the stepwise coding built-in modeling step inversion (KTD-9)", () => { const stepwise = getBuiltinWorkflow("builtin:stepwise-coding"); expect(stepwise).toBeDefined(); @@ -459,6 +467,7 @@ describe("built-in workflows", () => { const codeReview = byId("code-review"); expect(codeReview?.kind).toBe("gate"); + expect(codeReview?.config?.name).toBe("Code Review"); expect(codeReview?.config?.skillName).toBe("compound-engineering:ce-code-review"); expect(codeReview?.config?.gateMode).toBe("gate"); expect(codeReview?.config?.toolMode).toBe("coding"); diff --git a/packages/core/src/builtin-workflows.ts b/packages/core/src/builtin-workflows.ts index 5b32fafe80..d0e2844c49 100644 --- a/packages/core/src/builtin-workflows.ts +++ b/packages/core/src/builtin-workflows.ts @@ -216,7 +216,11 @@ export const BUILTIN_WORKFLOWS: WorkflowDefinition[] = [ id: "code-review", kind: "gate", config: { - name: "Code review", + /* + * FNXC:Workflows 2026-06-25-00:00: + * FN-7045 requires every built-in code-review step display name to be title-case "Code Review" so compound-engineering matches WORKFLOW_STEP_TEMPLATES and the optional Code Review group. + */ + name: "Code Review", executor: "skill", skillName: "compound-engineering:ce-code-review", gateMode: "gate",