jat-feedback 3.2.0 → 3.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jat-feedback",
-  "version": "3.2.0",
+  "version": "3.3.0",
   "description": "Embeddable feedback widget for bug reports and feature requests. Captures screenshots, console logs, and user context as a web component.",
   "type": "module",
   "main": "dist/jat-feedback.js",

package/supabase/functions/jat-webhook/index.ts

@@ -82,10 +82,8 @@ interface WebhookPayload {
 
 // Tables and the columns that JAT is allowed to update.
 // Add more tables here if you want JAT to sync status for other record types.
-// "feedback_reports" kept as alias for backward compat with pre-v3 callers.
 const TABLE_CONFIG: Record<string, { statusCol: string; taskIdCol: string }> = {
   project_tasks: { statusCol: "status", taskIdCol: "jat_task_id" },
-  feedback_reports: { statusCol: "status", taskIdCol: "jat_task_id" },
 }
 
 Deno.serve(async (req) => {
@@ -139,8 +137,14 @@ Deno.serve(async (req) => {
     )
   }
 
-  // Resolve actual table name — "feedback_reports" alias maps to "project_tasks"
-  const actualTable = reference_table === "feedback_reports" ? "project_tasks" : reference_table
+  // Skip transient voice-pipeline statuses — only sync when the task
+  // has a meaningful status that JAT should know about.
+  if (data.status === "transcribing" || data.status === "failed") {
+    return new Response(
+      JSON.stringify({ skipped: true, reason: `Status "${data.status}" is transient, not synced to JAT` }),
+      { status: 200, headers: { "Content-Type": "application/json" } },
+    )
+  }
 
   // Build the update object from the payload fields
   const update: Record<string, unknown> = {}
@@ -162,14 +166,14 @@ Deno.serve(async (req) => {
 
   const supabase = createClient(supabaseUrl, supabaseServiceKey)
   const result = await supabase
-    .from(actualTable)
+    .from(reference_table)
     .update(update)
     .eq("id", reference_id)
     .select("id")
 
   if (result.error) {
     console.error(`JAT webhook failed: ${result.error.message}`, {
-      table: actualTable,
+      table: reference_table,
       reference_id,
       update,
     })
@@ -180,11 +184,11 @@ Deno.serve(async (req) => {
   }
 
   if (!result.data || result.data.length === 0) {
-    console.warn(`JAT webhook: no rows matched ${actualTable}[${reference_id}]`, update)
+    console.warn(`JAT webhook: no rows matched ${reference_table}[${reference_id}]`, update)
     return new Response(
       JSON.stringify({
-        error: `No rows matched: ${actualTable} id=${reference_id}`,
-        table: actualTable,
+        error: `No rows matched: ${reference_table} id=${reference_id}`,
+        table: reference_table,
         id: reference_id,
         rowsAffected: 0,
       }),
@@ -192,12 +196,12 @@ Deno.serve(async (req) => {
     )
   }
 
-  console.log(`JAT webhook: ${event} → ${actualTable}[${reference_id}] (${result.data.length} row(s))`, update)
+  console.log(`JAT webhook: ${event} → ${reference_table}[${reference_id}] (${result.data.length} row(s))`, update)
 
   return new Response(
     JSON.stringify({
       success: true,
-      table: actualTable,
+      table: reference_table,
       id: reference_id,
       updated: update,
       rowsAffected: result.data.length,

package/supabase/migrations/3.3.0_realtime_voice.sql

@@ -0,0 +1,78 @@
+-- jat-feedback v3.3.0 — Realtime + voice transcription support
+--
+-- Enables Supabase Realtime on project_tasks (REPLICA IDENTITY FULL),
+-- adds audio_url column for voice recordings, expands status CHECK
+-- to include 'transcribing' and 'failed', and creates a private
+-- Storage bucket for voice audio files.
+--
+-- This is an additive migration (nullable column + new bucket + constraint update).
+-- Safe for 3.x — no existing data affected.
+--
+-- Apply to each consuming project:
+--   cp node_modules/jat-feedback/supabase/migrations/3.3.0_realtime_voice.sql \
+--      supabase/migrations/$(date +%Y%m%d%H%M%S)_feedback_3_3_0.sql
+--   supabase db push
+--
+-- After applying, enable Realtime for project_tasks in the Supabase dashboard:
+--   Database → Replication → supabase_realtime → toggle project_tasks ON
+
+-- ============================================================================
+-- 1. Enable Realtime (REPLICA IDENTITY FULL for complete row data in changes)
+-- ============================================================================
+
+ALTER TABLE project_tasks REPLICA IDENTITY FULL;
+
+-- ============================================================================
+-- 2. Add audio_url column (voice recording path in Storage)
+-- ============================================================================
+
+-- Separate from recording_url (which stores rrweb session replay data).
+-- audio_url points to raw voice audio files in the voice-recordings bucket.
+ALTER TABLE project_tasks ADD COLUMN IF NOT EXISTS audio_url TEXT;
+
+-- ============================================================================
+-- 3. Expand status CHECK constraint to include voice transcription states
+-- ============================================================================
+
+-- Drop the existing constraint (inherited from 1.0.0_feedback_reports.sql)
+-- and recreate with additional values for the voice pipeline.
+ALTER TABLE project_tasks DROP CONSTRAINT IF EXISTS feedback_reports_status_check;
+
+ALTER TABLE project_tasks ADD CONSTRAINT project_tasks_status_check
+  CHECK (status IN (
+    'submitted',
+    'in_progress',
+    'completed',
+    'accepted',
+    'rejected',
+    'wontfix',
+    'closed',
+    'transcribing',
+    'failed'
+  ));
+
+-- ============================================================================
+-- 4. Storage bucket for voice recordings (private)
+-- ============================================================================
+
+INSERT INTO storage.buckets (id, name, public)
+  VALUES ('voice-recordings', 'voice-recordings', false)
+  ON CONFLICT (id) DO NOTHING;
+
+-- RLS: service role can read and write
+CREATE POLICY "Service role can manage voice recordings"
+  ON storage.objects FOR ALL TO service_role
+  USING (bucket_id = 'voice-recordings')
+  WITH CHECK (bucket_id = 'voice-recordings');
+
+-- RLS: authenticated users can upload voice recordings
+CREATE POLICY "Authenticated users can upload voice recordings"
+  ON storage.objects FOR INSERT TO authenticated
+  WITH CHECK (bucket_id = 'voice-recordings');
+
+-- RLS: authenticated users can read own voice recordings
+CREATE POLICY "Authenticated users can read voice recordings"
+  ON storage.objects FOR SELECT TO authenticated
+  USING (bucket_id = 'voice-recordings');
+
+-- Anon role is blocked by default (no policy = no access)