PM-2179 update logic

Author: himaniraghav3

src/apps/review/src/lib/components/Scorecard/ScorecardAttachments/ScorecardAttachments.module.scss

@@ -25,10 +25,19 @@
 
 .expired {
   cursor: not-allowed;
-  opacity: 0.4;
   color: #999;
 
   &:hover {
     text-decoration: none;
   }
+}
+
+.artifactType {
+    display: flex;
+    gap: 5px;
+    align-items: center; 
+
+    .artifactIcon {
+        stroke: #00797A;
+    }
 }

src/apps/review/src/lib/components/Scorecard/ScorecardAttachments/ScorecardAttachments.tsx

@@ -45,8 +45,6 @@ const ScorecardAttachments: FC<ScorecardAttachmentsProps> = (props: ScorecardAtt
         [handleDownload],
     )
 
-    console.log('attachments', artifacts)
-
     const columns = useMemo<TableColumn<AiWorkflowRunArtifact>[]>(
         () => [
             {
@@ -68,7 +66,7 @@ const ScorecardAttachments: FC<ScorecardAttachmentsProps> = (props: ScorecardAtt
                             onClick={!isExpired ? createDownloadHandler(attachment.id) : undefined}
                         >
                             <span>{attachment.name}</span>
-                            {isExpired && <span>Expired</span>}
+                            {isExpired && <span>(Link Expired)</span>}
                         </div>
                     )
                 },
@@ -79,7 +77,7 @@ const ScorecardAttachments: FC<ScorecardAttachmentsProps> = (props: ScorecardAtt
                 label: 'Type',
                 renderer: () => (
                     <div className={styles.artifactType}>
-                        <IconOutline.CubeIcon />
+                        <IconOutline.CubeIcon className={styles.artifactIcon} width={26} />
                         <span>Artifact</span>
                     </div>
                 ),

src/apps/review/src/lib/hooks/useFetchAiWorkflowRuns.ts

@@ -2,7 +2,7 @@ import { useEffect, useState } from 'react'
 import useSWR, { SWRResponse } from 'swr'
 
 import { EnvironmentConfig } from '~/config'
-import { xhrGetAsync } from '~/libs/core'
+import { xhrGetAsync, xhrGetBlobAsync } from '~/libs/core'
 import { handleError } from '~/libs/shared/lib/utils/handle-error'
 
 import { AiFeedbackItem, Scorecard } from '../models'
@@ -216,26 +216,22 @@ export function useDownloadAiWorkflowsRunArtifact(
 
             const url = `${TC_API_BASE_URL}/workflows/${workflowId}/runs/${runId}/attachments/${artifactId}/zip`
 
-            const response = await fetch(url, {
-                credentials: 'include',
-                method: 'GET',
-            })
-
-            if (!response.ok) {
-                throw new Error(`Download failed with status ${response.status}`)
-            }
-
-            const blob = await response.blob()
-
-            // Create a blob URL and trigger browser download
-            const objectUrl = window.URL.createObjectURL(blob)
-            const link = document.createElement('a')
-            link.href = objectUrl
-            link.download = `artifact-${artifactId}.zip`
-            link.click()
-
-            // Cleanup
-            window.URL.revokeObjectURL(objectUrl)
+            xhrGetBlobAsync<Blob>(url)
+                .then(blob => {
+                    const objectUrl = window.URL.createObjectURL(blob)
+                    const link = document.createElement('a')
+                    link.href = objectUrl
+                    link.download = `artifact-${artifactId}.zip`
+
+                    document.body.appendChild(link)
+                    link.click()
+                    link.remove()
+
+                    window.URL.revokeObjectURL(objectUrl)
+                })
+                .catch(err => {
+                    handleError(err as Error)
+                })
         } catch (err) {
             handleError(err as Error)
         } finally {