fix(autocomplete): suggest clause keywords and correct columns after …

[thomaswasle]

closes #293.

Summary

• Keywords never suggested after FROM clause — The completion provider only
  offered keyword completions when no context columns were available. Removed the
  contextColumnSuggestions.length === 0 guard so clause keywords (WHERE, ORDER BY,
  GROUP BY, LIMIT, …) are always included alongside column and table suggestions.

• SQL keywords captured as table aliases — parseTablesFromQuery matched FROM users WHERE and registered WHERE as the alias, poisoning the alias map. Added a
  SQL_RESERVED set; any captured alias that is a reserved word is discarded and the
  table name is used as its own alias.

• Wrong columns in MySQL multiDb mode — When activeSchema is null (multiDb),
  getTableColumns was called without a schema, causing MySQL to fall back to
  information_schema and return system columns instead of the user's table columns.
  Fixed by adding an optional schema?: string field to TableInfo and populating it
  with the source database name when building effectiveTables in multiDb mode. Both
  the context-aware and dot-trigger paths now pass t.schema ?? schema.

• Column cache poisoning — The fallback that attempts column fetches before the
  tables list loads could receive an empty response (schema mismatch / connection not
  yet ready) and cache it, blocking all subsequent fetches for that table. Empty results
  are now never written to the cache.

Files changed

File	Change
src/utils/autocomplete.ts	Remove keyword gate; add schema-aware table lookup for dot-trigger; use t.schema in context-aware path; skip caching empty results
src/utils/sqlAnalysis.ts	Add SQL_RESERVED set; filter captured aliases against it
src/contexts/DatabaseContext.ts	Add optional schema?: string to TableInfo
src/pages/Editor.tsx	Annotate each table with its source database when building effectiveTables in multiDb mode

Test plan

• MySQL multiDb: type SELECT * FROM <table> → columns from the correct database
  appear
• MySQL multiDb: type SELECT * FROM <table> WHERE → clause keywords (AND, OR,
  …) and column names both appear
• Type SELECT * FROM <table> WHERE <partial> → matching column names filter
  correctly
• Dot-trigger table. → columns from the correct per-database table appear
• Single-database connections: keywords still appear; dot-trigger still works
• Non-MySQL drivers: no regression in keyword or column suggestions

[kilo-code-bot]

CRITICAL: Missing 'using' in SQL_RESERVED set

USING is a standard SQL keyword in JOIN ... USING(...) clauses. The fromPattern regex captures the word after a table name as a potential alias, so FROM a JOIN b USING(id) incorrectly registers using as the alias for b. This breaks dot-trigger autocomplete for that table.

Suggested change

	'asc', 'desc', 'null', 'with', 'exists', 'all', 'any',
	'asc', 'desc', 'null', 'with', 'exists', 'all', 'any', 'using',

[kilo-code-bot]

WARNING: Alias dot-trigger uses wrong schema lookup in multiDb mode

When typedName is an alias (e.g. u aliasing users), this looks up the table list by the alias instead of the resolved actualTableName. In multiDb mode foundTable will be undefined, so schema falls back to activeSchema (which is null), and MySQL receives no schema filter.

The context-aware path already looks up by actual table name — the dot-trigger path should do the same.

Suggested change

	const foundTable = tables.find(t => t.name.toLowerCase() === typedName);
	const foundTable = tables.find(t => t.name.toLowerCase() === actualTableName.toLowerCase());

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The previous issues have been resolved in the latest commit.

Resolved Issues (2)

Severity	File	Line	Issue
CRITICAL	src/utils/sqlAnalysis.ts	10	Missing 'using' in SQL_RESERVED set
WARNING	src/utils/autocomplete.ts	157	Alias dot-trigger uses wrong schema lookup in multiDb mode

Files Reviewed (5 files)

• src/utils/autocomplete.ts — no issues
• src/utils/sqlAnalysis.ts — no issues
• src/contexts/DatabaseContext.ts — no issues
• src/pages/Editor.tsx — no issues
• src-tauri/Cargo.lock — generated file, skipped

---

_{Reviewed by kimi-k2.6-20260420 · 273,521 tokens}