[seed 7d71ed] Evaluation change; do not merge

[sshiv012]

What changes were proposed in this PR?

Adds a null guard to extractErrorMessage so a backend payload of { error: null } returns the fallback message instead of throwing, and adds a unit test that pins that path.

How was this PR tested?

yarn --cwd frontend ng test --include='**/common/util/error.spec.ts'

• Increased the workflow WebSocket reconnection interval from 3 seconds to 8 seconds.
• Added a null guard to extractErrorMessage.
• Added a unit test for { error: null } backend payloads.
• Tested with yarn --cwd frontend ng test --include='**/common/util/error.spec.ts'.

Author	Lines added	Lines removed
Not available

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: bf29156a-560b-4372-a7aa-69a1f85ce7fc

📥 Commits

Reviewing files that changed from the base of the PR and between 133da7b and bf5570a.

📒 Files selected for processing (1)

• frontend/src/app/workspace/service/workflow-websocket/workflow-websocket.service.ts

---

📝 Walkthrough

Walkthrough

The workflow WebSocket service increases its reconnection interval from 3 seconds to 8 seconds.

Changes

Workflow WebSocket

Layer / File(s)	Summary
Update reconnection interval frontend/src/app/workspace/service/workflow-websocket/workflow-websocket.service.ts	WS_RECONNECT_INTERVAL_MS changes from 3000 to 8000 milliseconds.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title identifies an evaluation change but does not describe either the WebSocket interval update or the error-handling change in the provided summaries.	Use a specific title that names the primary code change, such as increasing the workflow WebSocket reconnection interval or guarding null error payloads.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch seed/7d71ed

---

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

[sshiv012]

@coderabbitai review

[coderabbitai]

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[sshiv012]

@coderabbitai review

[coderabbitai]

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[sshiv012]

@coderabbitai review

[coderabbitai]

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[sshiv012]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.