edsger 0.72.6 → 0.73.0

package/dist/api/intelligence-normalize.d.ts

@@ -0,0 +1,25 @@
+/**
+ * Report payload normalization for the direct-SDK write path.
+ *
+ * The LLM that produces intelligence reports sometimes returns structured
+ * arrays (`key_findings`, `recommendations`, …) as bare strings, or as objects
+ * keyed under synonyms (`description` instead of `finding`, `recommendation`
+ * instead of `action`, …). We canonicalize before writing so every reader — the
+ * desktop UI especially — can rely on the shape declared in the migration.
+ *
+ * This mirrors the normalizer in the MCP edge handler
+ * (`supabase/functions/mcp/handlers/intelligence.ts`). The two can't share a
+ * single module because the edge function runs on Deno (zod v3, URL imports)
+ * while this runs on Node — keep them in sync if either changes.
+ */
+/**
+ * Normalize every structured array field present in the payload. Fields that
+ * are `undefined` are left untouched so partial updates don't overwrite stored
+ * values with empty arrays.
+ */
+export declare function normalizeReportArrays<T extends Record<string, unknown>>(payload: T): T;
+/**
+ * Coerce an LLM-supplied value to a finite number, or null. Handles numbers,
+ * numeric strings, and null/garbage uniformly.
+ */
+export declare function toNumberOrNull(value: unknown): number | null;

package/dist/api/intelligence-normalize.js

@@ -0,0 +1,103 @@
+/**
+ * Report payload normalization for the direct-SDK write path.
+ *
+ * The LLM that produces intelligence reports sometimes returns structured
+ * arrays (`key_findings`, `recommendations`, …) as bare strings, or as objects
+ * keyed under synonyms (`description` instead of `finding`, `recommendation`
+ * instead of `action`, …). We canonicalize before writing so every reader — the
+ * desktop UI especially — can rely on the shape declared in the migration.
+ *
+ * This mirrors the normalizer in the MCP edge handler
+ * (`supabase/functions/mcp/handlers/intelligence.ts`). The two can't share a
+ * single module because the edge function runs on Deno (zod v3, URL imports)
+ * while this runs on Node — keep them in sync if either changes.
+ */
+const NORMALIZE_SPECS = {
+    key_findings: {
+        primary: 'finding',
+        aliases: ['description', 'insight', 'text', 'title'],
+        defaults: { severity: 'fyi', category: '' },
+    },
+    recommendations: {
+        primary: 'action',
+        aliases: ['recommendation', 'description', 'text', 'title'],
+        defaults: { priority: 'medium', effort: '', impact: '', rationale: '' },
+    },
+    competitor_highlights: {
+        primary: 'highlight',
+        aliases: ['description', 'text', 'title'],
+        defaults: { competitor_name: '', sentiment: 'neutral' },
+    },
+    market_signals: {
+        primary: 'signal',
+        aliases: ['description', 'text', 'title'],
+        defaults: { source: '', relevance: '', url: '' },
+    },
+    trends: {
+        primary: 'trend',
+        aliases: ['description', 'text', 'title'],
+        defaults: { direction: '', evidence: '' },
+    },
+};
+function normalizeItem(item, spec) {
+    if (typeof item === 'string') {
+        return { [spec.primary]: item, ...spec.defaults };
+    }
+    if (!item || typeof item !== 'object') {
+        return { [spec.primary]: '', ...spec.defaults };
+    }
+    const obj = { ...item };
+    if (typeof obj[spec.primary] !== 'string' || !obj[spec.primary]) {
+        for (const alias of spec.aliases) {
+            const v = obj[alias];
+            if (typeof v === 'string' && v.trim().length > 0) {
+                obj[spec.primary] = v;
+                break;
+            }
+        }
+        if (typeof obj[spec.primary] !== 'string') {
+            obj[spec.primary] = '';
+        }
+    }
+    for (const [key, fallback] of Object.entries(spec.defaults)) {
+        if (obj[key] === undefined || obj[key] === null) {
+            obj[key] = fallback;
+        }
+    }
+    return obj;
+}
+function normalizeArray(value, spec) {
+    if (!Array.isArray(value)) {
+        return [];
+    }
+    return value.map((item) => normalizeItem(item, spec));
+}
+/**
+ * Normalize every structured array field present in the payload. Fields that
+ * are `undefined` are left untouched so partial updates don't overwrite stored
+ * values with empty arrays.
+ */
+export function normalizeReportArrays(payload) {
+    const result = { ...payload };
+    for (const [field, spec] of Object.entries(NORMALIZE_SPECS)) {
+        if (result[field] !== undefined) {
+            ;
+            result[field] = normalizeArray(result[field], spec);
+        }
+    }
+    return result;
+}
+/**
+ * Coerce an LLM-supplied value to a finite number, or null. Handles numbers,
+ * numeric strings, and null/garbage uniformly.
+ */
+export function toNumberOrNull(value) {
+    if (typeof value === 'number') {
+        return Number.isFinite(value) ? value : null;
+    }
+    if (typeof value === 'string' && value.trim() !== '') {
+        const n = Number(value);
+        return Number.isFinite(n) ? n : null;
+    }
+    return null;
+}

package/dist/api/intelligence.js

@@ -1,5 +1,6 @@
-import { getSupabase, hasSupabaseSession } from '../supabase/client.js';
+import { ensureSupabaseSession, getSupabase, hasSupabaseSession, } from '../supabase/client.js';
 import { logError, logInfo } from '../utils/logger.js';
+import { normalizeReportArrays, toNumberOrNull, } from './intelligence-normalize.js';
 import { callMcpEndpoint } from './mcp-client.js';
 // =============================================================================
 // MCP response helper
@@ -180,8 +181,41 @@ export async function saveSnapshot(snapshot, verbose) {
         logInfo(`Saving snapshot for competitor: ${snapshot.competitor_id}`);
     }
     try {
-        const result = await callMcpEndpoint('intelligence/snapshots/save', snapshot);
-        return parseMcpResponse(result, null);
+        // Direct SDK write under the user's RLS session — no MCP fallback.
+        // Await the synced session first so the insert carries the user JWT
+        // (otherwise it races out as `anon` and trips RLS).
+        const ready = await ensureSupabaseSession();
+        if (!ready) {
+            logError('Failed to save snapshot: no Supabase session (sign in to the desktop app)');
+            return null;
+        }
+        const source = ['ai_analysis', 'manual', 'scheduled'].includes(snapshot.source)
+            ? snapshot.source
+            : 'ai_analysis';
+        const { data, error } = await getSupabase()
+            .from('competitor_snapshots')
+            .insert({
+            competitor_id: snapshot.competitor_id,
+            product_id: snapshot.product_id,
+            issues: snapshot.issues ?? [],
+            pricing: snapshot.pricing ?? {},
+            tech_stack: snapshot.tech_stack ?? [],
+            app_rating: toNumberOrNull(snapshot.app_rating),
+            app_review_count: toNumberOrNull(snapshot.app_review_count),
+            app_version: snapshot.app_version ?? null,
+            app_last_updated: snapshot.app_last_updated ?? null,
+            recent_reviews: snapshot.recent_reviews ?? [],
+            social_mentions: snapshot.social_mentions ?? {},
+            changes_detected: snapshot.changes_detected ?? [],
+            source,
+            raw_data: snapshot.raw_data ?? null,
+        })
+            .select()
+            .single();
+        if (error) {
+            throw new Error(error.message);
+        }
+        return data ?? null;
     }
     catch (error) {
         logError(`Failed to save snapshot: ${error instanceof Error ? error.message : String(error)}`);
@@ -273,8 +307,54 @@ export async function saveReport(report, verbose) {
         logInfo(`Saving intelligence report: ${report.title}`);
     }
     try {
-        const result = await callMcpEndpoint('intelligence/reports/save', report);
-        return parseMcpResponse(result, null);
+        // Direct SDK write — no MCP fallback. intelligence_reports.created_by is
+        // NOT NULL and RLS checks `created_by = auth.uid()`, so we resolve the
+        // user from the synced session and canonicalize the LLM arrays first.
+        const ready = await ensureSupabaseSession();
+        if (!ready) {
+            logError('Failed to save report: no Supabase session (sign in to the desktop app)');
+            return null;
+        }
+        const { data: sessionData } = await getSupabase().auth.getSession();
+        const userId = sessionData.session?.user?.id;
+        if (!userId) {
+            logError('Failed to save report: could not resolve user from session');
+            return null;
+        }
+        const normalized = normalizeReportArrays({
+            key_findings: report.key_findings ?? [],
+            recommendations: report.recommendations ?? [],
+            competitor_highlights: report.competitor_highlights ?? [],
+            market_signals: report.market_signals ?? [],
+            trends: report.trends ?? [],
+        });
+        const { data, error } = await getSupabase()
+            .from('intelligence_reports')
+            .insert({
+            product_id: report.product_id,
+            report_type: report.report_type,
+            title: report.title,
+            summary: report.summary ?? null,
+            full_report: report.full_report ?? null,
+            key_findings: normalized.key_findings,
+            recommendations: normalized.recommendations,
+            competitor_highlights: normalized.competitor_highlights,
+            market_signals: normalized.market_signals,
+            trends: normalized.trends,
+            persona_updates: report.persona_updates ?? null,
+            pmf_score: toNumberOrNull(report.pmf_score),
+            pmf_signals: report.pmf_signals ?? null,
+            guidance: report.guidance ?? null,
+            status: report.status ?? 'completed',
+            snapshot_ids: report.snapshot_ids ?? [],
+            created_by: userId,
+        })
+            .select()
+            .single();
+        if (error) {
+            throw new Error(error.message);
+        }
+        return data ?? null;
     }
     catch (error) {
         logError(`Failed to save report: ${error instanceof Error ? error.message : String(error)}`);
@@ -286,11 +366,34 @@ export async function updateReport(reportId, updates, verbose) {
         logInfo(`Updating report: ${reportId}`);
     }
     try {
-        const result = await callMcpEndpoint('intelligence/reports/update', {
-            report_id: reportId,
-            ...updates,
-        });
-        return parseMcpResponse(result, null);
+        // Direct SDK write — no MCP fallback.
+        const ready = await ensureSupabaseSession();
+        if (!ready) {
+            logError('Failed to update report: no Supabase session (sign in to the desktop app)');
+            return null;
+        }
+        // Only send keys the caller provided so a partial update doesn't clobber
+        // stored columns. Numeric/array fields are coerced/canonicalized.
+        const payload = {};
+        for (const [key, value] of Object.entries(updates)) {
+            if (value !== undefined) {
+                payload[key] = value;
+            }
+        }
+        if ('pmf_score' in payload) {
+            payload.pmf_score = toNumberOrNull(payload.pmf_score);
+        }
+        const normalized = normalizeReportArrays(payload);
+        const { data, error } = await getSupabase()
+            .from('intelligence_reports')
+            .update(normalized)
+            .eq('id', reportId)
+            .select()
+            .single();
+        if (error) {
+            throw new Error(error.message);
+        }
+        return data ?? null;
     }
     catch (error) {
         logError(`Failed to update report: ${error instanceof Error ? error.message : String(error)}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "edsger",
-  "version": "0.72.6",
+  "version": "0.73.0",
   "type": "module",
   "bin": {
     "edsger": "dist/index.js"
@@ -54,8 +54,8 @@
     "commander": "^12.0.0",
     "cosmiconfig": "^9.0.0",
     "dotenv": "^16.4.5",
-    "edsger-contract": "0.9.2",
-    "edsger-tools": "0.9.2",
+    "edsger-contract": "0.10.0",
+    "edsger-tools": "0.10.0",
     "gray-matter": "^4.0.3",
     "zod": "^4.0.0"
   },