jat-feedback 3.4.2 → 3.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jat-feedback",
-  "version": "3.4.2",
+  "version": "3.5.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/routes/api/tasks/[id]/comments/+server.ts

@@ -0,0 +1,107 @@
+/**
+ * Widget comments API — consumer route template.
+ * Copy to: src/routes/api/tasks/[id]/comments/+server.ts
+ *
+ * Requires jat-feedback v3.5.0 migration (external column on project_tasks_comments).
+ *
+ * GET  ?external=true  → external comments for a task (for widget thread view)
+ * POST               → reporter posts a reply comment (always external=true)
+ */
+import { json } from '@sveltejs/kit'
+import type { RequestHandler } from './$types'
+
+export const GET: RequestHandler = async ({ params, locals, url }) => {
+  const { id } = params
+  const externalOnly = url.searchParams.get('external') === 'true'
+
+  // Verify task exists
+  const { data: task, error: taskErr } = await locals.supabaseServiceRole
+    .from('project_tasks')
+    .select('id')
+    .eq('id', id)
+    .single()
+
+  if (taskErr || !task) {
+    return json({ error: 'Task not found' }, { status: 404 })
+  }
+
+  let query = locals.supabaseServiceRole
+    .from('project_tasks_comments')
+    .select('id, text, author, author_type, comment_type, external, created_at, metadata')
+    .eq('task_id', id)
+    .order('created_at', { ascending: true })
+
+  if (externalOnly) {
+    query = (query as any).eq('external', true)
+  }
+
+  const { data, error } = await query
+
+  if (error) {
+    return json({ error: error.message }, { status: 500 })
+  }
+
+  const comments = (data || []).map((c: any) => ({
+    ...c,
+    metadata: c.metadata ? safeParse(c.metadata) : null,
+    external: c.external !== false,
+  }))
+
+  return json({ comments })
+}
+
+export const POST: RequestHandler = async ({ params, request, locals }) => {
+  const { id } = params
+
+  let body: Record<string, unknown>
+  try {
+    body = await request.json()
+  } catch {
+    return json({ error: 'Invalid JSON body' }, { status: 400 })
+  }
+
+  const { text, author, author_email, author_type, comment_type, metadata } = body
+
+  if (!text || typeof text !== 'string' || !text.trim()) {
+    return json({ error: 'text is required' }, { status: 400 })
+  }
+  if (!author || typeof author !== 'string') {
+    return json({ error: 'author is required' }, { status: 400 })
+  }
+
+  // Verify task exists
+  const { data: task, error: taskErr } = await locals.supabaseServiceRole
+    .from('project_tasks')
+    .select('id')
+    .eq('id', id)
+    .single()
+
+  if (taskErr || !task) {
+    return json({ error: 'Task not found' }, { status: 404 })
+  }
+
+  const { data: comment, error } = await locals.supabaseServiceRole
+    .from('project_tasks_comments')
+    .insert({
+      task_id: id,
+      text: (text as string).trim(),
+      author,
+      author_type: author_type ?? 'user',
+      comment_type: comment_type ?? 'note',
+      external: true,
+      metadata: metadata != null ? JSON.stringify(metadata) : null,
+    })
+    .select('id, text, author, author_type, comment_type, external, created_at, metadata')
+    .single()
+
+  if (error) {
+    return json({ error: error.message }, { status: 500 })
+  }
+
+  return json({ comment: { ...comment, external: true } }, { status: 201 })
+}
+
+function safeParse(s: unknown) {
+  if (typeof s !== 'string') return s
+  try { return JSON.parse(s) } catch { return s }
+}

package/supabase/migrations/3.5.0_extend_comments_schema.sql

@@ -0,0 +1,30 @@
+-- jat-feedback v3.5.0 — Extended comments schema
+--
+-- Adds first-class fields for agent question/answer threads:
+--   author_type   TEXT        -- 'agent' | 'user' | 'system'
+--   comment_type  TEXT        -- 'question' | 'answer' | 'note' | 'event'
+--   session_id    TEXT        -- agent session to resume on answer (NULL = human thread)
+--   metadata      TEXT        -- JSON blob, extensible
+--   external      BOOLEAN     -- true = visible to client; false = internal agent/team thread
+--
+-- All columns are additive and nullable-safe (IF NOT EXISTS guards).
+-- Existing rows get external=true, preserving current behaviour where every
+-- comment was visible everywhere.
+--
+-- Apply to each consuming project:
+--   cp node_modules/jat-feedback/supabase/migrations/3.5.0_extend_comments_schema.sql \
+--      supabase/migrations/$(date +%Y%m%d%H%M%S)_feedback_3_5_0.sql
+--   supabase db push
+
+ALTER TABLE project_tasks_comments
+  ADD COLUMN IF NOT EXISTS author_type  TEXT,
+  ADD COLUMN IF NOT EXISTS comment_type TEXT,
+  ADD COLUMN IF NOT EXISTS session_id   TEXT,
+  ADD COLUMN IF NOT EXISTS metadata     TEXT,
+  ADD COLUMN IF NOT EXISTS external     BOOLEAN NOT NULL DEFAULT true;
+
+CREATE INDEX IF NOT EXISTS idx_project_tasks_comments_type
+  ON project_tasks_comments(task_id, comment_type);
+
+CREATE INDEX IF NOT EXISTS idx_project_tasks_comments_external
+  ON project_tasks_comments(task_id, external);