@adia-ai/web-modules 0.6.20 → 0.6.21

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog — @adia-ai/web-modules
 
+## [0.6.21] — 2026-05-21
+
+### Changed — `<admin-entity-item>` row insets aligned to shared select / nav-item-ui row convention
+
+- **`--entity-item-gap`: `--a-space-2` → `--a-space-1`** (matches `--select-trigger-gap` + `--nav-item-row-gap`) and **NEW `--entity-item-px: --a-ui-px` token** with `padding: 0 var(--entity-item-px)` on the host rule (matches `--select-px` + `--nav-item-row-px`). Identity rows now read as the same shared row vocabulary used inside `<select-ui>` triggers and `<nav-item-ui>` rows — visibly tighter gap between icon / label / badge, plus consistent inline padding. Both tunable: a consumer who finds the inset double-pads inside a particular parent context can set `--entity-item-px: 0` on a per-use basis. Files: `css/admin-shell.entity-item.css`.
+
+### Fixed — `admin-shell` no longer emits a spurious slot-contract `console.warn`
+
+- **The one-shot `console.warn` for `<admin-topbar>` / `<admin-statusbar>` missing `slot="header"` / `slot="footer"` (shipped in 0.6.20, FEEDBACK-37) is reverted — it was a false positive.** `<admin-content>` and `<admin-sidebar>` are CSS-only light-DOM elements with no shadow root, so a `slot=` attribute is inert (no projection), and no CSS selector targets `[slot="header"]` / `[slot="footer"]` for them — the topbar/statusbar are positioned purely by tag + DOM order (`admin-content > admin-topbar { … }`). The bars render identically with or without the slot; the warn fired on correct markup with a message that was factually wrong ("renders in the default body slot"). FEEDBACK-37's premise — that the slot is load-bearing — did not hold. Removed: `admin-shell.js` `#checkSlotContracts()` + the `MUST carry slot` `a2ui.rules` in `admin-topbar.yaml` / `admin-statusbar.yaml`.
+
+### Fixed — page-header title row no longer stacks under the `?chunks` dev overlay
+
+- **The `<admin-page-header>` title row (`h1` left, actions right) used a `:first-child` selector that broke when the `?chunks=1` dev overlay was active.** `site/dev-chunks.js` prepends a `<span data-chunk-marker>` into every `[data-chunk]` element; on a page-header `<header data-chunk="…">` that span displaced the title `<div>` from `:first-child`, so the flex-row rule (`display: flex; justify-content: space-between`) stopped matching — the title and the action buttons stacked vertically instead of sharing a row. The selector is now `div:first-of-type`, which a prepended non-`<div>` sibling cannot displace. Production (overlay off) was unaffected; this fixes the chunks-on dev view. Files: `css/admin-shell.templates.css`.
+
 ## [0.6.20] — 2026-05-21
 
 ### Added — `<admin-entity-item>` shell identity-row primitive (FEEDBACK-38)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adia-ai/web-modules",
-  "version": "0.6.20",
+  "version": "0.6.21",
   "description": "AdiaUI composite custom elements \u2014 shell, chat, editor, runtime clusters built from @adia-ai/web-components primitives. Subpath exports per cluster.",
   "type": "module",
   "exports": {

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

@@ -40,10 +40,6 @@ class AdminShell extends UIElement {
 
   static template = () => null;
 
-  // FEEDBACK-37: one-shot per-element warn for <admin-topbar>/<admin-statusbar>
-  // mounted without their slot. WeakSet so it never pins the element from GC.
-  static #warnedSlots = new WeakSet();
-
   connected() {
     // Apply the v0.6.13 default mode if (and only if) the author did
     // not supply a `mode` attribute on the tag. `hasAttribute` returns
@@ -52,7 +48,6 @@ class AdminShell extends UIElement {
     if (!this.hasAttribute('mode')) this.mode = DEFAULT_MODE;
     this.#wireToggleButtons();
     this.#wireCommandTriggers();
-    this.#checkSlotContracts();
   }
 
   // No #disconnected — children own their own cleanup; the host registers
@@ -100,33 +95,6 @@ class AdminShell extends UIElement {
       if (item) nav.select(item);
     });
   }
-
-  // ── 3. Slot-contract diagnostics (FEEDBACK-37) ──
-  // <admin-topbar>/<admin-statusbar> are CSS-only — placed inside
-  // <admin-sidebar>/<admin-content> without slot="header"/"footer" they
-  // land in the default slot and render in the body column instead of the
-  // chrome band, with no diagnostic. One-shot warn per misused element.
-  // (admin-statusbar folded in — identical failure mode to admin-topbar.)
-  #checkSlotContracts() {
-    const want = { 'admin-topbar': 'header', 'admin-statusbar': 'footer' };
-    for (const parent of this.querySelectorAll('admin-sidebar, admin-content')) {
-      for (const child of parent.children) {
-        const slot = want[child.localName];
-        if (slot
-            && child.getAttribute('slot') !== slot
-            && !AdminShell.#warnedSlots.has(child)) {
-          AdminShell.#warnedSlots.add(child);
-          // eslint-disable-next-line no-console
-          console.warn(
-            `[admin-shell] <${child.localName}> inside <${parent.localName}> is missing ` +
-            `slot="${slot}" — it renders in the default body slot instead of the chrome ` +
-            `band. Add slot="${slot}". See ${child.localName}.yaml.`,
-            child,
-          );
-        }
-      }
-    }
-  }
 }
 
 customElements.define('admin-shell', AdminShell);

package/shell/admin-shell/css/admin-shell.entity-item.css

@@ -22,8 +22,13 @@
    ═══════════════════════════════════════════════════════════════ */
 
 admin-entity-item {
-  /* ── tunable tokens ── */
-  --entity-item-gap:         var(--a-space-2);
+  /* ── tunable tokens ──
+     Gap + horizontal padding match <select-ui>'s trigger row and
+     <nav-item-ui>'s row (shared convention: --a-space-1 between
+     slots, --a-ui-px on each inline edge). Consumers can opt out
+     by setting --entity-item-px: 0 on a per-use basis. */
+  --entity-item-gap:         var(--a-space-1);
+  --entity-item-px:          var(--a-ui-px);
   --entity-item-icon-size:   1rem;
   --entity-item-icon-color:  var(--a-fg-muted);
   --entity-item-avatar-size: 1.5rem;
@@ -31,6 +36,7 @@ admin-entity-item {
   display: inline-flex;
   align-items: center;
   gap: var(--entity-item-gap);
+  padding: 0 var(--entity-item-px);
   min-width: 0;
   overflow: hidden;
 }

package/shell/admin-shell/css/admin-shell.templates.css

@@ -83,8 +83,14 @@ admin-page-header > :is(header, header-ui)[data-compact] {
   padding-block: var(--page-header-px);
 }
 
-/* Title row: h1 left, actions right */
-admin-page-header > :is(header, header-ui) > div:first-child {
+/* Title row: h1 left, actions right.
+   `:first-of-type` (not `:first-child`) so the row still matches when a
+   non-<div> element is prepended ahead of it — e.g. the
+   `<span data-chunk-marker>` the `?chunks` dev overlay (site/dev-chunks.js)
+   injects as the first child of every `[data-chunk]` element. With
+   `:first-child` the prepended span won the match and the title + actions
+   stacked instead of sitting on one row. */
+admin-page-header > :is(header, header-ui) > div:first-of-type {
   display: flex;
   align-items: center;
   justify-content: space-between;

package/shell/admin-statusbar/admin-statusbar.yaml

@@ -50,12 +50,6 @@ a2ui:
       admin-statusbar replaces <footer-ui> at shell-tier. Same slot
       vocabulary as <admin-topbar>; visual treatment differs (the
       shell css applies a top-border instead of bottom).
-    - >-
-      When placed inside <admin-sidebar> or <admin-content>, admin-statusbar
-      MUST carry slot="footer". Without it the statusbar lands in the
-      default body slot instead of the chrome footer band. Canonical:
-      <admin-statusbar slot="footer">…</admin-statusbar>. admin-shell logs
-      a one-shot console.warn when the slot is missing.
     - >-
       For a user-identity row (avatar + name + role badge), slot a single
       <admin-entity-item slot="heading"> rather than separate slot="icon" +

package/shell/admin-topbar/admin-topbar.yaml

@@ -56,12 +56,6 @@ a2ui:
       admin-topbar replaces <header-ui> at shell-tier — use it for
       chrome bars inside admin-shell, admin-content, admin-sidebar.
       Use <header-ui> for primitive containers (Card / Drawer / Modal).
-    - >-
-      When placed inside <admin-sidebar> or <admin-content>, admin-topbar
-      MUST carry slot="header". Without it the topbar lands in the default
-      body slot and renders below the content instead of as the chrome
-      header band. Canonical: <admin-topbar slot="header">…</admin-topbar>.
-      admin-shell logs a one-shot console.warn when the slot is missing.
     - >-
       For an icon + label (+ optional badge) identity row — workspace or
       product identity — slot a single <admin-entity-item slot="heading">