Repository navigation
feat: add sqlite-node package - #29
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
02f4dbc to
c71ded7
Compare
9397360 to
0657932
Compare
c71ded7 to
86d713a
Compare
The SQLite graph store on Node's built-in node:sqlite through drizzle-orm/node-sqlite, for the command line and servers. openNodeGraphStore opens a file or an in-memory database, passes node:sqlite's options through, and puts a file database in write-ahead logging so a reader in another process is not blocked by a writer. The conformance kit runs against an in-memory database in process and a file database over a MessageChannel.
|
@codex security review |
🛡️ Codex Security ReviewSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
0657932 to
9b4f303
Compare
|
@codex security review |
🛡️ Codex Security ReviewSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
@codex security review |
🛡️ Codex Security ReviewSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Adds
@exadev/process-graph-sqlite-node, the Node driver for the SQLite graph store, stacked on #27.openNodeGraphStore(path, options)opens anode:sqlitedatabase (a file or":memory:") throughdrizzle-orm/node-sqliteand returns the store, the Drizzle database andclose. A file database is put in write-ahead logging so a reader in another process is not blocked by a writer;node:sqlite's own options (such astimeout) pass straight through rather than being defaulted here.The schema is migrated on the store's first operation by
graph-store-sqlite'smigrate, the same statements the browser driver applies. That package's tests already checkmigrateagainst drizzle's ownnode-sqlitemigrator on the committed folder.The conformance kit runs against an in-memory database in process and against a file database over a
MessageChannel, and a further test checks write-ahead logging and a reopen.npm name to reserve before the first release:
npm-reserve @exadev/process-graph-sqlite-node --trust ExaDev/process-graph --workflow ci.yml.