@kapeta/local-cluster-service 0.71.0 → 0.71.2

package/CHANGELOG.md

@@ -1,3 +1,18 @@
+## [0.71.2](https://github.com/kapetacom/local-cluster-service/compare/v0.71.1...v0.71.2) (2024-09-18)
+
+
+### Bug Fixes
+
+* send initial screen errors too ([#253](https://github.com/kapetacom/local-cluster-service/issues/253)) ([386dabb](https://github.com/kapetacom/local-cluster-service/commit/386dabb0acb07384691242ecf4cc3081e0d04a05))
+
+## [0.71.1](https://github.com/kapetacom/local-cluster-service/compare/v0.71.0...v0.71.1) (2024-09-17)
+
+
+### Bug Fixes
+
+* emit all events except chunks from screen endpoints ([8e67b58](https://github.com/kapetacom/local-cluster-service/commit/8e67b58b6cc70c359556edfc61dd904254f9d3bc))
+* replace fetch retry handler library to avoid fetch failed ([59a4342](https://github.com/kapetacom/local-cluster-service/commit/59a4342203cd8efb706beba254284ce321fe313a))
+
 # [0.71.0](https://github.com/kapetacom/local-cluster-service/compare/v0.70.12...v0.71.0) (2024-09-16)
 
 

package/dist/cjs/src/storm/PageGenerator.js

@@ -268,7 +268,7 @@ class PageQueue extends node_events_1.EventEmitter {
         });
         await screenStream.waitForDone();
         if (!pageEvent) {
-            throw new Error('No page was generated');
+            throw new Error('No page was generated for ' + prompt.name);
         }
         await this.processPageEventWithReferences(pageEvent);
     }

package/dist/cjs/src/storm/routes.js

@@ -129,6 +129,12 @@ router.post('/ui/screen', async (req, res) => {
             console.error('Failed to process page', err);
             sendError(err, res);
         });
+        queue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
+        });
         await queue.addPrompt(aiRequest, conversationId, true);
         await queue.wait();
         await Promise.allSettled(promises);
@@ -207,8 +213,11 @@ router.post('/:handle/ui/iterative', async (req, res) => {
             pageQueue.cancel();
         });
         pageQueue.on('page', (screenData) => sendPageEvent(landingPagesStream.getConversationId(), screenData, res));
-        pageQueue.on('event', (screenData) => {
-            sendEvent(res, screenData);
+        pageQueue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
         });
         pageQueue.on('error', (err) => {
             console.error('Failed to process page', err);
@@ -360,8 +369,11 @@ router.post('/:handle/ui', async (req, res) => {
             queue.cancel();
         });
         queue.on('page', (screenData) => sendPageEvent(outerConversationId, screenData, res));
-        queue.on('event', (screenData) => {
-            sendEvent(res, screenData);
+        queue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
         });
         queue.on('error', (err) => {
             console.error('Failed to process page', err);
@@ -382,6 +394,7 @@ router.post('/:handle/ui', async (req, res) => {
             })
                 .catch((e) => {
                 console.error('Failed to generate page for screen %s', screen.name, e);
+                sendError(e, res);
             });
         }
         if (userJourneysStream.isAborted()) {
@@ -417,10 +430,11 @@ router.post('/ui/edit', async (req, res) => {
                 return promise;
             }
         });
-        queue.on('event', (data) => {
-            if (data.type === 'FILE_START' || data.type === 'FILE_DONE' || data.type === 'FILE_STATE') {
-                sendEvent(res, data);
+        queue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
             }
+            sendEvent(res, event);
         });
         queue.on('error', (err) => {
             console.error('Failed to process page', err);

package/dist/cjs/src/storm/stormClient.d.ts

@@ -1,3 +1,4 @@
+/// <reference types="node" />
 import { ConversationItem, ImplementAPIClientsRequest, StormFileImplementationPrompt, StormStream, StormUIImplementationPrompt, StormUIListPrompt } from './stream';
 import { Page, StormEventPageUrl } from './events';
 export declare const STORM_ID = "storm";
@@ -65,7 +66,7 @@ declare class StormClient {
     createUIShells(prompt: UIShellsPrompt, conversationId?: string): Promise<StormStream>;
     createUILandingPages(prompt: BasePromptRequest, conversationId?: string): Promise<StormStream>;
     createUIPage(prompt: UIPagePrompt, conversationId?: string, history?: ConversationItem[]): Promise<StormStream>;
-    voteUIPage(topic: string, conversationId: string, vote: -1 | 0 | 1, mainConversationId?: string): Promise<import("undici").Response>;
+    voteUIPage(topic: string, conversationId: string, vote: -1 | 0 | 1, mainConversationId?: string): Promise<Response>;
     getVoteUIPage(topic: string, conversationId: string, mainConversationId?: string): Promise<{
         vote: -1 | 0 | 1;
     }>;

package/dist/cjs/src/storm/stormClient.js

@@ -13,22 +13,8 @@ const utils_1 = require("../utils/utils");
 const promises_1 = __importDefault(require("node:readline/promises"));
 const node_stream_1 = require("node:stream");
 const stream_1 = require("./stream");
-const undici_1 = require("undici");
-// Will only retry on error codes and GET requests by default
-// See https://github.com/nodejs/undici/blob/990df2c7e37cbe5bb44fe2f576dddeaeb5916590/docs/docs/api/RetryAgent.md
-const retryAgent = new undici_1.RetryAgent(new undici_1.Agent(), {
-    methods: [
-        // Added methods ↓ (not idempotent), but we want to retry on POST:
-        'POST',
-        // defaults below
-        'GET',
-        'HEAD',
-        'OPTIONS',
-        'PUT',
-        'DELETE',
-        'TRACE',
-    ],
-});
+const fetch_retry_1 = __importDefault(require("fetch-retry"));
+const fetchWithRetries = (0, fetch_retry_1.default)(global.fetch, { retries: 5, retryDelay: 10 });
 exports.STORM_ID = 'storm';
 exports.ConversationIdHeader = 'Conversation-Id';
 class StormClient {
@@ -54,7 +40,6 @@ class StormClient {
             method: method,
             body: JSON.stringify(body),
             headers,
-            dispatcher: retryAgent,
         };
     }
     async send(path, body, method = 'POST') {
@@ -65,7 +50,7 @@ class StormClient {
         });
         const abort = new AbortController();
         options.signal = abort.signal;
-        const response = await (0, undici_1.fetch)(options.url, options);
+        const response = await fetchWithRetries(options.url, options);
         if (response.status !== 200) {
             throw new Error(`Got error response from ${options.url}: ${response.status}\nContent: ${await response.text()}`);
         }
@@ -138,19 +123,19 @@ class StormClient {
             prompt: JSON.stringify({ topic, vote, mainConversationId }),
             conversationId,
         });
-        return (0, undici_1.fetch)(options.url, options);
+        return fetch(options.url, options);
     }
     async getVoteUIPage(topic, conversationId, mainConversationId) {
         const options = await this.createOptions('/v2/ui/get-vote', 'POST', {
             prompt: JSON.stringify({ topic, mainConversationId }),
             conversationId,
         });
-        const response = await (0, undici_1.fetch)(options.url, options);
+        const response = await fetch(options.url, options);
         return response.json();
     }
     async implementAPIClients(prompt) {
         const u = `${this._baseUrl}/v2/ui/implement-api-clients`;
-        const response = await (0, undici_1.fetch)(u, {
+        const response = await fetch(u, {
             method: 'POST',
             body: JSON.stringify({
                 fileName: prompt.fileName,
@@ -162,7 +147,7 @@ class StormClient {
     }
     async generatePrompt(pages) {
         const u = `${this._baseUrl}/v2/ui/prompt`;
-        const response = await (0, undici_1.fetch)(u, {
+        const response = await fetch(u, {
             method: 'POST',
             body: JSON.stringify({
                 pages: pages,
@@ -235,7 +220,7 @@ class StormClient {
             prompt: '',
             conversationId: conversationId,
         });
-        const response = await (0, undici_1.fetch)(options.url, options);
+        const response = await fetch(options.url, options);
         return response.text();
     }
 }

package/dist/esm/src/storm/PageGenerator.js

@@ -268,7 +268,7 @@ class PageQueue extends node_events_1.EventEmitter {
         });
         await screenStream.waitForDone();
         if (!pageEvent) {
-            throw new Error('No page was generated');
+            throw new Error('No page was generated for ' + prompt.name);
         }
         await this.processPageEventWithReferences(pageEvent);
     }

package/dist/esm/src/storm/routes.js

@@ -129,6 +129,12 @@ router.post('/ui/screen', async (req, res) => {
             console.error('Failed to process page', err);
             sendError(err, res);
         });
+        queue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
+        });
         await queue.addPrompt(aiRequest, conversationId, true);
         await queue.wait();
         await Promise.allSettled(promises);
@@ -207,8 +213,11 @@ router.post('/:handle/ui/iterative', async (req, res) => {
             pageQueue.cancel();
         });
         pageQueue.on('page', (screenData) => sendPageEvent(landingPagesStream.getConversationId(), screenData, res));
-        pageQueue.on('event', (screenData) => {
-            sendEvent(res, screenData);
+        pageQueue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
         });
         pageQueue.on('error', (err) => {
             console.error('Failed to process page', err);
@@ -360,8 +369,11 @@ router.post('/:handle/ui', async (req, res) => {
             queue.cancel();
         });
         queue.on('page', (screenData) => sendPageEvent(outerConversationId, screenData, res));
-        queue.on('event', (screenData) => {
-            sendEvent(res, screenData);
+        queue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
         });
         queue.on('error', (err) => {
             console.error('Failed to process page', err);
@@ -382,6 +394,7 @@ router.post('/:handle/ui', async (req, res) => {
             })
                 .catch((e) => {
                 console.error('Failed to generate page for screen %s', screen.name, e);
+                sendError(e, res);
             });
         }
         if (userJourneysStream.isAborted()) {
@@ -417,10 +430,11 @@ router.post('/ui/edit', async (req, res) => {
                 return promise;
             }
         });
-        queue.on('event', (data) => {
-            if (data.type === 'FILE_START' || data.type === 'FILE_DONE' || data.type === 'FILE_STATE') {
-                sendEvent(res, data);
+        queue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
             }
+            sendEvent(res, event);
         });
         queue.on('error', (err) => {
             console.error('Failed to process page', err);

package/dist/esm/src/storm/stormClient.d.ts

@@ -1,3 +1,4 @@
+/// <reference types="node" />
 import { ConversationItem, ImplementAPIClientsRequest, StormFileImplementationPrompt, StormStream, StormUIImplementationPrompt, StormUIListPrompt } from './stream';
 import { Page, StormEventPageUrl } from './events';
 export declare const STORM_ID = "storm";
@@ -65,7 +66,7 @@ declare class StormClient {
     createUIShells(prompt: UIShellsPrompt, conversationId?: string): Promise<StormStream>;
     createUILandingPages(prompt: BasePromptRequest, conversationId?: string): Promise<StormStream>;
     createUIPage(prompt: UIPagePrompt, conversationId?: string, history?: ConversationItem[]): Promise<StormStream>;
-    voteUIPage(topic: string, conversationId: string, vote: -1 | 0 | 1, mainConversationId?: string): Promise<import("undici").Response>;
+    voteUIPage(topic: string, conversationId: string, vote: -1 | 0 | 1, mainConversationId?: string): Promise<Response>;
     getVoteUIPage(topic: string, conversationId: string, mainConversationId?: string): Promise<{
         vote: -1 | 0 | 1;
     }>;

package/dist/esm/src/storm/stormClient.js

@@ -13,22 +13,8 @@ const utils_1 = require("../utils/utils");
 const promises_1 = __importDefault(require("node:readline/promises"));
 const node_stream_1 = require("node:stream");
 const stream_1 = require("./stream");
-const undici_1 = require("undici");
-// Will only retry on error codes and GET requests by default
-// See https://github.com/nodejs/undici/blob/990df2c7e37cbe5bb44fe2f576dddeaeb5916590/docs/docs/api/RetryAgent.md
-const retryAgent = new undici_1.RetryAgent(new undici_1.Agent(), {
-    methods: [
-        // Added methods ↓ (not idempotent), but we want to retry on POST:
-        'POST',
-        // defaults below
-        'GET',
-        'HEAD',
-        'OPTIONS',
-        'PUT',
-        'DELETE',
-        'TRACE',
-    ],
-});
+const fetch_retry_1 = __importDefault(require("fetch-retry"));
+const fetchWithRetries = (0, fetch_retry_1.default)(global.fetch, { retries: 5, retryDelay: 10 });
 exports.STORM_ID = 'storm';
 exports.ConversationIdHeader = 'Conversation-Id';
 class StormClient {
@@ -54,7 +40,6 @@ class StormClient {
             method: method,
             body: JSON.stringify(body),
             headers,
-            dispatcher: retryAgent,
         };
     }
     async send(path, body, method = 'POST') {
@@ -65,7 +50,7 @@ class StormClient {
         });
         const abort = new AbortController();
         options.signal = abort.signal;
-        const response = await (0, undici_1.fetch)(options.url, options);
+        const response = await fetchWithRetries(options.url, options);
         if (response.status !== 200) {
             throw new Error(`Got error response from ${options.url}: ${response.status}\nContent: ${await response.text()}`);
         }
@@ -138,19 +123,19 @@ class StormClient {
             prompt: JSON.stringify({ topic, vote, mainConversationId }),
             conversationId,
         });
-        return (0, undici_1.fetch)(options.url, options);
+        return fetch(options.url, options);
     }
     async getVoteUIPage(topic, conversationId, mainConversationId) {
         const options = await this.createOptions('/v2/ui/get-vote', 'POST', {
             prompt: JSON.stringify({ topic, mainConversationId }),
             conversationId,
         });
-        const response = await (0, undici_1.fetch)(options.url, options);
+        const response = await fetch(options.url, options);
         return response.json();
     }
     async implementAPIClients(prompt) {
         const u = `${this._baseUrl}/v2/ui/implement-api-clients`;
-        const response = await (0, undici_1.fetch)(u, {
+        const response = await fetch(u, {
             method: 'POST',
             body: JSON.stringify({
                 fileName: prompt.fileName,
@@ -162,7 +147,7 @@ class StormClient {
     }
     async generatePrompt(pages) {
         const u = `${this._baseUrl}/v2/ui/prompt`;
-        const response = await (0, undici_1.fetch)(u, {
+        const response = await fetch(u, {
             method: 'POST',
             body: JSON.stringify({
                 pages: pages,
@@ -235,7 +220,7 @@ class StormClient {
             prompt: '',
             conversationId: conversationId,
         });
-        const response = await (0, undici_1.fetch)(options.url, options);
+        const response = await fetch(options.url, options);
         return response.text();
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@kapeta/local-cluster-service",
-    "version": "0.71.0",
+    "version": "0.71.2",
     "description": "Manages configuration, ports and service discovery for locally running Kapeta systems",
     "type": "commonjs",
     "exports": {
@@ -70,6 +70,7 @@
         "download-git-repo": "^3.0.2",
         "express": "4.17.1",
         "express-promise-router": "^4.1.1",
+        "fetch-retry": "^6.0.0",
         "fs-extra": "^11.1.0",
         "glob": "^7.1.6",
         "gunzip-maybe": "^1.4.2",
@@ -88,7 +89,6 @@
         "stream-json": "^1.8.0",
         "tar-stream": "^3.1.6",
         "typescript": "^5.1.6",
-        "undici": "^6.19.8",
         "uuid": "^9.0.1",
         "yaml": "^1.6.0"
     },

package/src/storm/PageGenerator.ts

@@ -259,7 +259,7 @@ export class PageQueue extends EventEmitter {
         // Add safeguard to avoid generating images for nonsense URLs
         // Sometimes we get entries for Base URLs that will then cause issues on the filesystem
         // Example: https://www.kapeta.com/images/
-        const mimeType = mimetypes.lookup(prompt.url) as string | false;
+        const mimeType = mimetypes.lookup(prompt.url);
         if (!mimeType || !mimeType.startsWith('image/')) {
             console.warn('Skipping image reference of type %s for url %s', mimeType, prompt.url);
             return;
@@ -300,7 +300,7 @@ export class PageQueue extends EventEmitter {
 
         await screenStream.waitForDone();
         if (!pageEvent) {
-            throw new Error('No page was generated');
+            throw new Error('No page was generated for ' + prompt.name);
         }
         await this.processPageEventWithReferences(pageEvent);
     }

package/src/storm/routes.ts

@@ -181,6 +181,13 @@ router.post('/ui/screen', async (req: KapetaBodyRequest, res: Response) => {
             sendError(err as any, res);
         });
 
+        queue.on('event', (event: StormEvent) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
+        });
+
         await queue.addPrompt(aiRequest, conversationId, true);
 
         await queue.wait();
@@ -274,8 +281,11 @@ router.post('/:handle/ui/iterative', async (req: KapetaBodyRequest, res: Respons
             sendPageEvent(landingPagesStream.getConversationId(), screenData, res)
         );
 
-        pageQueue.on('event', (screenData: StormEvent) => {
-            sendEvent(res, screenData);
+        pageQueue.on('event', (event: StormEvent) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
         });
 
         pageQueue.on('error', (err) => {
@@ -460,8 +470,11 @@ router.post('/:handle/ui', async (req: KapetaBodyRequest, res: Response) => {
 
         queue.on('page', (screenData: StormEventPage) => sendPageEvent(outerConversationId, screenData, res));
 
-        queue.on('event', (screenData: StormEvent) => {
-            sendEvent(res, screenData);
+        queue.on('event', (event: StormEvent) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
+            }
+            sendEvent(res, event);
         });
 
         queue.on('error', (err) => {
@@ -484,6 +497,7 @@ router.post('/:handle/ui', async (req: KapetaBodyRequest, res: Response) => {
                 })
                 .catch((e) => {
                     console.error('Failed to generate page for screen %s', screen.name, e);
+                    sendError(e as any, res);
                 });
         }
 
@@ -527,10 +541,11 @@ router.post('/ui/edit', async (req: KapetaBodyRequest, res: Response) => {
             }
         });
 
-        queue.on('event', (data) => {
-            if (data.type === 'FILE_START' || data.type === 'FILE_DONE' || data.type === 'FILE_STATE') {
-                sendEvent(res, data);
+        queue.on('event', (event) => {
+            if (event.type === 'FILE_CHUNK') {
+                return;
             }
+            sendEvent(res, event);
         });
 
         queue.on('error', (err) => {

package/src/storm/stormClient.ts

@@ -16,23 +16,8 @@ import {
     StormUIListPrompt,
 } from './stream';
 import { Page, StormEventPageUrl } from './events';
-import { fetch, RequestInit, Agent, RetryAgent } from 'undici';
-
-// Will only retry on error codes and GET requests by default
-// See https://github.com/nodejs/undici/blob/990df2c7e37cbe5bb44fe2f576dddeaeb5916590/docs/docs/api/RetryAgent.md
-const retryAgent = new RetryAgent(new Agent(), {
-    methods: [
-        // Added methods ↓ (not idempotent), but we want to retry on POST:
-        'POST',
-        // defaults below
-        'GET',
-        'HEAD',
-        'OPTIONS',
-        'PUT',
-        'DELETE',
-        'TRACE',
-    ],
-});
+import createFetch from 'fetch-retry';
+const fetchWithRetries = createFetch(global.fetch, { retries: 5, retryDelay: 10 });
 
 export const STORM_ID = 'storm';
 
@@ -129,7 +114,6 @@ class StormClient {
             method: method,
             body: JSON.stringify(body),
             headers,
-            dispatcher: retryAgent,
         };
     }
 
@@ -148,7 +132,7 @@ class StormClient {
         const abort = new AbortController();
         options.signal = abort.signal;
 
-        const response = await fetch(options.url, options);
+        const response = await fetchWithRetries(options.url, options);
 
         if (response.status !== 200) {
             throw new Error(