fix: address code review - simplify fallback logic and add error handling

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/plugin-detail/src/DetailView.tsx

@@ -95,25 +95,17 @@ export const DetailView: React.FC<DetailViewProps> = ({
           setLoading(false);
           return;
         }
-        // Fallback: if not found and resourceId starts with objectName prefix,
-        // retry with the prefix stripped (handles legacy ID conventions)
-        if (resourceId.startsWith(prefix)) {
-          const strippedId = resourceId.slice(prefix.length);
-          return dataSource.findOne(objectName, strippedId).then((fallbackResult) => {
-            setData(fallbackResult);
-            setLoading(false);
-          });
-        }
-        // Fallback: if not found and resourceId does NOT have the prefix,
-        // retry with the prefix prepended (handles bare IDs)
-        if (!resourceId.startsWith(prefix)) {
-          return dataSource.findOne(objectName, `${prefix}${resourceId}`).then((fallbackResult) => {
-            setData(fallbackResult);
-            setLoading(false);
-          });
-        }
-        setData(null);
-        setLoading(false);
+        // Fallback: try alternate ID format for backward compatibility
+        const altId = resourceId.startsWith(prefix)
+          ? resourceId.slice(prefix.length)   // strip prefix
+          : `${prefix}${resourceId}`;          // prepend prefix
+        return dataSource.findOne(objectName, altId).then((fallbackResult) => {
+          setData(fallbackResult);
+          setLoading(false);
+        }).catch(() => {
+          setData(null);
+          setLoading(false);
+        });
       }).catch((err) => {
          console.error('Failed to fetch detail data:', err);
          setLoading(false);