Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: taozhi8833998/node-sql-parser
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: OdatNurd/node-sql-parser
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 2 commits
  • 7 files changed
  • 1 contributor

Commits on Sep 26, 2025

  1. Fix SQLite's grammar 

    This is a small, targeted fix to the SQLite grammer to allow:
  - Bind parameters for the LIMIT and OFFSET fields
  - Numeric bind parameters (?1, etc)

 Both of these are supported by SQLite in general, but the grammar was
 not properly picking them up.

 This also chooses to make a ?1 and a ? behave the same (in that they
 are bind parameters but without making a new node type) which may or
 may not be what is useful to most, but that's what's useful to me.
    @OdatNurd
    OdatNurd authored and Terence Martin committed Sep 26, 2025
    Configuration menu
    Copy the full SHA
    cd30a76 View commit details
    Browse the repository at this point in the history

  2. Implement a translation build 

    Include a rollup config and command to directly create an ESM version of
the sqlite.js file directly from sources that would otherwise be used
by webpack to build the other version, since this thing uses an old and
decrepit version of webpack that does not seem to be capable of
generating an ESM file on its own.

This file also bundles within it the big-integer package, which is the
only dependency this has, which means that this file is a single file
dependency now (which is nice cause SQLite as a C library does the
same thing).

There is also added an additional test file that exercises this new
version using the existing SQLite tests, to ensure that the built
version is fully functional.
    @OdatNurd
    OdatNurd authored and Terence Martin committed Sep 26, 2025
    Configuration menu
    Copy the full SHA
    99b439b View commit details
    Browse the repository at this point in the history
Loading