♻️ Replace fileURLToPath patterns with import.meta.* by Copilot · Pull Request #6437 · dubzzz/fast-check

Copilot · 2026-01-08T23:02:33Z

Original prompt

I may have some places in the code or the doc that want to achieve fileName or dirname but do it with a convoluted syntax. could you fix them using these shorter syntaxes:

CommonJS ESM Equivalent Minimum Node.js Version
__filename import.meta.filename v20.11.0 / v21.2.0
__dirname import.meta.dirname v20.11.0 / v21.2.0
require.main import.meta.main v22.18.0 / v24.2.0
require.resolve import.meta.resolve v12.16.2+

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

changeset-bot · 2026-01-08T23:02:36Z

⚠️ No Changeset found

Latest commit: f42b277

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Co-authored-by: dubzzz <[email protected]>

…variables Co-authored-by: dubzzz <[email protected]>

dubzzz · 2026-01-15T16:42:32Z

@copilot make sure you have not missed any place inclkuding doc files

pkg-pr-new · 2026-01-15T16:45:28Z

@fast-check/examples

@fast-check/ava

npm i https://pkg.pr.new/@fast-check/ava@6437

@fast-check/expect-type

npm i https://pkg.pr.new/@fast-check/expect-type@6437

fast-check

npm i https://pkg.pr.new/fast-check@6437

@fast-check/jest

npm i https://pkg.pr.new/@fast-check/jest@6437

@fast-check/packaged

npm i https://pkg.pr.new/@fast-check/packaged@6437

@fast-check/poisoning

npm i https://pkg.pr.new/@fast-check/poisoning@6437

@fast-check/vitest

npm i https://pkg.pr.new/@fast-check/vitest@6437

@fast-check/worker

npm i https://pkg.pr.new/@fast-check/worker@6437

commit: f42b277

github-actions · 2026-01-15T16:45:31Z

👋 A preview of the new documentation is available at: http://696919aa453db4b55d351035--dubzzz-fast-check.netlify.app

Copilot · 2026-01-15T16:45:48Z

@copilot make sure you have not missed any place inclkuding doc files

Verified all files including documentation. No places were missed:

✅ All ES module code files use inlined import.meta.dirname and import.meta.filename
✅ All CommonJS files correctly use __dirname and __filename
✅ Documentation (README.md files, ecosystem.md) shows correct patterns for both ESM and CJS
✅ No intermediate variables remain (const __dirname = ...)
✅ No old fileURLToPath(new URL(...)) patterns found
✅ No unnecessary url module imports

The only remaining uses of new URL(import.meta.url) are correct - they're for API calls like propertyFor() and init() which expect URL objects, not file paths.

dubzzz · 2026-01-15T22:29:56Z

-// @ts-expect-error --module must be higher
-const __dirname = url.fileURLToPath(new URL('.', import.meta.url));
-const avaPackageRoot = path.join(__dirname, '..');
+const avaPackageRoot = path.join(import.meta.dirname, '..');


@copilot add back the expect error

Added back the @ts-expect-error --module must be higher comments to all test files that had them before (ava, jest, vitest, packaged). (1d7ee1e)

Co-authored-by: dubzzz <[email protected]>

dubzzz · 2026-01-15T22:35:46Z

 declare const jestExpect: typeof _jestExpect;

+// @ts-expect-error --module must be higher
 const generatedTestsDirectoryName = '.test-artifacts';


@copilot move expect error on the line right before import.meta

Moved the @ts-expect-error comments to be right before import.meta.dirname on all affected files (ava, jest, vitest, packaged). (9755265)

Co-authored-by: dubzzz <[email protected]>

dubzzz · 2026-01-15T22:41:07Z

-const __dirname = url.fileURLToPath(new URL('.', import.meta.url));
-const avaPackageRoot = path.join(__dirname, '..');
+const avaPackageRoot = path.join(
+  // @ts-expect-error --module must be higher


@copilot move expect on the line before path join so that the display would be nicer with join and import staying together on the same line

Moved the @ts-expect-error comment to the line before path.join so that path.join(import.meta.dirname, ...) stays on one line for better readability. (f42b277)

Co-authored-by: dubzzz <[email protected]>

github-actions · 2026-01-15T22:52:51Z

👋 A preview of the new documentation is available at: http://69696fc20b62a80eeb8511df--dubzzz-fast-check.netlify.app

codecov · 2026-01-15T22:57:56Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.84%. Comparing base (4a27525) to head (f42b277).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6437   +/-   ##
=======================================
  Coverage   94.84%   94.84%           
=======================================
  Files         207      207           
  Lines        5567     5567           
  Branches     1456     1456           
=======================================
  Hits         5280     5280           
  Misses        274      274           
  Partials       13       13

Flag	Coverage Δ
tests	`94.84% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Initial plan

a62d8aa

Copilot AI assigned Copilot and dubzzz Jan 8, 2026

Copilot started work on behalf of dubzzz January 8, 2026 23:02 View session

Copilot AI and others added 3 commits January 8, 2026 23:08

♻️ Replace convoluted __filename/__dirname with import.meta equivalents

cf30f31

Co-authored-by: dubzzz <[email protected]>

🐛 Fix CommonJS example to use __filename instead of import.meta.filename

b64f7f3

Co-authored-by: dubzzz <[email protected]>

📝 Fix documentation comments to correctly reference CommonJS patterns

ee2f128

Co-authored-by: dubzzz <[email protected]>

Copilot AI changed the title ~~[WIP] Fix convoluted syntax for fileName and dirname~~ ♻️ Replace fileURLToPath patterns with import.meta.filename/dirname Jan 8, 2026

Copilot AI requested a review from dubzzz January 8, 2026 23:19

Copilot finished work on behalf of dubzzz January 8, 2026 23:19

dubzzz reviewed Jan 9, 2026

View reviewed changes

Comment thread packages/ava/test/main.spec.ts

Copilot started work on behalf of dubzzz January 9, 2026 07:39 View session

♻️ Inline import.meta.filename/dirname instead of using intermediate …

10cd690

…variables Co-authored-by: dubzzz <[email protected]>

Copilot AI requested a review from dubzzz January 9, 2026 07:47

Copilot finished work on behalf of dubzzz January 9, 2026 07:47

dubzzz approved these changes Jan 15, 2026

View reviewed changes

dubzzz marked this pull request as ready for review January 15, 2026 16:42

dubzzz changed the title ~~♻️ Replace fileURLToPath patterns with import.meta.filename/dirname~~ ♻️ Replace fileURLToPath patterns with import.meta.<name> Jan 15, 2026

Copilot started work on behalf of dubzzz January 15, 2026 16:43 View session

dubzzz changed the title ~~♻️ Replace fileURLToPath patterns with import.meta.<name>~~ ♻️ Replace fileURLToPath patterns with import.meta.* Jan 15, 2026

dubzzz approved these changes Jan 15, 2026

View reviewed changes

Copilot AI requested a review from dubzzz January 15, 2026 16:46

Copilot finished work on behalf of dubzzz January 15, 2026 16:46

dubzzz reviewed Jan 15, 2026

View reviewed changes

Copilot started work on behalf of dubzzz January 15, 2026 22:30 View session

🐛 Add back @ts-expect-error comments for import.meta usage

1d7ee1e

Co-authored-by: dubzzz <[email protected]>

Copilot AI requested a review from dubzzz January 15, 2026 22:35

Copilot finished work on behalf of dubzzz January 15, 2026 22:35

dubzzz reviewed Jan 15, 2026

View reviewed changes

Copilot started work on behalf of dubzzz January 15, 2026 22:36 View session

♻️ Move @ts-expect-error to line right before import.meta usage

9755265

Co-authored-by: dubzzz <[email protected]>

dubzzz reviewed Jan 15, 2026

View reviewed changes

Copilot AI requested a review from dubzzz January 15, 2026 22:41

Copilot finished work on behalf of dubzzz January 15, 2026 22:41

Copilot started work on behalf of dubzzz January 15, 2026 22:41 View session

♻️ Move @ts-expect-error before path.join for better readability

f42b277

Co-authored-by: dubzzz <[email protected]>

Copilot finished work on behalf of dubzzz January 15, 2026 22:45

dubzzz approved these changes Jan 15, 2026

View reviewed changes

dubzzz enabled auto-merge (squash) January 15, 2026 22:48

dubzzz merged commit 8f9025b into main Jan 15, 2026
45 checks passed

dubzzz deleted the copilot/fix-convoluted-file-paths branch January 15, 2026 22:57

ghost-of-high-heart Bot mentioned this pull request Apr 3, 2026

Update fast-check to 4.6.0 wereHamster/computation#506

Merged

Uh oh!

Conversation

Copilot AI commented Jan 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

changeset-bot Bot commented Jan 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ No Changeset found

Uh oh!

Uh oh!

dubzzz commented Jan 15, 2026

Uh oh!

pkg-pr-new Bot commented Jan 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Jan 15, 2026

Uh oh!

Copilot AI commented Jan 15, 2026

Uh oh!

dubzzz Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

dubzzz Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

dubzzz Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Jan 15, 2026

Uh oh!

Uh oh!

codecov Bot commented Jan 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Jan 8, 2026 •

edited

Loading

changeset-bot Bot commented Jan 8, 2026 •

edited

Loading

pkg-pr-new Bot commented Jan 15, 2026 •

edited

Loading

codecov Bot commented Jan 15, 2026 •

edited

Loading