@caweb/a11y-webpack-plugin 2.0.1 → 2.0.3

package/aceconfig.js

@@ -2,6 +2,11 @@
  * Configuration for Accessibility Checker
  * @link https://www.npmjs.com/package/accessibility-checker
  */
+/**
+ * External dependencies
+ */
+import path from 'path';
+import { getArgVal, getAllFlags } from '@caweb/webpack/lib/args.js';
 
 let levels = [
     'violation', 
@@ -14,31 +19,32 @@ let levels = [
 let reportLevels = levels;
 let failLevels = levels;
 
-// process args
-process.argv.forEach((arg) => {
+// remove any levels based on flags
+Object.keys(getAllFlags()).forEach((flag) => {
     // remove any report levels
-    if( arg.includes('--no-report-levels-') ){  
-      let r = arg.replace('--no-report-levels-', '')
+    if( flag.includes('no-report-levels-') ){  
+      let r = flag.replace('no-report-levels-', '')
       delete reportLevels[reportLevels.indexOf(r)]
     }
     // remove any fails levels
-    if( arg.includes('--no-fail-levels-') ){  
-      let f = arg.replace('--no-fail-levels-', '')
+    if( flag.includes('no-fail-levels-') ){  
+      let f = flag.replace('no-fail-levels-', '')
       delete failLevels[failLevels.indexOf(f)]
     }
   })
 
+
 export default {
     ruleArchive: "latest",
     policies: [
         'WCAG_2_1'
     ],
-    failLevels: failLevels.filter(e=>e),
-    reportLevels: reportLevels.filter(e=>e),
+    failLevels: failLevels.filter( Boolean ),
+    reportLevels: reportLevels.filter( Boolean ),
     outputFilename: 'reports',
-    outputFolder: "/audits/a11y",
+    outputFolder: '/audits/a11y',
     outputFormat: [
-        'html', 'json'
+        'html'
     ],
     outputFilenameTimestamp: false
 }

package/index.js

@@ -4,63 +4,46 @@
  * External dependencies
  */
 import spawn from 'cross-spawn';
-import { getAllFilesSync } from 'get-all-files'
 import EntryDependency from "webpack/lib/dependencies/EntryDependency.js";
 import path from 'path';
 import { isUrl, isValidUrl } from 'check-valid-url';
 import fs from 'fs';
 import deepmerge from 'deepmerge';
 import chalk from 'chalk';
-import { fileURLToPath, URL } from 'url';
+import { fileURLToPath, URL, pathToFileURL } from 'url';
+import { createRequire } from 'module';
+
+const require = createRequire(import.meta.url);
 
 /**
  * Internal dependencies
  */
-import { landingPage, reporter } from './reporter.js';
+import { landingPage } from './reporter.js';
 
 // default configuration
 import {default as DefaultConfig} from './aceconfig.js';
 
-const boldWhite = chalk.bold.white;
-const boldGreen = chalk.bold.green;
-const boldBlue = chalk.bold.hex('#03a7fc');
-const currentPath = path.dirname(fileURLToPath(import.meta.url));
-
 const pluginName = 'CAWebA11yWebpackPlugin';
 
 // IBM Accessibility Checker Plugin
-class A11yPlugin {
+class CAWebA11yWebpackPlugin {
     config = {}
 
     constructor(opts = {}) {
-      // the default publicPath is always the outputFolder
-      DefaultConfig.publicPath = DefaultConfig.outputFolder;
-
-      // the default output folder is always relative to the current working directory.
-      DefaultConfig.outputFolder = path.join( process.cwd(), DefaultConfig.outputFolder );
-
-      // if opts.outputFolder is defined
-      if( opts.outputFolder && ! path.isAbsolute(opts.outputFolder)  ){
-        opts.publicPath = opts.outputFolder;
-
-        // we join the current working directory with the opts.outputFolder
-        opts.outputFolder = path.join( process.cwd(), opts.outputFolder );
-      }
-      
       this.config = deepmerge(DefaultConfig, opts);
     }
 
     
     apply(compiler) {
       const staticDir = {
-        directory: this.config.outputFolder,
-        publicPath: encodeURI(this.config.publicPath).replace(':', ''),
+        directory: path.join( process.cwd(), this.config.outputFolder ),
+        publicPath: encodeURI( this.config.outputFolder ).replace(':', ''),
         watch: true
       }
 
       let { devServer } = compiler.options;
       let auditUrl = `${devServer.server}://${devServer.host}:${devServer.port}`;
-      let nodeModulePath = encodeURI(this.config.publicPath).replace(':', '') + '/node_modules';
+      let nodeModulePath = encodeURI(staticDir.publicPath) + '/node_modules/';
       let pathRewrite = {};
       pathRewrite[`^${nodeModulePath}`] = '';
       
@@ -77,7 +60,7 @@ class A11yPlugin {
         devServer.proxy = [].concat(devServer.proxy, proxy );
       }
 
-      // add our static directory to the devServer
+      // // add our static directory to the devServer
       if( Array.isArray(devServer.static) ){
         devServer.static.push(staticDir)
       }else{
@@ -87,61 +70,61 @@ class A11yPlugin {
       // if dev server allows for multiple pages to be opened
       // add outputFilename.html to open property.
       if( Array.isArray(devServer.open) ){
-        devServer.open.push(`${staticDir.publicPath}/${this.config.outputFilename}.html`)
+        devServer.open.push(`${auditUrl}${this.config.outputFolder}/${this.config.outputFilename}.html`)
       }else if( 'object' === typeof devServer.open && Array.isArray(devServer.open.target) ){
-        devServer.open.target.push(`${staticDir.publicPath}/${this.config.outputFilename}.html`)
+        devServer.open.target.push(`${auditUrl}${this.config.outputFolder}/${this.config.outputFilename}.html`)
       }
 
-      // we always make sure the output folder exists
-      fs.mkdirSync( staticDir.directory, { recursive: true } );
+      // we always make sure the output folder is cleared
+      fs.rmSync( staticDir.directory, { recursive: true, force: true } );
 
       // Hot Module Replacement
-      if( compiler?.options?.devServer?.hot ){
-        // we create a blank file for the hot update to compile on our page.
-        // this is required for the hot-update to work.
-        fs.writeFileSync(
-          path.join(staticDir.directory, `a11y.update.js`),
-          `` // required for hot-update to compile on our page, blank script for now
-        );
-
-        // we add the entry to the dependency factory during compilation
-        compiler.hooks.compilation.tap(
-          pluginName,
-          (compilation, { normalModuleFactory }) => {
-            compilation.dependencyFactories.set(
-              EntryDependency,
-              normalModuleFactory
-            );
-
-          }
-        );
-
-        // we add the entry before the compilation ends
-        compiler.hooks.make.tapAsync(
-          pluginName, 
-          (compilation, callback) => {
-            const { entry, options, context } = {
-            entry: path.join( staticDir.directory, 'a11y.update.js'),
-            options: {
-              name: 'a11y.update'
-            },
-            context: 'a11y'
-          };
-
-          const dep = new EntryDependency(entry);
-          dep.loc = {
-            name: options.name
-          };
+      // if( compiler?.options?.devServer?.hot ){
+      //   // we create a blank file for the hot update to compile on our page.
+      //   // this is required for the hot-update to work.
+      //   fs.writeFileSync(
+      //     path.join(staticDir.directory, `a11y.update.js`),
+      //     `` // required for hot-update to compile on our page, blank script for now
+      //   );
+
+      //   // we add the entry to the dependency factory during compilation
+      //   compiler.hooks.compilation.tap(
+      //     pluginName,
+      //     (compilation, { normalModuleFactory }) => {
+      //       compilation.dependencyFactories.set(
+      //         EntryDependency,
+      //         normalModuleFactory
+      //       );
+
+      //     }
+      //   );
+
+      //   // we add the entry before the compilation ends
+      //   compiler.hooks.make.tapAsync(
+      //     pluginName, 
+      //     (compilation, callback) => {
+      //       const { entry, options, context } = {
+      //       entry: path.join( staticDir.directory, 'a11y.update.js'),
+      //       options: {
+      //         name: 'a11y.update'
+      //       },
+      //       context: 'a11y'
+      //     };
+
+      //     const dep = new EntryDependency(entry);
+      //     dep.loc = {
+      //       name: options.name
+      //     };
           
-          compilation.addEntry(
-            context,
-            dep, 
-            options, 
-            err => {
-              callback(err);
-            });
-          });
-      }
+      //     compilation.addEntry(
+      //       context,
+      //       dep, 
+      //       options, 
+      //       err => {
+      //         callback(err);
+      //       });
+      //     });
+      // }
 
       compiler.hooks.thisCompilation.tap(pluginName, (compilation) => {
         // We can audit the html files now that the compilation is done.
@@ -150,29 +133,23 @@ class A11yPlugin {
           pluginName,
           (stats, callback) => {
 
-            console.log(`<i> ${boldGreen('[webpack-dev-middleware] Running IBM Accessibility scan...')}`);
-
-            // we check the compilers output.publicPath
-            // if it is not set, we scan the output.path as the publicPath
-            let result = this.a11yCheck(
-              ! compiler.options.output.publicPath || 
-              'auto' === compiler.options.output.publicPath ? 
-                compiler.options.output.path : 
-                compiler.options.output.publicPath,
+            console.log('<i> \x1b[32m[webpack-dev-middleware] Running IBM Accessibility scan...\x1b[0m');
+            /**
+             * We run the accessibility checker
+             * 
+             * we scan the output.publicPath if its set and not set to 'auto'
+             * otherwise we scan the output.path
+             */
+            let scanDir = compiler.options.output.publicPath && 'auto' !== compiler.options.output.publicPath ?
+              compiler.options.output.publicPath :
+              compiler.options.output.path;
+
+            this.a11yCheck(
+              scanDir,
               this.config 
             );
 
-            // if( result ){
-            //   // we have to inject the a11y.update.js file into the head in order for the webpack-dev-server scripts to load.
-            //   // let pageContent = fs.readFileSync(path.join(staticDir.directory, `${this.config.outputFilename}.html`))
-              
-            //   // fs.writeFileSync(
-            //   //   path.join(staticDir.directory, `${this.config.outputFilename}.html`),
-            //   //   pageContent.toString().replace('</head>', `<script src="./a11y.update.js"></script>\n</head>`)
-            //   // )
-            // }
-
-            console.log(`<i> ${boldGreen('[webpack-dev-middleware] IBM Accessibilty Report can be viewed at')} ${ boldBlue(new URL(`${auditUrl}${staticDir.publicPath}/${this.config.outputFilename}.html`).toString())  }`);
+            console.log(`<i> \x1b[32m[webpack-dev-middleware] IBM Accessibilty Report can be viewed at \x1b[34m ${new URL(`${auditUrl}${staticDir.publicPath}/${this.config.outputFilename}.html`).toString()}\x1b[0m`);
 
         });
       })
@@ -182,6 +159,7 @@ class A11yPlugin {
     /**
      * Run accessibility checks
      * 
+     * @param {string}  target   The target URL or directory to scan.
      * @param {Object}  options
      * @param {boolean} options.debug            True if debug mode is enabled.
      * @param {boolean} options.ruleArchive   Specify the rule archive.
@@ -194,7 +172,7 @@ class A11yPlugin {
      * @param {boolean} options.outputFolder   Where the scan results should be saved.
      * @param {boolean} options.outputFilenameTimestamp   Should the timestamp be included in the filename of the reports?
      */
-    a11yCheck(url, {
+    a11yCheck( target, {
       debug, 
       ruleArchive,
       policies,
@@ -207,39 +185,38 @@ class A11yPlugin {
       outputFilenameTimestamp
     }){
 
-
-      let htmlOutput = outputFormat && outputFormat.includes('html');
-
-      // we remove the html output since we generate our own html based on the json output.
-      // outputFormat = outputFormat.filter(o => 'html' !== o );
+      // outputFolder should not be absolute
+      outputFolder = path.isAbsolute(outputFolder) ? path.join( process.cwd(), outputFolder ) : outputFolder;
       
+      // accessibility-checker CLI arguments
       let acheckerArgs = [
         '--ruleArchive',
         ruleArchive,
         '--policies',
-        Array.isArray(policies) ? policies.filter(e => e).join(',') : policies,
+        Array.isArray(policies) ? policies.filter(Boolean).join(',') : policies,
         '--failLevels',
-        Array.isArray(failLevels) ? failLevels.filter(e => e).join(',') : failLevels,
+        Array.isArray(failLevels) ? failLevels.filter(Boolean).join(',') : failLevels,
         '--reportLevels',
-        Array.isArray(reportLevels) ? reportLevels.filter(e => e).join(',') : reportLevels,
+        Array.isArray(reportLevels) ? reportLevels.filter(Boolean).join(',') : reportLevels,
         '--outputFolder',
         outputFolder,
         '--outputFormat',
         outputFormat,
         '---outputFilenameTimestamp',
         outputFilenameTimestamp,
-        url
+        target
       ];
     
-      let isValid = fs.existsSync( url ) || 'localhost' === new URL(url).hostname || isUrl( url );
-      let isDirectory = fs.existsSync( url ) && fs.statSync(url).isDirectory();
+      // checks for the target validity
+      let isValid = fs.existsSync( target ) || 'localhost' === new URL(target).hostname || isUrl( target );
+      // let isDirectory = fs.existsSync( target ) && fs.statSync(target).isDirectory();
 
+      // if the target is valid, we run the accessibility checker
       if( isValid ){
        
-        let outputDir = path.resolve('.',  outputFolder );
-
+        // we run the accessibility-checker CLI
         let {stderr, stdout}  = spawn.sync( 
-          path.resolve(currentPath, '..', '..', 'accessibility-checker', 'bin', 'achecker.js'),
+          path.resolve(require.resolve('accessibility-checker'), '..', '..' , 'bin', 'achecker.js'),
           acheckerArgs,
           {
             stdio: 'pipe'
@@ -255,53 +232,39 @@ class A11yPlugin {
 
           // we iterate thru the output directory in reverse order,
           // this way we can remove any empty directories at the end, since files are cycled first.
-          fs.readdirSync(outputDir, {recursive: true}).reverse().forEach( file => {
+          fs.readdirSync(outputFolder, {recursive: true}).reverse().forEach( file => {
+            let filePath = path.join(outputFolder, file);
+
             // process each json file file
-            if( fs.statSync(path.join(outputDir, file)).isFile() ){
+            if( fs.statSync(filePath).isFile() ){
               if ( file.startsWith('summary_') ){
                 // remove the summary files
-                fs.rmSync( path.join(outputDir, file) )
+                fs.rmSync( filePath )
                 return;
               }
-
-              let oldName = file;
-              let newName = file;
-
-              // remove the original output directory from the file name
-              newName = isDirectory ? file.replace(url.replace(':', '_'), '') : file;
               
-              newName = newName.replace(/^\\/, '');
+              // if the fle ends with .json or .html
+              if( file.endsWith('.json') || file.endsWith('.html') ){
+                let newName = file.replace( target.replace(':', '_') + '\\', '').replace('.html.html', '.html');
 
-              // for some reason .html files have an extra .html in the name
-              newName = newName.endsWith('.html.html') ? newName.replace('.html.html', '.html') : newName;
-              
-              // if the new name is not the same as the old name.
-              if( newName !== file ){
-                // rename the file
+                // we rename the json/html file to remove the target from the filename
                 fs.renameSync(
-                  path.join(outputDir, oldName ), 
-                  path.join(outputDir, newName) );
-              }
-
-              // we add the file to the audit index
-              if( ! auditIndex.includes(newName) && newName.endsWith('.html') ){
-                auditIndex.push( newName );
-              }
-              
-              // if we are generating html output, we need to generate the html file.
-              if( htmlOutput ){
-                // let jsonObj = JSON.parse(fs.readFileSync(path.join(outputDir, newName)));
-
-                // we generate the html file
-                // reporter( jsonObj, { outputFolder, outputFilename: newName.replace(/\.json$/, '') } );
+                  path.join(outputFolder, file),
+                  path.join(outputFolder, newName)
+                );
+
+                // we add the file to the audit index
+                if( file.endsWith('.html') && ! auditIndex.includes(newName) ){
+                  auditIndex.push( newName );
+                }
               }
             
-            }else if ( fs.statSync(path.join(outputDir, file)).isDirectory() ){
+            }else if ( fs.statSync(path.join(outputFolder, file)).isDirectory() ){
               // process each directory
               // delete any empty directories.
-              if( 0 === fs.readdirSync(path.join(outputDir, file)).length  ){
+              if( 0 === fs.readdirSync(path.join(outputFolder, file)).length  ){
                 // remove the directory
-                fs.rmSync(path.join(outputDir, file), {recursive: true});
+                fs.rmSync(path.join(outputFolder, file), {recursive: true});
               }
             }
 
@@ -312,14 +275,13 @@ class A11yPlugin {
 
           // we generate the .
           if( 'a11y' === process.argv[2] ){
-              // console.log( reportedFile )
+              console.log( 'Accessibility Report Generated File: ', path.join(outputFolder, `${outputFilename}.html`) )
           }else{
             return true;
-              // return reportedFile;
           }
         }
       }else{
-        console.log( `${url} is not a valid url.` )
+        console.log( `${target} is not a valid url or directory.` )
       }
 
     } // end of a11yCheck
@@ -327,4 +289,4 @@ class A11yPlugin {
 } // end of class
   
 
-export default A11yPlugin;
+export default CAWebA11yWebpackPlugin;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@caweb/a11y-webpack-plugin",
-  "version": "2.0.1",
+  "version": "2.0.3",
   "description": "CAWebPublishing Webpack Plugin to run Accessibility Scans",
   "type": "module",
   "main": "index.js",
@@ -11,6 +11,8 @@
     "README.md"
   ],
   "scripts": {
+    "webpack": "webpack",
+    "serve": "webpack serve --config ./node_modules/@caweb/webpack/webpack.config.js ./tests/webpack.tests.js --merge",
     "test": "echo \"Error: run tests from root\" && exit 0"
   },
   "author": "CAWebPublishing",
@@ -33,14 +35,12 @@
     "webpack"
   ],
   "dependencies": {
-    "accessibility-checker": "^4.0.7",
+    "accessibility-checker": "^4.0.10",
     "check-valid-url": "^0.1.0"
   },
   "devDependencies": {
-    "webpack": "^5.101.0",
+    "@caweb/html-webpack-plugin": "^2.0.2",
+    "webpack": "^5.104.1",
     "webpack-cli": "^6.0.1"
-  },
-  "peerDependencies": {
-    "@caweb/template": "^1.0.2"
   }
 }

package/reporter.js

@@ -173,7 +173,7 @@ function initHandleBars(){
   // Register custom helpers.
   HandleBars.registerHelper('endsWith', endsWith )
 
-  return HandleBars.compile(fs.readFileSync(path.resolve(templateDir, 'patterns', 'index.html')).toString() )
+  return HandleBars.compile(fs.readFileSync(path.resolve(templateDir, 'patterns', 'default.html')).toString() )
 
 }