Fix this.load meta override

Author: PepsRyuu

examples/example-react-refresh/package.json

@@ -12,10 +12,10 @@
         "@babel/core": "^7.13.14",
         "@babel/preset-react": "^7.13.13",
         "@rollup/plugin-babel": "^5.3.0",
-        "@rollup/plugin-node-resolve": "^11.2.1",
+        "@rollup/plugin-node-resolve": "^13.3.0",
         "cross-env": "^7.0.3",
         "react-refresh": "^0.9.0",
-        "rollup": "^1.28.0",
+        "rollup": "^2.77.0",
         "rollup-plugin-commonjs-alternate": "^0.8.0",
         "rollup-plugin-hot-css": "^0.4.0",
         "rollup-plugin-react-refresh": "0.0.3",

lib/impl/ConfigLoader.js

@@ -38,7 +38,7 @@ module.exports = class ConfigLoader {
             // so mark it as external so the require call persists.
             external: id => (id[0] !== '.' && !path.isAbsolute(id)) || id.slice(-5, id.length) === '.json',
             // Support --configPlugin flag to transform this config file before loading
-            plugins: getPlugins(configPlugins)
+            plugins: getPlugins(configPlugins || [])
         });
 
         let isESM = filepath.endsWith('.mjs');

lib/impl/NollupContext.js

@@ -201,7 +201,9 @@ class NollupContext {
         });
 
         this.plugins.onLoad(async ({ id, syntheticNamedExports, resolveDependencies, meta}) => {
-            this.plugins.__meta[id] = meta;
+            if (meta) {
+                this.plugins.__meta[id] = meta;
+            }
             await this.load(id, null, false, syntheticNamedExports, resolveDependencies);
         });
 
@@ -222,6 +224,7 @@ class NollupContext {
 
             return this.resolvedInputs;
         } 
+
     }
 
     /**

lib/impl/PluginLifecycle.js

@@ -153,7 +153,6 @@ function handleMetaProperty (container, filePath, meta) {
             container.__meta[filePath] = fileMeta;
         }
 
-
         for (let prop in meta) {
             fileMeta[prop] = meta[prop];
         }
@@ -371,7 +370,7 @@ const PluginLifecycle = {
                 container.__currentMapChain = null;
                 container.__currentOriginalCode = null;
                 container.__currentModuleId = null;
-
+                
                 return { 
                     code: hr.code,
                     map: await combineSourceMapChainFast(mapChain, originalCode, id),

test/cases/api/context.js

@@ -2764,6 +2764,31 @@ describe ('API: Plugin Context', () => {
             expect(result).to.deep.equal({ hello: true });
         });
 
+        it ('should not override meta if not is provided', async () => {
+            fs.stub('./src/subdep.js', () => `export default 789`);
+            fs.stub('./src/dep.js', () => `import './subdep'; export default 456`);
+            fs.stub('./src/main.js', () => `import './dep'; export default 123`);
+
+            let result;
+            let bundle = await nollup({
+                input: './src/main.js',
+                plugins: [{
+                    async resolveId (id, parent) {
+                        if (id.indexOf('main') > - 1) {
+                            let target = await this.resolve('./dep.js', path.resolve(process.cwd(), './src/main.js'));
+                            (await this.load({ ...target, meta: { hello: true } }));
+                            let modInfo = (await this.load({ ...target, meta: undefined }));
+                            result = modInfo.meta;
+                            
+                        }
+                    }
+                }]
+            });
+
+            await bundle.generate({ format: 'esm' });
+            expect(result).to.deep.equal({ hello: true });
+        });
+
         it ('should support meta via importedIdResolutions', async () => {
             fs.stub('./src/subdep.js', () => `export default 789`);
             fs.stub('./src/dep.js', () => `import './subdep'; export default 456`);
@@ -2928,7 +2953,7 @@ describe ('API: Plugin Context', () => {
 
             const DYNAMIC_IMPORT_PROXY_PREFIX = '\0dynamic-import:';
 
-            let bundle = await rollup({
+            let bundle = await nollup({
                 input: './src/main.js',
                 plugins: [{
                     async resolveDynamicImport(specifier, importer) {