@mohasinac/appkit 4.44.0 → 4.46.0

package/dist/ui/index.js

@@ -1,140 +1,160 @@
-import "./components/index.style.css";
-import "./forms/FormShell.style.css";
-export { FormShellProvider, useFormShell } from "./forms";
-export { FieldInput } from "./forms";
-export { FieldSelect } from "./forms";
-export { FieldCheckbox } from "./forms";
-export { FieldTextarea } from "./forms";
-export { FormErrorSummary } from "./forms";
+/*
+ * 🛑 NO CSS IMPORTS HERE. Both moved into src/styles.css's @import chain, which
+ * is what `@mohasinac/appkit/styles` resolves to and what bundle-css.mjs
+ * flattens.
+ *
+ * They used to be `import "./components/index.style.css"` and
+ * `import "./forms/FormShell.style.css"`, and that was wrong twice over:
+ *
+ *   - components/index.style.css was ALREADY in the @import chain, so the JS
+ *     import was pure duplication.
+ *   - forms/FormShell.style.css was NOT, so it shipped only to consumers who
+ *     happened to pull this barrel — a real gap, measured as zero occurrences
+ *     of `.appkit-formshell` in the built styles.css.
+ *
+ * And a JS->CSS edge on a BARREL poisons every graph that touches it. Node
+ * cannot load CSS at any path, so this one import pair is what made
+ * `serverExternalPackages: ["@mohasinac/appkit"]` fail: appkit's server entry
+ * reaches this barrel in a few hops and died on ERR_UNKNOWN_FILE_EXTENSION.
+ *
+ * Styles now arrive exactly one way — the stylesheet — which is also the only
+ * way a consumer can control ordering.
+ */
+export { FormShellProvider, useFormShell } from "./forms/index.js";
+export { FieldInput } from "./forms/index.js";
+export { FieldSelect } from "./forms/index.js";
+export { FieldCheckbox } from "./forms/index.js";
+export { FieldTextarea } from "./forms/index.js";
+export { FormErrorSummary } from "./forms/index.js";
 // --- Column utilities ---------------------------------------------------------
-export { buildColumns, createColumnBuilder, renderBoolean, renderCurrency, renderCurrencyCompact, renderCount, renderNullable, renderRating, getStatusTone, getResourceIcon, } from "./columns";
-export { Section, sectionBackgroundStyle, Article, Main, Aside, Nav, BlockHeader, BlockFooter, Ul, Ol, Li, Table, Thead, Tbody, Tr, Th, Td, Code, Pre, Blockquote, Figure, Figcaption, Dl, Dt, Dd, } from "./components/Semantic";
-export { Div } from "./components/Div";
-export { Heading, Text, Label, Caption, Span } from "./components/Typography";
+export { buildColumns, createColumnBuilder, renderBoolean, renderCurrency, renderCurrencyCompact, renderCount, renderNullable, renderRating, getStatusTone, getResourceIcon, } from "./columns/index.js";
+export { Section, sectionBackgroundStyle, Article, Main, Aside, Nav, BlockHeader, BlockFooter, Ul, Ol, Li, Table, Thead, Tbody, Tr, Th, Td, Code, Pre, Blockquote, Figure, Figcaption, Dl, Dt, Dd, } from "./components/Semantic.js";
+export { Div } from "./components/Div.js";
+export { Heading, Text, Label, Caption, Span } from "./components/Typography.js";
 // --- Filter chip group (SB10-C completion, S8 2026-05-13) --------------------
-export { FilterChipGroup } from "./components/FilterChipGroup";
-export { Spinner } from "./components/Spinner";
-export { PageLoader } from "./components/PageLoader";
-export { SiteLogo } from "./components/SiteLogo";
-export { SiteMark } from "./components/SiteMark";
-export { Skeleton } from "./components/Skeleton";
-export { Button } from "./components/Button";
+export { FilterChipGroup } from "./components/FilterChipGroup.js";
+export { Spinner } from "./components/Spinner.js";
+export { PageLoader } from "./components/PageLoader.js";
+export { SiteLogo } from "./components/SiteLogo.js";
+export { SiteMark } from "./components/SiteMark.js";
+export { Skeleton } from "./components/Skeleton.js";
+export { Button } from "./components/Button.js";
 /*
  * The one primitive allowed to own a raw <input type="file">. Rule #9 sends
  * every raw form element to a primitive and there was none for a file picker,
  * so the only feature that needed one had to carry a suppression marker that
  * audit-no-suppression-comments (strict-zero) correctly refuses.
  */
-export { FilePickerButton } from "./components/FilePickerButton";
-export { IconButton } from "./components/IconButton";
-export { Icon } from "./components/Icon";
-export { ICONS, ICON_SIZE, isIconKey, resolveIcon, } from "./icons/icon-registry";
-export { TextLink } from "./components/TextLink";
-export { Tooltip } from "./components/Tooltip";
-export { ActionRow } from "./components/ActionRow";
-export { Accordion, AccordionItem } from "./components/Accordion";
-export { Form, FormGroup, FormFieldSpan, FormActions } from "./components/Form";
-export { Badge } from "./components/Badge";
-export { Alert } from "./components/Alert";
-export { Divider } from "./components/Divider";
-export { Progress, IndeterminateProgress } from "./components/Progress";
-export { Pagination } from "./components/Pagination";
-export { StatusBadge } from "./components/StatusBadge";
-export { Modal, ModalFooter } from "./components/Modal";
-export { QuickCreateModal } from "./components/QuickCreateModal";
-export { VacationBanner } from "./components/VacationBanner";
-export { UnsavedChangesModal } from "./components/UnsavedChangesModal";
-export { LoginRequiredModal } from "./components/LoginRequiredModal";
-export { Drawer } from "./components/Drawer";
-export { BackgroundRenderer } from "./components/BackgroundRenderer";
-export { ResponsiveView } from "./components/ResponsiveView";
-export { Select } from "./components/Select";
-export { Input } from "./components/Input";
-export { OtpInput } from "./components/OtpInput";
-export { DateInput, DateRangeInput } from "./components/DateInput";
-export { Textarea } from "./components/Textarea";
-export { Slider } from "./components/Slider";
-export { Checkbox } from "./components/Checkbox";
-export { RadioGroup, RadioItem } from "./components/Radio";
-export { Toggle } from "./components/Toggle";
-export { QuantityStepper } from "./components/QuantityStepper";
-export { ToastProvider, useToast } from "./components/Toast";
-export { StarRating } from "./components/StarRating";
-export { Breadcrumb } from "./components/Breadcrumb";
-export { ImageLightbox } from "./components/ImageLightbox";
-export { TagInput } from "./components/TagInput";
-export { StepperNav } from "./components/StepperNav";
-export { ViewToggle } from "./components/ViewToggle";
-export { RatingDisplay } from "./components/RatingDisplay";
-export { PriceDisplay } from "./components/PriceDisplay";
-export { GatedPrice, PricesOnly, GATED_PRICE_CLASS } from "./components/GatedPrice";
-export { StatsGrid } from "./components/StatsGrid";
-export { SummaryCard } from "./components/SummaryCard";
-export { CountdownDisplay } from "./components/CountdownDisplay";
-export { ItemRow } from "./components/ItemRow";
-export { HorizontalScroller } from "./components/HorizontalScroller";
-export { FormGrid, FormField as FormGridField } from "./components/FormGrid";
-export { SettingsSection, ToggleRow } from "./components/SettingsSection";
-export { DetailPageHero } from "./components/DetailPageHero";
-export { DetailPageGallery } from "./components/DetailPageGallery";
-export { DetailPageTabs } from "./components/DetailPageTabs";
-export { FormField } from "./components/FormField";
-export { DescriptionField } from "./components/DescriptionField";
-export { Card, CardHeader, CardBody, CardFooter } from "./components/Card";
-export { EmptyState } from "./components/EmptyState";
-export { Avatar, AvatarGroup } from "./components/Avatar";
-export { Dropdown, DropdownTrigger, DropdownMenu, DropdownItem, DropdownSeparator, } from "./components/Dropdown";
-export { Menu, MenuTrigger, MenuContent, MenuItem, MenuSeparator, } from "./components/Menu";
-export { PaginatedSelect } from "./components/PaginatedSelect";
-export { RoleBadge } from "./components/RoleBadge";
-export { SkipToMain } from "./components/SkipToMain";
-export { NavigationLoader } from "./components/NavigationLoader";
-export { Tabs, TabsList, TabsTrigger, TabsContent } from "./components/Tabs";
-export { TabBarShell, TabBarButton, TabsNavSelect } from "./components/TabBarShell";
-export { FilterDrawer } from "./components/FilterDrawer";
-export { RowActionMenu } from "./components/RowActionMenu";
-export { PasswordStrengthIndicator } from "./components/PasswordStrengthIndicator";
-export { AvatarDisplay } from "./components/AvatarDisplay";
-export { ActiveFilterChips } from "./components/ActiveFilterChips";
-export { SortDropdown } from "./components/SortDropdown";
-export { ListingFilterDrawer } from "./components/ListingFilterDrawer";
-export { ListingToolbar } from "./components/ListingToolbar";
-export { TablePagination } from "./components/TablePagination";
-export { DataTable } from "./DataTable";
-export { DEFAULT_TABLE_CONFIG, DEFAULT_PAGINATION_CONFIG, DEFAULT_STICKY_CONFIG, mergeTableConfig, } from "../contracts";
-export { SURFACE_MAP, PADDING_MAP, ROUNDED_MAP, BORDER_MAP, SHADOW_MAP, buildSurfaceClasses, } from "./components/surface-tokens";
-export { Container, Stack, Row, Grid, GRID_MAP } from "./components/Layout";
-export { HandModeRow } from "./components/HandModeRow";
+export { FilePickerButton } from "./components/FilePickerButton.js";
+export { IconButton } from "./components/IconButton.js";
+export { Icon } from "./components/Icon.js";
+export { ICONS, ICON_SIZE, isIconKey, resolveIcon, } from "./icons/icon-registry.js";
+export { TextLink } from "./components/TextLink.js";
+export { Tooltip } from "./components/Tooltip.js";
+export { ActionRow } from "./components/ActionRow.js";
+export { Accordion, AccordionItem } from "./components/Accordion.js";
+export { Form, FormGroup, FormFieldSpan, FormActions } from "./components/Form.js";
+export { Badge } from "./components/Badge.js";
+export { Alert } from "./components/Alert.js";
+export { Divider } from "./components/Divider.js";
+export { Progress, IndeterminateProgress } from "./components/Progress.js";
+export { Pagination } from "./components/Pagination.js";
+export { StatusBadge } from "./components/StatusBadge.js";
+export { Modal, ModalFooter } from "./components/Modal.js";
+export { QuickCreateModal } from "./components/QuickCreateModal.js";
+export { VacationBanner } from "./components/VacationBanner.js";
+export { UnsavedChangesModal } from "./components/UnsavedChangesModal.js";
+export { LoginRequiredModal } from "./components/LoginRequiredModal.js";
+export { Drawer } from "./components/Drawer.js";
+export { BackgroundRenderer } from "./components/BackgroundRenderer.js";
+export { ResponsiveView } from "./components/ResponsiveView.js";
+export { Select } from "./components/Select.js";
+export { Input } from "./components/Input.js";
+export { OtpInput } from "./components/OtpInput.js";
+export { DateInput, DateRangeInput } from "./components/DateInput.js";
+export { Textarea } from "./components/Textarea.js";
+export { Slider } from "./components/Slider.js";
+export { Checkbox } from "./components/Checkbox.js";
+export { RadioGroup, RadioItem } from "./components/Radio.js";
+export { Toggle } from "./components/Toggle.js";
+export { QuantityStepper } from "./components/QuantityStepper.js";
+export { ToastProvider, useToast } from "./components/Toast.js";
+export { StarRating } from "./components/StarRating.js";
+export { Breadcrumb } from "./components/Breadcrumb.js";
+export { ImageLightbox } from "./components/ImageLightbox.js";
+export { TagInput } from "./components/TagInput.js";
+export { StepperNav } from "./components/StepperNav.js";
+export { ViewToggle } from "./components/ViewToggle.js";
+export { RatingDisplay } from "./components/RatingDisplay.js";
+export { PriceDisplay } from "./components/PriceDisplay.js";
+export { GatedPrice, PricesOnly, GATED_PRICE_CLASS } from "./components/GatedPrice.js";
+export { StatsGrid } from "./components/StatsGrid.js";
+export { SummaryCard } from "./components/SummaryCard.js";
+export { CountdownDisplay } from "./components/CountdownDisplay.js";
+export { ItemRow } from "./components/ItemRow.js";
+export { HorizontalScroller } from "./components/HorizontalScroller.js";
+export { FormGrid, FormField as FormGridField } from "./components/FormGrid.js";
+export { SettingsSection, ToggleRow } from "./components/SettingsSection.js";
+export { DetailPageHero } from "./components/DetailPageHero.js";
+export { DetailPageGallery } from "./components/DetailPageGallery.js";
+export { DetailPageTabs } from "./components/DetailPageTabs.js";
+export { FormField } from "./components/FormField.js";
+export { DescriptionField } from "./components/DescriptionField.js";
+export { Card, CardHeader, CardBody, CardFooter } from "./components/Card.js";
+export { EmptyState } from "./components/EmptyState.js";
+export { Avatar, AvatarGroup } from "./components/Avatar.js";
+export { Dropdown, DropdownTrigger, DropdownMenu, DropdownItem, DropdownSeparator, } from "./components/Dropdown.js";
+export { Menu, MenuTrigger, MenuContent, MenuItem, MenuSeparator, } from "./components/Menu.js";
+export { PaginatedSelect } from "./components/PaginatedSelect.js";
+export { RoleBadge } from "./components/RoleBadge.js";
+export { SkipToMain } from "./components/SkipToMain.js";
+export { NavigationLoader } from "./components/NavigationLoader.js";
+export { Tabs, TabsList, TabsTrigger, TabsContent } from "./components/Tabs.js";
+export { TabBarShell, TabBarButton, TabsNavSelect } from "./components/TabBarShell.js";
+export { FilterDrawer } from "./components/FilterDrawer.js";
+export { RowActionMenu } from "./components/RowActionMenu.js";
+export { PasswordStrengthIndicator } from "./components/PasswordStrengthIndicator.js";
+export { AvatarDisplay } from "./components/AvatarDisplay.js";
+export { ActiveFilterChips } from "./components/ActiveFilterChips.js";
+export { SortDropdown } from "./components/SortDropdown.js";
+export { ListingFilterDrawer } from "./components/ListingFilterDrawer.js";
+export { ListingToolbar } from "./components/ListingToolbar.js";
+export { TablePagination } from "./components/TablePagination.js";
+export { DataTable } from "./DataTable.js";
+export { DEFAULT_TABLE_CONFIG, DEFAULT_PAGINATION_CONFIG, DEFAULT_STICKY_CONFIG, mergeTableConfig, } from "../contracts/index.js";
+export { SURFACE_MAP, PADDING_MAP, ROUNDED_MAP, BORDER_MAP, SHADOW_MAP, buildSurfaceClasses, } from "./components/surface-tokens.js";
+export { Container, Stack, Row, Grid, GRID_MAP } from "./components/Layout.js";
+export { HandModeRow } from "./components/HandModeRow.js";
 // --- UI Helpers ---------------------------------------------------------------
-export { classNames, mergeTailwindClasses } from "./style.helper";
-export { SideModal } from "./components/SideModal";
-export { RichText } from "./rich-text/RichText";
-export { RichTextRenderer } from "./rich-text/RichTextRenderer";
-export { RichTextEditor } from "./components/RichTextEditor";
-export { ListingLayout } from "./components/ListingLayout";
-export { SlottedListingView } from "./components/SlottedListingView";
-export { DetailViewShell } from "./components/DetailViewShell";
-export { StackedViewShell } from "./components/StackedViewShell";
-export { BulkActionBar } from "./components/BulkActionBar";
-export { ClaimCouponButton } from "./components/ClaimCouponButton";
-export { FlowDiagram } from "./components/FlowDiagram";
-export { BaseListingCard } from "./components/BaseListingCard";
-export { default as ImageGallery } from "./components/ImageGallery";
-export { SideDrawer } from "./components/SideDrawer";
-export { FormActionBar } from "./components/FormActionBar";
-export { ConfirmDeleteModal } from "./components/ConfirmDeleteModal";
-export { FadeIn, SlideUp, SlideIn, ScaleIn, Collapse, PressScale, HoverLift, AnimatedList, AnimatedDiv, AnimatedStack, AnimatedRow, Draggable, Swipeable, AnimatePresence, } from "./components/Motion";
-export { MOTION_PRESETS, SPRING_SNAPPY, SPRING_GENTLE, EASE_OUT, } from "../tokens/motion";
+export { classNames, mergeTailwindClasses } from "./style.helper.js";
+export { SideModal } from "./components/SideModal.js";
+export { RichText } from "./rich-text/RichText.js";
+export { RichTextRenderer } from "./rich-text/RichTextRenderer.js";
+export { RichTextEditor } from "./components/RichTextEditor.js";
+export { ListingLayout } from "./components/ListingLayout.js";
+export { SlottedListingView } from "./components/SlottedListingView.js";
+export { DetailViewShell } from "./components/DetailViewShell.js";
+export { StackedViewShell } from "./components/StackedViewShell.js";
+export { BulkActionBar } from "./components/BulkActionBar.js";
+export { ClaimCouponButton } from "./components/ClaimCouponButton.js";
+export { FlowDiagram } from "./components/FlowDiagram.js";
+export { BaseListingCard } from "./components/BaseListingCard.js";
+export { default as ImageGallery } from "./components/ImageGallery.js";
+export { SideDrawer } from "./components/SideDrawer.js";
+export { FormActionBar } from "./components/FormActionBar.js";
+export { ConfirmDeleteModal } from "./components/ConfirmDeleteModal.js";
+export { FadeIn, SlideUp, SlideIn, ScaleIn, Collapse, PressScale, HoverLift, AnimatedList, AnimatedDiv, AnimatedStack, AnimatedRow, Draggable, Swipeable, AnimatePresence, } from "./components/Motion.js";
+export { MOTION_PRESETS, SPRING_SNAPPY, SPRING_GENTLE, EASE_OUT, } from "../tokens/motion.js";
 // --- W-E primitives (Details/Summary/HorizontalRule/Anchor) ------------------
-export { Details, Summary } from "./components/Details";
-export { HorizontalRule } from "./components/HorizontalRule";
-export { Anchor } from "./components/Anchor";
-export { Scrim } from "./components/Scrim";
-export { Kbd } from "./components/Kbd";
-export { StickyToolbar } from "./components/StickyToolbar";
-export { Show, Hide } from "./components/Responsive";
-export { CollapsibleSection } from "./components/CollapsibleSection";
+export { Details, Summary } from "./components/Details.js";
+export { HorizontalRule } from "./components/HorizontalRule.js";
+export { Anchor } from "./components/Anchor.js";
+export { Scrim } from "./components/Scrim.js";
+export { Kbd } from "./components/Kbd.js";
+export { StickyToolbar } from "./components/StickyToolbar.js";
+export { Show, Hide } from "./components/Responsive.js";
+export { CollapsibleSection } from "./components/CollapsibleSection.js";
 // RecordDetailModal — shared "open one listing row and read it" surface.
 // Backs the 2026-08-21 dead-end-listing sweep; see the component's own
 // header comment and scripts/audit-listing-detail-affordance.mjs.
-export { RecordDetailModal } from "./components/RecordDetailModal";
+export { RecordDetailModal } from "./components/RecordDetailModal.js";

package/dist/ui/rich-text/RichText.js

@@ -1,8 +1,8 @@
 "use client";
 import { jsx as _jsx } from "react/jsx-runtime";
 import { useMemo } from "react";
-import { Div } from "../components/Div";
-import { normalizeRichTextHtml } from "../../utils/string.formatter";
+import { Div } from "../components/Div.js";
+import { normalizeRichTextHtml } from "../../utils/string.formatter.js";
 // --- Allowed HTML tags (safe subset — no script/object; iframe allowed for YouTube only) -------
 const ALLOWED_TAGS = new Set([
     "p",

package/dist/ui/rich-text/index.js

@@ -1 +1 @@
-export { RichText } from "./RichText";
+export { RichText } from "./RichText.js";

package/dist/utils/action-response.js

@@ -1,5 +1,5 @@
-import { serverLogger } from "../monitoring";
-import { AuthorizationError, NotFoundError, ValidationError, } from "../errors";
+import { serverLogger } from "../monitoring/index.js";
+import { AuthorizationError, NotFoundError, ValidationError, } from "../errors/index.js";
 export function handleActionError(err) {
     const devDebug = process.env.NODE_ENV === "development"
         ? { stack: err instanceof Error ? err.stack : String(err) }

package/dist/utils/action-result.js

@@ -7,4 +7,4 @@
  * actions inside appkit; this file is the canonical public surface so
  * consumers (and the `appkit/src/client.ts` barrel) import from here.
  */
-export { isOk, unwrap } from "../_internal/shared/types/action-result";
+export { isOk, unwrap } from "../_internal/shared/types/action-result.js";

package/dist/utils/auth-error.js

@@ -2,4 +2,4 @@
 // `../errors/normalize.ts`. This file is a thin re-export shim so existing
 // consumers (17 imports across the monorepo) continue to compile while
 // migrating to `import { isAuthError } from "@mohasinac/appkit"`.
-export { isAuthError } from "../errors/normalize";
+export { isAuthError } from "../errors/normalize.js";

package/dist/utils/client.js

@@ -1 +1 @@
-export { GlobalEventManager, globalEventManager, throttle, debounce, addGlobalScrollHandler, addGlobalResizeHandler, addGlobalClickHandler, addGlobalKeyHandler, removeGlobalHandler, isMobileDevice, hasTouchSupport, getViewportDimensions, isInViewport, smoothScrollTo, preventBodyScroll, } from "./event-manager";
+export { GlobalEventManager, globalEventManager, throttle, debounce, addGlobalScrollHandler, addGlobalResizeHandler, addGlobalClickHandler, addGlobalKeyHandler, removeGlobalHandler, isMobileDevice, hasTouchSupport, getViewportDimensions, isInViewport, smoothScrollTo, preventBodyScroll, } from "./event-manager.js";

package/dist/utils/date.formatter.js

@@ -1,7 +1,7 @@
 /**
  * Date Formatting Utilities
  */
-import { getDefaultLocale } from "../core/baseline-resolver";
+import { getDefaultLocale } from "../core/baseline-resolver.js";
 /**
  * Resolve any date-like value (Date, ISO string, number, or Firestore Timestamp JSON)
  * into a native Date object. Returns null for falsy or unparseable input.

package/dist/utils/id-generators.d.ts

@@ -118,6 +118,36 @@ export interface GenerateOfferIdInput {
 }
 export declare function generateOfferId(input: GenerateOfferIdInput): string;
 export declare function idExists(getExistingId: () => Promise<unknown>): Promise<boolean>;
+/**
+ * Find the first id a `checkExists` probe says is free, trying
+ * `generateId(0)`, `generateId(1)`, … up to `maxAttempts`.
+ *
+ * 🛑 **The loop starts at 0, and it used to start at 1.** That one-character
+ * difference made the first candidate unreachable, because its only caller
+ * passes:
+ *
+ *     (count) => (count === 0 ? baseSlug : `${baseSlug}-${count}`)
+ *
+ * so `count === 0` — the BARE slug, which is obviously what the author
+ * intended as the first choice — was dead code. Every product created through
+ * the UI got a gratuitous suffix: the first-ever Dragoon G listing became
+ * `product-dragoon-g-1`, not `product-dragoon-g`.
+ *
+ * Verified by simulating the real helper against the real closure: three
+ * creations yielded `-1`, `-2`, `-3` and never the bare slug.
+ *
+ * It stayed invisible because all 70 production products come from the seed,
+ * which hand-writes clean ids and never calls this — so the catalogue looks
+ * right and only a UI-created listing shows the defect. Same shape as Root
+ * Cause #101: a path exercised by no fixture.
+ *
+ * 🛑 A product id is a URL and a `categorySlugs` reference, and the slug is
+ * immutable after creation by convention, so this is not cosmetic — a listing
+ * created today keeps the wrong URL forever.
+ *
+ * `maxAttempts` still bounds the ATTEMPTS, so the loop runs `0..maxAttempts`
+ * and the random-suffix fallback is unchanged.
+ */
 export declare function generateUniqueId(generateId: (count: number) => string, checkExists: (id: string) => Promise<boolean>, maxAttempts?: number): Promise<string>;
 export declare function generateBarcodeFromId(id: string): string;
 export declare function generateQRCodeData(id: string, baseUrl: string): string;

package/dist/utils/id-generators.js

@@ -5,8 +5,8 @@
  * non-empty it is returned as-is, letting callers bring their own IDs without
  * duplicating generator logic.
  */
-import { slugify } from "./string.formatter";
-import { normalizeError } from "../errors/normalize";
+import { slugify } from "./string.formatter.js";
+import { normalizeError } from "../errors/normalize.js";
 function generateRandomString(length = 6) {
     const chars = "abcdefghijklmnopqrstuvwxyz0123456789";
     const indices = new Uint8Array(length);
@@ -171,8 +171,38 @@ export async function idExists(getExistingId) {
         return false;
     }
 }
+/**
+ * Find the first id a `checkExists` probe says is free, trying
+ * `generateId(0)`, `generateId(1)`, … up to `maxAttempts`.
+ *
+ * 🛑 **The loop starts at 0, and it used to start at 1.** That one-character
+ * difference made the first candidate unreachable, because its only caller
+ * passes:
+ *
+ *     (count) => (count === 0 ? baseSlug : `${baseSlug}-${count}`)
+ *
+ * so `count === 0` — the BARE slug, which is obviously what the author
+ * intended as the first choice — was dead code. Every product created through
+ * the UI got a gratuitous suffix: the first-ever Dragoon G listing became
+ * `product-dragoon-g-1`, not `product-dragoon-g`.
+ *
+ * Verified by simulating the real helper against the real closure: three
+ * creations yielded `-1`, `-2`, `-3` and never the bare slug.
+ *
+ * It stayed invisible because all 70 production products come from the seed,
+ * which hand-writes clean ids and never calls this — so the catalogue looks
+ * right and only a UI-created listing shows the defect. Same shape as Root
+ * Cause #101: a path exercised by no fixture.
+ *
+ * 🛑 A product id is a URL and a `categorySlugs` reference, and the slug is
+ * immutable after creation by convention, so this is not cosmetic — a listing
+ * created today keeps the wrong URL forever.
+ *
+ * `maxAttempts` still bounds the ATTEMPTS, so the loop runs `0..maxAttempts`
+ * and the random-suffix fallback is unchanged.
+ */
 export async function generateUniqueId(generateId, checkExists, maxAttempts = 100) {
-    for (let count = 1; count <= maxAttempts; count++) {
+    for (let count = 0; count <= maxAttempts; count++) {
         const id = generateId(count);
         if (!(await checkExists(id)))
             return id;

package/dist/utils/index.js

@@ -1,20 +1,20 @@
-export * from "./auth-error";
-export * from "./date.formatter";
-export * from "./number.formatter";
-export * from "./string.formatter";
-export * from "./type.converter";
-export * from "./cookie.converter";
-export * from "./id-generators";
-export * from "./media-field";
-export * from "./array.helper";
-export * from "./object.helper";
-export * from "./sorting.helper";
-export * from "./animation.helper";
-export * from "./color.helper";
-export * from "./business-day";
-export * from "./schema-ui";
-export * from "./listing-params";
-export * from "./action-response";
-export * from "./search-txt";
+export * from "./auth-error.js";
+export * from "./date.formatter.js";
+export * from "./number.formatter.js";
+export * from "./string.formatter.js";
+export * from "./type.converter.js";
+export * from "./cookie.converter.js";
+export * from "./id-generators.js";
+export * from "./media-field.js";
+export * from "./array.helper.js";
+export * from "./object.helper.js";
+export * from "./sorting.helper.js";
+export * from "./animation.helper.js";
+export * from "./color.helper.js";
+export * from "./business-day.js";
+export * from "./schema-ui.js";
+export * from "./listing-params.js";
+export * from "./action-response.js";
+export * from "./search-txt.js";
 // Re-export order utilities for checkout flows
-export { splitCartIntoOrderGroups, } from "../features/orders/utils/order-splitter";
+export { splitCartIntoOrderGroups, } from "../features/orders/utils/order-splitter.js";

package/dist/utils/media-field.js

@@ -1 +1 @@
-export { coerceMediaField, coerceMediaFieldArray, getMediaUrl, inferMediaTypeFromMime, mediaFieldSchema, } from "../features/media/types/index";
+export { coerceMediaField, coerceMediaFieldArray, getMediaUrl, inferMediaTypeFromMime, mediaFieldSchema, } from "../features/media/types/index.js";

package/dist/utils/media-url.js

@@ -1,5 +1,5 @@
-import { MEDIA_ENDPOINTS } from "../constants/api-endpoints";
-import { normalizeError } from "../errors/normalize";
+import { MEDIA_ENDPOINTS } from "../constants/api-endpoints.js";
+import { normalizeError } from "../errors/normalize.js";
 /**
  * Single source of truth for the Firebase Storage host literal. Every other
  * appkit + consumer module that needs to detect or allowlist this host

package/dist/utils/number.formatter.js

@@ -1,7 +1,7 @@
 /**
  * Number Formatting Utilities
  */
-import { getDefaultCurrency, getDefaultLocale, } from "../core/baseline-resolver";
+import { getDefaultCurrency, getDefaultLocale, } from "../core/baseline-resolver.js";
 export function formatCurrency(amount, currency, locale) {
     const resolvedCurrency = currency ?? getDefaultCurrency();
     const resolvedLocale = locale ?? getDefaultLocale();

package/dist/utils/physical-location.d.ts

@@ -21,3 +21,71 @@ export interface PhysicalLocationParts {
  * from "zone A, shelf 3".
  */
 export declare function formatPhysicalLocation(loc: PhysicalLocationParts | null | undefined): string | undefined;
+/**
+ * Normalise an inbound location, or `undefined` when it carries nothing — C4.
+ *
+ * 🛑 THE ONE VALIDATOR. There were two and NEITHER was the one in use.
+ *
+ * `physicalLocationSchema` exists in `features/orders/schemas/index.ts` and is
+ * used by neither bulk-location route; both routes instead hand-checked
+ * `typeof loc.zone === "string"`, which **accepts the empty string**. So
+ * `PhysicalLocationModal`'s promise that "all fields are optional" was honoured
+ * only by accident — `typeof "" === "string"` is what made it true — while
+ * OMITTING a key, which is the sensible reading of "optional", returned 400.
+ *
+ * This replaces the hand-check at both routes: a missing key and an empty
+ * string now mean the same thing, which is what the UI says.
+ *
+ * Returns `undefined` for an all-blank location so a caller writes NOTHING
+ * rather than `{zone:"",shelf:"",bin:""}` — a stored all-blank object is what
+ * made `formatPhysicalLocation` need its own "never print `//`" guard, and the
+ * cleanest fix is to stop creating it.
+ */
+export declare function normalisePhysicalLocation(loc: PhysicalLocationParts | null | undefined): {
+    zone: string;
+    shelf: string;
+    bin: string;
+} | undefined;
+/**
+ * Search tokens for a location — what makes it findable in BOTH directions,
+ * which is C4's whole requirement.
+ *
+ * "Where is item X" is answered by the Location column once the field renders.
+ * "What is in bin A-3" is the hard direction, and it falls out of the same
+ * index.
+ *
+ * 🛑 Tokens, not a Firestore range query, and that is deliberate. A
+ * `physicalLocation.bin == "3"` clause paired with any sort needs its own
+ * composite index per sort field, and the `FAILED_PRECONDITION` would be
+ * swallowed into an empty page (Root Cause #59). `searchTxt` is already an
+ * `array-contains`-indexed field, so this costs **more tokens in an array that
+ * is already indexed** and no new index at all.
+ *
+ * ## 🛑 How the match actually happens — measured, because my first version
+ * ## of this comment was wrong about it
+ *
+ * The first draft emitted `a-3-2` and `a/3/2` and claimed the composite was
+ * what let a seller search a bin label. **It is not, and the composites were
+ * dead weight.** `buildSearchTxt` normalises with
+ * `.replace(/[^a-z0-9]+/g, " ")` and then splits — so `a-3-2` is stored as
+ * three tokens `a`, `3`, `2`, exactly the same as the parts. The verification
+ * caught it by asserting the composite was indexed and finding it absent.
+ *
+ * What makes it work is that `parseSearchTxtQuery` applies the **same**
+ * normaliser: a seller typing `A-3` yields terms `["a", "3"]`, which
+ * AND-refine against the parts. So the parts alone are sufficient, and the
+ * separators are irrelevant on both sides — a label written `A/3` and searched
+ * `A-3` matches either way.
+ *
+ * The UNSEPARATED form (`a32`) is kept, and it is the one composite that
+ * survives tokenisation: bin labels are often written without separators, and
+ * a seller typing `A32` must not come back empty when `A-3-2` would have
+ * matched.
+ *
+ * Honest limitation, stated rather than papered over: single-character parts
+ * make low-signal tokens — `a` is an edge n-gram of every word beginning with
+ * "a" — so a one-letter zone searched alone is a broad pushdown that the
+ * in-memory refine then narrows. It is correct, just not selective. A seller
+ * who wants precision has the nested Sieve filters (`physicalLocation.zone`).
+ */
+export declare function physicalLocationTokens(loc: PhysicalLocationParts | null | undefined): string[];