spine-framework-portal 0.2.21 → 0.2.22

package/components/KBGenerator.tsx

@@ -1,5 +1,4 @@
 import { useState } from 'react'
-import { getTypeIdAsync } from '../hooks/useTypeRegistry'
 import { apiFetch } from '@core/lib/api'
 import { Card, CardContent, CardHeader } from '@core/components/ui/card'
 import { Button } from '@core/components/ui/button'
@@ -40,13 +39,12 @@ export function KBGenerator({ ticket, onGenerated, onCancel }: KBGeneratorProps)
   const handleSave = async () => {
     if (!generatedArticle) return
     try {
-      const kbArticleTypeId = await getTypeIdAsync('kb_article')
       const res = await apiFetch('/.netlify/functions/admin-data', {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
         body: JSON.stringify({
           entity: 'items',
-          type_id: kbArticleTypeId,
+          type_slug: 'kb_article',
           title: generatedArticle.title,
           status: 'published',
           description: editedContent,

package/hooks/useCommunity.ts

@@ -1,6 +1,5 @@
 import { useState, useEffect, useCallback } from 'react'
 import { apiFetch } from '@core/lib/api'
-import { getTypeIdAsync } from './useTypeRegistry'
 
 export interface CommunityPost {
   id: string
@@ -59,12 +58,9 @@ export function useCreatePost() {
     setLoading(true)
     setError(null)
     try {
-      const typeId = await getTypeIdAsync('community_post')
-      if (!typeId) throw new Error('community_post type not found')
-      
       return await fetchJSON('/.netlify/functions/admin-data', {
         method: 'POST',
-        body: JSON.stringify({ entity: 'items', type_id: typeId, ...fields }),
+        body: JSON.stringify({ entity: 'items', type_slug: 'community_post', ...fields }),
       })
     } catch (e: any) {
       setError(e.message)

package/hooks/useItemProgress.ts

@@ -76,7 +76,7 @@ export function useUpsertProgress() {
   const upsert = useCallback(async (params: {
     personId: string
     itemId: string
-    typeId: string
+    typeSlug: string
     accountId: string
     appId?: string
     status?: string
@@ -92,7 +92,7 @@ export function useUpsertProgress() {
         body: JSON.stringify({
           person_id:  params.personId,
           item_id:    params.itemId,
-          type_id:    params.typeId,
+          type_slug:  params.typeSlug,
           account_id: params.accountId,
           app_id:     params.appId,
           status:     params.status,

package/hooks/usePortalThreads.ts

@@ -1,6 +1,5 @@
 import { useState, useEffect, useCallback } from 'react'
 import { apiFetch } from '@core/lib/api'
-import { getTypeIdAsync } from './useTypeRegistry'
 
 export interface PortalMessage {
   id: string
@@ -63,11 +62,8 @@ export function usePortalThread(targetType: string, targetId: string | null, dom
     setLoading(true)
     setError(null)
     try {
-      const threadTypeId = await getTypeIdAsync(threadTypeSlug)
-      if (!threadTypeId) throw new Error(`${threadTypeSlug} type not found`)
-
       const threads = await fetchJSON(
-        `/.netlify/functions/admin-data?entity=threads&target_type=${targetType}&target_id=${targetId}&type_id=${threadTypeId}`
+        `/.netlify/functions/admin-data?entity=threads&target_type=${targetType}&target_id=${targetId}&type_slug=${threadTypeSlug}`
       )
       const found: PortalThread | null = Array.isArray(threads) ? (threads[0] ?? null) : null
       setThread(found)
@@ -92,14 +88,11 @@ export function usePortalThread(targetType: string, targetId: string | null, dom
     let activeThread = thread
     
     if (!activeThread?.id) {
-      const threadTypeId = await getTypeIdAsync(threadTypeSlug)
-      if (!threadTypeId) throw new Error(`${threadTypeSlug} type not found`)
-      
       activeThread = await fetchJSON('/.netlify/functions/admin-data', {
         method: 'POST',
         body: JSON.stringify({
           entity: 'threads',
-          type_id: threadTypeId,
+          type_slug: threadTypeSlug,
           target_type: targetType,
           target_id: targetId,
           status: 'open',
@@ -108,9 +101,6 @@ export function usePortalThread(targetType: string, targetId: string | null, dom
       setThread(activeThread)
     }
     
-    const messageTypeId = await getTypeIdAsync(messageTypeSlug)
-    if (!messageTypeId) throw new Error(`${messageTypeSlug} type not found`)
-    
     // Calculate next sequence based on current messages to avoid stale state
     const nextSequence = messages.reduce((max, m) => Math.max(max, m.sequence || 0), 0) + 1
     
@@ -118,7 +108,7 @@ export function usePortalThread(targetType: string, targetId: string | null, dom
       method: 'POST',
       body: JSON.stringify({
         entity: 'messages',
-        type_id: messageTypeId,
+        type_slug: messageTypeSlug,
         thread_id: activeThread!.id,
         content,
         direction: 'inbound',

package/hooks/useTickets.ts

@@ -1,6 +1,5 @@
 import { useState, useEffect, useCallback } from 'react'
 import { apiFetch } from '@core/lib/api'
-import { getTypeIdAsync } from './useTypeRegistry'
 
 export interface Ticket {
   id: string
@@ -77,12 +76,9 @@ export function useCreateTicket() {
     setLoading(true)
     setError(null)
     try {
-      const typeId = await getTypeIdAsync('support_ticket')
-      if (!typeId) throw new Error('support_ticket type not found')
-      
       return await fetchJSON('/.netlify/functions/admin-data', {
         method: 'POST',
-        body: JSON.stringify({ entity: 'items', type_id: typeId, ...fields }),
+        body: JSON.stringify({ entity: 'items', type_slug: 'support_ticket', ...fields }),
       })
     } catch (e: any) {
       setError(e.message)

package/manifest.json

@@ -2,7 +2,7 @@
   "name": "Portal",
   "slug": "portal",
   "description": "Self-service portal for customers to access tickets, knowledge base, courses, and community",
-  "version": "0.2.21",
+  "version": "0.2.22",
   "app_type": "full",
   "route_prefix": "/portal",
   "required_roles": ["member", "member_admin"],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spine-framework-portal",
-  "version": "0.2.21",
+  "version": "0.2.22",
   "private": false,
   "description": "Customer Portal — self-service portal app for Spine Framework",
   "type": "module",

package/pages/CoursesPage.tsx

@@ -1,10 +1,9 @@
-import { useState, useEffect, useRef } from 'react'
+import { useState, useEffect } from 'react'
 import { CheckCircle, Circle, PlayCircle, GraduationCap, Send } from 'lucide-react'
 import { useCourseLessons, type CourseItem } from '../hooks/useCourses'
 import { usePortalSignal } from '../hooks/usePortalSignal'
 import { useItemProgress, useUpsertProgress } from '../hooks/useItemProgress'
 import { usePortalThread } from '../hooks/usePortalThreads'
-import { getTypeIdAsync } from '../hooks/useTypeRegistry'
 import { useAuth } from '@core/contexts/AuthContext'
 import { SearchFilterBar } from '../components/SearchFilterBar'
 import { Button } from '@core/components/ui/button'
@@ -42,7 +41,6 @@ export function CoursesPage() {
   const [selectedId, setSelectedId] = useState<string | null>(() => localStorage.getItem(LAST_LESSON_KEY))
   const [replyText, setReplyText] = useState('')
   const [replying, setReplying] = useState(false)
-  const progressTypeIdRef = useRef<string | null>(null)
 
   const { lessons, loading, error } = useCourseLessons()
   const lessonIds = lessons.map((l) => l.id)
@@ -60,10 +58,6 @@ export function CoursesPage() {
     }
   }, [lessons, selectedId])
 
-  useEffect(() => {
-    getTypeIdAsync(PROGRESS_TYPE_SLUG).then((id) => { progressTypeIdRef.current = id })
-  }, [])
-
   const courseMap = groupByCourse(lessons)
   const courseNames = Array.from(courseMap.keys())
   const filteredCourses = courseNames.filter((c) => c.toLowerCase().includes(search.toLowerCase()))
@@ -75,11 +69,10 @@ export function CoursesPage() {
   const handleSelectLesson = async (lessonId: string) => {
     setSelectedId(lessonId)
     localStorage.setItem(LAST_LESSON_KEY, lessonId)
-    const typeId = progressTypeIdRef.current
-    if (!typeId || !user?.id || !user?.account_id) return
+    if (!user?.id || !user?.account_id) return
     const existing = progressMap.get(lessonId)
     if (!existing || existing.status === 'not_started') {
-      await upsert({ personId: user.id, itemId: lessonId, typeId, accountId: user.account_id, status: 'in_progress' })
+      await upsert({ personId: user.id, itemId: lessonId, typeSlug: PROGRESS_TYPE_SLUG, accountId: user.account_id, status: 'in_progress' })
       refetchProgress()
       sendSignal('lesson_start', 'Started a course lesson')
     }
@@ -87,9 +80,7 @@ export function CoursesPage() {
 
   const handleComplete = async () => {
     if (!selectedId || !user?.id || !user?.account_id) return
-    const typeId = progressTypeIdRef.current
-    if (!typeId) return
-    await upsert({ personId: user.id, itemId: selectedId, typeId, accountId: user.account_id, status: 'completed' })
+    await upsert({ personId: user.id, itemId: selectedId, typeSlug: PROGRESS_TYPE_SLUG, accountId: user.account_id, status: 'completed' })
     refetchProgress()
     sendSignal('lesson_complete', 'Completed a course lesson')
   }

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
-}