Flagged during v0.5.22 release review (Tech Lead persona).
The data_diff feature ships 1,164 lines in:
packages/opencode/src/altimate/native/connections/data-diff.tspackages/opencode/src/altimate/tools/data-diff.ts
Coverage gap — no dedicated test file exercises the pure functions:
resolveTableSources, injectCteisAuditColumn, isAutoTimestampDefaultbuildColumnDiscoverySQL, parseColumnInfoquoteIdentForDialect, dateTruncExprpartitionMode, buildPartitionWhereClausemergeOutcomes, extractStats
Create packages/opencode/test/altimate/data-diff.test.ts with table-driven tests per dialect (postgres, snowflake, bigquery, clickhouse, mysql, oracle, tsql). Estimated ~2 hours.
Deferred because: v0.5.22 ships adversarial tests covering the main attack surface via tool-facing paths. Dedicated unit coverage is a maintainability improvement, not a shipping blocker.
Flagged during v0.5.22 release review (Tech Lead persona).
The data_diff feature ships 1,164 lines in:
packages/opencode/src/altimate/native/connections/data-diff.tspackages/opencode/src/altimate/tools/data-diff.tsCoverage gap — no dedicated test file exercises the pure functions:
resolveTableSources,injectCteisAuditColumn,isAutoTimestampDefaultbuildColumnDiscoverySQL,parseColumnInfoquoteIdentForDialect,dateTruncExprpartitionMode,buildPartitionWhereClausemergeOutcomes,extractStatsCreate
packages/opencode/test/altimate/data-diff.test.tswith table-driven tests per dialect (postgres, snowflake, bigquery, clickhouse, mysql, oracle, tsql). Estimated ~2 hours.Deferred because: v0.5.22 ships adversarial tests covering the main attack surface via tool-facing paths. Dedicated unit coverage is a maintainability improvement, not a shipping blocker.