@rootreeweb/linting 5.2.0 → 5.4.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @rootreeweb/linting
 
+## 5.4.0
+
+### Minor Changes
+
+- Add `presetNetSuite`, and ship agent skills (format/lint and a SOLID audit) for `skills experimental_sync`.
+
+## 5.3.0
+
+### Minor Changes
+
+- Add a preset for packages run directly by node.
+
 ## 5.2.0
 
 ### Minor Changes

package/README.md

@@ -29,6 +29,7 @@ Available presets:
 - presetBase
 - presetReact
 - presetNext
+- presetNetSuite (SuiteScript globals and `@N*` JSDoc tags)
 
 ```javascript
 import { presetBase } from "@rootreeweb/linting";
@@ -110,6 +111,20 @@ import { composeConfig, presetNext, turbo } from "@rootreeweb/linting";
 export default composeConfig(presetNext, turbo);
 ```
 
+## Agent Skills
+
+This package ships agent skills (`format-and-lint-js` and `solid-audit`, plus `domain-modeling`, `grill-me`, `improve-codebase-architecture` and `dry-refactoring` from upstream) using the [`skills`](https://www.npmjs.com/package/skills) CLI's npm sync.
+
+```bash
+yarn add -D skills
+npm pkg set scripts.prepare="husky && skills experimental_sync -y"
+yarn skills experimental_sync -y
+```
+
+This links the skills into `.agents/skills` (and `.claude/skills` for Claude Code).
+`dry-refactoring` expects [`jscpd`](https://www.npmjs.com/package/jscpd) to be installed.
+`experimental_sync` is experimental in the `skills` CLI, so its flags may change.
+
 ## Stylelint
 
 `stylelint.config.js` example:

package/dist/esm/eslint/index.d.ts

@@ -3,6 +3,7 @@ import { type TailwindOptions } from "./tailwind.js";
 export * from "./astro.js";
 export * from "./basic.js";
 export * from "./jsdoc.js";
+export * from "./netsuite.js";
 export * from "./next.js";
 export * from "./packageJson.js";
 export * from "./react.js";
@@ -10,7 +11,9 @@ export * from "./tailwind.js";
 export * from "./turbo.js";
 export { composeConfig } from "./utils.js";
 export declare const presetBase: Linter.Config[];
-export declare const presetReactLite: Linter.Config[];
+export declare const presetNode: Linter.Config<import("@eslint/core").RulesConfig>[];
+export declare const presetNetSuite: Linter.Config<import("@eslint/core").RulesConfig>[];
+export declare const presetReactLite: Linter.Config<import("@eslint/core").RulesConfig>[];
 export declare const getPresetReact: (options: TailwindOptions) => Linter.Config[];
 export declare const getPresetNext: (options: TailwindOptions) => Linter.Config[];
 export declare const getPresetAstro: (options: TailwindOptions) => Linter.Config[];

package/dist/esm/eslint/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/eslint/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAOrC,OAAO,EAAe,KAAK,eAAe,EAAE,MAAM,eAAe,CAAC;AAGlE,cAAc,YAAY,CAAC;AAC3B,cAAc,YAAY,CAAC;AAC3B,cAAc,YAAY,CAAC;AAC3B,cAAc,WAAW,CAAC;AAC1B,cAAc,kBAAkB,CAAC;AACjC,cAAc,YAAY,CAAC;AAC3B,cAAc,eAAe,CAAC;AAC9B,cAAc,YAAY,CAAC;AAC3B,OAAO,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE3C,eAAO,MAAM,UAAU,EAAE,MAAM,CAAC,MAAM,EAAyB,CAAC;AAChE,eAAO,MAAM,eAAe,EAAE,MAAM,CAAC,MAAM,EAG1C,CAAC;AACF,eAAO,MAAM,cAAc,YAAa,eAAe,KAAG,MAAM,CAAC,MAAM,EAChB,CAAC;AACxD,eAAO,MAAM,aAAa,YAAa,eAAe,KAAG,MAAM,CAAC,MAAM,EACzB,CAAC;AAC9C,eAAO,MAAM,cAAc,YAAa,eAAe,KAAG,MAAM,CAAC,MAAM,EACzB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/eslint/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AASrC,OAAO,EAAe,KAAK,eAAe,EAAE,MAAM,eAAe,CAAC;AAGlE,cAAc,YAAY,CAAC;AAC3B,cAAc,YAAY,CAAC;AAC3B,cAAc,YAAY,CAAC;AAC3B,cAAc,eAAe,CAAC;AAC9B,cAAc,WAAW,CAAC;AAC1B,cAAc,kBAAkB,CAAC;AACjC,cAAc,YAAY,CAAC;AAC3B,cAAc,eAAe,CAAC;AAC9B,cAAc,YAAY,CAAC;AAC3B,OAAO,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE3C,eAAO,MAAM,UAAU,EAAE,MAAM,CAAC,MAAM,EAAyB,CAAC;AAChE,eAAO,MAAM,UAAU,qDAAoC,CAAC;AAC5D,eAAO,MAAM,cAAc,qDAAsC,CAAC;AAClE,eAAO,MAAM,eAAe,qDAAmC,CAAC;AAChE,eAAO,MAAM,cAAc,YAAa,eAAe,KAAG,MAAM,CAAC,MAAM,EAChB,CAAC;AACxD,eAAO,MAAM,aAAa,YAAa,eAAe,KAAG,MAAM,CAAC,MAAM,EACzB,CAAC;AAC9C,eAAO,MAAM,cAAc,YAAa,eAAe,KAAG,MAAM,CAAC,MAAM,EACzB,CAAC"}

package/dist/esm/eslint/index.js

@@ -1,13 +1,16 @@
 import { astro } from "./astro.js";
 import { basic } from "./basic.js";
 import { jsdoc } from "./jsdoc.js";
+import { netsuite } from "./netsuite.js";
 import { next } from "./next.js";
+import { nodeTS } from "./nodeTS.js";
 import { react } from "./react.js";
 import { getTailwind } from "./tailwind.js";
 import { composeConfig } from "./utils.js";
 export * from "./astro.js";
 export * from "./basic.js";
 export * from "./jsdoc.js";
+export * from "./netsuite.js";
 export * from "./next.js";
 export * from "./packageJson.js";
 export * from "./react.js";
@@ -15,6 +18,8 @@ export * from "./tailwind.js";
 export * from "./turbo.js";
 export { composeConfig } from "./utils.js";
 export const presetBase = [...basic, ...jsdoc];
+export const presetNode = composeConfig(presetBase, nodeTS);
+export const presetNetSuite = composeConfig(presetBase, netsuite);
 export const presetReactLite = composeConfig(presetBase, react);
 export const getPresetReact = (options) => composeConfig(presetBase, react, getTailwind(options));
 export const getPresetNext = (options) => composeConfig(getPresetReact(options), next);

package/dist/esm/eslint/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/eslint/index.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,WAAW,EAAwB,MAAM,eAAe,CAAC;AAClE,OAAO,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE3C,cAAc,YAAY,CAAC;AAC3B,cAAc,YAAY,CAAC;AAC3B,cAAc,YAAY,CAAC;AAC3B,cAAc,WAAW,CAAC;AAC1B,cAAc,kBAAkB,CAAC;AACjC,cAAc,YAAY,CAAC;AAC3B,cAAc,eAAe,CAAC;AAC9B,cAAc,YAAY,CAAC;AAC3B,OAAO,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE3C,MAAM,CAAC,MAAM,UAAU,GAAoB,CAAC,GAAG,KAAK,EAAE,GAAG,KAAK,CAAC,CAAC;AAChE,MAAM,CAAC,MAAM,eAAe,GAAoB,aAAa,CAC5D,UAAU,EACV,KAAK,CACL,CAAC;AACF,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,OAAwB,EAAmB,EAAE,CAC3E,aAAa,CAAC,UAAU,EAAE,KAAK,EAAE,WAAW,CAAC,OAAO,CAAC,CAAC,CAAC;AACxD,MAAM,CAAC,MAAM,aAAa,GAAG,CAAC,OAAwB,EAAmB,EAAE,CAC1E,aAAa,CAAC,cAAc,CAAC,OAAO,CAAC,EAAE,IAAI,CAAC,CAAC;AAC9C,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,OAAwB,EAAmB,EAAE,CAC3E,aAAa,CAAC,cAAc,CAAC,OAAO,CAAC,EAAE,KAAK,CAAC,CAAC","sourcesContent":["import type { Linter } from \"eslint\";\n\nimport { astro } from \"./astro.js\";\nimport { basic } from \"./basic.js\";\nimport { jsdoc } from \"./jsdoc.js\";\nimport { next } from \"./next.js\";\nimport { react } from \"./react.js\";\nimport { getTailwind, type TailwindOptions } from \"./tailwind.js\";\nimport { composeConfig } from \"./utils.js\";\n\nexport * from \"./astro.js\";\nexport * from \"./basic.js\";\nexport * from \"./jsdoc.js\";\nexport * from \"./next.js\";\nexport * from \"./packageJson.js\";\nexport * from \"./react.js\";\nexport * from \"./tailwind.js\";\nexport * from \"./turbo.js\";\nexport { composeConfig } from \"./utils.js\";\n\nexport const presetBase: Linter.Config[] = [...basic, ...jsdoc];\nexport const presetReactLite: Linter.Config[] = composeConfig(\n\tpresetBase,\n\treact,\n);\nexport const getPresetReact = (options: TailwindOptions): Linter.Config[] =>\n\tcomposeConfig(presetBase, react, getTailwind(options));\nexport const getPresetNext = (options: TailwindOptions): Linter.Config[] =>\n\tcomposeConfig(getPresetReact(options), next);\nexport const getPresetAstro = (options: TailwindOptions): Linter.Config[] =>\n\tcomposeConfig(getPresetReact(options), astro);\n"]}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/eslint/index.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,QAAQ,EAAE,MAAM,eAAe,CAAC;AACzC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AACrC,OAAO,EAAE,KAAK,EAAE,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,WAAW,EAAwB,MAAM,eAAe,CAAC;AAClE,OAAO,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE3C,cAAc,YAAY,CAAC;AAC3B,cAAc,YAAY,CAAC;AAC3B,cAAc,YAAY,CAAC;AAC3B,cAAc,eAAe,CAAC;AAC9B,cAAc,WAAW,CAAC;AAC1B,cAAc,kBAAkB,CAAC;AACjC,cAAc,YAAY,CAAC;AAC3B,cAAc,eAAe,CAAC;AAC9B,cAAc,YAAY,CAAC;AAC3B,OAAO,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE3C,MAAM,CAAC,MAAM,UAAU,GAAoB,CAAC,GAAG,KAAK,EAAE,GAAG,KAAK,CAAC,CAAC;AAChE,MAAM,CAAC,MAAM,UAAU,GAAG,aAAa,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;AAC5D,MAAM,CAAC,MAAM,cAAc,GAAG,aAAa,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAC;AAClE,MAAM,CAAC,MAAM,eAAe,GAAG,aAAa,CAAC,UAAU,EAAE,KAAK,CAAC,CAAC;AAChE,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,OAAwB,EAAmB,EAAE,CAC3E,aAAa,CAAC,UAAU,EAAE,KAAK,EAAE,WAAW,CAAC,OAAO,CAAC,CAAC,CAAC;AACxD,MAAM,CAAC,MAAM,aAAa,GAAG,CAAC,OAAwB,EAAmB,EAAE,CAC1E,aAAa,CAAC,cAAc,CAAC,OAAO,CAAC,EAAE,IAAI,CAAC,CAAC;AAC9C,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,OAAwB,EAAmB,EAAE,CAC3E,aAAa,CAAC,cAAc,CAAC,OAAO,CAAC,EAAE,KAAK,CAAC,CAAC","sourcesContent":["import type { Linter } from \"eslint\";\n\nimport { astro } from \"./astro.js\";\nimport { basic } from \"./basic.js\";\nimport { jsdoc } from \"./jsdoc.js\";\nimport { netsuite } from \"./netsuite.js\";\nimport { next } from \"./next.js\";\nimport { nodeTS } from \"./nodeTS.js\";\nimport { react } from \"./react.js\";\nimport { getTailwind, type TailwindOptions } from \"./tailwind.js\";\nimport { composeConfig } from \"./utils.js\";\n\nexport * from \"./astro.js\";\nexport * from \"./basic.js\";\nexport * from \"./jsdoc.js\";\nexport * from \"./netsuite.js\";\nexport * from \"./next.js\";\nexport * from \"./packageJson.js\";\nexport * from \"./react.js\";\nexport * from \"./tailwind.js\";\nexport * from \"./turbo.js\";\nexport { composeConfig } from \"./utils.js\";\n\nexport const presetBase: Linter.Config[] = [...basic, ...jsdoc];\nexport const presetNode = composeConfig(presetBase, nodeTS);\nexport const presetNetSuite = composeConfig(presetBase, netsuite);\nexport const presetReactLite = composeConfig(presetBase, react);\nexport const getPresetReact = (options: TailwindOptions): Linter.Config[] =>\n\tcomposeConfig(presetBase, react, getTailwind(options));\nexport const getPresetNext = (options: TailwindOptions): Linter.Config[] =>\n\tcomposeConfig(getPresetReact(options), next);\nexport const getPresetAstro = (options: TailwindOptions): Linter.Config[] =>\n\tcomposeConfig(getPresetReact(options), astro);\n"]}

package/dist/esm/eslint/netsuite.d.ts

@@ -0,0 +1,3 @@
+import type { Linter } from "eslint";
+export declare const netsuite: Linter.Config[];
+//# sourceMappingURL=netsuite.d.ts.map

package/dist/esm/eslint/netsuite.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"netsuite.d.ts","sourceRoot":"","sources":["../../../src/eslint/netsuite.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAErC,eAAO,MAAM,QAAQ,EAAE,MAAM,CAAC,MAAM,EA6BnC,CAAC"}

package/dist/esm/eslint/netsuite.js

@@ -0,0 +1,31 @@
+export const netsuite = [
+    {
+        languageOptions: {
+            globals: {
+                define: "readonly",
+                log: "readonly",
+            },
+        },
+        rules: {
+            // N/* modules are provided by NetSuite at runtime.
+            "sonarjs/no-implicit-dependencies": "off",
+        },
+    },
+    {
+        files: ["**/*.ts"],
+        rules: {
+            "jsdoc/check-tag-names": [
+                "error",
+                {
+                    definedTags: [
+                        "NApiVersion",
+                        "NScriptType",
+                        "NModuleScope",
+                        "NAmdConfig",
+                    ],
+                },
+            ],
+        },
+    },
+];
+//# sourceMappingURL=netsuite.js.map

package/dist/esm/eslint/netsuite.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"netsuite.js","sourceRoot":"","sources":["../../../src/eslint/netsuite.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,QAAQ,GAAoB;IACxC;QACC,eAAe,EAAE;YAChB,OAAO,EAAE;gBACR,MAAM,EAAE,UAAU;gBAClB,GAAG,EAAE,UAAU;aACf;SACD;QACD,KAAK,EAAE;YACN,mDAAmD;YACnD,kCAAkC,EAAE,KAAK;SACzC;KACD;IACD;QACC,KAAK,EAAE,CAAC,SAAS,CAAC;QAClB,KAAK,EAAE;YACN,uBAAuB,EAAE;gBACxB,OAAO;gBACP;oBACC,WAAW,EAAE;wBACZ,aAAa;wBACb,aAAa;wBACb,cAAc;wBACd,YAAY;qBACZ;iBACD;aACD;SACD;KACD;CACD,CAAC","sourcesContent":["import type { Linter } from \"eslint\";\n\nexport const netsuite: Linter.Config[] = [\n\t{\n\t\tlanguageOptions: {\n\t\t\tglobals: {\n\t\t\t\tdefine: \"readonly\",\n\t\t\t\tlog: \"readonly\",\n\t\t\t},\n\t\t},\n\t\trules: {\n\t\t\t// N/* modules are provided by NetSuite at runtime.\n\t\t\t\"sonarjs/no-implicit-dependencies\": \"off\",\n\t\t},\n\t},\n\t{\n\t\tfiles: [\"**/*.ts\"],\n\t\trules: {\n\t\t\t\"jsdoc/check-tag-names\": [\n\t\t\t\t\"error\",\n\t\t\t\t{\n\t\t\t\t\tdefinedTags: [\n\t\t\t\t\t\t\"NApiVersion\",\n\t\t\t\t\t\t\"NScriptType\",\n\t\t\t\t\t\t\"NModuleScope\",\n\t\t\t\t\t\t\"NAmdConfig\",\n\t\t\t\t\t],\n\t\t\t\t},\n\t\t\t],\n\t\t},\n\t},\n];\n"]}

package/dist/esm/eslint/nodeTS.d.ts

@@ -0,0 +1,3 @@
+import type { Linter } from "eslint";
+export declare const nodeTS: Linter.Config;
+//# sourceMappingURL=nodeTS.d.ts.map

package/dist/esm/eslint/nodeTS.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"nodeTS.d.ts","sourceRoot":"","sources":["../../../src/eslint/nodeTS.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,QAAQ,CAAC;AAErC,eAAO,MAAM,MAAM,EAAE,MAAM,CAAC,MAI3B,CAAC"}

package/dist/esm/eslint/nodeTS.js

@@ -0,0 +1,6 @@
+export const nodeTS = {
+    rules: {
+        "import/extensions": ["error", "always", { ts: "always" }],
+    },
+};
+//# sourceMappingURL=nodeTS.js.map

package/dist/esm/eslint/nodeTS.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"nodeTS.js","sourceRoot":"","sources":["../../../src/eslint/nodeTS.ts"],"names":[],"mappings":"AAEA,MAAM,CAAC,MAAM,MAAM,GAAkB;IACpC,KAAK,EAAE;QACN,mBAAmB,EAAE,CAAC,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC;KAC1D;CACD,CAAC","sourcesContent":["import type { Linter } from \"eslint\";\n\nexport const nodeTS: Linter.Config = {\n\trules: {\n\t\t\"import/extensions\": [\"error\", \"always\", { ts: \"always\" }],\n\t},\n};\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rootreeweb/linting",
-  "version": "5.2.0",
+  "version": "5.4.0",
   "description": "Shared configurations for ESLint and Stylelint",
   "license": "UNLICENSED",
   "repository": "github:rootreeweb/meta-tools",
@@ -34,7 +34,8 @@
     "test:watch": "vitest --coverage --ui"
   },
   "files": [
-    "dist/**/*"
+    "dist/**/*",
+    "skills/**/*"
   ],
   "keywords": [
     "eslint",
@@ -46,17 +47,17 @@
   "dependencies": {
     "@eslint/js": "^10.0.1",
     "@ianvs/prettier-plugin-sort-imports": "^4.7.1",
-    "@next/eslint-plugin-next": "^16.4.0",
-    "@rootreeweb/js-lib": "^4.0.19",
-    "@tanstack/eslint-plugin-query": "^5.104.2",
+    "@next/eslint-plugin-next": "^16.4.2",
+    "@rootreeweb/js-lib": "^4.0.20",
+    "@tanstack/eslint-plugin-query": "^5.104.5",
     "@vitest/eslint-plugin": "^1.6.27",
     "eslint-import-resolver-typescript": "^4.4.5",
     "eslint-package-json": "^3.0.0",
     "eslint-plugin-astro": "^3.2.1",
     "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-jsdoc": "^65.3.2",
+    "eslint-plugin-jsdoc": "^65.4.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
-    "eslint-plugin-perfectionist": "^5.12.1",
+    "eslint-plugin-perfectionist": "^5.13.0",
     "eslint-plugin-promise": "^7.3.0",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-compiler": "^19.1.0-rc.2",
@@ -66,9 +67,9 @@
     "eslint-plugin-sonarjs": "^4.2.2",
     "eslint-plugin-tailwindcss": "^4.4.0",
     "eslint-plugin-testing-library": "^7.16.2",
-    "eslint-plugin-turbo": "^2.11.7",
+    "eslint-plugin-turbo": "^2.11.9",
     "globals": "^17.13.0",
-    "postcss": "^8.5.30",
+    "postcss": "^8.5.31",
     "prettier-plugin-astro": "^1.1.1",
     "prettier-plugin-jsdoc": "^1.8.1",
     "prettier-plugin-tailwindcss": "^0.8.1",
@@ -77,33 +78,35 @@
     "stylelint-config-tailwindcss": "^1.0.1",
     "stylelint-prettier": "^5.0.3",
     "tailwindcss": "^4.3.3",
-    "typescript-eslint": "^8.71.2"
+    "typescript-eslint": "^8.72.0"
   },
   "devDependencies": {
-    "@rootreeweb/tsconfigs": "^3.3.24",
+    "@rootreeweb/tsconfigs": "^3.4.1",
     "@types/eslint-plugin-jsx-a11y": "^6.10.1",
     "@types/eslint-plugin-security": "^3.0.1",
     "@types/node": "^26.6.5",
     "@types/react": "^19.3.0",
     "@typescript/native-preview": "^7.0.0-dev.20260707.2",
-    "@vitest/coverage-v8": "^5.0.4",
-    "@vitest/ui": "^5.0.4",
+    "@vitest/coverage-v8": "^5.0.5",
+    "@vitest/ui": "^5.0.5",
     "del-cli": "^8.0.1",
-    "eslint": "^10.12.0",
+    "eslint": "^10.13.0",
     "husky": "^9.1.7",
-    "knip": "^6.41.0",
+    "jscpd": "^5.4.1",
+    "knip": "^6.42.1",
     "lint-staged": "^17.6.0",
     "prettier": "~3.8.5",
-    "react": "^19.3.0",
+    "react": "^19.3.1",
     "stylelint": "~17.16.0",
-    "turbo": "~2.11.7",
+    "turbo": "~2.11.9",
     "typescript": "^6.0.3",
     "vite": "^8.3.4",
-    "vitest": "^5.0.4"
+    "vitest": "^5.0.5"
   },
   "peerDependencies": {
     "eslint": "^10",
     "husky": "^9.1.7",
+    "jscpd": "*",
     "lint-staged": "^17.2.0",
     "prettier": "~3.8.0",
     "stylelint": "~17.16.0",
@@ -116,5 +119,11 @@
     "turbo": {
       "optional": true
     }
-  }
+  },
+  "skills": [
+    "mattpocock/skills@domain-modeling",
+    "mattpocock/skills@grill-me",
+    "mattpocock/skills@improve-codebase-architecture",
+    "kucherenko/jscpd@dry-refactoring"
+  ]
 }

package/skills/format-lint-js/SKILL.md

@@ -0,0 +1,18 @@
+---
+name: format-and-lint-js
+description: Type-checks, lints and formats a JavaScript/TypeScript project using the @rootreeweb/linting setup. Use after making changes to JS or TS code.
+---
+
+# Format and Lint JS
+
+## When to use this skill
+
+- Use this after making changes to the JavaScript and TypeScript in the project
+
+## How to use it
+
+1. Execute `yarn check` to check for type errors.
+2. Execute `yarn lint` to check for lint errors.
+   - If the repo contains the `@rootreeweb/linting` package itself, run `yarn build` first so the rules are up to date.
+3. Execute `yarn format` to format the code.
+4. Fix any errors that arise and then repeat until there are no errors.

package/skills/solid-audit/SKILL.md

@@ -0,0 +1,83 @@
+---
+name: solid-audit
+description: Audits code or a design against SOLID and Clean Code (Uncle Bob) principles and reports concrete, prioritised refactors. Use on demand when asked for a SOLID, clean code, Uncle Bob, design or code-quality audit/review. Also use passively whenever planning or designing an implementation (plan mode, "how should we build X", new modules, refactors) to sanity-check the proposed structure before code is written.
+---
+
+# SOLID Audit
+
+Judge code by how cheaply it can change. Every finding must name a concrete cost (what change becomes hard, what breaks, what gets duplicated) and a concrete fix. A principle being "violated" with no cost is not a finding.
+
+## Modes
+
+### Planning (passive)
+
+Run silently while drafting any implementation plan. Do not announce the skill or produce a report.
+
+1. Check the proposed modules, functions and data flow against the principles and house preferences below.
+2. Adjust the plan itself where a problem is cheap to avoid now (split a responsibility, invert a dependency, narrow an interface).
+3. Only if a trade-off remains worth the user's attention, add at most 3 bullets under a short **Design notes** heading in the plan. Otherwise add nothing.
+4. Never add abstractions to the plan "for SOLID". Prefer the simplest design that does not create one of the costs listed below.
+
+### Audit (on demand)
+
+1. **Scope**: use what the user named (files, package, diff, PR). If nothing is named, audit the current diff against the main branch; if there is no diff, ask.
+2. **Read**: read the code in scope fully, plus its direct callers and dependencies when needed to judge coupling. Don't audit files you haven't read.
+3. **Assess** each principle, heuristic and house preference below. Skip what doesn't apply.
+4. **Report** using the output format below, most severe first. Cap at ~10 findings unless asked for more.
+5. **Don't edit** unless the user asks. When asked to fix, do one finding at a time with tests/type checks green between steps.
+
+## Principles
+
+Interpret for idiomatic TypeScript: modules and functions count as units, not just classes. Types and function parameters are interfaces.
+
+| Principle                                                 | Signals of a real problem                                                                                                                                                     | Typical fix                                                                            |
+| --------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | -------------------------------------------------------------------------------------- |
+| **S**ingle Responsibility: one reason to change           | A unit mixes I/O, business rules and formatting; edits for unrelated features touch the same function; name needs "and"/"Manager"/"Utils"/"Helper"; file well over ~100 lines | Split by reason to change; keep pure logic separate from side effects                  |
+| **O**pen/Closed: extend without editing                   | Growing `switch`/`if` chains on a type tag repeated in several places; every new variant edits core code                                                                      | Discriminated union + lookup map/strategy object; only once there are ≥2 real variants |
+| **L**iskov Substitution: substitutes honour the contract  | Overrides/implementations that throw "not supported", ignore params, narrow accepted inputs or widen errors; callers check concrete type                                      | Fix the contract or stop pretending it's the same type                                 |
+| **I**nterface Segregation: depend only on what you use    | Functions take a whole config/entity but read one field; optional-property soup; mocks in tests need many unused stubs                                                        | Accept a narrow type (`Pick<>`, specific params)                                       |
+| **D**ependency Inversion: policy doesn't depend on detail | Business logic imports fs, network, env, SDKs or global singletons directly; tests need module mocking to run                                                                 | Pass the dependency in (parameter, factory arg); keep the edge thin                    |
+
+## Clean Code heuristics
+
+- **Functions**: do one thing at one level of abstraction; >3 params or boolean flag params usually mean two functions or an options object.
+- **Names**: reveal intent; same concept, same word; no type-encoded or abbreviated names.
+- **Side effects**: hidden mutation of args/globals; functions named as queries that also write.
+- **Errors**: swallowed errors, `catch` that logs and continues into invalid state, error handling mixed into happy-path logic.
+- **Law of Demeter**: long `a.b().c().d` chains reaching into another module's internals.
+- **Comments**: inline comments that restate code or compensate for a bad name. Prefer renaming/extracting. Comments that explain non-obvious _why_ are fine. JSDoc blocks are exempt: never flag them as over-commenting.
+- **Tests**: untested branches in changed logic; tests coupled to implementation details.
+- **Design smells**: rigidity (one change → many edits), fragility (changes break distant code), immobility (can't reuse without dragging deps), needless complexity (speculative abstraction), opacity.
+
+## House preferences
+
+These are this team's defaults. Apply them in both modes alongside SOLID.
+
+- **Files under ~100 lines.** A longer file is a signal of more than one responsibility; propose how to split it (by responsibility, not arbitrarily). JSDoc lines count less: don't split a file just because its docs are thorough.
+- **Use folder structure.** Group related files in a folder named for the concept (e.g. `eslint/netsuite.ts`, `presets/core.ts`) rather than prefixing file names or growing a flat directory.
+- **Minimise churn in existing files.** Prefer designs where new behaviour lands in new files and existing files change little; in particular avoid plans that rewrite import lists across many consumers.
+- **Barrels where they don't complicate things.** A folder's `index.ts` re-exporting its public surface lets consumers import from the folder, so adding or splitting files inside it doesn't touch their imports. Skip a barrel when it would create an import cycle, re-export across package boundaries for no gain, or hide which module owns a side effect.
+
+## Guardrails
+
+- YAGNI beats OCP: don't recommend an abstraction for a single implementation or a hypothetical future variant.
+- Don't recommend classes/interfaces where plain functions and types suffice.
+- Style, formatting and lint-enforced rules are out of scope; the linter owns them.
+- Duplication findings: hand off to the `dry-refactoring` skill. Module-boundary/deep-module redesigns: hand off to `improve-codebase-architecture`.
+- Say so plainly when code is in good shape; an empty or short report is a valid result.
+
+## Output format (audit mode)
+
+```markdown
+## SOLID audit: <scope>
+
+<one-sentence overall verdict>
+
+| #   | Severity | Principle | Location        | Problem (cost)                                                                              | Suggested fix                    |
+| --- | -------- | --------- | --------------- | ------------------------------------------------------------------------------------------- | -------------------------------- |
+| 1   | High     | DIP       | `src/foo.ts:42` | `syncOrders` calls the NetSuite SDK directly, so it can't be tested without mocking modules | Pass a `fetchOrders` function in |
+
+**Quick wins:** <1–3 low-effort, high-value items by #>
+```
+
+Severity: **High** = actively causing bugs or blocking likely changes; **Medium** = real cost on the next change in this area; **Low** = worth doing when touching the code anyway.