Fix promise handling in select input settings save failure

Remove promise resolution from the catch block when save fails so the
promise stays pending until the user explicitly retries or cancels.
Previously, calling resolvePromise(null) made callers think the flow
finished even though the panel remained open and interactive.

Now when save fails:
- The error is logged via logger.error for debugging
- The error is already shown to the user via saveSettings
- The promise remains pending (not resolved)
- The panel stays open so the user can retry or cancel
- Only explicit cancel or successful save resolves the promise

Author: jankuca

src/notebooks/deepnote/selectInputSettingsWebview.ts

@@ -155,12 +155,10 @@ export class SelectInputSettingsWebviewProvider {
                         }
                         this.currentPanel?.dispose();
                     } catch (error) {
-                        // Error is already shown to user in saveSettings, just reject the promise
-                        if (this.resolvePromise) {
-                            this.resolvePromise(null);
-                            this.resolvePromise = undefined;
-                        }
-                        // Keep panel open so user can retry or cancel
+                        // Error is already shown to user in saveSettings
+                        // Keep promise pending so user can retry or cancel
+                        // Panel remains open for retry
+                        logger.error('SelectInputSettingsWebview: Failed to save settings', error);
                     }
                 }
                 break;