Skip to content

ci(embed): build workspace deps and fetch dotenv-linter before verify - #14612

Merged
dylanjeffers merged 1 commit into
mainfrom
fix/embed-verify-ci
Sep 24, 2026
Merged

dylanjeffers merged 1 commit into
mainfrom
fix/embed-verify-ci

Conversation

@dylanjeffers

Copy link
Copy Markdown
Contributor

Follow-up to #14611. The jest test now passes in the Embed Deploy job (run 36062074996), but npm run verify in the same step still fails:

  • dotenv-linter: not found: the binary is downloaded by the package's install script into packages/dotenv-linter/bin, which isn't in the node_modules cache, and the job only runs rebuild --ignore-scripts. I've added the same missing-binary check mobile.yml uses.
  • eslint import/no-unresolved for @audius/harmony, @audius/sdk and @audius/fixed-decimal: their dist/ isn't built yet. I've added turbo run build --filter='embed^...', the same step embed.yml runs. build:prod afterwards reuses the turbo cache.

This job only runs on main pushes, so it can't be exercised from the PR.

🤖 Generated with Claude Code

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>
@changeset-bot

changeset-bot Bot commented Sep 24, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: fb00caf

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@socket-security

Copy link
Copy Markdown

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.

@dylanjeffers
dylanjeffers merged commit d68b0ba into main Sep 24, 2026
18 checks passed
@dylanjeffers
dylanjeffers deleted the fix/embed-verify-ci branch September 24, 2026 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant