fix(api-service): Workflow schema duplication fixes NV-7040

[scopsy]

What changed? Why was the change needed?

This PR resolves NV-7040, where the workflow schema (payloadSchema, validatePayload, and severity) was not being duplicated when a workflow was copied.

The duplicate-workflow.usecase.ts has been updated to include these missing fields during the duplication process. A new e2e test case has also been added to workflow.controller.e2e.ts to ensure the correct duplication of these properties.

Screenshots

N/A

Expand for optional sections

Related enterprise PR

N/A

Special notes for your reviewer

The approach for duplicating these fields aligns with the existing logic in sync-to-environment.usecase.ts.

---

Linear Issue: NV-7040

 

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}

[linear]

NV-7040 Workflow schema not duplicated when duplicating workflow

[netlify]

✅ Deploy Preview for dashboard-v2-novu-staging canceled.

Name	Link
🔨 Latest commit	250040d
🔍 Latest deploy log	https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/696738e352cad20008216622

[coderabbitai]

Walkthrough

This pull request extends the workflow duplication feature to preserve three additional fields when duplicating workflows. The DuplicateWorkflowUseCase now copies payloadSchema, validatePayload, and severity from the origin workflow to the duplicated workflow, with payloadSchema defaulting to null if not present. Corresponding end-to-end tests are included to verify these fields are correctly preserved during the duplication process.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description provides relevant context about the change, references the resolved issue, explains what was fixed, and describes the implementation approach.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title clearly and specifically describes the main change: fixing workflow schema duplication during workflow copying, with the referenced issue ticket NV-7040.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}