skyltmax_config 2.0.0 → 3.0.0

data/prettier-ignored-plugin.js

@@ -0,0 +1,33 @@
+// Vendored replacement for `prettier-plugin-ignored` (1.0.0 — one release ever, 2024-08-30, single maintainer). The
+// package was 20 lines of no-op parser/printer; owning them here removes a registry dependency from every consumer's
+// tree without changing behaviour.
+//
+// Why the shape exists at all: prettier has no shareable ignore mechanism. `.prettierignore`, `--ignore-path` and
+// `.gitignore` all live in the consumer's repo, so a shared config can only exclude a file by formatting it to exactly
+// what it already was. The parser hands the source through untouched and the printer prints it back verbatim; an
+// `overrides` entry in `prettier.js` opts a glob in via `parser: "ignored"`.
+
+/** @type {import("prettier").SupportLanguage[]} */
+export const languages = [
+  {
+    name: "ignored",
+    parsers: ["ignored"],
+  },
+]
+
+/** @type {Record<string, import("prettier").Parser<string>>} */
+export const parsers = {
+  ignored: {
+    parse: source => source,
+    astFormat: "ignored-ast",
+    locStart: () => 0,
+    locEnd: node => node.length,
+  },
+}
+
+/** @type {Record<string, import("prettier").Printer<string>>} */
+export const printers = {
+  "ignored-ast": {
+    print: path => path.node ?? "",
+  },
+}

data/prettier.js

@@ -46,9 +46,10 @@ export const config = {
       },
     },
   ],
-  // Resolved from this package's own dependencies. Bare names would be resolved by prettier's import() from the
-  // consumer's context, which is exactly the phantom-dependency setup v2 removes.
-  plugins: [import.meta.resolve("prettier-plugin-ignored"), import.meta.resolve("prettier-plugin-tailwindcss")],
+  // Resolved from this package itself — the `ignored` parser is vendored here, tailwindcss comes from our own
+  // dependencies. Bare names would be resolved by prettier's import() from the consumer's context, which is exactly
+  // the phantom-dependency setup v2 removes.
+  plugins: [import.meta.resolve("./prettier-ignored-plugin.js"), import.meta.resolve("prettier-plugin-tailwindcss")],
   tailwindAttributes: ["class", "className", ".*[cC]lassName"],
   tailwindFunctions: ["clsx", "cn", "twcn"],
 }

data/reset.d.ts

@@ -1,5 +1,6 @@
+// `/dom` is the whole `recommended` set (filter-boolean, json-parse, is-array, includes/has/indexOf, promise-catch,
+// map-constructor) plus the Storage overrides — importing any of those entrypoints separately is a no-op.
 import "@total-typescript/ts-reset/dom"
-import "@total-typescript/ts-reset/filter-boolean"
 
 import "react"
 

data/tests/configs.test.js

@@ -1,3 +1,4 @@
+import { format } from "prettier"
 import { describe, expect, test } from "vitest"
 
 import { config as eslintConfig } from "../eslint.js"
@@ -13,17 +14,33 @@ describe("prettier config", () => {
       await expect(import(plugin)).resolves.toBeDefined()
     }
   })
+
+  // The vendored `ignored` parser (prettier-ignored-plugin.js) is how the shared config excludes files without a
+  // .prettierignore in the consumer's repo — it has to return the source byte for byte, whitespace included.
+  test("the ignored parser formats a file to exactly its input", async () => {
+    const source = "packages:\n  - 'apps/*'\n\n\n   trailing spaces   \n"
+
+    await expect(format(source, { parser: "ignored", plugins: prettierConfig.plugins })).resolves.toBe(source)
+  })
+
+  test("the pnpm manifests are the globs routed to the ignored parser", () => {
+    const ignoredGlobs = prettierConfig.overrides
+      .filter(override => override.options.parser === "ignored")
+      .flatMap(override => override.files)
+
+    expect(ignoredGlobs).toEqual(["**/pnpm-lock.yaml", "**/pnpm-workspace.yaml"])
+  })
 })
 
 // Every plugin eslint.js loads, alphabetically. All of them resolve from this package's own dependencies, so a missing
 // entry means a dependency or an import in eslint.js broke — never a consumer misconfiguration.
 const EXPECTED_ESLINT_PLUGINS = [
+  "@eslint-react",
   "@typescript-eslint",
   "import",
   "jest-dom",
   "jsx-a11y",
   "prettier",
-  "react",
   "react-hooks",
   "testing-library",
   "vitest",

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: skyltmax_config
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Signmax AB
@@ -110,6 +110,7 @@ files:
 - fixture/pnpm-workspace.yaml
 - fixture/src/index.ts
 - fixture/src/ui/button.tsx
+- fixture/src/ui/card.tsx
 - fixture/tsconfig.json
 - index.js
 - lib/skyltmax_config.rb
@@ -117,6 +118,7 @@ files:
 - package.json
 - pnpm-lock.yaml
 - pnpm-workspace.yaml
+- prettier-ignored-plugin.js
 - prettier.js
 - reset.d.ts
 - rubocop.rails.yml