Fix types in source_manager and llm_manager packages

We were generating types wrongly and when using these packages types
were not recognized by the LSP (Language Server Protocol)

.changeset/loud-parents-roll.md

@@ -0,0 +1,6 @@
+---
+"@latitude-data/source-manager": minor
+"@latitude-data/llm-manager": minor
+---
+
+Fix types in Source Manager and LLManager

packages/llm_manager/package.json

@@ -30,6 +30,8 @@
     "@types/node": "^20.10.6",
     "mock-fs": "^5.2.0",
     "rollup": "^4.10.0",
+    "rollup-plugin-dts": "^6.1.0",
+    "@rollup/plugin-alias": "^5.1.0",
     "tslib": "^2.4.1",
     "typescript": "^5.4.5",
     "vite-tsconfig-paths": "^4.3.1",

packages/llm_manager/rollup.config.mjs

@@ -1,25 +1,43 @@
+import * as path from 'path'
+import * as url from 'url'
 import typescript from '@rollup/plugin-typescript'
+import { dts } from 'rollup-plugin-dts'
+import alias from '@rollup/plugin-alias'
 
-export default {
-  input: 'src/index.ts',
-  output: [
-    {
-      file: 'dist/index.js',
-      sourcemap: true,
-    },
-  ],
-  plugins: [
-    typescript({
-      exclude: ['**/__tests__', '**/*.test.ts'],
-    })
-  ],
-  external: [
-    'fs',
-    'path',
-    'yaml',
-    'openai',
-    '@latitude-data/source-manager',
-    '@latitude-data/sql-compiler',
-    'dotenv/config',
-  ],
+const __dirname = path.dirname(url.fileURLToPath(import.meta.url))
+const aliasEntries = {
+  entries: [{ find: '$', replacement: path.resolve(__dirname, 'src') }],
 }
+
+/** @type {import('rollup').RollupOptions[]} */
+export default [
+  {
+    input: 'src/index.ts',
+    output: [
+      {
+        file: 'dist/index.js',
+      },
+    ],
+    plugins: [
+      typescript({
+        noEmit: true,
+        tsconfig: './tsconfig.json',
+        exclude: ['**/__tests__', '**/*.test.ts'],
+      }),
+    ],
+    external: [
+      'fs',
+      'path',
+      'yaml',
+      'openai',
+      '@latitude-data/source-manager',
+      '@latitude-data/sql-compiler',
+      'dotenv/config',
+    ],
+  },
+  {
+    input: 'src/index.ts',
+    output: [{ file: 'dist/index.d.ts', format: 'es' }],
+    plugins: [alias(aliasEntries), dts()],
+  },
+]

packages/llm_manager/src/model/supportedMethods/readQuery.ts

@@ -36,7 +36,7 @@ const buildReadQueryMethod = ({
       // way to know which is which is by its appearance in the query
       const regex = new RegExp(`\\${resolvedAs}`, 'g')
 
-      query = query.replace(regex, value)
+      query = query.replace(regex, value as string)
     })
 
     return query

packages/llm_manager/src/types.ts

@@ -1,4 +1,5 @@
-import { Model } from './model'
+import { Model } from '$/model'
+
 export { CompileError } from '@latitude-data/sql-compiler'
 
 export enum ModelType {

packages/llm_manager/tsconfig.json

@@ -2,10 +2,11 @@
   "extends": "@latitude-data/typescript/base.json",
   "compilerOptions": {
     "baseUrl": ".",
+    "declaration": false,
+    "declarationMap": false,
     "rootDir": "./src",
     "outDir": "./dist",
     "paths": {
-      "$": ["./src"],
       "$/*": ["./src/*"]
     }
   },

packages/source_manager/package.json

@@ -31,12 +31,12 @@
     "@rollup/plugin-alias": "^5.1.0",
     "@rollup/plugin-commonjs": "^25.0.7",
     "@rollup/plugin-typescript": "^11.1.6",
+    "rollup-plugin-dts": "^6.1.0",
     "@types/mock-fs": "^4.13.4",
     "@types/node": "^20.10.6",
     "@types/uuid": "^9.0.8",
     "mock-fs": "^5.2.0",
     "rollup": "^4.10.0",
-    "rollup-plugin-dts": "^6.1.0",
     "uuid": "^9.0.1",
     "vite-tsconfig-paths": "^4.3.1",
     "vitest": "^1.2.2"

packages/source_manager/rollup.config.mjs

@@ -1,22 +1,28 @@
 import typescript from '@rollup/plugin-typescript'
+import * as path from 'path'
+import * as url from 'url'
 import alias from '@rollup/plugin-alias'
-import dts from 'rollup-plugin-dts'
+import { dts } from 'rollup-plugin-dts'
 import commonjs from '@rollup/plugin-commonjs'
 
+const __dirname = path.dirname(url.fileURLToPath(import.meta.url))
+const aliasEntries = {
+  entries: [{ find: '@', replacement: path.resolve(__dirname, 'src') }],
+}
+
+/** @type {import('rollup').RollupOptions[]} */
 export default [
   {
     input: 'src/index.ts',
     output: [
       {
         file: 'dist/index.js',
-        sourcemap: true,
       },
     ],
     plugins: [
       typescript({
+        noEmit: true,
         tsconfig: './tsconfig.json',
-        declaration: true,
-        declarationDir: './dist/types',
         exclude: ['**/__tests__', '**/*.test.ts', '**/*.d.ts'],
       }),
       commonjs(),
@@ -32,21 +38,12 @@ export default [
       '@latitude-data/sql-compiler',
       '@latitude-data/query_result',
       '@latitude-data/storage-driver',
-    '@dsnp/parquetjs',
+      '@dsnp/parquetjs',
     ],
   },
   {
-    input: 'dist/types/index.d.ts',
-    output: {
-      file: 'dist/index.d.ts',
-      format: 'es',
-    },
-    plugins: [
-      alias({
-        entries: [{ find: '@', replacement: './src/' }],
-      }),
-      dts(),
-    ],
-    external: (id) => id.startsWith('@'),
+    input: 'src/index.ts',
+    output: [{ file: 'dist/index.d.ts', format: 'es' }],
+    plugins: [alias(aliasEntries), dts()],
   },
 ]

packages/source_manager/tsconfig.json

@@ -1,6 +1,8 @@
 {
   "extends": "@latitude-data/typescript/base.json",
   "compilerOptions": {
+    "declaration": false,
+    "declarationMap": false,
     "baseUrl": ".",
     "rootDir": "./src",
     "outDir": "./dist",