ci(embed): build workspace deps and fetch dotenv-linter before verify - #14612
Conversation
On a node_modules cache hit the dotenv-linter binary is missing, and eslint can't resolve @audius/* until their dists are built. Mirrors embed.yml and mobile.yml. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
|
Dependency limit exceeded — report not shown. This pull request scan exceeded the 10,000-dependency limit applied to this scan, so the results are incomplete and may be inaccurate. To avoid reporting false positives, Socket has not posted a report. Upgrade your plan to raise the dependency limit and get complete reports, or view the partial scan in the dashboard. Socket is always free for open source. If this is a non-commercial open source project, contact us to request a free Team account. |
Follow-up to #14611. The jest test now passes in the Embed Deploy job (run 36062074996), but
npm run verifyin the same step still fails:dotenv-linter: not found: the binary is downloaded by the package'sinstallscript intopackages/dotenv-linter/bin, which isn't in the node_modules cache, and the job only runsrebuild --ignore-scripts. I've added the same missing-binary checkmobile.ymluses.import/no-unresolvedfor@audius/harmony,@audius/sdkand@audius/fixed-decimal: theirdist/isn't built yet. I've addedturbo run build --filter='embed^...', the same stepembed.ymlruns.build:prodafterwards reuses the turbo cache.This job only runs on main pushes, so it can't be exercised from the PR.
🤖 Generated with Claude Code