detailed methods removed

Author: pyramation

CHANGELOG.md

@@ -1,6 +1,5 @@
 # 17.2.1
 * Add normalize() and normalizeSync() functions for SQL normalization
-* Add parseQueryDetailed() and parseQueryDetailedSync() with enhanced error reporting
 * Add fingerprint() and fingerprintSync() functions for query fingerprinting
 * Add isReady() function to check WASM module initialization status
 * Improve memory management and error handling in WASM wrapper

README.md

@@ -157,32 +157,6 @@ const normalized = normalizeSync('SELECT * FROM users WHERE active = true');
 // Returns: string - normalized SQL query
 ```
 
-### `parseQueryDetailed(sql: string): Promise<DetailedParseResult>`
-
-Parses a SQL query with enhanced error reporting that includes detailed location information for parse errors. Returns a Promise for either the parse tree or detailed error information.
-
-```typescript
-import { parseQueryDetailed } from 'libpg-query';
-
-try {
-  const result = await parseQueryDetailed('SELECT * FROM users WHERE');
-} catch (error) {
-  // Enhanced error with line number, position, and context information
-  console.log(error.message); // "Parse error: syntax error at end of input at line 1, position 26"
-}
-```
-
-### `parseQueryDetailedSync(sql: string): DetailedParseResult`
-
-Synchronous version of detailed parsing with enhanced error reporting.
-
-```typescript
-import { parseQueryDetailedSync } from 'libpg-query';
-
-const result = parseQueryDetailedSync('SELECT * FROM users WHERE active = true');
-// Returns: DetailedParseResult with enhanced error information if parsing fails
-```
-
 ### Initialization
 
 The library provides both async and sync methods. Async methods handle initialization automatically, while sync methods require explicit initialization.

test/parsing.test.js

@@ -80,23 +80,4 @@ describe("Query Parsing", () => {
       );
     });
   });
-
-  describe("Detailed Error Parsing", () => {
-    it("should provide detailed error information for invalid queries", async () => {
-      try {
-        await query.parseQueryDetailed("SELECT FROM");
-        throw new Error("should have thrown");
-      } catch (e) {
-        expect(e.message).to.include("Parse error:");
-        expect(e.message).to.include("line");
-        expect(e.message).to.include("position");
-      }
-    });
-
-    it("should parse valid queries with detailed parsing", async () => {
-      const result = await query.parseQueryDetailed("SELECT 1");
-      expect(result.stmts).to.have.lengthOf(1);
-      expect(result.stmts[0].stmt.SelectStmt).to.exist;
-    });
-  });
 });

wasm/index.cjs

@@ -154,45 +154,6 @@ const normalize = awaitInit(async (query) => {
   }
 });
 
-const parseQueryDetailed = awaitInit(async (query) => {
-  const queryPtr = stringToPtr(query);
-  let resultPtr;
-  
-  try {
-    resultPtr = wasmModule._wasm_parse_query_detailed(queryPtr);
-    
-    const hasError = wasmModule.HEAPU32[resultPtr >> 2];
-    
-    if (hasError) {
-      const messagePtr = wasmModule.HEAPU32[(resultPtr + 4) >> 2];
-      const funcnamePtr = wasmModule.HEAPU32[(resultPtr + 8) >> 2];
-      const filenamePtr = wasmModule.HEAPU32[(resultPtr + 12) >> 2];
-      const lineno = wasmModule.HEAPU32[(resultPtr + 16) >> 2];
-      const cursorpos = wasmModule.HEAPU32[(resultPtr + 20) >> 2];
-      const contextPtr = wasmModule.HEAPU32[(resultPtr + 24) >> 2];
-      
-      const error = {
-        message: messagePtr ? ptrToString(messagePtr) : '',
-        funcname: funcnamePtr ? ptrToString(funcnamePtr) : null,
-        filename: filenamePtr ? ptrToString(filenamePtr) : null,
-        lineno: lineno,
-        cursorpos: cursorpos,
-        context: contextPtr ? ptrToString(contextPtr) : null
-      };
-      
-      wasmModule._wasm_free_detailed_result(resultPtr);
-      throw new Error(error.message);
-    } else {
-      const dataPtr = wasmModule.HEAPU32[(resultPtr + 28) >> 2];
-      const result = JSON.parse(ptrToString(dataPtr));
-      wasmModule._wasm_free_detailed_result(resultPtr);
-      return result;
-    }
-  } finally {
-    wasmModule._free(queryPtr);
-  }
-});
-
 // Sync versions
 function parseQuerySync(query) {
   if (!wasmModule) {
@@ -326,60 +287,16 @@ function normalizeSync(query) {
   }
 }
 
-function parseQueryDetailedSync(query) {
-  if (!wasmModule) {
-    throw new Error('WASM module not initialized. Call loadModule() first.');
-  }
-  const queryPtr = stringToPtr(query);
-  let resultPtr;
-  
-  try {
-    resultPtr = wasmModule._wasm_parse_query_detailed(queryPtr);
-    
-    const hasError = wasmModule.HEAPU32[resultPtr >> 2];
-    
-    if (hasError) {
-      const messagePtr = wasmModule.HEAPU32[(resultPtr + 4) >> 2];
-      const funcnamePtr = wasmModule.HEAPU32[(resultPtr + 8) >> 2];
-      const filenamePtr = wasmModule.HEAPU32[(resultPtr + 12) >> 2];
-      const lineno = wasmModule.HEAPU32[(resultPtr + 16) >> 2];
-      const cursorpos = wasmModule.HEAPU32[(resultPtr + 20) >> 2];
-      const contextPtr = wasmModule.HEAPU32[(resultPtr + 24) >> 2];
-      
-      const error = {
-        message: messagePtr ? ptrToString(messagePtr) : '',
-        funcname: funcnamePtr ? ptrToString(funcnamePtr) : null,
-        filename: filenamePtr ? ptrToString(filenamePtr) : null,
-        lineno: lineno,
-        cursorpos: cursorpos,
-        context: contextPtr ? ptrToString(contextPtr) : null
-      };
-      
-      wasmModule._wasm_free_detailed_result(resultPtr);
-      throw new Error(error.message);
-    } else {
-      const dataPtr = wasmModule.HEAPU32[(resultPtr + 28) >> 2];
-      const result = JSON.parse(ptrToString(dataPtr));
-      wasmModule._wasm_free_detailed_result(resultPtr);
-      return result;
-    }
-  } finally {
-    wasmModule._free(queryPtr);
-  }
-}
-
 module.exports = {
   loadModule,
   parseQuery,
   deparse,
   parsePlPgSQL,
   fingerprint,
   normalize,
-  parseQueryDetailed,
   parseQuerySync,
   deparseSync,
   parsePlPgSQLSync,
   fingerprintSync,
-  normalizeSync,
-  parseQueryDetailedSync
+  normalizeSync
 };

wasm/index.d.ts

@@ -12,7 +12,5 @@ export function fingerprint(sql: string): Promise<string>;
 export function fingerprintSync(sql: string): string;
 export function normalize(sql: string): Promise<string>;
 export function normalizeSync(sql: string): string;
-export function parseQueryDetailed(sql: string): Promise<ParseResult>;
-export function parseQueryDetailedSync(sql: string): ParseResult;
 
 export * from '@pgsql/types'; 

wasm/index.js

@@ -154,45 +154,6 @@ export const normalize = awaitInit(async (query) => {
   }
 });
 
-export const parseQueryDetailed = awaitInit(async (query) => {
-  const queryPtr = stringToPtr(query);
-  let resultPtr;
-  
-  try {
-    resultPtr = wasmModule._wasm_parse_query_detailed(queryPtr);
-    
-    const hasError = wasmModule.HEAPU32[resultPtr >> 2];
-    
-    if (hasError) {
-      const messagePtr = wasmModule.HEAPU32[(resultPtr + 4) >> 2];
-      const funcnamePtr = wasmModule.HEAPU32[(resultPtr + 8) >> 2];
-      const filenamePtr = wasmModule.HEAPU32[(resultPtr + 12) >> 2];
-      const lineno = wasmModule.HEAPU32[(resultPtr + 16) >> 2];
-      const cursorpos = wasmModule.HEAPU32[(resultPtr + 20) >> 2];
-      const contextPtr = wasmModule.HEAPU32[(resultPtr + 24) >> 2];
-      
-      const error = {
-        message: messagePtr ? ptrToString(messagePtr) : '',
-        funcname: funcnamePtr ? ptrToString(funcnamePtr) : null,
-        filename: filenamePtr ? ptrToString(filenamePtr) : null,
-        lineno: lineno,
-        cursorpos: cursorpos,
-        context: contextPtr ? ptrToString(contextPtr) : null
-      };
-      
-      wasmModule._wasm_free_detailed_result(resultPtr);
-      throw new Error(error.message);
-    } else {
-      const dataPtr = wasmModule.HEAPU32[(resultPtr + 28) >> 2];
-      const result = JSON.parse(ptrToString(dataPtr));
-      wasmModule._wasm_free_detailed_result(resultPtr);
-      return result;
-    }
-  } finally {
-    wasmModule._free(queryPtr);
-  }
-});
-
 // Sync versions
 export function parseQuerySync(query) {
   if (!wasmModule) {
@@ -325,45 +286,3 @@ export function normalizeSync(query) {
     }
   }
 }
-
-export function parseQueryDetailedSync(query) {
-  if (!wasmModule) {
-    throw new Error('WASM module not initialized. Call loadModule() first.');
-  }
-  const queryPtr = stringToPtr(query);
-  let resultPtr;
-  
-  try {
-    resultPtr = wasmModule._wasm_parse_query_detailed(queryPtr);
-    
-    const hasError = wasmModule.HEAPU32[resultPtr >> 2];
-    
-    if (hasError) {
-      const messagePtr = wasmModule.HEAPU32[(resultPtr + 4) >> 2];
-      const funcnamePtr = wasmModule.HEAPU32[(resultPtr + 8) >> 2];
-      const filenamePtr = wasmModule.HEAPU32[(resultPtr + 12) >> 2];
-      const lineno = wasmModule.HEAPU32[(resultPtr + 16) >> 2];
-      const cursorpos = wasmModule.HEAPU32[(resultPtr + 20) >> 2];
-      const contextPtr = wasmModule.HEAPU32[(resultPtr + 24) >> 2];
-      
-      const error = {
-        message: messagePtr ? ptrToString(messagePtr) : '',
-        funcname: funcnamePtr ? ptrToString(funcnamePtr) : null,
-        filename: filenamePtr ? ptrToString(filenamePtr) : null,
-        lineno: lineno,
-        cursorpos: cursorpos,
-        context: contextPtr ? ptrToString(contextPtr) : null
-      };
-      
-      wasmModule._wasm_free_detailed_result(resultPtr);
-      throw new Error(error.message);
-    } else {
-      const dataPtr = wasmModule.HEAPU32[(resultPtr + 28) >> 2];
-      const result = JSON.parse(ptrToString(dataPtr));
-      wasmModule._wasm_free_detailed_result(resultPtr);
-      return result;
-    }
-  } finally {
-    wasmModule._free(queryPtr);
-  }
-}