add eslint rules for ts parser

Author: angrychow

ts-parser/.eslintrc.json

@@ -0,0 +1,34 @@
+{
+  "root": true,
+  "parser": "@typescript-eslint/parser",
+  "plugins": ["@typescript-eslint"],
+  "extends": [
+    "eslint:recommended",
+    "plugin:@typescript-eslint/recommended",
+    "prettier"
+  ],
+  "parserOptions": {
+    "ecmaVersion": 2022,
+    "sourceType": "module"
+  },
+  "rules": {
+    "@typescript-eslint/no-unused-vars": ["error", { "argsIgnorePattern": "^_" }],
+    "@typescript-eslint/no-explicit-any": "warn",
+    "prefer-const": "error",
+    "no-var": "error",
+    "object-shorthand": "error",
+    "prefer-arrow-callback": "error",
+    "@typescript-eslint/ban-types": "off"
+  },
+  "env": {
+    "node": true,
+    "es2022": true
+  },
+  "ignorePatterns": [
+    "dist/",
+    "node_modules/",
+    "**/*.js",
+    "**/*.d.ts",
+    "coverage/"
+  ]
+}

ts-parser/.prettierignore

@@ -0,0 +1,33 @@
+# Dependencies
+node_modules/
+
+# Build outputs
+dist/
+build/
+
+# Test coverage
+coverage/
+
+# Logs
+*.log
+
+# Environment files
+.env*
+
+# Package files
+package-lock.json
+yarn.lock
+
+# Generated files
+*.d.ts
+*.js.map
+*.ts.map
+
+# Temporary files
+*.tmp
+*.temp
+.DS_Store
+
+# IDE
+.vscode/
+.idea/

ts-parser/.prettierrc

@@ -0,0 +1,10 @@
+{
+  "semi": true,
+  "trailingComma": "es5",
+  "singleQuote": true,
+  "printWidth": 100,
+  "tabWidth": 2,
+  "useTabs": false,
+  "bracketSpacing": true,
+  "arrowParens": "avoid"
+}

ts-parser/src/parser/FunctionParser.ts

@@ -58,7 +58,7 @@ export class FunctionParser {
     // Parse method declarations in classes
     const classes = sourceFile.getClasses();
     for (const cls of classes) {
-      let sym = cls.getSymbol();
+      const sym = cls.getSymbol();
       let className = ""
       if (sym) {
         className = assignSymbolName(sym)
@@ -104,7 +104,7 @@ export class FunctionParser {
     for (const varDecl of variableDeclarations) {
       const initializer = varDecl.getInitializer();
       if (initializer && (Node.isArrowFunction(initializer) || Node.isFunctionExpression(initializer))) {
-        let sym = varDecl.getSymbol()
+        const sym = varDecl.getSymbol()
         let funcName = ""
         if (sym) {
           funcName = assignSymbolName(sym)
@@ -140,7 +140,7 @@ export class FunctionParser {
   }
 
   private parseFunction(func: FunctionDeclaration, moduleName: string, packagePath: string, sourceFile: SourceFile): UniFunction {
-    let symbol = func.getSymbol();
+    const symbol = func.getSymbol();
     let name = 'anonymous_' + func.getStart();
     if (symbol) {
       name = assignSymbolName(symbol)
@@ -189,7 +189,7 @@ export class FunctionParser {
   }
 
   private parseMethod(method: MethodDeclaration, moduleName: string, packagePath: string, sourceFile: SourceFile, className: string): UniFunction {
-    let symbol = method.getSymbol();
+    const symbol = method.getSymbol();
     let methodName = ""
     if (symbol) {
       methodName = assignSymbolName(symbol)
@@ -254,7 +254,7 @@ export class FunctionParser {
   }
 
   private parseInterfaceMethod(method: MethodSignature, moduleName: string, packagePath: string, sourceFile: SourceFile): UniFunction {
-    let symbol = method.getSymbol();
+    const symbol = method.getSymbol();
     let methodName = ""
     if (symbol) {
       methodName = assignSymbolName(symbol)
@@ -290,7 +290,7 @@ export class FunctionParser {
   }
 
   private parseConstructor(ctor: ConstructorDeclaration, moduleName: string, packagePath: string, sourceFile: SourceFile, className: string): UniFunction {
-    let symbol = ctor.getSymbol();
+    const symbol = ctor.getSymbol();
     let name = ""
     if (symbol) {
       name = assignSymbolName(symbol)
@@ -386,14 +386,14 @@ export class FunctionParser {
   }
 
   // TODO: parse parameters
-  private parseParameters(parameters: ParameterDeclaration[], moduleName: string, packagePath: string, sourceFile: SourceFile): Dependency[] {
+  private parseParameters(_parameters: ParameterDeclaration[], _moduleName: string, _packagePath: string, _sourceFile: SourceFile): Dependency[] {
     const dependencies: Dependency[] = [];
 
     return dependencies;
   }
 
   // TODO: parse return types
-  private parseReturnTypes(func: FunctionDeclaration | MethodSignature, moduleName: string, packagePath: string, sourceFile: SourceFile): Dependency[] {
+  private parseReturnTypes(_func: FunctionDeclaration | MethodSignature, _moduleName: string, _packagePath: string, _sourceFile: SourceFile): Dependency[] {
     const results: Dependency[] = [];
     return results;
   }
@@ -402,7 +402,7 @@ export class FunctionParser {
     node: FunctionDeclaration | MethodDeclaration | ConstructorDeclaration | ArrowFunction | FunctionExpression,
     moduleName: string,
     packagePath: string,
-    sourceFile: SourceFile
+    _sourceFile: SourceFile
   ): Dependency[] {
     const calls: Dependency[] = [];
     const visited = new Set<string>();
@@ -429,7 +429,7 @@ export class FunctionParser {
       }
 
       visited.add(key);
-      let dep: Dependency = {
+      const dep: Dependency = {
         ModPath: resolvedSymbol.moduleName || moduleName,
         PkgPath: this.getPkgPath(resolvedSymbol.packagePath || packagePath),
         Name: resolvedSymbol.name,
@@ -509,7 +509,7 @@ export class FunctionParser {
     newExpr: Identifier,
     moduleName: string,
     packagePath: string,
-    sourceFile: SourceFile,
+    _sourceFile: SourceFile,
     calls: Dependency[],
     visited: Set<string>
   ): void {
@@ -521,7 +521,7 @@ export class FunctionParser {
     if (!resolvedSymbol) return;
 
     // Handle method names like 'getX' -> 'getX'
-    let nameFormat = resolvedSymbol.name;
+    const nameFormat = resolvedSymbol.name;
 
     const key = `${resolvedSymbol.moduleName}?${resolvedSymbol.packagePath}#${nameFormat}`;
 
@@ -530,7 +530,7 @@ export class FunctionParser {
     }
     visited.add(key);
 
-    let dep: Dependency = {
+    const dep: Dependency = {
       ModPath: resolvedSymbol.moduleName || moduleName,
       PkgPath: this.getPkgPath(resolvedSymbol.packagePath || packagePath),
       Name: nameFormat,
@@ -566,7 +566,7 @@ export class FunctionParser {
     propAccess: PropertyAccessExpression,
     moduleName: string,
     packagePath: string,
-    sourceFile: SourceFile,
+    _sourceFile: SourceFile,
     calls: Dependency[],
     visited: Set<string>
   ): void {
@@ -578,7 +578,7 @@ export class FunctionParser {
     if (!resolvedSymbol) return;
 
     // Handle method names like 'getX' -> 'getX'
-    let nameFormat = resolvedSymbol.name;
+    const nameFormat = resolvedSymbol.name;
 
     const key = `${resolvedSymbol.moduleName}?${resolvedSymbol.packagePath}#${nameFormat}`;
 
@@ -587,7 +587,7 @@ export class FunctionParser {
     }
     visited.add(key);
 
-    let dep: Dependency = {
+    const dep: Dependency = {
       ModPath: resolvedSymbol.moduleName || moduleName,
       PkgPath: this.getPkgPath(resolvedSymbol.packagePath || packagePath),
       Name: nameFormat,
@@ -623,7 +623,7 @@ export class FunctionParser {
     node: FunctionDeclaration | MethodDeclaration | ConstructorDeclaration | ArrowFunction | FunctionExpression,
     moduleName: string,
     packagePath: string,
-    sourceFile: SourceFile
+    _sourceFile: SourceFile
   ): Dependency[] {
     const types: Dependency[] = [];
     const visited = new Set<string>();
@@ -683,7 +683,7 @@ export class FunctionParser {
         }
 
         visited.add(key);
-        let dep: Dependency = {
+        const dep: Dependency = {
           ModPath: resolvedSymbol.moduleName || moduleName,
           PkgPath: this.getPkgPath(resolvedSymbol.packagePath || packagePath),
           Name: resolvedSymbol.name,
@@ -712,7 +712,7 @@ export class FunctionParser {
     node: FunctionDeclaration | MethodDeclaration | ConstructorDeclaration | ArrowFunction | FunctionExpression,
     moduleName: string,
     packagePath: string,
-    sourceFile: SourceFile
+    _sourceFile: SourceFile
   ): Dependency[] {
     const body = node.getBody();
     if (!body) return [];
@@ -795,7 +795,7 @@ export class FunctionParser {
 
       visited.add(key);
 
-      let dep: Dependency = {
+      const dep: Dependency = {
         ModPath: resolvedSymbol.moduleName || moduleName,
         PkgPath: this.getPkgPath(resolvedSymbol.packagePath || packagePath),
         Name: resolvedSymbol.name,

ts-parser/src/parser/ModuleParser.ts

@@ -1,7 +1,7 @@
 import * as path from 'path';
 import * as fs from 'fs';
-import { Project, SourceFile, SyntaxKind } from 'ts-morph';
-import { Module, Package, Import } from '../types/uniast';
+import { Project, SourceFile } from 'ts-morph';
+import { Module, Package } from '../types/uniast';
 import { PackageParser } from './PackageParser';
 import { TypeScriptStructureAnalyzer } from '../utils/typescript-structure';
 import { TsConfigCache } from '../utils/tsconfig-cache';
@@ -72,7 +72,7 @@ export class ModuleParser {
       Object.assign(dependencies, packageJson.peerDependencies);
     }
 
-    let pathUitl = new PathUtils(this.projectRoot)
+    const pathUitl = new PathUtils(this.projectRoot)
 
     // Build files map with detailed import analysis
     const files: Record<string, any> = {};
@@ -99,12 +99,8 @@ export class ModuleParser {
     };
   }
 
-  private extractImports(sourceFile: SourceFile, modulePath: string): Array<{ Path: string }> {
+  private extractImports(sourceFile: SourceFile, _modulePath: string): Array<{ Path: string }> {
     const imports: Array<{ Path: string }> = [];
-    const sourceFilePath = sourceFile.getFilePath();
-    
-    // Cache path aliases for this module
-    const pathAliases = this.tsConfigCache.getPathAliases(modulePath);
 
     // Track unique import paths to avoid duplicates
     const uniquePaths = new Set<string>();
@@ -115,7 +111,12 @@ export class ModuleParser {
       const originalPath = importDecl.getModuleSpecifierValue();
       const resolvedPath = importDecl.getModuleSpecifierSourceFile()?.getFilePath();
       if (resolvedPath) {
-        let relativePath = path.relative(this.projectRoot, resolvedPath);
+        const relativePath = path.relative(this.projectRoot, resolvedPath);
+
+        if(uniquePaths.has(relativePath)) {
+          continue;
+        }
+        uniquePaths.add(relativePath);
         imports.push({ Path: relativePath });
       } else {
         imports.push({ Path: "external:" + originalPath });
@@ -129,7 +130,7 @@ export class ModuleParser {
       if (originalPath) {
         const resolvedPath = exportDecl.getModuleSpecifierSourceFile()?.getFilePath();
         if (resolvedPath) {
-          let relativePath = path.relative(this.projectRoot, resolvedPath);
+          const relativePath = path.relative(this.projectRoot, resolvedPath);
           imports.push({ Path: relativePath });
         } else {
           imports.push({ Path: "external:" + originalPath });