@forgeax/interface 0.9.1 → 0.9.3

package/dist/package/browser/appHostBootstrap.js

@@ -61,14 +61,12 @@ export async function bootstrapAppHost(overrides = {}) {
         ...m,
         async setup(ctx) {
             control.beginSetup(m);
-            const menuCleanups = [];
+            const cleanups = [];
+            let pageCleanup;
             try {
-                const cleanups = [];
-                let pageCleanup;
                 for (const menu of m.contributes?.menus ?? []) {
-                    menuCleanups.push(host.menus.register(menu));
+                    cleanups.push(host.menus.register(menu));
                 }
-                cleanups.push(...menuCleanups);
                 if (m.contributes?.panels) {
                     cleanups.push(control.contributePanels(m.id, m.contributes.panels));
                 }
@@ -100,7 +98,10 @@ export async function bootstrapAppHost(overrides = {}) {
                 };
             }
             catch (error) {
-                for (const cleanup of menuCleanups.slice().reverse())
+                // Preserve the same page-close decision boundary on failed setup.
+                // A dirty page must not lose its surrounding contributions underneath it.
+                await pageCleanup?.();
+                for (const cleanup of cleanups.slice().reverse())
                     await cleanup();
                 throw error;
             }

package/dist/package/browser/core/panels/panel-action-registry.d.ts

@@ -1,6 +1,4 @@
-import type { Cleanup } from '../extension-foundation/types';
-import type { PanelActionContribution, PanelActionsApi } from './types';
-export interface PanelActionRegistry extends PanelActionsApi {
-    contribute(owner: string, actions: readonly PanelActionContribution[]): Cleanup;
-}
-export declare function createPanelActionRegistry(): PanelActionRegistry;
+import { type PanelActionRegistry as Registry } from '@forgeax/app-shell/application';
+import type { PanelActionContribution } from './types';
+export type PanelActionRegistry = Registry<PanelActionContribution>;
+export declare const createPanelActionRegistry: () => Registry<PanelActionContribution>;

package/dist/package/browser/core/panels/panel-action-registry.js

@@ -1,62 +1,2 @@
-export function createPanelActionRegistry() {
-    const entries = [];
-    const listeners = new Set();
-    let cache = null;
-    let version = 0;
-    const emit = () => {
-        cache = null;
-        version++;
-        for (const listener of [...listeners])
-            listener();
-    };
-    // Batch-deferred notification: data mutations (push/splice) happen
-    // immediately so reads (list/all) always return fresh data, but listener
-    // notifications are deferred to a microtask. This avoids triggering
-    // forceStoreRerender (useSyncExternalStore) during React 19's commit
-    // phase, which otherwise causes "Maximum update depth exceeded".
-    let emitScheduled = false;
-    const scheduleEmit = () => {
-        cache = null;
-        if (emitScheduled)
-            return;
-        emitScheduled = true;
-        queueMicrotask(() => {
-            emitScheduled = false;
-            emit();
-        });
-    };
-    const all = () => {
-        if (cache)
-            return cache;
-        cache = entries.flatMap((entry) => entry.actions).sort((a, b) => (a.order ?? 0) - (b.order ?? 0));
-        return cache;
-    };
-    return {
-        contribute(owner, actions) {
-            const entry = { owner, actions };
-            entries.push(entry);
-            scheduleEmit();
-            let removed = false;
-            return () => {
-                if (removed)
-                    return;
-                removed = true;
-                const index = entries.indexOf(entry);
-                if (index >= 0)
-                    entries.splice(index, 1);
-                scheduleEmit();
-            };
-        },
-        list(panelId) {
-            return all().filter((action) => action.panelId === panelId);
-        },
-        all,
-        onChange(listener) {
-            listeners.add(listener);
-            return () => { listeners.delete(listener); };
-        },
-        version() {
-            return version;
-        },
-    };
-}
+import { createPanelActionRegistry as createRegistry, } from '@forgeax/app-shell/application';
+export const createPanelActionRegistry = (createRegistry);

package/dist/package/browser/core/panels/panel-control-registry.d.ts

@@ -1,6 +1,4 @@
-import type { Cleanup } from '../extension-foundation/types';
-import type { PanelControlContribution, PanelControlsApi } from './types';
-export interface PanelControlRegistry extends PanelControlsApi {
-    contribute(owner: string, controls: readonly PanelControlContribution[]): Cleanup;
-}
-export declare function createPanelControlRegistry(): PanelControlRegistry;
+import { type PanelControlRegistry as Registry } from '@forgeax/app-shell/application';
+import type { PanelControlContribution } from './types';
+export type PanelControlRegistry = Registry<PanelControlContribution>;
+export declare const createPanelControlRegistry: () => Registry<PanelControlContribution>;

package/dist/package/browser/core/panels/panel-control-registry.js

@@ -1,54 +1,2 @@
-export function createPanelControlRegistry() {
-    const entries = [];
-    const listeners = new Set();
-    let version = 0;
-    const emit = () => {
-        version++;
-        for (const listener of [...listeners])
-            listener();
-    };
-    // Batch-deferred notification: data mutations (push/splice) happen
-    // immediately so reads (list/get) always return fresh data, but listener
-    // notifications are deferred to a microtask. This avoids triggering
-    // forceStoreRerender (useSyncExternalStore) during React 19's commit
-    // phase, which otherwise causes "Maximum update depth exceeded".
-    let emitScheduled = false;
-    const scheduleEmit = () => {
-        if (emitScheduled)
-            return;
-        emitScheduled = true;
-        queueMicrotask(() => {
-            emitScheduled = false;
-            emit();
-        });
-    };
-    const list = () => entries.flatMap((entry) => entry.controls);
-    return {
-        contribute(owner, controls) {
-            const entry = { owner, controls };
-            entries.push(entry);
-            scheduleEmit();
-            let removed = false;
-            return () => {
-                if (removed)
-                    return;
-                removed = true;
-                const index = entries.indexOf(entry);
-                if (index >= 0)
-                    entries.splice(index, 1);
-                scheduleEmit();
-            };
-        },
-        get(id) {
-            return list().find((control) => control.id === id);
-        },
-        list,
-        onChange(listener) {
-            listeners.add(listener);
-            return () => { listeners.delete(listener); };
-        },
-        version() {
-            return version;
-        },
-    };
-}
+import { createPanelControlRegistry as createRegistry, } from '@forgeax/app-shell/application';
+export const createPanelControlRegistry = (createRegistry);

package/dist/package/node/appHostBootstrap.js

@@ -61,14 +61,12 @@ export async function bootstrapAppHost(overrides = {}) {
         ...m,
         async setup(ctx) {
             control.beginSetup(m);
-            const menuCleanups = [];
+            const cleanups = [];
+            let pageCleanup;
             try {
-                const cleanups = [];
-                let pageCleanup;
                 for (const menu of m.contributes?.menus ?? []) {
-                    menuCleanups.push(host.menus.register(menu));
+                    cleanups.push(host.menus.register(menu));
                 }
-                cleanups.push(...menuCleanups);
                 if (m.contributes?.panels) {
                     cleanups.push(control.contributePanels(m.id, m.contributes.panels));
                 }
@@ -100,7 +98,10 @@ export async function bootstrapAppHost(overrides = {}) {
                 };
             }
             catch (error) {
-                for (const cleanup of menuCleanups.slice().reverse())
+                // Preserve the same page-close decision boundary on failed setup.
+                // A dirty page must not lose its surrounding contributions underneath it.
+                await pageCleanup?.();
+                for (const cleanup of cleanups.slice().reverse())
                     await cleanup();
                 throw error;
             }

package/dist/package/node/core/panels/panel-action-registry.js

@@ -1,62 +1,2 @@
-export function createPanelActionRegistry() {
-    const entries = [];
-    const listeners = new Set();
-    let cache = null;
-    let version = 0;
-    const emit = () => {
-        cache = null;
-        version++;
-        for (const listener of [...listeners])
-            listener();
-    };
-    // Batch-deferred notification: data mutations (push/splice) happen
-    // immediately so reads (list/all) always return fresh data, but listener
-    // notifications are deferred to a microtask. This avoids triggering
-    // forceStoreRerender (useSyncExternalStore) during React 19's commit
-    // phase, which otherwise causes "Maximum update depth exceeded".
-    let emitScheduled = false;
-    const scheduleEmit = () => {
-        cache = null;
-        if (emitScheduled)
-            return;
-        emitScheduled = true;
-        queueMicrotask(() => {
-            emitScheduled = false;
-            emit();
-        });
-    };
-    const all = () => {
-        if (cache)
-            return cache;
-        cache = entries.flatMap((entry) => entry.actions).sort((a, b) => (a.order ?? 0) - (b.order ?? 0));
-        return cache;
-    };
-    return {
-        contribute(owner, actions) {
-            const entry = { owner, actions };
-            entries.push(entry);
-            scheduleEmit();
-            let removed = false;
-            return () => {
-                if (removed)
-                    return;
-                removed = true;
-                const index = entries.indexOf(entry);
-                if (index >= 0)
-                    entries.splice(index, 1);
-                scheduleEmit();
-            };
-        },
-        list(panelId) {
-            return all().filter((action) => action.panelId === panelId);
-        },
-        all,
-        onChange(listener) {
-            listeners.add(listener);
-            return () => { listeners.delete(listener); };
-        },
-        version() {
-            return version;
-        },
-    };
-}
+import { createPanelActionRegistry as createRegistry, } from '@forgeax/app-shell/application';
+export const createPanelActionRegistry = (createRegistry);

package/dist/package/node/core/panels/panel-control-registry.js

@@ -1,54 +1,2 @@
-export function createPanelControlRegistry() {
-    const entries = [];
-    const listeners = new Set();
-    let version = 0;
-    const emit = () => {
-        version++;
-        for (const listener of [...listeners])
-            listener();
-    };
-    // Batch-deferred notification: data mutations (push/splice) happen
-    // immediately so reads (list/get) always return fresh data, but listener
-    // notifications are deferred to a microtask. This avoids triggering
-    // forceStoreRerender (useSyncExternalStore) during React 19's commit
-    // phase, which otherwise causes "Maximum update depth exceeded".
-    let emitScheduled = false;
-    const scheduleEmit = () => {
-        if (emitScheduled)
-            return;
-        emitScheduled = true;
-        queueMicrotask(() => {
-            emitScheduled = false;
-            emit();
-        });
-    };
-    const list = () => entries.flatMap((entry) => entry.controls);
-    return {
-        contribute(owner, controls) {
-            const entry = { owner, controls };
-            entries.push(entry);
-            scheduleEmit();
-            let removed = false;
-            return () => {
-                if (removed)
-                    return;
-                removed = true;
-                const index = entries.indexOf(entry);
-                if (index >= 0)
-                    entries.splice(index, 1);
-                scheduleEmit();
-            };
-        },
-        get(id) {
-            return list().find((control) => control.id === id);
-        },
-        list,
-        onChange(listener) {
-            listeners.add(listener);
-            return () => { listeners.delete(listener); };
-        },
-        version() {
-            return version;
-        },
-    };
-}
+import { createPanelControlRegistry as createRegistry, } from '@forgeax/app-shell/application';
+export const createPanelControlRegistry = (createRegistry);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forgeax/interface",
-  "version": "0.9.1",
+  "version": "0.9.3",
   "type": "module",
   "description": "ForgeaX product interface shell and application composition boundary.",
   "license": "Apache-2.0",
@@ -92,7 +92,7 @@
     "lint:dep": "depcruise -c .dependency-cruiser.cjs src"
   },
   "dependencies": {
-    "@forgeax/app-shell": "0.88.0",
+    "@forgeax/app-shell": "0.89.0",
     "@forgeax/extension-host": "0.3.2",
     "@forgeax/extension-platform": "0.5.0",
     "@forgeax/toolkit": "0.1.2",