Skip to content

[Bug]: use a better variable name for require_* / import_* with id's including . #8028

@sapphi-red

Description

@sapphi-red

Reproduction link or steps

https://repl.rolldown.rs/#eNqFUVtqwzAQvMqiHydg7H+BD9Az1KWo0cqoyNqgRxowunslP7AJCdGPZl+a0c7EFOMT01bivfn1BVvG97hmlxKOV3IBFBEoRyNUGTU/wlW96+2FrCeDjaHhlPPnkstzyHhwEVPNHBkj6c82uVPp4UDzpLIQ4n0mlKhENAGm8ibA1cRBW8/hc4lhK5RjxYgcKodZzA1dVe+lNfchT1qejzMAWkFOQtd1+6ceWsp8iM5C1VqS+D2SjAZ9u7a3267mZWwjaQ/SQYkh8UbEG46X2paN+aZ41EHP8t2zl5I2uIKvcs14MU4J4zFl627ZCyMC+sDSP8M3rVM=

What is expected?

The output to be like

//#region node_modules/foo.bar/index.js
var require_foo_bar = /* @__PURE__ */ __commonJSMin(((exports) => {
	exports.foo = "foo";
}));

//#endregion
//#region index.js
var import_foo_bar = /* @__PURE__ */ __toESM(require_foo_bar());

What is actually happening?

The output is

//#region node_modules/foo.bar/index.js
var require_foo = /* @__PURE__ */ __commonJSMin(((exports) => {
	exports.foo = "foo";
}));

//#endregion
//#region index.js
var import_foo = /* @__PURE__ */ __toESM(require_foo());

System Info

REPL 1.0.0-rc.1

Any additional comments?

No response

Metadata

Metadata

Labels

No labels
No labels

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions