Skip to content

Conversation

@github-actions
Copy link
Contributor

This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.

@vercel
Copy link

vercel bot commented Dec 12, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
app (staging) Skipped Skipped Dec 12, 2025 0:23am
portal (staging) Skipped Skipped Dec 12, 2025 0:23am

@cursor
Copy link

cursor bot commented Dec 12, 2025

PR Summary

Updates buildspec to detect and package either dist/apps/api/src or dist/src outputs, ensuring main.js is found and copied correctly.

  • CI/CD (buildspec)
    • Build output detection: List and verify main.js in either dist/apps/api/src or dist/src.
    • Packaging logic: Conditionally copy build artifacts from dist/apps/api/* or dist/* into ../docker-build/ based on detected structure.

Written by Cursor Bugbot for commit 89188e4. This will update automatically on new commits. Configure here.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Marfuen
❌ github-actions[bot]
You have signed the CLA already but the status is still pending? Let us recheck it.

@graphite-app graphite-app bot requested a review from Marfuen December 12, 2025 00:23
@graphite-app
Copy link

graphite-app bot commented Dec 12, 2025

Graphite Automations

"Auto-assign PRs to Author" took an action on this PR • (12/12/25)

1 reviewer was added to this PR based on Mariano Fuentes's automation.

else
echo "Using standard output structure..."
cp -r dist/* ../docker-build/
fi
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: File vs directory check mismatch causes wrong copy path

The validation on line 54 checks for file existence using [ -f "dist/apps/api/src/main.js" ] with OR logic, while the copy decision on line 59 checks for directory existence using [ -d "dist/apps/api/src" ]. If the directory exists but is empty or lacks main.js, while dist/src/main.js exists in the standard location, validation passes but the copy selects the wrong source path. The directory check at line 59 needs to match the file-based check logic, such as [ -f "dist/apps/api/src/main.js" ].

Additional Locations (1)

Fix in Cursor Fix in Web

@Marfuen Marfuen merged commit fe84bf9 into release Dec 12, 2025
14 of 16 checks passed
@claudfuen
Copy link
Contributor

🎉 This PR is included in version 1.71.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants