sparkecoder 0.1.110 → 0.1.112

package/web/.next/standalone/web/src/app/(main)/page.tsx

@@ -172,8 +172,42 @@ function HomeContent() {
     return () => window.removeEventListener('keydown', handleKeyDown);
   }, [creating, sessions.length, config]);
 
-  // Show API key setup if not configured
-  if (config && !config.apiKeyConfigured) {
+  // Distinguish three states:
+  //   1. config is null → still loading (handled below the early returns)
+  //   2. !serverConnected → API unreachable (wrong apiUrl, server down, etc).
+  //   3. serverConnected && !apiKeyConfigured → local AI key needed.
+  //
+  // In remote-inference mode the server reports apiKeyConfigured=true even
+  // without a local Gateway key (the remote-server has the key), so the
+  // prompt should NOT show. The original check (`!apiKeyConfigured`) was
+  // correct for that, but treated state #2 (unreachable) identically to #3.
+  if (config && !config.serverConnected) {
+    return (
+      <div className="flex items-center justify-center h-full bg-gradient-to-b from-background to-muted/20">
+        <div className="text-center max-w-lg mx-auto p-8">
+          <div className="flex justify-center mb-8">
+            <div className="relative size-28 rounded-2xl overflow-hidden shadow-xl ring-1 ring-white/10">
+              <Image src="/sparke-coder.png" alt="Sparke" fill className="object-cover" priority />
+            </div>
+          </div>
+          <h1 className="text-3xl font-bold mb-3 text-primary">SparkECoder API unreachable</h1>
+          <div className="bg-muted/50 rounded-lg p-6 mb-6 text-left">
+            <p className="text-sm text-muted-foreground mb-2">
+              The browser couldn&apos;t reach the SparkECoder API at <code className="px-1 py-0.5 rounded bg-muted text-[11px] break-all">{config.apiBaseUrl}</code>.
+            </p>
+            <p className="text-sm text-muted-foreground mb-2">Common causes:</p>
+            <ul className="text-sm text-muted-foreground list-disc pl-5 space-y-1">
+              <li>The auto-detected URL is wrong for this hostname. Append <code className="px-1 py-0.5 rounded bg-muted text-[11px]">?apiUrl=https://your-api-host</code> to the page URL to override (saves to localStorage).</li>
+              <li>Cloudflared tunnel down or DNS not yet propagated.</li>
+              <li>Sparkecoder service stopped — run <code className="px-1 py-0.5 rounded bg-muted text-[11px]">launchctl list | grep sparkecoder</code>.</li>
+            </ul>
+          </div>
+        </div>
+      </div>
+    );
+  }
+  // Show API key setup only when server is connected but no AI key is configured.
+  if (config && config.serverConnected && !config.apiKeyConfigured) {
     return (
       <div className="flex items-center justify-center h-full bg-gradient-to-b from-background to-muted/20">
         <div className="text-center max-w-lg mx-auto p-8">

package/web/.next/standalone/web/src/app/(main)/settings/page.tsx

@@ -368,6 +368,7 @@ function SlackCard({ data, onChange }: { data: IntegrationsResponse; onChange: (
       {showSetup && (
         <SlackSetupModal
           eventsUrl={data.slack.eventsUrl}
+          agentName={data.slack.defaultOrchestratorName || 'orchestrator'}
           onClose={() => setShowSetup(false)}
         />
       )}
@@ -491,9 +492,9 @@ function SlackAllowlistEditor({ data, onChange }: { data: IntegrationsResponse;
   );
 }
 
-function SlackSetupModal({ eventsUrl, onClose }: { eventsUrl: string; onClose: () => void }) {
+function SlackSetupModal({ eventsUrl, agentName, onClose }: { eventsUrl: string; agentName: string; onClose: () => void }) {
   const [copied, setCopied] = useState<string | null>(null);
-  const manifest = buildSlackManifest(eventsUrl);
+  const manifest = buildSlackManifest(eventsUrl, agentName);
   const copy = async (key: string, text: string) => {
     await navigator.clipboard.writeText(text);
     setCopied(key); setTimeout(() => setCopied(null), 1500);
@@ -566,39 +567,79 @@ function CopyBlock({ label, value, onCopy, copied, mono }: { label: string; valu
   );
 }
 
-function buildSlackManifest(eventsUrl: string): string {
-  // Trim trailing whitespace; Slack rejects manifests with weird indents.
-  return `display_information:
-  name: sparkecoder
-  description: Your orchestrator agent. Spawns worker agents, posts updates, answers questions.
-  background_color: "#1f1f24"
-features:
-  bot_user:
-    display_name: sparkecoder
-    always_online: true
-oauth_config:
-  scopes:
-    bot:
-      - app_mentions:read
-      - channels:history
-      - chat:write
-      - groups:history
-      - im:history
-      - im:read
-      - im:write
-      - users:read
-settings:
-  event_subscriptions:
-    request_url: ${eventsUrl}
-    bot_events:
-      - app_mention
-      - message.im
-  interactivity:
-    is_enabled: false
-  org_deploy_enabled: false
-  socket_mode_enabled: false
-  token_rotation_enabled: false
-`;
+function buildSlackManifest(eventsUrl: string, agentName: string): string {
+  // Slack constraints:
+  //   display_information.name:    1-35 chars, used as the app's
+  //                                listing name. Visible spaces OK
+  //                                but we keep it safe / app-store-ish.
+  //   bot_user.display_name:       1-80 chars, what users @-mention.
+  //                                Spaces NOT allowed; Slack rejects.
+  // We slug-ify the user-provided agent name for both, preserving case
+  // for display_information.name (humans see it) but stripping spaces
+  // / punctuation from bot_user.display_name (Slack rule).
+  const safe = (agentName || 'orchestrator').trim() || 'orchestrator';
+  const displayName = safe.slice(0, 35);
+  const botName = safe
+    .replace(/[^A-Za-z0-9_-]+/g, '-') // Slack: alphanumerics, _ and - only
+    .replace(/^-+|-+$/g, '')
+    .slice(0, 80) || 'orchestrator';
+
+  // JSON manifest. Slack accepts both YAML and JSON, but JSON parses
+  // more reliably across browser clipboards (no whitespace surprises).
+  const manifest = {
+    display_information: {
+      name: displayName,
+      description: `${displayName} — your orchestrator agent. Spawns worker agents, posts updates, answers questions.`,
+      background_color: '#1f1f24',
+    },
+    features: {
+      bot_user: {
+        display_name: botName,
+        always_online: true,
+      },
+      // Required for users to DM the app. Without these, Slack disables
+      // the message composer and shows "Sending messages to this app
+      // has been turned off".
+      app_home: {
+        home_tab_enabled: false,
+        messages_tab_enabled: true,
+        messages_tab_read_only_enabled: false,
+      },
+    },
+    oauth_config: {
+      scopes: {
+        bot: [
+          'app_mentions:read',
+          'channels:history',
+          'chat:write',
+          'groups:history',
+          'im:history',
+          'im:read',
+          'im:write',
+          'users:read',
+        ],
+      },
+    },
+    settings: {
+      event_subscriptions: {
+        request_url: eventsUrl,
+        // app_mention      → user @-pings the bot in a channel
+        // message.im       → user DMs the bot (every message in DM is a reply)
+        // message.channels → message in a public channel the bot is in.
+        //                    Used so thread replies (no re-mention needed)
+        //                    keep reaching the bot. Server-side filters
+        //                    these to ONLY messages with thread_ts that
+        //                    match a session the bot is participating in.
+        // message.groups   → same, but for private channels.
+        bot_events: ['app_mention', 'message.im', 'message.channels', 'message.groups'],
+      },
+      interactivity: { is_enabled: false },
+      org_deploy_enabled: false,
+      socket_mode_enabled: false,
+      token_rotation_enabled: false,
+    },
+  };
+  return JSON.stringify(manifest, null, 2);
 }
 
 function CloudflaredCard({ data }: { data: IntegrationsResponse }) {

package/web/.next/standalone/web/src/components/chat-interface.tsx

@@ -1724,6 +1724,40 @@ export function ChatInterface({ session, isEmbed = false }: ChatInterfaceProps)
           });
           cancelRef.current = cancel;
         }
+
+        // Also subscribe to coarse-grained session updates (Slack /
+        // inbox / schedule-triggered daemon turns). These don't go
+        // through the resumable-stream pipeline above, so without this
+        // the UI would only see new messages after a refresh.
+        try {
+          const { getApiUrl } = await import('@/lib/config');
+          const updatesUrl = `${getApiUrl()}/sessions/${session.id}/updates`;
+          const es = new EventSource(updatesUrl);
+          es.onmessage = async (msg) => {
+            try {
+              const payload = JSON.parse(msg.data) as { type: string };
+              if (payload.type === 'turn-complete') {
+                const fresh = await getSessionMessages(session.id);
+                if (isStale) return;
+                const sortedFresh = [...fresh].sort((a, b) =>
+                  new Date(a.createdAt).getTime() - new Date(b.createdAt).getTime(),
+                );
+                setChatItems(convertApiMessages(sortedFresh));
+              }
+            } catch { /* malformed event — ignore */ }
+          };
+          es.onerror = () => {
+            // Browser will auto-reconnect; nothing to do here.
+          };
+          // Cancel on session change / unmount.
+          const prevCancel = cancelRef.current;
+          cancelRef.current = () => {
+            prevCancel?.();
+            es.close();
+          };
+        } catch (err) {
+          console.warn('Failed to open session-updates EventSource:', err);
+        }
       } catch (err) {
         if (!isStale) {
           console.error('Failed to load messages:', err);