fix: refresh extension versions on delete conflict

Author: gnugomez

webui/CHANGELOG.md

@@ -13,6 +13,10 @@ This change log covers only the frontend library (webui) of Open VSX.
 - Migrate admin dashboard to use `@tanstack/react-query` ([#1917](https://github.com/eclipse-openvsx/openvsx/pull/1917)
 - Replace formatting from `stylistic` with `prettier` ([#1916](https://github.com/eclipse-openvsx/openvsx/pull/1916))
 
+### Fixed
+
+- Refresh the extension version list in the delete views when a delete fails with a conflict
+
 ## [v1.0.2] (23/06/2026)
 
 ### Changed

webui/src/context.ts

@@ -17,7 +17,7 @@ import { ErrorResponse } from './server-request';
 export interface MainContext {
     service: ExtensionRegistryService;
     pageSettings: PageSettings;
-    handleError: (err: Error | Partial<ErrorResponse>) => void;
+    handleError: (err: Error | Partial<ErrorResponse>, options?: { onClose?: () => void }) => void;
     user?: UserData;
     updateUser: () => void;
     loginProviders?: Record<string, string>;

webui/src/main.tsx

@@ -37,6 +37,8 @@ export const Main: FunctionComponent<MainProps> = props => {
     const [error, setError] = useState<{ message: string; code?: number | string }>();
     const [isErrorDialogOpen, setIsErrorDialogOpen] = useState<boolean>(false);
     const abortController = useRef<AbortController>(new AbortController());
+    // Optional callback to run when the error dialog is dismissed (e.g. re-fetch stale data).
+    const errorDialogOnClose = useRef<(() => void) | undefined>(undefined);
 
     useEffect(() => {
         getLoginProviders();
@@ -83,20 +85,24 @@ export const Main: FunctionComponent<MainProps> = props => {
         }
     };
 
-    const onError = (err: Error | Partial<ErrorResponse> | ReportedError) => {
+    const onError = (err: Error | Partial<ErrorResponse> | ReportedError, options?: { onClose?: () => void }) => {
         if (err instanceof DOMException && err.message.trim() === 'The operation was aborted.') {
             // ignore error caused by AbortController.abort()
             return;
         }
 
         const message = handleError(err);
         const code = (err as ReportedError).code;
+        errorDialogOnClose.current = options?.onClose;
         setError({ message, code });
         setIsErrorDialogOpen(true);
     };
 
     const onErrorDialogClose = () => {
         setIsErrorDialogOpen(false);
+        const onClose = errorDialogOnClose.current;
+        errorDialogOnClose.current = undefined;
+        onClose?.();
     };
 
     const renderPageContent = (): ReactNode => {

webui/src/pages/admin-dashboard/extension-admin.tsx

@@ -82,15 +82,20 @@ export const ExtensionAdmin: FunctionComponent = () => {
             return;
         }
 
-        await deleteExtension({
-            namespace: extension.namespace,
-            extension: extension.name,
-            targetPlatformVersions: targetPlatformVersions?.map(({ version, targetPlatform }) => ({
-                version,
-                targetPlatform
-            }))
-        });
-        await refetch();
+        try {
+            await deleteExtension({
+                namespace: extension.namespace,
+                extension: extension.name,
+                targetPlatformVersions: targetPlatformVersions?.map(({ version, targetPlatform }) => ({
+                    version,
+                    targetPlatform
+                }))
+            });
+            await refetch();
+        } catch (err) {
+            const isConflict = (err as { status?: number }).status === 409;
+            handleError(err, isConflict ? { onClose: () => refetch() } : undefined);
+        }
     };
 
     return (

webui/src/pages/admin-dashboard/extension-remove-dialog.tsx

@@ -37,11 +37,13 @@ export const ExtensionRemoveDialog: FunctionComponent<ExtensionRemoveDialogProps
         try {
             setWorking(true);
             await props.onRemove(props.targetPlatformVersions);
-            setDialogOpen(false);
         } catch (err) {
+            // onRemove is expected to handle its own errors; this is a defensive fallback so an
+            // unexpected rejection still surfaces to the user rather than being swallowed.
             handleError(err);
         } finally {
             setWorking(false);
+            setDialogOpen(false);
         }
     };
 

webui/src/pages/user/user-settings-delete-extension.tsx

@@ -63,15 +63,20 @@ export const UserSettingsDeleteExtension: FunctionComponent<UserSettingsDeleteEx
             return;
         }
 
-        await service.deleteExtensions(abortController.current, {
-            namespace: extension.namespace,
-            extension: extension.name,
-            targetPlatformVersions: targetPlatformVersions?.map(({ version, targetPlatform }) => ({
-                version,
-                targetPlatform
-            }))
-        });
-        await findExtension();
+        try {
+            await service.deleteExtensions(abortController.current, {
+                namespace: extension.namespace,
+                extension: extension.name,
+                targetPlatformVersions: targetPlatformVersions?.map(({ version, targetPlatform }) => ({
+                    version,
+                    targetPlatform
+                }))
+            });
+            await findExtension();
+        } catch (err) {
+            const isConflict = (err as { status?: number }).status === 409;
+            handleError(err, isConflict ? { onClose: () => findExtension() } : undefined);
+        }
     };
 
     return (