@abcnews/aunty 14.0.1 → 15.0.0

package/README.md

@@ -120,7 +120,7 @@ If you don't need to override any of the project defaults, your entire aunty con
 | `hasBundleAnalysis` | `false`       | Setting this to true will spin up a second server on another port, allowing you to inspect your bundle. The address will be logged to the console when the dev server starts.                                                                  |
 | `host`              | `"localhost"` | The hostname of your dev server. If you're on the ABC internal network, the default will change to your machine's hostname (`ws<number>.aus.aunty.abc.net.au`). The `AUNTY_HOST` environment variable, if present, will override this setting. |
 | `hot`               | `true`        | Should the dev server enable hot reloading. If `NODE_ENV !== "development"`, the default will change to `false`.                                                                                                                               |
-| `https`             | `true`        | Should the dev server use SSL (with a self-signed certificate matching the `host`). You can alternatively supply your own `{cert: string, key: string}` object if you've generated your certificate some other way.                             |
+| `https`             | `true`        | Should the dev server use SSL (with a self-signed certificate matching the `host`). You can alternatively supply your own `{cert: string, key: string}` object if you've generated your certificate some other way.                            |
 | `port`              | `8000`        | The port number of your dev server. If the port specified is unavailable, **aunty** will try incrementing the port number until it finds an available one. The `AUNTY_PORT` environment variable, if present, will override this setting.      |
 
 #### `deploy` config properties
@@ -142,18 +142,6 @@ You _could_ also specify `type` (`"ftp"`/`"ssh"`), `host`, `port`, `username` &
 
 Aunty comes with a few basic generators. Run `aunty generate --help` for the full list, or `aunty generate <generator> --help` for further details.
 
-### Async/await
-
-One way to add `async`/`await` and generators/`yield` to your project is with the [`regenerator-runtime`](https://www.npmjs.com/package/regenerator-runtime) package.
-
-`npm install regenerator-runtime` and then:
-
-```js
-import 'regenerator-runtime/runtime';
-```
-
-Note: You may also need a `Promise` polyfill for IE11.
-
 ## Authors
 
 - Colin Gourlay

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abcnews/aunty",
-  "version": "14.0.1",
+  "version": "15.0.0",
   "description": "A toolkit for working with ABC News projects",
   "repository": "abcnews/aunty",
   "license": "MIT",
@@ -12,7 +12,8 @@
   ],
   "scripts": {
     "postinstall": "node scripts/postinstall.js",
-    "release": "release-it"
+    "release": "release-it",
+    "test": "jest test/test.js"
   },
   "files": [
     "assets",
@@ -29,7 +30,7 @@
   },
   "main": "src/api",
   "engines": {
-    "node": ">=7.6.0"
+    "node": ">=16"
   },
   "dependencies": {
     "@babel/core": "^7.11.4",
@@ -45,7 +46,7 @@
     "basic-ftp": "^5.0.2",
     "chalk": "^4.1.2",
     "cli-select": "^1.1.2",
-    "copy-webpack-plugin": "^8.1.1",
+    "copy-webpack-plugin": "^11.0.0",
     "css-loader": "^6.7.1",
     "csv-loader": "^3.0.3",
     "deepmerge": "^4.0.0",
@@ -77,11 +78,11 @@
     "sass-loader": "^11.0.1",
     "semver": "^7.3.2",
     "style-loader": "^2.0.0",
-    "svelte": "^3.24.1",
-    "svelte-loader": "^3.1.1",
-    "svelte-preprocess": "^4.1.1",
+    "svelte": "^4.2.8",
+    "svelte-loader": "^3.1.9",
+    "svelte-preprocess": "^5.1.1",
     "tcp-ping-sync": "^1.0.0",
-    "typescript": "^4.0.2",
+    "typescript": "^5.3.2",
     "update-notifier": "^6.0.2",
     "webpack": "^5.33.2",
     "webpack-bundle-analyzer": "^4.5.0",
@@ -94,6 +95,7 @@
     "babel-eslint": "^10.1.0",
     "eslint": "^7.7.0",
     "eslint-config-prettier": "^8.2.0",
-    "release-it": "^15.1.4"
+    "release-it": "^15.1.4",
+    "yeoman-automation-adapter": "^2.0.0"
   }
 }

package/src/cli/build/index.js

@@ -19,77 +19,82 @@ const { command } = require('../');
 const cleanCommand = require('../clean');
 const { MESSAGES, OPTIONS } = require('./constants');
 
-module.exports = command(
-  {
-    name: 'build',
-    nodeEnv: 'production',
-    options: OPTIONS,
-    usage: MESSAGES.usage
-  },
-  async argv => {
-    const { root } = getProjectConfig();
-    const webpackConfig = getWebpackConfig();
-    let deployConfig;
-
-    if (!argv.local) {
-      deployConfig = getDeployConfig({ id: argv.id });
-      webpackConfig.forEach(config => {
-        config.output.publicPath = deployConfig.targets[0].publicPath;
-      });
-    }
-
-    if (argv.dry) {
-      return dry(
-        combine(
-          {
-            'Webpack config': webpackConfig
-          },
-          deployConfig
-            ? {
-                'Deploy config': deployConfig
-              }
-            : {}
-        )
-      );
-    }
+const build = async argv => {
+  const { root } = getProjectConfig();
+  const webpackConfig = getWebpackConfig();
+  let deployConfig;
 
-    throws(await cleanCommand(['--quiet']));
+  if (!argv.local) {
+    deployConfig = getDeployConfig({ id: argv.id });
+    webpackConfig.forEach(config => {
+      config.output.publicPath = deployConfig.targets[0].publicPath;
+    });
+  }
 
-    info(
-      MESSAGES.build({
-        id: deployConfig ? deployConfig.id : null,
-        publicPaths: deployConfig ? deployConfig.targets.map(x => x.publicPath) : [webpackConfig[0].output.publicPath]
-      })
+  if (argv.dry) {
+    return dry(
+      combine(
+        {
+          'Webpack config': webpackConfig
+        },
+        deployConfig
+          ? {
+              'Deploy config': deployConfig
+            }
+          : {}
+      )
     );
+  }
 
-    let spinner = spin('Building');
-    const startTime = Date.now();
-    const compiler = webpack(webpackConfig);
-    const stats = unpack(await packs(pify(compiler.run.bind(compiler)))());
+  throws(await cleanCommand(['--quiet']));
 
-    if (stats.hasErrors()) {
-      const errors = stats.toJson({}, true).errors;
+  info(
+    MESSAGES.build({
+      id: deployConfig ? deployConfig.id : null,
+      publicPaths: deployConfig ? deployConfig.targets.map(x => x.publicPath) : [webpackConfig[0].output.publicPath]
+    })
+  );
 
-      spinner.fail();
+  let spinner = spin('Building');
+  const startTime = Date.now();
+  const compiler = webpack(webpackConfig);
+  const stats = unpack(await packs(pify(compiler.run.bind(compiler)))());
 
-      if (errors.length > 1) {
-        throw MESSAGES.multipleErrors(errors.map(error => error.message));
-      }
+  if (stats.hasErrors()) {
+    const errors = stats.toJson({}, true).errors;
 
-      throw errors[0];
-    }
+    spinner.fail();
 
-    if (stats.hasWarnings()) {
-      spinner.warn();
-      stats.toJson({}, true).warnings.forEach(warning => warn(warning.message));
-    } else {
-      spinner.succeed(`Built in ${((Date.now() - startTime) / 1000).toFixed(2)}s`);
+    if (errors.length > 1) {
+      throw MESSAGES.multipleErrors(errors.map(error => error.message));
     }
 
-    if (deployConfig) {
-      spinner = spin('Creating deploy configuration');
-      writeJsonFile.sync(join(root, OUTPUT_DIRECTORY_NAME, DEPLOY_FILE_NAME), deployConfig);
-      spinner.succeed('Created deploy configuration');
-    }
+    throw errors[0];
   }
+
+  if (stats.hasWarnings()) {
+    spinner.warn();
+    stats.toJson({}, true).warnings.forEach(warning => warn(warning.message));
+  } else {
+    spinner.succeed(`Built in ${((Date.now() - startTime) / 1000).toFixed(2)}s`);
+  }
+
+  if (deployConfig) {
+    spinner = spin('Creating deploy configuration');
+    writeJsonFile.sync(join(root, OUTPUT_DIRECTORY_NAME, DEPLOY_FILE_NAME), deployConfig);
+    spinner.succeed('Created deploy configuration');
+  }
+};
+
+module.exports = command(
+  {
+    name: 'build',
+    nodeEnv: 'production',
+    options: OPTIONS,
+    usage: MESSAGES.usage
+  },
+  build
 );
+
+// Expose this for the tests
+module.exports._testBuild = build;

package/src/cli/constants.js

@@ -1,7 +1,6 @@
 // Ours
 const { createLogo } = require('../utils/branding');
 const { cmd, dim, hvy, opt, req, sec } = require('../utils/color');
-const { setOfValues } = require('../utils/structures');
 
 const COMMAND_ALIASES = (module.exports.COMMAND_ALIASES = {
   b: 'build',
@@ -14,7 +13,7 @@ const COMMAND_ALIASES = (module.exports.COMMAND_ALIASES = {
   t: 'test'
 });
 
-module.exports.COMMANDS = setOfValues(COMMAND_ALIASES);
+module.exports.COMMANDS = new Set(Object.values(COMMAND_ALIASES));
 
 module.exports.DEFAULTS = {
   name: '__command__',

package/src/cli/generate/constants.js

@@ -1,6 +1,5 @@
 // Ours
 const { cmd, hvy, opt, req, sec } = require('../../utils/color');
-const { setOfValues } = require('../../utils/structures');
 
 module.exports.OPTIONS = {
   boolean: ['announce'],
@@ -14,7 +13,7 @@ const GENERATOR_ALIASES = (module.exports.GENERATOR_ALIASES = {
   p: 'project'
 });
 
-module.exports.GENERATORS = setOfValues(GENERATOR_ALIASES);
+module.exports.GENERATORS = new Set(Object.values(GENERATOR_ALIASES));
 
 module.exports.MESSAGES = {
   generatorDoesNotExist: name => `The generator '${name}' does not exist.`,

package/src/cli/generate/index.js

@@ -10,55 +10,63 @@ const { log } = require('../../utils/logging');
 const { command } = require('../');
 const { GENERATOR_ALIASES, GENERATORS, MESSAGES, OPTIONS } = require('./constants');
 
-module.exports = command(
-  {
-    name: 'generate',
-    hasSubcommands: true,
-    options: OPTIONS
-  },
-  async argv => {
-    if (argv.dry) {
-      throw MESSAGES.noDryRuns;
-    }
-
-    const generatorName = GENERATOR_ALIASES[argv._[0]] || argv._[0];
+const generate = async argv => {
+  if (argv.dry) {
+    throw MESSAGES.noDryRuns;
+  }
 
-    // If we didn't supply a known generator name, blow up, unless we wanted
-    // the generate command's usage message (in which case, print it, then exit).
+  const generatorName = GENERATOR_ALIASES[argv._[0]] || argv._[0];
 
-    if (!GENERATORS.has(generatorName)) {
-      if (!generatorName || generatorName.indexOf('-') === 0 || argv.help) {
-        return log(MESSAGES.usage);
-      }
+  // If we didn't supply a known generator name, blow up, unless we wanted
+  // the generate command's usage message (in which case, print it, then exit).
 
-      throw MESSAGES.generatorDoesNotExist(generatorName);
+  if (!GENERATORS.has(generatorName)) {
+    if (!generatorName || generatorName.indexOf('-') === 0 || argv.help) {
+      return log(MESSAGES.usage);
     }
 
-    log(createCommandLogo(`generate ${generatorName}`));
+    throw MESSAGES.generatorDoesNotExist(generatorName);
+  }
 
-    const env = yeoman.createEnv();
+  log(createCommandLogo(`generate ${generatorName}`));
 
-    // Register the generator
+  // If we're in a test environment, insert the auto adapter with the provided answers
+  const env = global.auntyYeomanAnswers
+    ? yeoman.createEnv([], {}, new (require('yeoman-automation-adapter').AutoAdapter)(global.auntyYeomanAnswers, true))
+    : yeoman.createEnv();
 
-    const generatorPath = require.resolve(`../../generators/${generatorName}`);
+  // Register the generator
 
-    env.register(generatorPath, generatorName);
+  const generatorPath = require.resolve(`../../generators/${generatorName}`);
 
-    // Run the generator, including known arguments
+  env.register(generatorPath, generatorName);
 
-    const runArgs = [generatorName]
-      .concat(argv['--'])
-      .concat(argv.help ? ['--help'] : [])
-      .join(' ');
+  // Run the generator, including known arguments
 
-    const [err] = await pack(env.run(runArgs));
+  const runArgs = [generatorName]
+    .concat(argv['--'])
+    .concat(argv.help ? ['--help'] : [])
+    .join(' ');
 
-    if (err) {
-      throw err;
-    }
+  const [err] = await pack(env.run(runArgs));
 
-    if (argv.announce) {
-      announce();
-    }
+  if (err) {
+    throw err;
+  }
+
+  if (argv.announce) {
+    announce();
   }
+};
+
+module.exports = command(
+  {
+    name: 'generate',
+    hasSubcommands: true,
+    options: OPTIONS
+  },
+  generate
 );
+
+// Expose this for the tests
+module.exports._testGenerate = generate;

package/src/config/serve.js

@@ -86,6 +86,16 @@ const findPort = async (port, max = port + 100, host = '0.0.0.0') => {
       // If the connection is refused, it's assumed nothing is listening and the port is available.
       if (e.code === 'ECONNREFUSED') {
         found();
+      } else if (e.code === 'ENOTFOUND' && e.syscall === 'getaddrinfo' && e.hostname?.includes(INTERNAL_SUFFIX)) {
+        console.error(
+          [
+            'Could not resolve hostname ' + e.hostname,
+            'You appear to be on the ABC network without a hostname attached to your computer.',
+            "Aunty can't continue. Consider reconnecting, or add your hostname to your hosts file.",
+            ''
+          ].join('\n')
+        );
+        process.exit(1);
       } else {
         // Not sure what to do with other errors. Log the code & keep seeking a free port.
         next(`error code ${e.code}`);

package/src/config/webpack.js

@@ -47,7 +47,7 @@ const PROJECT_TYPES_CONFIG = {
     config.resolve = {
       // Make sure that only one copy of the Svelte runtime is bundled in the app
       alias: {
-        svelte: resolve('node_modules', 'svelte')
+        svelte: resolve('node_modules', 'svelte/src/runtime')
       },
       // Recognise .svelte files
       extensions: [...config.resolve.extensions, '.svelte'],

package/src/generators/project/index.js

@@ -217,7 +217,7 @@ Shorthand examples (assuming xyz is your project name):
         break;
       case 'svelte':
         devDependencies.push('@testing-library/svelte');
-        dependencies.push('svelte');
+        dependencies.push('svelte@4');
         break;
       default:
         break;

package/src/generators/project/templates/preact/src/index.tsx

@@ -26,7 +26,7 @@ if (module.hot) {
   module.hot.accept('./components/App', () => {
     try {
       renderApp();
-    } catch (err) {
+    } catch (err<% if (isTS) { %>: any<% } %>) {
       import('./components/ErrorBox').then(({ default: ErrorBox }) => {
         if (appMountEl) {
           render(<ErrorBox error={err} />, appMountEl);

package/src/utils/structures.js

@@ -31,5 +31,3 @@ module.exports.combine = (...sources) => sources.reduce(combine, {});
  * @see combine
  */
 module.exports.merge = (...sources) => sources.reduce(combineDeep, {});
-
-module.exports.setOfValues = source => new Set(Object.keys(source).map(key => source[key]));

package/ts/config.json

@@ -14,7 +14,7 @@
     "esModuleInterop": true,
     "forceConsistentCasingInFileNames": true,
     "allowSyntheticDefaultImports": true,
-    "importsNotUsedAsValues": "preserve",
+    "verbatimModuleSyntax": true,
     "resolveJsonModule": true,
     "isolatedModules": true,
     "strictNullChecks": true