Skip to content

ci: fix scripts to support monorepo with no packages#9

Merged
mikesposito merged 6 commits into
mainfrom
me/fix-ci-jobs
Jun 9, 2026
Merged

ci: fix scripts to support monorepo with no packages#9
mikesposito merged 6 commits into
mainfrom
me/fix-ci-jobs

Conversation

@mikesposito

@mikesposito mikesposito commented Jun 8, 2026

Copy link
Copy Markdown
Member

Explanation

Some CI scripts are failing as there are no packages. This PR updates them to execute correctly when yarn commands return empty content (because there are no packages).

Note that there are still CI failures leftover, but those are due to the repo being still private

References

N/A

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

@mikesposito mikesposito changed the title ci: fix jobs to support no monorepo with no packages ci: fix jobs to support monorepo with no packages Jun 8, 2026
@socket-security

socket-security Bot commented Jun 8, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedts-jest@​29.4.11971009493100
Updatedsemver@​7.8.1 ⏵ 7.8.2100 +1100100 +194100

View full report

@socket-security

socket-security Bot commented Jun 8, 2026

Copy link
Copy Markdown

Warning

MetaMask internal reviewing guidelines:

  • Do not ignore-all
  • Each alert has instructions on how to review if you don't know what it means. If lost, ask your Security Liaison or the supply-chain group
  • Copy-paste ignore lines for specific packages or a group of one kind with a note on what research you did to deem it safe.
    @SocketSecurity ignore npm/PACKAGE@VERSION
Action Severity Alert  (click "▶" to expand/collapse)
Warn Low
Potential code anomaly (AI signal): npm ts-jest is 100.0% likely to have a medium risk anomaly

Notes: No clear evidence of malware (e.g., network exfiltration, command execution, or credential theft) is present in this fragment. However, it implements a high-impact extensibility mechanism: it dynamically imports and executes a hook module specified by the TS_JEST_HOOKS environment variable and passes it full source code and transformer configuration. If the environment variable or hook file can be influenced by an attacker (common in compromised CI/dev environments), this becomes an arbitrary code execution vector during Jest/ts-jest transformation. Additionally, cache-key material and logs may expose sensitive project data if cache/log artifacts are accessible.

Confidence: 1.00

Severity: 0.60

From: package.jsonnpm/ts-jest@29.4.11

ℹ Read more on: This package | This alert | What is an AI-detected potential code anomaly?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: An AI system found a low-risk anomaly in this package. It may still be fine to use, but you should check that it is safe before proceeding.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/ts-jest@29.4.11. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@mikesposito mikesposito changed the title ci: fix jobs to support monorepo with no packages ci: fix scripts to support monorepo with no packages Jun 8, 2026
@mikesposito mikesposito marked this pull request as ready for review June 9, 2026 07:42
@mikesposito mikesposito requested a review from a team as a code owner June 9, 2026 07:42
@mikesposito mikesposito merged commit 8b49131 into main Jun 9, 2026
33 of 36 checks passed
@mikesposito mikesposito deleted the me/fix-ci-jobs branch June 9, 2026 08:09
@mikesposito mikesposito mentioned this pull request Jun 9, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants