Tweaks from review

Author: weaverryan

README.md

@@ -57,7 +57,7 @@ Remove the following line in the mentioned file: it's not needed anymore:
 
 The bridge works by reading a `controllers.json` file that is automatically
 updated by Symfony Flex whenever you download a UX-powered package. For
-example, after running `composer require symfony/ux-dropzone.js`, it will
+example, after running `composer require symfony/ux-dropzone`, it will
 look like this:
 
 ```json
@@ -81,15 +81,17 @@ Each item under `controllers` will cause a Stimulus controller to be
 registered with a specific name - in this case the controller would
 be called `symfony--ux-dropzone--dropzone` (the `/` becomes `--`).
 
-By default, the new controller will be always be included in your
+By default, the new controller will always be included in your
 JavaScript package. You can control that with the `webpackMode` option,
 ordered from least to most lazy:
 
-* `webpackMode: 'eager'`: include in JavaScript that is downloaded on page load
-* `webpackMode: 'lazy'`: isolate into a separate JavaScript file, but immediately
-  download this after the page is loaded.
-* `webpackMode: 'controller-lazy'`: wait until this controller HTML is
-  first included on the page and *then* download the controller and execute it.
+* `webpackMode: 'eager'`: controller & dependencies are included in the JavaScript
+  that's downloaded when the page is loaded.
+* `webpackMode: 'lazy'`: controller & dependencies are isolated into a separate
+  JavaScript file, but immediately downloaded asynchronously after the page is loaded.
+* `webpackMode: 'controller-lazy'`: controller & dependencies are isolated into a
+  separate file and only downloaded asynchronously if (and when) the `data-controller`
+  HTML appears on the page.
 
 ## Run tests
 

src/webpack/create-controllers-module.js

@@ -49,7 +49,7 @@ module.exports = function createControllersModule(config) {
             const controllerMain = packageName + '/' + controllerPackageConfig.main;
             const webpackMode = controllerUserConfig.webpackMode;
 
-            let moduleValueContents = '';
+            let moduleValueContents = `import(/* webpackMode: "${webpackMode}" */ '${controllerMain}')`;
             if (webpackMode === 'lazy-controller') {
                 hasLazyControllers = true;
                 moduleValueContents = `
@@ -63,8 +63,6 @@ new Promise((resolve, reject) => resolve({ default:
       }
   }))
                 `.trim();
-            } else {
-                moduleValueContents = `import(/* webpackMode: "${webpackMode}" */ '${controllerMain}')`;
             }
 
             controllerContents += `\n  '${controllerNormalizedName}': ${moduleValueContents},`;