wineater-bot 4.21.0 → 4.22.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wineater-bot",
-  "version": "4.21.0",
+  "version": "4.22.0",
   "description": "",
   "main": "dist/wineater-chatbot.mjs",
   "module": "src/index.js",

package/src/App.svelte

@@ -171,6 +171,8 @@
 		userActed = true;
 
 		let query = '';
+		// id этого поиска: ответы и finally более старого поиска не трогают новый
+		let myQueryId: string | undefined;
 		try {
 			isLoading = true;
 			clearError();
@@ -178,6 +180,7 @@
 			query = searchQuery.trim() + ';';
 			lastSearchedQuery = query;
 			currentQueryId = newQueryId();
+			myQueryId = currentQueryId;
 			try {
 				trackWidgetEvent(WIDGET_EVENTS.SEARCH_PERFORMED, storeId, { query });
 			} catch (error) {
@@ -185,9 +188,46 @@
 			}
 
 			// Запрос уходит сразу, карточки уезжают вверх, пока он в полёте.
+			// v2 идёт потоком: 4 вина приходят, как только их выбрал реранкер,
+			// причины («почему») — следом, в том же ответе. Они видны только в
+			// попапе, поэтому плитки показываем, не дожидаясь их.
+			let animationDone = false;
+			let shown = false;
+			let pendingEarly: RecommendedWine[] | null = null;
+
+			const showResults = (searchResults: RecommendedWine[]) => {
+				if (!searchResults || searchResults.length === 0) {
+					results = [];
+					translatedPopups = [];
+					errorMessage = getTranslation({ key: 'noResultsMessage', lang: appLanguage });
+					trackWidgetEvent(WIDGET_EVENTS.NO_RESULTS_FOUND, storeId, { query });
+					return;
+				}
+				results = searchResults;
+				isLoading = false;
+				if (shown) return;
+				shown = true;
+				// Пагинация доступна только для search/v2 — у instant/legacy-путей нет
+				// "more"-эндпоинта. Неполная выдача (< 4) означает, что кандидаты и так
+				// кончились: кнопка в этом случае только мигнула бы впустую.
+				hasMore = apiVersion === '2' && searchResults.length >= 4;
+				feedbackShown(searchResults, query);
+				trackWidgetEvent(WIDGET_EVENTS.SEARCH_RESULTS_DISPLAYED, storeId, {
+					results_count: searchResults.length,
+					results: searchResults,
+					query
+				});
+			};
+
+			const onWines = (early: RecommendedWine[]) => {
+				if (myQueryId !== currentQueryId) return; // устаревший поиск
+				if (!animationDone) pendingEarly = early;
+				else showResults(early);
+			};
+
+			const request = wineaterApi.searchStream(query, currentQueryId, onWines);
 			// Пустой catch нужен, чтобы отказ запроса во время анимации не стал
 			// unhandled rejection: ошибку обработает await ниже.
-			const request = wineaterApi.getWineRecommendationsSearch(query, currentQueryId);
 			request.catch(() => {});
 
 			// шаг 1: плавно поднять карточки
@@ -198,29 +238,26 @@
 			isAnimating = false;
 			hasMore = false;
 			seenIds = [];
-
-			const searchResults = await request;
-
-			if (!searchResults || searchResults.length === 0) {
-				results = [];
-				translatedPopups = [];
-				errorMessage = getTranslation({ key: 'noResultsMessage', lang: appLanguage });
-
-				trackWidgetEvent(WIDGET_EVENTS.NO_RESULTS_FOUND, storeId, { query });
+			animationDone = true;
+			if (pendingEarly) showResults(pendingEarly);
+
+			const finalResults = await request;
+			if (myQueryId !== currentQueryId) return;
+
+			// Тот же состав — дописываем причины в уже показанные плитки, не
+			// перерисовывая сетку под курсором; иначе (fallback, редкие расхождения) — заменяем.
+			const sameWines =
+				shown &&
+				finalResults.length === results.length &&
+				finalResults.every((w, i) => w.id === results[i]?.id);
+			if (sameWines) {
+				const byId = new Map(finalResults.map((w) => [w.id, w]));
+				results = results.map((w) => ({ ...w, reason: byId.get(w.id)?.reason ?? w.reason }));
 			} else {
-				results = searchResults;
-				// Пагинация доступна только для search/v2 — у instant/legacy-путей нет
-				// "more"-эндпоинта. Неполная выдача (< 4) означает, что кандидаты и так
-				// кончились: кнопка в этом случае только мигнула бы впустую.
-				hasMore = apiVersion === '2' && searchResults.length >= 4;
-				feedbackShown(searchResults, query);
-				trackWidgetEvent(WIDGET_EVENTS.SEARCH_RESULTS_DISPLAYED, storeId, {
-					results_count: searchResults.length,
-					results: searchResults,
-					query
-				});
+				showResults(finalResults);
 			}
 		} catch (error) {
+			if (myQueryId !== currentQueryId) return;
 			results = [];
 			console.error('[wineater] search error:', error);
 			trackWidgetEvent(WIDGET_EVENTS.ERROR_OCCURRED, storeId, {
@@ -229,7 +266,7 @@
 			});
 			setSearchError();
 		} finally {
-			isLoading = false;
+			if (myQueryId === currentQueryId) isLoading = false;
 		}
 	}
 

package/src/services/api/api.ts

@@ -120,6 +120,69 @@ class WineaterApi {
         }
     }
 
+    /**
+     * v2 search over NDJSON (`search/v2/stream`): `onWines` fires as soon as the
+     * reranker has picked the 4 wines (~1 s before the reasons are written), with
+     * `reason` empty; the promise resolves with the final wines including reasons.
+     * Any transport problem (no stream support, HTTP error, old backend without
+     * the endpoint) falls back to the plain JSON request — same result, later.
+     */
+    async searchStream(
+        text: string,
+        queryId: string | undefined,
+        onWines: (wines: RecommendedWine[]) => void,
+    ): Promise<RecommendedWine[]> {
+        const fallback = () => this.getWineRecommendationsSearch(text, queryId);
+        if (this.#apiVersion !== '2') return fallback();
+
+        let response: Response;
+        try {
+            response = await fetch(`${this.#baseUrl}dish/recommendations/search/v2/stream`, {
+                method: 'POST',
+                headers: this.#headers(),
+                body: JSON.stringify(queryId ? { text, query_id: queryId } : { text }),
+            });
+        } catch {
+            // network / CORS (incl. the uid header) — the JSON path knows how to recover
+            return fallback();
+        }
+        if (!response.ok || !response.body) return fallback();
+
+        const reader = response.body.getReader();
+        const decoder = new TextDecoder();
+        let buffer = '';
+        let early: RecommendedWine[] | null = null;
+        const toWines = (recs: any[]) => (recs ?? []).filter(Boolean).map(deserializeRecommendedWine);
+
+        try {
+            for (;;) {
+                const { done, value } = await reader.read();
+                if (value) buffer += decoder.decode(value, { stream: true });
+                const lines = buffer.split('\n');
+                buffer = done ? '' : lines.pop() ?? '';
+                for (const line of done ? lines.filter(Boolean) : lines) {
+                    if (!line.trim()) continue;
+                    const event = JSON.parse(line);
+                    if (event.type === 'wines') {
+                        // reasons are not written yet — show none rather than a placeholder
+                        early = toWines(event.recommendations).map((w) => ({ ...w, reason: '' }));
+                        onWines(early);
+                    } else if (event.type === 'final') {
+                        return toWines(event.recommendations);
+                    } else if (event.type === 'error') {
+                        if (early) return early;
+                        return fallback();
+                    }
+                }
+                if (done) break;
+            }
+        } catch (error) {
+            if (early) return early;
+            return fallback();
+        }
+        return early ?? fallback();
+    }
+
     async getMoreWineRecommendations(text: string, excludeIds: string[]): Promise<{ recommendations: RecommendedWine[]; hasMore: boolean }> {
         try {
             const data = await this.#request<any>('dish/recommendations/search/v2/more', { text, exclude_ids: excludeIds });