Skip to content

Commit df28359

Browse files
committed
fix: simply put onRequire module content onto package space
1 parent c403add commit df28359

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

spec/bundler.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -452,8 +452,8 @@ test('Bundler supports implementation returned by onRequire', t => {
452452
{contents: 'dumber-module-loader;'},
453453
{contents: 'define.switchToUserSpace();'},
454454
{path: 'src/app.js', contents: "define('app',[\"foo\"],1);", sourceMap: undefined},
455-
{path: '__on_require__/foo.js', contents: "define('foo',[\"loo\"],1);", sourceMap: undefined},
456455
{contents: 'define.switchToPackageSpace();'},
456+
{path: '__on_require__/foo.js', contents: "define('foo',[\"loo\"],1);", sourceMap: undefined},
457457
{path: 'node_modules/loo/loo.js', contents: "define('loo/loo',[],1);define('loo',['loo/loo'],function(m){return m;});", sourceMap: undefined},
458458
{contents: 'define.switchToUserSpace();'},
459459
],

src/index.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,8 @@ export default class Bundler {
267267
return this.capture({
268268
path: '__on_require__/' + parsedId.bareId + (parsedId.ext ? '' : '.js'),
269269
contents: result,
270-
moduleId: parsedId.bareId
270+
moduleId: parsedId.bareId,
271+
packageName: parsedId.parts[0]
271272
});
272273
}
273274

0 commit comments

Comments
 (0)