Merge pull request #1 from daniel-lxs/semantic-search-cleanup

Semantic search cleanup

Author: daniel-lxs

.eslintrc.json

@@ -3,21 +3,31 @@
 	"parser": "@typescript-eslint/parser",
 	"parserOptions": {
 		"ecmaVersion": 6,
-		"sourceType": "module"
+		"sourceType": "module",
+		"project": "./tsconfig.json"
 	},
 	"plugins": ["@typescript-eslint"],
 	"rules": {
-		"@typescript-eslint/naming-convention": [
-			"warn",
-			{
-				"selector": "import",
-				"format": ["camelCase", "PascalCase"]
-			}
-		],
+		"no-var": "error",
+		"no-unused-vars": "off",
+		"@typescript-eslint/no-unused-vars": "warn",
+		"@typescript-eslint/no-explicit-any": "warn",
+		"@typescript-eslint/no-misused-promises": "error",
+		"@typescript-eslint/await-thenable": "warn",
+		"@typescript-eslint/return-await": "error",
+		"@typescript-eslint/explicit-function-return-type": "warn",
 		"@typescript-eslint/semi": "off",
 		"eqeqeq": "warn",
 		"no-throw-literal": "warn",
 		"semi": "off"
 	},
-	"ignorePatterns": ["out", "dist", "**/*.d.ts"]
+	"ignorePatterns": ["out", "dist", "**/*.d.ts", "jest.*.js"],
+	"overrides": [
+		{
+			"files": ["webview-ui/src/**/*.tsx"],
+			"parserOptions": {
+				"project": "./webview-ui/tsconfig.json"
+			}
+		}
+	]
 }

.husky/pre-commit

@@ -1,4 +1,5 @@
 const esbuild = require("esbuild")
+const { copy } = require("esbuild-plugin-copy")
 const fs = require("fs")
 const path = require("path")
 
@@ -25,40 +26,14 @@ const esbuildProblemMatcherPlugin = {
 	},
 }
 
-const copyWasmFiles = {
-	name: "copy-wasm-files",
+// Plugin to handle native .node files
+const nativeNodeModulesPlugin = {
+	name: "native-node-modules",
 	setup(build) {
-		build.onEnd(() => {
-			// tree sitter
-			const sourceDir = path.join(__dirname, "node_modules", "web-tree-sitter")
-			const targetDir = path.join(__dirname, "dist")
-
-			// Copy tree-sitter.wasm
-			fs.copyFileSync(path.join(sourceDir, "tree-sitter.wasm"), path.join(targetDir, "tree-sitter.wasm"))
-
-			// Copy language-specific WASM files
-			const languageWasmDir = path.join(__dirname, "node_modules", "tree-sitter-wasms", "out")
-			const languages = [
-				"typescript",
-				"tsx",
-				"python",
-				"rust",
-				"javascript",
-				"go",
-				"cpp",
-				"c",
-				"c_sharp",
-				"ruby",
-				"java",
-				"php",
-				"swift",
-			]
-
-			languages.forEach((lang) => {
-				const filename = `tree-sitter-${lang}.wasm`
-				fs.copyFileSync(path.join(languageWasmDir, filename), path.join(targetDir, filename))
-			})
-		})
+		build.onResolve({ filter: /\.node$/ }, (args) => ({
+			path: args.path,
+			external: true, // Already correctly marked as external
+		}))
 	},
 }
 
@@ -67,9 +42,27 @@ const extensionConfig = {
 	minify: production,
 	sourcemap: !production,
 	logLevel: "silent",
+	assetNames: "[name]",
 	plugins: [
-		copyWasmFiles,
-		/* add to the end of plugins array */
+		copy({
+			resolveFrom: "cwd",
+			assets: [
+				{
+					from: "./node_modules/tree-sitter-wasms/out/**/*.wasm",
+					to: "./dist/",
+				},
+				{
+					from: "./node_modules/web-tree-sitter/tree-sitter.wasm",
+					to: "./dist/",
+				},
+				{
+					from: "./node_modules/@lancedb/**/*.node",
+					to: "./dist/",
+				},
+			],
+			watch: true,
+		}),
+		nativeNodeModulesPlugin,
 		esbuildProblemMatcherPlugin,
 	],
 	entryPoints: ["src/extension.ts"],
@@ -78,6 +71,7 @@ const extensionConfig = {
 	platform: "node",
 	outfile: "dist/extension.js",
 	external: ["vscode"],
+	loader: { ".node": "file" },
 }
 
 async function main() {

esbuild.js

@@ -7,14 +7,7 @@ module.exports = {
 		"^.+\\.tsx?$": [
 			"ts-jest",
 			{
-				tsconfig: {
-					module: "CommonJS",
-					moduleResolution: "node",
-					esModuleInterop: true,
-					allowJs: true,
-				},
-				diagnostics: false,
-				isolatedModules: true,
+				tsconfig: "tsconfig.json",
 			},
 		],
 	},
@@ -30,9 +23,11 @@ module.exports = {
 		"^strip-ansi$": "<rootDir>/src/__mocks__/strip-ansi.js",
 		"^default-shell$": "<rootDir>/src/__mocks__/default-shell.js",
 		"^os-name$": "<rootDir>/src/__mocks__/os-name.js",
+		"^(\\.{1,2}/.*)\\.js$": "$1",
+		"^@xenova/transformers$": "<rootDir>/src/services/semantic-search/embeddings/__mocks__/transformers.ts",
 	},
 	transformIgnorePatterns: [
-		"node_modules/(?!(@modelcontextprotocol|delay|p-wait-for|globby|serialize-error|strip-ansi|default-shell|os-name)/)",
+		"node_modules/(?!(@modelcontextprotocol|delay|p-wait-for|globby|serialize-error|strip-ansi|default-shell|os-name|@xenova/transformers)/)",
 	],
 	roots: ["<rootDir>/src", "<rootDir>/webview-ui/src"],
 	modulePathIgnorePatterns: [".vscode-test"],