@adia-ai/web-modules 0.8.42 → 0.8.43

package/shell/admin-sidebar/admin-sidebar.js

@@ -30,9 +30,9 @@
  *   sidebar-resize  — bubbles. detail: { name, width }
  *
  * The drag handle is conventional: a child `[data-sidebar-resize]` element
- * (`[data-resize]` still works — deprecated, dual-read compat shim, removed
- * in 0.9.0: docs/ops/plan/plan-090-rename-wave.md). Authors may provide a
- * custom one, or omit (no resize affordance).
+ * (its deprecated pre-0.9.0 name was cut at the rename-wave W3, gh#1562:
+ * docs/ops/plan/plan-090-rename-wave.md). Authors may provide a custom one,
+ * or omit (no resize affordance).
  *
  * Backwards compat: <admin-shell> still recognizes the legacy
  * `<aside data-sidebar="leading">` shape via :is() selector. New
@@ -57,15 +57,10 @@ class AdminSidebar extends UIElement {
 
   static template = () => null;
 
-  // FEEDBACK-17: elements already warned about a missing [data-resize]
+  // FEEDBACK-17: elements already warned about a missing [data-sidebar-resize]
   // handle — one-shot per element so re-renders stay quiet. GC-friendly.
   static #warnedNoHandle = new WeakSet();
 
-  // Rename-wave compat (docs/ops/plan/plan-090-rename-wave.md W1): one-shot
-  // deprecation warn per element when the legacy `[data-resize]` name is the
-  // one that resolved the handle. Same GC-friendly WeakSet pattern as above.
-  static #warnedOldResizeName = new WeakSet();
-
   // The width the sidebar had before being collapsed — used for restore.
   // Map keyed by sidebar name allows multiple sidebars on one host.
   #previousExpandedWidth = '';
@@ -173,22 +168,10 @@ class AdminSidebar extends UIElement {
   // ── Resize drag handle ──
 
   #setupResizeHandle() {
-    // Rename-wave compat (plan-090 W1): `[data-sidebar-resize]` is the
-    // current name; `[data-resize]` still dual-reads until the 0.9.0 cut
-    // (W3), warning once per element when it's the name that resolved.
-    let handle = this.querySelector(':scope > [data-sidebar-resize]');
-    if (!handle) {
-      handle = this.querySelector(':scope > [data-resize]');
-      if (handle && !AdminSidebar.#warnedOldResizeName.has(this)) {
-        AdminSidebar.#warnedOldResizeName.add(this);
-        // eslint-disable-next-line no-console
-        console.warn(
-          '[admin-sidebar] `[data-resize]` is deprecated and removed in ' +
-          '0.9.0 — rename the drag-handle child to `[data-sidebar-resize]`.',
-          this,
-        );
-      }
-    }
+    // Rename-wave cut (plan-090 W3, gh#1562): `[data-sidebar-resize]` is the
+    // only recognized handle name — its deprecated pre-0.9.0 dual-read was
+    // removed at the cut.
+    const handle = this.querySelector(':scope > [data-sidebar-resize]');
     if (!handle) {
       // FEEDBACK-17: `resizable` opts in to drag-resize but relies on an
       // author-supplied `[data-sidebar-resize]` child. A silent return left