sh3-core 0.11.7 → 0.12.0

package/dist/shards/activate.svelte.js

@@ -47,6 +47,7 @@ export const registeredShards = $state(new Map());
  */
 const active = new Map();
 export const activeShards = $state(new Map());
+export const erroredShards = $state(new Map());
 /**
  * Register (or re-register) a shard with the framework so it can later be
  * activated. Records the shard in `registeredShards` but does not run
@@ -63,6 +64,9 @@ export function registerShard(shard) {
         deactivateShard(id);
     }
     registeredShards.set(id, shard);
+    // Re-registering wipes any prior error: the new shard module gets a
+    // clean slate, and a hot-reload of a fixed shard removes the stale entry.
+    erroredShards.delete(id);
 }
 /**
  * Activate a registered shard. Builds a `ShardContext`, calls `shard.activate`,
@@ -70,11 +74,18 @@ export function registerShard(shard) {
  * calls `shard.autostart` if defined. Idempotent — calling on an already-active
  * shard is a no-op.
  *
+ * If `shard.activate` throws, partial state (registered views, verbs,
+ * contributions, document handles, actions, env subscription) is unwound
+ * and the failure is recorded in `erroredShards` before the error is
+ * re-thrown. Callers in `host.ts` (autostart loop) and `launchApp`
+ * (required-shard loop) decide how to react.
+ *
  * @param id - The `ShardManifest.id` of the shard to activate. Must be registered.
- * @throws If the shard is not registered, or if a manifest view has no factory after activation.
+ * @param opts - Optional. `phase` is recorded in `erroredShards` on failure (default 'launch').
+ * @throws If the shard is not registered, if `shard.activate` throws, or if a manifest view has no factory after activation.
  */
-export async function activateShard(id) {
-    var _a, _b, _c, _d, _e;
+export async function activateShard(id, opts) {
+    var _a, _b, _c, _d, _e, _f;
     const shard = registeredShards.get(id);
     if (!shard) {
         throw new Error(`Cannot activate shard "${id}": not registered`);
@@ -201,24 +212,57 @@ export async function activateShard(id) {
     }
     active.set(id, entry);
     activeShards.set(id, shard);
-    await shard.activate(ctx);
-    for (const view of shard.manifest.views) {
-        if (!entry.viewIds.has(view.id)) {
-            throw new Error(`Shard "${id}" declared view "${view.id}" in its manifest but registered no factory for it.`);
+    try {
+        await shard.activate(ctx);
+        for (const view of shard.manifest.views) {
+            if (!entry.viewIds.has(view.id)) {
+                throw new Error(`Shard "${id}" declared view "${view.id}" in its manifest but registered no factory for it.`);
+            }
         }
-    }
-    // Hydrate env state if the shard declared it via ctx.env().
-    if (envState.proxy && envState.defaults) {
-        try {
-            const stored = await fetchEnvState(id);
-            const merged = Object.assign({}, envState.defaults, stored);
-            Object.assign(envState.proxy, merged);
+        // Hydrate env state if the shard declared it via ctx.env().
+        if (envState.proxy && envState.defaults) {
+            try {
+                const stored = await fetchEnvState(id);
+                const merged = Object.assign({}, envState.defaults, stored);
+                Object.assign(envState.proxy, merged);
+            }
+            catch (err) {
+                console.warn(`[sh3] Failed to hydrate env state for shard "${id}":`, err instanceof Error ? err.message : err);
+            }
         }
-        catch (err) {
-            console.warn(`[sh3] Failed to hydrate env state for shard "${id}":`, err instanceof Error ? err.message : err);
+    }
+    catch (err) {
+        // Unwind partial state. Mirror deactivateShard's body, minus the
+        // shard.deactivate?.() call — the shard never finished activating.
+        // Each cleanup fn runs inside its own swallow so a teardown failure
+        // cannot mask the original activation error.
+        for (const fn of entry.cleanupFns) {
+            try {
+                void fn();
+            }
+            catch (_g) {
+                // intentionally swallowed: original error is what matters.
+            }
         }
+        for (const name of entry.verbNames)
+            fwUnregisterVerb(name);
+        for (const viewId of entry.viewIds)
+            unregisterView(viewId);
+        clearSelectionForShard(id);
+        active.delete(id);
+        activeShards.delete(id);
+        erroredShards.set(id, {
+            id,
+            error: err,
+            phase: (_e = opts === null || opts === void 0 ? void 0 : opts.phase) !== null && _e !== void 0 ? _e : 'launch',
+            timestamp: Date.now(),
+        });
+        console.error(`[sh3] Shard "${id}" failed to activate:`, err);
+        throw err;
     }
-    void ((_e = shard.autostart) === null || _e === void 0 ? void 0 : _e.call(shard, ctx));
+    // Activation succeeded — clear any prior error record for this shard.
+    erroredShards.delete(id);
+    void ((_f = shard.autostart) === null || _f === void 0 ? void 0 : _f.call(shard, ctx));
 }
 /**
  * Deactivate an active shard. Calls `shard.deactivate`, flushes and disposes
@@ -288,4 +332,5 @@ export function __resetShardRegistryForTest() {
     active.clear();
     activeShards.clear();
     registeredShards.clear();
+    erroredShards.clear();
 }

package/dist/version.d.ts

@@ -1,2 +1,2 @@
 /** Auto-generated from package.json — do not edit manually. */
-export declare const VERSION = "0.11.7";
+export declare const VERSION = "0.12.0";

package/dist/version.js

@@ -1,2 +1,2 @@
 /** Auto-generated from package.json — do not edit manually. */
-export const VERSION = '0.11.7';
+export const VERSION = '0.12.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sh3-core",
-  "version": "0.11.7",
+  "version": "0.12.0",
   "type": "module",
   "files": [
     "dist"