Fix: Ultimate governance resolution via Database Views (Phase 8)

database/FINAL_SUPABASE_FIX.sql

@@ -1,42 +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.audit_logs DROP CONSTRAINT IF EXISTS audit_logs_user_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
+-- FINAL ROBUST FIX: DATABASE VIEWS FOR GOVERNANCE (Phase 8)
+-- RUN THIS IN SUPABASE SQL EDITOR TO BYPASS RELATIONSHIP DISCOVERY ISSUES
+
+-- 1. TEAM MEMBERS VIEW
+CREATE OR REPLACE VIEW public.team_members_with_profiles AS
+SELECT 
+    tm.id,
+    tm.team_id,
+    tm.user_id,
+    tm.role,
+    tm.created_at,
+    p.full_name,
+    p.email
+FROM public.team_members tm
+LEFT JOIN public.profiles p ON tm.user_id = p.id;
+
+GRANT SELECT ON public.team_members_with_profiles TO authenticated;
+
+-- 2. AUDIT LOGS VIEW
+CREATE OR REPLACE VIEW public.audit_logs_with_details AS
+SELECT 
+    al.id,
+    al.user_id,
+    al.action,
+    al.agent_id,
+    al.task_id,
+    al.metadata,
+    al.created_at,
+    p.full_name AS actor_name,
+    p.email AS actor_email,
+    ag.name AS agent_name,
+    t.title AS task_title
+FROM public.audit_logs al
+LEFT JOIN public.profiles p ON al.user_id = p.id
+LEFT JOIN public.agents ag ON al.agent_id = ag.id
+LEFT JOIN public.tasks t ON al.task_id = t.id;
+
+GRANT SELECT ON public.audit_logs_with_details TO authenticated;
+
+-- 3. RE-SYNC SCHEMA
 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

@@ -52,13 +52,8 @@ const AuditView: React.FC = () => {
     setLoading(true);
     try {
       const { data, error: fetchError } = await supabase
-        .from('audit_logs')
-        .select(`
-          *,
-          profiles!user_id(full_name, email),
-          agents!agent_id(name),
-          tasks!task_id(title)
-        `)
+        .from('audit_logs_with_details')
+        .select('*')
         .order('created_at', { ascending: false })
         .range(page * pageSize, (page + 1) * pageSize - 1);
 
@@ -76,9 +71,9 @@ const AuditView: React.FC = () => {
     const rows = logs.map(log => [
       log.created_at,
       log.action,
-      (log as any).profiles?.email || 'System',
-      (log as any).agents?.name || 'N/A',
-      (log as any).tasks?.title || 'N/A',
+      (log as any).actor_email || 'System',
+      (log as any).agent_name || 'N/A',
+      (log as any).task_title || 'N/A',
       JSON.stringify(log.metadata)
     ]);
     
@@ -93,8 +88,8 @@ const AuditView: React.FC = () => {
 
   const filteredLogs = logs.filter(log => 
     log.action.toLowerCase().includes(searchTerm.toLowerCase()) ||
-    ((log as any).profiles?.email || '').toLowerCase().includes(searchTerm.toLowerCase()) ||
-    ((log as any).agents?.name || '').toLowerCase().includes(searchTerm.toLowerCase())
+    ((log as any).actor_email || '').toLowerCase().includes(searchTerm.toLowerCase()) ||
+    ((log as any).agent_name || '').toLowerCase().includes(searchTerm.toLowerCase())
   );
 
   const formatTimestamp = (ts: string) => {
@@ -191,21 +186,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 as any).profiles?.email || <span style={{ color: 'var(--text-dim)', fontSize: '0.8rem' }}>System</span>}
+                        {(log as any).actor_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 as any).agents && (
+                        {log.agent_id && (
                           <div style={{ display: 'flex', alignItems: 'center', gap: '4px', color: 'var(--accent)', fontSize: '0.8rem' }}>
                             <Bot size={12} />
-                            {(log as any).agents.name}
+                            {(log as any).agent_name}
                           </div>
                         )}
-                        {log.task_id && (log as any).tasks && (
+                        {log.task_id && (
                           <div style={{ display: 'flex', alignItems: 'center', gap: '4px', color: 'var(--text-dim)', fontSize: '0.8rem' }}>
                             <FileText size={12} />
-                            {(log as any).tasks.title}
+                            {(log as any).task_title}
                           </div>
                         )}
                         {!log.agent_id && !log.task_id && <span style={{ color: 'var(--text-dim)', fontSize: '0.8rem' }}>-</span>}

frontend/src/components/TeamsView.tsx

@@ -85,19 +85,16 @@ const TeamsView: React.FC = () => {
   const fetchMembers = async (teamId: string) => {
     try {
       const { data, error } = await supabase
-        .from('team_members')
-        .select(`
-          *,
-          profiles(full_name, email)
-        `)
+        .from('team_members_with_profiles')
+        .select('*')
         .eq('team_id', teamId);
 
       if (error) throw error;
 
       const formattedMembers = data.map(m => ({
         ...m,
-        full_name: m.profiles?.full_name,
-        email: m.profiles?.email
+        full_name: m.full_name,
+        email: m.email
       }));
 
       setMembers(formattedMembers);