fix(ui): show artifact URI instead of transfer job destination URI in version details (#2554)

* fix(ui): show artifact URI instead of transfer job destination URI in version details

The StorageLocationSection was displaying the URI from the transfer job's
destination field, which stores bare OCI image references without the oci://
scheme prefix. The registered model artifact has the correct URI with the
prefix. Use the artifact URI for display instead.

Assisted-by: Claude Code (claude-opus-4-6)
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>

* test(ui): update storage location URI assertion to expect oci:// prefix

Assisted-by: Claude Code (claude-opus-4-6)
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>

---------

Signed-off-by: Mike Turley <mike.turley@alum.cs.umass.edu>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: mturley

clients/ui/frontend/src/__tests__/cypress/cypress/tests/mocked/modelRegistry/modelVersionDetails.cy.ts

@@ -620,7 +620,7 @@ describe('Model version details', () => {
 
         modelVersionDetails
           .findStorageLocationUri()
-          .should('contain.text', 'quay.io/my-org/my-model:v1.0.0');
+          .should('contain.text', 'oci://quay.io/my-org/my-model:v1.0.0');
       });
 
       it('should show expandable source details with origin location', () => {

clients/ui/frontend/src/app/pages/modelRegistry/screens/ModelVersionDetails/ModelVersionDetailsView.tsx

@@ -170,6 +170,7 @@ const ModelVersionDetailsView: React.FC<ModelVersionDetailsViewProps> = ({
                 )}
                 {isTransferJobSource && modelArtifact ? (
                   <StorageLocationSection
+                    artifactUri={modelArtifact.uri}
                     fallbackNamespace={transferJobParams.jobNamespace}
                     transferJob={transferJob}
                     transferJobLoaded={transferJobLoaded}

clients/ui/frontend/src/app/pages/modelRegistry/screens/ModelVersionDetails/StorageLocationSection.tsx

@@ -27,11 +27,11 @@ import {
   getModelUriLabel,
   getModelUriPopoverContent,
   getSourceLabel,
-  getDestinationUri,
   getSourcePath,
 } from '~/app/utils';
 
 type StorageLocationSectionProps = {
+  artifactUri?: string;
   fallbackNamespace: string;
   transferJob: ModelTransferJob | null;
   transferJobLoaded: boolean;
@@ -117,6 +117,7 @@ const StorageLocationTitle: React.FC = () => (
 );
 
 const StorageLocationSection: React.FC<StorageLocationSectionProps> = ({
+  artifactUri,
   fallbackNamespace,
   transferJob,
   transferJobLoaded,
@@ -185,7 +186,6 @@ const StorageLocationSection: React.FC<StorageLocationSectionProps> = ({
   const destType = transferJob.destination.type;
   const sourceType = transferJob.source.type;
   const namespace = transferJob.namespace || '';
-  const destinationUri = getDestinationUri(transferJob);
   const sourcePath = getSourcePath(transferJob);
   const sourceLabel = getSourceLabel(sourceType);
 
@@ -209,13 +209,15 @@ const StorageLocationSection: React.FC<StorageLocationSectionProps> = ({
           <DashboardDescriptionListGroup
             title={getModelUriLabel(destType)}
             popover={getModelUriPopoverContent(destType)}
-            isEmpty={!destinationUri}
+            isEmpty={!artifactUri}
             contentWhenEmpty="No URI"
           >
-            <InlineTruncatedClipboardCopy
-              testId="storage-location-uri"
-              textToCopy={destinationUri}
-            />
+            {artifactUri && (
+              <InlineTruncatedClipboardCopy
+                testId="storage-location-uri"
+                textToCopy={artifactUri}
+              />
+            )}
           </DashboardDescriptionListGroup>
         </DescriptionList>
       </StackItem>