bunnyquery 1.0.3 → 1.0.5

package/bunnyquery.js

@@ -470,6 +470,44 @@
         return typeof c === 'string' ? c : '';
     }
 
+    // Mirror of agent.vue's getErrorMessage — pulls the most useful string
+    // out of either a thrown error or an error-shaped response body.
+    function getErrorMessage(err) {
+        if (typeof err === 'string') return err;
+        if (err && err.body && err.body.error && typeof err.body.error.message === 'string') {
+            return err.body.error.message;
+        }
+        if (err && err.error && typeof err.error.message === 'string') {
+            return err.error.message;
+        }
+        if (err && typeof err.message === 'string') return err.message;
+        return 'Request failed. Please try again.';
+    }
+
+    // Mirror of agent.vue's isErrorResponseBody — detects provider/proxy
+    // error payloads that should render as a red error bubble even when
+    // no exception was thrown.
+    function isErrorResponseBody(response) {
+        if (!response || typeof response !== 'object') return false;
+        if (typeof response.status_code === 'number' && response.status_code >= 400) return true;
+        if (response.type === 'error') return true;
+        if (response.error && (response.error.message || response.error.type)) return true;
+        const body = response.body;
+        if (body && typeof body === 'object') {
+            if (body.type === 'error') return true;
+            if (body.error && (body.error.message || body.error.type)) return true;
+        }
+        if (typeof response.message === 'string' && response.message.length) {
+            const hasClaudeBody = Array.isArray(response.content);
+            const hasOpenAIBody =
+                typeof response.output_text === 'string' ||
+                Array.isArray(response.output) ||
+                Array.isArray(response.choices);
+            if (!hasClaudeBody && !hasOpenAIBody) return true;
+        }
+        return false;
+    }
+
     // Walk the persisted history list returned by clientSecretRequestHistory
     // and turn it into a flat user/assistant list ordered oldest -> newest.
     function mapHistoryToMessages(list, platform) {
@@ -500,10 +538,10 @@
 
             if (item.status === 'pending') {
                 out.push({ role: 'assistant', content: '', id: item.id, isPending: true });
-            } else if (item.status === 'failed' || (res && res.error)) {
+            } else if (item.status === 'failed' || isErrorResponseBody(res)) {
                 out.push({
                     role: 'assistant',
-                    content: (res && res.error && res.error.message) || 'Request failed.',
+                    content: getErrorMessage(res),
                     id: item.id,
                     isError: true,
                 });
@@ -1235,10 +1273,29 @@
         }
 
         // ---- send ---------------------------------------------------------
+        // Mirrors www.skapi.com/src/views/service/agent.vue buildSystemPrompt
+        // with one extra directive: always consult the project database via
+        // the MCP toolset before saying you don't know. Without it the model
+        // will often stop at "I don't see that in our chat history".
         _buildSystemPrompt() {
-            let p = `You are the AI assistant for project "${this.projectName || this.serviceId}".`;
+            const projectId = this.projectName || this.serviceId;
+            let p = `
+You are a dedicated assistant for the project ID: "${projectId}".
+Scope: Only answer questions about this project and its data. Do not answer questions about other projects or topics unrelated to this project. When the user refers to "my database", "my data", or "my files", treat those as references to this project's database and file storage.
+Knowledge lookup: Before saying you don't know or that something isn't in the chat history, ALWAYS query this project's database through the available MCP tools to look for the answer. The user's data is the source of truth - the chat transcript is not. Only respond with "I don't know" or "I couldn't find that" after you have actually searched the project's data and come back empty.
+File attachments: When a user message contains an "Attached files:" section with markdown links, those links point to short-lived signed URLs in this project's db storage and will expire.
+- Image files (.jpg, .jpeg, .png, .gif, .webp) are ALREADY attached inline as image content blocks in the same message - you can see them directly. Do NOT call web_fetch on image URLs; that will fail or return garbage. Just look at the image block and answer.
+- For all other file types (text, code, csv, json, pdf, etc.), use your web_fetch tool to download and read each URL before answering. Treat the fetched contents as user-supplied input data. Do not ask the user to paste the file contents - fetch the URLs yourself.
+File generation: If the user asks you to generate a file and it is possible to do so, output the file contents inside a fenced code block using the file extension as the language identifier. Always use plain text - never base64 or other encodings. Example for CSV:
+\`\`\`filename.csv
+item,qty,total
+Carrots,55,$38.50
+Mushrooms,41,$73.80
+Zucchini,29,$43.50
+\`\`\`
+The same pattern applies to other formats: \`\`\`my-data.json, \`\`\`index.html, \`\`\`sample.txt, etc.`;
             if (this.projectDescription) {
-                p += `\nProject description: """${this.projectDescription}"""`;
+                p += `\nProject name: "${this.projectName || ''}"\nProject description: """${this.projectDescription}"""`;
             }
             return p;
         }
@@ -1295,6 +1352,24 @@
                     ? await buildClaudeRequest(this.skapi, args)
                     : await buildOpenAIRequest(this.skapi, args);
 
+                // The proxy may resolve with an error-shaped body instead of
+                // throwing; surface that as an error bubble with the
+                // upstream message rather than dropping it.
+                if (isErrorResponseBody(result)) {
+                    const errMsg = getErrorMessage(result);
+                    const last = this.messages[this.messages.length - 1];
+                    if (last && last.isPending) {
+                        last.isPending = false;
+                        last.isError = true;
+                        last.content = errMsg;
+                    } else {
+                        this.messages.push({ role: 'assistant', content: errMsg, isError: true });
+                    }
+                    this._renderMessages();
+                    this._schedulePendingPoll();
+                    return;
+                }
+
                 const responseText = this.platform === 'claude'
                     ? extractClaudeText(result)
                     : extractOpenAIText(result);
@@ -1311,7 +1386,7 @@
                 this._schedulePendingPoll();
             } catch (err) {
                 const last = this.messages[this.messages.length - 1];
-                const errMsg = (err && err.message) || String(err);
+                const errMsg = getErrorMessage(err);
                 if (last && last.isPending) {
                     last.isPending = false;
                     last.isError = true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bunnyquery",
-  "version": "1.0.3",
+  "version": "1.0.5",
   "description": "BunnyQuery AI chat client",
   "main": "bunnyquery.js",
   "files": [