skyltmax_config 1.0.0 → 2.0.0.beta.1

data/pnpm-workspace.yaml

@@ -0,0 +1,3 @@
+allowBuilds:
+  esbuild: true
+  unrs-resolver: true

data/prettier.js

@@ -46,7 +46,9 @@ export const config = {
       },
     },
   ],
-  plugins: ["prettier-plugin-ignored", "prettier-plugin-tailwindcss"],
+  // 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")],
   tailwindAttributes: ["class", "className", ".*[cC]lassName"],
   tailwindFunctions: ["clsx", "cn", "twcn"],
 }

data/tests/configs.test.js

@@ -0,0 +1,47 @@
+import { describe, expect, test } from "vitest"
+
+import { config as eslintConfig } from "../eslint.js"
+import { config as prettierConfig } from "../prettier.js"
+
+describe("prettier config", () => {
+  test("plugins are resolved to URLs inside this package, not bare names", async () => {
+    expect(prettierConfig.plugins).toHaveLength(2)
+
+    for (const plugin of prettierConfig.plugins) {
+      expect(plugin).toMatch(/^file:\/\//)
+      // a bare name would make prettier resolve from the consumer's context; a URL is import()-able from anywhere
+      await expect(import(plugin)).resolves.toBeDefined()
+    }
+  })
+})
+
+// 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 = [
+  "@typescript-eslint",
+  "import",
+  "jest-dom",
+  "jsx-a11y",
+  "prettier",
+  "react",
+  "react-hooks",
+  "testing-library",
+  "vitest",
+]
+
+describe("eslint config", () => {
+  test("exports a flat config array with every plugin loaded from our own dependencies", () => {
+    expect(Array.isArray(eslintConfig)).toBe(true)
+    expect(eslintConfig.length).toBeGreaterThan(0)
+
+    const plugins = new Map(eslintConfig.flatMap(entry => Object.entries(entry.plugins ?? {})))
+
+    for (const expected of EXPECTED_ESLINT_PLUGINS) {
+      // The value, not just the key: a failed import leaves the name in place with nothing behind it.
+      expect(plugins.get(expected), `plugin "${expected}" should be loaded`).toBeInstanceOf(Object)
+    }
+
+    // Exact roster, so adding or dropping a plugin has to be a deliberate edit to the list above.
+    expect([...plugins.keys()].sort()).toEqual(EXPECTED_ESLINT_PLUGINS)
+  })
+})

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: skyltmax_config
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 2.0.0.beta.1
 platform: ruby
 authors:
 - Signmax AB
@@ -95,7 +95,6 @@ files:
 - ".gitignore"
 - ".prettierignore"
 - ".rubocop.yml"
-- AGENTS.md
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock
@@ -108,6 +107,7 @@ files:
 - fixture/eslint.config.js
 - fixture/package.json
 - fixture/pnpm-lock.yaml
+- fixture/pnpm-workspace.yaml
 - fixture/src/index.ts
 - fixture/src/ui/button.tsx
 - fixture/tsconfig.json
@@ -116,17 +116,13 @@ files:
 - lib/skyltmax_config/version.rb
 - package.json
 - pnpm-lock.yaml
+- pnpm-workspace.yaml
 - prettier.js
 - reset.d.ts
 - rubocop.rails.yml
 - rubocop.yml
-- scripts/peer-deps/audit.js
-- scripts/peer-deps/install.js
 - skyltmax_config.gemspec
-- tests/check-peers.test.js
-- tests/fixtures/manifest-no-peers.json
-- tests/fixtures/manifest-with-peers.json
-- tests/install-peers.test.js
+- tests/configs.test.js
 - tsconfig.json
 - typescript.json
 homepage: https://github.com/skyltmax/config