Skip to content

node-executor: skip npm audit in build_deps - #641

Open
stickerdaniel wants to merge 1 commit into
convex:mainfrom
stickerdaniel:fix/build-deps-no-audit
Open

stickerdaniel wants to merge 1 commit into
convex:mainfrom
stickerdaniel:fix/build-deps-no-audit

Conversation

@stickerdaniel

Copy link
Copy Markdown
Contributor

build_deps runs npm install, and npm also sends the installed dependency tree to the registry's audit endpoint. Nothing reads the audit report here. When that endpoint stalls, the install waits for it, and on a self-hosted backend this ran pushes into the 300 s evaluate_push timeout. --no-audit skips only the audit request.

I ran both commands against a local registry proxy that never answers the audit request. npm install was still waiting after 60 s, and npm install --no-audit finished in 2 s with the same packages installed.

Implemented with Claude Opus 5.5 (xhigh) in Claude Code via T3 Code.


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copilot AI balanced review requested due to automatic review settings November 11, 2026 04:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

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