smart-home-engine 0.23.0 → 0.23.2

package/dist/web/assets/{tsMode-Cvp4skWL.js → tsMode-Bt_apCqT.js}

@@ -1,4 +1,4 @@
-import{m as O}from"./monaco-langs-BW2J83t5.js";import{t as I}from"./index-BWPgohd9.js";/*!-----------------------------------------------------------------------------
+import{m as O}from"./monaco-langs-BW2J83t5.js";import{t as I}from"./index-IaTuET9d.js";/*!-----------------------------------------------------------------------------
  * Copyright (c) Microsoft Corporation. All rights reserved.
  * Version: 0.52.2(404545bded1df6ffa41ea0af4e8ddb219018c6c1)
  * Released under the MIT license

package/dist/web/index.html

@@ -155,7 +155,7 @@
                 }
             })();
         </script>
-      <script type="module" crossorigin src="/assets/index-BWPgohd9.js"></script>
+      <script type="module" crossorigin src="/assets/index-IaTuET9d.js"></script>
       <link rel="modulepreload" crossorigin href="/assets/monaco-langs-BW2J83t5.js">
       <link rel="stylesheet" crossorigin href="/assets/monaco-langs-DyX1CsEw.css">
       <link rel="stylesheet" crossorigin href="/assets/index-BPk8Jr3B.css">

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "smart-home-engine",
-    "version": "0.23.0",
+    "version": "0.23.2",
     "description": "Node.js based script runner for use in MQTT based Smart Home environments",
     "main": "src/index.js",
     "scripts": {

package/src/index.js

@@ -1041,14 +1041,17 @@ function unloadScript(file) {
     log.info(unloadLabel, 'unloading');
     scriptOrigins.delete(file);
 
+    let removedCallbacks = 0;
+    let removedTimers = 0;
+
     // Remove MQTT subscriptions belonging to this script
     for (let i = subscriptions.length - 1; i >= 0; i--) {
-        if (subscriptions[i]._script === file) subscriptions.splice(i, 1);
+        if (subscriptions[i]._script === file) { subscriptions.splice(i, 1); removedCallbacks++; }
     }
 
     // Remove MQTT event callbacks (connect/disconnect) belonging to this script
     for (let i = mqttEventCallbacks.length - 1; i >= 0; i--) {
-        if (mqttEventCallbacks[i]._script === file) mqttEventCallbacks.splice(i, 1);
+        if (mqttEventCallbacks[i]._script === file) { mqttEventCallbacks.splice(i, 1); removedCallbacks++; }
     }
 
     // Remove HTTP routes registered by this script via she.api
@@ -1059,6 +1062,7 @@ function unloadScript(file) {
     const jobs = scriptJobs.get(file);
     if (jobs) {
         jobs.forEach((job) => job && job.cancel());
+        removedTimers += jobs.length;
         scriptJobs.delete(file);
     }
 
@@ -1067,6 +1071,7 @@ function unloadScript(file) {
         if (sunEvents[i]._script === file) {
             if (sunEvents[i]._job) sunEvents[i]._job.cancel();
             sunEvents.splice(i, 1);
+            removedTimers++;
         }
     }
 
@@ -1074,6 +1079,7 @@ function unloadScript(file) {
     const timers = scriptTimers.get(file);
     if (timers) {
         timers.forEach((id) => clearTimeout(id));
+        removedTimers += timers.size;
         scriptTimers.delete(file);
     }
 
@@ -1082,6 +1088,7 @@ function unloadScript(file) {
         if (varSubscriptions[i]._script === file) {
             store.removeListener('change', varSubscriptions[i].handler);
             varSubscriptions.splice(i, 1);
+            removedCallbacks++;
         }
     }
 
@@ -1095,6 +1102,13 @@ function unloadScript(file) {
         matterSandbox.cleanup(file);
     }
 
+    if (removedCallbacks > 0 || removedTimers > 0) {
+        const parts = [];
+        if (removedCallbacks > 0) parts.push(`${removedCallbacks} callback${removedCallbacks !== 1 ? 's' : ''}`);
+        if (removedTimers > 0) parts.push(`${removedTimers} timer${removedTimers !== 1 ? 's' : ''}`);
+        log.debug(unloadLabel, `unregistered ${parts.join(' and ')}`);
+    }
+
     // Remove from scripts map so it can be re-loaded
     delete scripts[file];
 }

package/src/web/ai-api.js

@@ -441,14 +441,15 @@ router.post('/prompt', (req, res) => {
 // POST /she/ai/chat — non-streaming
 router.post('/chat', async (req, res) => {
     const ai = readAiConfig(req.app.locals.configPath);
-    if (!ai?.provider || !ai?.model) {
+    const { messages = [], currentScript, currentView, currentDoc, context = {}, modelOverride, extraFiles } = req.body || {};
+    const effectiveModel = (modelOverride && typeof modelOverride === 'string') ? modelOverride : ai?.model;
+    if (!ai?.provider || !effectiveModel) {
         return res.status(400).json({ error: 'AI provider not configured. Set ai.provider and ai.model in Config.' });
     }
 
-    const { messages = [], currentScript, currentView, currentDoc, context = {}, modelOverride, extraFiles } = req.body || {};
     if (!Array.isArray(messages)) return res.status(400).json({ error: 'messages must be an array' });
 
-    const aiWithModel = modelOverride && typeof modelOverride === 'string' ? { ...ai, model: modelOverride } : ai;
+    const aiWithModel = { ...ai, model: effectiveModel };
     const systemPrompt = buildSystemPrompt(context, currentScript ?? null, currentView ?? null, currentDoc ?? null, _store, extraFiles || []);
     const fullMessages = [{ role: 'system', content: systemPrompt }, ...messages];
 
@@ -471,14 +472,15 @@ router.post('/chat', async (req, res) => {
 // POST /she/ai/chat/stream — SSE streaming
 router.post('/chat/stream', async (req, res) => {
     const ai = readAiConfig(req.app.locals.configPath);
-    if (!ai?.provider || !ai?.model) {
+    const { messages = [], currentScript, currentView, currentDoc, context = {}, modelOverride, extraFiles } = req.body || {};
+    const effectiveModel = (modelOverride && typeof modelOverride === 'string') ? modelOverride : ai?.model;
+    if (!ai?.provider || !effectiveModel) {
         return res.status(400).json({ error: 'AI provider not configured. Set ai.provider and ai.model in Config.' });
     }
 
-    const { messages = [], currentScript, currentView, currentDoc, context = {}, modelOverride, extraFiles } = req.body || {};
     if (!Array.isArray(messages)) return res.status(400).json({ error: 'messages must be an array' });
 
-    const aiWithModel = modelOverride && typeof modelOverride === 'string' ? { ...ai, model: modelOverride } : ai;
+    const aiWithModel = { ...ai, model: effectiveModel };
 
     // Build system prompt BEFORE flushing headers so errors can still return a proper HTTP status
     let systemPrompt;