@playcanvas/attribute-parser 1.0.4 → 1.0.6

package/.eslintignore

@@ -0,0 +1 @@
+/**/program.invalid.js

package/.eslintrc

@@ -0,0 +1,19 @@
+{
+    "extends": [
+      "@playcanvas/eslint-config"
+    ],
+    "rules": {
+      "no-unused-expressions": "off",
+      "jsdoc/check-tag-names": "off",
+      "import/no-unresolved": "off"
+    },
+    "overrides": [
+        {
+            "files": ["./**/*.test.js"],
+            "rules": {
+              "prefer-arrow-callback": "off",
+          }
+        }
+      ]
+  }
+  

package/.github/workflows/ci.yml

@@ -24,5 +24,8 @@ jobs:
     - name: Install dependencies
       run: npm clean-install --progress=false --no-fund
 
+    - name: Run ESLint
+      run: npm run lint
+
     - name: Run unit tests
       run: npm test

package/package.json

@@ -5,18 +5,22 @@
   "engines": {
     "node": ">=18.0.0"
   },
-  "version": "1.0.4",
+  "version": "1.0.6",
   "dependencies": {
     "@microsoft/tsdoc": "^0.15.0",
+    "@playcanvas/eslint-config": "^1.7.4",
     "@typescript/vfs": "^1.6.0",
+    "eslint": "^8.56.0",
     "typescript": "^5.5.3"
   },
   "devDependencies": {
     "chai": "^5.1.1",
+    "globals": "^15.9.0",
     "mocha": "^10.6.0"
   },
   "scripts": {
-    "test": "mocha test/**/*.test.js --timeout 10000"
+    "test": "mocha test/**/*.test.js --timeout 10000",
+    "lint": "eslint ."
   },
   "publishConfig": {
     "access": "public"

package/src/index.js

@@ -1,22 +1,21 @@
-import * as ts from 'typescript';
 import { createSystem, createDefaultMapFromNodeModules, createVirtualTypeScriptEnvironment } from '@typescript/vfs';
-import globalTypes from './global.types.js';
+import * as ts from 'typescript';
+
 import { ScriptParser } from './parsers/script-parser.js';
 import { createDefaultMapFromCDN, flatMapAnyNodes, getExportedNodes, getType, inheritsFrom, isAliasedClassDeclaration } from './utils/ts-utils.js';
 
 const toLowerCamelCase = str => str[0].toLowerCase() + str.substring(1);
 
 const COMPILER_OPTIONS = {
-    strictPropertyInitialization: false, // Allow uninitialized properties
+    strict: false,
     skipLibCheck: true, // Skip type checking of declaration files
     target: ts.ScriptTarget.ES2022, // If this version changes, the types must be updated in the /rollup.config.mjs
     module: ts.ModuleKind.CommonJS,
     checkJs: true, // Enable JSDoc parsing
     allowJs: true,
-    noImplicitAny: false, // Allow implicit any
     baseUrl: './',
     paths: {
-        "playcanvas": ["/playcanvas.js"]
+        'playcanvas': ['/playcanvas.js']
     }
 };
 
@@ -35,7 +34,7 @@ export class JSDocParser {
 
     /**
      * Initializes the JSDocParser with the standard library files
-     * 
+     *
      * @param {string} libPath - The path to standard library files
      * @returns {Promise<JSDocParser>} - The initialized JSDocParser
      */
@@ -55,9 +54,6 @@ export class JSDocParser {
         const system = createSystem(fsMap);
         this._env = createVirtualTypeScriptEnvironment(system, Array.from(fsMap.keys()), ts, COMPILER_OPTIONS);
 
-        // Add global types to the parser
-        this._env.createFile('/global.d.ts', globalTypes);
-
         return this;
     }
 
@@ -100,7 +96,7 @@ export class JSDocParser {
 
     /**
      * Returns all the valid ESM Scripts within a file
-     * @param fileName - The file name in the program to check
+     * @param {string} fileName - The file name in the program to check
      * @returns {import('typescript').Node[]} - An array of any exported ESM Script nodes within the file
      */
     getAllEsmScripts(fileName) {
@@ -111,16 +107,16 @@ export class JSDocParser {
         const pcTypes = this.program.getSourceFile('/playcanvas.d.ts');
 
         if (!pcTypes) {
-            throw new Error(`PlayCanvas Types must be supplied`);
+            throw new Error('PlayCanvas Types must be supplied');
         }
-        
+
         // Parse the source file and pc types
         const sourceFile = this.program.getSourceFile(fileName);
-        
+
         if (!sourceFile) {
             throw new Error(`Source file ${fileName} not found`);
         }
-        
+
         // Extract all exported nodes
         const nodes = getExportedNodes(this.program, sourceFile);
 
@@ -141,7 +137,7 @@ export class JSDocParser {
         const results = {};
         const errors = [...this._errors];
 
-        if(errors.length > 0 ){
+        if (errors.length > 0) {
             return [results, errors];
         }
 
@@ -167,7 +163,7 @@ export class JSDocParser {
         const results = {};
         const errors = [...this._errors];
 
-        if(errors.length > 0 ){
+        if (errors.length > 0) {
             return [results, errors];
         }
 
@@ -205,7 +201,7 @@ export class JSDocParser {
                 }
 
                 // Extract JSDoc tags
-                const tags = member.jsDoc.map(jsdoc => jsdoc.tags?.map(tag => tag.tagName.getText()) ?? []).flat();
+                const tags = member.jsDoc ? member.jsDoc.map(jsdoc => jsdoc.tags?.map(tag => tag.tagName.getText()) ?? []).flat() : [];
 
                 const name = member.name.getText();
                 const type = getType(member, typeChecker);

package/src/parsers/attribute-parser.js

@@ -1,10 +1,10 @@
+import { TSDocConfiguration, TSDocTagDefinition, TSDocTagSyntaxKind, TextRange, TSDocParser } from '@microsoft/tsdoc';
 import * as ts from 'typescript';
-import { TSDocConfiguration, TSDocTagDefinition, TSDocTagSyntaxKind, TextRange, TSDocParser } from "@microsoft/tsdoc";
-import { extractTextFromDocNode, getLeadingBlockCommentRanges, getType } from '../utils/ts-utils.js';
 
-import { ParsingError } from "./parsing-error.js";
-import { hasTag } from "../utils/attribute-utils.js";
-import { parseTag, validateTag } from "../utils/tag-utils.js";
+import { ParsingError } from './parsing-error.js';
+import { hasTag } from '../utils/attribute-utils.js';
+import { parseTag, validateTag } from '../utils/tag-utils.js';
+import { extractTextFromDocNode, getLeadingBlockCommentRanges, getType } from '../utils/ts-utils.js';
 
 /**
  * A class to parse JSDoc comments and extract attribute metadata.
@@ -197,7 +197,7 @@ export class AttributeParser {
         } else {
             // Check for JSDoc annotations
             const jsDocs = ts.getJSDocTags(node);
-            const typeTag = jsDocs.find(doc => doc.tagName.text === "type");
+            const typeTag = jsDocs.find(doc => doc.tagName.text === 'type');
             if (typeTag && ts.isJSDocTypeTag(typeTag)) {
                 typeNode = typeTag.typeExpression;
             }
@@ -227,7 +227,7 @@ export class AttributeParser {
 
                         // Additionally check for JSDoc enum tag
                         const jsDocs = ts.getJSDocTags(declaration);
-                        const enumTag = jsDocs.find(doc => doc.tagName.text === "enum");
+                        const enumTag = jsDocs.find(doc => doc.tagName.text === 'enum');
                         if (enumTag && ts.isJSDocEnumTag(enumTag)) {
                             // This assumes enum members are correctly annotated in JSDoc comments
                             // Example: @enum {type} EnumName { Member1, Member2 }
@@ -245,10 +245,9 @@ export class AttributeParser {
      * Extract the public members and their values from a TypeScript node.
      *
      * @param {ts.Node} node - The node to parse
-     * @param {ParsingError[]} errors - An array to store any parsing errors
      * @returns {any[]} - The extracted members
      */
-    getMembers(node, errors = []) {
+    getMembers(node) {
 
         const members = [];
 

package/src/parsers/script-parser.js

@@ -1,11 +1,11 @@
-import * as ts from 'typescript';
 import { TextRange } from '@microsoft/tsdoc';
+import * as ts from 'typescript';
 
 import { AttributeParser } from './attribute-parser.js';
 import { ParsingError } from './parsing-error.js';
-import { zipArrays } from '../utils/generic-utils.js';
 import { hasTag } from '../utils/attribute-utils.js';
-import { flatMapAnyNodes, getJSDocCommentRanges, parseArrayLiteral, parseBooleanNode, parseFloatNode, parseStringNode } from "../utils/ts-utils.js";
+import { zipArrays } from '../utils/generic-utils.js';
+import { flatMapAnyNodes, getJSDocCommentRanges, parseArrayLiteral, parseBooleanNode, parseFloatNode, parseStringNode } from '../utils/ts-utils.js';
 
 /**
  * @typedef {object} Attribute
@@ -72,27 +72,27 @@ const createArrayArgumentParser = (name, defaultArr = []) => (
  * eg `new Vec3(1, 2, 3)`
  */
 const SUPPORTED_INITIALIZABLE_TYPE_NAMES = new Map([
-    ["Curve", createArrayArgumentParser("Curve")],
-    ["Vec2", createNumberArgumentParser("Vec2", [0, 0])],
-    ["Vec3", createNumberArgumentParser("Vec3", [0, 0, 0])],
-    ["Vec4", createNumberArgumentParser("Vec4", [0, 0, 0, 0])],
-    ["Color", createNumberArgumentParser("Color", [1, 1, 1, 1])],
-    ["number", parseFloatNode],
-    ["string", parseStringNode],
-    ["boolean", parseBooleanNode]
+    ['Curve', createArrayArgumentParser('Curve')],
+    ['Vec2', createNumberArgumentParser('Vec2', [0, 0])],
+    ['Vec3', createNumberArgumentParser('Vec3', [0, 0, 0])],
+    ['Vec4', createNumberArgumentParser('Vec4', [0, 0, 0, 0])],
+    ['Color', createNumberArgumentParser('Color', [1, 1, 1, 1])],
+    ['number', parseFloatNode],
+    ['string', parseStringNode],
+    ['boolean', parseBooleanNode]
 ]);
 
 /**
  * The full set of supported PlayCanvas types.
  */
 const SUPPORTED_PLAYCANVAS_TYPES = new Set([
-    "Curve",
-    "Asset",
-    "Entity",
-    "Vec2",
-    "Vec3",
-    "Vec4",
-    "Color"
+    'Curve',
+    'Asset',
+    'Entity',
+    'Vec2',
+    'Vec3',
+    'Vec4',
+    'Color'
 ]);
 
 /**
@@ -260,7 +260,8 @@ export class ScriptParser {
     validateProgram(errors = []) {
 
         // Report any compiler errors
-        const compilerDiagnostics = this.program.getSemanticDiagnostics();
+        const compilerDiagnostics = this.program.getSyntacticDiagnostics();
+
         if (compilerDiagnostics.length > 0) {
             for (const diagnostic of compilerDiagnostics) {
                 const message = ts.flattenDiagnosticMessageText(diagnostic.messageText, '\n');

package/src/utils/attribute-utils.js

@@ -13,26 +13,24 @@ export const getTagFromJsdoc = (tag, doc) => {
     return doc?.tags?.find(doc => doc.tagName.text === tag);
 };
 
-
 /**
  * Checks if a node has a specific JSDoc tag.
  * @param {string} tag - The tag to search for
  * @param {import("typescript").Node} node - The node to analyze
- * @returns {boolean} - True if the tag is found
+ * @returns {import('typescript').Node} - True if the tag is found
  */
-export const hasTag = (tag, node) => {
-    return !!getTag(tag, node);
+export const getTag = (tag, node) => {
+    return node?.jsDoc?.find(doc => getTagFromJsdoc(tag, doc));
 };
 
-
 /**
  * Checks if a node has a specific JSDoc tag.
  * @param {string} tag - The tag to search for
  * @param {import("typescript").Node} node - The node to analyze
- * @returns {import('typescript').Node} - True if the tag is found
+ * @returns {boolean} - True if the tag is found
  */
-export const getTag = (tag, node) => {
-    return node?.jsDoc?.find(doc => getTagFromJsdoc(tag, doc));
+export const hasTag = (tag, node) => {
+    return !!getTag(tag, node);
 };
 
 

package/src/utils/tag-utils.js

@@ -22,12 +22,12 @@ export function parseTag(input = '') {
     try {
         const arr = parseStringToNumericalArray(input);
         if (Array.isArray(arr)) return arr;
-    } catch (e) {}
+    } catch { /* eslint-disable-next-line no-empty */ }
 
     try {
         const arr = JSON.parse(input);
         if (Array.isArray(arr)) return arr;
-    } catch (e) {}
+    } catch { /* eslint-disable-next-line no-empty */ }
 
     // If none of the above, return the string
     return input;
@@ -45,7 +45,7 @@ export function parseTag(input = '') {
 export function validateTag(value, typeAnnotation, env) {
 
     const sourceText = `let a: ${typeAnnotation} = ${value};`;
-    env.createFile("/___virtual__.ts", sourceText);
+    env.createFile('/___virtual__.ts', sourceText);
 
     // Get the program and check for semantic errors
     const program = env.languageService.getProgram();

package/src/utils/ts-utils.js

@@ -1,6 +1,7 @@
 import { DocNodeKind } from '@microsoft/tsdoc';
 import { knownLibFilesForCompilerOptions } from '@typescript/vfs';
 import * as ts from 'typescript';
+
 import { ParsingError } from '../parsers/parsing-error.js';
 
 export function createDefaultMapFromCDN(options, prefix, ts) {
@@ -19,9 +20,7 @@ export function createDefaultMapFromCDN(options, prefix, ts) {
 
         const contents = await Promise.all(contentPromises);
 
-        contents.forEach(function (text, index) {
-            return fsMap.set('/' + files[index], text);
-        });
+        contents.forEach((text, index) => fsMap.set(`/${files[index]}`, text));
 
         return fsMap;
     }
@@ -87,7 +86,7 @@ export function isAliasedClassDeclaration(node, typeChecker) {
         return true;
     }
     // Try to resolve the original declaration
-    if (ts.isIdentifier(node.name)) {
+    if (node?.name && ts.isIdentifier(node.name)) {
         const symbol = typeChecker.getSymbolAtLocation(node.name);
         if (symbol) {
             const resolvedSymbol = resolveAliasedSymbol(typeChecker, symbol);

package/test/fixtures/asset.invalid.js

@@ -1,3 +1,4 @@
+// eslint-disable-next-line
 import { Script, Asset } from 'playcanvas';
 
 class Example extends Script {
@@ -21,7 +22,6 @@ class Example extends Script {
      * @resource
      */
     c;
-
 }
 
 export { Example };

package/test/fixtures/asset.valid.js

@@ -1,3 +1,4 @@
+// eslint-disable-next-line
 import { Script, Asset } from 'playcanvas';
 
 class Example extends Script {

package/test/fixtures/curve.invalid.js

@@ -1,3 +1,4 @@
+// eslint-disable-next-line
 import { Script, Curve } from 'playcanvas';
 
 class Example extends Script {

package/test/fixtures/enum.valid.js

@@ -12,15 +12,17 @@ const NumberEnum = {
 /**
  * @enum {string}
  */
+// eslint-disable-next-line
 const StringEnum = {
     A: 'a',
     B: 'b',
     C: 'c'
-}
+};
 
 /**
  * @enum {Vec3}
  */
+// eslint-disable-next-line
 const Vec3Enum = {
     A: new Vec3(1, 2, 3),
     B: new Vec3(4, 5, 6),
@@ -30,11 +32,12 @@ const Vec3Enum = {
 /**
  * @enum {NumberEnum}
  */
+// eslint-disable-next-line
 const NumberNumberEnum = {
     A: NumberEnum.A,
     B: NumberEnum.B,
     C: NumberEnum.C
-}
+};
 
 class Example extends Script {
     /**

package/test/fixtures/export.import.js

@@ -1,3 +1,3 @@
 import { Script } from 'playcanvas';
 
-export class ExampleImport extends Script {}
+export class ExampleImport extends Script {}

package/test/fixtures/export.valid.js

@@ -1,18 +1,18 @@
-
 import { Script } from 'playcanvas';
 
 import { ExampleImport } from './export.import.js';
 
-class ExampleImportExtend extends ExampleImport {};
+class ExampleImportExtend extends ExampleImport {}
 
 export { ExampleImport as ExampleImportExportAs } from './export.import.js';
 
-class Example extends Script {};
+class Example extends Script {}
 
-export default class ExampleDefault extends Script {};
+export default class ExampleDefault extends Script {}
 
 export class ExampleExport extends Script {}
 
+// eslint-disable-next-line
 class ExampleNotExported extends Script {}
 
 export {
@@ -20,4 +20,4 @@ export {
     Example as ExampleExportAs,
     ExampleImport,
     ExampleImportExtend
-}
+};

package/test/fixtures/inherit.valid.js

@@ -1,4 +1,3 @@
-
 import { Script } from 'playcanvas';
 
 class Example extends Script {
@@ -7,7 +6,7 @@ class Example extends Script {
      * @type {boolean}
      */
     a;
-};
+}
 
 class ExampleExtended extends Example {
     /**
@@ -15,7 +14,7 @@ class ExampleExtended extends Example {
      * @type {number}
      */
     b;
-};
+}
 
 class ExampleExtendedExtended extends ExampleExtended {
     /**

package/test/fixtures/json.invalid.js

@@ -1,5 +1,6 @@
 import { Script } from 'playcanvas';
 
+// eslint-disable-next-line
 class NoInterfaceFolder {
     /**
      * @attribute

package/test/fixtures/json.valid.js

@@ -3,6 +3,7 @@ import { Script, Vec3 } from 'playcanvas';
 /**
  * @interface
  */
+// eslint-disable-next-line
 class Folder {
     /**
      * @attribute
@@ -26,7 +27,9 @@ class Folder {
 
 /**
  * @interface
+ *
  */
+// eslint-disable-next-line
 class NestedFolder {
     /**
      * @attribute

package/test/fixtures/misc.invalid.js

@@ -1,7 +1,6 @@
-import { Script } from 'playcanvas';
-import { GraphNode } from './playcanvas';
+import { Script, GraphNode } from 'playcanvas';
 
-class Vec3 {};
+class Vec3 {}
 
 class Example extends Script {
     /**

package/test/fixtures/misc.valid.js

@@ -3,7 +3,7 @@ import { Script } from 'playcanvas';
 class Example extends Script {
     /**
      * This is a description of the attribute.
-     * 
+     *
      * @attribute
      * @type {number}
      * @title A number

package/test/fixtures/program.invalid.js

@@ -1,6 +1,7 @@
+/* eslint-disable */
 import { Script } from 'playcanvas';
 
-class Example extends Script {
+Class Intentionally extends Invalid {
     /**
      * @attribute
      * @type {boolean}

package/test/fixtures/program.valid.js

@@ -1,7 +1,9 @@
+import * as TWEEN from 'https://cdnjs.cloudflare.com/ajax/libs/tween.js/23.1.2/tween.esm.js';
+import confetti from 'https://esm.sh/canvas-confetti@1.6.0';
 import { Script } from 'playcanvas';
 
-// The parser should ignore https imports and not throw an error
-import confetti from "https://esm.sh/canvas-confetti@1.6.0"
+/** @enum {number} */
+export const MyEnum = { value: 0 };
 
 class Example extends Script {
     /**
@@ -12,6 +14,7 @@ class Example extends Script {
 
     initialize() {
         confetti();
+        new TWEEN.Tween({ x: 0 }).to({ x: 100 }, 1000).start();
     }
 }
 

package/test/fixtures/text.valid.js

@@ -1,3 +1,4 @@
+// eslint-disable
 import { Script } from 'playcanvas';
 
 class Example extends Script {

package/test/fixtures/vector.invalid.js

@@ -1,5 +1,6 @@
 import { Script, Vec3 } from 'playcanvas';
 
+// eslint-disable-next-line
 class Vec3Extended extends Vec3 {};
 
 class Example extends Script {

package/test/fixtures/vector.valid.js

@@ -1,3 +1,4 @@
+// eslint-disable-next-line
 import { Script, Vec2, Vec3, Vec4 } from 'playcanvas';
 
 class Example extends Script {

package/test/tests/invalid/asset.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/invalid/curve.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/invalid/json.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/invalid/misc.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/invalid/numeric.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/invalid/program.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 
@@ -13,6 +12,6 @@ describe('INVALID: Program ', function () {
     it('only errors should exist (1)', function () {
         expect(data).to.exist;
         expect(data[0]).to.be.empty;
-        expect(data[1].length).to.equal(1);
+        expect(data[1].length).to.equal(4);
     });
 });

package/test/tests/invalid/vector.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/asset.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/checkbox.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/color.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/curve.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/entity.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/enum.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/export.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/inherit.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/intellisense.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { getAttributes } from '../../utils.js';
 

package/test/tests/valid/json.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/misc.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/numeric.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/program.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/text.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/tests/valid/vector.test.js

@@ -1,6 +1,5 @@
-/* eslint-disable no-unused-expressions */
-import { describe, it, before } from 'mocha';
 import { expect } from 'chai';
+import { describe, it, before } from 'mocha';
 
 import { parseAttributes } from '../../utils.js';
 

package/test/utils.js

@@ -1,6 +1,7 @@
 import fs from 'fs/promises';
-import { fileURLToPath } from 'url';
 import path, { dirname } from 'path';
+import { fileURLToPath } from 'url';
+
 import { JSDocParser } from '../src/index.js';
 
 const __filename = fileURLToPath(import.meta.url);

package/src/global.types.js

@@ -1,18 +0,0 @@
-/**
- * These types are use to extend the global types in the program being parsed.
- * It's useful to add types to modules that are not written in TypeScript.
- */
-
-// This allows http/s modules to be imported without TypeScript errors
-
-export default `
-declare module 'http://*' {
-  const value: any;
-  export default value;
-}
-
-declare module 'https://*' {
-  const value: any;
-  export default value;
-}
-`