Changeset 3738724

Timestamp:

10/10/2026 09:28:56 PM (3 weeks ago)

Author:

dtpm

Message:

Version 1.1.3

Location:

dtpm-ai-sale-assistant-woocommerce

Files:

• tags/1.1.3 (added)
• tags/1.1.3/LICENSE (added)
• tags/1.1.3/admin (added)
• tags/1.1.3/admin/assets (added)
• tags/1.1.3/admin/assets/banner-dashboard.svg (added)
• tags/1.1.3/admin/availability-sync.php (added)
• tags/1.1.3/admin/category-analysis-page.php (added)
• tags/1.1.3/admin/causal-chain-page.php (added)
• tags/1.1.3/admin/content-page.php (added)
• tags/1.1.3/admin/content-processing-resume.php (added)
• tags/1.1.3/admin/css (added)
• tags/1.1.3/admin/css/ki-berater-design.css (added)
• tags/1.1.3/admin/dashboard-page.php (added)
• tags/1.1.3/admin/diagnosis-page.php (added)
• tags/1.1.3/admin/help-page.php (added)
• tags/1.1.3/admin/js (added)
• tags/1.1.3/admin/js/category-analysis.js (added)
• tags/1.1.3/admin/js/causal-chain.js (added)
• tags/1.1.3/admin/js/connect.js (added)
• tags/1.1.3/admin/js/content.js (added)
• tags/1.1.3/admin/js/diagnosis.js (added)
• tags/1.1.3/admin/js/settings.js (added)
• tags/1.1.3/admin/js/status.js (added)
• tags/1.1.3/admin/js/tabs.js (added)
• tags/1.1.3/admin/lizenz-check.php (added)
• tags/1.1.3/admin/settings-page.php (added)
• tags/1.1.3/admin/status-page.php (added)
• tags/1.1.3/admin/testmonat-check.php (added)
• tags/1.1.3/changelog.txt (added)
• tags/1.1.3/frontend (added)
• tags/1.1.3/frontend/chat-widget.php (added)
• tags/1.1.3/frontend/css (added)
• tags/1.1.3/frontend/css/chat-widget.css (added)
• tags/1.1.3/frontend/js (added)
• tags/1.1.3/frontend/js/chat-widget.js (added)
• tags/1.1.3/ki-berater.php (added)
• tags/1.1.3/languages (added)
• tags/1.1.3/languages/dtpm-ai-sale-assistant-woocommerce.pot (added)
• tags/1.1.3/readme.txt (added)
• trunk/admin/content-processing-resume.php (modified) (5 diffs)
• trunk/admin/js/causal-chain.js (modified) (3 diffs)
• trunk/admin/js/content.js (modified) (3 diffs)
• trunk/admin/js/tabs.js (modified) (3 diffs)
• trunk/changelog.txt (modified) (1 diff)
• trunk/ki-berater.php (modified) (2 diffs)
• trunk/readme.txt (modified) (2 diffs)

dtpm-ai-sale-assistant-woocommerce/trunk/admin/content-processing-resume.php

@@ -64 +64 @@
  * ki-berater.php bei Plugin-Aktivierung/-Deaktivierung, ruft aber die
  * Funktionen dieser Datei auf - gleiches Prinzip wie admin/availability-sync.php/
- * admin/testmonat-check.php. Braucht ki_berater_fetch_sample_products() aus
- * admin/category-analysis-page.php - deshalb erst NACH dieser Datei in
- * ki-berater.php eingebunden (siehe dort).
+ * admin/testmonat-check.php. Braucht ki_berater_fetch_content_products_for_category()
+ * aus admin/content-page.php (Fund 09.10.2026, siehe
+ * ki_berater_resume_content_processing_for_subcategory() unten fuer die volle
+ * Begruendung) - deshalb erst NACH dieser Datei in ki-berater.php eingebunden
+ * (siehe dort).
  */
 
@@ -216 +218 @@
 
 /**
+ * GET /variation-attributes (admin/variation_attributes.py) - die
+ * bestaetigten RELEVANTEN Variationsattribut-Namen einer Subkategorie,
+ * dieselbe Berechnung (bestaetigt UND relevant) wie
+ * fetchRelevantAttributeNames() in js/content.js fuer den Browser-Pfad
+ * (Fund 09.10.2026, siehe ki_berater_resume_content_processing_for_subcategory()
+ * unten). Leeres Array bei jedem Fehler (Netzwerk, Nicht-200, kaputtes
+ * JSON) - dieselbe konservative "zaehlt wie nicht relevant"-Regel wie in
+ * ki_berater_expand_variable_product_for_content() (admin/content-page.php):
+ * erzeugt im Zweifel WENIGER statt mehr Varianten-Zeilen, nie mehr.
+ */
+function ki_berater_fetch_relevant_attribute_names_for_resume( $subcategory_id, $tenant_id ) {
+        $response = wp_remote_get(
+                trailingslashit( KI_BERATER_API_URL ) . 'variation-attributes?subcategory_id=' . rawurlencode( $subcategory_id )
+                        . '&tenant_id=' . rawurlencode( $tenant_id ),
+                array(
+                        'headers' => ki_berater_auth_headers(),
+                        'timeout' => 20,
+                )
+        );
+
+        if ( is_wp_error( $response ) || 200 !== wp_remote_retrieve_response_code( $response ) ) {
+                return array();
+        }
+
+        $body = json_decode( wp_remote_retrieve_body( $response ), true );
+        if ( ! is_array( $body ) ) {
+                return array();
+        }
+
+        $relevant_attributes = array();
+        foreach ( $body as $entry ) {
+                if ( ! empty( $entry['bestaetigt'] ) && ! empty( $entry['relevant'] ) ) {
+                        $relevant_attributes[] = $entry['attribut'];
+                }
+        }
+        return $relevant_attributes;
+}
+
+/**
  * Ermittelt fuer EINE als "angefangen, aber unvollstaendig" erkannte
  * Subkategorie (processed_count > 0, siehe ki_berater_run_content_processing_resume())
- * die aktuelle, vollstaendige WooCommerce-Produktliste (gleiches Muster wie
- * ki_berater_sync_availability_for_subcategory() in admin/availability-sync.php -
- * ueber vaterprodukt_id dedupliziert, -1 = kein Limit), vergleicht sie gegen
+ * die aktuelle, vollstaendige WooCommerce-Produktliste, vergleicht sie gegen
  * die bereits gespeicherten SKUs (GET /content/state) und stoesst
  * POST /content/process-subcategory NUR fuer die tatsaechlich noch
@@ -228 +267 @@
  * still ab (return), gleiches "kein eigener Aufrufer/Fehlerkanal"-Prinzip
  * wie die uebrigen Cron-Callbacks in diesem Plugin.
+ *
+ * BUGFIX (Fund 09.10.2026, Abgleich Shopify-Projekt gegen Kern-API): bisher
+ * ueber ki_berater_fetch_sample_products() (admin/category-analysis-page.php)
+ * ermittelt - die liefert IMMER genau EINE Zeile je WooCommerce-Vaterprodukt
+ * mit dessen eigener (oft leerer) SKU/Preis, OHNE jede Varianten-
+ * Aufschluesselung. Der Browser-Pfad (ki_berater_fetch_content_products_for_category(),
+ * admin/content-page.php) gruppiert dagegen korrekt nach den RELEVANTEN
+ * unterscheidenden Attributen (siehe ki_berater_expand_variable_product_for_content()
+ * dort). Ein Variationsprodukt mit mehreren relevanten Gruppen wurde dadurch
+ * im stuendlichen Cron faelschlich nur als EIN Eintrag mit der falschen
+ * SKU/dem falschen Preis (Vaterprodukt statt Variante) gezaehlt/gemeldet -
+ * jetzt dieselbe Funktion wie der Browser-Pfad, mit denselben bestaetigten
+ * relevant_attributes (siehe ki_berater_fetch_relevant_attribute_names_for_resume()
+ * oben). Dedup jetzt ueber sku statt vaterprodukt_id - echte, unterschiedliche
+ * Varianten-SKUs desselben Vaterprodukts waeren sonst faelschlich als
+ * "Duplikat" zusammengefallen (gleiche Begruendung wie
+ * ki_berater_ajax_fetch_content_products() in admin/content-page.php).
  */
 function ki_berater_resume_content_processing_for_subcategory( $subcategory_id, $tenant_id, $processed_count ) {
@@ -235 +291 @@
         }
 
-        $categories = array_map(
-                function ( $category ) {
-                        return array( 'id' => $category['external_category_id'] );
-                },
-                $external_categories
-        );
-
-        $products_per_category = ki_berater_fetch_sample_products( $categories, -1 );
-
-        $products_by_vaterprodukt_id = array();
-        foreach ( $products_per_category as $products ) {
+        $relevant_attributes = ki_berater_fetch_relevant_attribute_names_for_resume( $subcategory_id, $tenant_id );
+
+        $products_by_sku = array();
+        foreach ( $external_categories as $external_category ) {
+                $products = ki_berater_fetch_content_products_for_category(
+                        $external_category['external_category_id'],
+                        $relevant_attributes
+                );
                 foreach ( $products as $product ) {
-                        $products_by_vaterprodukt_id[ $product['vaterprodukt_id'] ] = $product;
+                        $products_by_sku[ $product['sku'] ] = $product;
                 }
         }
 
-        $total = count( $products_by_vaterprodukt_id );
+        $total = count( $products_by_sku );
         if ( 0 === $total || $processed_count >= $total ) {
                 // Entweder der Katalog ist gerade leer (voruebergehender
@@ -269 +322 @@
 
         $remaining = array();
-        foreach ( $products_by_vaterprodukt_id as $product ) {
+        foreach ( $products_by_sku as $product ) {
                 if ( ! isset( $already_processed_lookup[ $product['sku'] ] ) ) {
                         $remaining[] = $product;

dtpm-ai-sale-assistant-woocommerce/trunk/admin/js/causal-chain.js

@@ -726 +726 @@
                 var picker = document.getElementById( 'ki-berater-causal-chain-subcategory-picker' );
                 var hint = document.getElementById( 'ki-berater-causal-chain-no-subcategories-hint' );
+                // BUGFIX (Fund 10.10.2026, siehe js/tabs.js "ki-berater-tab-activated"):
+                // dieser Aufruf laeuft jetzt bei JEDEM Wechsel zu diesem Tab erneut,
+                // nicht mehr nur einmal beim Seitenaufbau - populateSubcategorySelect()
+                // baut die <select>-Optionen dabei komplett neu auf UND wuerde ohne
+                // diese Zwischenspeicherung die Auswahl stillschweigend auf "Bitte
+                // auswaehlen" zuruecksetzen, selbst wenn der Haendler mitten in der
+                // Bearbeitung GENAU dieser Subkategorie war (selectedSubcategoryId
+                // selbst bleibt davon unberuehrt, nur die sichtbare <select>-Anzeige
+                // wuerde sonst davon abweichen).
+                var select = document.getElementById( 'ki-berater-causal-chain-subcategory-select' );
+                var previouslySelectedId = select ? select.value : '';
 
                 hideApiSecretHint();
@@ -748 +759 @@
                                         }
                                 } else {
+                                        // BUGFIX (Fund 10.10.2026): dieser Zweig setzte picker/hint
+                                        // bisher NIE explizit zurueck - unschaedlich, SOLANGE
+                                        // loadSubcategories() nur einmal pro Seitenaufbau lief (der
+                                        // server-seitige HTML-Ausgangszustand zeigt den picker
+                                        // bereits/versteckt den hint bereits, siehe
+                                        // causal-chain-page.php). Seit dem erneuten Aufruf bei jedem
+                                        // Tab-Wechsel (siehe js/tabs.js) MUSS dieser Zweig den
+                                        // "keine Subkategorien"-Zustand eines fruehen Aufrufs
+                                        // explizit wieder aufheben koennen, sonst bleiben picker/hint
+                                        // auf dem letzten "leer"-Stand haengen, obwohl die neue
+                                        // Antwort bereits Subkategorien enthaelt.
+                                        if ( picker ) {
+                                                picker.style.display = 'block';
+                                        }
+                                        if ( hint ) {
+                                                hint.style.display = 'none';
+                                        }
                                         populateSubcategorySelect( subcategories );
+                                        // Nur wiederherstellen, wenn diese Subkategorie in der
+                                        // FRISCHEN Liste tatsaechlich noch vorkommt - sonst bleibt
+                                        // bewusst der Platzhalter stehen (z. B. tatsaechlich
+                                        // geloescht).
+                                        if ( previouslySelectedId && subcategoryNameById[ previouslySelectedId ] ) {
+                                                select.value = previouslySelectedId;
+                                        }
                                 }
 
@@ -2160 +2195 @@
                         loadSubcategories();
                         loadOpenOverview();
+
+                        // BUGFIX (Fund 10.10.2026, Dirk, siehe js/tabs.js fuer die volle
+                        // Begruendung): ohne Seiten-Reload zeigte dieser Tab nach einem
+                        // rein client-seitigen Wechsel weiterhin den Stand von VOR einer
+                        // zwischenzeitlich in "Kategorien-Analyse" bestaetigten Analyse
+                        // ("Kategorie-Analyse noch nicht gemacht"). loadSubcategories()/
+                        // loadOpenOverview() sind beide bereits vollstaendig
+                        // idempotent (bauen Auswahl/Uebersicht jedes Mal komplett neu
+                        // aus der Server-Antwort auf) - ein erneuter Aufruf bei JEDEM
+                        // Wechsel zu diesem Tab ist deshalb gefahrlos.
+                        if ( typeof document.addEventListener === 'function' ) {
+                                document.addEventListener( 'ki-berater-tab-activated', function ( event ) {
+                                        if ( event.detail && event.detail.panelId === 'ki-berater-tab-panel-causal-chain' ) {
+                                                loadSubcategories();
+                                                loadOpenOverview();
+                                        }
+                                } );
+                        }
                 } );
         }

dtpm-ai-sale-assistant-woocommerce/trunk/admin/js/content.js

@@ -726 +726 @@
                 var noSubcategoriesHint = document.getElementById( 'ki-berater-content-no-subcategories-hint' );
                 var noCausalChainsHint = document.getElementById( 'ki-berater-content-no-causal-chains-hint' );
+                // BUGFIX (Fund 10.10.2026, siehe js/tabs.js "ki-berater-tab-activated"):
+                // dieser Aufruf laeuft jetzt bei JEDEM Wechsel zu diesem Tab erneut -
+                // populateSubcategorySelect() baut die <select>-Optionen dabei
+                // komplett neu auf und wuerde ohne diese Zwischenspeicherung die
+                // Auswahl stillschweigend auf "Bitte auswaehlen" zuruecksetzen, selbst
+                // wenn der Haendler mitten in der Bearbeitung GENAU dieser
+                // Subkategorie war.
+                var select = document.getElementById( 'ki-berater-content-subcategory-select' );
+                var previouslySelectedId = select ? select.value : '';
 
                 if ( picker ) {
@@ -768 +777 @@
                                         }
                                         populateSubcategorySelect( subcategoriesWithCausalChain );
+                                        // Nur wiederherstellen, wenn diese Subkategorie in der
+                                        // FRISCHEN, gefilterten Liste tatsaechlich noch vorkommt.
+                                        var bleibtGueltig = subcategoriesWithCausalChain.some( function ( subcategory ) {
+                                                return subcategory.id === previouslySelectedId;
+                                        } );
+                                        if ( previouslySelectedId && bleibtGueltig ) {
+                                                select.value = previouslySelectedId;
+                                        }
                                 } );
                         } )
@@ -1758 +1775 @@
                         loadSubcategories();
                         loadOpenOverview();
+
+                        // BUGFIX (Fund 10.10.2026, Dirk, siehe js/tabs.js fuer die volle
+                        // Begruendung): ohne Seiten-Reload zeigte dieser Tab nach einem
+                        // rein client-seitigen Wechsel weiterhin den Stand von VOR einer
+                        // zwischenzeitlich in "Kausalketten" bestaetigten Kausalkette
+                        // ("noch keine Kausalkette bestaetigt"). loadSubcategories()/
+                        // loadOpenOverview() sind beide bereits vollstaendig idempotent -
+                        // ein erneuter Aufruf bei JEDEM Wechsel zu diesem Tab ist deshalb
+                        // gefahrlos.
+                        if ( typeof document.addEventListener === 'function' ) {
+                                document.addEventListener( 'ki-berater-tab-activated', function ( event ) {
+                                        if ( event.detail && event.detail.panelId === 'ki-berater-tab-panel-content' ) {
+                                                loadSubcategories();
+                                                loadOpenOverview();
+                                        }
+                                } );
+                        }
                 } );
         }

dtpm-ai-sale-assistant-woocommerce/trunk/admin/js/tabs.js

@@ -1 +1 @@
 /**
- * Reines Tab-Umschalten fuer admin/dashboard-page.php - greift NICHT in
- * settings.js/category-analysis.js/causal-chain.js/content.js ein, kennt
- * nur die generischen .ki-berater-tab-trigger/.ki-berater-tab-panel-
- * Elemente und deren data-tab-target/id-Kopplung. Gleicher Code-Stil wie
- * die vier Feature-Skripte (IIFE, var, DOMContentLoaded).
+ * Reines Tab-Umschalten fuer admin/dashboard-page.php - kennt nur die
+ * generischen .ki-berater-tab-trigger/.ki-berater-tab-panel-Elemente und
+ * deren data-tab-target/id-Kopplung. Gleicher Code-Stil wie die vier
+ * Feature-Skripte (IIFE, var, DOMContentLoaded).
  *
  * Kein URL-Sync (siehe Auftrag - entspricht dem v0-Mockup, dort auch nur
@@ -10 +9 @@
  * serverseitigen Standard-Tab (Einstellungen, siehe dashboard-page.php)
  * zurueck.
+ *
+ * BUGFIX (Fund 10.10.2026, Dirk): category-analysis.js/causal-chain.js/
+ * content.js laden ihren jeweiligen Anfangszustand (u. a. die
+ * Subkategorie-Liste, "Noch offen"-Uebersicht) bisher NUR EINMAL beim
+ * DOMContentLoaded dieser Seite - ein rein client-seitiger Tab-Wechsel
+ * (GENAU das, was dieses Skript tut) loeste seitdem NIE einen erneuten
+ * Abruf aus. Wer z. B. in "Kategorien-Analyse" etwas bestaetigt und DANN
+ * ohne Seiten-Reload zu "Kausalketten" wechselt, sah dort weiterhin den
+ * Stand von vor der Bestaetigung ("Kategorie-Analyse noch nicht gemacht") -
+ * derselbe Effekt beim Wechsel von "Kausalketten" zu "Produktabdeckung".
+ * activateTab() loest deshalb jetzt zusaetzlich ein generisches
+ * CustomEvent "ki-berater-tab-activated" aus (detail.panelId = die Ziel-
+ * Panel-ID) - jedes Feature-Skript kann darauf lauschen und seinen
+ * eigenen Anfangszustand bei Bedarf erneut laden, OHNE dass dieses Skript
+ * (oder ein anderes Feature-Skript) die Feature-Skripte direkt kennen
+ * oder aufrufen muss (bleibt bewusst entkoppelt, siehe Modul-Docstrings
+ * dort - "bewusst dupliziert, nicht importiert"). NUR bei einem
+ * tatsaechlichen Klick ausgeloest (siehe handleTriggerClick() unten),
+ * NICHT beim initialen Seitenaufbau - der server-seitig aktive Standard-
+ * Tab hat seinen Anfangszustand ohnehin schon ueber sein eigenes
+ * DOMContentLoaded geladen, ein zusaetzlicher Abruf dafuer waere
+ * redundant.
  */
 ( function () {
@@ -35 +56 @@
                         panel.hidden = panel.id !== targetPanelId;
                 } );
+
+                document.dispatchEvent(
+                        new CustomEvent( 'ki-berater-tab-activated', { detail: { panelId: targetPanelId } } )
+                );
         }
 

dtpm-ai-sale-assistant-woocommerce/trunk/changelog.txt

@@ -1 +1 @@
 *** DT-PM AI Sale Assistant for WooCommerce Changelog ***
+
+2026-10-10 - version 1.1.3
+Requires Kern-API >= commit 3c31fd7 (unchanged from 1.1.2) - this is a pure frontend/JS fix, no Kern-API dependency.
+* Fixed - `admin/js/causal-chain.js` and `admin/js/content.js` loaded their subcategory list exactly once, at page load (`DOMContentLoaded`) - a purely client-side tab switch (`admin/js/tabs.js`) never triggered a re-fetch. After confirming a Category Analysis and switching to the Causal Chains tab without reloading the page (or, equally, after confirming a Causal Chain and switching to Product Coverage), the newly confirmed subcategory still showed as "not done yet" until the page was reloaded. `tabs.js` now dispatches a `ki-berater-tab-activated` custom event whenever a tab is actually clicked, which both feature scripts listen for on their own panel to refresh their state - kept the two scripts decoupled (no direct cross-file calls, consistent with the existing "duplicated, not imported" convention). Also fixed a related, previously latent bug this exposed: `causal-chain.js`'s loader never explicitly reset its "no subcategories yet" hint/picker visibility back to normal once shown, relying on a one-time page load always starting from the correct default HTML state - harmless as long as it only ran once, but would have left the hint stuck visible forever on any later successful re-fetch. Also preserves the currently selected subcategory across a re-fetch in both tabs (previously, re-running the loader would have silently reset an in-progress selection back to the placeholder).
+
+2026-10-10 - version 1.1.2
+Requires Kern-API >= commit 3c31fd7 (unchanged from 1.1.1) - this fix calls only the already-existing `GET /variation-attributes` endpoint, no new Kern-API dependency.
+* Fixed - the hourly catch-up cron for incomplete product processing (`admin/content-processing-resume.php`) used a different, simpler product-fetching routine (`ki_berater_fetch_sample_products()`) than the one the browser's "Process remaining" button uses (`ki_berater_fetch_content_products_for_category()`/`ki_berater_expand_variable_product_for_content()`) - for a variable product with at least one merchant-confirmed relevant distinguishing attribute (e.g. color), the cron reported only a single row using the parent product's own (often empty) SKU/price, instead of one row per actual relevant variant combination with that variant's own SKU/price. The cron now fetches the merchant's confirmed relevant attributes itself (`GET /variation-attributes`) and uses the same variant-aware logic as the browser path; deduplication switched from `vaterprodukt_id` to `sku` accordingly, since a variant-expanded product can now report multiple distinct SKUs.
 
 2026-10-08 - version 1.1.1

dtpm-ai-sale-assistant-woocommerce/trunk/ki-berater.php

@@ -3 +3 @@
  * Plugin Name: DT-PM AI Sale Assistant for WooCommerce
  * Description: Thin wp-admin interface for the AI product advisor (ARCHITECTURE.md section 2.1). Renders no business logic itself, only calls the Kern-API.
- * Version: 1.1.1
+ * Version: 1.1.3
  * Author: KI-Produktberater-Team
  * License: GPLv2 or later
@@ -64 +64 @@
 // BEIDE hochzaehlen (war zuletzt auseinandergelaufen - 0.1.13 im Header,
 // 0.1.12 hier - hiermit wieder angeglichen).
-define( 'KI_BERATER_VERSION', '1.1.1' );
+define( 'KI_BERATER_VERSION', '1.1.3' );
 
 require_once KI_BERATER_PLUGIN_DIR . 'admin/diagnosis-page.php';

dtpm-ai-sale-assistant-woocommerce/trunk/readme.txt

@@ -8 +8 @@
 WC tested up to: 11.1
 Requires Plugins: woocommerce
-Stable tag: 1.1.1
+Stable tag: 1.1.3
 License: GPLv2 or later
 License URI: https://www.gnu.org/licenses/gpl-2.0.html
@@ -94 +94 @@
 == Changelog ==
 
+= 1.1.3 =
+* Fixed: after confirming a Category Analysis, the Causal Chains tab (and, after confirming a Causal Chain, the Product Coverage tab) still said "not done yet" for it until the page was reloaded - switching tabs without reloading now refreshes that status correctly.
+
+= 1.1.2 =
+* Fixed: the hourly catch-up check for incomplete product processing could report the wrong price/SKU for a product with multiple customer-relevant option combinations (e.g. several colors), using the overall product's own (often blank) price/SKU instead of the matching option's - the manual "Process remaining" button already handled this correctly; the hourly check now uses the same logic.
+
 = 1.1.1 =
 * Fixed: an automatic catch-up run for a handful of still-missing products (the hourly background check, or clicking "Process remaining" right after a few products failed to process) could wipe out the rest of an already-processed subcategory's recommendations - the run now only clears out genuinely removed products when it actually has the complete, current product list, not a partial one.