dslinter 0.0.16 → 0.0.27

package/CHANGELOG.md

@@ -1,5 +1,104 @@
 # Changelog
 
+## v0.0.27
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.26...v0.0.27)
+
+### 🩹 Fixes
+
+- **ci:** Prepublish skip optional npm publish ([b69b368](https://github.com/jrmybtlr/DSLinter/commit/b69b368))
+- **ci:** Disable npm provenance on publish (retry-safe) ([5aaece8](https://github.com/jrmybtlr/DSLinter/commit/5aaece8))
+
+### ❤️ Contributors
+
+- Jeremy Butler <jeremy.butler@laravel.com>
+
+## v0.0.26
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.25...v0.0.26)
+
+### 🩹 Fixes
+
+- **ci:** Skip gh-release in napi prepublish ([cdcda0e](https://github.com/jrmybtlr/DSLinter/commit/cdcda0e))
+
+### ❤️ Contributors
+
+- Jeremy Butler <jeremy.butler@laravel.com>
+
+## v0.0.25
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.24...v0.0.25)
+
+### 🩹 Fixes
+
+- **ci:** Single macos job + mold on publish ([b2a193d](https://github.com/jrmybtlr/DSLinter/commit/b2a193d))
+
+### ❤️ Contributors
+
+- Jeremy Butler <jeremy.butler@laravel.com>
+
+## v0.0.24
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.23...v0.0.24)
+
+### 🩹 Fixes
+
+- **ci:** Skip lifecycle scripts and install mold for Rust job ([e85d8d9](https://github.com/jrmybtlr/DSLinter/commit/e85d8d9))
+- **ci:** Build x86_64-apple-darwin on macos-latest ([bcc1af8](https://github.com/jrmybtlr/DSLinter/commit/bcc1af8))
+
+### ❤️ Contributors
+
+- Jeremy Butler <jeremy.butler@laravel.com>
+
+## v0.0.23
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.22...v0.0.23)
+
+## v0.0.22
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.21...v0.0.22)
+
+## v0.0.21
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.19...v0.0.21)
+
+### 🏡 Chore
+
+- **release:** V0.0.20 ([c07bb96](https://github.com/jrmybtlr/DSLinter/commit/c07bb96))
+
+### ❤️ Contributors
+
+- Jeremy Butler <jeremy.butler@laravel.com>
+
+## v0.0.20
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.19...v0.0.20)
+
+## v0.0.19
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.18...v0.0.19)
+
+## v0.0.18
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.17...v0.0.18)
+
+## v0.0.17
+
+[compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.16...v0.0.17)
+
+### 🚀 Enhancements
+
+- Refactor release process and update workflows ([20af684](https://github.com/jrmybtlr/DSLinter/commit/20af684))
+- Add contributing guidelines and enhance documentation ([ac14afc](https://github.com/jrmybtlr/DSLinter/commit/ac14afc))
+
+### 🏡 Chore
+
+- Update Node version and GitHub Actions workflows ([612a9b0](https://github.com/jrmybtlr/DSLinter/commit/612a9b0))
+
+### ❤️ Contributors
+
+- Jeremy Butler <jeremy.butler@laravel.com>
+
 ## v0.0.16
 
 [compare changes](https://github.com/jrmybtlr/DSLinter/compare/v0.0.15...v0.0.16)

package/README.md

@@ -1,6 +1,6 @@
 # dslinter
 
-React UI for the **DSLinter dashboard**: component playground shell, token wall, and governance panels. It expects a **`dslint-report.json`** file produced by the **`dslint`** CLI (Rust scanner in this repo).
+React UI for the **DSLinter dashboard**: component playground shell, token wall, and governance panels. It expects a **`dslint-report.json`** file produced by the **`dslinter`** CLI (Rust scanner powered by [Oxc](https://oxc.rs) in this repo).
 
 **Previously published as `@dslinter/dashboard`.** Migrate with `npm install dslinter` and replace imports `@dslinter/dashboard` → `dslinter`, and `@dslinter/dashboard/theme.css` → `dslinter/theme.css`.
 
@@ -19,56 +19,27 @@ This package is **source-first**: entry points resolve to TypeScript/TSX under `
 
 ## CLI (`npx dslinter`)
 
-The **`dslinter` binary** runs the **`dslint`** scanner with the same flags as the Rust CLI (`--json`, `-o`, `--serve`, etc.).
+The **`dslinter`** command runs the design-system scanner with the same flags as the Rust CLI (`--json`, `-o`, `--serve`, etc.) via a **napi-rs** native binding (same distribution model as **`oxlint`**).
 
 ### Without installing Rust
 
-On **`npm install dslinter`**, a **`postinstall`** script tries to download a **prebuilt `dslint`** for your OS/arch from this repo’s **GitHub Releases**, using the **same tag as the npm version** (for example npm `dslinter@0.0.6` → release **`v0.0.6`** and assets like `dslint-x86_64-unknown-linux-gnu`). The binary is stored under `node_modules/dslinter/vendor/` and `dslinter` / `npx dslinter` prefer it over `PATH`.
-
-**Release workflow:** push git tag `v*` (after bumping the npm version) so [.github/workflows/release-dslint-binaries.yml](https://github.com/jrmybtlr/DSLinter/blob/main/.github/workflows/release-dslint-binaries.yml) uploads the platform binaries, **then** publish `dslinter` to npm (or publish after the workflow finishes so installs resolve the assets).
-
-Environment variables:
+On **`npm install dslinter`**, npm installs the platform **`@dslinter/binding-*`** optional dependency (darwin/linux/windows). No postinstall download or GitHub Releases API is required.
 
 | Variable | Purpose |
 |----------|---------|
-| `DSLINT_SKIP_DOWNLOAD=1` | Skip postinstall download (air-gapped / you only use `PATH`). |
-| `DSLINT_RELEASE_TAG` | Override release tag (default `v` + `dslinter` version from `package.json`). |
-| `DSLINT_GITHUB_REPO` | Override `owner/repo` for downloads (default from `package.json` → `jrmybtlr/DSLinter`). |
-| `DSLINT_VERBOSE=1` | Log which GitHub releases/assets were tried when downloading. |
-| `GITHUB_TOKEN` / `GH_TOKEN` | **Required for private repos** (`jrmybtlr/DSLinter`). Token needs read access to releases. |
-
-### How this differs from `oxlint`
-
-**`oxlint`** on npm ships **Node native addons** as **`optionalDependencies`** (`@oxlint/binding-darwin-arm64`, …) built with **napi-rs** — each package is a small prebuilt library loaded by Node.
-
-**`dslinter`** is a **standalone executable**. The practical pattern here is **download on install** from **GitHub Releases** (similar in spirit to tools that pull a platform binary once), instead of publishing dozens of `@dslinter/binding-*` packages.
+| `DSLINT_BIN` | Use a cargo-built `dslinter` binary instead of the NAPI binding. |
+| `DSLINT_ALLOW_PATH=1` | Allow `dslinter` on `PATH` when the binding is missing. |
+| `NAPI_RS_NATIVE_LIBRARY_PATH` | Point at a specific `.node` file (napi-rs escape hatch). |
 
 ### Do not `cargo install dslint`
 
-The crates.io crate **`dslint`** (v0.0.x) is a **different project** (design-file linting). It is **not** this design-system scanner. Installing it will break `npx dslinter` if it ends up on your `PATH`.
-
-Use **`cargo install --git https://github.com/jrmybtlr/DSLinter dslinter --locked`** or set **`DSLINT_BIN`** to a local `target/release/dslinter` build.
-
-### If there is no matching release asset yet
-
-You’ll see a **warning** during install (install still succeeds). **`dslinter`** will try to download on first run; if no GitHub release exists yet, you get a clear error (not a silent fallback to the wrong `dslint` on crates.io):
-
-```bash
-npx dslinter /path/to/repo --json -o dslint-report.json
-```
-
-| Distribution | How users get the scanner |
-|--------------|-------------------------|
-| **npm + GitHub Releases** | Default: download when release `vX.Y.Z` includes your platform asset. |
-| **GitHub Releases** | Manual download of `dslinter-*` from the release; run directly or set `DSLINT_BIN`. |
-| **From source** | `cargo install --git https://github.com/jrmybtlr/DSLinter dslinter --locked` (not `cargo install dslint`). |
+The crates.io crate **`dslint`** is a **different project**. Use **`cargo install --git https://github.com/jrmybtlr/DSLinter dslinter --locked`** or **`DSLINT_BIN`** for local Rust builds.
 
 Typical usage:
 
 ```bash
 dslinter /path/to/repo --json -o dslint-report.json
 # or --serve for live reload while developing a dashboard
-# (npm `npx dslinter` runs the same binary)
 ```
 
 ## Styles (Tailwind v4)

package/bin/dslinter.mjs

@@ -1,72 +1,77 @@
 #!/usr/bin/env node
 /**
- * Runs the design-system scanner (`dslinter` binary from GitHub Releases or DSLINT_BIN).
- * Does NOT use bare `dslint` on PATH — that name is a different crate on crates.io.
+ * Runs the design-system scanner via the NAPI native binding (oxlint-style).
+ * Falls back to DSLINT_BIN (cargo-built binary) when set.
  */
 import { spawnSync } from "node:child_process";
+import { createRequire } from "node:module";
 import { existsSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
-import { ensureDslintBinary } from "../scripts/ensure-dslint.mjs";
-import {
-  SCANNER_VERSION_MARKER,
-  vendorBinaryPath,
-} from "../scripts/resolve-dslint-binary.mjs";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const packageRoot = join(__dirname, "..");
+const require = createRequire(import.meta.url);
 
 const args = process.argv.slice(2);
 
+const SCANNER_VERSION_MARKER = "design system linting";
+
 function isOurScanner(binary) {
   const help = spawnSync(binary, ["--help"], { encoding: "utf8" });
   const out = `${help.stdout ?? ""}${help.stderr ?? ""}`;
   return out.includes(SCANNER_VERSION_MARKER);
 }
 
-async function resolveCommand() {
-  const fromEnv = process.env.DSLINT_BIN?.trim();
-  if (fromEnv) {
-    if (!existsSync(fromEnv)) {
-      process.stderr.write(`dslinter: DSLINT_BIN not found: ${fromEnv}\n`);
-      process.exit(127);
-    }
-    if (!isOurScanner(fromEnv)) {
-      process.stderr.write(
-        `dslinter: DSLINT_BIN does not look like the DSLint design-system scanner.\n` +
-          `  Expected output containing "${SCANNER_VERSION_MARKER}".\n`,
-      );
-      process.exit(1);
-    }
-    return fromEnv;
+function runViaNapi() {
+  let runCli;
+  try {
+    ({ runCli } = require(join(packageRoot, "index.cjs")));
+  } catch (err) {
+    process.stderr.write(
+      `dslinter: failed to load native binding.\n` +
+        `  Run \`pnpm --filter dslinter run build:napi\` from the repo root, or reinstall dslinter.\n` +
+        `  ${err instanceof Error ? err.message : err}\n`,
+    );
+    process.exit(127);
   }
 
-  const vendored = vendorBinaryPath(packageRoot);
-  if (!existsSync(vendored)) {
-    await ensureDslintBinary(packageRoot, { quiet: false });
-  }
-  if (existsSync(vendored)) {
-    return vendored;
-  }
+  const argv = ["dslinter", ...args];
+  const code = runCli(argv);
+  process.exit(code);
+}
 
-  if (process.env.DSLINT_ALLOW_PATH === "1") {
-    const onPath = spawnSync("dslinter", ["--version"], { encoding: "utf8" });
-    if (onPath.status === 0 && `${onPath.stdout}`.includes(SCANNER_VERSION_MARKER)) {
-      return "dslinter";
-    }
+function runViaBinary(binary) {
+  const child = spawnSync(binary, args, { stdio: "inherit" });
+  if (child.error && "code" in child.error && child.error.code === "ENOENT") {
+    process.stderr.write(`dslinter: failed to execute ${binary}\n`);
+    process.exit(127);
   }
-
-  await import("../scripts/print-missing-scanner.mjs");
-  process.exit(127);
+  process.exit(child.status === null ? 1 : child.status);
 }
 
-const cmd = await resolveCommand();
-const child = spawnSync(cmd, args, { stdio: "inherit" });
+const fromEnv = process.env.DSLINT_BIN?.trim();
+if (fromEnv) {
+  if (!existsSync(fromEnv)) {
+    process.stderr.write(`dslinter: DSLINT_BIN not found: ${fromEnv}\n`);
+    process.exit(127);
+  }
+  if (!isOurScanner(fromEnv)) {
+    process.stderr.write(
+      `dslinter: DSLINT_BIN does not look like the DSLint design-system scanner.\n` +
+        `  Expected output containing "${SCANNER_VERSION_MARKER}".\n`,
+    );
+    process.exit(1);
+  }
+  runViaBinary(fromEnv);
+}
 
-if (child.error && "code" in child.error && child.error.code === "ENOENT") {
-  process.stderr.write(`dslinter: failed to execute ${cmd}\n`);
-  process.exit(127);
+if (process.env.DSLINT_ALLOW_PATH === "1") {
+  const onPath = spawnSync("dslinter", ["--help"], { encoding: "utf8" });
+  const out = `${onPath.stdout ?? ""}${onPath.stderr ?? ""}`;
+  if (onPath.status === 0 && out.includes(SCANNER_VERSION_MARKER)) {
+    runViaBinary("dslinter");
+  }
 }
 
-const code = child.status === null ? 1 : child.status;
-process.exit(code);
+runViaNapi();