From 3b86be74204fc9dcfb1c5357adea690b261f5596 Mon Sep 17 00:00:00 2001 From: Q - Workflow Optimizer Date: Mon, 22 Dec 2025 16:35:09 +0000 Subject: [PATCH] fix: upload-assets not processing files from MCP server The upload_assets.cjs script was filtering for both 'upload_assets' and 'upload_asset' types and combining them into allUploadItems, but then only iterating over uploadAssetItems (the legacy singular type). This caused files uploaded via the modern MCP server (which uses 'upload_asset' type) to be saved to artifacts but never published to the orphaned git branch. Fixed by changing both loops to iterate over allUploadItems instead of uploadAssetItems, ensuring both legacy and modern upload types are processed correctly. Fixes #7290 --- pkg/workflow/js/upload_assets.cjs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/workflow/js/upload_assets.cjs b/pkg/workflow/js/upload_assets.cjs index 4d59c17eb0..ee71a03ac3 100644 --- a/pkg/workflow/js/upload_assets.cjs +++ b/pkg/workflow/js/upload_assets.cjs @@ -121,7 +121,7 @@ async function main() { } // Process each asset - for (const asset of uploadAssetItems) { + for (const asset of allUploadItems) { try { const { fileName, sha, size, targetFileName } = asset; @@ -179,7 +179,7 @@ async function main() { core.info(`Successfully uploaded ${uploadCount} assets to branch ${normalizedBranchName}`); } - for (const asset of uploadAssetItems) { + for (const asset of allUploadItems) { if (asset.fileName && asset.sha && asset.size && asset.url) { core.summary.addRaw(`- [\`${asset.fileName}\`](${asset.url}) → \`${asset.targetFileName}\` (${asset.size} bytes)`); }