@jini-ai/admin 0.4.7 → 0.4.9

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.4.9 — 2026-10-13
+
+- Sitemap modal: Regenerate no longer reads `/sitemap.xml` twice. The regenerate mutation already invalidates the cached sitemap read (since 1c5d476f), so the modal's extra refetch is gone; the modal still shows the fresh XML after a successful regenerate and keeps the old XML after a failed one.
+
+## 0.4.8 — 2026-10-13
+
+- Share-image warning checks decoded size against 1200×630 (width and height independently; was width ≥ 600). New copy key: `The share image is only {height} px tall; use an image at least 630 px tall.`; the width key now says 1200 px.
+- SEO rules import `interpolate` from the universal `@jini-ai/ui/i18n`, so the SEO entries stay React-free. UI peer floor ^0.5.1.
+
 ## 0.4.7 — 2026-10-13
 
 - Accept UI ^0.5.0 for the chart contract extraction.

package/dist/seo/models.d.ts

@@ -57,9 +57,9 @@ export interface SeoController {
     defaultOgImage: string;
     setDefaultOgImage: (value: string) => void;
     /** Resolves `true` on success, `false` on a caught failure (`error` is set either way this
-     *  already did) — `SitemapModal.tsx`'s own footer Regenerate button uses the boolean to decide
-     *  whether to refetch `/sitemap.xml`, so a failed attempt never re-fetches the same stale
-     *  content. The pre-existing caller, `react/pages/SeoPage.tsx`'s own Regenerate button, still just fires this as
+     *  already did). Success invalidates the cached `/sitemap.xml` read, so `SitemapModal.tsx`'s
+     *  footer Regenerate button shows fresh XML without its own refetch; a failed attempt leaves
+     *  the viewed XML alone. The pre-existing caller, `react/pages/SeoPage.tsx`'s own Regenerate button, still just fires this as
      *  a plain `onClick` and discards the return value — this is a widening, not a breaking change. */
     regenerateSitemap: () => Promise<boolean>;
     /** Whether `SitemapModal.tsx` ("View sitemap") is open. `react/pages/SeoPage.tsx` stays markup-only (this file's
@@ -136,9 +136,8 @@ export interface SitemapModalController {
     setFilter: (value: string) => void;
     view: SitemapModalView;
     setView: (view: SitemapModalView) => void;
-    /** Re-runs the fetch — `SitemapModal.tsx`'s footer Regenerate button calls this after a
-     *  successful regenerate, so the count/table/raw view all reflect the freshly rebuilt sitemap
-     *  without the operator closing and reopening the modal. */
+    /** Re-runs the fetch. Retained for the hook ABI; after a successful regenerate the mutation's
+     *  cache invalidation already refreshes the count/table/raw view, so the modal no longer calls it. */
     refetch: () => void;
 }
 export interface MediaRefFieldController {

package/dist/seo/react/components/SitemapModal.d.ts

@@ -12,9 +12,8 @@ import { useWiredSitemapModal } from "../hooks/use-sitemap-modal.hooks.js";
  * single scrolling region between fixed header and footer.
  *
  * Domain state (fetch, parse, filter, view toggle) lives in `hooks/use-sitemap-modal
- * .hooks.ts`; this file is markup only — the one bit of sequencing that belongs to neither that
- * hook nor the outer `useSeo()` (refetching after a successful regenerate) lives in its own
- * colocated `react/hooks/SitemapModal.hooks.ts`, see that file's header for why it isn't folded into either.
+ * .hooks.ts`; this file is markup only. The outer `useSeo()` mutation owns regeneration and
+ * cache invalidation; `react/hooks/SitemapModal.hooks.ts` forwards that action without another read.
  *
  * ## Agent handles
  * Every interactive element carries its own `agentHandle` (grep `agentHandle(` in `react/pages/SeoPage.tsx` for
@@ -34,9 +33,8 @@ export interface SitemapModalProps {
     /** `saving` from `useSeo()` — shared with the outer page's own Save/Regenerate buttons, same as
      *  today's single-flag convention (`react/pages/SeoPage.tsx`'s existing `disabled={saving}` on both). */
     regenerating: boolean;
-    /** `regenerateSitemap` from `useSeo()`. Resolves `true` on success, `false` on a caught failure —
-     *  `handleRegenerate` below only refetches on `true`, matching REQ 7 ("after a SUCCESSFUL
-     *  regenerate it refetches"). */
+    /** `regenerateSitemap` from `useSeo()`. Its mutation invalidates the sitemap cache on
+     * success; a caught failure returns `false` and leaves the viewed XML alone. */
     onRegenerate: () => Promise<boolean>;
     onClose: () => void;
     /** Injectable seam for the modal's fetch/parse/filter/view state. Defaults to the real

package/dist/seo/react/components/SitemapModal.js

@@ -62,7 +62,7 @@ export function SitemapModal({ locale, sitemapEnabled, regenerating, onRegenerat
     const t = useSeoCopy();
     const modal = useModal({ enabled: sitemapEnabled, onClose });
     const { title } = useSitemapPresentation({ locale, modal });
-    const { handleRegenerate } = useSitemapModalRegenerate({ onRegenerate, refetch: modal.refetch });
+    const { handleRegenerate } = useSitemapModalRegenerate({ onRegenerate });
     // Native modality keeps Tab from reaching the background, as MediaPickerDialog now does too.
     return (_jsxs(Dialog, { open: true, title: title, onClose: () => onClose(), className: "settings-dialog tovu-domain-dialog sitemap-modal", children: [_jsx("div", { className: "sitemap-modal-header", children: _jsx(SitemapModalHeaderActions, { locale: locale, modal: modal }) }), _jsx("div", { className: "sitemap-modal-body", children: _jsx(SitemapModalBody, { locale: locale, sitemapEnabled: sitemapEnabled, modal: modal }) }), _jsx("div", { className: "widget-picker-footer", children: _jsxs("span", { className: "editor-actions", children: [_jsx("button", { type: "button", className: "btn-secondary", onClick: handleRegenerate, disabled: regenerating, ...agentHandle({ handle: "seo-sitemap-modal-regenerate" }, {
                                 role: "button",

package/dist/seo/react/hooks/SitemapModal.hooks.d.ts

@@ -1,10 +1,13 @@
 import type { SitemapModalController } from '../../models.js';
-/** Refetch only after accepted regeneration; a failed write must leave the viewed XML alone. */
-export declare function useSitemapModalRegenerate({ onRegenerate, refetch }: {
+/** The regeneration mutation owns cache invalidation; forwarding it must not start a second
+ * XML read. Failed writes leave the viewed XML alone.
+ * @returns The mutation callback as the modal's click handler; no additional I/O.
+ * @complexity O(1).
+ */
+export declare function useSitemapModalRegenerate({ onRegenerate }: {
     onRegenerate: () => Promise<boolean>;
-    refetch: () => void;
 }, _optional?: Record<string, never>): {
-    handleRegenerate: () => Promise<void>;
+    handleRegenerate: () => Promise<boolean>;
 };
 export declare function useSitemapTable({ modal }: {
     modal: SitemapModalController;

package/dist/seo/react/hooks/SitemapModal.hooks.js

@@ -1,9 +1,12 @@
 import { useSeoCopy, useSeoOptions } from './SeoPorts.hooks.js';
 import { buildAgentListHandles } from '@jini-ai/agentic';
-/** Refetch only after accepted regeneration; a failed write must leave the viewed XML alone. */
-export function useSitemapModalRegenerate({ onRegenerate, refetch }, _optional = {}) {
-    return { handleRegenerate: async () => { if (await onRegenerate())
-            refetch(); } };
+/** The regeneration mutation owns cache invalidation; forwarding it must not start a second
+ * XML read. Failed writes leave the viewed XML alone.
+ * @returns The mutation callback as the modal's click handler; no additional I/O.
+ * @complexity O(1).
+ */
+export function useSitemapModalRegenerate({ onRegenerate }, _optional = {}) {
+    return { handleRegenerate: onRegenerate };
 }
 export function useSitemapTable({ modal }, _optional = {}) {
     return { rowHandles: buildAgentListHandles({ prefix: 'seo-sitemap-row', ids: modal.filteredEntries.map(entry => entry.loc) }) };

package/dist/seo/react/hooks/use-share-image-warning.hooks.d.ts

@@ -1,10 +1,10 @@
 import type { AdminSeoPort } from '../../ports.js';
 import type { MediaRefFieldController, SeoTranslator } from '../../models.js';
-/** Probe original file size through the host port; decoded width comes from the existing preview.
+/** Probe original file size through the host port; decoded dimensions come from the existing preview.
  * Unknown size is not zero. URL-keyed state and cancellation prevent a prior choice's response
  * or image event from warning against the current selection. The probe has a 10-second bound.
  * External HEAD responses need CORS, so only same-origin files are probed; decoded previews
- * still report external loading failures and known widths. HTTP copy is translated at render.
+ * still report external loading failures and known dimensions. HTTP copy is translated at render.
  * @param required Controlled reference, source preview URL and injected SEO API.
  * @returns Non-blocking warning and the preview's native load/error handlers.
  * @complexity O(1) state and one HEAD per selection; no image bytes are downloaded by the probe.

package/dist/seo/react/hooks/use-share-image-warning.hooks.js

@@ -1,11 +1,11 @@
 import { useEffect, useRef, useState } from 'react';
-import { interpolate } from '@jini-ai/ui/panel-kit';
+import { interpolate } from '@jini-ai/ui/i18n';
 import { shareImageWarning } from '../../rules.js';
-/** Probe original file size through the host port; decoded width comes from the existing preview.
+/** Probe original file size through the host port; decoded dimensions come from the existing preview.
  * Unknown size is not zero. URL-keyed state and cancellation prevent a prior choice's response
  * or image event from warning against the current selection. The probe has a 10-second bound.
  * External HEAD responses need CORS, so only same-origin files are probed; decoded previews
- * still report external loading failures and known widths. HTTP copy is translated at render.
+ * still report external loading failures and known dimensions. HTTP copy is translated at render.
  * @param required Controlled reference, source preview URL and injected SEO API.
  * @returns Non-blocking warning and the preview's native load/error handlers.
  * @complexity O(1) state and one HEAD per selection; no image bytes are downloaded by the probe.
@@ -62,12 +62,12 @@ export function useShareImageWarning({ value, previewUrl, api }, { t = key => ke
     const checked = inspection?.url === previewUrl ? inspection : null;
     const error = checked?.error;
     return {
-        warning: shareImageWarning({ value, previewUrl, width: source?.width, previewFailed: source?.failed, byteSize: checked?.byteSize,
+        warning: shareImageWarning({ value, previewUrl, width: source?.width, height: source?.height, previewFailed: source?.failed, byteSize: checked?.byteSize,
             error: error ? interpolate({ template: t(error.key, error.vars), vars: error.vars ?? {} }) : undefined }, { t }),
         onPreviewLoad: ({ currentTarget }) => {
             const url = currentTarget.getAttribute('src');
             if (url && url === currentUrl.current)
-                setDecoded({ url, width: currentTarget.naturalWidth });
+                setDecoded({ url, width: currentTarget.naturalWidth, height: currentTarget.naturalHeight });
         },
         onPreviewError: ({ currentTarget }) => {
             const url = currentTarget.getAttribute('src');

package/dist/seo/rules.d.ts

@@ -121,14 +121,15 @@ export declare function resolveMediaRefPreviewUrl({ value, mediaOriginalUrl }: {
     siteOrigin?: string | undefined;
 }): string | null;
 /** Diagnostic policy: unknown measurements stay unknown; measured failures name their reason.
- * @param required Controlled field, decoded preview width and original HTTP metadata.
+ * @param required Controlled field, decoded preview dimensions and original HTTP metadata.
  * @returns A non-blocking explanation, or null for an image without a measured problem.
  * @complexity O(n) in value/error length; fixed-count checks, no I/O.
  */
-export declare function shareImageWarning({ value, previewUrl, width, byteSize, error, previewFailed }: {
+export declare function shareImageWarning({ value, previewUrl, width, height, byteSize, error, previewFailed }: {
     value: string;
     previewUrl: string | null;
     width?: number | undefined;
+    height?: number | undefined;
     byteSize?: number | null | undefined;
     error?: string | null | undefined;
     previewFailed?: boolean | undefined;

package/dist/seo/rules.js

@@ -1,4 +1,4 @@
-import { interpolate } from '@jini-ai/ui/panel-kit';
+import { interpolate } from '@jini-ai/ui/i18n';
 /**
  * @file Pure logic for the `seo` feature — everything that computes a value rather than rendering
  * one.
@@ -180,11 +180,11 @@ function localRenditionAssetKey({ value, siteOrigin }, _optional = {}) {
     }
 }
 /** Diagnostic policy: unknown measurements stay unknown; measured failures name their reason.
- * @param required Controlled field, decoded preview width and original HTTP metadata.
+ * @param required Controlled field, decoded preview dimensions and original HTTP metadata.
  * @returns A non-blocking explanation, or null for an image without a measured problem.
  * @complexity O(n) in value/error length; fixed-count checks, no I/O.
  */
-export function shareImageWarning({ value, previewUrl, width, byteSize, error, previewFailed }, { t = key => key } = {}) {
+export function shareImageWarning({ value, previewUrl, width, height, byteSize, error, previewFailed }, { t = key => key } = {}) {
     if (typeof value !== 'string' || !value.trim())
         return t('No share image is selected. Link previews may show no picture.');
     if (!previewUrl)
@@ -194,8 +194,10 @@ export function shareImageWarning({ value, previewUrl, width, byteSize, error, p
     if (previewFailed)
         return t('The share image could not be loaded. Choose an available image.');
     const reasons = [];
-    if (width !== undefined && width < 600)
-        reasons.push(interpolate({ template: t('The share image is only {width} px wide; use an image at least 600 px wide.', { width }), vars: { width } }));
+    if (width !== undefined && width < 1200)
+        reasons.push(interpolate({ template: t('The share image is only {width} px wide; use an image at least 1200 px wide.', { width }), vars: { width } }));
+    if (height !== undefined && height < 630)
+        reasons.push(interpolate({ template: t('The share image is only {height} px tall; use an image at least 630 px tall.', { height }), vars: { height } }));
     if (byteSize !== undefined && byteSize !== null && byteSize < 5120)
         reasons.push(interpolate({ template: t('The share image is only {byteSize} bytes; use an image of at least 5 KB (5120 bytes).', { byteSize }), vars: { byteSize } }));
     return reasons.length ? reasons.join(' ') : null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jini-ai/admin",
-  "version": "0.4.7",
+  "version": "0.4.9",
   "description": "Composable admin surface for Jini-hosted products: framework-free contracts, panel registry, and route model in `/core`; React shell and panels in `/react`. Import only the layers you need.",
   "license": "Apache-2.0",
   "repository": {
@@ -434,7 +434,7 @@
     "react": "^18.3.0 || ^19.0.0",
     "react-dom": "^18.3.0 || ^19.0.0",
     "@jini-ai/agentic": "^0.4.4",
-    "@jini-ai/ui": "^0.4.8 || ^0.5.0",
+    "@jini-ai/ui": "^0.5.1",
     "@jini-ai/ui-kit": "^0.1.3",
     "@jini-ai/agent-plugins": "^0.4.3",
     "@jini-ai/cms": "^0.5.6",
@@ -476,12 +476,12 @@
     "react": "^19.2.0",
     "react-dom": "^19.2.0",
     "@testing-library/user-event": "^14.6.1",
-    "@jini-ai/ui-kit": "^0.1.3",
-    "@jini-ai/core": "^0.4.4",
+    "@jini-ai/ui": "^0.5.1",
     "@jini-ai/agentic": "^0.4.4",
+    "@jini-ai/ui-kit": "^0.1.4",
+    "@jini-ai/agent-plugins": "^0.4.6",
     "@jini-ai/cms": "^0.5.10",
-    "@jini-ai/ui": "^0.5.0",
-    "@jini-ai/agent-plugins": "^0.4.6"
+    "@jini-ai/core": "^0.4.5"
   },
   "typesVersions": {
     "*": {