yaver-feedback-react-native 0.6.1 → 0.7.1

package/dist/deviceDedup.d.ts

@@ -0,0 +1,44 @@
+/**
+ * Device deduplication + online-signal merging, ported from the Yaver
+ * mobile app's DeviceContext.collapseAliasDevices.
+ *
+ * Convex stores one row per pair (device, re-install). After a re-pair
+ * or hostname change the list can contain 2-3 rows for the same
+ * physical machine, with different hwid/publicKey values. The picker
+ * then shows duplicates and the user can't tell which one is live.
+ *
+ * This file collapses rows in three passes:
+ *   1. Identity key (hwid → publicKey → "host:os:name" → id → name)
+ *   2. Alias key (os + normalized-hostname) — catches re-pairs that
+ *      mint a new hwid
+ *   3. Endpoint key (host:port)
+ *
+ * When collapsing, `mergeDeviceEntries` prefers: authenticated over
+ * needsAuth, online over offline, freshest lastHeartbeat.
+ */
+import type { RemoteDevice } from './auth';
+/**
+ * Collapse a Convex device list so each physical machine appears once.
+ * Safe on an empty list; idempotent on an already-deduped list.
+ */
+export declare function collapseRemoteDevices(devices: RemoteDevice[]): RemoteDevice[];
+/**
+ * Threshold for "online" based on heartbeat age, in milliseconds.
+ * Matches the mobile app (HEARTBEAT_STALE_MS = 90 s). The SDK used
+ * 60 s, which flashed yellow on single missed beats.
+ */
+export declare const HEARTBEAT_STALE_MS = 90000;
+/**
+ * Returns a freshness flag consistent with the mobile app. A device is
+ * "fresh" when it was online per Convex AND its heartbeat is within
+ * `HEARTBEAT_STALE_MS`.
+ */
+export declare function isDeviceFresh(d: RemoteDevice): boolean;
+/**
+ * Pick the best candidate for an auto-connect attempt. Preference:
+ *   1. matches the preferred deviceId when supplied + still fresh
+ *   2. fresh (online + recent heartbeat) + has a quicHost
+ *   3. online + has a quicHost
+ *   4. first with a quicHost
+ */
+export declare function pickTargetDevice(devices: RemoteDevice[], preferredDeviceId?: string): RemoteDevice | null;

package/dist/deviceDedup.js

@@ -0,0 +1,184 @@
+"use strict";
+/**
+ * Device deduplication + online-signal merging, ported from the Yaver
+ * mobile app's DeviceContext.collapseAliasDevices.
+ *
+ * Convex stores one row per pair (device, re-install). After a re-pair
+ * or hostname change the list can contain 2-3 rows for the same
+ * physical machine, with different hwid/publicKey values. The picker
+ * then shows duplicates and the user can't tell which one is live.
+ *
+ * This file collapses rows in three passes:
+ *   1. Identity key (hwid → publicKey → "host:os:name" → id → name)
+ *   2. Alias key (os + normalized-hostname) — catches re-pairs that
+ *      mint a new hwid
+ *   3. Endpoint key (host:port)
+ *
+ * When collapsing, `mergeDeviceEntries` prefers: authenticated over
+ * needsAuth, online over offline, freshest lastHeartbeat.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.HEARTBEAT_STALE_MS = void 0;
+exports.collapseRemoteDevices = collapseRemoteDevices;
+exports.isDeviceFresh = isDeviceFresh;
+exports.pickTargetDevice = pickTargetDevice;
+function normalizedName(name) {
+    return String(name || '').trim().toLowerCase().replace(/\.local$/i, '');
+}
+function normalizedHost(host) {
+    return String(host || '').trim().toLowerCase().replace(/\.local$/i, '');
+}
+function identityKey(d) {
+    if (d.hwid)
+        return `hwid:${d.hwid}`;
+    if (d.publicKey)
+        return `pub:${d.publicKey}`;
+    if (d.isGuest) {
+        const scope = d.hostEmail || d.hostName || 'guest';
+        return `guest:${scope}:${d.deviceId || d.name}`;
+    }
+    const n = normalizedName(d.name);
+    const os = String(d.platform || '').trim().toLowerCase();
+    if (n && os)
+        return `host:${os}:${n}`;
+    if (d.deviceId)
+        return `id:${d.deviceId}`;
+    return `name:${d.name}`;
+}
+function aliasKey(d) {
+    if (d.isGuest)
+        return null;
+    const n = normalizedName(d.name);
+    const os = String(d.platform || '').trim().toLowerCase();
+    if (!n || !os)
+        return null;
+    return `${os}:${n}`;
+}
+function endpointKey(d) {
+    if (d.isGuest)
+        return null;
+    const h = normalizedHost(d.quicHost);
+    if (!h)
+        return null;
+    return `${h}:${d.quicPort || 0}`;
+}
+function mergeEntries(existing, incoming) {
+    const incomingWins = (!!existing.needsAuth && !incoming.needsAuth) ||
+        (incoming.lastHeartbeat || 0) > (existing.lastHeartbeat || 0) ||
+        (!!incoming.isOnline && !existing.isOnline);
+    const base = incomingWins ? incoming : existing;
+    const other = incomingWins ? existing : incoming;
+    return {
+        ...other,
+        ...base,
+        quicHost: base.quicHost || other.quicHost,
+        quicPort: base.quicPort || other.quicPort,
+        isOnline: base.isOnline || other.isOnline,
+        runnerDown: base.runnerDown && other.runnerDown,
+        publicKey: base.publicKey || other.publicKey,
+        lastHeartbeat: Math.max(existing.lastHeartbeat || 0, incoming.lastHeartbeat || 0),
+    };
+}
+// When two rows share the same alias key (hostname + OS) but differ on
+// hwid/publicKey, pick the active one over the stale needs-auth leftover.
+function pickActiveOverStaleNeedsAuth(a, b) {
+    const aDead = a.needsAuth && !a.isOnline;
+    const bDead = b.needsAuth && !b.isOnline;
+    const aLive = !a.needsAuth && a.isOnline;
+    const bLive = !b.needsAuth && b.isOnline;
+    if (aDead && bLive)
+        return b;
+    if (bDead && aLive)
+        return a;
+    return null;
+}
+/**
+ * Collapse a Convex device list so each physical machine appears once.
+ * Safe on an empty list; idempotent on an already-deduped list.
+ */
+function collapseRemoteDevices(devices) {
+    if (!Array.isArray(devices) || devices.length === 0)
+        return [];
+    const byIdentity = new Map();
+    for (const d of devices) {
+        const k = identityKey(d);
+        const prev = byIdentity.get(k);
+        byIdentity.set(k, prev ? mergeEntries(prev, d) : d);
+    }
+    const byAlias = new Map();
+    for (const d of byIdentity.values()) {
+        const k = aliasKey(d);
+        if (!k) {
+            byAlias.set(`id:${d.deviceId}`, d);
+            continue;
+        }
+        const prev = byAlias.get(k);
+        if (!prev) {
+            byAlias.set(k, d);
+            continue;
+        }
+        const strongIdentityConflict = !!prev.publicKey && !!d.publicKey && prev.publicKey !== d.publicKey;
+        if (strongIdentityConflict) {
+            const winner = pickActiveOverStaleNeedsAuth(prev, d);
+            if (winner) {
+                byAlias.set(k, winner);
+                continue;
+            }
+        }
+        byAlias.set(k, mergeEntries(prev, d));
+    }
+    const byEndpoint = new Map();
+    for (const d of byAlias.values()) {
+        const k = endpointKey(d);
+        if (!k) {
+            byEndpoint.set(`id:${d.deviceId}`, d);
+            continue;
+        }
+        const prev = byEndpoint.get(k);
+        byEndpoint.set(k, prev ? mergeEntries(prev, d) : d);
+    }
+    return [...byEndpoint.values()];
+}
+/**
+ * Threshold for "online" based on heartbeat age, in milliseconds.
+ * Matches the mobile app (HEARTBEAT_STALE_MS = 90 s). The SDK used
+ * 60 s, which flashed yellow on single missed beats.
+ */
+exports.HEARTBEAT_STALE_MS = 90000;
+/**
+ * Returns a freshness flag consistent with the mobile app. A device is
+ * "fresh" when it was online per Convex AND its heartbeat is within
+ * `HEARTBEAT_STALE_MS`.
+ */
+function isDeviceFresh(d) {
+    if (!d.isOnline)
+        return false;
+    if (!d.lastHeartbeat)
+        return true;
+    return Date.now() - d.lastHeartbeat < exports.HEARTBEAT_STALE_MS;
+}
+/**
+ * Pick the best candidate for an auto-connect attempt. Preference:
+ *   1. matches the preferred deviceId when supplied + still fresh
+ *   2. fresh (online + recent heartbeat) + has a quicHost
+ *   3. online + has a quicHost
+ *   4. first with a quicHost
+ */
+function pickTargetDevice(devices, preferredDeviceId) {
+    if (!devices.length)
+        return null;
+    if (preferredDeviceId) {
+        const preferred = devices.find((d) => d.deviceId === preferredDeviceId && d.quicHost);
+        if (preferred && isDeviceFresh(preferred))
+            return preferred;
+        if (preferred)
+            return preferred;
+    }
+    const fresh = devices.find((d) => isDeviceFresh(d) && d.quicHost);
+    if (fresh)
+        return fresh;
+    const online = devices.find((d) => d.isOnline && d.quicHost);
+    if (online)
+        return online;
+    return devices.find((d) => d.quicHost) || devices[0] || null;
+}

package/dist/expo.d.ts

@@ -7,7 +7,6 @@ import type { FeedbackConfig } from './types';
  *
  * Defaults:
  * - trigger: 'shake'
- * - feedbackMode: 'batch'
  * - enabled: __DEV__ (only active in development)
  *
  * @param overrides - Optional partial config to override defaults

package/dist/expo.js

@@ -32,7 +32,6 @@ const YaverFeedback_1 = require("./YaverFeedback");
  *
  * Defaults:
  * - trigger: 'shake'
- * - feedbackMode: 'batch'
  * - enabled: __DEV__ (only active in development)
  *
  * @param overrides - Optional partial config to override defaults
@@ -54,7 +53,6 @@ function initExpo(overrides) {
     YaverFeedback_1.YaverFeedback.init({
         authToken: '', // LAN auto-discovery doesn't require a token
         trigger: 'shake',
-        feedbackMode: 'batch',
         enabled: __DEV__,
         ...overrides,
         ...(agentUrl ? { agentUrl } : {}),

package/dist/index.d.ts

@@ -1,24 +1,29 @@
 /**
- * @yaver/feedback-react-native — Visual feedback SDK for Yaver.
+ * yaver-feedback-react-native — Visual feedback SDK for Yaver.
  *
- * Shake-to-report, screenshots, voice annotations, P2P connection,
- * device discovery, and live/narrated/batch feedback modes for vibe coding.
+ * Shake-to-report surface with five one-tap actions:
+ *  1. Hot Reload              — instant JS reload
+ *  2. Screenshot & Fix        — capture the screen under the modal and
+ *                               kick a fix task on the agent
+ *  3. Vibing                  — open a vibing session on the agent
+ *  4. Start / Stop Recording  — screen recording toggle
+ *  5. Send Video              — submit the last recording
  *
  * @example
  * ```tsx
- * import { YaverFeedback, FeedbackProvider } from '@yaver/feedback-react-native';
+ * import { YaverFeedback, FeedbackModal } from 'yaver-feedback-react-native';
  *
  * YaverFeedback.init({
  *   agentUrl: 'http://192.168.1.10:18080',
  *   authToken: 'your-token',
  *   trigger: 'shake',
- *   feedbackMode: 'live',
+ *   strictNativeAuth: true,
  * });
  *
- * // Wrap your app root:
- * <FeedbackProvider>
+ * <>
  *   <App />
- * </FeedbackProvider>
+ *   <FeedbackModal />
+ * </>
  * ```
  */
 export { YaverFeedback } from './YaverFeedback';
@@ -40,9 +45,9 @@ export { FeedbackModal } from './FeedbackModal';
 export { FixReport } from './FixReport';
 export { configureAuthEndpoints, getConvexSiteUrl, getWebBaseUrl, getToken, saveToken, clearToken, getUser, saveUser, getSelectedDeviceId, saveSelectedDeviceId, clearSelectedDeviceId, validateToken, signInWithApple, signInWithOAuth, signupWithEmail, loginWithEmail, listReachableDevices, DEFAULT_CONVEX_SITE_URL, DEFAULT_WEB_BASE_URL, DEFAULT_OAUTH_REDIRECT, } from './auth';
 export type { OAuthProvider, User, RemoteDevice, DeviceList, } from './auth';
-export { captureScreenshot, startAudioRecording, stopAudioRecording } from './capture';
+export { captureScreenshot, startVideoRecording, stopVideoRecording, isVideoRecording, } from './capture';
 export { uploadFeedback } from './upload';
-export type { FeedbackConfig, FeedbackBundle, FeedbackMetadata, DeviceInfo, AppInfo, TimelineEvent, FeedbackReport, AgentCommentary, FeedbackStreamEvent, VoiceCapability, CapturedError, TestFix, TestSession, } from './types';
+export type { FeedbackConfig, FeedbackBundle, FeedbackMetadata, DeviceInfo, AppInfo, TimelineEvent, FeedbackReport, FeedbackStreamEvent, VoiceCapability, CapturedError, TestFix, TestSession, } from './types';
 export type { BlackBoxEvent, BlackBoxConfig, BlackBoxCommand, CommandHandler } from './BlackBox';
 export type { DiscoveryResult } from './Discovery';
 export type { FeedbackEvent } from './P2PClient';

package/dist/index.js

@@ -1,29 +1,34 @@
 "use strict";
 /**
- * @yaver/feedback-react-native — Visual feedback SDK for Yaver.
+ * yaver-feedback-react-native — Visual feedback SDK for Yaver.
  *
- * Shake-to-report, screenshots, voice annotations, P2P connection,
- * device discovery, and live/narrated/batch feedback modes for vibe coding.
+ * Shake-to-report surface with five one-tap actions:
+ *  1. Hot Reload              — instant JS reload
+ *  2. Screenshot & Fix        — capture the screen under the modal and
+ *                               kick a fix task on the agent
+ *  3. Vibing                  — open a vibing session on the agent
+ *  4. Start / Stop Recording  — screen recording toggle
+ *  5. Send Video              — submit the last recording
  *
  * @example
  * ```tsx
- * import { YaverFeedback, FeedbackProvider } from '@yaver/feedback-react-native';
+ * import { YaverFeedback, FeedbackModal } from 'yaver-feedback-react-native';
  *
  * YaverFeedback.init({
  *   agentUrl: 'http://192.168.1.10:18080',
  *   authToken: 'your-token',
  *   trigger: 'shake',
- *   feedbackMode: 'live',
+ *   strictNativeAuth: true,
  * });
  *
- * // Wrap your app root:
- * <FeedbackProvider>
+ * <>
  *   <App />
- * </FeedbackProvider>
+ *   <FeedbackModal />
+ * </>
  * ```
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.uploadFeedback = exports.stopAudioRecording = exports.startAudioRecording = exports.captureScreenshot = exports.DEFAULT_OAUTH_REDIRECT = exports.DEFAULT_WEB_BASE_URL = exports.DEFAULT_CONVEX_SITE_URL = exports.listReachableDevices = exports.loginWithEmail = exports.signupWithEmail = exports.signInWithOAuth = exports.signInWithApple = exports.validateToken = exports.clearSelectedDeviceId = exports.saveSelectedDeviceId = exports.getSelectedDeviceId = exports.saveUser = exports.getUser = exports.clearToken = exports.saveToken = exports.getToken = exports.getWebBaseUrl = exports.getConvexSiteUrl = exports.configureAuthEndpoints = exports.FixReport = exports.FeedbackModal = exports.FloatingButton = exports.ShakeDetector = exports.AuthOverlay = exports.YaverMachinePickerScreen = exports.YaverLoginScreen = exports.YaverConnectionScreen = exports.P2PClient = exports.YaverDiscovery = exports.initExpo = exports.YaverUpdates = exports.BlackBox = exports.YaverFeedback = void 0;
+exports.uploadFeedback = exports.isVideoRecording = exports.stopVideoRecording = exports.startVideoRecording = exports.captureScreenshot = exports.DEFAULT_OAUTH_REDIRECT = exports.DEFAULT_WEB_BASE_URL = exports.DEFAULT_CONVEX_SITE_URL = exports.listReachableDevices = exports.loginWithEmail = exports.signupWithEmail = exports.signInWithOAuth = exports.signInWithApple = exports.validateToken = exports.clearSelectedDeviceId = exports.saveSelectedDeviceId = exports.getSelectedDeviceId = exports.saveUser = exports.getUser = exports.clearToken = exports.saveToken = exports.getToken = exports.getWebBaseUrl = exports.getConvexSiteUrl = exports.configureAuthEndpoints = exports.FixReport = exports.FeedbackModal = exports.FloatingButton = exports.ShakeDetector = exports.AuthOverlay = exports.YaverMachinePickerScreen = exports.YaverLoginScreen = exports.YaverConnectionScreen = exports.P2PClient = exports.YaverDiscovery = exports.initExpo = exports.YaverUpdates = exports.BlackBox = exports.YaverFeedback = void 0;
 var YaverFeedback_1 = require("./YaverFeedback");
 Object.defineProperty(exports, "YaverFeedback", { enumerable: true, get: function () { return YaverFeedback_1.YaverFeedback; } });
 var BlackBox_1 = require("./BlackBox");
@@ -75,7 +80,8 @@ Object.defineProperty(exports, "DEFAULT_WEB_BASE_URL", { enumerable: true, get:
 Object.defineProperty(exports, "DEFAULT_OAUTH_REDIRECT", { enumerable: true, get: function () { return auth_1.DEFAULT_OAUTH_REDIRECT; } });
 var capture_1 = require("./capture");
 Object.defineProperty(exports, "captureScreenshot", { enumerable: true, get: function () { return capture_1.captureScreenshot; } });
-Object.defineProperty(exports, "startAudioRecording", { enumerable: true, get: function () { return capture_1.startAudioRecording; } });
-Object.defineProperty(exports, "stopAudioRecording", { enumerable: true, get: function () { return capture_1.stopAudioRecording; } });
+Object.defineProperty(exports, "startVideoRecording", { enumerable: true, get: function () { return capture_1.startVideoRecording; } });
+Object.defineProperty(exports, "stopVideoRecording", { enumerable: true, get: function () { return capture_1.stopVideoRecording; } });
+Object.defineProperty(exports, "isVideoRecording", { enumerable: true, get: function () { return capture_1.isVideoRecording; } });
 var upload_1 = require("./upload");
 Object.defineProperty(exports, "uploadFeedback", { enumerable: true, get: function () { return upload_1.uploadFeedback; } });

package/dist/types.d.ts

@@ -65,27 +65,6 @@ export interface FeedbackConfig {
     reportingOnly?: boolean;
     /** Max screen recording duration in seconds. Default: 120 */
     maxRecordingDuration?: number;
-    /**
-     * Feedback mode:
-     * - 'live': stream events to the agent as they happen
-     * - 'narrated': record everything, send on stop
-     * - 'batch': dump everything at end (default)
-     */
-    feedbackMode?: 'live' | 'narrated' | 'batch';
-    /**
-     * Agent commentary level (0-10).
-     * 0 = silent, 10 = agent comments on everything it sees.
-     * Only relevant in live mode. Default: 0.
-     */
-    agentCommentaryLevel?: number;
-    /**
-     * Enable voice input for feedback annotations. Always true by default.
-     * Audio is recorded on the device and sent to the agent for transcription.
-     * Works regardless of whether a speech-to-speech provider is configured —
-     * if STT is available on the agent, audio is auto-transcribed; otherwise
-     * raw audio is attached to the feedback report.
-     */
-    voiceEnabled?: boolean;
     /**
      * Maximum number of captured errors to keep in memory (ring buffer).
      * Oldest errors are evicted when the buffer is full.
@@ -179,13 +158,10 @@ export interface FeedbackConfig {
 }
 export interface FeedbackBundle {
     metadata: FeedbackMetadata;
+    /** Screen-recording file path, when produced by the "Start Recording" action. */
     video?: string;
-    /** Voice annotation audio file path (WAV). Always available when voiceEnabled. */
-    audio?: string;
-    /** Transcribed text from voice annotation (if STT/S2S provider is available on agent). */
-    audioTranscript?: string;
     screenshots: string[];
-    /** Captured errors with stack traces, attached automatically when captureErrors is enabled. */
+    /** Captured errors with stack traces, attached via attachError / wrapErrorHandler. */
     errors?: CapturedError[];
 }
 /** An error captured by the SDK's global error handler. */
@@ -231,12 +207,6 @@ export interface FeedbackReport {
     status: 'pending' | 'uploading' | 'uploaded' | 'failed';
     error?: string;
 }
-export interface AgentCommentary {
-    id: string;
-    timestamp: string;
-    message: string;
-    type: 'observation' | 'suggestion' | 'question' | 'action';
-}
 export interface FeedbackStreamEvent {
     type: string;
     timestamp: string;

package/dist/upload.d.ts

@@ -5,9 +5,14 @@ import { FeedbackBundle } from './types';
  * The agent receives the bundle at POST /feedback with:
  * - `metadata` (JSON string)
  * - `screenshot_0`, `screenshot_1`, ... (image files)
- * - `audio` (audio file, if present)
  * - `video` (video file, if present)
  *
- * @returns The feedback report ID from the agent response.
+ * Returns the parsed agent response — typically `{ ok, id, reportId }`.
+ * Callers can inspect `.id` / `.reportId` to drive a follow-up
+ * `/feedback/{id}/fix` kick.
  */
-export declare function uploadFeedback(agentUrl: string, authToken: string, bundle: FeedbackBundle): Promise<string>;
+export declare function uploadFeedback(agentUrl: string, authToken: string, bundle: FeedbackBundle): Promise<{
+    id?: string;
+    reportId?: string;
+    [k: string]: unknown;
+}>;

package/dist/upload.js

@@ -8,10 +8,11 @@ const react_native_1 = require("react-native");
  * The agent receives the bundle at POST /feedback with:
  * - `metadata` (JSON string)
  * - `screenshot_0`, `screenshot_1`, ... (image files)
- * - `audio` (audio file, if present)
  * - `video` (video file, if present)
  *
- * @returns The feedback report ID from the agent response.
+ * Returns the parsed agent response — typically `{ ok, id, reportId }`.
+ * Callers can inspect `.id` / `.reportId` to drive a follow-up
+ * `/feedback/{id}/fix` kick.
  */
 async function uploadFeedback(agentUrl, authToken, bundle) {
     const formData = new FormData();
@@ -26,14 +27,6 @@ async function uploadFeedback(agentUrl, authToken, bundle) {
             name: `screenshot_${i}.png`,
         });
     }
-    // Attach audio
-    if (bundle.audio) {
-        formData.append('audio', {
-            uri: react_native_1.Platform.OS === 'android' ? `file://${bundle.audio}` : bundle.audio,
-            type: 'audio/m4a',
-            name: 'voice_note.m4a',
-        });
-    }
     // Attach video
     if (bundle.video) {
         formData.append('video', {
@@ -54,6 +47,6 @@ async function uploadFeedback(agentUrl, authToken, bundle) {
         const text = await response.text().catch(() => '');
         throw new Error(`[YaverFeedback] Upload failed (${response.status}): ${text}`);
     }
-    const result = await response.json();
-    return result.id ?? result.reportId ?? 'unknown';
+    const result = await response.json().catch(() => ({}));
+    return result;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "yaver-feedback-react-native",
-  "version": "0.6.1",
+  "version": "0.7.1",
   "description": "Visual feedback SDK for Yaver — bug reports, screen recording, voice annotations, and local-first developer workflows",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",