[#noissue] Address PR review: fix type mismatch, broken link, and stack loss

- Change contentOption type from PopoverContentProps to DialogContent props
- Remove unsupported collisionPadding prop from DialogContent
- Conditionally render <a> only when url is present for instance
- Preserve original Error stack/cause in ThrowError by rethrowing

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: jihea-park

web-frontend/src/main/v3/packages/ui/src/components/Error/ErrorDetailDialog.tsx

@@ -9,15 +9,14 @@ import {
   DialogTrigger,
   Separator,
 } from '../ui';
-import * as PopoverPrimitive from '@radix-ui/react-popover';
 import { ErrorLike } from '@pinpoint-fe/ui/src/constants';
 import { cn } from '../../lib';
 import { HighLightCode } from '../HighLightCode';
 import { RxChevronDown, RxChevronUp } from 'react-icons/rx';
 
 export interface ErrorDetailDialogProps {
   error: Error | ErrorLike;
-  contentOption?: PopoverPrimitive.PopoverContentProps;
+  contentOption?: React.ComponentPropsWithoutRef<typeof DialogContent>;
   contentClassName?: string;
 }
 
@@ -62,7 +61,6 @@ export const ErrorDetailDialog = ({
       </DialogTrigger>
       <DialogContent
         className={cn('max-h-[90%] overflow-auto max-w-5xl', contentClassName)}
-        collisionPadding={16}
         onMouseDown={(e) => e.stopPropagation()}
         {...contentOption}
       >
@@ -75,14 +73,20 @@ export const ErrorDetailDialog = ({
 
             {serverError?.instance && (
               <div className="flex gap-1 items-center">
-                <a
-                  className="text-sm font-semibold text-primary hover:underline"
-                  href={serverError?.url}
-                  target="_blank"
-                  rel="noreferrer"
-                >
-                  {serverError.instance}
-                </a>
+                {serverError?.url ? (
+                  <a
+                    className="text-sm font-semibold text-primary hover:underline"
+                    href={serverError.url}
+                    target="_blank"
+                    rel="noreferrer"
+                  >
+                    {serverError.instance}
+                  </a>
+                ) : (
+                  <span className="text-sm font-semibold">
+                    {serverError.instance}
+                  </span>
+                )}
               </div>
             )}
             {hasMethod && (

web-frontend/src/main/v3/packages/ui/src/components/Error/ThrowError.tsx

@@ -1,10 +1,16 @@
 import { ErrorLike } from '@pinpoint-fe/ui/src/constants';
 
 export const ThrowError = ({ error }: { error: Error | ErrorLike }) => {
-  const err = new Error(
-    (error as ErrorLike).detail || (error as ErrorLike).title || 'An error occurred.',
-  ) as Error;
+  const detail = (error as ErrorLike).detail;
+  const title = (error as ErrorLike).title;
+  const normalizedMessage = detail || title || (error instanceof Error ? error.message : 'An error occurred.');
+
+  if (error instanceof Error) {
+    error.message = normalizedMessage;
+    throw error;
+  }
+
+  const err = new Error(normalizedMessage);
   Object.assign(err, error);
-  err.message = (error as ErrorLike).detail || (error as ErrorLike).title || err.message;
   throw err;
 };