refactor: improve sqlite-introspector metadata query

[tgriesser]

Minor change to follow up on #1344

The change to using a union all means that although there's now only one query, the query grows to be as long as the number of tables in the DB, which feels a little off.

Decided to see if there was a way to handle this that was a little cleaner and came up with:

with "table_list" as (
  select "name", "sql", "type" from "sqlite_master"
  where "type" in ('table', 'view')
  and "name" not like 'sqlite_%' order by "name"
)
select
  "tl"."name" as "table",
  "p"."cid",
  "p"."name",
  "p"."type",
  "p"."notnull",
  "p"."dflt_value",
  "p"."pk"
from
  "table_list" as "tl",
  pragma_table_info(tl.name) as "p"
order by "tl.name", "p.cid"

Which feels a little nicer than an really large union query

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
kysely	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 8, 2025 9:54pm

[pkg-pr-new]

Open in Stackblitz • kysely_koa_example

npm i https://pkg.pr.new/kysely-org/kysely@1349

commit: f5f005b

[igalklebanov]

LGTM! 🚀

Thanks! 🙏