Improve 'Install x to use y' and 'Unrecognized method' error messages.

Author: Lyrkan

index.js

@@ -1279,7 +1279,13 @@ const EncoreProxy = new Proxy(new Encore(), {
             // Find the property with the closest Levenshtein distance
             let similarProperty;
             let minDistance = Number.MAX_VALUE;
-            for (const apiProperty in target) {
+
+            for (const apiProperty of Object.getOwnPropertyNames(Encore.prototype)) {
+                // Ignore class constructor
+                if (apiProperty === 'constructor') {
+                    continue;
+                }
+
                 const distance = levenshtein.get(apiProperty, prop);
                 if (distance <= minDistance) {
                     similarProperty = apiProperty;
@@ -1292,8 +1298,14 @@ const EncoreProxy = new Proxy(new Encore(), {
                 errorMessage += ` Did you mean ${chalk.green(`Encore.${similarProperty}`)}?`;
             }
 
+            // Prettify the error message.
+            // Only keep the 2nd line of the stack trace:
+            // - First line should be this file (index.js)
+            // - Second line should be the Webpack config file
+            const pe = new PrettyError();
+            pe.skip((traceLine, lineNumber) => lineNumber !== 1);
             const error = new Error(errorMessage);
-            console.log(new PrettyError().render(error));
+            console.log(pe.render(error));
             process.exit(1); // eslint-disable-line
         }
 

lib/package-helper.js

@@ -18,11 +18,10 @@ function ensurePackagesExist(packagesConfig, requestedFeature) {
     const missingPackagesRecommendation = getMissingPackageRecommendations(packagesConfig, requestedFeature);
 
     if (missingPackagesRecommendation) {
-        throw new Error(`
+        throw `
 ${missingPackagesRecommendation.message}
   ${missingPackagesRecommendation.installCommand}
-`
-        );
+`;
     }
 
     // check for invalid versions & warn

test/bin/encore.js

@@ -193,4 +193,41 @@ module.exports = Encore.getWebpackConfig();
             done();
         });
     });
+
+    it('Display an error when calling an unknown method', (done) => {
+        testSetup.emptyTmpDir();
+        const testDir = testSetup.createTestAppDir();
+
+        fs.writeFileSync(
+            path.join(testDir, 'package.json'),
+            `{
+                "devDependencies": {
+                    "@symfony/webpack-encore": "*"
+                }
+            }`
+        );
+
+        fs.writeFileSync(
+            path.join(testDir, 'webpack.config.js'),
+            `
+const Encore = require('../../index.js');
+Encore
+    .setOutputPath('build/')
+    .setPublicPath('/build')
+    .enableSingleRuntimeChuck()
+    .addEntry('main', './js/no_require')
+;
+
+module.exports = Encore.getWebpackConfig();
+            `
+        );
+
+        const binPath = path.resolve(__dirname, '../', '../', 'bin', 'encore.js');
+        exec(`node ${binPath} dev --context=${testDir}`, { cwd: testDir }, (err, stdout, stderr) => {
+            expect(err).not.to.be.null;
+            expect(stdout).to.contain('is not a recognized property or method');
+            expect(stdout).to.contain('Did you mean');
+            done();
+        });
+    });
 });