From 7dd3098af5286d23a769eef3055f63001c3ea44f Mon Sep 17 00:00:00 2001 From: Benjamin Lu Date: Mon, 2 Feb 2026 03:42:07 -0800 Subject: [PATCH] fix: use NodeId for output key parts (#8547) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary Use the shared NodeId type for output asset key construction to keep node id typing consistent across assets and workflow schemas. ## Changes - **What**: replace the local `string | number` nodeId type in output key parts with the shared `NodeId` alias ## Review Focus Type consistency for `nodeId` used in output asset key generation. ## Screenshots (if applicable) N/A ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-8547-fix-use-NodeId-for-output-key-parts-2fb6d73d36508127a022cc6052e5f59e) by [Unito](https://www.unito.io) --- src/platform/assets/utils/outputAssetUtil.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/platform/assets/utils/outputAssetUtil.ts b/src/platform/assets/utils/outputAssetUtil.ts index 540ba3cc8..e23cad988 100644 --- a/src/platform/assets/utils/outputAssetUtil.ts +++ b/src/platform/assets/utils/outputAssetUtil.ts @@ -1,5 +1,6 @@ import type { OutputAssetMetadata } from '@/platform/assets/schemas/assetMetadataSchema' import type { AssetItem } from '@/platform/assets/schemas/assetSchema' +import type { NodeId } from '@/platform/workflow/validation/schemas/workflowSchema' import { getJobDetail, getPreviewableOutputsFromJobDetail @@ -21,7 +22,7 @@ type ResolveOutputAssetItemsOptions = { } type OutputKeyParts = { - nodeId?: string | number | null + nodeId?: NodeId | null subfolder?: string | null filename?: string | null }