# Decisions Register

<!-- Append-only. Never edit or remove existing rows.
     To reverse a decision, add a new row that supersedes it.
     Read this file at the start of any planning or research phase. -->

| # | When | Scope | Decision | Choice | Rationale | Revisable? | Made By |
|---|------|-------|----------|--------|-----------|------------|---------|
| D001 |  | architecture | Storage layer selection | PostgreSQL for structured data, Qdrant (existing instance at 10.0.0.10) for vector embeddings, local filesystem for transcript JSON files | Spec specifies PostgreSQL for JSONB support and Qdrant already running on hypervisor. File store on local filesystem organized by creator slug. Qdrant gets a dedicated collection, not a new instance. | Yes | collaborative |
| D002 |  | architecture | Timestamp handling for asyncpg with TIMESTAMP WITHOUT TIME ZONE columns | Use datetime.now(timezone.utc).replace(tzinfo=None) in _now() helper | asyncpg rejects timezone-aware datetimes for TIMESTAMP WITHOUT TIME ZONE columns. Stripping tzinfo at the application layer preserves UTC semantics while staying compatible with the existing schema. | Yes | agent |
| D003 |  | requirement | R002 Transcript Ingestion API status | validated | 6 passing integration tests prove the full POST /api/v1/ingest flow: creator auto-detection, SourceVideo upsert, TranscriptSegment bulk insert, raw JSON persistence, idempotent re-upload, and invalid input rejection. | Yes | agent |
| D004 |  | architecture | Sync vs async approach for Celery worker pipeline tasks | Use sync openai.OpenAI, sync QdrantClient, and sync SQLAlchemy (create_engine with psycopg2) inside Celery tasks. Convert DATABASE_URL from postgresql+asyncpg:// to postgresql:// for the sync engine. | Celery workers run in a synchronous context. Using asyncio.run() inside tasks risks nested event loop errors with gevent/eventlet workers. Using sync clients throughout eliminates this class of bug entirely. The async engine/session from database.py is only used by FastAPI (ASGI); the worker gets its own sync engine. | Yes | agent |
| D005 |  | architecture | Embedding/Qdrant failure handling strategy in pipeline | Embedding/Qdrant failures (stage 6) log errors but do not fail the pipeline. Processing_status is set by stages 2-5 only. Embeddings can be regenerated by manual re-trigger. | Qdrant is at 10.0.0.10 on the hypervisor network and may not be reachable during all pipeline runs. Making embedding a non-blocking side-effect ensures core pipeline output (KeyMoments, TechniquePages in PostgreSQL) is never lost due to vector store issues. The manual re-trigger endpoint allows regenerating embeddings at any time. | Yes | agent |
| D006 |  | requirement | R013 Prompt Template System status | validated | 4 prompt template files in prompts/ directory loaded from configurable settings.prompts_path. Templates use XML-style content fencing. Pipeline stages read templates from disk at runtime, enabling edits without code changes. Manual re-trigger endpoint (POST /api/v1/pipeline/trigger/{video_id}) allows re-processing after prompt edits. | Yes | agent |
| D007 | M001/S04 | architecture | Runtime review mode toggle persistence mechanism | Store review mode toggle in Redis key `chrysopedia:review_mode` with async redis client. Fall back to `settings.review_mode` config default when key is absent. | The config.py `review_mode` setting is loaded via lru_cache from environment variables and cannot be mutated at runtime. Redis is already used by the project (Celery broker, stage 4 classification data) so it adds no new infrastructure. A system_settings DB table would work but Redis is simpler for a single boolean toggle on a single-admin tool. The pipeline's stages.py reads settings.review_mode from config — the admin toggle only affects new pipeline runs if stages.py is updated to check Redis too, but that's deferred since the toggle is primarily a UI-level concept for the review queue. | Yes | agent |
| D008 | M001/S04 | requirement | R004 Review Queue UI status | validated | All R004 capabilities delivered and verified: 9 API endpoints (approve, reject, edit, split, merge, queue list, stats, mode get/set) with 24 passing integration tests covering happy paths and error boundaries. React+TypeScript frontend with queue page (filter tabs, stats, pagination), moment detail page (all review actions with modals), and review-vs-auto mode toggle. Frontend builds with zero TypeScript errors. | Yes | agent |
| D009 | M001/S05 | architecture | Async search service pattern for FastAPI request path | Create a separate `SearchService` class using `openai.AsyncOpenAI` and `qdrant_client.AsyncQdrantClient` for the search endpoint. Keep existing sync `EmbeddingClient` and `QdrantManager` for Celery pipeline. Search endpoint has 300ms timeout on embedding API and falls back to SQL ILIKE keyword search on Qdrant/embedding failure. | The existing EmbeddingClient and QdrantManager are sync (using `openai.OpenAI` and `QdrantClient`) because Celery tasks run synchronously. FastAPI request handlers are async — reusing sync clients would block the event loop. Creating a thin async wrapper avoids modifying the battle-tested pipeline code while providing non-blocking search. The 300ms timeout and keyword fallback ensure the search endpoint always returns results, even when Qdrant or the embedding service is degraded. | Yes | agent |
| D010 | M001/S05 | requirement | R005 Search-First Web UI status | validated | Search endpoint (GET /api/v1/search) with async embedding + Qdrant + keyword fallback implemented and tested. Frontend Home.tsx has prominent search bar with 300ms debounced typeahead, scope toggle via URL params, nav cards for Topics/Creators, Recently Added section. SearchResults.tsx displays grouped results. 5 integration tests verify search happy path, empty query, keyword fallback, scope filter, and no-results. Frontend production build succeeds with zero TypeScript errors. | Yes | agent |
| D011 | M001/S05 | requirement | R006 Technique Page Display status | validated | TechniquePage.tsx renders all specified sections: header with topic_category badge and topic_tags pills, creator name linked to creator detail, source_quality indicator, amber banner for unstructured content, body_sections JSONB prose (handles both string and object values), key moments index ordered by start_time, signal chains, plugins pill list, and related techniques links. Backend GET /api/v1/techniques/{slug} returns full detail with eager-loaded key_moments and related links. 404 for unknown slug tested. | Yes | agent |
| D012 | M001/S05 | requirement | R007 Creators Browse Page status | validated | CreatorsBrowse.tsx implements genre filter pills, type-to-narrow name filter, sort toggle (Random default/A-Z/Views). Each creator row shows name, genre tags, technique_count, video_count. Links to CreatorDetail page. Backend GET /api/v1/creators supports sort=random\|alpha\|views and genre filter. Integration tests verify random sort, alpha sort, genre filter, detail endpoint, 404, and counts. | Yes | agent |
| D013 | M001/S05 | requirement | R008 Topics Browse Page status | validated | TopicsBrowse.tsx renders two-level topic hierarchy (6 categories from canonical_tags.yaml with expandable sub-topics showing technique_count and creator_count). Filter input narrows categories/sub-topics. Clicking sub-topic navigates to search with scope=topics. Backend GET /api/v1/topics aggregates counts from DB per sub-topic. Integration test verifies topic hierarchy response shape. | Yes | agent |
| D014 | M001/S05 | requirement | R014 Creator Equity status | validated | CreatorsBrowse.tsx defaults to sort=random. Backend uses func.random() ORDER BY for randomized sort. Integration test verifies random sort returns all creators (order may vary). All creators get equal visual weight in the UI — no featured/highlighted treatment. Equal-weight row layout confirmed in CSS. | Yes | agent |
| D015 | M002/S01 | architecture | Docker network subnet for Chrysopedia on ub01 | 172.32.0.0/24 — free on ub01, replaces 172.24.0.0/24 which is used by xpltd_docs_default | 172.24.0.0/24 was already allocated to xpltd_docs_default network on ub01. Scanned all existing subnets and 172.32.0.0/24 was the first unoccupied /24 in the 172.x range. | Yes | agent |
| D016 | M002/S01 | architecture | Embedding service for the pipeline and search — OpenWebUI doesn't serve /v1/embeddings | Add Ollama container (ollama/ollama:latest) to the compose stack running nomic-embed-text on CPU | The FYN OpenWebUI instance at chat.forgetyour.name returns 404/500 on /v1/embeddings. No Ollama instance exists on the network. Ollama provides the standard OpenAI-compatible /v1/embeddings endpoint that both EmbeddingClient and SearchService expect. | Yes | agent |
| D017 |  | frontend | CSS theming approach for dark mode | 77 semantic CSS custom properties in :root block, all hardcoded colors replaced with var(--*) references | A complete variable-based palette enables theme switching, ensures consistency, and makes future color changes single-point edits. 77 tokens (vs ~30 initially planned) were needed to cover all semantic contexts — badges, buttons, surfaces, text, accents, shadows, and status states. Cyan (#22d3ee) replaces indigo as the accent color throughout. | Yes | agent |
| D018 | M004/S04 | architecture | Version snapshot failure handling strategy in stage 5 | Best-effort versioning — snapshot INSERT failure logs ERROR but does not block page update (existing content overwrites proceed regardless) | Versioning is a diagnostic/benchmarking feature. Losing a version snapshot is far less damaging than failing the entire page synthesis. Follows the same non-blocking side-effect pattern established in D005 for embedding/Qdrant failures. | Yes | agent |
| D019 | M005/S02 | frontend-layout | Technique page layout structure | CSS grid 2-column layout: 1fr main + 22rem sticky sidebar, collapsing to single column at 768px. Page max-width widened from 48rem to 64rem. | 22rem sidebar provides enough room for moment cards and plugin lists without cramming. 64rem total width accommodates both columns comfortably on standard desktop displays. Sticky sidebar keeps navigation aids visible while scrolling prose. 768px breakpoint aligns with existing mobile styles in the codebase. | Yes | agent |
| D020 | M006/S05 | frontend | Topics page card layout visual differentiation approach | 3px colored left border + small colored dot next to category name, using existing badge CSS custom properties per category | Subtler than a full colored header — maintains dark theme cohesion while providing clear visual differentiation between 7 category cards. Reuses existing --color-badge-cat-*-bg/text custom properties, avoiding new color definitions. | Yes | agent |
| D021 | M011 | scope | Which UI/UX assessment findings to implement in M011 | 12 of 16 findings approved; F01 (beginner paths), F02 (YouTube links), F03 (hide admin), F15 (CTA label) denied | User triaged each finding individually. Denied F01 because audience knows what they want. Denied F02 because no video URLs / don't want to link out. Denied F03 because admin dropdown is fine as-is. Denied F15 as low-value. | Yes | human |
| D022 |  | requirement | R025 | validated | useDocumentTitle hook called in all 10 pages. Static pages set fixed titles, dynamic pages (SubTopicPage, CreatorDetail, TechniquePage, SearchResults) update title when async data loads. | Yes | agent |
| D023 | M012/S01 | architecture | Qdrant embedding text enrichment strategy | Prepend creator_name and join topic_tags into embedding text for technique pages and key moments. Batch-resolve creator names at stage 6 start. | Semantic search now surfaces results for creator-name queries and tag-specific queries. Batch resolution avoids N+1 lookups during embedding. Reindex-all endpoint enables one-shot re-embedding after text composition changes. | Yes | agent |
| D024 | M014/S01 | architecture | Content model for sections with subsections | Sections with subsections use empty-string content field; substance lives in subsections | Avoids duplication between section-level content and subsection content. The section heading serves as H2 container; all prose lives in subsection content fields. Sections without subsections use the section content field directly. | Yes | agent |
| D025 | M015 | architecture | Search query storage and popular searches architecture | PostgreSQL search_log table + Redis read-through cache with 5-min TTL | PostgreSQL gives full historical data for future analytics (zero-result queries, time-of-day patterns). Redis cache prevents DB query on every homepage load. 5-min TTL balances freshness with load. Volume is tiny at current scale. | Yes | collaborative |
| D026 |  | requirement | R033 | validated | Creators browse page shows "Last updated: Apr 2/3" per creator with techniques, omits for 0-technique creators. Homepage recently-added cards show subtle date stamps. Both verified live on ub01:8096. | Yes | agent |
| D027 |  | requirement | R034 | validated | Homepage renders stats block with real counts from the API: GET /api/v1/stats returns {"technique_count":21,"creator_count":7}, and the frontend scorecard displays "21 ARTICLES" and "7 CREATORS" in cyan-on-dark design. Visual and API verification both pass. | Yes | agent |
| D028 |  | requirement | R036 | validated | AdminDropdown.tsx now opens on hover at desktop widths (≥769px) via matchMedia guard with 150ms leave delay, while mobile retains tap-to-toggle. Build passes. Satisfies R036 criteria. | Yes | agent |
| D029 |  | requirement | R039 | validated | Build output confirms favicon.svg, favicon-32.png, apple-touch-icon.png, og-image.png in dist/. index.html contains all required OG, Twitter card, and favicon meta tags. Inline SVG logo mark present in header. All slice plan verification checks pass. | Yes | agent |
