feat: introduce FakerCore

[ST-DDT]

First part of the standalone module function feature #2667

• Proposal: Design for standalone module functions #2667

[netlify]

✅ Deploy Preview for fakerjs ready!

Name	Link
🔨 Latest commit	7353a69
🔍 Latest deploy log	https://app.netlify.com/sites/fakerjs/deploys/6648f5ce488d8e00081f25af
😎 Deploy Preview	https://deploy-preview-2838.fakerjs.dev
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[Shinigami92]

I know the PR is still in draft but I would like to provide some early-feedback

1. I think the refDate should still be named something like default or fallback or base, so the intent is more clear and there might be no variable naming conflict when using an option's refDate + the ...RefDate
2. I suggest to not do something like super({ fakerCore: fakerCore ?? { locale, randomizer, config: {} } }) but only pass the locale and randomizer so the config get's set like every other method call to a default internally. Or is there something special about that I currently overlook? 👀

[ST-DDT]

1. Done
2. Changed slightly to improve readability.

[ST-DDT]

Does anybody know why this test is failing with that particular error message?

❯ test/simple-faker.spec.ts > simpleFaker > should not log anything on startup
→ Unexpected token ','

It seems to happen only with vitest/CJS.

[Shinigami92]

Does anybody know why this test is failing with that particular error message?

❯ test/simple-faker.spec.ts > simpleFaker > should not log anything on startup
→ Unexpected token ','

It seems to happen only with vitest/CJS.

If I set minify: false in tsup.config.ts everything runs fine. Please report a bug-issue to https://github.com/egoist/tsup

Or it is related to https://github.com/egoist/tsup/issues/1091

[ST-DDT]

The error is caused by this part:

_class.prototype.__init23.call(this);,this.definitions=_chunkWAQHUEHTcjs.d.call(void 0, this.rawDefinitions)
                                     ^

Which is equivalent to this source code:

faker/src/faker.ts

Lines 195 to 197 in e4da2f5

	super({ fakerCore });
	this.definitions = createLocaleProxy(this.rawDefinitions);

[ST-DDT]

I created an issue in tsup with a minimal reproduction:

• Bug when bundling commonjs when there is code before super() egoist/tsup#1124

And it has reeeaaally specific conditions, check the issue for more details.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.95%. Comparing base (7dc8a18) to head (7353a69).

Additional details and impacted files

@@            Coverage Diff             @@
##             next    #2838      +/-   ##
==========================================
- Coverage   99.96%   99.95%   -0.01%     
==========================================
  Files        2977     2977              
  Lines      215422   215463      +41     
  Branches      950      954       +4     
==========================================
+ Hits       215351   215375      +24     
- Misses         71       88      +17     

Files	Coverage Δ
src/faker.ts	100.00% <100.00%> (ø)
src/modules/number/index.ts	100.00% <100.00%> (ø)
src/simple-faker.ts	100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

[ST-DDT]

Should we deprecate this field?