FN-7045: capitalize Code Review step name

Normalize the built-in Code Review workflow label and cover the display-name invariant.

- Rename the compound-engineering built-in code-review node to "Code Review".
- Add regression coverage that rejects lowercase built-in Code review node names.
- Add a patch changeset for the published CLI package.

Files changed:
 .changeset/fn-7045-code-review-step-name.md           | 7 +++++++
 packages/core/src/__tests__/builtin-workflows.test.ts | 9 +++++++++
 packages/core/src/builtin-workflows.ts                | 6 +++++-
 3 files changed, 21 insertions(+), 1 deletion(-)

Fusion-Task-Id: FN-7045

Fusion-Task-Lineage: 4608df38-dc5f-4d16-b33e-9f4ce00fe96d
This commit is contained in:
gsxdsm
2026-06-25 22:10:59 -07:00
parent 429143ecf0
commit 07209a4732
3 changed files with 21 additions and 1 deletions

View File

@@ -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.

View File

@@ -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");

View File

@@ -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",