Merge pull request #84 from aminya/ci

test: add integration tests for the plugins + fix: update dependencies

Author: aminya

.github/workflows/CI.yml

@@ -3,7 +3,6 @@ on:
   pull_request:
   push:
     branches:
-      - master
       - main
 
 jobs:
@@ -15,22 +14,12 @@ jobs:
       matrix:
         os:
           - ubuntu-latest
-          # - macos-latest
-          # - windows-latest
-        node:
-          - 20
-        pnpm:
-          - 8
     steps:
       - uses: actions/checkout@v4
-
       - uses: pnpm/action-setup@v4
-        with:
-          version: ${{ matrix.pnpm }}
-
       - uses: actions/setup-node@v4
         with:
-          node-version: ${{ matrix.node }}
+          node-version-file: ".nvmrc"
           cache: "pnpm"
 
       - name: Install and build

.nvmrc

@@ -0,0 +1 @@
+22.14.0

cspell.config.yaml

@@ -12,10 +12,12 @@ words:
   - applellvm
   - Astrojs
   - buildtools
+  - Cascadia
   - caxa
   - ccache
   - choco
   - cmake
+  - Consolas
   - CPATH
   - Cppcheck
   - CPPFLAGS
@@ -44,16 +46,19 @@ words:
   - msys
   - multilib
   - mxschmitt
+  - Neue
   - noconfirm
   - noprogressbar
   - nothrow
   - npmrc
+  - nvmrc
   - Opencppcoverage
   - OSSDK
   - parceljs
   - pnpm
   - pwsh
   - pypy
+  - Roboto
   - setupcpp
   - setx
   - Syuu

package.json

@@ -11,32 +11,35 @@
   "scripts": {
     "build": "pnpm -r run build",
     "bump": "pnpm exec ncu -u && pnpm -r exec ncu -u && typesync --ignoredeps=dev && pnpm update",
-    "clean": "shx rm -rf ./packages/*/dist/",
+    "clean": "shx rm -rf ./packages/*/dist/ ./test/dist/ ./test/.astro",
     "doc": "shx rm -rf packages/*/README.md && pnpm -r exec readme --path ../../dev/readme/template.md -y && pnpm -r exec ts-readme",
     "format": "prettier --write .",
     "lint": "eslint . --fix",
     "prepare": "pnpm run build",
     "publish": "pnpm -r publish",
-    "test": "run-p test.cspell test.lint",
+    "test": "run-p test.cspell test.lint && run-s test.posthtml test.htmlnano",
     "test.cspell": "cspell lint --no-progress --show-suggestions",
     "test.format": "prettier . --check",
-    "test.lint": "eslint ."
+    "test.htmlnano": "run-s clean build && cd test && cross-env NODE_ENV=production TEST=htmlnano pnpm build",
+    "test.lint": "eslint .",
+    "test.posthtml": "run-s clean build && cd test && cross-env NODE_ENV=production TEST=posthtml pnpm build"
   },
   "prettier": "prettier-config-atomic",
   "devDependencies": {
-    "@types/node": "^20.11.30",
-    "cspell": "^8.6.0",
-    "eslint": "^8.57.0",
-    "eslint-config-atomic": "^1.21.2",
-    "npm-check-updates": "^16.14.17",
-    "npm-run-all2": "^6.1.2",
-    "prettier": "3.2.5",
-    "prettier-config-atomic": "^4.0.0",
-    "readme-md-generator": "^1.0.0",
-    "shx": "^0.3.4",
-    "ts-readme": "^1.1.3",
-    "typescript": "^5.4.3",
-    "typesync": "^0.12.1"
+    "@types/node": "22.13.14",
+    "cross-env": "^7.0.3",
+    "cspell": "8.18.0",
+    "eslint": "^8",
+    "eslint-config-atomic": "1.22.1",
+    "npm-check-updates": "17.1.16",
+    "npm-run-all2": "7.0.2",
+    "prettier": "3.5.3",
+    "prettier-config-atomic": "4.0.0",
+    "readme-md-generator": "1.0.0",
+    "shx": "0.4.0",
+    "ts-readme": "1.1.3",
+    "typescript": "5.8.2",
+    "typesync": "0.14.1"
   },
   "engines": {
     "node": ">=16"
@@ -60,7 +63,7 @@
     "relative-attribute",
     "attribute"
   ],
-  "packageManager": "pnpm@9.12.1",
+  "packageManager": "pnpm@10.7.0",
   "packages": [
     "./packages/*"
   ]

packages/astro-htmlnano/package.json

@@ -14,20 +14,20 @@
     "prepublishOnly": "pnpm run build"
   },
   "dependencies": {
-    "astro-posthtml": "workspace:*",
-    "cssnano": "^7.0.0",
-    "htmlnano": "2.1.1",
-    "postcss": "^8.4.38",
+    "astro-posthtml": "^1.1.3",
+    "cssnano": "^7.0.6",
+    "htmlnano": "^2.1.1",
+    "postcss": "^8.5.3",
     "relateurl": "^0.2.7",
     "srcset": "^5.0.1",
-    "svgo": "^3.2.0",
-    "terser": "^5.29.2"
+    "svgo": "^3.3.2",
+    "terser": "^5.39.0"
   },
   "devDependencies": {
     "@types/relateurl": "^0.2.33"
   },
   "peerDependencies": {
-    "astro": "^4 || ^5.0.0"
+    "astro": "^4 || ^5"
   },
   "engines": {
     "node": ">=16.x"

packages/astro-htmlnano/src/index.ts

@@ -24,6 +24,7 @@ export function getAstroHTMLNano(options?: HtmlnanoOptions, preset?: Presets[key
       const htmlnanoOptions: HtmlnanoOptions = options ?? {}
 
       // disable `removeComments` due to Astro's use of comments for hydration
+      // eslint-disable-next-line @typescript-eslint/strict-boolean-expressions
       if (htmlnanoOptions.removeComments) {
         console.warn("`removeComments` is disabled due to Astro's use of comments for hydration.")
       }

packages/astro-posthtml/src/index.ts

@@ -19,7 +19,7 @@ import posthtml, { type Options, type Plugin } from "posthtml"
  */
 export function getAstroPostHTML<PostHTMLUseThis, PostHTMLMessage>(
   plugins?: Plugin<PostHTMLUseThis>[],
-  options?: Options
+  options?: Options,
 ) {
   return async (_context: APIContext, next: MiddlewareNext) => {
     const response = await next()
@@ -33,7 +33,7 @@ export function getAstroPostHTML<PostHTMLUseThis, PostHTMLMessage>(
       // modify HTML using posthtml
       const { html: modifiedHTML, messages } = await posthtml<PostHTMLUseThis, PostHTMLMessage>(plugins).process(
         originalHTML,
-        options
+        options,
       )
 
       // log posthtml messages