@arcgis/lumina 4.31.0-next.98 → 4.31.1

package/dist/LitElement.d.ts

@@ -156,8 +156,17 @@ export declare class LitElement extends OriginalLitElement implements ComponentL
      * }
      * ```
      */
-    listen<K extends keyof HTMLElementEventMap>(type: K, listener: (this: this, event: HTMLElementEventMap[K]) => unknown, options?: AddEventListenerOptions | boolean): void;
-    listen(type: string, listener: (this: this, event: Event) => unknown, options?: AddEventListenerOptions | boolean): void;
+    listen<K extends keyof HTMLElementEventMap>(name: K, listener: (this: this, event: HTMLElementEventMap[K]) => unknown, options?: AddEventListenerOptions | boolean): void;
+    listen(name: string, listener: (this: this, event: Event) => unknown, options?: AddEventListenerOptions | boolean): void;
+    listen<EventType extends Event = CustomEvent<"Provide type like this.listenOn<ToEvents<ArcgisCounter>['arcgisClick']>() to get type-checked payload type">>(name: string, 
+    /**
+     * The "NoInfer" here forces type argument to be specified explicitly.
+     * Without it, the following would be allowed:
+     * ```tsx
+     * this.listen("focus",(event:NotFocusEvent)=>{....})
+     * ```
+     */
+    listener: (this: this, event: NoInfer<EventType>) => unknown, options?: AddEventListenerOptions | boolean): void;
     /**
      * A helper for setting even listener on any element (or window / document).
      *
@@ -195,7 +204,15 @@ export declare class LitElement extends OriginalLitElement implements ComponentL
     listenOn<Name extends keyof HTMLElementEventMap>(target: HTMLElement, name: Name, listener: Listener<this, HTMLElementEventMap[Name] & {
         currentTarget: HTMLElement;
     }>, options?: AddEventListenerOptions | boolean): void;
-    listenOn<EventType extends Event = CustomEvent<"Provide type like this.listenOn<ToEvents<ArcgisCounter>['arcgisClick']>() to get type-checked payload type">, Target = EventTarget>(target: Target, name: string, listener: Listener<this, EventType & {
+    listenOn<EventType extends Event = CustomEvent<"Provide type like this.listenOn<ToEvents<ArcgisCounter>['arcgisClick']>() to get type-checked payload type">, Target = EventTarget>(target: Target, name: string, 
+    /**
+     * The "NoInfer" here forces type argument to be specified explicitly.
+     * Without it, the following would be allowed:
+     * ```tsx
+     * this.listen("focus",(event:NotFocusEvent)=>{....})
+     * ```
+     */
+    listener: Listener<this, NoInfer<EventType> & {
         currentTarget: Target;
     }>, options?: AddEventListenerOptions | boolean): void;
     /**

package/dist/devOnlyDetectIncorrectLazyUsages.d.ts

@@ -4,7 +4,10 @@ import type { LitElement } from "./LitElement";
  * Instead, a proxy element is present and should be used for all DOM actions.
  *
  * In practice, this means using this.el.getAttribute() instead of
- * this.getAttribute() and etc for each DOM method
+ * this.getAttribute() and etc for each DOM method.
+ *
+ * To detect incorrect usages, this function overwrites each DOM member on the
+ * LitElement prototype with an exception-throwing function.
  *
  * This code does not ship in production builds.
  */

package/dist/index.js

@@ -61,11 +61,22 @@ import { Deferred, camelToKebab } from "@arcgis/components-utils";
 // src/devOnlyDetectIncorrectLazyUsages.ts
 function devOnlyDetectIncorrectLazyUsages(LitClass) {
   const genericPrototype = LitClass.prototype;
+  const firstGetter = Object.getOwnPropertyDescriptor(genericPrototype, "innerText");
+  const secondGetter = Object.getOwnPropertyDescriptor(genericPrototype, "innerHTML");
+  if (firstGetter === secondGetter && firstGetter !== void 0) {
+    return;
+  }
   const allowList = /* @__PURE__ */ new Set([
     // We shouldn't be overwriting this property
     "constructor",
     // Called by Lit - we proxy it to this.el in ProxyComponent
-    "setAttribute"
+    "setAttribute",
+    // Called by Lit SSR - we proxy it to this.el in ProxyComponent
+    "removeAttribute",
+    // Called by Lit - we proxy it to this.el in ProxyComponent
+    "isConnected",
+    // Called by Lit, but only in dev mode for warnings, so we don't have to proxy.
+    "localName"
   ]);
   const customErrorMessages = {
     addEventListener: "use this.listen() or this.el.addEventListener()"
@@ -75,28 +86,36 @@ function devOnlyDetectIncorrectLazyUsages(LitClass) {
     ...Object.getOwnPropertyDescriptors(Element.prototype),
     ...Object.getOwnPropertyDescriptors(Node.prototype),
     ...Object.getOwnPropertyDescriptors(EventTarget.prototype)
-  }).filter(([key, { value }]) => typeof value === "function" && !allowList.has(key)).forEach(([key]) => {
-    genericPrototype[key] = (...args) => {
+  }).forEach(([key, value]) => {
+    if (allowList.has(key)) {
+      return;
+    }
+    const callback = (...args) => {
       if (key === "hasAttribute" && args[0] === "defer-hydration") {
         return false;
       }
       throw new Error(
-        `You should not be calling this.${key}() directly as it won't work correctly in lazy-builds. Instead, ${customErrorMessages[key] ?? `use this.el.${key}()`}`
+        `You should not be trying to access this.${key} directly as it won't work correctly in lazy-builds. Instead, ${customErrorMessages[key] ?? `use this.el.${key}`}`
       );
     };
+    if (typeof value === "function") {
+      genericPrototype[key] = callback;
+    } else {
+      Object.defineProperty(genericPrototype, key, { get: callback, set: callback });
+    }
   });
 }
 
 // src/lifecycleSupport.ts
-function attachToAncestor(child, el) {
-  let ancestor = el;
+function attachToAncestor(child) {
+  let ancestor = child;
   while (ancestor = ancestor.parentNode ?? ancestor.host) {
     if (ancestor?.constructor?.lumina) {
       const litParent = ancestor;
       if (!litParent.manager?.loadedCalled) {
         litParent._offspring.push(child);
       }
-      return litParent._postLoad?.promise;
+      return litParent._postLoad.promise;
     }
   }
   return false;
@@ -151,6 +170,7 @@ function createLazyElement([tagName, [load, compactMeta = ""]]) {
   };
   customElements.define(tagName, ProxyClass);
 }
+var defineProperty = Object.defineProperty;
 function parseCondensedProp(propAndAttribute) {
   const name = propAndAttribute.split(lazyMetaSubItemJoiner);
   return name.length === 1 ? [name[0], camelToKebab(name[0])] : name;
@@ -194,7 +214,10 @@ var ProxyComponent = class extends HtmlElement {
     if (ProxyClass._LitConstructor) {
       this._initializeComponent({ a: ProxyClass._LitConstructor });
     } else {
-      void ProxyClass._loadPromise.then(this._initializeComponent.bind(this)).catch(this._postLoaded.reject);
+      void ProxyClass._loadPromise.then(this._initializeComponent.bind(this)).catch((error) => {
+        console.error(error);
+        this._postLoaded.reject(error);
+      });
     }
     if (process.env.NODE_ENV !== "production") {
       ProxyClass._hmrInstances ??= [];
@@ -211,7 +234,7 @@ var ProxyComponent = class extends HtmlElement {
     this._syncMethods?.forEach(this._bindSync, this);
   }
   static _bindProp(propName) {
-    Object.defineProperty(this.prototype, propName, {
+    defineProperty(this.prototype, propName, {
       configurable: true,
       enumerable: true,
       get() {
@@ -226,7 +249,7 @@ var ProxyComponent = class extends HtmlElement {
     });
   }
   static _bindAsync(methodName) {
-    Object.defineProperty(this.prototype, methodName, {
+    defineProperty(this.prototype, methodName, {
       async value(...args) {
         if (!this._litElement) {
           await this._postLoaded.promise;
@@ -240,7 +263,7 @@ var ProxyComponent = class extends HtmlElement {
     });
   }
   static _bindSync(methodName) {
-    Object.defineProperty(this.prototype, methodName, {
+    defineProperty(this.prototype, methodName, {
       value(...args) {
         if (process.env.NODE_ENV === "development" && !this._litElement) {
           const ProxyClass = this.constructor;
@@ -313,7 +336,7 @@ var ProxyComponent = class extends HtmlElement {
       this._litElement?.connectedCallback();
     } else {
       queueMicrotask(() => {
-        this._ancestorLoad = attachToAncestor(this, this);
+        this._ancestorLoad = attachToAncestor(this);
       });
     }
   }
@@ -352,10 +375,23 @@ var ProxyComponent = class extends HtmlElement {
     const isFirstInitialization = !ProxyClass._LitConstructor;
     if (isFirstInitialization) {
       ProxyClass._LitConstructor = LitConstructor;
-      LitConstructor.prototype.removeAttribute = removeAttribute;
-      LitConstructor.prototype.setAttribute = setAttribute;
+      LitConstructor.prototype.removeAttribute = function(qualifiedName) {
+        HTMLElement.prototype.removeAttribute.call(this.el, qualifiedName);
+      };
+      LitConstructor.prototype.setAttribute = function(qualifiedName, value) {
+        HTMLElement.prototype.setAttribute.call(this.el, qualifiedName, value);
+      };
+      defineProperty(LitConstructor.prototype, "isConnected", {
+        get() {
+          return this.el.isConnected;
+        }
+      });
       if (process.env.NODE_ENV !== "production" && LitConstructor.shadowRootOptions !== noShadowRoot) {
-        devOnlyDetectIncorrectLazyUsages(Object.getPrototypeOf(LitConstructor));
+        let prototype = LitConstructor;
+        while (prototype && !Object.hasOwn(prototype, "lumina")) {
+          prototype = Object.getPrototypeOf(prototype);
+        }
+        devOnlyDetectIncorrectLazyUsages(LitConstructor);
       }
       customElements.define(lazyTagName, LitConstructor);
     }
@@ -381,26 +417,20 @@ var ProxyComponent = class extends HtmlElement {
       const missingFromLit = lazyObserved.filter((attribute) => !litObserved.includes(attribute));
       if (missingFromLazy.length > 0) {
         console.warn(
-          `The following attributes on <${ProxyClass._name}> are present on the Lit element, but are missing from the lazy proxy component: ${missingFromLazy.join(", ")}. This either indicates a bug in Lumina, or you are creating the attribute dynamically in a way that compiler can not infer statically. For these attributes, lazy-loading version of your component won't work correctly, thus this must be resolved`
+          `The following attributes on <${ProxyClass._name}> are present on the Lit element, but are missing from the lazy proxy component: ${missingFromLazy.join(", ")}. This either indicates a bug in Lumina, or you are creating the attribute dynamically in a way that compiler cannot infer statically. For these attributes, lazy-loading version of your component won't work correctly, thus this must be resolved`
         );
       }
       if (missingFromLit.length > 0) {
         console.warn(
-          `The following attributes on <${ProxyClass._name}> are defined on the lazy proxy component, but not on the actual Lit element: ${missingFromLit.join(", ")}. This either indicates a bug in Lumina, or you are creating the attribute dynamically in a way that compiler can not infer statically. This is a non-critical issue, but does indicate that something is going wrong and should be fixed`
+          `The following attributes on <${ProxyClass._name}> are defined on the lazy proxy component, but not on the actual Lit element: ${missingFromLit.join(", ")}. This either indicates a bug in Lumina, or you are creating the attribute dynamically in a way that compiler cannot infer statically. This is a non-critical issue, but does indicate that something is going wrong and should be fixed`
         );
       }
     }
     if (this.isConnected) {
-      litElement.connectedCallback();
+      litElement.connectedCallback?.();
     }
   }
 };
-function removeAttribute(qualifiedName) {
-  HTMLElement.prototype.removeAttribute.call(this.el, qualifiedName);
-}
-function setAttribute(qualifiedName, value) {
-  HTMLElement.prototype.setAttribute.call(this.el, qualifiedName, value);
-}
 function syncLitElement([key, value]) {
   this[key] = value;
 }
@@ -542,13 +572,6 @@ var LitElement = class _LitElement extends OriginalLitElement {
      * @internal
      */
     this._offspring = this.constructor.lazy?._offspring ?? [];
-    /**
-     * Promise that resolves once parent's load() completed. False if there is no
-     * parent
-     *
-     * @internal
-     */
-    this._ancestorLoad = this.constructor.lazy?._ancestorLoad;
     this._postLoaded = this.constructor.lazy?._postLoaded ?? new Deferred2();
     this._enableUpdating = this.enableUpdating;
     this.enableUpdating = emptyFunction;
@@ -618,7 +641,12 @@ var LitElement = class _LitElement extends OriginalLitElement {
     const isFirstCall = !this.manager.connectedCalled;
     super.connectedCallback();
     if (isFirstCall) {
-      queueMicrotask(() => void this._load().catch(this._postLoaded.reject));
+      queueMicrotask(
+        () => void this._load().catch((error) => {
+          console.error(error);
+          this._postLoaded.reject(error);
+        })
+      );
     }
   }
   /**
@@ -652,7 +680,7 @@ var LitElement = class _LitElement extends OriginalLitElement {
   }
   /** Do asynchronous component load */
   async _load() {
-    const parentLoadPromise = this._ancestorLoad ?? attachToAncestor(this, this);
+    const parentLoadPromise = this.el._ancestorLoad ?? attachToAncestor(this.el);
     if (parentLoadPromise) {
       await parentLoadPromise;
     }
@@ -746,8 +774,9 @@ function makeRuntime(options) {
 }
 
 // src/jsx/jsx.ts
-import { directive as litDirective } from "lit-html/directive.js";
+import { directive as litDirective, Directive } from "lit-html/directive.js";
 import { noChange as litNoChange, nothing as litNothing } from "lit-html";
+var Fragment = void 0;
 var bindAttribute = void 0;
 var bindBooleanAttribute = void 0;
 var bindProperty = void 0;
@@ -755,6 +784,33 @@ var bindEvent = void 0;
 var nothing = litNothing;
 var noChange = litNoChange;
 var directive = litDirective;
+var DynamicHtmlValueDirective = class extends Directive {
+  update(part, [prop, value]) {
+    if (process.env.NODE_ENV !== "production") {
+      if (part.type !== 6) {
+        throw new Error("DynamicHtmlValueDirective can only be used in the element part position");
+      }
+      if (prop !== "value" && prop !== "defaultValue") {
+        throw new Error('Expected the first argument to DynamicHtmlValueDirective to be "value" or "defaultValue"');
+      }
+      if (typeof value === "object" && value != null) {
+        throw new Error(
+          `Tried to set an object as the value/defaultValue prop in a <${part.element.tagName}> element.`
+        );
+      }
+    }
+    const element = part.element;
+    const tagName = element.tagName;
+    if (prop === "value" ? tagName === "INPUT" : tagName === "BUTTON" || tagName === "DATA") {
+      element[prop] = value;
+    } else if (value != null) {
+      element.setAttribute("value", value);
+    } else {
+      element.removeAttribute("value");
+    }
+  }
+};
+var dynamicValueDirective = directive(DynamicHtmlValueDirective);
 
 // src/jsx/directives.ts
 import { classMap } from "lit-html/directives/class-map.js";
@@ -782,6 +838,7 @@ function createPrototypeProxy(tagName) {
   return customElement;
 }
 export {
+  Fragment,
   LitElement,
   bindAttribute,
   bindBooleanAttribute,
@@ -790,6 +847,7 @@ export {
   createEvent,
   createPrototypeProxy,
   directive,
+  dynamicValueDirective,
   handleComponentMetaUpdate,
   handleHmrUpdate,
   makeDefineCustomElements,

package/dist/jsx/jsx.d.ts

@@ -9,12 +9,10 @@
 /// <reference path="../typings/jsxGlobals.d.ts" />
 import type { Properties as CssProperties } from "csstype";
 import type { TemplateResult } from "lit-html";
-import type { DirectiveResult } from "lit-html/directive.js";
-import type { ClassMapDirective } from "lit-html/directives/class-map.js";
+import type { DirectiveResult, DirectiveClass } from "lit-html/directive.js";
+import type { ClassMapDirective, ClassInfo } from "lit-html/directives/class-map.js";
 import type { StyleMapDirective } from "lit-html/directives/style-map.js";
 import type { Ref } from "lit-html/directives/ref.js";
-import type { ClassInfo } from "lit/directives/class-map.js";
-import type { DirectiveClass } from "lit/directive.js";
 /**
  * The "h" namespace is used to import JSX types for elements and attributes.
  * It is imported in order to avoid conflicting global JSX issues.
@@ -24,31 +22,21 @@ export declare namespace h {
     export type { LuminaJsx as JSX };
 }
 /**
- * Fragment
- *
- * @remarks
- * This function is not actually defined in the code since references to it will
- * be removed at build time by the jsxToLitHtml plugin.
+ * The references to this function are removed at build time.
  */
-export declare function Fragment(props: {
+export declare const Fragment: (props: {
     children?: JsxNode;
-}): TemplateResult;
-/**
- * @remarks
- * This function is not actually defined in the code since references to it will
- * be removed at build time by the jsxToLitHtml plugin.
- */
-export declare function h(sel: any, data?: any, text?: any): TemplateResult;
+}) => TemplateResult;
 /**
- * @remarks
- * This function is not actually defined in the code since references to it will
- * be removed at build time by the jsxToLitHtml plugin.
+ * @internal
+ * The references to this function are removed at build time. You do not need
+ * to import it directly
  */
 export declare function jsx(type: string, props: unknown, key?: unknown): JsxNode;
 /**
- * @remarks
- * This function is not actually defined in the code since references to it will
- * be removed at build time by the jsxToLitHtml plugin.
+ * @internal
+ * The references to this function are removed at build time. You do not need
+ * to import it directly
  */
 export declare function jsxs(type: string, props: unknown, key?: unknown): JsxNode;
 /**
@@ -215,6 +203,14 @@ export declare const noChange: never;
  * "never" to allow it be set as a value for any JSX attribute.
  */
 export declare const directive: <C extends DirectiveClass>(c: C) => (...values: Parameters<InstanceType<C>["render"]>) => never;
+/**
+ * Do not import this directly. It will be inserted automatically by JSX to
+ * lit-html transformer when you are setting "value" or "defaultValue" JSX prop
+ * in an element with dynamic tag name.
+ *
+ * @internal
+ */
+export declare const dynamicValueDirective: (...values: never) => never;
 /**
  * this.el property on a component only has the public properties of the
  * component. All internal methods, properties, as well as LitElement methods
@@ -1078,6 +1074,7 @@ export declare namespace LuminaJsx {
         fetchPriority?: "high" | "low" | "auto";
     }
     interface InputHTMLAttributes<T> extends HTMLAttributes<T> {
+        autocomplete?: AutoFill;
         accept?: string;
         alt?: string;
         autofocus?: boolean;
@@ -1409,6 +1406,7 @@ export declare namespace LuminaJsx {
         required?: boolean;
         rows?: number | string;
         value?: string | number;
+        defaultValue?: string | number;
         wrap?: "hard" | "soft" | "off";
         maxLength?: number | string;
         minLength?: number | string;

package/dist/lazyLoad.d.ts

@@ -105,7 +105,7 @@ export declare abstract class ProxyComponent extends HtmlElement {
     /**
      * Direct offspring that should be awaited before loaded() is emitted
      */
-    _offspring: (Partial<Pick<LitElement, "manager">> & Pick<LitElement, "componentOnReady">)[];
+    _offspring: (Element & Partial<Pick<LitElement, "manager">> & Pick<LitElement, "componentOnReady">)[];
     /**
      * Promise that resolves once parent's load() completed. False if there is no
      * parent

package/dist/lifecycleSupport.d.ts

@@ -5,4 +5,4 @@ import type { ProxyComponent } from "./lazyLoad";
  * Also, return a promise that will delay our load() until parent's load()
  * is completed.
  */
-export declare function attachToAncestor(child: ProxyComponent["_offspring"][number], el: HTMLElement): Promise<void> | false;
+export declare function attachToAncestor(child: ProxyComponent["_offspring"][number]): Promise<void> | false;

package/dist/runtime.d.ts

@@ -3,7 +3,7 @@ import type { LitElement } from "./LitElement";
 /**
  * `@arcgis/lumina` package may be bundled once but used by multiple packages with
  * different configuration options. For that reason, the configuration options
- * can not be a global object, but has to be an object that you pass around.
+ * cannot be a global object, but has to be an object that you pass around.
  */
 export type Runtime = RuntimeOptions & {
     /**

package/dist/stencil-ssr-compatibility/index.js

@@ -2,7 +2,7 @@
 import { getWindow } from "@lit-labs/ssr/lib/dom-shim.js";
 import { render } from "@lit-labs/ssr/lib/render.js";
 import { html } from "@lit-labs/ssr/lib/server-template.js";
-import { unsafeStatic, withStatic } from "lit/static-html.js";
+import { unsafeStatic, withStatic } from "lit-html/static.js";
 import { collectResult } from "@lit-labs/ssr/lib/render-result.js";
 import { RenderResultReadable } from "@lit-labs/ssr/lib/render-result-readable.js";
 var staticHtml = withStatic(html);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arcgis/lumina",
-  "version": "4.31.0-next.98",
+  "version": "4.31.1",
   "type": "module",
   "main": "dist/index.cjs",
   "module": "dist/index.js",
@@ -17,33 +17,14 @@
   "files": [
     "dist/"
   ],
-  "publishConfig": {
-    "registry": "https://registry.npmjs.org/",
-    "access": "public"
-  },
   "license": "SEE LICENSE IN LICENSE.md",
-  "scripts": {
-    "build": "tsup",
-    "build:dev": "tsup --sourcemap",
-    "watch": "yarn build:dev --watch",
-    "clean": "rimraf ./dist ./build ./turbo ./node_modules"
-  },
   "dependencies": {
-    "@arcgis/components-controllers": "4.31.0-next.98",
-    "@arcgis/components-utils": "4.31.0-next.98",
+    "@arcgis/components-controllers": "4.31.1",
+    "@arcgis/components-utils": "4.31.1",
     "@lit-labs/ssr": "^3.2.2",
     "@lit-labs/ssr-client": "^1.1.7",
     "csstype": "^3.1.3",
     "lit": "^3.1.2",
     "tslib": "^2.7.0"
-  },
-  "devDependencies": {
-    "@arcgis/typescript-config": "4.31.0-next.98",
-    "@types/node": "^20.2.5",
-    "eslint": "^8.55.0",
-    "rimraf": "^5.0.0",
-    "tsup": "^8.2.4",
-    "typescript": "~5.4.0"
-  },
-  "gitHead": "cbe08697f9589db42635f6184bf7bd3dafcb2d79"
-}
+  }
+}