From 0664c5293602631d088e027aa0ef2a5e0335b5b8 Mon Sep 17 00:00:00 2001 From: Diego Imbert Date: Tue, 2 Jun 2026 13:58:56 +0200 Subject: [PATCH] fix(frontend): FlowMetadata view link targets the job's workspace The "view script/flow" link in FlowMetadata still used flowPathToHref (which resolves the active workspace) for flows and a bare /scripts/get for scripts, so for a foreign-workspace job it pointed at the active workspace. Make it hub-aware and use the job's workspace (job.workspace_id ?? $workspaceStore), matching JobDetailHeader. Hub paths stay workspace-free. Co-Authored-By: Claude Opus 4.8 (1M context) --- frontend/src/lib/components/FlowMetadata.svelte | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/frontend/src/lib/components/FlowMetadata.svelte b/frontend/src/lib/components/FlowMetadata.svelte index 3728ca7ffb..a90c787d62 100644 --- a/frontend/src/lib/components/FlowMetadata.svelte +++ b/frontend/src/lib/components/FlowMetadata.svelte @@ -97,11 +97,15 @@ {/if} {#if (job && job.job_kind == 'flow') || job?.job_kind == 'script'} - {@const stem = `${job?.job_kind}s`} {@const isScript = job?.job_kind === 'script'} - {@const viewHref = isScript - ? `${base}/${stem}/get/${job?.script_hash}` - : flowPathToHref(job?.script_path ?? '')} + {@const jobWorkspace = job?.workspace_id ?? $workspaceStore} + {@const viewHref = (job?.script_path ?? '').startsWith('hub/') + ? isScript + ? `${base}/scripts/get/${job?.script_hash}` + : flowPathToHref(job?.script_path ?? '') + : isScript + ? `${base}/scripts/get/${job?.script_hash}?workspace=${jobWorkspace}` + : `${base}/flows/get/${job?.script_path}?workspace=${jobWorkspace}`}
{#if isScript}