kirby-api 0.2.0__tar.gz → 0.2.2__tar.gz

{kirby_api-0.2.0/kirby_api.egg-info → kirby_api-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kirby-api
-Version: 0.2.0
+Version: 0.2.2
 Summary: Kirby — HERO System VTT backend API (FastAPI mono)
 Author: PeterB
 License: PolyForm Noncommercial License 1.0.0
@@ -32,7 +32,7 @@ Requires-Dist: pgvector>=0.3
 Requires-Dist: pypdfium2>=4.30
 Requires-Dist: kirby-combat==0.18.5
 Requires-Dist: kirby-dice>=0.1.0
-Requires-Dist: kirby-ai>=0.6.0
+Requires-Dist: kirby-ai>=0.7.0
 Requires-Dist: prometheus-client>=0.21
 Requires-Dist: prometheus-fastapi-instrumentator>=7.0
 Requires-Dist: tenacity>=8.0

kirby_api-0.2.2/kirby/_migrations/versions/20261005_0001_knowledge_indexes.py

@@ -0,0 +1,41 @@
+"""the three knowledge indexes the models declared and no migration made
+
+A schema-drift check (models vs a migrated database) found three
+``index=True`` columns that no migration ever created, in production as
+well as in a fresh ``alembic upgrade head``:
+
+* ``knowledge_link.campaign_id``  -> ``ix_knowledge_link_campaign_id``
+* ``knowledge_link.tenant_id``    -> ``ix_knowledge_link_tenant_id``
+* ``knowledge_unlock.tenant_id``  -> ``ix_knowledge_unlock_tenant_id``
+
+Purely additive and idempotent (``if_not_exists``): safe on a database
+where someone already created them by hand. The fourth drift item,
+``codex_chunk.idx_codex_chunk_embedding``, is deliberately NOT here —
+codex is moving to kirby-codex.
+
+Revision ID: 20261005_0001
+Revises: 20260918_0002
+Create Date: 2026-10-05
+"""
+from alembic import op
+
+revision = "20261005_0001"
+down_revision = "20260918_0002"
+branch_labels = None
+depends_on = None
+
+_INDEXES = (
+    ("ix_knowledge_link_campaign_id", "knowledge_link", "campaign_id"),
+    ("ix_knowledge_link_tenant_id", "knowledge_link", "tenant_id"),
+    ("ix_knowledge_unlock_tenant_id", "knowledge_unlock", "tenant_id"),
+)
+
+
+def upgrade() -> None:
+    for name, table, column in _INDEXES:
+        op.create_index(name, table, [column], if_not_exists=True)
+
+
+def downgrade() -> None:
+    for name, table, _column in _INDEXES:
+        op.drop_index(name, table_name=table, if_exists=True)

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby/builds/category_extraction.py

@@ -13,7 +13,7 @@ from __future__ import annotations
 import json
 import logging
 
-from kirby_ai import GenerationClient, strip_code_fences
+from kirby_ai import GenerationClient, extract_json_array
 
 from kirby.settings import settings
 
@@ -27,26 +27,11 @@ not the game mechanic. Emit ONLY the JSON array, no prose."""
 
 
 def _extract_json_array(raw: str) -> list:
-    """Parse a JSON array out of model output.
-
-    The sibling ``kirby_ai.extract_json_object`` handles objects; this is its
-    array counterpart — kirby-ai has no array extractor. Falls back to
-    slicing the outermost brackets so a model that wraps the array in prose
-    still parses.
-    """
-    text = strip_code_fences(raw)
+    """The model's slug array, or ``[]`` when there is none to find."""
     try:
-        v = json.loads(text)
+        return extract_json_array(raw)
     except json.JSONDecodeError:
-        s, e = text.find("["), text.rfind("]")
-        if s != -1 and e != -1 and e > s:
-            try:
-                v = json.loads(text[s:e + 1])
-            except json.JSONDecodeError:
-                return []
-        else:
-            return []
-    return v if isinstance(v, list) else []
+        return []
 
 
 def extract_category_slugs(

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby/builds/combatant_loader.py

@@ -1,4 +1,4 @@
-"""``HeroCombatant.from_db()`` and the to-legacy bridge.
+"""``HeroCombatant.from_db()`` — a stored build becomes a combatant.
 
 These helpers attach to (rather than monkey-patch into)
 ``kirby_combat.hero_view``. The reason: hero_view.py lives in the
@@ -18,12 +18,7 @@ from uuid import UUID
 
 from sqlalchemy.orm import Session
 
-from kirby_combat.hero_view import (
-    HeroCombatant,
-    HeroCombatState,
-    _compute_stats_from_hero,
-)
-from kirby_combat.models import Combatant
+from kirby_combat.hero_view import HeroCombatant
 
 from kirby.builds.build_doc_store import build_doc_from_rows
 from kirby.characters.models import CharacterDef
@@ -82,17 +77,11 @@ def hero_combatant_from_build_doc(
     if id is None:
         id = (hero.name.lower().replace(" ", "_") if hero.name
               else str(fallback_id))
-    stats = _compute_stats_from_hero(hero)
-    return HeroCombatant(
-        id=id,
-        hero=hero,
-        state=HeroCombatState(
-            current_stun=stats.max_stun,
-            current_body=stats.max_body,
-            current_end=stats.max_end,
-        ),
-        knockback_resistance=0,
-    )
+    # The engine's own door, not a copy of it: this used to construct the
+    # HeroCombatant here from the engine's PRIVATE `_compute_stats_from_hero`,
+    # agreeing with `from_build` only because the two happen to call the same
+    # function today.
+    return HeroCombatant.from_build(hero, id=id)
 
 
 def hero_combatant_from_db(
@@ -116,65 +105,3 @@ def hero_combatant_from_db(
         source=f"character {character_id}",
     )
 
-
-def to_legacy_combatant(hc: HeroCombatant) -> Combatant:
-    """Convert a HeroCombatant to the legacy flat ``Combatant`` shape.
-
-    DEPRECATED as of 2026-05-02 (combatant-redesign step 4 complete).
-    The engine now accepts HeroCombatant directly via the uniform
-    ``combat_stats()`` / flat-property interface added by step 6.
-    Pass ``HeroCombatant`` straight into ``AttackInput``; the engine
-    handles either shape.
-
-    Kept for backward compatibility with callers that haven't migrated.
-    Will be removed in a future kirby-api release.
-    """
-    s = hc.combat_stats()
-    defs = hc.defense_view()
-    attacks = []
-    attack_xmlids = {"ENERGYBLAST", "RKA", "HKA", "EGOATTACK",
-                     "MENTALBLAST", "KILLINGATTACKRANGED",
-                     "KILLINGATTACKHTH"}
-    seen_xmlids: set[str] = set()
-
-    def _collect_attacks(power_list) -> None:
-        """Walk top-level + nested (multipower/VPP slot) powers."""
-        for p in power_list or []:
-            x = (getattr(p, "xmlid", None) or "").upper()
-            if x in attack_xmlids and x not in seen_xmlids:
-                # attack_view picks the first match by xmlid; if a
-                # combatant has multiple ENERGYBLASTs we still get
-                # only the first slot. That's fine for the v1 demo;
-                # step 3 wires slot_xmlid for proper Multipower
-                # selection.
-                try:
-                    attacks.append(hc.attack_view(p.xmlid))
-                    seen_xmlids.add(x)
-                except ValueError:
-                    pass
-            # Recurse into sub_powers — Multipower / VPP slots live here.
-            sub = getattr(p, "sub_powers", None)
-            if sub:
-                _collect_attacks(sub)
-
-    _collect_attacks(hc.hero.powers)  # type: ignore[union-attr]
-    name = (
-        getattr(hc.hero, "name", None) or hc.id
-    )
-    return Combatant(
-        id=hc.id,
-        name=name,
-        ocv=s.ocv, dcv=s.dcv, omcv=s.omcv, dmcv=s.dmcv,
-        spd=s.spd,
-        dex=s.dex, ego=s.ego, str_=s.str_, con=s.con, pre=s.pre, rec=s.rec,
-        pd=s.pd, ed=s.ed,
-        rpd=s.rpd, red=s.red, md=s.md,
-        power_defense=s.power_defense, flash_defense=s.flash_defense,
-        max_stun=s.max_stun, max_body=s.max_body, max_end=s.max_end,
-        current_stun=hc.state.current_stun,
-        current_body=hc.state.current_body,
-        current_end=hc.state.current_end,
-        attacks=attacks,
-        defenses=defs,
-        knockback_resistance=hc.knockback_resistance,
-    )

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby/builds/power_library_service.py

@@ -219,40 +219,3 @@ def get_power_tree(db: Session, power_root_object_id: int) -> dict[str, Any] | N
 
 
 # ── convenience: render a hit for prompt context ────────────────────────────
-
-
-def render_hit_for_prompt(
-    db: Session,
-    hit: PowerLibraryHit,
-    *,
-    include_tree: bool = False,
-) -> str:
-    """Format one hit as a prompt-ready block. Pass 1 typically uses
-    ``include_tree=False`` (prose + tags only); Pass 2 uses
-    ``include_tree=True`` to give the structured-output model the exact
-    JSON shape it should produce.
-    """
-    parts: list[str] = []
-    parts.append(f"## {hit.name}")
-    if hit.base_name and hit.base_name != hit.name:
-        parts.append(f"  (variant of {hit.base_name})")
-    tags = ", ".join(hit.sfx_tags + hit.mechanical_tags)
-    parts.append(f"  Tags: {tags}")
-    ap = f"{hit.active_cost}" if hit.active_cost is not None else "?"
-    rc = f"{hit.real_cost}" if hit.real_cost is not None else "?"
-    parts.append(f"  Active: {ap}  Real: {rc}  Kind: {hit.kind}")
-    if hit.description:
-        snippet = hit.description.strip()
-        if len(snippet) > 1200:
-            snippet = snippet[:1200].rsplit(" ", 1)[0] + "…"
-        parts.append("")
-        parts.append(snippet)
-
-    if include_tree:
-        import json
-        tree = get_power_tree(db, hit.power_root_object_id)
-        parts.append("")
-        parts.append("```json")
-        parts.append(json.dumps(tree, indent=2, default=str))
-        parts.append("```")
-    return "\n".join(parts)

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby/campaigns/services/extract.py

@@ -28,12 +28,11 @@ section/paragraph they came from.
 from __future__ import annotations
 
 import json
-import re
 from dataclasses import asdict, dataclass, field
 from typing import Any, Optional
 from uuid import UUID
 
-from kirby_ai import GenerationClient
+from kirby_ai import GenerationClient, extract_json_array
 from sqlalchemy.orm import Session
 
 from kirby.settings import settings
@@ -123,48 +122,17 @@ def parse_drafts_json(raw: str) -> list[ExtractedDraft]:
 
     Tolerant of:
     - Markdown code fences around the JSON (strip them)
-    - Trailing prose after the array (find the first complete `[...]`)
+    - Prose on either side of the array (kirby_ai.extract_json_array)
     - Individual draft items missing optional fields
 
     Returns the drafts that parsed cleanly. Skips malformed entries
     rather than failing the whole import.
     """
-    cleaned = raw.strip()
-    # Strip markdown fences
-    if cleaned.startswith("```"):
-        cleaned = re.sub(r"^```(?:json)?\s*", "", cleaned)
-        cleaned = re.sub(r"\s*```\s*$", "", cleaned)
-    cleaned = cleaned.strip()
-
-    # Find the first balanced [...] in the string. The LLM occasionally
-    # emits trailing chatter; bracket-counting locates the canonical array.
-    start = cleaned.find("[")
-    if start < 0:
-        return []
-
-    depth = 0
-    end = -1
-    for i in range(start, len(cleaned)):
-        c = cleaned[i]
-        if c == "[":
-            depth += 1
-        elif c == "]":
-            depth -= 1
-            if depth == 0:
-                end = i + 1
-                break
-    if end < 0:
-        return []
-
-    json_str = cleaned[start:end]
     try:
-        parsed = json.loads(json_str)
+        parsed = extract_json_array(raw)
     except json.JSONDecodeError:
         return []
 
-    if not isinstance(parsed, list):
-        return []
-
     drafts: list[ExtractedDraft] = []
     for item in parsed:
         if not isinstance(item, dict):

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby/characters/routes.py

@@ -462,28 +462,27 @@ async def import_hdc(
     content = await file.read()
     filename = file.filename or "unknown.hdc"
 
-    # Parse the HDC via EntityService (uses kirby.engines.cost — the
-    # canonical kirby-cost loader). The relational
-    # ``character_objects`` rows are the source of truth for combat
-    # state; PR-4 dropped the legacy ``combatant_jsonb`` projection.
-    # We still grab the parsed Combatant dataclass to seed the
-    # transient ``current_*`` columns.
-    from kirby.characters.services.entity_service import EntityService
+    # The canonical import: the engine loads and costs the file, the build is
+    # persisted relationally (so combat can load it), and the opening vitals
+    # are the engine's own (`HeroCombatant.from_build`, the same door a fight
+    # uses). This route used to run a private translator that built its own
+    # combatant -- base + CHARACTERISTICS levels only, so a character with
+    # Resistant Protection got rPD 0 -- kept only STUN/BODY/END from it, and
+    # never persisted the build at all, so combat refused the character.
+    from kirby.builds.build_doc_store import persist_loaded
+    from kirby.engines.cost.hdc import load_hdc
+    from kirby_combat.hero_view import HeroCombatant
 
     try:
-        imported = EntityService().import_hdc(content)
-    except (ValueError, Exception) as exc:
+        imported = load_hdc(content)
+    except Exception as exc:
         raise HTTPException(
             status_code=400,
             detail=f"Failed to parse HDC: {exc}",
         )
-
-    # Bases (kind=='base') don't produce a Combatant — they're scene
-    # fragments. Allow the import (so the file can be referenced) but
-    # the combatant-derived transient state stays None and the row
-    # can't be added to a session.
-    name = imported.name or filename.replace(".hdc", "").replace("_", " ")
-    cb = imported.combatant
+    hero = imported.hero
+    vitals = HeroCombatant.from_build(hero).state
+    name = (hero.name or "").strip() or filename.replace(".hdc", "").replace("_", " ")
 
     char = CharacterDef(
         tenant_id=tid,
@@ -491,14 +490,15 @@ async def import_hdc(
         name=name,
         hdc_filename=filename,
         source="hdc_import",
-        is_npc=(imported.kind == "npc"),
-        # Seed transient state from the combatant's max values so the
-        # character is ready to take a hit on first session start.
-        current_stun=cb.current_stun if cb is not None else None,
-        current_body=cb.current_body if cb is not None else None,
-        current_end=cb.current_end if cb is not None else None,
+        # A file upload is a character; the GM marks NPCs as with any field.
+        is_npc=False,
+        current_stun=vitals.current_stun,
+        current_body=vitals.current_body,
+        current_end=vitals.current_end,
     )
     db.add(char)
+    db.flush()
+    persist_loaded(db, char, hero)
     db.commit()
     db.refresh(char)
     return char

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby/codex/services/villain_extractor.py

@@ -7,10 +7,9 @@ from __future__ import annotations
 
 import json
 import logging
-import re
 from dataclasses import dataclass, field
 
-from kirby_ai import GenerationClient
+from kirby_ai import GenerationClient, extract_json_object
 
 from kirby.settings import settings
 
@@ -78,16 +77,10 @@ minions", "Avoid melee with bricks") tied to a source_snippet quoted from the te
 
 def _extract_json(raw: str) -> dict:
     """Pull the first JSON object out of model output (tolerates fences/prose)."""
-    fenced = re.search(r"```(?:json)?\s*(\{.*?\})\s*```", raw, re.DOTALL)
-    candidate = fenced.group(1) if fenced else raw
-    start = candidate.find("{")
-    end = candidate.rfind("}")
-    if start == -1 or end == -1 or end < start:
-        raise ExtractionError(f"no JSON object in model output: {raw[:200]!r}")
     try:
-        return json.loads(candidate[start:end + 1])
+        return extract_json_object(raw)
     except json.JSONDecodeError as exc:
-        raise ExtractionError(f"invalid JSON: {exc}; raw={raw[:200]!r}") from exc
+        raise ExtractionError(f"no JSON object in model output: {raw[:200]!r}") from exc
 
 
 def parse_villain_writeup(raw: str) -> VillainWriteup:

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby/combat/__init__.py

@@ -18,15 +18,13 @@ tables, its routes and its tests. What stands today:
                        runner's turn and step caps; its seeding and its
                        worker await the harness (plan C5)
 
-``services/entity_service`` IS NOT HERE ANY MORE. HDC import is a
-characters/builds concern and its only caller is
-``kirby/characters/routes.py``; it lived here until the 2026-09-17 final
-review found it was a parser, a classifier and a dozen substituted 6E
-numbers inside the one module that claims to hold no rule. It is
-``kirby/characters/services/entity_service.py`` now, and
-``tests/combat/test_no_rules_here.py`` grew a fourth property — the
-engine's rule vocabulary, turned on this SOURCE tree — so the next one
-is caught rather than certified.
+``services/entity_service`` IS GONE. It moved from here to
+``kirby/characters/`` after the 2026-09-17 review found a parser, a classifier
+and substituted 6E numbers inside it, and was deleted 2026-10-04: the HDC
+upload now takes the canonical import (``load_hdc`` -> ``persist_loaded``) and
+the engine's own ``HeroCombatant.from_build`` for its opening vitals.
+``tests/combat/test_no_rules_here.py`` scans the WHOLE ``kirby/`` source tree
+for rule numbers now, so moving a file out of ``combat/`` hides nothing.
 
 ``async_db`` IS NOT HERE ANY MORE EITHER. It was the deleted
 ``websocket.py``'s dependency, it had no consumer left anywhere in

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby/comic_canon/__init__.py

@@ -7,15 +7,16 @@ Design mirrors codex:
                      comic_character, comic_storyarc, comic_chunk,
                      comic_archetype, comic_book_tenant)
   services/
-    comic_vine.py  — async Comic Vine API client (port from
-                     comic-mcp standalone)
     archetypes.py  — YAML loader → DB sync, tenant-scoped
-    prewarm.py     — pre-warm pipeline; tenant + group aware
     search.py      — cosine search over comic_chunk + retrieval
                      helpers used by brain.py
   routes.py        — FastAPI routes for archetype CRUD + cache stats
   schemas.py       — Pydantic schemas
 
+The Comic Vine client and pre-warm pipeline were removed 2026-10-04: nothing
+imported them. Their live home is the standalone comic-mcp
+(`comic_mcp/{comic_vine,prewarm,text_clean}.py`).
+
 Tenant pattern: comic_book_tenant junction grants tenant access to
 specific Comic Vine resources, paralleling codex_book_tenant. Tenants
 inherit a universal-baseline archetype set and can author their own

{kirby_api-0.2.0 → kirby_api-0.2.2/kirby_api.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kirby-api
-Version: 0.2.0
+Version: 0.2.2
 Summary: Kirby — HERO System VTT backend API (FastAPI mono)
 Author: PeterB
 License: PolyForm Noncommercial License 1.0.0
@@ -32,7 +32,7 @@ Requires-Dist: pgvector>=0.3
 Requires-Dist: pypdfium2>=4.30
 Requires-Dist: kirby-combat==0.18.5
 Requires-Dist: kirby-dice>=0.1.0
-Requires-Dist: kirby-ai>=0.6.0
+Requires-Dist: kirby-ai>=0.7.0
 Requires-Dist: prometheus-client>=0.21
 Requires-Dist: prometheus-fastapi-instrumentator>=7.0
 Requires-Dist: tenacity>=8.0

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby_api.egg-info/SOURCES.txt

@@ -105,6 +105,7 @@ kirby/_migrations/versions/20260917_0001_session_names_a_seat.py
 kirby/_migrations/versions/20260918_0000_drop_combat_tables.py
 kirby/_migrations/versions/20260918_0001_harness_tables.py
 kirby/_migrations/versions/20260918_0002_narration_speaker_id.py
+kirby/_migrations/versions/20261005_0001_knowledge_indexes.py
 kirby/_migrations/versions/21980c836860_add_asset_id_refs.py
 kirby/_migrations/versions/ba28e71c955d_codex_init.py
 kirby/admin/__init__.py
@@ -121,8 +122,6 @@ kirby/builds/category_extraction.py
 kirby/builds/cli.py
 kirby/builds/codex_linker.py
 kirby/builds/combatant_loader.py
-kirby/builds/concept_to_build.py
-kirby/builds/hdc_exporter.py
 kirby/builds/importer.py
 kirby/builds/json_exporter.py
 kirby/builds/json_importer.py
@@ -171,7 +170,6 @@ kirby/characters/proposals/service.py
 kirby/characters/services/__init__.py
 kirby/characters/services/character_info_backfill.py
 kirby/characters/services/effective.py
-kirby/characters/services/entity_service.py
 kirby/characters/services/writeup_persistence.py
 kirby/codex/__init__.py
 kirby/codex/cli.py
@@ -215,10 +213,7 @@ kirby/comic_canon/models.py
 kirby/comic_canon/routes.py
 kirby/comic_canon/services/__init__.py
 kirby/comic_canon/services/archetypes.py
-kirby/comic_canon/services/comic_vine.py
 kirby/comic_canon/services/embeddings.py
-kirby/comic_canon/services/prewarm.py
-kirby/comic_canon/services/text_clean.py
 kirby/engines/__init__.py
 kirby/engines/cost/__init__.py
 kirby/engines/cost/hdc.py
@@ -249,6 +244,7 @@ kirby_api.egg-info/dependency_links.txt
 kirby_api.egg-info/entry_points.txt
 kirby_api.egg-info/requires.txt
 kirby_api.egg-info/top_level.txt
+tests/test_knowledge_indexes_migration.py
 tests/test_public_vocabulary.py
 tests/test_teardown.py
 tests/test_wheel_contents.py

{kirby_api-0.2.0 → kirby_api-0.2.2}/kirby_api.egg-info/requires.txt

@@ -23,7 +23,7 @@ pgvector>=0.3
 pypdfium2>=4.30
 kirby-combat==0.18.5
 kirby-dice>=0.1.0
-kirby-ai>=0.6.0
+kirby-ai>=0.7.0
 prometheus-client>=0.21
 prometheus-fastapi-instrumentator>=7.0
 tenacity>=8.0

{kirby_api-0.2.0 → kirby_api-0.2.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "kirby-api"
-version = "0.2.0"
+version = "0.2.2"
 description = "Kirby — HERO System VTT backend API (FastAPI mono)"
 # Published to public PyPI. Every runtime dependency below resolves from
 # public PyPI too — kirby-cost, kirby-combat, kirby-dice and kirby-ai all
@@ -196,9 +196,7 @@ dependencies = [
     # better a public `EVENT_KINDS` mapping the engine owns and both
     # repos read (it already builds one to key its own serializer).
     # When that lands, `events.py` reads the public name and this goes
-    # back to `>=`. `kirby/builds/hdc_exporter.py:32` has the same shape
-    # against kirby-cost (`from kirby_cost.io.build_json import
-    # _root_from_doc`) and wants the same ask.
+    # back to `>=`.
     "kirby-combat==0.18.5",
 
     # The dice roller left kirby-combat at 0.10.0 for its own package, so
@@ -217,7 +215,7 @@ dependencies = [
     # `AttributeError` inside a fight — and every earlier version silently
     # substitutes `FirstLegalChooser`, which is a model playing on without
     # having answered. `IntentChooser` also arrives in 0.3.0.
-    "kirby-ai>=0.6.0",
+    "kirby-ai>=0.7.0",
     "prometheus-client>=0.21",
     "prometheus-fastapi-instrumentator>=7.0",
 

kirby_api-0.2.2/tests/test_knowledge_indexes_migration.py

@@ -0,0 +1,28 @@
+"""The knowledge indexes the models declare must exist after migrating."""
+import pytest
+from sqlalchemy import text
+
+from kirby.shared.db import SessionLocal
+
+EXPECTED = {
+    ("knowledge_link", "ix_knowledge_link_campaign_id"),
+    ("knowledge_link", "ix_knowledge_link_tenant_id"),
+    ("knowledge_unlock", "ix_knowledge_unlock_tenant_id"),
+}
+
+
+def test_knowledge_indexes_exist():
+    db = SessionLocal()
+    try:
+        if db.get_bind().dialect.name != "postgresql":
+            pytest.skip("pg_indexes needs Postgres")
+        rows = db.execute(
+            text(
+                "SELECT tablename, indexname FROM pg_indexes "
+                "WHERE indexname = ANY(:names)"
+            ),
+            {"names": [n for _, n in EXPECTED]},
+        ).all()
+        assert {tuple(r) for r in rows} == EXPECTED
+    finally:
+        db.close()