@toiroakr/lines-db 0.1.2 → 0.2.1

package/src/sqlite-adapter.ts

@@ -1,5 +1,5 @@
 /**
- * SQLite adapter that works across Node.js, Bun, and Deno
+ * SQLite adapter for Node.js
  */
 
 import { RUNTIME } from './runtime.js';
@@ -20,51 +20,16 @@ export interface SQLiteStatement {
 }
 
 /**
- * Create a SQLite database instance based on the runtime environment
+ * Create a SQLite database instance for Node.js
  */
 export function createDatabase(path: string = ':memory:'): SQLiteDatabase {
-  if (RUNTIME === 'bun') {
-    return createBunDatabase(path);
-  } else if (RUNTIME === 'node' || RUNTIME === 'deno') {
+  if (RUNTIME === 'node') {
     return createNodeDatabase(path);
   } else {
     throw new Error(`Unsupported runtime: ${RUNTIME}`);
   }
 }
 
-/**
- * Create a Bun SQLite database
- */
-function createBunDatabase(path: string): SQLiteDatabase {
-  // Dynamic import for Bun
-  // eslint-disable-next-line @typescript-eslint/no-require-imports
-  const { Database } = require('bun:sqlite');
-  const db = new Database(path);
-
-  return {
-    prepare(sql: string): SQLiteStatement {
-      const stmt = db.prepare(sql);
-      return {
-        run(...params: any[]) {
-          return stmt.run(...params);
-        },
-        get(...params: any[]) {
-          return stmt.get(...params);
-        },
-        all(...params: any[]) {
-          return stmt.all(...params);
-        },
-      };
-    },
-    exec(sql: string): void {
-      db.exec(sql);
-    },
-    close(): void {
-      db.close();
-    },
-  };
-}
-
 /**
  * Create a Node.js SQLite database
  */

package/src/type-generator.ts

@@ -71,9 +71,7 @@ export class TypeGenerator {
           const schemaFilePath = join(this.dataDirPath, schemaFileName);
 
           // Check if schema file exists
-          const hasSchema = entries.some(
-            e => e.isFile() && e.name === schemaFileName
-          );
+          const hasSchema = entries.some((e) => e.isFile() && e.name === schemaFileName);
 
           tables.push({
             tableName,
@@ -137,7 +135,7 @@ export class TypeGenerator {
     return `// Auto-generated by lines-db
 // Do not edit this file manually
 
-${importSection}import type { DatabaseConfig${inferOutputImport} } from 'lines-db';
+${importSection}import type { DatabaseConfig${inferOutputImport} } from '@toiroakr/lines-db';
 import { fileURLToPath } from 'node:url';
 import { dirname } from 'node:path';
 

package/src/validator.test.ts

@@ -46,6 +46,7 @@ describe('Validator', () => {
 
       expect(result.valid).toBe(true);
       expect(result.errors).toHaveLength(0);
+      expect(result.warnings).toHaveLength(0);
     });
 
     it('should detect validation errors', async () => {
@@ -77,7 +78,7 @@ describe('Validator', () => {
 
       expect(result.valid).toBe(false);
       expect(result.errors).toHaveLength(1);
-      expect(result.errors[0].rowIndex).toBe(2);
+      expect(result.errors[0].rowIndex).toBe(1); // 0-indexed: line 2 is index 1
       expect(result.errors[0].tableName).toBe('users');
       expect(result.errors[0].issues).toHaveLength(1);
     });
@@ -119,8 +120,8 @@ describe('Validator', () => {
 
       expect(result.valid).toBe(false);
       expect(result.errors).toHaveLength(2);
-      expect(result.errors[0].rowIndex).toBe(1);
-      expect(result.errors[1].rowIndex).toBe(2);
+      expect(result.errors[0].rowIndex).toBe(0); // 0-indexed: line 1 is index 0
+      expect(result.errors[1].rowIndex).toBe(1); // 0-indexed: line 2 is index 1
     });
 
     it('should return 0-based rowIndex (first line = index 0)', async () => {
@@ -128,10 +129,7 @@ describe('Validator', () => {
       const schemaPath = join(testDir, 'users.schema.ts');
 
       // Line 1: valid, Line 2: invalid (no name), Line 3: invalid (no name)
-      await writeFile(
-        jsonlPath,
-        '{"id":1,"name":"Alice"}\n{"id":2}\n{"id":3}\n',
-      );
+      await writeFile(jsonlPath, '{"id":1,"name":"Alice"}\n{"id":2}\n{"id":3}\n');
       await writeFile(
         schemaPath,
         `
@@ -239,6 +237,7 @@ describe('Validator', () => {
 
       expect(result.valid).toBe(true);
       expect(result.errors).toHaveLength(0);
+      expect(result.warnings).toHaveLength(0);
     });
 
     it('should collect errors from multiple files', async () => {
@@ -295,6 +294,37 @@ describe('Validator', () => {
 
       await expect(validator.validate()).rejects.toThrow('No JSONL files found');
     });
+
+    it('should skip validation and warn for JSONL files without schema', async () => {
+      const usersPath = join(testDir, 'users.jsonl');
+      const productsPath = join(testDir, 'products.jsonl');
+      const usersSchemaPath = join(testDir, 'users.schema.ts');
+
+      // Only users.jsonl has a schema
+      await writeFile(usersPath, '{"id":1,"name":"Alice"}\n');
+      await writeFile(productsPath, '{"id":1,"name":"Product"}\n');
+      await writeFile(
+        usersSchemaPath,
+        `
+        export const schema = {
+          '~standard': {
+            version: 1,
+            vendor: 'test',
+            validate: (data) => ({ value: data, issues: [] })
+          }
+        };
+      `,
+      );
+
+      const validator = new Validator({ path: testDir });
+      const result = await validator.validate();
+
+      expect(result.valid).toBe(true);
+      expect(result.errors).toHaveLength(0);
+      expect(result.warnings).toHaveLength(1);
+      expect(result.warnings[0]).toContain('products');
+      expect(result.warnings[0]).toContain('schema file not found');
+    });
   });
 
   describe('error handling', () => {

package/src/validator.ts

@@ -8,6 +8,7 @@ import type { BiDirectionalSchema } from './schema.js';
 export interface ValidationResult {
   valid: boolean;
   errors: ValidationErrorDetail[];
+  warnings: string[];
 }
 
 export interface ValidationErrorDetail {
@@ -69,20 +70,37 @@ export class Validator {
     }
 
     const allErrors: ValidationErrorDetail[] = [];
+    const allWarnings: string[] = [];
+    const filesWithSchema: string[] = [];
 
-    // First, validate schema for each file
+    // Filter files with schema and collect warnings for files without schema
     for (const file of jsonlFiles) {
+      const hasSchema = await SchemaLoader.hasSchema(file);
+      if (hasSchema) {
+        filesWithSchema.push(file);
+      } else {
+        const tableName = basename(file, '.jsonl');
+        allWarnings.push(`Skipping validation for '${tableName}': schema file not found`);
+      }
+    }
+
+    // Validate schema for each file with schema
+    for (const file of filesWithSchema) {
       const result = await this.validateFile(file);
       allErrors.push(...result.errors);
+      allWarnings.push(...result.warnings);
     }
 
-    // Then, validate foreign keys across all tables
-    const fkErrors = await this.validateForeignKeys(dirPath, jsonlFiles);
-    allErrors.push(...fkErrors);
+    // Then, validate foreign keys across all tables (only for files with schema)
+    if (filesWithSchema.length > 0) {
+      const fkErrors = await this.validateForeignKeys(dirPath, filesWithSchema);
+      allErrors.push(...fkErrors);
+    }
 
     return {
       valid: allErrors.length === 0,
       errors: allErrors,
+      warnings: allWarnings,
     };
   }
 
@@ -202,6 +220,7 @@ export class Validator {
     return {
       valid: errors.length === 0,
       errors,
+      warnings: [],
     };
   }
 }