@kaelio/ktx 0.5.0 → 0.6.0

package/dist/context/scan/relationship-profiling.js

@@ -3,65 +3,6 @@ export function createKtxRelationshipProfileCache() {
     return { tableProfiles: new Map() };
 }
 const SAMPLE_VALUE_DELIMITER = '\u001f';
-function quoteStyle(driver) {
-    if (driver === 'mysql' || driver === 'clickhouse') {
-        return 'backtick';
-    }
-    if (driver === 'sqlserver') {
-        return 'bracket';
-    }
-    return 'double';
-}
-export function quoteKtxRelationshipIdentifier(driver, identifier) {
-    switch (quoteStyle(driver)) {
-        case 'backtick':
-            return `\`${identifier.replace(/`/g, '``')}\``;
-        case 'bracket':
-            return `[${identifier.replace(/\]/g, ']]')}]`;
-        case 'double':
-            return `"${identifier.replace(/"/g, '""')}"`;
-    }
-}
-export function formatKtxRelationshipTableRef(driver, table) {
-    const parts = driver === 'sqlite'
-        ? [table.name]
-        : [table.catalog, table.db, table.name].filter((value) => Boolean(value));
-    return parts.map((part) => quoteKtxRelationshipIdentifier(driver, part)).join('.');
-}
-function textLengthExpression(driver, columnSql) {
-    if (driver === 'mysql') {
-        return `CHAR_LENGTH(CAST(${columnSql} AS CHAR))`;
-    }
-    if (driver === 'sqlserver') {
-        return `LEN(CAST(${columnSql} AS NVARCHAR(MAX)))`;
-    }
-    if (driver === 'bigquery') {
-        return `LENGTH(CAST(${columnSql} AS STRING))`;
-    }
-    if (driver === 'clickhouse') {
-        return `length(toString(${columnSql}))`;
-    }
-    return `LENGTH(CAST(${columnSql} AS TEXT))`;
-}
-function limitSql(driver, limit) {
-    if (driver === 'sqlserver') {
-        return '';
-    }
-    return ` LIMIT ${Math.max(1, Math.floor(limit))}`;
-}
-function topSql(driver, limit) {
-    if (driver === 'sqlserver') {
-        return ` TOP (${Math.max(1, Math.floor(limit))})`;
-    }
-    return '';
-}
-function sampledTableSql(driver, tableSql, limit) {
-    const safeLimit = Math.max(1, Math.floor(limit));
-    if (driver === 'sqlserver') {
-        return `(SELECT TOP (${safeLimit}) * FROM ${tableSql}) AS relationship_profile_sample`;
-    }
-    return `(SELECT * FROM ${tableSql}${limitSql(driver, safeLimit)}) AS relationship_profile_sample`;
-}
 function firstRow(result) {
     return result.rows[0] ?? [];
 }
@@ -108,7 +49,7 @@ function tableProfileCacheKey(input) {
     return [
         input.ctx.runId,
         input.connectionId,
-        input.driver,
+        input.dialect.type,
         input.table.catalog ?? '',
         input.table.db ?? '',
         input.table.name,
@@ -119,42 +60,32 @@ function tableProfileCacheKey(input) {
 function sqlStringLiteral(value) {
     return `'${value.replace(/'/g, "''")}'`;
 }
-function sampleAggregateSql(driver, innerSql) {
-    if (driver === 'postgres') {
-        return `(SELECT STRING_AGG(CAST(value AS TEXT), CHR(31)) FROM (${innerSql}) AS relationship_profile_values)`;
-    }
-    if (driver === 'bigquery') {
-        return `(SELECT STRING_AGG(CAST(value AS STRING), '\\u001F') FROM (${innerSql}) AS relationship_profile_values)`;
-    }
-    if (driver === 'mysql') {
-        return `(SELECT GROUP_CONCAT(CAST(value AS CHAR) SEPARATOR CHAR(31)) FROM (${innerSql}) AS relationship_profile_values)`;
-    }
-    if (driver === 'sqlserver') {
-        return `(SELECT STRING_AGG(CAST(value AS NVARCHAR(MAX)), CHAR(31)) FROM (${innerSql}) AS relationship_profile_values)`;
-    }
-    if (driver === 'clickhouse') {
-        return `(SELECT arrayStringConcat(groupArray(toString(value)), '\\x1F') FROM (${innerSql}) AS relationship_profile_values)`;
-    }
-    if (driver === 'snowflake') {
-        return `(SELECT LISTAGG(CAST(value AS VARCHAR), '\\x1f') FROM (${innerSql}) AS relationship_profile_values)`;
+function sqlSuffix(fragment) {
+    return fragment ? ` ${fragment}` : '';
+}
+function sampledTableSql(dialect, tableSql, limit) {
+    const top = dialect.getTopClause(limit);
+    if (top) {
+        return `(SELECT ${top} * FROM ${tableSql}) AS relationship_profile_sample`;
     }
-    return `(SELECT GROUP_CONCAT(CAST(value AS TEXT), char(31)) FROM (${innerSql}) AS relationship_profile_values)`;
+    return `(SELECT * FROM ${tableSql}${sqlSuffix(dialect.getLimitOffsetClause(limit))}) AS relationship_profile_sample`;
 }
 function sampleValuesSql(input) {
+    const top = input.dialect.getTopClause(input.limit);
     return [
-        `SELECT${topSql(input.driver, input.limit)} ${input.columnSql} AS value`,
+        `SELECT${top ? ` ${top}` : ''} ${input.columnSql} AS value`,
         `FROM ${input.tableSql}`,
         `WHERE ${input.columnSql} IS NOT NULL`,
         `GROUP BY ${input.columnSql}`,
         `ORDER BY COUNT(*) DESC, ${input.columnSql} ASC`,
-        limitSql(input.driver, input.limit),
+        sqlSuffix(input.dialect.getLimitOffsetClause(input.limit)),
     ].join(' ');
 }
 function columnProfileSelectSql(input) {
-    const columnSql = quoteKtxRelationshipIdentifier(input.connectionDriver, input.column.name);
-    const textLengthSql = textLengthExpression(input.connectionDriver, columnSql);
-    const samplesSql = sampleAggregateSql(input.connectionDriver, sampleValuesSql({
-        driver: input.connectionDriver,
+    const columnSql = input.dialect.quoteIdentifier(input.column.name);
+    const textLengthSql = input.dialect.textLengthExpression(columnSql);
+    const samplesSql = input.dialect.getSampleValueAggregation(sampleValuesSql({
+        dialect: input.dialect,
         tableSql: input.profileTableSql,
         columnSql,
         limit: input.sampleValuesPerColumn,
@@ -183,7 +114,7 @@ function splitSampleValues(value) {
     return text.split(SAMPLE_VALUE_DELIMITER).filter((item) => item !== '');
 }
 async function queryCount(input) {
-    const tableSql = formatKtxRelationshipTableRef(input.driver, input.table);
+    const tableSql = input.dialect.formatTableName(input.table);
     const result = await input.executor.executeReadOnly({ connectionId: input.connectionId, sql: `SELECT COUNT(*) AS row_count FROM ${tableSql}`, maxRows: 1 }, input.ctx);
     return { rowCount: numberAt(result, 'row_count'), queryCount: 1 };
 }
@@ -191,7 +122,7 @@ async function queryTableProfile(input) {
     if (input.table.columns.length === 0) {
         const rowCount = await queryCount({
             connectionId: input.connectionId,
-            driver: input.driver,
+            dialect: input.dialect,
             table: input.table.ref,
             executor: input.executor,
             ctx: input.ctx,
@@ -202,11 +133,11 @@ async function queryTableProfile(input) {
             queryCount: rowCount.queryCount,
         };
     }
-    const tableSql = formatKtxRelationshipTableRef(input.driver, input.table.ref);
-    const profileTableSql = sampledTableSql(input.driver, tableSql, input.profileSampleRows);
+    const tableSql = input.dialect.formatTableName(input.table.ref);
+    const profileTableSql = sampledTableSql(input.dialect, tableSql, input.profileSampleRows);
     const sql = input.table.columns
         .map((column) => columnProfileSelectSql({
-        connectionDriver: input.driver,
+        dialect: input.dialect,
         tableSql,
         profileTableSql,
         column,
@@ -252,7 +183,7 @@ export async function profileKtxRelationshipSchema(input) {
     if (!input.executor) {
         return {
             connectionId: input.connectionId,
-            driver: input.driver,
+            driver: input.dialect.type,
             sqlAvailable: false,
             queryCount: 0,
             tables: [],
@@ -271,7 +202,7 @@ export async function profileKtxRelationshipSchema(input) {
         const profileSampleRows = input.profileSampleRows ?? 10000;
         const cacheKey = tableProfileCacheKey({
             connectionId: input.connectionId,
-            driver: input.driver,
+            dialect: input.dialect,
             ctx: input.ctx,
             table: table.ref,
             sampleValuesPerColumn,
@@ -284,7 +215,7 @@ export async function profileKtxRelationshipSchema(input) {
         try {
             const tableProfile = await queryTableProfile({
                 connectionId: input.connectionId,
-                driver: input.driver,
+                dialect: input.dialect,
                 table,
                 executor,
                 ctx: input.ctx,
@@ -324,7 +255,7 @@ export async function profileKtxRelationshipSchema(input) {
     }
     return {
         connectionId: input.connectionId,
-        driver: input.driver,
+        driver: input.dialect.type,
         sqlAvailable: true,
         queryCount: queryTotal,
         tables,

package/dist/context/scan/relationship-validation.d.ts

@@ -1,7 +1,8 @@
+import type { KtxDialect } from '../connections/dialects.js';
 import { type KtxRelationshipValidationBudget } from './relationship-budget.js';
 import type { KtxRelationshipDiscoveryCandidate } from './relationship-candidates.js';
 import { type KtxRelationshipProfileArtifact, type KtxRelationshipReadOnlyExecutor } from './relationship-profiling.js';
-import type { KtxConnectionDriver, KtxScanContext } from './types.js';
+import type { KtxScanContext } from './types.js';
 type KtxValidatedRelationshipStatus = 'accepted' | 'review' | 'rejected';
 interface KtxRelationshipValidationSettings {
     acceptThreshold: number;
@@ -33,7 +34,7 @@ export interface KtxValidatedRelationshipDiscoveryCandidate extends Omit<KtxRela
 }
 export interface ValidateKtxRelationshipDiscoveryCandidatesInput {
     connectionId: string;
-    driver: KtxConnectionDriver;
+    dialect: KtxDialect;
     candidates: readonly KtxRelationshipDiscoveryCandidate[];
     profiles: KtxRelationshipProfileArtifact;
     executor: KtxRelationshipReadOnlyExecutor | null;

package/dist/context/scan/relationship-validation.js

@@ -1,5 +1,4 @@
 import { applyKtxRelationshipValidationBudget } from './relationship-budget.js';
-import { formatKtxRelationshipTableRef, quoteKtxRelationshipIdentifier, } from './relationship-profiling.js';
 const DEFAULT_SETTINGS = {
     acceptThreshold: 0.85,
     reviewThreshold: 0.55,
@@ -41,28 +40,19 @@ function numberAt(result, header) {
     }
     return 0;
 }
-function limitSql(driver, limit) {
-    if (driver === 'sqlserver') {
-        return '';
-    }
-    return ` LIMIT ${Math.max(1, Math.floor(limit))}`;
-}
-function topSql(driver, limit) {
-    if (driver === 'sqlserver') {
-        return ` TOP (${Math.max(1, Math.floor(limit))})`;
-    }
-    return '';
+function sqlSuffix(fragment) {
+    return fragment ? ` ${fragment}` : '';
 }
 function buildCoverageSql(input) {
-    const childTable = formatKtxRelationshipTableRef(input.driver, { catalog: null, db: null, name: input.childTable });
-    const parentTable = formatKtxRelationshipTableRef(input.driver, { catalog: null, db: null, name: input.parentTable });
-    const childColumn = quoteKtxRelationshipIdentifier(input.driver, input.childColumn);
-    const parentColumn = quoteKtxRelationshipIdentifier(input.driver, input.parentColumn);
-    const limit = limitSql(input.driver, input.maxDistinctSourceValues);
-    const top = topSql(input.driver, input.maxDistinctSourceValues);
+    const childTable = input.dialect.formatTableName(input.childTable);
+    const parentTable = input.dialect.formatTableName(input.parentTable);
+    const childColumn = input.dialect.quoteIdentifier(input.childColumn);
+    const parentColumn = input.dialect.quoteIdentifier(input.parentColumn);
+    const limit = sqlSuffix(input.dialect.getLimitOffsetClause(input.maxDistinctSourceValues));
+    const top = input.dialect.getTopClause(input.maxDistinctSourceValues);
     return [
         'WITH child_values AS (',
-        `SELECT DISTINCT${top} ${childColumn} AS value FROM ${childTable} WHERE ${childColumn} IS NOT NULL${limit}`,
+        `SELECT DISTINCT${top ? ` ${top}` : ''} ${childColumn} AS value FROM ${childTable} WHERE ${childColumn} IS NOT NULL${limit}`,
         '), parent_values AS (',
         `SELECT DISTINCT ${parentColumn} AS value FROM ${parentTable} WHERE ${parentColumn} IS NOT NULL`,
         ')',
@@ -154,10 +144,10 @@ export async function validateKtxRelationshipDiscoveryCandidates(input) {
         const result = await executor.executeReadOnly({
             connectionId: input.connectionId,
             sql: buildCoverageSql({
-                driver: input.driver,
-                childTable: candidate.from.table.name,
+                dialect: input.dialect,
+                childTable: candidate.from.table,
                 childColumn: sourceColumn,
-                parentTable: candidate.to.table.name,
+                parentTable: candidate.to.table,
                 parentColumn: targetColumn,
                 maxDistinctSourceValues: settings.maxDistinctSourceValues,
             }),

package/dist/context/scan/table-ref.d.ts

@@ -17,8 +17,7 @@ export declare function tableRefFromKey(key: KtxTableRefKey): KtxTableRef;
 export declare function tableRefSet(refs: readonly KtxTableRef[]): ReadonlySet<KtxTableRefKey>;
 /**
  * Return the bare table names from a scope that fall within the given
- * (catalog, db) namespace. `catalog: null` is treated as a wildcard so that
- * legacy 2-part `"db.name"` entries continue to match. Same for `db: null`.
+ * (catalog, db) namespace.
  */
 export declare function scopedTableNames(scope: ReadonlySet<KtxTableRefKey>, namespace: {
     catalog?: string | null;

package/dist/context/scan/table-ref.js

@@ -17,8 +17,7 @@ export function tableRefSet(refs) {
 }
 /**
  * Return the bare table names from a scope that fall within the given
- * (catalog, db) namespace. `catalog: null` is treated as a wildcard so that
- * legacy 2-part `"db.name"` entries continue to match. Same for `db: null`.
+ * (catalog, db) namespace.
  */
 export function scopedTableNames(scope, namespace) {
     const names = new Set();
@@ -26,9 +25,9 @@ export function scopedTableNames(scope, namespace) {
     const wantDb = namespace.db ?? null;
     for (const key of scope) {
         const ref = tableRefFromKey(key);
-        if (wantCatalog !== null && ref.catalog !== null && ref.catalog !== wantCatalog)
+        if (ref.catalog !== wantCatalog)
             continue;
-        if (wantDb !== null && ref.db !== null && ref.db !== wantDb)
+        if (ref.db !== wantDb)
             continue;
         names.add(ref.name);
     }

package/dist/context/scan/types.d.ts

@@ -1,5 +1,5 @@
 import type { KtxTableRefKey } from './table-ref.js';
-export type KtxConnectionDriver = 'sqlite' | 'postgres' | 'postgresql' | 'sqlserver' | 'bigquery' | 'snowflake' | 'mysql' | 'clickhouse';
+export type KtxConnectionDriver = 'sqlite' | 'postgres' | 'sqlserver' | 'bigquery' | 'snowflake' | 'mysql' | 'clickhouse';
 export type KtxScanMode = 'structural' | 'relationships' | 'enriched';
 export type KtxScanTrigger = 'cli' | 'mcp' | 'schema_scan' | 'scheduled' | 'manual';
 export interface KtxConnectorCapabilities {
@@ -56,6 +56,7 @@ export interface KtxSchemaSnapshot {
     scope: KtxSchemaScope;
     tables: KtxSchemaTable[];
     metadata: Record<string, unknown>;
+    warnings?: KtxScanWarning[];
 }
 interface KtxCredentialEnvReference {
     kind: 'env';
@@ -227,6 +228,7 @@ export interface KtxQueryResult {
     rowCount: number | null;
 }
 export interface KtxTableListEntry {
+    catalog: string | null;
     schema: string;
     name: string;
     kind: 'table' | 'view';
@@ -241,6 +243,8 @@ export interface KtxScanConnector {
     capabilities: KtxConnectorCapabilities;
     eventStreamDiscovery?: KtxEventStreamDiscoveryPort;
     introspect(input: KtxScanInput, ctx: KtxScanContext): Promise<KtxSchemaSnapshot>;
+    listSchemas(): Promise<string[]>;
+    listTables(schemas?: string[]): Promise<KtxTableListEntry[]>;
     testConnection?(): Promise<KtxConnectorTestResult>;
     sampleColumn?(input: KtxColumnSampleInput, ctx: KtxScanContext): Promise<KtxColumnSampleResult>;
     sampleTable?(input: KtxTableSampleInput, ctx: KtxScanContext): Promise<KtxTableSampleResult>;
@@ -276,7 +280,7 @@ interface KtxScanArtifactPaths {
     manifestShards: string[];
     enrichmentArtifacts: string[];
 }
-type KtxScanWarningCode = 'connector_capability_missing' | 'sampling_failed' | 'statistics_failed' | 'llm_unavailable' | 'embedding_unavailable' | 'scan_enrichment_backend_not_configured' | 'relationship_validation_failed' | 'relationship_llm_invalid_reference' | 'relationship_llm_proposal_failed' | 'credential_redacted' | 'enrichment_failed' | 'description_fallback_used';
+type KtxScanWarningCode = 'connector_capability_missing' | 'sampling_failed' | 'statistics_failed' | 'llm_unavailable' | 'embedding_unavailable' | 'scan_enrichment_backend_not_configured' | 'relationship_validation_failed' | 'relationship_llm_invalid_reference' | 'relationship_llm_proposal_failed' | 'credential_redacted' | 'enrichment_failed' | 'description_fallback_used' | 'constraint_discovery_unauthorized';
 export interface KtxScanWarning {
     code: KtxScanWarningCode;
     message: string;

package/dist/context/scan/warehouse-catalog.js

@@ -1,4 +1,4 @@
-import { getDialectForDriver } from '../../context/connections/dialects.js';
+import { getDialectForDriver } from '../connections/dialects.js';
 function normalize(value) {
     return (value ?? '').toLowerCase();
 }
@@ -26,43 +26,20 @@ function splitDisplay(display) {
         .map(cleanIdentifierPart)
         .filter(Boolean);
 }
-function formatDisplay(driver, table) {
-    if (driver === 'sqlite' || driver === 'sqlite3') {
-        return table.name;
-    }
-    return [table.catalog, table.db, table.name].filter((part) => Boolean(part)).join('.');
+function formatDisplay(dialect, table) {
+    return dialect.formatDisplayRef(table);
 }
-function parseDisplay(driver, display) {
-    const parts = splitDisplay(display);
-    if (driver === 'sqlite' || driver === 'sqlite3') {
-        return parts.length === 1 ? { catalog: null, db: null, name: parts[0] } : null;
-    }
-    if (driver === 'bigquery' || driver === 'snowflake' || driver === 'sqlserver') {
-        if (parts.length !== 3) {
-            return null;
-        }
-        return { catalog: parts[0], db: parts[1], name: parts[2] };
-    }
-    if (parts.length === 2) {
-        return { catalog: null, db: parts[0], name: parts[1] };
-    }
-    if (parts.length === 3) {
-        return { catalog: parts[0], db: parts[1], name: parts[2] };
+function parseDisplay(dialect, display) {
+    const parsed = dialect.parseDisplayRef(display);
+    if (parsed) {
+        return parsed;
     }
+    const parts = splitDisplay(display);
     return parts.length === 1 ? { catalog: null, db: null, name: parts[0] } : null;
 }
-function expectedDisplayPartCount(driver) {
-    if (driver === 'sqlite' || driver === 'sqlite3') {
-        return 1;
-    }
-    if (driver === 'bigquery' || driver === 'snowflake' || driver === 'sqlserver') {
-        return 3;
-    }
-    return 2;
-}
-function parseColumnDisplay(driver, display) {
+function parseColumnDisplay(dialect, display) {
     const parts = splitDisplay(display);
-    const tablePartCount = expectedDisplayPartCount(driver);
+    const tablePartCount = dialect.columnDisplayTablePartCount();
     if (parts.length !== tablePartCount + 1) {
         return null;
     }
@@ -70,7 +47,7 @@ function parseColumnDisplay(driver, display) {
     if (!column) {
         return null;
     }
-    const table = parseDisplay(driver, parts.slice(0, -1).join('.'));
+    const table = dialect.parseDisplayRef(parts.slice(0, -1).join('.'));
     return table ? { ...table, column } : null;
 }
 function bestCandidates(tables, display, limit = 5) {
@@ -159,6 +136,7 @@ export class WarehouseCatalogService {
         if (!table) {
             return null;
         }
+        const dialect = getDialectForDriver(catalog.driver);
         const profileTables = catalog.profile?.tables ?? [];
         const profileTable = profileTables.find((candidate) => candidate.table && refsEqual(candidate.table, table));
         const profileColumns = catalog.profile?.columns ?? {};
@@ -167,7 +145,7 @@ export class WarehouseCatalogService {
             catalog: table.catalog,
             db: table.db,
             name: table.name,
-            display: formatDisplay(catalog.driver, table),
+            display: formatDisplay(dialect, table),
             kind: table.kind,
             comment: table.comment,
             description: firstDescription(table.descriptions),
@@ -195,34 +173,38 @@ export class WarehouseCatalogService {
         if (!catalog) {
             return { resolved: null, candidates: [], dialect: 'unknown' };
         }
-        const dialect = getDialectForDriver(catalog.driver).type;
-        const parsed = parseDisplay(catalog.driver, display);
+        const dialect = getDialectForDriver(catalog.driver);
+        const parsed = parseDisplay(dialect, display);
         if (!parsed) {
-            return { resolved: null, candidates: bestCandidates(catalog.tables, display), dialect };
+            return { resolved: null, candidates: bestCandidates(catalog.tables, display), dialect: dialect.type };
         }
-        const table = catalog.tables.find((candidate) => refsEqual(candidate, parsed));
+        const exactTable = catalog.tables.find((candidate) => refsEqual(candidate, parsed));
+        const looseNameMatches = parsed.catalog === null && parsed.db === null
+            ? catalog.tables.filter((candidate) => normalize(candidate.name) === normalize(parsed.name))
+            : [];
+        const table = exactTable ?? (looseNameMatches.length === 1 ? looseNameMatches[0] : undefined);
         if (!table) {
-            return { resolved: null, candidates: bestCandidates(catalog.tables, display), dialect };
+            return { resolved: null, candidates: bestCandidates(catalog.tables, display), dialect: dialect.type };
         }
-        return { resolved: { catalog: table.catalog, db: table.db, name: table.name }, candidates: [], dialect };
+        return { resolved: { catalog: table.catalog, db: table.db, name: table.name }, candidates: [], dialect: dialect.type };
     }
     async resolveDisplayTarget(connectionId, display) {
         const catalog = await this.loadCatalog(connectionId);
         if (!catalog) {
             return { resolved: null, candidates: [], dialect: 'unknown' };
         }
-        const dialect = getDialectForDriver(catalog.driver).type;
+        const dialect = getDialectForDriver(catalog.driver);
         const tableResolution = await this.resolveDisplay(connectionId, display);
         if (tableResolution.resolved) {
             return tableResolution;
         }
-        const parsedColumn = parseColumnDisplay(catalog.driver, display);
+        const parsedColumn = parseColumnDisplay(dialect, display);
         if (!parsedColumn) {
-            return { resolved: null, candidates: bestCandidates(catalog.tables, display), dialect };
+            return { resolved: null, candidates: bestCandidates(catalog.tables, display), dialect: dialect.type };
         }
         const table = catalog.tables.find((candidate) => refsEqual(candidate, parsedColumn));
         if (!table) {
-            return { resolved: null, candidates: bestCandidates(catalog.tables, display), dialect };
+            return { resolved: null, candidates: bestCandidates(catalog.tables, display), dialect: dialect.type };
         }
         return {
             resolved: {
@@ -232,7 +214,7 @@ export class WarehouseCatalogService {
                 column: parsedColumn.column,
             },
             candidates: [],
-            dialect,
+            dialect: dialect.type,
         };
     }
     async searchByName(connectionId, query, limit) {
@@ -240,6 +222,7 @@ export class WarehouseCatalogService {
         if (!catalog) {
             return [];
         }
+        const dialect = getDialectForDriver(catalog.driver);
         const hits = [];
         for (const table of catalog.tables) {
             const tableMatch = matchedOnTable(table, query);
@@ -248,7 +231,7 @@ export class WarehouseCatalogService {
                     kind: 'table',
                     connectionId,
                     ref: { catalog: table.catalog, db: table.db, name: table.name },
-                    display: formatDisplay(catalog.driver, table),
+                    display: formatDisplay(dialect, table),
                     matchedOn: tableMatch,
                 });
             }
@@ -261,7 +244,7 @@ export class WarehouseCatalogService {
                     kind: 'column',
                     connectionId,
                     ref: { catalog: table.catalog, db: table.db, name: table.name, column: column.name },
-                    display: `${formatDisplay(catalog.driver, table)}.${column.name}`,
+                    display: `${formatDisplay(dialect, table)}.${column.name}`,
                     matchedOn: columnMatch,
                 });
             }

package/dist/context/sl/local-query.js

@@ -21,17 +21,14 @@ function assertSafeConnectionId(connectionId) {
 function dialectForDriver(driver) {
     const normalized = (driver ?? 'postgres').toUpperCase();
     const map = {
-        POSTGRESQL: 'postgres',
         POSTGRES: 'postgres',
         BIGQUERY: 'bigquery',
         SNOWFLAKE: 'snowflake',
         MYSQL: 'mysql',
         SQLSERVER: 'tsql',
-        MSSQL: 'tsql',
         SQLITE: 'sqlite',
         DUCKDB: 'duckdb',
         CLICKHOUSE: 'clickhouse',
-        REDSHIFT: 'redshift',
         DATABRICKS: 'databricks',
     };
     return map[normalized] ?? 'postgres';

package/dist/context/sl/local-sl.js

@@ -176,19 +176,6 @@ export async function validateLocalSlSource(rawYaml, options) {
     try {
         const parsed = parseYamlRecord(rawYaml);
         const schema = parsed.table || parsed.sql ? sourceDefinitionSchema : sourceOverlaySchema;
-        if (schema === sourceOverlaySchema && Array.isArray(parsed.columns)) {
-            const sourceName = options?.sourceName ?? (typeof parsed.name === 'string' ? parsed.name : 'source');
-            const path = options?.connectionId && isSafeConnectionId(options.connectionId)
-                ? `semantic-layer/${options.connectionId}/${sourceName}.yaml`
-                : `${sourceName}.yaml`;
-            const legacyColumnPatchErrors = parsed.columns
-                .filter((column) => isRecord(column))
-                .filter((column) => typeof column.name === 'string' && (!column.expr || !column.type))
-                .map((column) => `${path}: column '${column.name}' patches a manifest column but is in 'columns:' — move it to 'column_overrides:'`);
-            if (legacyColumnPatchErrors.length > 0) {
-                return { valid: false, errors: legacyColumnPatchErrors };
-            }
-        }
         const result = schema.parse(parsed);
         const errors = [];
         if (options?.project && options.connectionId && 'table' in result && result.table) {

package/dist/context/sl/semantic-layer.service.js

@@ -920,17 +920,14 @@ export class SemanticLayerService {
     static mapDialect(connectionType) {
         const normalized = connectionType.toUpperCase();
         const map = {
-            POSTGRESQL: 'postgres',
             POSTGRES: 'postgres',
             BIGQUERY: 'bigquery',
             SNOWFLAKE: 'snowflake',
             MYSQL: 'mysql',
             SQLSERVER: 'tsql',
-            MSSQL: 'tsql',
             SQLITE: 'sqlite',
             DUCKDB: 'duckdb',
             CLICKHOUSE: 'clickhouse',
-            REDSHIFT: 'redshift',
             DATABRICKS: 'databricks',
         };
         return map[normalized] ?? 'postgres';
@@ -1263,7 +1260,7 @@ export function composeOverlay(base, overlay) {
     const computedColumns = normalizedOverlay.columns ?? [];
     for (const column of computedColumns) {
         if (baseByLowerName.has(column.name.toLowerCase())) {
-            throw new ColumnNameCollisionError(`column '${column.name}' in columns patches a manifest column on '${base.name}' — move it to 'column_overrides:'`);
+            throw new ColumnNameCollisionError(`column '${column.name}' in columns already exists on manifest source '${base.name}'`);
         }
         columnsByLowerName.set(column.name.toLowerCase(), column);
     }

package/dist/context/tools/context-candidate-write.tool.d.ts

@@ -14,8 +14,8 @@ declare const contextCandidateWriteInputSchema: z.ZodObject<{
         merge: "merge";
         conflict: "conflict";
         skip: "skip";
-        update: "update";
         create: "create";
+        update: "update";
     }>;
     durabilityScore: z.ZodNumber;
     authorityScore: z.ZodNumber;
@@ -49,8 +49,8 @@ export declare class ContextCandidateWriteTool extends BaseTool<typeof contextCa
             merge: "merge";
             conflict: "conflict";
             skip: "skip";
-            update: "update";
             create: "create";
+            update: "update";
         }>;
         durabilityScore: z.ZodNumber;
         authorityScore: z.ZodNumber;

package/dist/context-build-view.js

@@ -517,7 +517,7 @@ function isLocalSqlAnalysisConnectionRefused(input) {
 }
 function friendlyDriverName(driver) {
     const normalized = driver.toLowerCase();
-    if (normalized === 'postgres' || normalized === 'postgresql')
+    if (normalized === 'postgres')
         return 'PostgreSQL';
     if (normalized === 'mysql')
         return 'MySQL';