Skip to content

eshost: run module-flagged .js tests as their real file #3

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

Merged
Sam Gammon (sgammon) merged 1 commit into from
Jul 18, 2026
Merged

eshost: run module-flagged .js tests as their real file #3

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
15 changes: 6 additions & 9 deletions harness/src/eshost-elide/agent.elide.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,15 +144,12 @@ class ElideAgent extends ConsoleAgent {
// Fixture copying is best-effort; the test itself still runs.
}
}
if (this._elideModule && args[0].endsWith(".js")) {
const mjs = `${args[0].slice(0, -3)}.mjs`;
try {
fs.copyFileSync(args[0], mjs);
args = [mjs, ...args.slice(1)];
} catch {
// Fall back to the original `.js` file if the copy fails.
}
}
// A module-flagged `.js` entry runs as ESM via the `{"type":"module"}`
// marker written above — no `.mjs` copy needed. Copying to `.mjs` would
// give the entry a distinct file identity from any self/cyclic import that
// names the original `.js` (e.g. instn-*_FIXTURE re-exports), splitting one
// module into two instances and breaking TDZ/binding-identity tests. Run
// the `.js` in place so those imports dedupe to the single entry record.
}
return super.createChildProcess(args, options);
}
Expand Down