@mikrojs/firmware 0.20.1 → 0.20.2-next.20260907205905

package/components/mikrojs/mik_main.cpp

@@ -746,24 +746,19 @@ void MIK_Main(void) {
         if (safe_mode) {
             ESP_LOGW(TAG, "Safe mode: skipping entry point %s", app_config.entry_point);
         } else {
-            /* Catch async fatals (unhandled rejections) during an OTA trial. */
+            /* Catch fatals during an OTA trial: MIK_RunEntry hands a synchronous
+             * throw to this handler itself, the loop's rejection flush hands it
+             * unhandled rejections. Both note the failure and arm the panic
+             * restart so reconcile reverts on the next (clean-looking) boot. */
             MIK_SetErrorHandler(mik_rt, ota_trial_error_handler, nullptr);
-            char entry_err[160] = {0};
-            int rc = MIK_RunEntryErr(mik_rt, app_config.entry_point, entry_err, sizeof(entry_err));
+            int rc = MIK_RunEntry(mik_rt, app_config.entry_point);
             if (rc == -EINVAL) {
                 ESP_LOGW(TAG, "No entry point configured (no \"main\" field in package.json)");
             } else if (rc == -ENOENT) {
                 ESP_LOGW(TAG, "Entry point not found: %s", app_config.entry_point);
             } else if (rc == -EFAULT) {
+                /* Already reported to the console by the runtime. */
                 ESP_LOGE(TAG, "Failed to evaluate %s", app_config.entry_point);
-                /* A synchronous top-level throw doesn't reach the rejection handler
-                 * above. If this is a trial build, flag it and arm the panic restart
-                 * so reconcile reverts on the next (clean-looking) boot. */
-                if (mik__ota_in_trial()) {
-                    mik__ota_note_trial_failure(entry_err[0] ? entry_err
-                                                             : "entry failed to evaluate");
-                    MIK_Stop(mik_rt);
-                }
             }
         }
     }

package/components/mikrojs/mik_ota_client_module.cpp

@@ -75,6 +75,10 @@ struct MIKOtaClientState {
      * only copy left after ota.reconcile() ran. settle() marks it delivered. */
     bool has_last_install = false;
     MIKOtaDiagnostic last_install = {};
+    /* Set by report() when the stored decline made it into the body, so settle()
+     * clears only a record the registry has seen. A read that failed leaves the
+     * record for the next round. */
+    bool decline_reported = false;
 };
 
 int mik__ota_client_slot = -1;
@@ -350,7 +354,8 @@ JSValue kv_string_or_undefined(JSContext* ctx, const char* key) {
     MIKOtaClientState* state = state_of(ctx);
     if (!state || !state->env || !state->env->kv_get_str) return JS_UNDEFINED;
     char buf[256] = {};
-    if (!state->env->kv_get_str(state->env->opaque, key, buf, sizeof(buf)) || buf[0] == '\0') {
+    if (state->env->kv_get_str(state->env->opaque, key, buf, sizeof(buf)) != MIK_OTA_KV_OK ||
+        buf[0] == '\0') {
         return JS_UNDEFINED;
     }
     return JS_NewString(ctx, buf);
@@ -911,7 +916,8 @@ JSValue ota_report(JSContext* ctx, JSValue, int, JSValue*) {
      * decline(). Without it a registry cannot tell a device still working
      * through a download from one that has stopped. */
     MIKOtaDeclineRecord declined;
-    if (mikrojs::MIKOtaStore(env).GetDecline(&declined)) {
+    state->decline_reported = mikrojs::MIKOtaStore(env).GetDecline(&declined);
+    if (state->decline_reported) {
         JSValue last = JS_NewObject(ctx);
         JS_SetPropertyStr(ctx, last, "checksum", JS_NewString(ctx, declined.checksum));
         JS_SetPropertyStr(ctx, last, "reason", JS_NewString(ctx, declined.reason));
@@ -960,10 +966,13 @@ JSValue ota_settle(JSContext* ctx, JSValue, int argc, JSValue* argv) {
     /* Confirm before the deliveries, as the built-in does: it must settle the
      * document held BEFORE this round, never the one about to be armed. */
     mikrojs::mik__ota_policy_confirm(state->env);
-    /* The round completed, so the cached install report and the stored decline
-     * record were delivered. */
+    /* The round completed, so the cached install report was delivered, and so
+     * was the stored decline record if report() carried it. */
     state->has_last_install = false;
-    mikrojs::MIKOtaStore(state->env).ClearDecline();
+    if (state->decline_reported) {
+        mikrojs::MIKOtaStore(state->env).ClearDecline();
+        state->decline_reported = false;
+    }
 
     bool renamed = false;
     if (usable) {

package/components/mikrojs/mik_ota_env.cpp

@@ -105,11 +105,13 @@ bool env_kv_set_blob(void*, const char* key, const uint8_t* data, size_t len) {
     return kv_write_blob(key, data, len);
 }
 
-bool env_kv_get_str(void*, const char* key, char* out, size_t max_len) {
+MIKOtaKvStatus env_kv_get_str(void*, const char* key, char* out, size_t max_len) {
     uint8_t buf[320];
     size_t len = sizeof(buf);
-    if (kv_read_blob(key, buf, &len) != MIK_OTA_KV_OK) return false;
-    return mik__kv_decode_str(buf, len, out, max_len);
+    MIKOtaKvStatus status = kv_read_blob(key, buf, &len);
+    if (status != MIK_OTA_KV_OK) return status;
+    /* Stored bytes that will not decode are corruption, not absence. */
+    return mik__kv_decode_str(buf, len, out, max_len) ? MIK_OTA_KV_OK : MIK_OTA_KV_ERROR;
 }
 
 bool env_kv_set_str(void*, const char* key, const char* value) {
@@ -119,11 +121,12 @@ bool env_kv_set_str(void*, const char* key, const char* value) {
     return kv_write_blob(key, buf, needed);
 }
 
-bool env_kv_get_i32(void*, const char* key, int32_t* out) {
+MIKOtaKvStatus env_kv_get_i32(void*, const char* key, int32_t* out) {
     uint8_t buf[16];
     size_t len = sizeof(buf);
-    if (kv_read_blob(key, buf, &len) != MIK_OTA_KV_OK) return false;
-    return mik__kv_decode_i32(buf, len, out);
+    MIKOtaKvStatus status = kv_read_blob(key, buf, &len);
+    if (status != MIK_OTA_KV_OK) return status;
+    return mik__kv_decode_i32(buf, len, out) ? MIK_OTA_KV_OK : MIK_OTA_KV_ERROR;
 }
 
 bool env_kv_set_i32(void*, const char* key, int32_t value) {

package/components/mikrojs/test/ota_host/.gitignore

@@ -0,0 +1,2 @@
+.build/
+.cache/

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikrojs/firmware",
-  "version": "0.20.1",
+  "version": "0.20.2-next.20260907205905",
   "description": "Mikro.js ESP32 firmware: ESP-IDF component, build tools, and project template",
   "keywords": [
     "esp-idf",
@@ -46,9 +46,9 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "esbuild": "^0.28.0",
-    "@mikrojs/native": "0.20.1",
-    "@mikrojs/quickjs": "0.20.1"
+    "@mikrojs/native": "0.20.2-next.20260907205905+8b6f502",
+    "@mikrojs/quickjs": "0.20.2-next.20260907205905+8b6f502",
+    "esbuild": "^0.28.2"
   },
   "engines": {
     "node": ">=24.0.0"