Skip to content

fix(drive-integration): add MISSING_PARAMETER failure reason [INTEG-4369]#11068

Open
JuliRossi wants to merge 1 commit into
masterfrom
feat/drive-integration-missing-parameter-failure-reason
Open

fix(drive-integration): add MISSING_PARAMETER failure reason [INTEG-4369]#11068
JuliRossi wants to merge 1 commit into
masterfrom
feat/drive-integration-missing-parameter-failure-reason

Conversation

@JuliRossi

@JuliRossi JuliRossi commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

INTEG-4369

Purpose

The MISSING_PARAMETER failure reason was added to WorkflowFailureReason but never wired up — it had no error message, no entry in the failure reason message map, and no handler in ModalOrchestrator, so it would silently fall through to the generic error. This PR completes the implementation.

Summary

  • Fixed typo MISING_PARAMETERMISSING_PARAMETER in the enum
  • Added MISSING_PARAMETER error message to ERROR_MESSAGES
  • Wired MISSING_PARAMETER into FAILURE_REASON_MESSAGES in useWorkflowAgent
  • Added handler in ModalOrchestrator.showWorkflowError so the error surfaces with a specific title and message instead of falling through to generic

Test plan

  • Trigger a workflow failure with code: 'missing-parameter' from the backend and verify the "Missing parameter" error modal appears (not the generic error)
  • Verify all other failure reason error modals are unaffected
  • TypeScript compiles clean (tsc --noEmit)

Generated with Claude Code

…l error handling [INTEG-4369]

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@JuliRossi
JuliRossi requested review from a team as code owners July 13, 2026 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants