Fix eslint configuration

Author: jdesrosiers

eslint.config.js

@@ -8,6 +8,7 @@ export default tseslint.config(
   ...tseslint.configs.recommendedTypeChecked,
   ...tseslint.configs.stylisticTypeChecked,
   importPlugin.flatConfigs.recommended, // eslint-disable-line @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-member-access
+  importPlugin.flatConfigs.typescript, // eslint-disable-line @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-member-access
   stylistic.configs.customize({
     arrowParens: true,
     braceStyle: "1tbs",
@@ -48,7 +49,7 @@ export default tseslint.config(
       // }], // Doesn't respect @import
 
       // Stylistic
-      "@stylistic/yield-star-spacing": ["error", "after"],
+      "@stylistic/generator-star-spacing": ["error", { after: true, before: false }],
       "@stylistic/multiline-ternary": "off",
       "@stylistic/no-mixed-operators": "off",
       "@stylistic/no-multiple-empty-lines": ["error", { max: 2, maxEOF: 0, maxBOF: 0 }], // Allow max=2 for imports

src/hyperjump/hyperjump.js

@@ -287,7 +287,7 @@ export class Hyperjump {
    *
    * @type (node: JsonCompatible<JrefNode>) => AsyncGenerator<JsonCompatible<JrefNode>, void, unknown>
    */
-  async * iter(node) {
+  async* iter(node) {
     if (node.jsonType === "array") {
       for (const itemNode of node.children) {
         yield this.#followReferences(itemNode);
@@ -303,7 +303,7 @@ export class Hyperjump {
    *
    * @type (node: JsonCompatible<JrefNode>) => AsyncGenerator<JsonCompatible<JrefNode>, void, unknown>
    */
-  async * values(node) {
+  async* values(node) {
     if (node.jsonType === "object") {
       for (const propertyNode of node.children) {
         yield this.#followReferences(propertyNode.children[1]);
@@ -317,7 +317,7 @@ export class Hyperjump {
    *
    * @type (node: JsonCompatible<JrefNode>) => AsyncGenerator<[string, JsonCompatible<JrefNode>], void, unknown>
    */
-  async * entries(node) {
+  async* entries(node) {
     if (node.jsonType === "object") {
       for (const propertyNode of node.children) {
         yield [propertyNode.children[0].value, await this.#followReferences(propertyNode.children[1])];

src/hyperjump/index.d.ts

@@ -0,0 +1,7 @@
+export * from "./hyperjump.js";
+export * from "./uri-schemes/uri-scheme-plugin.d.ts";
+export * from "./uri-schemes/file-scheme-plugin.js";
+export * from "./uri-schemes/http-scheme-plugin.js";
+export * from "./media-types/media-type-plugin.d.ts";
+export * from "./media-types/json-media-type-plugin.js";
+export * from "./media-types/jref-media-type-plugin.js";

src/hyperjump/index.js

@@ -3,11 +3,7 @@
  */
 
 export * from "./hyperjump.js";
-// eslint-disable-next-line import/extensions
-export * from "./uri-schemes/uri-scheme-plugin.js";
 export * from "./uri-schemes/file-scheme-plugin.js";
 export * from "./uri-schemes/http-scheme-plugin.js";
-// eslint-disable-next-line import/extensions
-export * from "./media-types/media-type-plugin.js";
 export * from "./media-types/json-media-type-plugin.js";
 export * from "./media-types/jref-media-type-plugin.js";

src/hyperjump/media-types/media-type-plugin.js

@@ -1,4 +1,4 @@
-import { GetOptions } from "../hyperjump.js"; // eslint-disable-line import/named
+import { GetOptions } from "../hyperjump.js";
 
 
 // eslint-disable-next-line @typescript-eslint/consistent-type-definitions

src/hyperjump/uri-schemes/uri-scheme-plugin.d.ts

@@ -0,0 +1,5 @@
+export * from "./jref-util.js";
+export * from "./jref-parse.js";
+export * from "./jref-stringify.js";
+export * from "./jref.js";
+export * from "./jref-ast.d.ts";

src/hyperjump/uri-schemes/uri-scheme-plugin.js

@@ -2,5 +2,3 @@ export * from "./jref-util.js";
 export * from "./jref-parse.js";
 export * from "./jref-stringify.js";
 export * from "./jref.js";
-// eslint-disable-next-line import/extensions
-export * from "./jref-ast.js";

src/jref/index.d.ts

@@ -0,0 +1,5 @@
+export * from "./jsonast-util.js";
+export * from "./rejson-parse.js";
+export * from "./rejson-stringify.js";
+export * from "./rejson.js";
+export * from "./jsonast.d.ts";

src/jref/index.js

@@ -2,5 +2,3 @@ export * from "./jsonast-util.js";
 export * from "./rejson-parse.js";
 export * from "./rejson-stringify.js";
 export * from "./rejson.js";
-// eslint-disable-next-line import/extensions
-export * from "./jsonast.js";

src/jref/jref-ast.js

@@ -61,7 +61,7 @@ export class JsonLexer {
       error: moo.error
     });
 
-    this.#iterator = (function * (lexer) {
+    this.#iterator = (function* (lexer) {
       for (const token of lexer.reset(json)) {
         if (token.type === "WS") {
           continue;

src/json/index.d.ts

@@ -17,15 +17,7 @@ import {
 } from "./index.js";
 
 /**
- * @import {
- *   JsonArrayNode,
- *   JsonBooleanNode,
- *   JsonNode,
- *   JsonNullNode,
- *   JsonNumberNode,
- *   JsonObjectNode,
- *   JsonStringNode
- * } from "./index.js"
+ * @import { JsonNode } from "./index.js"
  */
 
 

src/json/index.js

@@ -3,11 +3,9 @@
   "include": ["src/**/*"],
   "exclude": [
     "**/*.test.js",
-    "src/hyperjump/media-types/media-type-plugin.js",
-    "src/hyperjump/uri-schemes/uri-scheme-plugin.js",
-    "src/json/jsonast.js",
-    "src/json/types.js",
-    "src/jref/jref-ast.js"
+    "src/hyperjump/index.js",
+    "src/jref/index.js",
+    "src/json/index.js"
   ],
   "compilerOptions": {
     "declaration": true,