Separate verification from test scripts.

feimosi · feimosi · commit 15820e5455c7 · 2026-03-09T23:55:20.000+01:00
Make lint, typecheck, and build explicit CI steps while keeping npm test as the standard entrypoint by delegating it to the Playwright suite.

Made-with: Cursor
diff --git a/.github/workflows/ui-tests.yml b/.github/workflows/ui-tests.yml
@@ -1,4 +1,4 @@
-name: UI Tests
+name: CI
 
 on:
   push:
@@ -8,7 +8,7 @@ permissions:
   contents: read
 
 jobs:
-  playwright:
+  verify:
     runs-on: macos-14
 
     steps:
@@ -28,11 +28,17 @@ jobs:
       - name: Install dependencies
         run: npm ci
 
-      - name: Run project verification
-        run: npm test
+      - name: Run ESLint
+        run: npm run lint
+
+      - name: Run TypeScript typecheck
+        run: npm run typecheck
+
+      - name: Build distributable assets
+        run: npm run build
 
       - name: Run Playwright UI tests
-        run: npm run test:ui
+        run: npm test
         env:
           CHROME_PATH: ${{ steps.chrome.outputs.chrome-path }}
 
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -18,7 +18,7 @@ Remember the script's core philosophy is to stay simple and minimal, doing one t
 
 ## Before you open a pull request
 - Follow Git best practices (especially use meaningful commit messages).
-- Run `npm test` and `npm run test:ui`.
+- Run `npm run lint`, `npm run typecheck`, `npm run build`, and `npm test`.
 - Describe thoroughly your work in a PR comment.
 - Be patient and understanding. It's a side project, done in free time.
 
@@ -32,9 +32,10 @@ Compatible node version: v22
 
 Build the script
 ```sh
+npm run lint
+npm run typecheck
 npm run build
 npm test
-npm run test:ui
 git add --update
 git commit -m "Build update"
 ```
diff --git a/README.md b/README.md
@@ -267,28 +267,30 @@ The repository includes the legacy Gulp build, standalone linting, strict TypeSc
 ### Run the full local verification flow
 
 ```sh
+npm run lint
+npm run typecheck
+npm run build
 npm test
 ```
 
-This runs:
-- `npm run build`
-- `npm run lint`
-- `npm run typecheck`
+`npm test` now delegates to `npm run test:ui`. Build, lint, and typecheck are intentionally separate commands.
 
 ### Run the UI test suite
 
 ```sh
 npm run test:ui
 ```
 
-This rebuilds the distributable assets first, then runs Playwright against the packaged minified CSS and JS files.
+This runs the Playwright suite against the packaged minified CSS and JS files. Run `npm run build` first if you want the tests to reflect fresh source changes.
 
 ### Update snapshot baselines
 
 ```sh
 npm run test:ui:update
 ```
 
+Run `npm run build` first so snapshot updates match the current built assets.
+
 Notes:
 - The Playwright config uses the local Google Chrome browser on macOS.
 - Snapshot files live in `tests/ui/smoke.spec.js-snapshots/`.
diff --git a/dist/baguetteBox.css b/dist/baguetteBox.css
@@ -1,7 +1,7 @@
 /*!
  * baguetteBox.js
  * @author  feimosi
- * @version %%INJECT_VERSION%%
+ * @version 1.13.0
  * @url https://github.com/feimosi/baguetteBox.js
  */
 
diff --git a/dist/baguetteBox.js b/dist/baguetteBox.js
@@ -2,7 +2,7 @@
 /*!
  * baguetteBox.js
  * @author  feimosi
- * @version %%INJECT_VERSION%%
+ * @version 1.13.0
  * @url https://github.com/feimosi/baguetteBox.js
  */
 (function (root, factory) {
diff --git a/dist/baguetteBox.min.css b/dist/baguetteBox.min.css
diff --git a/dist/baguetteBox.min.js b/dist/baguetteBox.min.js
diff --git a/package.json b/package.json
@@ -22,9 +22,9 @@
         "start": "gulp",
         "lint": "eslint src/**/*.ts gulpfile.js playwright.config.js eslint.config.js tests/**/*.js",
         "typecheck": "tsc --project tsconfig.json",
-        "test": "npm run build && npm run lint && npm run typecheck",
-        "test:ui": "npm run build && playwright test",
-        "test:ui:update": "npm run build && playwright test --update-snapshots",
+        "test": "npm run test:ui",
+        "test:ui": "playwright test",
+        "test:ui:update": "playwright test --update-snapshots",
         "build": "gulp build",
         "release": "gulp release",
         "patch": "gulp patch"
