@jini-ai/admin 0.4.8 → 0.4.9

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # 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.

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jini-ai/admin",
-  "version": "0.4.8",
+  "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": {
@@ -476,11 +476,11 @@
     "react": "^19.2.0",
     "react-dom": "^19.2.0",
     "@testing-library/user-event": "^14.6.1",
+    "@jini-ai/ui": "^0.5.1",
     "@jini-ai/agentic": "^0.4.4",
-    "@jini-ai/agent-plugins": "^0.4.6",
     "@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.1",
     "@jini-ai/core": "^0.4.5"
   },
   "typesVersions": {