mirror of
https://github.com/memohai/Memoh.git
synced 2026-04-25 07:00:48 +09:00
Acbox Liu
8d5c38f0e5
* refactor: unify providers and models tables
- Rename `llm_providers` → `providers`, `llm_provider_oauth_tokens` → `provider_oauth_tokens`
- Remove `tts_providers` and `tts_models` tables; speech models now live in the unified `models` table with `type = 'speech'`
- Replace top-level `api_key`/`base_url` columns with a JSONB `config` field on `providers`
- Rename `llm_provider_id` → `provider_id` across all references
- Add `edge-speech` client type and `conf/providers/edge.yaml` default provider
- Create new read-only speech endpoints (`/speech-providers`, `/speech-models`) backed by filtered views of the unified tables
- Remove old TTS CRUD handlers; simplify speech page to read-only + test
- Update registry loader to skip malformed YAML files instead of failing entirely
- Fix YAML quoting for model names containing colons in openrouter.yaml
- Regenerate sqlc, swagger, and TypeScript SDK
* fix: exclude speech providers from providers list endpoint
ListProviders now filters out client_type matching '%-speech' so Edge
and future speech providers no longer appear on the Providers page.
ListSpeechProviders uses the same pattern match instead of hard-coding
'edge-speech'.
* fix: use explicit client_type list instead of LIKE pattern
Replace '%-speech' pattern with explicit IN ('edge-speech') for both
ListProviders (exclusion) and ListSpeechProviders (inclusion). New
speech client types must be added to both queries.
* fix: use EXECUTE for dynamic SQL in migrations referencing old schema
PL/pgSQL pre-validates column/table references in static SQL statements
inside DO blocks before evaluating IF/RETURN guards. This caused
migrations 0010-0061 to fail on fresh databases where the canonical
schema uses `providers`/`provider_id` instead of `llm_providers`/
`llm_provider_id`.
Wrap all SQL that references potentially non-existent old schema objects
(llm_providers, llm_provider_id, tts_providers, tts_models, etc.) in
EXECUTE strings so they are only parsed at runtime when actually reached.
* fix: revert canonical schema to use llm_providers for migration compatibility
The CI migrations workflow (up → down → up) failed because 0061 down
renames `providers` back to `llm_providers`, but 0001 down only dropped
`providers` — leaving `llm_providers` as a remnant. On the second
migrate up, 0010 found the stale `llm_providers` and tried to reference
`models.llm_provider_id` which no longer existed.
Revert 0001 canonical schema to use original names (llm_providers,
tts_providers, tts_models) so incremental migrations work naturally and
0061 handles the final rename. Remove EXECUTE wrappers and unnecessary
guards from migrations that now always operate on llm_providers.
* fix: icons
* fix: sync canonical schema with 0061 migration to fix sqlc column mismatch
0001_init.up.sql still used old names (llm_providers, llm_provider_id)
and included dropped tts_providers/tts_models tables. sqlc could not
parse the PL/pgSQL EXECUTE in migration 0061, so generated code retained
stale columns (input_modalities, supports_reasoning) causing runtime
"column does not exist" errors when adding models.
- Update 0001_init.up.sql to current schema (providers, provider_id,
no tts tables, add provider_oauth_tokens)
- Use ALTER TABLE IF EXISTS in 0010/0041/0042 for backward compat
- Regenerate sqlc
* fix: guard all legacy migrations against fresh schema for CI compat
On fresh databases, 0001_init.up.sql creates providers/provider_id
(not llm_providers/llm_provider_id). Migrations 0013, 0041, 0046, 0047
referenced the old names without guards, causing CI migration failures.
- 0013: check llm_provider_id column exists before adding old constraint
- 0041: check llm_providers table exists before backfill/constraint DDL
- 0046: wrap CREATE TABLE in DO block with llm_providers existence check
- 0047: use ALTER TABLE IF EXISTS + DO block guard
68 lines
3.1 KiB
SQL
68 lines
3.1 KiB
SQL
-- 0041_provider_model_refactor
|
||
-- Move client_type to llm_providers, add icon, replace model columns with config JSONB.
|
||
|
||
-- 1. Add client_type and icon to llm_providers (IF EXISTS for fresh-schema compat)
|
||
ALTER TABLE IF EXISTS llm_providers
|
||
ADD COLUMN IF NOT EXISTS client_type TEXT NOT NULL DEFAULT 'openai-completions',
|
||
ADD COLUMN IF NOT EXISTS icon TEXT;
|
||
|
||
-- 2–6. Backfill and migrate only when old columns exist (idempotent for fresh DBs).
|
||
DO $$ BEGIN
|
||
-- Back-fill provider client_type from models.client_type (old column).
|
||
-- Only runs on pre-0061 schema where llm_providers table still exists.
|
||
IF EXISTS (SELECT 1 FROM information_schema.tables WHERE table_schema = 'public' AND table_name = 'llm_providers')
|
||
AND EXISTS (SELECT 1 FROM information_schema.columns WHERE table_name = 'models' AND column_name = 'client_type')
|
||
THEN
|
||
UPDATE llm_providers p
|
||
SET client_type = sub.client_type
|
||
FROM (
|
||
SELECT DISTINCT ON (llm_provider_id) llm_provider_id, client_type
|
||
FROM models
|
||
WHERE client_type IS NOT NULL AND client_type != ''
|
||
ORDER BY llm_provider_id, created_at ASC
|
||
) sub
|
||
WHERE p.id = sub.llm_provider_id;
|
||
END IF;
|
||
|
||
-- Add CHECK constraint (skip if already present or table renamed)
|
||
IF EXISTS (SELECT 1 FROM information_schema.tables WHERE table_schema = 'public' AND table_name = 'llm_providers')
|
||
AND NOT EXISTS (SELECT 1 FROM information_schema.table_constraints WHERE constraint_name = 'llm_providers_client_type_check')
|
||
THEN
|
||
ALTER TABLE llm_providers
|
||
ADD CONSTRAINT llm_providers_client_type_check
|
||
CHECK (client_type IN ('openai-responses', 'openai-completions', 'anthropic-messages', 'google-generative-ai'));
|
||
END IF;
|
||
|
||
-- Add config JSONB to models
|
||
IF NOT EXISTS (SELECT 1 FROM information_schema.columns WHERE table_name = 'models' AND column_name = 'config') THEN
|
||
ALTER TABLE models ADD COLUMN config JSONB NOT NULL DEFAULT '{}'::jsonb;
|
||
END IF;
|
||
|
||
-- Migrate existing columns into config (only when old columns exist)
|
||
IF EXISTS (SELECT 1 FROM information_schema.columns WHERE table_name = 'models' AND column_name = 'dimensions') THEN
|
||
UPDATE models SET config = jsonb_strip_nulls(jsonb_build_object(
|
||
'dimensions', dimensions,
|
||
'compatibilities', (
|
||
SELECT coalesce(jsonb_agg(c), '[]'::jsonb) FROM (
|
||
SELECT 'tool-call' AS c
|
||
UNION ALL
|
||
SELECT 'vision' WHERE 'image' = ANY(input_modalities)
|
||
UNION ALL
|
||
SELECT 'reasoning' WHERE supports_reasoning = true
|
||
) AS caps
|
||
),
|
||
'context_window', NULL
|
||
));
|
||
END IF;
|
||
END $$;
|
||
|
||
-- Drop old columns and constraints (IF EXISTS makes these safe on fresh DBs)
|
||
ALTER TABLE models DROP CONSTRAINT IF EXISTS models_client_type_check;
|
||
ALTER TABLE models DROP CONSTRAINT IF EXISTS models_chat_client_type_check;
|
||
ALTER TABLE models DROP CONSTRAINT IF EXISTS models_dimensions_check;
|
||
|
||
ALTER TABLE models DROP COLUMN IF EXISTS client_type;
|
||
ALTER TABLE models DROP COLUMN IF EXISTS dimensions;
|
||
ALTER TABLE models DROP COLUMN IF EXISTS input_modalities;
|
||
ALTER TABLE models DROP COLUMN IF EXISTS supports_reasoning;
|