fix: only consider errors (excluding warning/info messages)

.vscode/settings.json

@@ -6,5 +6,9 @@
   "editor.formatOnSave": true,
   "editor.codeActionsOnSave": {
     "source.fixAll": true
-  }
+  },
+  "eslint.options": {
+    "ignorePath": ".gitignore"
+  },
+  "prettier.ignorePath": ".gitignore"
 }

README.md

@@ -59,7 +59,7 @@ npm install --save-dev html-w3c-validator start-server-and-test
     "start": "serve \"./build\"",
 
     // Command to validate your HTML pages
-    "test:html-w3c-validator": "start-server-and-test \"start\" \"http://localhost:3000\" \"html-w3c-validator\""
+    "test:html-w3c-validator": "start-server-and-test \"start\" \"http://127.0.0.1:3000\" \"html-w3c-validator\""
   }
 }
 ```
@@ -68,7 +68,7 @@ npm install --save-dev html-w3c-validator start-server-and-test
 
 ```jsonc
 {
-  "urls": ["http://localhost:3000/", "http://localhost:3000/about"],
+  "urls": ["http://127.0.0.1:3000/", "http://127.0.0.1:3000/about"],
 
   // You can also specify HTML files instead of URLs
   "files": ["./index.html", "./about.html"]
@@ -84,8 +84,8 @@ npm run test:html-w3c-validator
 Example of output (in case of success):
 
 ```txt
-✔ Validating http://localhost:3000/
-✔ Validating http://localhost:3000/about
+✔ Validating http://127.0.0.1:3000/
+✔ Validating http://127.0.0.1:3000/about
 
 Success: HTML validation (W3C) passed! 🎉
 ```

example/.html-w3c-validatorrc.json

@@ -1,4 +1,4 @@
 {
-  "urls": ["http://localhost:3000/", "http://localhost:3000/about"],
+  "urls": ["http://127.0.0.1:3000/", "http://127.0.0.1:3000/about"],
   "files": ["./build/index.html", "./build/about.html"]
 }

example/package.json

@@ -3,10 +3,10 @@
   "private": true,
   "scripts": {
     "start": "serve \"./build\"",
-    "test:html-w3c-validator": "start-server-and-test \"start\" \"http://localhost:3000\" \"html-w3c-validator\""
+    "test:html-w3c-validator": "start-server-and-test \"start\" \"http://127.0.0.1:3000\" \"html-w3c-validator\""
   },
   "devDependencies": {
-    "serve": "13.0.2",
+    "serve": "14.0.1",
     "start-server-and-test": "1.14.0",
     "html-w3c-validator": "file:.."
   }

package.json

@@ -44,42 +44,42 @@
     "postpublish": "pinst --enable"
   },
   "dependencies": {
-    "chalk": "5.0.1",
+    "chalk": "5.1.2",
     "clipanion": "3.1.0",
-    "html-validator": "6.0.0",
-    "ora": "6.1.0",
+    "html-validator": "6.0.1",
+    "ora": "6.1.2",
     "read-pkg": "7.1.0",
     "table": "6.8.0"
   },
   "devDependencies": {
-    "@commitlint/cli": "16.2.3",
-    "@commitlint/config-conventional": "16.2.1",
+    "@commitlint/cli": "17.1.2",
+    "@commitlint/config-conventional": "17.1.0",
     "@swc/cli": "0.1.57",
-    "@swc/core": "1.2.163",
-    "@types/html-validator": "5.0.2",
+    "@swc/core": "1.3.9",
+    "@types/html-validator": "5.0.3",
     "@types/mock-fs": "4.13.1",
-    "@types/node": "17.0.23",
-    "@types/tap": "15.0.6",
-    "@typescript-eslint/eslint-plugin": "5.18.0",
+    "@types/node": "18.11.3",
+    "@types/tap": "15.0.7",
+    "@typescript-eslint/eslint-plugin": "5.40.1",
     "editorconfig-checker": "4.0.2",
-    "eslint": "8.12.0",
-    "eslint-config-conventions": "2.0.0",
+    "eslint": "8.25.0",
+    "eslint-config-conventions": "5.0.0",
     "eslint-config-prettier": "8.5.0",
     "eslint-plugin-import": "2.26.0",
-    "eslint-plugin-prettier": "4.0.0",
-    "eslint-plugin-promise": "6.0.0",
-    "eslint-plugin-unicorn": "42.0.0",
+    "eslint-plugin-prettier": "4.2.1",
+    "eslint-plugin-promise": "6.1.1",
+    "eslint-plugin-unicorn": "44.0.2",
     "execa": "6.1.0",
-    "husky": "7.0.4",
-    "lint-staged": "12.3.7",
-    "markdownlint-cli": "0.31.1",
-    "mock-fs": "5.1.2",
+    "husky": "8.0.1",
+    "lint-staged": "13.0.3",
+    "markdownlint-cli": "0.32.2",
+    "mock-fs": "5.1.4",
     "pinst": "3.0.0",
-    "prettier": "2.6.2",
+    "prettier": "2.7.1",
     "rimraf": "3.0.2",
-    "semantic-release": "19.0.2",
-    "serve": "13.0.2",
-    "tap": "16.0.1",
-    "typescript": "4.6.3"
+    "semantic-release": "19.0.5",
+    "serve": "14.0.1",
+    "tap": "16.3.0",
+    "typescript": "4.8.4"
   }
 }

src/HTMLValidatorCommand.ts

@@ -4,10 +4,11 @@ import fs from 'node:fs'
 import { Command } from 'clipanion'
 import chalk from 'chalk'
 import ora from 'ora'
-import validateHTML, {
+import type {
   ValidationMessageLocationObject,
   ParsedJsonAsValidationResults
 } from 'html-validator'
+import validateHTML from 'html-validator'
 import { table } from 'table'
 
 import { isExistingPath } from './utils/isExistingPath.js'
@@ -103,27 +104,27 @@ export class HTMLValidatorCommand extends Command {
           } else {
             throw new Error('Invalid type')
           }
-          const isValidHTML = result.messages.length === 0
-          if (isValidHTML) {
+          const hasErrors = result.messages.some((message) => {
+            return message.type === 'error'
+          })
+          if (!hasErrors) {
             loader.succeed()
           } else {
             loader.fail()
             const messagesTable: string[][] = []
             for (const message of result.messages) {
-              const row: string[] = []
               if (message.type === 'error') {
+                const row: string[] = []
                 row.push(chalk.red(message.type))
-              } else {
-                row.push(chalk.yellow(message.type))
+                row.push(message.message)
+                const violation = message as ValidationMessageLocationObject
+                if (violation.extract != null) {
+                  row.push(
+                    `line: ${violation.lastLine}, column: ${violation.firstColumn}-${violation.lastColumn}`
+                  )
+                }
+                messagesTable.push(row)
               }
-              row.push(message.message)
-              const violation = message as ValidationMessageLocationObject
-              if (violation.extract != null) {
-                row.push(
-                  `line: ${violation.lastLine}, column: ${violation.firstColumn}-${violation.lastColumn}`
-                )
-              }
-              messagesTable.push(row)
             }
             errors.push({ data, messagesTable })
             isValid = false