slicejs-web-framework 3.1.3 → 3.1.4

package/Slice/Slice.js

@@ -105,18 +105,23 @@ export default class Slice {
          return null;
       }
 
-      if (!this.controller.componentCategories.has(componentName)) {
-         this.logger.logError('Slice', null, `Component ${componentName} not found in components.js file`);
-         return null;
-      }
-
       // 📦 Try to load from bundles first
       const bundleName = this.controller.getBundleForComponent(componentName);
       if (bundleName && !this.controller.loadedBundles.has(bundleName)) {
          await this.controller.loadBundle(bundleName);
       }
 
+      // After bundle loading attempt, allow build when class is already available
+      // even if components map has no category entry (stale/components.js mismatch).
+      if (!this.controller.componentCategories.has(componentName) && !this.controller.classes.has(componentName)) {
+         this.logger.logError('Slice', null, `Component ${componentName} not found in components.js file`);
+         return null;
+      }
+
       let componentCategory = this.controller.componentCategories.get(componentName);
+      if (!componentCategory && this.controller.classes.has(componentName)) {
+         componentCategory = 'AppComponents';
+      }
 
       // 📦 Check if component is already available from loaded bundles
       const isFromBundle = this.controller.isComponentFromBundle(componentName);
@@ -130,7 +135,7 @@ export default class Slice {
          return null;
       }
 
-      let isVisual = slice.paths.components[componentCategory].type === 'Visual';
+      let isVisual = slice.paths.components[componentCategory]?.type === 'Visual';
       let modulePath = `${slice.paths.components[componentCategory].path}/${componentName}/${componentName}.js`;
       const isJsOnlyVisualComponent = isVisual && (componentName === 'MultiRoute' || componentName === 'Route');
 

package/Slice/tests/build-bundled-component-without-category.test.js

@@ -0,0 +1,103 @@
+import test from 'node:test';
+import assert from 'node:assert/strict';
+
+globalThis.alert = () => {};
+
+const { default: Slice } = await import('../Slice.js');
+
+class FakeController {
+  constructor() {
+    this.componentCategories = new Map();
+    this.templates = new Map();
+    this.classes = new Map();
+    this.requestedStyles = new Set();
+    this.loadedBundles = new Set();
+    this.activeComponents = new Map();
+    this.bundleLoads = 0;
+  }
+
+  getBundleForComponent(componentName) {
+    return componentName === 'DocumentationPage' ? 'multiroute-DocumentationPage--p1' : null;
+  }
+
+  async loadBundle(bundleName) {
+    this.bundleLoads += 1;
+    this.loadedBundles.add(bundleName);
+    this.classes.set(
+      'DocumentationPage',
+      class DocumentationPage {
+        constructor(props) {
+          this.props = props;
+          this.sliceId = 'DocumentationPage';
+        }
+        async init() {}
+      }
+    );
+  }
+
+  isComponentFromBundle(componentName) {
+    return componentName === 'DocumentationPage' && this.loadedBundles.has('multiroute-DocumentationPage--p1');
+  }
+
+  verifyComponentIds() {
+    return true;
+  }
+
+  registerComponent() {}
+
+  registerComponentsRecursively() {}
+}
+
+class FakeStylesManager {
+  registerComponentStyles() {}
+}
+
+function createSlice() {
+  const instance = new Slice(
+    {
+      paths: {
+        components: {
+          AppComponents: {
+            path: '/Components/AppComponents',
+            type: 'Visual'
+          }
+        }
+      },
+      themeManager: {},
+      stylesManager: {},
+      logger: {},
+      debugger: { enabled: false },
+      loading: {},
+      events: {}
+    },
+    {
+      Controller: FakeController,
+      StylesManager: FakeStylesManager
+    }
+  );
+
+  instance.logger = {
+    logError() {},
+    logWarning() {},
+    logInfo() {}
+  };
+
+  return instance;
+}
+
+test('build succeeds for bundled component even when componentCategories is missing entry', async () => {
+  const originalSlice = globalThis.slice;
+
+  try {
+    const sliceInstance = createSlice();
+    globalThis.slice = sliceInstance;
+
+    const component = await sliceInstance.build('DocumentationPage', { from: 'route' });
+
+    assert.ok(component);
+    assert.equal(component.sliceId, 'DocumentationPage');
+    assert.equal(sliceInstance.controller.bundleLoads, 1);
+  } finally {
+    globalThis.slice = originalSlice;
+  }
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slicejs-web-framework",
-  "version": "3.1.3",
+  "version": "3.1.4",
   "description": "",
   "engines": {
     "node": ">=20"