@arcgis/coding-components 4.31.0-next.6 → 4.31.0-next.61

package/dist/components/arcgis-sql-expression-editor.js

@@ -1,12 +1,13 @@
 /*!
  * All material copyright Esri, All Rights Reserved, unless otherwise specified.
  * See https://js.arcgis.com/4.31/esri/copyright.txt for details.
- * v4.31.0-next.6
+ * v4.31.0-next.61
  */
-import { proxyCustomElement, HTMLElement, forceUpdate, h, Host } from '@stencil/core/internal/client';
-import { u as useControllerManager } from './chunk-3IUJHIAA.js';
+import { proxyCustomElement, HTMLElement, createEvent, forceUpdate, h, Host } from '@stencil/core/internal/client';
+import { u as useControllerManager } from './chunk-WEX2VUZV.js';
 import { g as generateGuid } from './index2.js';
 import { s as sqlExprDefaults } from './sql-expr-defaults.js';
+import { u as useT9n } from './useT9n.js';
 import { d as defineCustomElement$4 } from './code-editor.js';
 import { d as defineCustomElement$3 } from './language-api-panel.js';
 import { d as defineCustomElement$2 } from './sql-expression-fields.js';
@@ -16,34 +17,49 @@ import "@esri/calcite-components/dist/components/calcite-action.js";
 const sqlExpressionEditorCss = "arcgis-sql-expression-editor{display:flex;flex-direction:column;position:relative;overflow:hidden;border:1px solid var(--calcite-color-border-3);box-sizing:border-box;background-color:var(--calcite-color-foreground-1);height:100%}arcgis-sql-expression-editor .flex-adjustable{flex:1 1 auto;overflow:hidden}arcgis-sql-expression-editor .flex-panel{flex:1 0 auto}arcgis-sql-expression-editor .position-relative{position:relative}arcgis-sql-expression-editor .flex-row{display:flex;flex-direction:row;overflow:hidden}arcgis-sql-expression-editor .flex-column{display:flex;flex-direction:column;overflow:hidden}arcgis-sql-expression-editor .border-inline-start{border-inline-start:1px solid var(--calcite-color-border-3);box-sizing:border-box}arcgis-sql-expression-editor .border-inline-end{border-inline-end:1px solid var(--calcite-color-border-3);box-sizing:border-box}arcgis-sql-expression-editor .side-action-bar{max-width:25%}arcgis-sql-expression-editor .side-panel{max-width:45%;width:600px}arcgis-sql-expression-editor .side-panel calcite-flow{height:100%}";
 const ArcgisSqlExpressionEditorStyle0 = sqlExpressionEditorCss;
 
-const __cmpMeta = [0,"arcgis-sql-expression-editor",{"profile":[1040],"_preparing":[32],"_editorProfile":[32],"_activeAction":[32],"_apiLibrary":[32]},undefined,{"profile":["_profileChanged"]}];
+const __cmpMeta = [0,"arcgis-sql-expression-editor",{"profile":[1040],"script":[1025],"_preparing":[32],"_editorProfile":[32],"_activeAction":[32],"_apiLibrary":[32],"messages":[32]},undefined,{"messages":["_messagesChanged"],"profile":["_profileChanged"]}];
 const SqlExpressionEditor = /*@__PURE__*/ proxyCustomElement(class SqlExpressionEditor extends HTMLElement {
     connectedCallback() { this.manager._connectedCallback(); }
     constructor() {
         super();
         this.__registerHost();
 this.constructor.__registerControllers = ()=>__cmpMeta[2]??{};
+        this.arcgisScriptChange = createEvent(this, "arcgisScriptChange", 3);
         this.manager = useControllerManager(this,forceUpdate);
         // #endregion
         // #region Private properties
         this._modelId = generateGuid();
         this._insertAsSnippet = (e) => void this._codeEditorElt?.insertSnippet(e.detail).catch(console.error);
+        this._updateApiLibrary = async () => {
+            this._apiLibrary = await sqlExprDefaults.getApiLibrary(this._modelId);
+        };
+        this._onCodeEditorValueChange = (e) => {
+            e.stopPropagation();
+            this.script = e.detail;
+            this.arcgisScriptChange.emit(e.detail);
+        };
         this._preparing = false;
         this._editorProfile = undefined;
         this._activeAction = undefined;
         this._apiLibrary = undefined;
+        this.messages = useT9n();
         this.profile = undefined;
+        this.script = "";
+    }
+    async _messagesChanged() {
+        await this._updateEditorProfile().catch(console.error);
+        await this._updateApiLibrary().catch(console.error);
     }
-    _profileChanged() {
-        this._updateSqlExprDefaults().catch(console.error);
+    async _profileChanged() {
+        await this._updateEditorProfile().catch(console.error);
+        await this._updateApiLibrary().catch(console.error);
     }
     // #endregion
     // #region Private methods API
-    async _updateSqlExprDefaults() {
+    async _updateEditorProfile() {
         this._preparing = true;
         try {
             await sqlExprDefaults.setProfileForModel(this._modelId, this.profile);
-            this._apiLibrary = await sqlExprDefaults.getApiLibrary(this._modelId);
         }
         catch {
             this._editorProfile = undefined;
@@ -63,23 +79,29 @@ this.constructor.__registerControllers = ()=>__cmpMeta[2]??{};
     // #endregion
     // #region Component lifecycle events
     async componentWillLoad() {
-        await this._updateSqlExprDefaults().catch(console.error);
+        await this._updateEditorProfile().catch(console.error);
         // The profile has to be refreshed on locale change
         // this.manager.watch("messages", () => void this._updateEditorProfile().catch(console.error));
     }
+    async componentDidLoad() {
+        // We do updateApiLibrary in componentDidLoad because we need to wait until code editor
+        // is rendered for worker to register and allow for fetching the sdk
+        await this._updateApiLibrary();
+    }
     disconnectedCallback() {
         this.manager._disconnectedCallback();
         sqlExprDefaults.disposeForModel(this._modelId);
     }
     // #endregion
     render() {
-        return (h(Host, { key: '67f7ebf7a28a8ffc3fe967721389d45c4e2003a1' }, h("div", { key: 'b3bf4ec2a8d04105e97489593ccec95cd33237f0', class: "flex-row flex-adjustable" }, h("div", { key: 'fdac47e22baf64c72f786f9e09b637050f417f94', class: "flex-column flex-adjustable" }, h("arcgis-code-editor", { key: 'f4151dd39ec3523907cd907ebb7558e05bca9cc1', language: "arcgis-sql-expression", modelId: this._modelId, ref: (e) => (this._codeEditorElt = e) })), this._activeAction === "fields" && (h("arcgis-sql-expression-fields", { key: 'aa0e5d88429d4bf865b9adc506028efefec1a73a', class: "side-panel flex-panel  border-inline-start", loading: this._preparing, modelId: this._modelId, "data-panel-name": "none", onArcgisFieldSelected: this._insertAsSnippet, onArcgisClose: () => this._setActiveAction(undefined) })), this._activeAction === "functions" && (h("arcgis-language-api-panel", { key: '7826dbb1b08894c6c6385806a3aef6cff77395d0', class: "side-panel flex-panel border-inline-start", onArcgisItemSelected: this._insertAsSnippet, onArcgisClose: () => this._setActiveAction(undefined), loading: this._preparing, apiLibrary: this._apiLibrary, "data-panel-name": "none" })), h("calcite-action-bar", { key: 'ee618d666b70405495627a9d15238e8723c6f961', class: "side-action-bar border-inline-start", expanded: false, position: "end" }, h("calcite-action", { key: '2c828df5dded6347e13e59ba9e313824baa606ff', id: "fields-action", text: "Fields", icon: "profile-variables", active: this._activeAction === "fields", "data-panel-name": "fields",
+        return (h(Host, { key: '63348b6e3e5aa7544203c5ec7270db7827c6f3ed' }, h("div", { key: '203ed656208717c877c540ed5c49fc12338d6b15', class: "flex-row flex-adjustable" }, h("div", { key: '4dfe400bbc9ab74ef86c4115ceeabbcb41913427', class: "flex-column flex-adjustable" }, h("arcgis-code-editor", { key: 'dfc9e247b39d2d9901ac911e147d993897809954', language: "arcgis-sql-expression", modelId: this._modelId, ref: (e) => (this._codeEditorElt = e), value: this.script ?? "", onArcgisValueChange: this._onCodeEditorValueChange })), this._activeAction === "fields" && (h("arcgis-sql-expression-fields", { key: 'b5a5516264aae278f3feec00a5e20e2d0855024d', class: "side-panel flex-panel  border-inline-start", loading: this._preparing, modelId: this._modelId, "data-panel-name": "none", onArcgisFieldSelected: this._insertAsSnippet, onArcgisClose: () => this._setActiveAction(undefined) })), this._activeAction === "functions" && (h("arcgis-language-api-panel", { key: '6da236e289e8bcdadc27cadece6e1fb34c55b1da', class: "side-panel flex-panel border-inline-start", onArcgisItemSelected: this._insertAsSnippet, onArcgisClose: () => this._setActiveAction(undefined), loading: this._preparing, apiLibrary: this._apiLibrary, "data-panel-name": "none" })), h("calcite-action-bar", { key: '91549ebac7e78165b911e5def1141010032dfa43', class: "side-action-bar border-inline-start", expanded: false, position: "end" }, h("calcite-action", { key: '32211e3b4e8db0530a047ba6c22234d15e504e17', id: "fields-action", text: this.messages.fields ?? "Fields", icon: "profile-variables", active: this._activeAction === "fields", "data-panel-name": "fields",
             // need to use an arrow function to automatically bind 'this'
-            onClick: (e) => this._handleActionClick(e) }), h("calcite-action", { key: 'eab785c5b14cadeb5047db1957134f5accfedc76', id: "functions-action", text: "Functions", icon: "function", active: this._activeAction === "functions", "data-panel-name": "functions", onClick: (e) => this._handleActionClick(e) })))));
+            onClick: (e) => this._handleActionClick(e) }), h("calcite-action", { key: '3c308d994b45c67df76414580d881a082333d2f8', id: "functions-action", text: this.messages.functions ?? "Functions", icon: "function", active: this._activeAction === "functions", "data-panel-name": "functions", onClick: (e) => this._handleActionClick(e) })))));
     }
     static get assetsDirs() { return ["assets"]; }
     get el() { return this; }
     static get watchers() { return {
+        "messages": ["_messagesChanged"],
         "profile": ["_profileChanged"]
     }; }
     static get style() { return ArcgisSqlExpressionEditorStyle0; }

package/dist/components/arcgis-sql-expression-fields.js

@@ -1,7 +1,7 @@
 /*!
  * All material copyright Esri, All Rights Reserved, unless otherwise specified.
  * See https://js.arcgis.com/4.31/esri/copyright.txt for details.
- * v4.31.0-next.6
+ * v4.31.0-next.61
  */
 import { S as SqlExpressionFields, d as defineCustomElement$1 } from './sql-expression-fields.js';
 

package/dist/components/{chunk-3IUJHIAA.js → chunk-WEX2VUZV.js}

@@ -1,7 +1,7 @@
 /*!
  * All material copyright Esri, All Rights Reserved, unless otherwise specified.
  * See https://js.arcgis.com/4.31/esri/copyright.txt for details.
- * v4.31.0-next.6
+ * v4.31.0-next.61
  */
 import { a as isNotUndefined, D as Deferred, k as safeAsyncCall, l as devToolsAwareTimeout, m as safeCall } from './index2.js';
 
@@ -15,7 +15,7 @@ function retrieveComponentMembers(component, isLit) {
         ([name, descriptor]) => descriptor.noAccessor ? void 0 : [
           name,
           [
-            (descriptor.readOnly ? 2048 /* ReadOnly */ : 0) | (descriptor.state ? 32 /* State */ : descriptor.type === Number ? 2 /* Number */ : descriptor.type === Boolean ? 4 /* Boolean */ : 1 /* String */)
+            (descriptor.readOnly ? 2048 /* ReadOnly */ : 0) | (descriptor.state ? 32 /* State */ : 16 /* Unknown */)
           ]
         ]
       ).filter(isNotUndefined)
@@ -145,6 +145,7 @@ var shouldBypass = {
 };
 var elementToInstance = /* @__PURE__ */ new WeakMap();
 var _a;
+_a = controllerSymbol;
 var Controller = class {
   constructor(component) {
     this._callbacks = {
@@ -160,8 +161,8 @@ var Controller = class {
     this._ready = new Deferred();
     this._lifecycleCleanups = [];
     this.connectedCalled = false;
-    this.willLoadCalled = false;
-    this.didLoadCalled = false;
+    this._loadCalled = false;
+    this.loadedCalled = false;
     this[_a] = true;
     this.ready = this._ready.promise;
     /*
@@ -193,12 +194,12 @@ var Controller = class {
     if (!connectedWillStillHappen && !this.connectedCalled) {
       this.triggerConnected();
     }
-    const loadWillStillHappen = !manager.willLoadCalled;
+    const loadWillStillHappen = !manager._loadCalled;
     if (loadWillStillHappen) {
       return;
     }
     this.triggerLoad().then(() => {
-      const loadedWillStillHappen = !manager.didLoadCalled;
+      const loadedWillStillHappen = !manager.loadedCalled;
       if (loadedWillStillHappen) {
         return;
       }
@@ -340,10 +341,10 @@ var Controller = class {
   }
   /** @internal */
   async triggerLoad() {
-    if (this.willLoadCalled) {
+    if (this._loadCalled) {
       return;
     }
-    this.willLoadCalled = true;
+    this._loadCalled = true;
     const genericController = this;
     if (genericController.hostLoad) {
       await safeAsyncCall(genericController.hostLoad, genericController);
@@ -355,7 +356,7 @@ var Controller = class {
   }
   /** @internal */
   triggerLoaded() {
-    if (this.didLoadCalled) {
+    if (this.loadedCalled) {
       return;
     }
     const genericController = this;
@@ -363,7 +364,7 @@ var Controller = class {
       safeCall(genericController.hostLoaded, genericController);
     }
     this._callbacks.hostLoaded.forEach(safeCall);
-    this.didLoadCalled = true;
+    this.loadedCalled = true;
   }
   /** @internal */
   triggerUpdate(changedProperties) {
@@ -410,7 +411,6 @@ var Controller = class {
     });
   }
 };
-_a = controllerSymbol;
 function callUpdate(callback) {
   safeCall(callback, void 0, this);
 }
@@ -419,9 +419,9 @@ function makeProvisionalValue(base) {
   if (typeof base !== "object" && typeof base !== "function" || base === null) {
     return base;
   }
-  const proxy2 = new Proxy(base, {
+  const proxy = new Proxy(base, {
     get(target, prop, receiver) {
-      if (cyclical.has(prop) && prop in target && target[prop] === proxy2) {
+      if (cyclical.has(prop) && prop in target && target[prop] === proxy) {
         return void 0;
       }
       if (prop in target || prop in Promise.prototype || typeof prop === "symbol") {
@@ -433,7 +433,7 @@ function makeProvisionalValue(base) {
       return Reflect.set(target, prop, newValue, receiver);
     }
   });
-  return proxy2;
+  return proxy;
 }
 var cyclical = /* @__PURE__ */ new Set(["exports", "_exports"]);
 function toControllerHost(component) {
@@ -500,14 +500,35 @@ function watch(component, property, callback) {
 var ComponentInternals = class {
   constructor(component) {
     /**
-     * When watchers are set, set then into `allWatchers`. When watchers are read
-     * in the setter, read from `enabledWatchers`.
-     * On connectedCallback(), controller manager does `enabledWatchers=allWatchers`.
-     * Reasoning:
+     * When watchers are set, they are set into `allWatchers`. When watchers are
+     * read in the setter, they are read from `enabledWatchers`.
+     *
+     * In Stencil, on connectedCallback(), controller manager does
+     * `enabledWatchers=allWatchers`. Reasoning:
      * - This disables watchers until connected callback (matches behavior of
      *   Stencil's watchers)
-     * - This removes in the setter to check if watchers were enabled already or
-     *   not (as getters/setters are hot path, and should be streamlined)
+     * - This removes the need for a check in the setter for whether the watchers
+     *   were enabled already or not (as getters/setters are hot path, and should
+     *   be streamlined)
+     *
+     * In Lit, we set enabledWatchers to allWatchers in the constructor.
+     * Reasoning:
+     * - While in Stencil all user provided properties are either set before the
+     *   component constructor (this is possible since Stencil's props store is
+     *   externalized) or after connectedCallback, in Lit the properties may be
+     *   set by attributeChangedCallback before connectedCallback. Thus, we need
+     *   to enable watchers even before connectedCallback.
+     * - This means, that a watcher set for some prop before your component got
+     *   a chance to set the default value will trigger the watcher when the
+     *   default value is set - but, that is inline with Lit's willUpdate behavior
+     *   of triggering for default values AND, the only way to set a watcher
+     *   before default value is if you called watcher inside a controller. For
+     *   now, there are no controllers written outside of arcgis-map-components
+     *   package, so I was able to verify that this change would have no
+     *   negative impact.
+     *
+     * In either way, I plan to remove watchers from controllers once Stencil
+     * support is removed.
      */
     this.enabledWatchers = {};
     this.allWatchers = {};
@@ -585,7 +606,10 @@ var ComponentInternals = class {
   _getSetProxy(prototype, name2, type) {
     const component = this.component;
     const propType = getMemberType(component, name2);
-    const descriptor = Object.getOwnPropertyDescriptor(prototype, name2);
+    let descriptor;
+    do {
+      descriptor = Object.getOwnPropertyDescriptor(prototype, name2);
+    } while (!descriptor && (prototype = Object.getPrototypeOf(prototype)));
     if (!descriptor) {
       return void 0;
     }
@@ -593,11 +617,11 @@ var ComponentInternals = class {
     const originalSet = descriptor.set;
     const isGetterAlreadyOverwritten = customAccessor in originalGet;
     const isSetterAlreadyOverwritten = customAccessor in originalSet;
-    const shouldOverwriteGet = !isGetterAlreadyOverwritten;
-    const shouldOverwriteSet = !isSetterAlreadyOverwritten;
-    if (!shouldOverwriteGet && !shouldOverwriteSet) {
+    if (isGetterAlreadyOverwritten && isSetterAlreadyOverwritten) {
       return;
     }
+    const shouldOverwriteGet = !isGetterAlreadyOverwritten;
+    const shouldOverwriteSet = !isSetterAlreadyOverwritten;
     const finalGetter = shouldOverwriteGet ? function getter() {
       let value = originalGet.call(this);
       const component2 = elementToInstance.get(this);
@@ -619,10 +643,20 @@ var ComponentInternals = class {
         originalSet.call(this, rawNewValue);
         return;
       }
-      let newValue = component2.manager.isLit ? rawNewValue : parsePropertyValue(rawNewValue, propType);
+      let newValue = component2.manager.isLit ? (
+        /*
+         * Cast `null` to `undefined`. Reasons:
+         * - component can declare properties simply using
+         *   `@property() name?: string;` and get accurate typings,
+         *   without being forced to use
+         *   `@property() name?: string | Nil;`.
+         * - simplifies checks in the code and thus reduces bugs
+         */
+        rawNewValue ?? void 0
+      ) : parsePropertyValue(rawNewValue, propType);
       const internals = component2.manager.internals;
       if (newValue === oldValue) {
-        originalSet.call(this, rawNewValue);
+        originalSet.call(this, newValue);
       } else {
         const setters = shouldBypass.setter ? emptyArray : internals.setters[name2] ?? emptyArray;
         for (let i = 0; i < setters.length; i++) {
@@ -683,7 +717,7 @@ var ControllerManager = class extends GenericController {
     const controllers = /* @__PURE__ */ new Set();
     function addController(controller) {
       controllers.add(controller);
-      if (isLit && !(controllerSymbol in controller) && component.el.isConnected) {
+      if (!(controllerSymbol in controller) && component.renderRoot && component.el.isConnected) {
         controller.hostConnected?.();
       }
     }
@@ -710,9 +744,6 @@ var ControllerManager = class extends GenericController {
     this._controllers = controllers;
     this.exports = void 0;
     this.hasDestroy = autoDestroyDisabledPropName in this.component && typeof this.component.destroy === "function";
-    if (this.hasDestroy) {
-      watch(component, autoDestroyDisabledPropName, () => this._setAutoDestroyTimeout());
-    }
     this._bindLifecycleMethods();
     const internals = this.internals;
     Object.keys(internals.members).forEach((name2) => {
@@ -720,7 +751,9 @@ var ControllerManager = class extends GenericController {
       internals.accessorSetter[name2] = defaultGetterSetter;
       internals.getSetProxy(name2);
     });
-    if (!isLit) {
+    if (isLit) {
+      this.internals.enabledWatchers = this.internals.allWatchers;
+    } else {
       Object.defineProperty(component, "updateComplete", {
         get: async () => await this._updatePromise.promise
       });
@@ -737,13 +770,13 @@ var ControllerManager = class extends GenericController {
     const isStencilDistBuild = component.el === component;
     this._originalLifecycles = {
       // These component's callbacks will be called by Lit, so we don't have to
-      connectedCallback: isLit || isStencilDistBuild ? void 0 : component.connectedCallback,
-      disconnectedCallback: isLit || isStencilDistBuild ? void 0 : component.disconnectedCallback,
-      componentWillLoad: isLit ? void 0 : component.componentWillLoad,
-      componentDidLoad: isLit ? void 0 : component.componentDidLoad,
-      componentWillUpdate: isLit ? void 0 : component.componentWillUpdate,
-      componentDidUpdate: isLit ? void 0 : component.componentDidUpdate,
-      destroy: component.destroy
+      _connectedCallback: isLit || isStencilDistBuild ? void 0 : component.connectedCallback,
+      _disconnectedCallback: isLit || isStencilDistBuild ? void 0 : component.disconnectedCallback,
+      _load: isLit ? component.load : component.componentWillLoad,
+      _loaded: isLit ? component.loaded : component.componentDidLoad,
+      _willUpdate: isLit ? void 0 : component.componentWillUpdate,
+      _updated: isLit ? void 0 : component.componentDidUpdate,
+      _destroy: component.destroy
     };
     const hostConnected = this._connectedCallback.bind(this);
     const hostDisconnected = this._disconnectedCallback.bind(this);
@@ -792,7 +825,7 @@ var ControllerManager = class extends GenericController {
     keyTrackResolve();
     internals.enableReadonly?.();
     this._controllers.forEach(callConnected);
-    this._originalLifecycles.connectedCallback?.call(this.component);
+    this._originalLifecycles._connectedCallback?.call(this.component);
   }
   /** @internal */
   _disconnectedCallback() {
@@ -800,7 +833,7 @@ var ControllerManager = class extends GenericController {
       return;
     }
     this._controllers.forEach(callDisconnected);
-    this._originalLifecycles.disconnectedCallback?.call(this.component);
+    this._originalLifecycles._disconnectedCallback?.call(this.component);
     if (this.hasDestroy) {
       this._setAutoDestroyTimeout();
     }
@@ -818,22 +851,25 @@ var ControllerManager = class extends GenericController {
       }
     }
     await Promise.allSettled(Array.from(this._controllers, callLoad, integrations));
-    await this._originalLifecycles.componentWillLoad?.call(this.component);
+    await this._originalLifecycles._load?.call(this.component);
+    if (this.hasDestroy) {
+      watch(this.component, autoDestroyDisabledPropName, () => this._setAutoDestroyTimeout());
+    }
   }
   /** @internal */
   _loaded() {
     this._controllers.forEach(callLoaded);
-    this._originalLifecycles.componentDidLoad?.call(this.component);
+    this._originalLifecycles._loaded?.call(this.component);
   }
   _update() {
     const changedProperties = this.component.changedProperties;
     this._controllers.forEach(callUpdate2, changedProperties);
-    void this._originalLifecycles.componentWillUpdate?.call(this.component);
+    this._originalLifecycles._willUpdate?.call(this.component);
   }
   _updated() {
     const maybeLitComponent = this.component;
     this._controllers.forEach(callUpdated, maybeLitComponent.changedProperties);
-    this._originalLifecycles.componentDidUpdate?.call(this.component);
+    this._originalLifecycles._updated?.call(this.component);
     if (this.isLit) {
       maybeLitComponent.changedProperties = /* @__PURE__ */ new Map();
     } else {
@@ -858,13 +894,13 @@ var ControllerManager = class extends GenericController {
     this.destroyed = true;
     this._controllers.forEach(callDestroy);
     this._controllers.clear();
-    await this._originalLifecycles.destroy?.call(this.component);
+    await this._originalLifecycles._destroy?.call(this.component);
   }
   _setAutoDestroyTimeout() {
+    if (this._autoDestroyTimeout !== void 0) {
+      clearTimeout(this._autoDestroyTimeout);
+    }
     if (!this.component.el.isConnected && !this.component.autoDestroyDisabled) {
-      if (this._autoDestroyTimeout !== void 0) {
-        clearTimeout(this._autoDestroyTimeout);
-      }
       const destroy = () => void this.destroy().catch(console.error);
       {
         this._autoDestroyTimeout = devToolsAwareTimeout(destroy, autoDestroyOnDisconnectTimeout);
@@ -946,7 +982,7 @@ function trackKey(hostsCandidates, onResolved, defaultValue) {
   const resolved = (resolution) => {
     leftToResolve -= 1;
     if (resolution !== void 0) {
-      leftToResolve = 0;
+      leftToResolve = Math.min(leftToResolve, 0);
     }
     if (leftToResolve === 0) {
       onResolved(resolution);
@@ -995,8 +1031,7 @@ function getSet(defaultValue, getSet2) {
       const genericComponent = component;
       const value = genericComponent[name2];
       const isStencilHotReload = "production" !== "production"  ;
-      const initialSet = getSet2.initialSet ?? true;
-      if (value != null && value !== defaultValue && typeof getSet2.set === "function" && initialSet && !isStencilHotReload) {
+      if (value != null && value !== defaultValue && typeof getSet2.set === "function" && !isStencilHotReload) {
         const newValue = getSet2.set(value, defaultValue, name2);
         if (newValue !== value) {
           if (manager.isLit) {
@@ -1111,44 +1146,4 @@ function setProxy(controller, { host, key, isReactive: assignedToProp }, initial
   });
 }
 
-// src/functional.ts
-var makeController = (constructor) => proxy(void 0, constructor);
-var FunctionalController = class extends Controller {
-  constructor(component, constructor) {
-    super(component);
-    const originalExports = this.exports;
-    try {
-      setAmbientComponent(this.component);
-      const value = constructor(this.component, this);
-      const constructorChangedExports = this.exports !== originalExports;
-      if (value instanceof Promise) {
-        if (!constructorChangedExports) {
-          this.setProvisionalExports(value);
-        }
-        const resolved = value.then((result) => {
-          this.exports = result;
-          super.catchUpLifecycle();
-        }).catch((error) => {
-          this._ready.reject(error);
-          console.error(error);
-        });
-        this.onLoad(async () => await resolved);
-      } else {
-        if (!constructorChangedExports || value !== void 0) {
-          this.exports = value;
-        }
-        queueMicrotask(() => super.catchUpLifecycle());
-      }
-    } catch (error) {
-      this._ready.reject(error);
-      console.error(error);
-    }
-  }
-  /** Noop - will be called in the constructor instead */
-  catchUpLifecycle() {
-    return;
-  }
-};
-var proxy = proxyExports(FunctionalController);
-
-export { getSet as g, makeController as m, useControllerManager as u, watch as w };
+export { Controller as C, getSet as g, proxyExports as p, setAmbientComponent as s, useControllerManager as u, watch as w };