eslint-config-tungsten-flat 0.2.4 → 0.3.0

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,OAAO,EAA+B,MAAM,EAAE,MAAM,eAAe,CAAA;;AAMnE,wBA+FE"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AACA,OAAO,EAA+B,MAAM,EAAE,MAAM,eAAe,CAAA;;AAKnE,wBA0FE"}

package/dist/index.js

@@ -2,7 +2,6 @@ import { defineConfig, globalIgnores } from 'eslint/config';
 //export type { CompatibleConfigArray } from 'typescript-eslint' // would use this instead of `as ConfigArray` if typescript-eslint exported this type
 import tungstenJs from 'eslint-config-tungsten-js-flat';
 import tseslint from 'typescript-eslint';
-import pluginNoTypeAssertion from 'eslint-plugin-no-type-assertion';
 export default defineConfig([
     {
         extends: [tungstenJs]
@@ -29,11 +28,6 @@ export default defineConfig([
     //"plugin:@typescript-eslint/recommended",
     //"plugin:import/typescript",
     globalIgnores([".tungsten"]),
-    {
-        plugins: {
-            'no-type-assertion': pluginNoTypeAssertion
-        }
-    },
     {
         "rules": {
             "prefer-const": "warn", // this is set in tungsten-js already, but for some reason gets reset to error
@@ -42,15 +36,15 @@ export default defineConfig([
             //     "skipBlankLines": true, 
             //     "ignoreComments": true
             // }],
-            "import/no-unresolved": "off", // bugs out with typescript, + typescript does this already
-            "import/extensions": ["off"], // imports of external packages may require extension-less imports due to exports config 
+            "import-x/no-unresolved": "off", // bugs out with typescript, + typescript does this already
+            "import-x/extensions": ["off"], // imports of external packages may require extension-less imports due to exports config
             //"@typescript-eslint/ban-types": "warn", // removed
             // which one to use? this had warn, but a project had it off
             "@typescript-eslint/no-empty-object-type": "warn",
             //"@typescript-eslint/no-empty-object-type": "off",
             "@typescript-eslint/no-unsafe-function-type": "warn",
             "@typescript-eslint/no-wrapper-object-types": "warn",
-            "@typescript-eslint/no-empty-interface": "warn",
+            //"@typescript-eslint/no-empty-interface": "warn", // deprecated, superseded by no-empty-object-type above
             "@typescript-eslint/no-empty-function": "off",
             "@typescript-eslint/no-namespace": "warn",
             "@typescript-eslint/no-unnecessary-type-assertion": "warn",
@@ -78,8 +72,8 @@ export default defineConfig([
             '@typescript-eslint/no-floating-promises': 'warn',
             // these can give false positives due to eslint info being stale
             '@typescript-eslint/no-unsafe-assignment': 'warn',
-            // plugin-no-type-assertion
-            "no-type-assertion/no-type-assertion": "warn",
+            // replaces the old plugin-no-type-assertion rule: permits widening, flags narrowing
+            "@typescript-eslint/no-unsafe-type-assertion": "warn",
             // was using this in a project, but doesn't seem to change the underlying config
             //"react/no-unknown-property": ["warn", { ignore: ["for"] }]
             "react/no-unknown-property": "off"

package/package.json

@@ -1,21 +1,21 @@
 {
   "name": "eslint-config-tungsten-flat",
-  "version": "0.2.4",
+  "version": "0.3.0",
   "description": "",
   "type": "module",
+  "files": [
+    "dist"
+  ],
   "main": "dist/index.js",
   "keywords": [],
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "@typescript-eslint/parser": "^8.57.1",
-    "eslint": "^9.39.4",
-    "eslint-config-tungsten-js-flat": "^0.2.0",
-    "eslint-plugin-no-type-assertion": "^1.3.0",
+    "eslint-config-tungsten-js-flat": "^0.3.0",
     "typescript-eslint": "^8.57.1"
   },
   "peerDependencies": {
-    "eslint": "^9.39.4"
+    "eslint": "^9.0.0 || ^10.0.0"
   },
   "__OldPeerDependencies": {
     "@babel/core": "^7.18.5",
@@ -26,11 +26,11 @@
   },
   "devDependencies": {
     "@types/node": "^22.19.15",
+    "eslint": "^9.39.4",
     "typescript": "^5.9.3"
   },
   "scripts": {
     "compile": "tsc",
-    "build": "tsc --watch",
-    "prepublishonly": "tsc"
+    "build": "tsc --watch"
   }
 }

package/NOTES.md

@@ -1,91 +0,0 @@
-# eslint-config-tungsten-flat — maintenance notes
-
-## `eslint-plugin-no-type-assertion` peer warning
-
-### Symptom
-
-Every project that installs this config gets a pnpm peer warning:
-
-```
-✕ unmet peer eslint
-  Installed: 9.39.5
-  Wanted:
-    "^7.0.0 || ^8.0.0":
-      eslint-plugin-no-type-assertion@1.3.0
-```
-
-### Why it happens
-
-This package depends on `eslint-plugin-no-type-assertion@^1.3.0`, whose declared
-peer range stops at ESLint 8. The plugin is unmaintained — 1.3.0 is the last
-release (~2021) — so the range was never bumped for ESLint 9.
-
-### Is it a real incompatibility?
-
-No. The plugin is a leaf plugin with no version-coupled ESLint API. Its entire
-entry point is:
-
-```js
-module.exports.rules = {
-  "no-type-assertion": require("./rules/no-type-assertion"),
-};
-```
-
-and the rule itself is ~55 lines using only `meta` + `create(context)` with
-`context.report({ node, messageId })` and three AST visitors (`TSTypeAssertion`,
-`TSAsExpression`, `TSNonNullExpression`). All of that is unchanged in ESLint 9.
-The rule is already enabled and firing under ESLint 9 — see
-[`src/index.ts`](src/index.ts), which registers the plugin under the
-`no-type-assertion` namespace and sets
-`"no-type-assertion/no-type-assertion": "warn"`.
-
-So the warning is stale metadata, not a broken install.
-
-### Workaround for consumers (current fix)
-
-Suppress it in the consuming project. **Note for pnpm 10+/11:** the `pnpm` field
-in `package.json` is no longer read — pnpm warns and ignores it. These settings
-now live in `pnpm-workspace.yaml`:
-
-```yaml
-peerDependencyRules:
-  allowedVersions:
-    # Unmaintained leaf plugin; its peer range stops at eslint 8 but the rule
-    # uses no version-coupled API and works fine on 9.
-    'eslint-plugin-no-type-assertion>eslint': '9'
-```
-
-Applied in `qtc-2` on 2026-07-22.
-
-### Proper fix (not yet done)
-
-Vendor the rule into this package and drop the dependency. That fixes the
-warning at source for every consumer instead of requiring each one to suppress
-it. Steps:
-
-1. Copy the rule body into `src/rules/noTypeAssertion.ts` (it is ~55 lines of
-   plain JS; typing it against `Rule.RuleModule` is optional).
-2. Register it inline in `src/index.ts` in place of the imported plugin —
-   `{ rules: { 'no-type-assertion': noTypeAssertion } }` — keeping the same
-   `no-type-assertion/no-type-assertion` rule id so consumer configs and inline
-   disable comments keep working.
-3. Remove `eslint-plugin-no-type-assertion` from `dependencies` and delete its
-   entry from `src/modules.d.ts` (which only exists to silence the missing-types
-   error for the untyped plugin).
-4. Consumers can then drop the `peerDependencyRules` entry above.
-
-Licence is MIT (Kirill Korolyov,
-<https://github.com/Dremora/eslint-plugin-no-type-assertion>), so vendoring is
-fine — preserve the copyright notice in the vendored file.
-
-### Possible alternative (unevaluated)
-
-`typescript-eslint` has a built-in rule that may cover the same ground:
-
-- <https://typescript-eslint.io/rules/consistent-type-assertions/>
-
-`typescript-eslint` is already a dependency of this package, so adopting it
-would remove the third-party plugin without vendoring anything. Not yet
-reviewed — the rule's options and exactly how its behaviour compares to
-`no-type-assertion` (particularly the `as const` and non-null `!` cases the
-current rule special-cases) still need checking.

package/dist/test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=test.d.ts.map

package/dist/test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"test.d.ts","sourceRoot":"","sources":["../src/test.ts"],"names":[],"mappings":""}

package/dist/test.js

@@ -1,2 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });

package/src/index.ts

@@ -1,105 +0,0 @@
-
-import { defineConfig, globalIgnores, Config } from 'eslint/config'
-//export type { CompatibleConfigArray } from 'typescript-eslint' // would use this instead of `as ConfigArray` if typescript-eslint exported this type
-import tungstenJs from 'eslint-config-tungsten-js-flat'
-import tseslint from 'typescript-eslint'
-import pluginNoTypeAssertion from 'eslint-plugin-no-type-assertion'
-
-export default defineConfig([
-    {
-        extends: [tungstenJs]
-    },
-    //...tungstenJs,
-    ///...tseslint.configs.recommended,
-    tseslint.configs.recommendedTypeChecked as Config[],
-    {
-        languageOptions: {
-            parserOptions: {
-                //projectService: true,
-                projectService: {
-                    allowDefaultProject: [
-                        '*.config.js',
-                        //'eslint.config.js',
-                        //'tungsten.config.js',
-                        //'sxy-test-runner.config.js'
-                    ],
-                },
-                //tsconfigRootDir: import.meta.dirname,
-            },
-        },
-    },
-    
-    //"plugin:@typescript-eslint/recommended",
-    //"plugin:import/typescript",
-    globalIgnores([".tungsten"]),
-    { 
-        plugins: { 
-            'no-type-assertion': pluginNoTypeAssertion
-        }
-    },
-    {
-        "rules": {
-            "prefer-const": "warn", // this is set in tungsten-js already, but for some reason gets reset to error
-
-            // //// style
-            // "no-trailing-spaces" : ["warn", { 
-            //     "skipBlankLines": true, 
-            //     "ignoreComments": true
-            // }],
-            
-            "import/no-unresolved": "off", // bugs out with typescript, + typescript does this already
-            "import/extensions": ["off"], // imports of external packages may require extension-less imports due to exports config 
-            
-            //"@typescript-eslint/ban-types": "warn", // removed
-
-            // which one to use? this had warn, but a project had it off
-            "@typescript-eslint/no-empty-object-type": "warn",
-            //"@typescript-eslint/no-empty-object-type": "off",
-
-            "@typescript-eslint/no-unsafe-function-type": "warn",
-            "@typescript-eslint/no-wrapper-object-types": "warn",
-
-            "@typescript-eslint/no-empty-interface": "warn",
-            "@typescript-eslint/no-empty-function": "off",
-            "@typescript-eslint/no-namespace": "warn",
-            "@typescript-eslint/no-unnecessary-type-assertion": "warn",
-            "@typescript-eslint/strict-boolean-expressions": "warn",
-            "@typescript-eslint/no-this-alias": "warn",
-            "@typescript-eslint/ban-ts-comment": "off",
-            "@typescript-eslint/no-inferrable-types": "off",
-            "@typescript-eslint/no-explicit-any": "off",
-            // would like to change to _unused or unused_, but conflicts with the camel case rule
-            //"@typescript-eslint/no-unused-vars": ["warn", { "varsIgnorePattern": "^__*$|^unused$|^unused|.+unused$", "argsIgnorePattern": "^__*$|^unused$|^unused_.+|.+_unused$|^reject$" }],
-            // "^_" (any leading underscore) subsumes the older "^__*$" (which only matched a
-            // name that was entirely underscores), and covers the `_name` convention too
-            "@typescript-eslint/no-unused-vars": ["warn", { "varsIgnorePattern": "^_|unused$", "argsIgnorePattern": "^_|unused$|^reject$" }],
-            "@typescript-eslint/switch-exhaustiveness-check": "warn",
-            "@typescript-eslint/require-await": "warn",
-            "@typescript-eslint/no-unsafe-member-access": "warn",
-            "@typescript-eslint/no-unsafe-call": "warn",
-            "@typescript-eslint/no-unused-expressions": "warn",
-
-            // added TS rules
-            '@typescript-eslint/no-misused-spread': 'warn',
-
-            // fires on every event handler and JSX prop taking an async function,
-            // which is the normal shape in a tungsten app
-            '@typescript-eslint/no-misused-promises': 'off',
-            '@typescript-eslint/no-for-in-array': 'off',
-            '@typescript-eslint/no-floating-promises': 'warn',
-
-            // these can give false positives due to eslint info being stale
-            '@typescript-eslint/no-unsafe-assignment': 'warn',
-
-            // plugin-no-type-assertion
-            "no-type-assertion/no-type-assertion": "warn",
-
-            // was using this in a project, but doesn't seem to change the underlying config
-            //"react/no-unknown-property": ["warn", { ignore: ["for"] }]
-            "react/no-unknown-property": "off"
-        },
-    }
-
-])
-
-

package/src/modules.d.ts

@@ -1,2 +0,0 @@
-
-declare module 'eslint-plugin-no-type-assertion'

package/tsconfig.json

@@ -1,104 +0,0 @@
-{
-    "include": ["src"],
-    "compilerOptions": {
-        /* Visit https://aka.ms/tsconfig to read more about this file */
-
-        /* Projects */
-        // "incremental": true,                              /* Save .tsbuildinfo files to allow for incremental compilation of projects. */
-        // "composite": true,                                /* Enable constraints that allow a TypeScript project to be used with project references. */
-        // "tsBuildInfoFile": "./.tsbuildinfo",              /* Specify the path to .tsbuildinfo incremental compilation file. */
-        // "disableSourceOfProjectReferenceRedirect": true,  /* Disable preferring source files instead of declaration files when referencing composite projects. */
-        // "disableSolutionSearching": true,                 /* Opt a project out of multi-project reference checking when editing. */
-        // "disableReferencedProjectLoad": true,             /* Reduce the number of projects loaded automatically by TypeScript. */
-
-        /* Language and Environment */
-        "target": "es2022",                                  /* Set the JavaScript language version for emitted JavaScript and include compatible library declarations. */
-        // "lib": [],                                        /* Specify a set of bundled library declaration files that describe the target runtime environment. */
-        "jsx": "preserve",                                   /* Specify what JSX code is generated. */
-        // "experimentalDecorators": true,                   /* Enable experimental support for TC39 stage 2 draft decorators. */
-        // "emitDecoratorMetadata": true,                    /* Emit design-type metadata for decorated declarations in source files. */
-        // "jsxFactory": "",                                 /* Specify the JSX factory function used when targeting React JSX emit, e.g. 'React.createElement' or 'h'. */
-        // "jsxFragmentFactory": "",                         /* Specify the JSX Fragment reference used for fragments when targeting React JSX emit e.g. 'React.Fragment' or 'Fragment'. */
-        // "jsxImportSource": "",                            /* Specify module specifier used to import the JSX factory functions when using 'jsx: react-jsx*'. */
-        // "reactNamespace": "",                             /* Specify the object invoked for 'createElement'. This only applies when targeting 'react' JSX emit. */
-        // "noLib": true,                                    /* Disable including any library files, including the default lib.d.ts. */
-        "useDefineForClassFields": true,                     /* Emit ECMAScript-standard-compliant class fields. */
-        // "moduleDetection": "auto",                        /* Control what method is used to detect module-format JS files. */
-
-        /* Modules */
-        "module": "Node16",                                /* Specify what module code is generated. */
-        "rootDir": "./src",                                  /* Specify the root folder within your source files. */
-        // "moduleResolution": "NodeNext",                   /* Specify how TypeScript looks up a file from a given module specifier. */
-        // "baseUrl": "./",                                  /* Specify the base directory to resolve non-relative module names. */
-        // "paths": {},                                      /* Specify a set of entries that re-map imports to additional lookup locations. */
-        // "rootDirs": [],                                   /* Allow multiple folders to be treated as one when resolving modules. */
-        // "typeRoots": [],                                  /* Specify multiple folders that act like './node_modules/@types'. */
-        // "types": [],                                      /* Specify type package names to be included without being referenced in a source file. */
-        // "allowUmdGlobalAccess": true,                     /* Allow accessing UMD globals from modules. */
-        // "moduleSuffixes": [],                             /* List of file name suffixes to search when resolving a module. */
-        "resolveJsonModule": true,                           /* Enable importing .json files. */
-        // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */
-
-        /* JavaScript Support */
-        // "allowJs": true,                                  /* Allow JavaScript files to be a part of your program. Use the 'checkJS' option to get errors from these files. */
-        // "checkJs": true,                                  /* Enable error reporting in type-checked JavaScript files. */
-        // "maxNodeModuleJsDepth": 1,                        /* Specify the maximum folder depth used for checking JavaScript files from 'node_modules'. Only applicable with 'allowJs'. */
-
-        /* Emit */
-        "declaration": true,                              /* Generate .d.ts files from TypeScript and JavaScript files in your project. */
-        "declarationMap": true,                           /* Create sourcemaps for d.ts files. */
-        // "emitDeclarationOnly": true,                      /* Only output d.ts files and not JavaScript files. */
-        // "sourceMap": true,                                /* Create source map files for emitted JavaScript files. */
-        // "outFile": "./",                                  /* Specify a file that bundles all outputs into one JavaScript file. If 'declaration' is true, also designates a file that bundles all .d.ts output. */
-        "outDir": "./dist",                                  /* Specify an output folder for all emitted files. */
-        // "removeComments": true,                           /* Disable emitting comments. */
-        // "noEmit": true,                                   /* Disable emitting files from a compilation. */
-        // "importHelpers": true,                            /* Allow importing helper functions from tslib once per project, instead of including them per-file. */
-        // "importsNotUsedAsValues": "remove",               /* Specify emit/checking behavior for imports that are only used for types. */
-        // "downlevelIteration": true,                       /* Emit more compliant, but verbose and less performant JavaScript for iteration. */
-        // "sourceRoot": "",                                 /* Specify the root path for debuggers to find the reference source code. */
-        // "mapRoot": "",                                    /* Specify the location where debugger should locate map files instead of generated locations. */
-        // "inlineSourceMap": true,                          /* Include sourcemap files inside the emitted JavaScript. */
-        // "inlineSources": true,                            /* Include source code in the sourcemaps inside the emitted JavaScript. */
-        // "emitBOM": true,                                  /* Emit a UTF-8 Byte Order Mark (BOM) in the beginning of output files. */
-        // "newLine": "crlf",                                /* Set the newline character for emitting files. */
-        // "stripInternal": true,                            /* Disable emitting declarations that have '@internal' in their JSDoc comments. */
-        // "noEmitHelpers": true,                            /* Disable generating custom helper functions like '__extends' in compiled output. */
-        // "noEmitOnError": true,                            /* Disable emitting files if any type checking errors are reported. */
-        // "preserveConstEnums": true,                       /* Disable erasing 'const enum' declarations in generated code. */
-        // "declarationDir": "./",                           /* Specify the output directory for generated declaration files. */
-        // "preserveValueImports": true,                     /* Preserve unused imported values in the JavaScript output that would otherwise be removed. */
-
-        /* Interop Constraints */
-        // "isolatedModules": true,                          /* Ensure that each file can be safely transpiled without relying on other imports. */
-        // "allowSyntheticDefaultImports": true,             /* Allow 'import x from y' when a module doesn't have a default export. */
-        "esModuleInterop": true,                             /* Emit additional JavaScript to ease support for importing CommonJS modules. This enables 'allowSyntheticDefaultImports' for type compatibility. */
-        // "preserveSymlinks": true,                         /* Disable resolving symlinks to their realpath. This correlates to the same flag in node. */
-        "forceConsistentCasingInFileNames": true,            /* Ensure that casing is correct in imports. */
-
-        /* Type Checking */
-        "strict": true,                                      /* Enable all strict type-checking options. */
-        // "noImplicitAny": true,                            /* Enable error reporting for expressions and declarations with an implied 'any' type. */
-        // "strictNullChecks": true,                         /* When type checking, take into account 'null' and 'undefined'. */
-        // "strictFunctionTypes": true,                      /* When assigning functions, check to ensure parameters and the return values are subtype-compatible. */
-        // "strictBindCallApply": true,                      /* Check that the arguments for 'bind', 'call', and 'apply' methods match the original function. */
-        // "strictPropertyInitialization": true,             /* Check for class properties that are declared but not set in the constructor. */
-        // "noImplicitThis": true,                           /* Enable error reporting when 'this' is given the type 'any'. */
-        // "useUnknownInCatchVariables": true,               /* Default catch clause variables as 'unknown' instead of 'any'. */
-        // "alwaysStrict": true,                             /* Ensure 'use strict' is always emitted. */
-        // "noUnusedLocals": true,                           /* Enable error reporting when local variables aren't read. */
-        // "noUnusedParameters": true,                       /* Raise an error when a function parameter isn't read. */
-        "exactOptionalPropertyTypes": true,                  /* Interpret optional property types as written, rather than adding 'undefined'. */
-        // "noImplicitReturns": true,                        /* Enable error reporting for codepaths that do not explicitly return in a function. */
-        // "noFallthroughCasesInSwitch": true,                  /* Enable error reporting for fallthrough cases in switch statements. */
-        "noUncheckedIndexedAccess": true,                    /* Add 'undefined' to a type when accessed using an index. */
-        // "noImplicitOverride": true,                       /* Ensure overriding members in derived classes are marked with an override modifier. */
-        // "noPropertyAccessFromIndexSignature": true,       /* Enforces using indexed accessors for keys declared using an indexed type. */
-        // "allowUnusedLabels": true,                        /* Disable error reporting for unused labels. */
-        // "allowUnreachableCode": true,                     /* Disable error reporting for unreachable code. */
-
-        /* Completeness */
-        // "skipDefaultLibCheck": true,                      /* Skip type checking .d.ts files that are included with TypeScript. */
-        "skipLibCheck": true                                /* Skip type checking all .d.ts files. */
-    }
-}