Conversation
added 10 commits
May 27, 2026 01:25
Add complete SQLite driver following the existing MySQL/MSSQL pattern: - SQLiteConnection: native sqlite3 extension, :memory: support, WAL mode, foreign key enforcement, prepared statement execution - SQLiteQuery: query builder with LIMIT/OFFSET, INSERT, UPDATE, DELETE, ALTER TABLE ADD COLUMN, RENAME COLUMN - SQLiteColumn: type affinity mapping (INTEGER, REAL, TEXT, BLOB), double-quote identifier quoting, AUTOINCREMENT support - SQLiteTable: DDL generation with CREATE TABLE IF NOT EXISTS, PRIMARY KEY, FOREIGN KEY, UNIQUE constraints - SQLiteInsertBuilder: positional parameter binding Integration points updated: - ConnectionInfo: accept 'sqlite' type - ConnectionPool: dispatch SQLiteConnection - Database: dispatch SQLiteQuery and SQLiteTable - TableFactory/ColumnFactory: create SQLite instances - DataType: register SQLite affinity types Refs: #141
…ping - Add setAutoUpdate/isAutoUpdate no-ops to SQLiteColumn for ColumnFactory compatibility - Handle auto-inc option for SQLite in ColumnFactory::primaryCheck - Map MSSQL 'identity' option to SQLite autoincrement in ColumnFactory::identityCheck - Update ConnectionInfoExtendedTest to expect 3 supported databases
Add bidirectional type mappings between SQLite and MySQL/MSSQL: - mysql→sqlite: int→integer, varchar→text, decimal→real, blob→blob, etc. - mssql→sqlite: nvarchar→text, money→real, bigint→integer, etc. - sqlite→mysql: integer→int, real→decimal, text→text, blob→blob - sqlite→mssql: integer→int, real→decimal, text→nvarchar, blob→varbinary Enables cross-engine table migration via TableFactory::map(). Refs: #141
- SQLiteColumnTest: 100% coverage - type mapping, quoting, asString, cleanValue - SQLiteTableTest: 100% coverage - DDL generation, PK, FK, unique, defaults - SQLiteQueryTest: 98% coverage - SELECT, INSERT, UPDATE, DELETE, LIMIT/OFFSET, ADD COLUMN, RENAME COLUMN, bindings, unsupported operations - SQLiteConnectionTest: 88% coverage - CRUD integration, transactions, rollback, connection lifecycle, constraint violations, error handling - SQLiteInsertBuilder: 100% coverage Overall SQLite coverage: 96% lines across all 5 classes. Remaining uncovered: connection error paths requiring DB corruption. Refs: #141
…5 classes - Remove unreachable catch block in connect() (SQLite3 throws directly) - Remove unreachable false-checks in runDirect/runPrepared (exceptions enabled) - Remove unreachable null-check in renameCol (getOldName never returns null) - Remove unused mapType method from SQLiteTable Coverage: SQLiteColumn 100%, SQLiteQuery 100%, SQLiteTable 100%, SQLiteInsertBuilder 100%, SQLiteConnection 97% (isAlive catch + __destruct)
- Add examples/17-sqlite/ with full CRUD, pagination, transactions, file-based DB - Update README.md: add SQLite to supported databases and examples list - Fix binding accumulation bug: reset bindings after query execution - Fix UPDATE SET clause: disable table prefix on column names in SET - Fix renameCol test to use Database instance with schema Refs: #141
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #156 +/- ##
============================================
+ Coverage 87.89% 88.53% +0.63%
- Complexity 2185 2325 +140
============================================
Files 54 59 +5
Lines 5427 5763 +336
============================================
+ Hits 4770 5102 +332
- Misses 657 661 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…ustion - Add tearDown with ConnectionPool::reset() to release connections between tests - Remove testGetNameEmpty that triggered PHP warnings on empty column names
…tQuery Extract shared binding logic (addBinding, getBindings, resetBinding, setBindings) into AbstractQuery as default implementations. MySQL overrides with its own typed-binding structure. Reduces code duplication between SQLite and MSSQL query builders.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add complete SQLite database engine support using the native PHP
sqlite3extension, following the existing MySQL/MSSQL driver pattern.Motivation
SQLite is widely used for local development, testing, embedded applications, and lightweight deployments. Adding SQLite support enables running the full test suite without external database servers (using
:memory:databases), improving CI speed and contributor onboarding.Closes #141
Changes
SQLiteConnection— native sqlite3 extension,:memory:and file-based support, WAL journal mode, FK enforcement, prepared statements with typed bindingSQLiteQuery— query builder with LIMIT/OFFSET, INSERT, UPDATE, DELETE, ALTER TABLE ADD/RENAME COLUMN, unsupported operations throw clear exceptionsSQLiteColumn— type affinity mapping (INTEGER, REAL, TEXT, BLOB), double-quote identifier quoting, AUTOINCREMENT supportSQLiteTable— DDL generation with CREATE TABLE IF NOT EXISTS, composite PRIMARY KEY, FOREIGN KEY, UNIQUE constraintsSQLiteInsertBuilder— positional parameter bindingConnectionInfo— acceptssqlitetype (host/port/user/password ignored)ConnectionPool— dispatchesSQLiteConnectionDatabase— dispatchesSQLiteQueryandSQLiteTableTableFactory/ColumnFactory— create SQLite instances, mapidentityto autoincrementDataType— registers SQLite affinity typesTypesMap— bidirectional type mappings (SQLite ↔ MySQL, SQLite ↔ MSSQL)README.md— updated supported databases and examples listexamples/17-sqlite/— complete working example with CRUD, pagination, transactions, file-based DBUI Changes
N/A
How to Test / Verify
85 SQLite tests pass. Coverage: SQLiteColumn 100%, SQLiteQuery 100%, SQLiteTable 100%, SQLiteInsertBuilder 100%, SQLiteConnection 97%.
Breaking Changes and Migration Steps
None. This is a purely additive feature. No existing APIs were modified.
Checklist
composer fix-cs)Related issues
Closes #141