Skip to content

Use Node.js compile cache #193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
+13 −0
Open

Use Node.js compile cache #193

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b310002
Use Node.js compile cache
outslept Sep 12, 2025
ecac95d
Disable NODE_DISABLE_COMPILE_CACHE in CI so that it does not interfer…
outslept Sep 12, 2025
a2b44b4
Use namespace import for node:module
outslept Sep 12, 2025
cdd601d
Switch from namespace to default import
outslept Sep 12, 2025
e909a52
Flush compile cache before spawning child process
outslept Sep 12, 2025
f53c3a9
Fix typo
outslept Sep 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ on:

env:
FORCE_COLOR: 2
NODE_DISABLE_COMPILE_CACHE: 1

jobs:
lint:
Expand Down
5 changes: 5 additions & 0 deletions lib/cjs.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
const mod = require('node:module')

// to use V8's code cache to speed up instantiation time
mod.enableCompileCache?.()

const { default: npmRunAll } = require('./esm.mjs')

// Didn't manage to get this into the last breaking change, so adding a cjs as the primary export
Expand Down
4 changes: 4 additions & 0 deletions lib/esm.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
// Requirements
// ------------------------------------------------------------------------------

import mod from 'node:module'
import shellQuote from 'shell-quote'
import matchTasks from './match-tasks.js'
import readPackageJson from './read-package-json.js'
Expand All @@ -18,6 +19,9 @@ import runTasks from './run-tasks.js'
// Helpers
// ------------------------------------------------------------------------------

// to use V8's code cache to speed up instantiation time
mod.enableCompileCache?.()

const ARGS_PATTERN = /\{(!)?([*@%]|\d+)([^}]+)?}/g
const ARGS_UNPACK_PATTERN = /\{(!)?([%])([^}]+)?}/g

Expand Down
3 changes: 3 additions & 0 deletions lib/run-task.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@

import fs from 'node:fs'
import path from 'node:path'
import mod from 'node:module'

import ansiStyles from 'ansi-styles'
import { parse as parseArgs } from 'shell-quote'
Expand Down Expand Up @@ -208,6 +209,8 @@ export default function runTask (task, options) {
}
Array.prototype.push.apply(spawnArgs, parseArgs(task).map(cleanTaskArg))

mod.flushCompileCache?.()

cp = await spawn(execPath, spawnArgs, spawnOptions)

// Piping stdio.
Expand Down