Skip to content

tools: avoid repeated subtree scans in must-call-assert - #66408

Open
inoway46 wants to merge 1 commit into
nodejs:mainfrom
inoway46:optimize-must-call-assert
Open

inoway46 wants to merge 1 commit into
nodejs:mainfrom
inoway46:optimize-must-call-assert

Conversation

@inoway46

@inoway46 inoway46 commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Match the .then() / mustCall() call chain before evaluating
:has(ReturnStatement), avoiding subtree scans for unrelated nodes
while preserving the existing lint checks.

Across 256 ubuntu-slim trials per version, the average duration of the
lint-js-and-md job from .github/workflows/linters.yml decreased
from 9m03s to 7m32s.

Runner Success rate Avg Min P90 P99 Max
ubuntu-slim (before) 243/256 (94.9%) 9m 03s 6m 15s 12m 41s 16m 17s 17m 41s
ubuntu-slim (after) 246/256 (96.1%) 7m 32s 5m 28s 10m 02s 16m 12s 17m 25s

Runs (64 combined-slim trials per batch; only this job is included in the table):

Refs: #64972

Match the promise call chain before checking for a descendant return
statement, preserving the existing lint checks.

Assisted-by: Codex
Signed-off-by: inoway46 <inoueyuya416@gmail.com>
@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Sep 30, 2026
@inoway46
inoway46 marked this pull request as ready for review September 30, 2026 08:19
@aduh95 aduh95 added author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue PRs queued for automated landing through the Commit Queue. labels Sep 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue PRs queued for automated landing through the Commit Queue. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants