Fix: Final column naming and relationship corrections (Phase 8 Governance)

database/FINAL_SUPABASE_FIX.sql

@@ -0,0 +1,41 @@
+-- FINAL GOVERNANCE & RELATIONSHIP FIX (Phase 8)
+-- RUN THIS IN SUPABASE SQL EDITOR TO RESOLVE 400 BAD REQUEST ERRORS
+
+-- 1. CLEANUP OLD CONSTRAINTS
+ALTER TABLE IF EXISTS public.team_members DROP CONSTRAINT IF EXISTS team_members_user_id_fkey;
+ALTER TABLE IF EXISTS public.projects DROP CONSTRAINT IF EXISTS projects_owner_id_fkey;
+ALTER TABLE IF EXISTS public.agents DROP CONSTRAINT IF EXISTS agents_user_id_fkey;
+ALTER TABLE IF EXISTS public.audit_logs DROP CONSTRAINT IF EXISTS audit_logs_actor_id_fkey;
+ALTER TABLE IF EXISTS public.agent_templates DROP CONSTRAINT IF EXISTS agent_templates_author_id_fkey;
+
+-- 2. CREATE NEW ROBUST RELATIONSHIPS POINTING TO PUBLIC.PROFILES
+-- This is critical for PostgREST join discovery.
+ALTER TABLE public.team_members 
+ADD CONSTRAINT team_members_user_id_fkey 
+FOREIGN KEY (user_id) REFERENCES public.profiles(id) ON DELETE CASCADE;
+
+ALTER TABLE public.projects 
+ADD CONSTRAINT projects_owner_id_fkey 
+FOREIGN KEY (owner_id) REFERENCES public.profiles(id) ON DELETE SET NULL;
+
+ALTER TABLE public.agents 
+ADD CONSTRAINT agents_user_id_fkey 
+FOREIGN KEY (user_id) REFERENCES public.profiles(id) ON DELETE CASCADE;
+
+ALTER TABLE public.audit_logs 
+ADD CONSTRAINT audit_logs_user_id_fkey 
+FOREIGN KEY (user_id) REFERENCES public.profiles(id) ON DELETE SET NULL;
+
+ALTER TABLE public.agent_templates 
+ADD CONSTRAINT agent_templates_author_id_fkey 
+FOREIGN KEY (author_id) REFERENCES public.profiles(id) ON DELETE SET NULL;
+
+-- 3. ENSURE GRANTS
+GRANT SELECT ON ALL TABLES IN SCHEMA public TO authenticated;
+GRANT USAGE ON SCHEMA public TO authenticated;
+
+-- 4. RELOAD POSTGREST SCHEMA CACHE
+NOTIFY pgrst, 'reload schema';
+
+-- 5. VERIFICATION COMMENT (For PostgREST hinting)
+COMMENT ON CONSTRAINT team_members_user_id_fkey ON public.team_members IS 'Relationship to profiles table for joins';

frontend/src/components/AuditView.tsx

@@ -18,7 +18,7 @@ import { supabase } from '../services/supabase';
 interface AuditLog {
   id: string;
   created_at: string;
-  actor_id: string | null;
+  user_id: string | null;
   action: string;
   agent_id: string | null;
   task_id: string | null;
@@ -55,9 +55,9 @@ const AuditView: React.FC = () => {
         .from('audit_logs')
         .select(`
           *,
-          profiles(full_name, email),
-          agents(name),
-          tasks(title)
+          profiles!user_id(full_name, email),
+          agents!agent_id(name),
+          tasks!task_id(title)
         `)
         .order('created_at', { ascending: false })
         .range(page * pageSize, (page + 1) * pageSize - 1);
@@ -76,9 +76,9 @@ const AuditView: React.FC = () => {
     const rows = logs.map(log => [
       log.created_at,
       log.action,
-      log.profiles?.email || 'System',
-      log.agents?.name || 'N/A',
-      log.tasks?.title || 'N/A',
+      (log as any).profiles?.email || 'System',
+      (log as any).agents?.name || 'N/A',
+      (log as any).tasks?.title || 'N/A',
       JSON.stringify(log.metadata)
     ]);
     
@@ -93,8 +93,8 @@ const AuditView: React.FC = () => {
 
   const filteredLogs = logs.filter(log => 
     log.action.toLowerCase().includes(searchTerm.toLowerCase()) ||
-    (log.profiles?.email || '').toLowerCase().includes(searchTerm.toLowerCase()) ||
-    (log.agents?.name || '').toLowerCase().includes(searchTerm.toLowerCase())
+    ((log as any).profiles?.email || '').toLowerCase().includes(searchTerm.toLowerCase()) ||
+    ((log as any).agents?.name || '').toLowerCase().includes(searchTerm.toLowerCase())
   );
 
   const formatTimestamp = (ts: string) => {
@@ -191,21 +191,21 @@ const AuditView: React.FC = () => {
                     <td style={{ padding: 'var(--space-md)', fontSize: '0.9rem' }}>
                       <div style={{ display: 'flex', alignItems: 'center', gap: '8px' }}>
                         <User size={14} style={{ opacity: 0.5 }} />
-                        {log.profiles?.email || <span style={{ color: 'var(--text-dim)', fontSize: '0.8rem' }}>System</span>}
+                        {(log as any).profiles?.email || <span style={{ color: 'var(--text-dim)', fontSize: '0.8rem' }}>System</span>}
                       </div>
                     </td>
                     <td style={{ padding: 'var(--space-md)', fontSize: '0.9rem' }}>
                       <div style={{ display: 'flex', flexDirection: 'column', gap: '4px' }}>
-                        {log.agent_id && log.agents && (
+                        {log.agent_id && (log as any).agents && (
                           <div style={{ display: 'flex', alignItems: 'center', gap: '4px', color: 'var(--accent)', fontSize: '0.8rem' }}>
                             <Bot size={12} />
-                            {log.agents.name}
+                            {(log as any).agents.name}
                           </div>
                         )}
-                        {log.task_id && log.tasks && (
+                        {log.task_id && (log as any).tasks && (
                           <div style={{ display: 'flex', alignItems: 'center', gap: '4px', color: 'var(--text-dim)', fontSize: '0.8rem' }}>
                             <FileText size={12} />
-                            {log.tasks.title}
+                            {(log as any).tasks.title}
                           </div>
                         )}
                         {!log.agent_id && !log.task_id && <span style={{ color: 'var(--text-dim)', fontSize: '0.8rem' }}>-</span>}