Merge pull request #66 from iamvirul/feature/mssql-support

feat(mssql): Add full MSSQL/SQL Server support

Author: iamvirul

.github/workflows/build.yml

@@ -17,7 +17,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: '1.23'
+          go-version: '1.25'
           cache-dependency-path: go.sum
 
       - name: Download dependencies

.github/workflows/ci.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        go-version: ['1.23', '1.24']
+        go-version: ['1.24', '1.25']
 
     steps:
       - name: Checkout code
@@ -41,15 +41,15 @@ jobs:
         continue-on-error: false
 
       - name: Merge coverage files
-        if: matrix.go-version == '1.24'
+        if: matrix.go-version == '1.25'
         run: |
           # Merge coverage files using go tool cover
           echo "mode: atomic" > coverage.txt
           grep -h -v "^mode:" coverage-unit.txt coverage-integration-basic.txt >> coverage.txt || true
         continue-on-error: false
 
       - name: Verify coverage file exists
-        if: matrix.go-version == '1.24'
+        if: matrix.go-version == '1.25'
         run: |
           if [ ! -f coverage.txt ]; then
             echo "Error: coverage.txt not found"
@@ -59,7 +59,7 @@ jobs:
           head -5 coverage.txt
 
       - name: Upload coverage artifact for merging
-        if: matrix.go-version == '1.24'
+        if: matrix.go-version == '1.25'
         uses: actions/upload-artifact@v4
         with:
           name: coverage-unit-${{ github.run_id }}
@@ -78,7 +78,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: '1.24'
+          go-version: '1.25'
           cache-dependency-path: go.sum
 
       - name: Verify dependencies
@@ -89,7 +89,7 @@ jobs:
 
       - name: Run testcontainers integration tests
         run: |
-          go test -v -timeout=20m -coverprofile=coverage-integration.txt -covermode=atomic -coverpkg=./internal/...,./pkg/... ./tests -run 'TestIntegration_MySQL_FullWorkflow|TestIntegration_PostgreSQL_FullWorkflow|TestIntegration_AllReportsGenerated'
+          go test -v -timeout=30m -coverprofile=coverage-integration.txt -covermode=atomic -coverpkg=./internal/...,./pkg/... ./tests -run 'TestIntegration_MySQL_FullWorkflow|TestIntegration_PostgreSQL_FullWorkflow|TestIntegration_MSSQL_FullWorkflow|TestIntegration_AllReportsGenerated'
         continue-on-error: false
 
       - name: Verify integration coverage file exists
@@ -130,7 +130,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: '1.24'
+          go-version: '1.25'
           cache-dependency-path: go.sum
 
       - name: Download unit test coverage artifact
@@ -173,13 +173,14 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: '1.24'
+          go-version: '1.25'
           cache-dependency-path: go.sum
 
       - name: Run linter
-        uses: golangci/golangci-lint-action@v4
+        uses: golangci/golangci-lint-action@v6
         with:
-          version: latest
+          version: v1.64.8
+          install-mode: goinstall
           args: --timeout=5m --out-format=colored-line-number
 
   build:
@@ -194,7 +195,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: '1.24'
+          go-version: '1.25'
           cache-dependency-path: go.sum
 
       - name: Download dependencies

.github/workflows/release.yml

@@ -39,7 +39,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: '1.23'
+          go-version: '1.25'
           cache-dependency-path: go.sum
 
       - name: Download dependencies

.golangci.yml

@@ -0,0 +1,49 @@
+run:
+  # golangci-lint is compiled from source via install-mode: goinstall in CI
+  # using Go 1.25, so it can type-check modules that declare go 1.25.x.
+  timeout: 5m
+
+linters:
+  enable-all: false
+  # Use golangci-lint's default linter set; explicitly disable linters that
+  # produce excessive noise or require Go 1.25-specific analysis.
+  disable:
+    - depguard
+    - exhaustruct
+    - gochecknoglobals
+    - nlreturn
+    - wsl
+    - paralleltest
+    - testpackage
+    - godox
+
+linters-settings:
+  errcheck:
+    # Type assertions in this codebase use the &holder SQL scan pattern where
+    # the assertion is guaranteed to succeed. Enabling this produces false
+    # positives on every sql.Rows.Scan destination dereference.
+    check-type-assertions: false
+    check-blank: false
+  govet:
+    enable-all: false
+    # shadow: disabled — flags every `if err := ...; err != nil` block which is
+    # idiomatic Go. Produces 40+ false positives across the codebase with zero
+    # real bugs caught.
+  gofmt:
+    simplify: true
+
+issues:
+  exclude-rules:
+    # Test files get relaxed rules
+    - path: "_test\\.go"
+      linters:
+        - errcheck
+        - gosec
+        - gocritic
+    # Seed/tooling binaries get relaxed rules
+    - path: "cmd/"
+      linters:
+        - forbidigo
+  # Don't fail on warnings from generated or vendor code
+  exclude-dirs:
+    - vendor

CHANGELOG.md

@@ -7,6 +7,33 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.8] - 2026-03-14
+
+### Added
+- **Microsoft SQL Server Support** (issue #13)
+  - New driver: `driver: "mssql"` in `deepdiffdb.config.yaml`; maps to `github.com/microsoft/go-mssqldb` (`sqlserver://` DSN)
+  - Schema introspection via `INFORMATION_SCHEMA.COLUMNS`, `sys.indexes`, `sys.index_columns`, `INFORMATION_SCHEMA.REFERENTIAL_CONSTRAINTS`, and `sys.key_constraints`
+  - Square-bracket identifier quoting: `[table]`, `[column]`, with `]]` escaping for literal `]`
+  - `OFFSET 0 ROWS FETCH NEXT n ROWS ONLY` pagination (MSSQL has no `LIMIT` clause)
+  - MSSQL-compatible `ALTER TABLE` generation: `ADD` (no `COLUMN` keyword), `ALTER COLUMN`, `DROP CONSTRAINT`, `DROP INDEX … ON …`
+  - FK control during pack application via `sp_msforeachtable`: `NOCHECK CONSTRAINT ALL` / `WITH CHECK CHECK CONSTRAINT ALL`
+  - Full type reconstruction: `varchar(max)`, `decimal(10,2)`, `nvarchar(255)` with `nchar`/`nvarchar` half-length correction (MSSQL stores byte-length)
+  - `CheckPrimaryKeys` extended with MSSQL case querying `sys.tables` / `sys.key_constraints`
+  - Port is optional for MSSQL (defaults to SQL Server standard 1433)
+  - Integration test `TestIntegration_MSSQL_FullWorkflow` using testcontainers + `mcr.microsoft.com/mssql/server:2022-latest`
+- **Sample 15: MSSQL Support** (`samples/15-mssql-support/`)
+  - Docker Compose with two SQL Server 2022 Express containers (prod on 1433, dev on 1434)
+  - Init SQL scripts with deliberate schema drift: new columns, nullability changes, new indexes, FK relationships
+  - Seed data with deliberate data drift: price updates, status changes, new rows
+  - Makefile targets: `up`, `seed`, `diff`, `diff-schema`, `diff-content`, `gen-pack`, `down`, `clean`
+
+### Changed
+- `pkg/config/config.go`: `mssql` added to valid driver list; port validation skipped for MSSQL (optional)
+- `internal/drivers/drivers.go`: MSSQL DSN builder added (`sqlserver://user:pass@host:port?database=db`)
+- `internal/content/hash.go` / `cursor.go`: MSSQL identifier quoting and `OFFSET/FETCH` pagination
+- `internal/content/pack.go`: MSSQL FK disable/enable, column type introspection, `ALTER TABLE ADD` syntax
+- `deepdiffdb.config.yaml.example`: Commented MSSQL connection example added
+
 ## [0.7] - 2026-03-14
 
 ### Added

README.md

@@ -24,7 +24,7 @@ DeepDiff DB makes the entire process deterministic, reviewable, and safe by:
 
 - **Fast Go-based diff engine** - Optimized for performance with efficient memory usage
 - **Single static binary** - Zero dependencies after download, works on any compatible system
-- **Multi-database support** - MySQL, PostgreSQL, and SQLite
+- **Multi-database support** - MySQL, PostgreSQL, SQLite, and Microsoft SQL Server
 - **Schema drift detection** - Identifies structural differences between databases
 - **Row-level comparison** - SHA-256 hashing for accurate change detection
 - **Conflict detection** - Identifies rows that exist in both databases but differ
@@ -172,7 +172,7 @@ Create a `deepdiffdb.config.yaml` file:
 
 ```yaml
 prod:
-  driver: "mysql"          # mysql, postgres, postgresql, or sqlite
+  driver: "mysql"          # mysql, postgres, postgresql, sqlite, or mssql
   host: "localhost"
   port: 3306
   user: "root"
@@ -218,9 +218,9 @@ conflict_resolution:
 ### Configuration Options
 
 **Database Configuration:**
-- `driver`: Database driver (`mysql`, `postgres`, `postgresql`, or `sqlite`)
+- `driver`: Database driver (`mysql`, `postgres`, `postgresql`, `sqlite`, or `mssql`)
 - `host`: Database hostname or IP address
-- `port`: Database port number (not required for SQLite)
+- `port`: Database port number (not required for SQLite; defaults to 1433 for MSSQL)
 - `user`: Database username
 - `password`: Database password
 - `database`: Database name
@@ -306,7 +306,7 @@ deepdiffdb schema-migrate --config deepdiffdb.config.yaml
 - `schema_migration.sql` - Transaction-wrapped SQL migration script
 
 **Features:**
-- Driver-specific SQL syntax (MySQL, PostgreSQL, SQLite)
+- Driver-specific SQL syntax (MySQL, PostgreSQL, SQLite, MSSQL)
 - Safe defaults (destructive operations commented out by default)
 - Proper dependency ordering for foreign keys and constraints
 - Transaction-wrapped for atomic execution
@@ -613,7 +613,7 @@ deepdiffdb gen-pack --config deepdiffdb.config.yaml --resume
 
 Current limitations and known constraints:
 
-- **Database Support** - MSSQL and Oracle are not yet supported (planned for future releases)
+- **Database Support** - Oracle is not yet supported (planned for future releases)
 - **Schema Auto-merge** - Schema differences must be resolved manually
 - **Primary Key Requirement** - All tables must have primary keys (unless explicitly ignored)
 - **Large Database Performance** - Very large tables are handled with keyset-paginated batching (v0.7+); diff output files may still be large for tables with many changed rows
@@ -640,7 +640,7 @@ go test ./tests/drivers
 go test ./tests -run TestIntegration -v
 ```
 
-Integration tests use testcontainers and automatically spin up MySQL and PostgreSQL containers for full workflow validation.
+Integration tests use testcontainers and automatically spin up MySQL, PostgreSQL, and MSSQL containers for full workflow validation.
 
 ## Contributing
 

ROADMAP.md

@@ -8,15 +8,15 @@ We release a new version every **Saturday**. Each release includes one or more f
 
 ---
 
-## Current Status: v0.7
+## Current Status: v0.8
 
 **Last Release:** 2026-03-14
 
 **Current Features:**
 - Schema drift detection and standalone schema migration (`schema-migrate`)
 - Row-level data comparison with SHA-256 hashing
 - Migration pack generation and transactional apply mode
-- MySQL, PostgreSQL, and SQLite support
+- MySQL, PostgreSQL, SQLite, and **Microsoft SQL Server** support
 - Conflict detection with `ours`/`theirs`/`manual` resolution strategies
 - Interactive `resolve-conflicts` command with `--auto` and `--resume` flags
 - Per-table conflict resolution strategies with `resolutions.json` persistence
@@ -119,17 +119,18 @@ We release a new version every **Saturday**. Each release includes one or more f
 
 ---
 
-### v0.8: MSSQL Support
-**Target Date:** Next Saturday
+### ~~v0.8: MSSQL Support~~ ✅ Released 2026-03-14
 
-**Features:**
-- Microsoft SQL Server driver support
-- MSSQL-specific schema introspection
-- MSSQL-specific SQL generation
-- Transaction handling for MSSQL
-- Testing with MSSQL 2019+ versions
+**Features delivered:**
+- Microsoft SQL Server driver (`github.com/microsoft/go-mssqldb`)
+- Schema introspection via `INFORMATION_SCHEMA` + `sys.*` catalog views
+- MSSQL-compatible SQL generation (square-bracket quoting, `ALTER COLUMN`, `DROP INDEX … ON …`)
+- FK control via `sp_msforeachtable` in pack application
+- `OFFSET/FETCH` pagination (no `LIMIT`)
+- Integration tests with SQL Server 2022 (testcontainers)
+- Sample 15: MSSQL Support
 
-**Impact:** Expands database support to enterprise SQL Server users
+**Impact:** Enterprise SQL Server users can now use DeepDiff DB in production
 
 ---
 
@@ -213,11 +214,9 @@ We release a new version every **Saturday**. Each release includes one or more f
 - ~~HTML Report Viewer~~ (v0.5)
 - ~~Enhanced Error Handling & Logging~~ (v0.6)
 - ~~Streaming Support for Large Datasets~~ (v0.7)
+- ~~MSSQL Support~~ (v0.8)
 - Documentation & Production Readiness
 
-### Medium Priority (Should Have)
-- MSSQL Support
-
 ### Low Priority (Nice to Have)
 - Oracle Support (can be post-v1.0)
 - Advanced features (post-v1.0)

deepdiffdb.config.yaml.example

@@ -1,5 +1,5 @@
 prod:
-  driver: "postgres"
+  driver: "postgres"       # Supported: mysql | postgres | postgresql | sqlite | mssql
   host: "127.0.0.1"
   port: 5432
   user: "postgres"
@@ -14,6 +14,27 @@ dev:
   password: "password"
   database: "dev_db"
 
+# ─────────────────────────────────────────────────────────
+# Microsoft SQL Server example (driver: mssql)
+# Requires SQL Server 2017+ (or Azure SQL Database)
+# Port is optional — defaults to 1433 when omitted.
+# ─────────────────────────────────────────────────────────
+# prod:
+#   driver: "mssql"
+#   host: "127.0.0.1"
+#   port: 1433           # optional; omit to use default 1433
+#   user: "sa"
+#   password: "StrongP@ss1word!"
+#   database: "prod_db"
+#
+# dev:
+#   driver: "mssql"
+#   host: "127.0.0.1"
+#   port: 1434
+#   user: "sa"
+#   password: "StrongP@ss1word!"
+#   database: "dev_db"
+
 ignore:
   tables:
     - "logs"