feat(pipeline): Add completion view support to pipeline definitions

[evanpurkhiser]

Add a completionView property to PipelineDefinition that allows pipelines to render a custom view after completion before firing onComplete and closing the modal. When set, onComplete is deferred until the component calls finish(). When null, existing behavior is preserved.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 585549a. Configure here.}

[cursor]

finish lacks guard against repeated onComplete invocations

Medium Severity

The finish callback has no guard to prevent multiple invocations. Each call fires onCompleteRef.current, so a rapid double-click on the "Done" button fires onComplete twice. Unlike advance, which exposes isPending so step components can disable their buttons, PipelineCompletionProps provides no equivalent mechanism for completion views to prevent repeated submission. In the modal, each finish() call triggers handleComplete, which calls both the consumer's onComplete and closeModal(), potentially causing duplicate side effects before the modal unmounts.

Additional Locations (1)

• static/app/components/pipeline/pipelineDummyProvider.tsx#L69-L82

 

^{Reviewed by Cursor Bugbot for commit 585549a. Configure here.}