yaver-feedback-react-native 0.8.11 → 0.8.13

package/src/VibeChatScreen.tsx

@@ -0,0 +1,362 @@
+// VibeChatScreen — the converged chat UI for the standalone feedback
+// SDK. Mirrors Yaver mobile's Tasks tab + the in-Yaver native pane:
+//
+//   1. User sees a live SSE transcript of agent stdout (PhaseStatusLine
+//      style "searching… / compiling…" while running, full markdown
+//      output once it lands).
+//   2. User can keep vibing — type a follow-up after the first turn
+//      lands and POST a /tasks/{id}/resume to multi-turn the same
+//      coding session.
+//   3. Reload button at the bottom hits client.reloadApp() so the user
+//      can see the change without leaving the chat.
+//
+// State machine:
+//   idle    — empty, waiting for first prompt (handled by parent screen)
+//   running — task is live, transcript streams, follow-up disabled
+//   done    — task finished, follow-up enabled, Reload prominent
+//   failed  — same as done but error tinted
+
+import React, { useCallback, useEffect, useRef, useState } from 'react';
+import {
+  ActivityIndicator,
+  ScrollView,
+  StyleSheet,
+  Text,
+  TextInput,
+  TouchableOpacity,
+  View,
+} from 'react-native';
+import type { P2PClient } from './P2PClient';
+
+export type VibeTurnRole = 'user' | 'assistant' | 'status';
+
+export interface VibeTurn {
+  id: string;
+  role: VibeTurnRole;
+  text: string;
+  timestamp: number;
+}
+
+interface Props {
+  client: P2PClient;
+  initialTaskId: string;
+  initialUserPrompt: string;
+  onClose?: () => void;
+  /** Called when the user taps Reload after a task completes — uses
+   *  P2PClient.reloadApp() with the active project context. */
+  onReload?: () => Promise<void>;
+}
+
+export function VibeChatScreen({
+  client,
+  initialTaskId,
+  initialUserPrompt,
+  onClose,
+  onReload,
+}: Props) {
+  const [taskId, setTaskId] = useState(initialTaskId);
+  const [turns, setTurns] = useState<VibeTurn[]>(() => [
+    {
+      id: `user-${Date.now()}`,
+      role: 'user',
+      text: initialUserPrompt,
+      timestamp: Date.now(),
+    },
+    {
+      id: `status-${Date.now()}`,
+      role: 'status',
+      text: 'starting…',
+      timestamp: Date.now(),
+    },
+  ]);
+  const [streamBuffer, setStreamBuffer] = useState('');
+  const [status, setStatus] = useState<'running' | 'done' | 'failed'>('running');
+  const [followUp, setFollowUp] = useState('');
+  const [isResuming, setIsResuming] = useState(false);
+  const [isReloading, setIsReloading] = useState(false);
+  const scrollRef = useRef<ScrollView | null>(null);
+  const abortRef = useRef<(() => void) | null>(null);
+
+  // Subscribe to the current task's SSE stream. Re-runs whenever the
+  // taskId changes (resumeTask reuses the same id, so this only fires
+  // once per task — which is fine).
+  useEffect(() => {
+    let live = true;
+    const acc: string[] = [];
+    const close = client.streamTaskOutput(
+      taskId,
+      (line) => {
+        if (!live) return;
+        // Filter our internal error sentinel from the SSE helper.
+        if (line.startsWith('__error__:')) {
+          setStatus('failed');
+          setStreamBuffer((prev) => prev + (prev ? '\n' : '') + line.slice('__error__:'.length).trim());
+          return;
+        }
+        acc.push(line);
+        // Throttle re-renders: flush every ~100ms.
+        setStreamBuffer(acc.join('\n'));
+      },
+      (terminal) => {
+        if (!live) return;
+        setStatus(terminal === 'completed' ? 'done' : 'failed');
+        // Move the buffered stream into a real assistant turn so the
+        // user sees a stable render and can scroll back, then clear
+        // the buffer for any follow-up.
+        setTurns((prev) => {
+          const collapsed = acc.join('\n').trim();
+          if (!collapsed) return prev.filter((t) => t.role !== 'status');
+          const next = prev.filter((t) => t.role !== 'status');
+          next.push({
+            id: `assistant-${taskId}-${Date.now()}`,
+            role: 'assistant',
+            text: collapsed,
+            timestamp: Date.now(),
+          });
+          return next;
+        });
+        setStreamBuffer('');
+      },
+    );
+    abortRef.current = close;
+    return () => {
+      live = false;
+      try { close(); } catch { /* ignore */ }
+    };
+  }, [client, taskId]);
+
+  // Auto-scroll the transcript when new content lands.
+  useEffect(() => {
+    const t = setTimeout(() => {
+      scrollRef.current?.scrollToEnd({ animated: true });
+    }, 50);
+    return () => clearTimeout(t);
+  }, [streamBuffer, turns]);
+
+  const handleSendFollowUp = useCallback(async () => {
+    const text = followUp.trim();
+    if (!text || isResuming) return;
+    setIsResuming(true);
+    // Add user turn immediately for snappy UX.
+    setTurns((prev) => [
+      ...prev,
+      { id: `user-${Date.now()}`, role: 'user', text, timestamp: Date.now() },
+      { id: `status-${Date.now()}`, role: 'status', text: 'thinking…', timestamp: Date.now() },
+    ]);
+    setFollowUp('');
+    setStatus('running');
+    setStreamBuffer('');
+    try {
+      await client.resumeTask({ taskId, userPrompt: text });
+      // resumeTask reuses the same taskId, so the SSE subscription
+      // above will pick up the new output stream automatically. To
+      // force a fresh subscription we momentarily flip taskId to a
+      // sentinel and back; cleaner than tearing down + re-attaching
+      // the SSE manually.
+      const same = taskId;
+      setTaskId(`${same}#`);
+      setTimeout(() => setTaskId(same), 0);
+    } catch (e) {
+      setStatus('failed');
+      setTurns((prev) => [
+        ...prev.filter((t) => t.role !== 'status'),
+        {
+          id: `assistant-err-${Date.now()}`,
+          role: 'assistant',
+          text: `Failed to send follow-up: ${e instanceof Error ? e.message : String(e)}`,
+          timestamp: Date.now(),
+        },
+      ]);
+    } finally {
+      setIsResuming(false);
+    }
+  }, [client, followUp, isResuming, taskId]);
+
+  const handleReload = useCallback(async () => {
+    if (isReloading || !onReload) return;
+    setIsReloading(true);
+    try {
+      await onReload();
+    } catch (e) {
+      setTurns((prev) => [
+        ...prev,
+        {
+          id: `assistant-reload-err-${Date.now()}`,
+          role: 'assistant',
+          text: `Reload failed: ${e instanceof Error ? e.message : String(e)}`,
+          timestamp: Date.now(),
+        },
+      ]);
+    } finally {
+      setIsReloading(false);
+    }
+  }, [isReloading, onReload]);
+
+  return (
+    <View style={styles.container}>
+      <View style={styles.header}>
+        <Text style={styles.title}>Vibe</Text>
+        {onClose && (
+          <TouchableOpacity onPress={onClose} accessibilityLabel="Close vibe chat">
+            <Text style={styles.close}>✕</Text>
+          </TouchableOpacity>
+        )}
+      </View>
+
+      <ScrollView
+        ref={scrollRef}
+        style={styles.transcript}
+        contentContainerStyle={styles.transcriptContent}
+        keyboardShouldPersistTaps="handled"
+      >
+        {turns.map((turn) => (
+          <View
+            key={turn.id}
+            style={[
+              styles.turn,
+              turn.role === 'user' && styles.turnUser,
+              turn.role === 'assistant' && styles.turnAssistant,
+              turn.role === 'status' && styles.turnStatus,
+            ]}
+          >
+            <Text style={styles.turnText}>{turn.text}</Text>
+          </View>
+        ))}
+        {/* Live streaming buffer rendered as a single trailing
+            assistant block while the task is running. Once the task
+            terminates the stream is moved into a real turn (above)
+            and this block clears. */}
+        {streamBuffer && status === 'running' && (
+          <View style={[styles.turn, styles.turnAssistant]}>
+            <Text style={styles.turnText}>{streamBuffer}</Text>
+          </View>
+        )}
+        {status === 'running' && (
+          <View style={styles.spinnerRow}>
+            <ActivityIndicator size="small" color="#9ca3af" />
+            <Text style={styles.spinnerText}>working…</Text>
+          </View>
+        )}
+      </ScrollView>
+
+      <View style={styles.footer}>
+        <TextInput
+          style={styles.input}
+          value={followUp}
+          onChangeText={setFollowUp}
+          placeholder={status === 'running' ? 'wait for the agent…' : 'follow up…'}
+          placeholderTextColor="#666"
+          editable={status !== 'running' && !isResuming}
+          multiline
+        />
+        <View style={styles.actions}>
+          {onReload && (
+            <TouchableOpacity
+              style={[
+                styles.actionBtn,
+                styles.reloadBtn,
+                (isReloading || status === 'running') && styles.actionBtnDisabled,
+              ]}
+              onPress={handleReload}
+              disabled={isReloading || status === 'running'}
+            >
+              <Text style={styles.actionText}>
+                {isReloading ? 'reloading…' : '⟳ reload'}
+              </Text>
+            </TouchableOpacity>
+          )}
+          <TouchableOpacity
+            style={[
+              styles.actionBtn,
+              styles.sendBtn,
+              (isResuming || status === 'running' || !followUp.trim()) && styles.actionBtnDisabled,
+            ]}
+            onPress={handleSendFollowUp}
+            disabled={isResuming || status === 'running' || !followUp.trim()}
+          >
+            <Text style={styles.actionText}>
+              {isResuming ? '…' : '↑ send'}
+            </Text>
+          </TouchableOpacity>
+        </View>
+      </View>
+    </View>
+  );
+}
+
+const styles = StyleSheet.create({
+  container: { flex: 1, backgroundColor: '#0a0a0a' },
+  header: {
+    flexDirection: 'row',
+    alignItems: 'center',
+    justifyContent: 'space-between',
+    paddingHorizontal: 16,
+    paddingTop: 14,
+    paddingBottom: 8,
+    borderBottomWidth: 1,
+    borderBottomColor: 'rgba(255,255,255,0.08)',
+  },
+  title: { color: '#fff', fontSize: 17, fontWeight: '600' },
+  close: { color: '#9ca3af', fontSize: 18 },
+  transcript: { flex: 1 },
+  transcriptContent: { padding: 12, paddingBottom: 24 },
+  turn: {
+    marginVertical: 4,
+    padding: 10,
+    borderRadius: 12,
+    maxWidth: '92%',
+  },
+  turnUser: {
+    backgroundColor: '#7582f5',
+    alignSelf: 'flex-end',
+  },
+  turnAssistant: {
+    backgroundColor: 'rgba(255,255,255,0.06)',
+    borderColor: 'rgba(255,255,255,0.10)',
+    borderWidth: 1,
+    alignSelf: 'flex-start',
+  },
+  turnStatus: {
+    backgroundColor: 'transparent',
+    alignSelf: 'flex-start',
+    paddingHorizontal: 4,
+  },
+  turnText: { color: '#f1f5f9', fontSize: 14, lineHeight: 20 },
+  spinnerRow: {
+    flexDirection: 'row',
+    alignItems: 'center',
+    marginTop: 8,
+    paddingHorizontal: 4,
+  },
+  spinnerText: { color: '#9ca3af', fontSize: 12, marginLeft: 8 },
+  footer: {
+    borderTopWidth: 1,
+    borderTopColor: 'rgba(255,255,255,0.08)',
+    padding: 10,
+  },
+  input: {
+    minHeight: 40,
+    maxHeight: 120,
+    color: '#f1f5f9',
+    fontSize: 14,
+    backgroundColor: 'rgba(255,255,255,0.04)',
+    borderRadius: 10,
+    paddingHorizontal: 12,
+    paddingVertical: 8,
+  },
+  actions: {
+    flexDirection: 'row',
+    justifyContent: 'flex-end',
+    marginTop: 8,
+  },
+  actionBtn: {
+    paddingHorizontal: 14,
+    paddingVertical: 8,
+    borderRadius: 10,
+    marginLeft: 8,
+  },
+  actionBtnDisabled: { opacity: 0.5 },
+  reloadBtn: { backgroundColor: 'rgba(255,255,255,0.08)' },
+  sendBtn: { backgroundColor: '#7582f5' },
+  actionText: { color: '#fff', fontSize: 13, fontWeight: '600' },
+});

package/src/YaverFeedback.ts

@@ -815,6 +815,16 @@ export class YaverFeedback {
     return config;
   }
 
+  /** Returns the resolved relay password the SDK is currently using.
+   *  Empty string when no relay routing is in play (direct LAN agent
+   *  URLs need no password). Callers attaching it to relay-routed
+   *  HTTP requests must check for empty before setting the header,
+   *  since "X-Relay-Password: " is treated as invalid by the relay
+   *  and would 401 the request. */
+  static getRelayPassword(): string {
+    return p2pRelayPassword;
+  }
+
   /**
    * Manually attach an error with optional metadata.
    * Use this in catch blocks to give the agent extra context.
@@ -1040,7 +1050,7 @@ export class YaverFeedback {
         errors: errorBuffer.length > 0 ? [...errorBuffer] : undefined,
       };
 
-      await uploadFeedback(config.agentUrl, config.authToken ?? '', bundle);
+      await uploadFeedback(config.agentUrl, config.authToken ?? '', bundle, p2pRelayPassword);
       console.log('[YaverFeedback] Auto-report sent');
     } catch (err) {
       console.warn('[YaverFeedback] Auto-report failed:', err);

package/src/_core/buildFeedbackPrompt.ts

@@ -0,0 +1,102 @@
+// buildFeedbackPrompt — shared prompt enrichment used by every Yaver
+// feedback surface, in-Yaver native pane (mirrored in Swift + Kotlin)
+// AND the standalone RN feedback SDK (this file). Keep all three
+// implementations in lockstep — the wording is what the AI on the
+// remote is conditioned to expect.
+//
+// The bare user text on its own loses crucial context: WHICH app the
+// user is testing, WHICH screen they're looking at, and whether a
+// screenshot is attached for visual reference. Without that the agent
+// guesses, edits the wrong project, or asks clarifying questions
+// instead of acting. The wrapper below tells the agent:
+//   - this feedback comes from the in-app drawer while the user is
+//     mid-test,
+//   - which project the user is in (when known),
+//   - that the FIRST attached image (when present) is a snapshot of
+//     the current screen — open it to see what the user is pointing
+//     at,
+//   - that changes should be applied to that project's source +
+//     saved so the user can trigger a Hermes reload to see them.
+//
+// Cross-reference: mobile/ios/Yaver/YaverFeedbackPane.swift's
+// `buildFeedbackPrompt` and mobile/android/.../YaverFeedbackPane.kt's
+// `buildFeedbackPrompt`. All three must match.
+
+export interface BuildFeedbackPromptInput {
+  userPrompt: string;
+  /** Hot-Reload project name when running inside Yaver mobile, OR
+   *  the host app's bundle/package name when running standalone. */
+  projectName?: string;
+  /** Absolute path on the host where the project lives (only known
+   *  when running inside Yaver mobile via Hot Reload). */
+  projectPath?: string;
+  /** True when the caller has attached a screenshot of the current
+   *  screen as the first image in the task's images array. */
+  hasScreenshot: boolean;
+}
+
+export function buildFeedbackPrompt(input: BuildFeedbackPromptInput): string {
+  const userPrompt = input.userPrompt ?? "";
+  const projectName = (input.projectName ?? "").trim();
+  const projectPath = (input.projectPath ?? "").trim();
+  const hasScreenshot = !!input.hasScreenshot;
+
+  const lines: string[] = [];
+  lines.push("[Mobile feedback from inside Yaver]");
+  lines.push(
+    "The user is providing this feedback while running a mobile app inside the Yaver mobile container " +
+      "and is currently looking at a specific screen of that app."
+  );
+  lines.push("");
+  if (projectName || projectPath) {
+    lines.push("App being tested:");
+    if (projectName) lines.push(`  name: ${projectName}`);
+    if (projectPath) lines.push(`  path: ${projectPath}`);
+    lines.push("");
+  }
+  if (hasScreenshot) {
+    lines.push(
+      "A screenshot of the current screen is attached as the first image. " +
+        "Open it before deciding what to change — the user is pointing at what they SEE, " +
+        "not necessarily what is named most prominently in the source."
+    );
+    lines.push("");
+  } else {
+    lines.push("(The user chose not to attach a screenshot for this round.)");
+    lines.push("");
+  }
+  lines.push("Operation contract:");
+  lines.push(
+    "1. Locate the file(s) responsible for what the user described and EDIT them in place. " +
+      "Save the changes — that is the deliverable."
+  );
+  lines.push(
+    "2. Stream a CONCISE Claude-Code / Codex-style narration as you work: " +
+      "one short line per step (e.g. \"Reading app/index.tsx\", " +
+      "\"Editing safe.backgroundColor\", \"Saved app/index.tsx\"). Show small diffs only — " +
+      "never dump entire files, never paste node_modules contents, never echo build / install logs."
+  );
+  lines.push(
+    "3. Do NOT run npm install / yarn / pnpm / git clone / cargo build / docker pull or any other " +
+      "long-running install / fetch command. The repo is already prepared on this machine. " +
+      "If a dependency is genuinely missing, say so in one line and stop — the user will install it."
+  );
+  lines.push(
+    "4. Do NOT trigger a Hermes reload yourself. The user has a Reload button in the drawer " +
+      "and decides when to refresh."
+  );
+  lines.push(
+    "5. Keep total output under a few hundred lines. Heavy ripgrep / find / cat with no filter " +
+      "are usually the wrong tool — use targeted reads."
+  );
+  if (!projectName && !projectPath) {
+    lines.push(
+      "6. If you can identify the project from the prompt or the screenshot, work there. " +
+        "Otherwise ask the user briefly which project to target — one short line, no exhaustive list."
+    );
+  }
+  lines.push("");
+  lines.push("User feedback:");
+  lines.push(userPrompt);
+  return lines.join("\n");
+}

package/src/capture.ts

@@ -38,6 +38,37 @@ export async function captureScreenshot(): Promise<string> {
   }
 }
 
+/**
+ * Capture a screenshot AND return it as base64 + mime type, ready to
+ * embed in a `/tasks` payload's `images` array. Used by the converged
+ * vibe-feedback flow (FeedbackModal → P2PClient.createFeedbackTask).
+ *
+ * Returns null when capture isn't possible (peer dep missing / user
+ * permission denied / running in a context where view-shot can't
+ * grab the screen). Caller should treat null as "send without
+ * screenshot" rather than aborting the whole feedback.
+ */
+export async function captureScreenshotBase64(): Promise<{
+  base64: string;
+  mimeType: string;
+} | null> {
+  try {
+    const ViewShot = require('react-native-view-shot');
+    const result = await ViewShot.captureScreen({
+      format: 'jpg',
+      quality: 0.7,
+      result: 'base64',
+    });
+    if (typeof result === 'string' && result.length > 0) {
+      // ViewShot returns a bare base64 string (no `data:` prefix).
+      return { base64: result, mimeType: 'image/jpeg' };
+    }
+    return null;
+  } catch {
+    return null;
+  }
+}
+
 export interface PickedFeedbackFile {
   path: string;
   name: string;

package/src/preferences.ts

@@ -149,3 +149,59 @@ export async function setQuickIconColorPreset(
 export async function clearQuickIconColorPreset(): Promise<void> {
   await setQuickIconColorPreset(null);
 }
+
+// ── Preferred coding agent + model (used by the standalone feedback
+// SDK's vibe chat to mirror what Yaver mobile's Tasks tab would send.
+// The agent on the remote DOES read userSettings.primaryRunnerByDevice
+// from Convex, but the standalone SDK has no DeviceContext to push the
+// per-device pick. We persist the user's last choice locally; first
+// run picks whatever's signed-in via getRunnerStatus().)
+
+const PREFERRED_RUNNER_KEY = 'yaver_feedback_preferred_runner';
+const PREFERRED_MODEL_KEY = 'yaver_feedback_preferred_model';
+
+export async function getPreferredRunner(): Promise<string | null> {
+  if (!AsyncStorage) return null;
+  try {
+    const v = await AsyncStorage.getItem(PREFERRED_RUNNER_KEY);
+    return v && v.trim() ? v.trim() : null;
+  } catch {
+    return null;
+  }
+}
+
+export async function setPreferredRunner(runner: string | null): Promise<void> {
+  if (!AsyncStorage) return;
+  try {
+    if (!runner || !runner.trim()) {
+      await AsyncStorage.removeItem(PREFERRED_RUNNER_KEY);
+      return;
+    }
+    await AsyncStorage.setItem(PREFERRED_RUNNER_KEY, runner.trim());
+  } catch {
+    /* best-effort */
+  }
+}
+
+export async function getPreferredModel(): Promise<string | null> {
+  if (!AsyncStorage) return null;
+  try {
+    const v = await AsyncStorage.getItem(PREFERRED_MODEL_KEY);
+    return v && v.trim() ? v.trim() : null;
+  } catch {
+    return null;
+  }
+}
+
+export async function setPreferredModel(model: string | null): Promise<void> {
+  if (!AsyncStorage) return;
+  try {
+    if (!model || !model.trim()) {
+      await AsyncStorage.removeItem(PREFERRED_MODEL_KEY);
+      return;
+    }
+    await AsyncStorage.setItem(PREFERRED_MODEL_KEY, model.trim());
+  } catch {
+    /* best-effort */
+  }
+}

package/src/types.ts

@@ -123,6 +123,15 @@ export interface FeedbackConfig {
   preferredDeviceId?: string;
   /** How feedback collection is triggered */
   trigger?: 'shake' | 'floating-button' | 'manual';
+  /**
+   * App slug used by the in-modal Deploy panel when calling the agent's
+   * `/fleet/deploy-options` and `/deploy/ship` endpoints. Should match an
+   * `apps[].name` entry in the agent's `yaver.workspace.yaml`. When omitted
+   * the panel falls back to the last dot-segment of `bundleId` (e.g.
+   * `io.yaver.sfmg` → `sfmg`). Set explicitly when the workspace name
+   * differs from the bundleId tail.
+   */
+  deployAppSlug?: string;
   /**
    * Non-default escape hatch for host apps that want the SDK without
    * shake gesture handling. When enabled:

package/src/upload.ts

@@ -18,6 +18,7 @@ export async function uploadFeedback(
   agentUrl: string,
   authToken: string,
   bundle: FeedbackBundle,
+  relayPassword: string = '',
 ): Promise<{ id?: string; reportId?: string; [k: string]: unknown }> {
   const formData = new FormData();
 
@@ -54,11 +55,15 @@ export async function uploadFeedback(
 
   const url = agentUrl.replace(/\/$/, '') + '/feedback';
 
+  const headers: Record<string, string> = {
+    Authorization: `Bearer ${authToken}`,
+  };
+  if (relayPassword) {
+    headers['X-Relay-Password'] = relayPassword;
+  }
   const response = await fetch(url, {
     method: 'POST',
-    headers: {
-      Authorization: `Bearer ${authToken}`,
-    },
+    headers,
     body: formData,
   });