From cbe9594a3dcac1f91a6baa7ac99a138c22a71a8a Mon Sep 17 00:00:00 2001
From: Simon Willison <swillison@gmail.com>
Date: Thu, 28 May 2026 11:00:04 -0700
Subject: [PATCH] Use SQLiteTableType directly in SQL analysis

Remove the redundant SQLTableKind alias from the write SQL analysis model. Operation.table_kind and the analyzer cache now use the SQLite metadata classification type directly, making the source of table-kind values clearer.
---
 datasette/utils/sql_analysis.py | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/datasette/utils/sql_analysis.py b/datasette/utils/sql_analysis.py
index b5d7ada8..0a3a947c 100644
--- a/datasette/utils/sql_analysis.py
+++ b/datasette/utils/sql_analysis.py
@@ -42,7 +42,6 @@ SQLTargetType = Literal[
 SQLTableOperation = Literal["read", "insert", "update", "delete"]
 SQLSchemaOperation = Literal["create", "drop"]
 SQLSchemaTargetType = Literal["index", "table", "trigger", "view", "virtual-table"]
-SQLTableKind = SQLiteTableType
 
 
 @dataclass(frozen=True)
@@ -52,7 +51,7 @@ class Operation:
     database: str | None
     table: str | None
     sqlite_schema: str | None
-    table_kind: SQLTableKind | None = None
+    table_kind: SQLiteTableType | None = None
     target: str | None = None
     columns: tuple[str, ...] = ()
     source: str | None = None
@@ -428,7 +427,7 @@ def analyze_sql_tables(
         )
         return sqlite3.SQLITE_OK
 
-    table_kind_cache: dict[tuple[str | None, str], SQLTableKind | None] = {}
+    table_kind_cache: dict[tuple[str | None, str], SQLiteTableType | None] = {}
 
     conn.set_authorizer(authorizer)
     try:
@@ -523,7 +522,7 @@ def analyze_sql_tables(
             return True
         return False
 
-    def table_kind_for(key: OperationKey) -> SQLTableKind | None:
+    def table_kind_for(key: OperationKey) -> SQLiteTableType | None:
         if (
             key.target_type != "table"
             or key.operation not in {"read", "insert", "update", "delete"}