spine-framework-cortex 0.2.25 → 0.2.26

package/manifest.json

@@ -2,7 +2,7 @@
   "name": "Cortex",
   "slug": "cortex",
   "description": "Unified workspace for CRM, Support, Community, and Knowledge Base",
-  "version": "0.2.25",
+  "version": "0.2.26",
   "app_type": "full",
   "route_prefix": "/cortex",
   "required_roles": ["support", "support_admin"],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spine-framework-cortex",
-  "version": "0.2.25",
+  "version": "0.2.26",
   "private": false,
   "description": "Cortex — AI-powered support, CRM, and knowledge base app for Spine Framework",
   "keywords": [

package/pages/community/CommunityPage.tsx

@@ -1,7 +1,6 @@
 import { useState, useEffect } from 'react'
 import { Send, Hash, MessageSquarePlus, AlertCircle } from 'lucide-react'
 import { apiFetch } from '@core/lib/api'
-import { getTypeIdAsync } from '../../hooks/useTypeRegistry'
 import { Button } from '@core/components/ui/button'
 import { Input } from '@core/components/ui/input'
 import { Badge } from '@core/components/ui/badge'
@@ -21,28 +20,20 @@ function ThreadPane({ post }: { post: Post }) {
   const [loading, setLoading] = useState(true)
   const [replyText, setReplyText] = useState('')
   const [sending, setSending] = useState(false)
-  const [threadTypeId, setThreadTypeId] = useState<string | null>(null)
-  const [messageTypeId, setMessageTypeId] = useState<string | null>(null)
 
   useEffect(() => {
-    getTypeIdAsync('community_thread').then(id => setThreadTypeId(id))
-    getTypeIdAsync('community_reply').then(id => setMessageTypeId(id))
-  }, [])
-
-  useEffect(() => {
-    if (!threadTypeId) return
     setLoading(true)
-    apiFetch(`/api/admin-data?action=list&entity=threads&target_id=${post.id}&type_id=${threadTypeId}&limit=5`)
+    apiFetch(`/api/admin-data?action=list&entity=threads&target_id=${post.id}&type_slug=community_thread&limit=5`)
       .then(r => r.json()).then(j => {
         const t = (j?.data ?? j)?.[0] ?? null
         setThread(t)
         if (t) return apiFetch(`/api/admin-data?action=list&entity=messages&thread_id=${t.id}&limit=100`).then(r => r.json())
         return []
       }).then(raw => { const msgs = raw?.data ?? raw; setMessages(Array.isArray(msgs) ? msgs : []) }).catch(() => {}).finally(() => setLoading(false))
-  }, [post.id, threadTypeId])
+  }, [post.id])
 
   const handleSend = async () => {
-    if (!replyText.trim() || !messageTypeId || !threadTypeId) return
+    if (!replyText.trim()) return
     setSending(true)
     try {
       let activeThread = thread
@@ -51,7 +42,7 @@ function ThreadPane({ post }: { post: Post }) {
           method: 'POST', headers: { 'Content-Type': 'application/json' },
           body: JSON.stringify({
             entity: 'threads',
-            type_id: threadTypeId,
+            type_slug: 'community_thread',
             target_type: 'items',
             target_id: post.id,
             status: 'open',
@@ -67,7 +58,7 @@ function ThreadPane({ post }: { post: Post }) {
         method: 'POST', headers: { 'Content-Type': 'application/json' },
         body: JSON.stringify({
           thread_id: activeThread.id,
-          type_id: messageTypeId,
+          type_slug: 'community_reply',
           content: replyText.trim(),
           direction: 'outbound',
           sequence: messages.reduce((max, m) => Math.max(max, m.sequence || 0), 0) + 1,
@@ -101,7 +92,7 @@ function ThreadPane({ post }: { post: Post }) {
       <div className="border-t p-3 flex gap-2 shrink-0">
         <Input placeholder="Reply as agent…" value={replyText} onChange={e => setReplyText(e.target.value)}
           onKeyDown={e => e.key === 'Enter' && !e.shiftKey && handleSend()} className="flex-1" />
-        <Button size="icon" onClick={handleSend} disabled={sending || !replyText.trim() || !thread || !messageTypeId}><Send size={14} /></Button>
+        <Button size="icon" onClick={handleSend} disabled={sending || !replyText.trim() || !thread}><Send size={14} /></Button>
       </div>
     </div>
   )

package/pages/courses/CoursesPage.tsx

@@ -1,7 +1,6 @@
-import { useState, useEffect, useRef } from 'react'
+import { useState, useEffect } from 'react'
 import { useNavigate } from 'react-router-dom'
 import { apiFetch } from '@core/lib/api'
-import { getTypeIdAsync } from '../../hooks/useTypeRegistry'
 import { Button } from '@core/components/ui/button'
 import { Input } from '@core/components/ui/input'
 import { Badge } from '@core/components/ui/badge'
@@ -31,13 +30,6 @@ function DiscussionPanel({ lesson }: { lesson: Lesson }) {
   const [loading, setLoading] = useState(true)
   const [newMessage, setNewMessage] = useState('')
   const [submitting, setSubmitting] = useState(false)
-  const messageTypeIdRef = useRef<string>('')
-
-  useEffect(() => {
-    getTypeIdAsync('message').then(id => {
-      if (id) messageTypeIdRef.current = id
-    })
-  }, [])
 
   useEffect(() => {
     setLoading(true)
@@ -53,10 +45,6 @@ function DiscussionPanel({ lesson }: { lesson: Lesson }) {
   const handleSubmit = async (e: React.FormEvent) => {
     e.preventDefault()
     if (!newMessage.trim() || submitting) return
-    if (!messageTypeIdRef.current) {
-      console.error('Message type not loaded')
-      return
-    }
 
     setSubmitting(true)
     try {
@@ -83,7 +71,7 @@ function DiscussionPanel({ lesson }: { lesson: Lesson }) {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
         body: JSON.stringify({
-          type_id: messageTypeIdRef.current,
+          type_slug: 'message',
           thread_id: currentThread.id,
           content: newMessage.trim(),
           direction: 'inbound',

package/pages/kb/KBEditorPage.tsx

@@ -2,7 +2,6 @@ import { useState, useEffect } from 'react'
 import { useParams, useNavigate } from 'react-router-dom'
 import { useAppPath } from '@core/hooks/useAppPath'
 import { apiFetch } from '@core/lib/api'
-import { getTypeIdAsync } from '../../hooks/useTypeRegistry'
 import { RichTextEditor } from '@core/components/ui/RichTextEditor'
 import { Button } from '@core/components/ui/button'
 import { Input } from '@core/components/ui/input'
@@ -73,7 +72,6 @@ export default function KBEditorPage() {
   const [loading, setLoading] = useState(!isNew)
   const [saving, setSaving] = useState(false)
   const [error, setError] = useState<string | null>(null)
-  const [kbArticleTypeId, setKbArticleTypeId] = useState<string>('')
 
   useEffect(() => {
     if (isNew) return
@@ -97,23 +95,16 @@ export default function KBEditorPage() {
       .finally(() => setLoading(false))
   }, [id, isNew])
 
-  useEffect(() => {
-    getTypeIdAsync('kb_article').then(id => {
-      if (id) setKbArticleTypeId(id)
-    })
-  }, [])
-
   const handleSave = async (publish?: boolean) => {
     if (!form.title.trim()) { setError('Title is required'); return }
     if (!form.kb_type) { setError('KB Type is required'); return }
-    if (!kbArticleTypeId) { setError('Type not loaded'); return }
     setSaving(true)
     setError(null)
     try {
       const payload = {
         title: form.title.trim(),
         description: form.description,
-        type_id: kbArticleTypeId,
+        type_slug: 'kb_article',
         status: publish ? 'published' : form.status,
         data: {
           kb_type: form.kb_type,

package/pages/support/RedactionReview.tsx

@@ -2,7 +2,6 @@ import { useState, useEffect, useCallback } from 'react'
 import { useParams, useNavigate } from 'react-router-dom'
 import { useAppPath } from '@core/hooks/useAppPath'
 import { apiFetch } from '@core/lib/api'
-import { getTypeIdAsync } from '../../hooks/useTypeRegistry'
 import { Button } from '@core/components/ui/button'
 import { Badge } from '@core/components/ui/badge'
 import { ScrollArea } from '@core/components/ui/scroll-area'
@@ -107,7 +106,6 @@ export default function RedactionReview() {
   const [finalContent, setFinalContent] = useState('')
   const [articleTitle, setArticleTitle] = useState('')
   const [viewMode, setViewMode] = useState<'review' | 'final'>('review')
-  const [kbArticleTypeId, setKbArticleTypeId] = useState<string>('')
 
   // Load ticket and trigger redaction analysis
   useEffect(() => {
@@ -155,11 +153,6 @@ export default function RedactionReview() {
     }
 
     loadAndAnalyze()
-
-    // Load KB article type ID
-    getTypeIdAsync('kb_article').then(id => {
-      if (id) setKbArticleTypeId(id)
-    })
   }, [id])
 
   // Build content from ticket for redaction analysis.
@@ -264,10 +257,6 @@ export default function RedactionReview() {
   // Publish KB article
   const publishKB = async () => {
     if (!id || !finalContent || !articleTitle) return
-    if (!kbArticleTypeId) {
-      console.error('KB article type not loaded')
-      return
-    }
 
     setProcessing(true)
     try {
@@ -276,7 +265,7 @@ export default function RedactionReview() {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
         body: JSON.stringify({
-          type_id: kbArticleTypeId,
+          type_slug: 'kb_article',
           title: articleTitle,
           status: 'published',
           description: finalContent,

package/pages/support/TicketDetailPage.tsx

@@ -1,4 +1,4 @@
-import { useEffect, useRef, useState } from 'react'
+import { useCallback, useEffect, useRef, useState } from 'react'
 import { useParams, useNavigate } from 'react-router-dom'
 import { useAppPath } from '@core/hooks/useAppPath'
 import { apiFetch } from '@core/lib/api'
@@ -11,7 +11,6 @@ import { Tabs, TabsList, TabsTrigger, TabsContent } from '@core/components/ui/ta
 import { Textarea } from '@core/components/ui/textarea'
 import { ArrowLeft, Send, Lock, Globe, Bot, CheckCircle, AlertCircle, FileText, Building2, CreditCard, Package, BookOpen, Eye, EyeOff, Brain, Clock, TrendingUp, Tag as TagIcon, Sparkles } from 'lucide-react'
 import { useAuth } from '@core/contexts/AuthContext'
-import { getTypeIdAsync } from '../../hooks/useTypeRegistry'
 
 interface Ticket { 
   id: string; 
@@ -83,14 +82,12 @@ function MergedThreadPanel({
   ticketId,
   externalThread,
   internalThread,
-  threadTypeId,
-  messageTypeId,
+  refreshThreads,
 }: {
   ticketId: string
   externalThread: Thread | null
   internalThread: Thread | null
-  threadTypeId: string | null
-  messageTypeId: string | null
+  refreshThreads: () => Promise<void>
 }) {
   const [messages, setMessages] = useState<Message[]>([])
   const [loading, setLoading] = useState(true)
@@ -137,32 +134,24 @@ function MergedThreadPanel({
     try {
       // Create internal thread if it doesn't exist
       if (!targetThread && replyType === 'internal') {
-        if (!threadTypeId) {
-          console.error('support_thread type not resolved')
-          return
-        }
-
         const threadRes = await apiFetch('/api/admin-data?action=create&entity=threads', {
           method: 'POST', headers: { 'Content-Type': 'application/json' },
           body: JSON.stringify({
             target_type: 'item',
             target_id: ticketId,
             visibility: 'internal',
-            type_id: threadTypeId
+            type_slug: 'support_thread'
           }),
         })
         const newThread = await threadRes.json()
         if (newThread?.id) {
           targetThread = newThread
-          // Refresh threads list
-          const thr = await apiFetch(`/api/admin-data?action=list&entity=threads&target_id=${ticketId}&type_id=${threadTypeId}&limit=10`).then(r => r.json())
-          const threadList = thr?.data ?? thr
-          setThreads(Array.isArray(threadList) ? threadList : [])
+          await refreshThreads()
         }
       }
 
-      if (!targetThread || !messageTypeId) {
-        console.error('No thread or message type available for message')
+      if (!targetThread) {
+        console.error('No thread available for message')
         return
       }
 
@@ -177,7 +166,7 @@ function MergedThreadPanel({
           thread_id: targetThread.id,
           content: reply.trim(),
           direction: 'outbound',
-          type_id: messageTypeId,
+          type_slug: 'support_reply',
           sequence: nextSeq,
           visibility: replyType
         }),
@@ -734,25 +723,25 @@ export default function TicketDetailPage() {
   const [watcherId, setWatcherId] = useState<string | null>(null)
   const [watchLoading, setWatchLoading] = useState(false)
   const [analysisLoading, setAnalysisLoading] = useState(false)
-  const [threadTypeId, setThreadTypeId] = useState<string | null>(null)
-  const [messageTypeId, setMessageTypeId] = useState<string | null>(null)
-
-  useEffect(() => {
-    getTypeIdAsync('support_thread').then(id => setThreadTypeId(id))
-    getTypeIdAsync('support_reply').then(id => setMessageTypeId(id))
-  }, [])
 
-  useEffect(() => {
-    if (!id || !threadTypeId) return
-    Promise.all([
-      apiFetch(`/api/admin-data?action=get&entity=items&id=${id}`).then(r => r.json()),
-      apiFetch(`/api/admin-data?action=list&entity=threads&target_id=${id}&type_id=${threadTypeId}&limit=10`).then(r => r.json()),
-    ]).then(([ir, thr]) => {
+  const loadTicketAndThreads = useCallback(async () => {
+    if (!id) return
+    setLoading(true)
+    try {
+      const [ir, thr] = await Promise.all([
+        apiFetch(`/api/admin-data?action=get&entity=items&id=${id}`).then(r => r.json()),
+        apiFetch(`/api/admin-data?action=list&entity=threads&target_id=${id}&type_slug=support_thread&limit=10`).then(r => r.json()),
+      ])
       setTicket(ir?.data ?? ir ?? null)
       const threadList = thr?.data ?? thr
       setThreads(Array.isArray(threadList) ? threadList : [])
-    }).catch(() => {}).finally(() => setLoading(false))
-  }, [id, threadTypeId])
+    } catch {
+    } finally {
+      setLoading(false)
+    }
+  }, [id])
+
+  useEffect(() => { loadTicketAndThreads() }, [loadTicketAndThreads])
 
   // Check if current user is watching this ticket
   useEffect(() => {
@@ -778,16 +767,12 @@ export default function TicketDetailPage() {
         setIsWatching(false)
         setWatcherId(null)
       } else {
-        // Resolve watcher type_id via types API
-        const typeRes = await apiFetch('/api/types?kind=watcher&limit=1').then(r => r.json())
-        const types = Array.isArray(typeRes?.data) ? typeRes.data : Array.isArray(typeRes) ? typeRes : []
-        if (!types.length) { console.error('No watcher type found'); return }
         const res = await apiFetch('/api/admin-data?action=create&entity=watchers', {
           method: 'POST',
           headers: { 'Content-Type': 'application/json' },
           body: JSON.stringify({
             entity: 'watchers',
-            type_id: types[0].id,
+            type_slug: 'watcher',
             target_type: 'item',
             target_id: id,
             person_id: user.id,
@@ -893,8 +878,7 @@ export default function TicketDetailPage() {
             ticketId={id || ''}
             externalThread={externalThread}
             internalThread={internalThread}
-            threadTypeId={threadTypeId}
-            messageTypeId={messageTypeId}
+            refreshThreads={loadTicketAndThreads}
           />
         </div>
 

package/hooks/useTypeRegistry.ts

@@ -1,74 +0,0 @@
-import { useState, useEffect } from 'react'
-import { supabase } from '@core/lib/supabase'
-
-interface TypeRecord {
-  id: string
-  slug: string
-  name: string
-  description?: string
-}
-
-// Module-level cache - persists across renders, shared across hooks
-let typeCache: Map<string, TypeRecord> | null = null
-let loadPromise: Promise<Map<string, TypeRecord>> | null = null
-
-async function fetchTypes(): Promise<Map<string, TypeRecord>> {
-  if (typeCache) return typeCache
-  if (loadPromise) return loadPromise
-
-  loadPromise = (async () => {
-    try {
-      // Query Supabase directly - types is a config table, not exposed via admin-data
-      const { data: types, error } = await supabase
-        .from('types')
-        .select('id, slug, name, description')
-        .eq('is_active', true)
-        .limit(100)
-
-      if (error) throw error
-
-      typeCache = new Map((types || []).map(t => [t.slug, t]))
-      return typeCache
-    } catch (e) {
-      console.error('Failed to load types:', e)
-      typeCache = new Map() // Empty cache on error
-      return typeCache
-    } finally {
-      loadPromise = null
-    }
-  })()
-
-  return loadPromise
-}
-
-export function useTypeRegistry() {
-  const [types, setTypes] = useState<Map<string, TypeRecord>>(typeCache || new Map())
-  const [loading, setLoading] = useState(!typeCache)
-  const [error, setError] = useState<string | null>(null)
-
-  useEffect(() => {
-    if (typeCache) return
-
-    fetchTypes()
-      .then(cache => {
-        setTypes(cache)
-        setLoading(false)
-      })
-      .catch(e => {
-        setError(e.message)
-        setLoading(false)
-      })
-  }, [])
-
-  const getTypeId = (slug: string): string | null => {
-    return types.get(slug)?.id || null
-  }
-
-  return { types, loading, error, getTypeId }
-}
-
-// Synchronous version for use inside async functions
-export async function getTypeIdAsync(slug: string): Promise<string | null> {
-  const cache = await fetchTypes()
-  return cache.get(slug)?.id || null
-}