@dvukovic/style-guide 0.14.0 → 0.14.2

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@dvukovic/style-guide",
-    "version": "0.14.0",
+    "version": "0.14.2",
     "description": "My own style guide",
     "repository": {
         "type": "git",
@@ -43,10 +43,12 @@
         "src/knip",
         "src/package-json",
         "src/prettier",
-        "src/stylelint"
+        "src/stylelint",
+        "!**/*.test.js",
+        "!**/*.test.ts"
     ],
     "scripts": {
-        "build": "tsc",
+        "build": "tsc --project tsconfig.build.json",
         "lint": "yarn lint:eslint && yarn lint:prettier && yarn lint:stylelint && yarn lint:cspell && yarn lint:knip",
         "lint:cspell": "cspell --no-progress --no-summary --unique '**'",
         "lint:eslint": "eslint . --cache --concurrency=auto",

package/src/eslint/index.js

@@ -1,4 +1,3 @@
-// @ts-nocheck
 import { defineConfig, globalIgnores } from "eslint/config"
 
 /**

package/dist/scripts/changelog-report.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/scripts/changelog-report.types.d.ts

@@ -1,28 +0,0 @@
-export type OutdatedPackageType = {
-    current: string;
-    latest: string;
-    name: string;
-};
-export type NpmRegistryResponseType = {
-    repository?: {
-        url?: string;
-    };
-};
-export type GitHubReleaseType = {
-    body: string;
-    name: string;
-    published_at: string;
-    tag_name: string;
-};
-export type PackageChangelogType = {
-    changelog: string;
-    current: string;
-    latest: string;
-    name: string;
-};
-export type NpmOutdatedEntryType = {
-    current: string;
-    latest: string;
-    wanted: string;
-};
-export type NpmOutdatedOutputType = Record<string, NpmOutdatedEntryType>;

package/dist/src/eslint/configs/aws.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/core.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/jest.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/mobx.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/next.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/node.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/package-json.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/playwright.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/react.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/storybook.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/typescript-strict.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/typescript.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/configs/vitest.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/plugins/eslint.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/plugins/validate-rules.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/rules/document-todos/document-todos.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/rules/no-commented-out-code/no-commented-out-code.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/rules/no-restricted-dependencies/no-restricted-dependencies.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/rules/no-t/no-t.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/rules/require-properties/require-properties.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/eslint/rules/valid-engines-node/valid-engines-node.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/prettier/configs/core.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/src/stylelint/configs/core.test.d.ts

@@ -1 +0,0 @@
-export {};

package/src/eslint/configs/aws.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { awsConfig } from "./aws.js"
-
-const eslint = new ESLint({
-    overrideConfig: awsConfig,
-    overrideConfigFile: true,
-})
-
-describe("aws", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.js" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/core.test.js

@@ -1,23 +0,0 @@
-import { ESLint } from "eslint"
-
-import { coreConfig } from "./core.js"
-
-const eslint = new ESLint({
-    overrideConfig: coreConfig,
-    overrideConfigFile: true,
-})
-
-describe("core", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.js" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-
-    test("detects violations", async () => {
-        const results = await eslint.lintText("var x = 1\n", { filePath: "test.js" })
-
-        expect(results[0]?.errorCount).toBeGreaterThan(0)
-    })
-})

package/src/eslint/configs/jest.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { jestConfig } from "./jest.js"
-
-const eslint = new ESLint({
-    overrideConfig: jestConfig,
-    overrideConfigFile: true,
-})
-
-describe("jest", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.test.js" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/mobx.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { mobxConfig } from "./mobx.js"
-
-const eslint = new ESLint({
-    overrideConfig: mobxConfig,
-    overrideConfigFile: true,
-})
-
-describe("mobx", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.js" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/next.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { nextConfig } from "./next.js"
-
-const eslint = new ESLint({
-    overrideConfig: nextConfig,
-    overrideConfigFile: true,
-})
-
-describe("next", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.tsx" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/node.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { nodeConfig } from "./node.js"
-
-const eslint = new ESLint({
-    overrideConfig: nodeConfig,
-    overrideConfigFile: true,
-})
-
-describe("node", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.js" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/package-json.test.js

@@ -1,154 +0,0 @@
-import { ESLint } from "eslint"
-import { defineConfig } from "eslint/config"
-
-import { packageJsonConfigs, packageJsonWorkspace } from "./package-json.js"
-
-const eslint = new ESLint({
-    overrideConfig: [...packageJsonConfigs, { files: ["**/package.json"] }],
-    overrideConfigFile: true,
-})
-
-describe("package-json", () => {
-    test("loads without errors", async () => {
-        const validPackageJson = JSON.stringify(
-            {
-                author: "Test Author",
-                description: "A test package",
-                engines: {
-                    node: ">=20.0.0",
-                },
-                license: "MIT",
-                name: "test-package",
-                repository: {
-                    type: "git",
-                    url: "https://github.com/test/test",
-                },
-                version: "1.0.0",
-            },
-            null,
-            4,
-        )
-
-        const results = await eslint.lintText(validPackageJson, { filePath: "package.json" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-
-    test("detects missing required fields", async () => {
-        const invalidPackageJson = JSON.stringify(
-            {
-                name: "test-package",
-            },
-            null,
-            4,
-        )
-
-        const results = await eslint.lintText(invalidPackageJson, { filePath: "package.json" })
-
-        expect(results[0]?.errorCount).toBeGreaterThan(0)
-    })
-})
-
-describe("packageJsonWorkspace", () => {
-    test("returns array of two configs", () => {
-        const configs = packageJsonWorkspace()
-
-        expect(configs).toHaveLength(2)
-    })
-
-    test("root config targets package.json", () => {
-        const configs = packageJsonWorkspace()
-
-        expect(configs[0].files).toEqual(["package.json"])
-    })
-
-    test("nested config targets default workspace patterns", () => {
-        const configs = packageJsonWorkspace()
-
-        expect(configs[1].files).toEqual(["**/packages/**/package.json", "**/apps/**/package.json"])
-    })
-
-    test("nested config disables require-properties rule", () => {
-        const configs = packageJsonWorkspace()
-
-        expect(configs[1].rules).toEqual({
-            "dvukovic/require-properties": "off",
-        })
-    })
-
-    test("accepts custom workspace patterns", () => {
-        const configs = packageJsonWorkspace({
-            workspacePatterns: ["**/libs/**/package.json"],
-        })
-
-        expect(configs[1].files).toEqual(["**/libs/**/package.json"])
-    })
-
-    test("passes through extends to root config", () => {
-        const customExtends = [{ rules: { "custom/rule": "error" } }]
-        const configs = packageJsonWorkspace({ extends: customExtends })
-
-        expect(configs[0].extends).toContain(customExtends[0])
-    })
-
-    test("root config requires volta.node", async () => {
-        const workspaceEslint = new ESLint({
-            overrideConfig: defineConfig(...packageJsonWorkspace()),
-            overrideConfigFile: true,
-        })
-
-        const packageWithoutVolta = JSON.stringify(
-            {
-                author: "Test Author",
-                description: "A test package",
-                engines: { node: ">=24.0.0" },
-                license: "MIT",
-                name: "test-package",
-                repository: { type: "git", url: "https://github.com/test/test" },
-                version: "1.0.0",
-            },
-            null,
-            4,
-        )
-
-        const results = await workspaceEslint.lintText(packageWithoutVolta, {
-            filePath: "package.json",
-        })
-
-        const hasVoltaError = results[0]?.messages.some((message) => {
-            return message.ruleId === "dvukovic/require-properties"
-        })
-        expect(hasVoltaError).toBe(true)
-    })
-
-    test("nested config does not require volta.node", async () => {
-        const workspaceEslint = new ESLint({
-            overrideConfig: defineConfig(...packageJsonWorkspace()),
-            overrideConfigFile: true,
-        })
-
-        const packageWithoutVolta = JSON.stringify(
-            {
-                author: "Test Author",
-                description: "A test package",
-                engines: { node: ">=24.0.0" },
-                license: "MIT",
-                name: "test-package",
-                repository: { type: "git", url: "https://github.com/test/test" },
-                version: "1.0.0",
-            },
-            null,
-            4,
-        )
-
-        const results = await workspaceEslint.lintText(packageWithoutVolta, {
-            filePath: "packages/my-package/package.json",
-        })
-
-        const hasVoltaError = results[0]?.messages.some((message) => {
-            return message.ruleId === "dvukovic/require-properties"
-        })
-        expect(hasVoltaError).toBe(false)
-    })
-})

package/src/eslint/configs/playwright.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { playwrightConfig } from "./playwright.js"
-
-const eslint = new ESLint({
-    overrideConfig: playwrightConfig,
-    overrideConfigFile: true,
-})
-
-describe("playwright", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.spec.ts" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/react.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { reactConfig } from "./react.js"
-
-const eslint = new ESLint({
-    overrideConfig: reactConfig,
-    overrideConfigFile: true,
-})
-
-describe("react", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.tsx" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/storybook.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { storybookConfig } from "./storybook.js"
-
-const eslint = new ESLint({
-    overrideConfig: storybookConfig,
-    overrideConfigFile: true,
-})
-
-describe("storybook", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.stories.tsx" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/typescript-strict.test.js

@@ -1,19 +0,0 @@
-import { ESLint } from "eslint"
-
-import { typescriptStrictConfig } from "./typescript-strict.js"
-
-const eslint = new ESLint({
-    overrideConfig: typescriptStrictConfig,
-    overrideConfigFile: true,
-})
-
-describe("typescript-strict", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x: number = 1\n", {
-            filePath: "src/eslint/configs/typescript-strict.test.ts",
-        })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/typescript.test.js

@@ -1,19 +0,0 @@
-import { ESLint } from "eslint"
-
-import { typescriptConfig } from "./typescript.js"
-
-const eslint = new ESLint({
-    overrideConfig: typescriptConfig,
-    overrideConfigFile: true,
-})
-
-describe("typescript", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", {
-            filePath: "src/eslint/configs/typescript.test.js",
-        })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/configs/vitest.test.js

@@ -1,17 +0,0 @@
-import { ESLint } from "eslint"
-
-import { vitestConfig } from "./vitest.js"
-
-const eslint = new ESLint({
-    overrideConfig: vitestConfig,
-    overrideConfigFile: true,
-})
-
-describe("vitest", () => {
-    test("loads without errors", async () => {
-        const results = await eslint.lintText("const x = 1\n", { filePath: "test.test.ts" })
-
-        expect(results).toBeDefined()
-        expect(results[0]?.fatalErrorCount).toBe(0)
-    })
-})

package/src/eslint/plugins/eslint.test.js

@@ -1,42 +0,0 @@
-import { ESLint } from "eslint"
-import tseslint from "typescript-eslint"
-
-import { eslint as base } from "./eslint.js"
-
-const eslint = new ESLint({
-    overrideConfig: [
-        {
-            files: ["**/*.ts"],
-            languageOptions: {
-                parser: tseslint.parser,
-            },
-        },
-        base,
-    ],
-    overrideConfigFile: true,
-})
-
-describe("eslint no-enum", () => {
-    test("detects enum declarations", async () => {
-        const code = `enum Status { Active, Inactive }\n`
-        const results = await eslint.lintText(code, { filePath: "test.ts" })
-
-        const enumErrors = results[0]?.messages.filter((message) => {
-            return message.ruleId === "no-restricted-syntax"
-        })
-
-        expect(enumErrors?.length).toBe(1)
-        expect(enumErrors?.[0]?.message).toContain("enum")
-    })
-
-    test("allows union types", async () => {
-        const code = `type Status = "active" | "inactive"\n`
-        const results = await eslint.lintText(code, { filePath: "test.ts" })
-
-        const restrictedSyntaxErrors = results?.[0]?.messages.filter((message) => {
-            return message.ruleId === "no-restricted-syntax"
-        })
-
-        expect(restrictedSyntaxErrors?.length).toBe(0)
-    })
-})

package/src/eslint/plugins/validate-rules.test.js

@@ -1,68 +0,0 @@
-import { baseline } from "./baseline.js"
-import { dvukovic } from "./dvukovic.js"
-import { eslint } from "./eslint.js"
-import { eslintComments } from "./eslint-comments.js"
-import { importX } from "./import-x.js"
-import { jest } from "./jest.js"
-import { mobx } from "./mobx.js"
-import { nodeN } from "./n.js"
-import { next } from "./next.js"
-import { packageJson } from "./package-json.js"
-import { playwright } from "./playwright.js"
-import { promise } from "./promise.js"
-import { react } from "./react.js"
-import { reactHooks } from "./react-hooks.js"
-import { rimac } from "./rimac.js"
-import { securityNode } from "./security-node.js"
-import { simpleImportSort } from "./simple-import-sort.js"
-import { sonarjs } from "./sonarjs.js"
-import { sonarjsAws } from "./sonarjs-aws.js"
-import { sortDestructureKeys } from "./sort-destructure-keys.js"
-import { sortKeysFix } from "./sort-keys-fix.js"
-import { storybook } from "./storybook.js"
-import { stylistic } from "./stylistic.js"
-import { typescriptEslint } from "./typescript-eslint.js"
-import { typescriptSortKeys } from "./typescript-sort-keys.js"
-import { unicorn } from "./unicorn.js"
-import { unusedImports } from "./unused-imports.js"
-import { validatePluginRules } from "./validate-rules.js"
-import { vitest } from "./vitest.js"
-
-const allPlugins = {
-    baseline,
-    dvukovic,
-    eslint,
-    eslintComments,
-    importX,
-    jest,
-    mobx,
-    next,
-    nodeN,
-    packageJson,
-    playwright,
-    promise,
-    react,
-    reactHooks,
-    rimac,
-    securityNode,
-    simpleImportSort,
-    sonarjs,
-    sonarjsAws,
-    sortDestructureKeys,
-    sortKeysFix,
-    storybook,
-    stylistic,
-    typescriptEslint,
-    typescriptSortKeys,
-    unicorn,
-    unusedImports,
-    vitest,
-}
-
-describe("plugin rules", () => {
-    for (const [name, config] of Object.entries(allPlugins)) {
-        test(`${name} has no invalid rules`, () => {
-            validatePluginRules(config)
-        })
-    }
-})