Remove rendering logic from collections

Author: brianpetro

smart-chats/smart_thread.js

@@ -83,7 +83,7 @@ export class SmartThread extends SmartSource {
    * @returns {Promise<DocumentFragment>} The rendered thread interface.
    */
   async render(container = this.container, opts = {}) {
-    const frag = await this.render_component('thread', opts);
+    const frag = await this.env.render_component('thread', this, opts);
     if (container) {
       container.empty();
       if (container.classList.contains('sc-thread')) {

smart-collections/collection.js

@@ -460,15 +460,6 @@ export class Collection {
     this.env.collections[this.collection_key] = null;
   }
 
-  /**
-   * Helper function to render a component in the collection scope
-   * @param {*} component_key 
-   * @param {*} opts 
-   * @returns 
-   */
-  async render_component(component_key, opts = {}) {
-    return await this.env.render_component(component_key, this, opts);
-  }
   // only show process notice if taking longer than 1 second
   show_process_notice(process, opts = {}) {
     if(!this.debounce_process_notice) this.debounce_process_notice = {};

smart-collections/item.js

@@ -258,16 +258,6 @@ export class CollectionItem {
    */
   parse() { /* NO-OP by default */ }
 
-  /**
-   * Helper function to render a component in the item scope
-   * @param {*} component_key 
-   * @param {*} opts 
-   * @returns 
-   */
-  async render_component(component_key, opts = {}) {
-    return await this.env.render_component(component_key, this, opts);
-  }
-
   get actions() {
     if(!this._actions) {
       this._actions = Object.entries(this.env.opts.items[this.item_type_key].actions || {}).reduce((acc, [k,v]) => {
@@ -357,19 +347,19 @@ export class CollectionItem {
     this.env.smart_settings.save();
   }
 
-  /**
-   * Render this item into a container using the item's component.
-   * @deprecated 2024-12-02 Use explicit component pattern from environment
-   * @param {HTMLElement} container
-   * @param {Object} opts
-   * @returns {Promise<HTMLElement>}
-   */
-  async render_item(container, opts = {}) {
-    const frag = await this.component.call(this.smart_view, this, opts);
-    this.env.smart_view.empty(container);
-    container.appendChild(frag);
-    return container;
-  }
+  // /**
+  //  * Render this item into a container using the item's component.
+  //  * @deprecated 2024-12-02 Use explicit component pattern from environment
+  //  * @param {HTMLElement} container
+  //  * @param {Object} opts
+  //  * @returns {Promise<HTMLElement>}
+  //  */
+  // async render_item(container, opts = {}) {
+  //   const frag = await this.component.call(this.smart_view, this, opts);
+  //   this.env.smart_view.empty(container);
+  //   container.appendChild(frag);
+  //   return container;
+  // }
 
   /**
    * @deprecated use env.smart_view
@@ -380,12 +370,12 @@ export class CollectionItem {
     return this._smart_view;
   }
 
-  /**
-   * Override in child classes to set the component for this item
-   * @deprecated 2024-12-02
-   * @returns {Function} The render function for this component
-   */
-  get component() { return item_component; }
+  // /**
+  //  * Override in child classes to set the component for this item
+  //  * @deprecated 2024-12-02
+  //  * @returns {Function} The render function for this component
+  //  */
+  // get component() { return item_component; }
 }
 
 /**

smart-entities/smart_entity.js

@@ -306,33 +306,13 @@ export class SmartEntity extends CollectionItem {
    */
   get path() { return this.data.path; }
 
-  /**
-   * Gets the component responsible for rendering the entity.
-   * @readonly
-   * @returns {Function} The render function for the entity component.
-   */
-  get component() { return render_entity_component; }
 
   get is_unembedded() {
     if(!this.vec) return true;
     if(!this.embed_hash || this.embed_hash !== this.read_hash) return true;
     return false;
   }
 
-  // COMPONENTS 2024-11-27
-  get connections_component() {
-    if(!this._connections_component) this._connections_component = this.components?.connections?.bind(this.smart_view);
-    return this._connections_component;
-  }
-  async render_connections(container, opts={}) {
-    if(container) this.env.smart_view.safe_inner_html(container, 'Loading connections...');
-    const frag = await this.env.render_component('connections', this, opts);
-    if(container) {
-      this.env.smart_view.empty(container);
-      container.appendChild(frag);
-    }
-    return frag;
-  }
 }
 
 import { find_connections } from "./actions/find_connections.js";