@businessdash/sdk 0.9.70 → 0.9.81

package/src/sdk.ts

@@ -33,12 +33,14 @@ import {
 	BiabDevCouponsClient,
 	BiabDevCustomerPortalClient,
 	BiabDevFollowersClient,
+	BiabDevLegalClient,
 	BiabDevMarketingClient,
 	BiabDevMarketingPagesClient,
 	BiabDevNotificationsClient,
 	BiabDevParallelPagesClient,
 	BiabDevReviewsClient,
 	BiabDevShippingClient,
+	type BiabDevSiteClient,
 	BiabDevStorefrontClient,
 	BiabDevSubscriptionsClient,
 } from "./client.js";
@@ -1619,6 +1621,16 @@ export interface BiabClient {
 	customerPortal: BiabDevCustomerPortalClient;
 	notifications: BiabDevNotificationsClient;
 	followers: BiabDevFollowersClient;
+	// The org's published legal pages — what `@businessdash/sdk/legal`'s
+	// resolver expects a client to carry, so the starters' catch-all
+	// route works on THIS facade, not only on the raw BiabDevClient.
+	legal: BiabDevLegalClient;
+	// Site-scoped surfaces (collections, rows, data model, analytics,
+	// scheduling, …). A METHOD mirroring `BiabDevClient.site(siteId)` —
+	// `@businessdash/sdk/sitemap`'s collector calls
+	// `client.site(siteId).sitemapEntries()`, so the starters' sitemap
+	// route needs the same shape on this facade.
+	site(siteId: string): BiabDevSiteClient;
 	marketing: BiabDevMarketingClient;
 	marketingPages: BiabDevMarketingPagesClient;
 	parallelPages: BiabDevParallelPagesClient;
@@ -1664,8 +1676,30 @@ export function createBiabClient(opts: CreateBiabClientOptions): BiabClient {
 		customerPortal: new BiabDevCustomerPortalClient(transport, null),
 		notifications: new BiabDevNotificationsClient(transport, state.siteId),
 		followers: new BiabDevFollowersClient(transport, state.siteId),
+		legal: new BiabDevLegalClient(transport),
+		site: (siteId: string) => transport.site(siteId),
 		marketing: new BiabDevMarketingClient(transport, state.siteId),
 		marketingPages: new BiabDevMarketingPagesClient(transport, state.siteId),
 		parallelPages: new BiabDevParallelPagesClient(transport, state.siteId),
 	};
 }
+
+/* ── Facade ↔ helper-module contracts ──────────────────────────────────
+ * The standalone helpers (`/legal`, `/sitemap`) accept "any client that
+ * carries the right surface", and every file-routed starter hands them
+ * THIS facade. 0.9.80 shipped without `legal` and `site`, and three
+ * starters stopped typechecking against the published package — a drift
+ * only the template fleet could see. These aliases move that failure
+ * into `pnpm typecheck`: drop a required surface and the build breaks
+ * here, before a release. Type-only; erased from the emitted JS. */
+type MustBeTrue<T extends true> = T;
+export type FacadeServesLegalRoutes = MustBeTrue<
+	BiabClient extends import("./legal-core/index.js").LegalClientLike
+		? true
+		: false
+>;
+export type FacadeServesSitemapRoutes = MustBeTrue<
+	BiabClient extends import("./sitemap-core/index.js").SitemapClientLike
+		? true
+		: false
+>;

package/src/sitemap-core/index.ts

@@ -74,8 +74,11 @@ export type SitemapRoutes = {
 export type SitemapEntry = {
 	/** Absolute URL. */
 	loc: string;
-	/** ISO 8601. Omitted when nothing meaningful is known. */
-	lastmod?: string;
+	/** ISO 8601. Omitted when nothing meaningful is known. The explicit
+	 *  `| undefined` matters under exactOptionalPropertyTypes: the real
+	 *  client's `sitemapEntries()` rows carry `lastmod?: string | undefined`,
+	 *  and without it the facade fails the SitemapClientLike contract. */
+	lastmod?: string | undefined;
 };
 
 export type BuildSitemapOptions = {
@@ -106,16 +109,21 @@ export type BuildSitemapOptions = {
 	onSkip?: (section: string, reason: string) => void;
 };
 
-/** The subset of the SDK client this needs. */
+/** The subset of the SDK client this needs. Shapes mirror the REAL
+ *  client responses (`{ items, … }` — blogListPostsResponseSchema and
+ *  storefrontListProductsResponseSchema): this type once said `posts`/
+ *  `products`, so the destructure came back undefined, the loop threw,
+ *  and every real sitemap silently reported its blog and product
+ *  sections as "unreachable". */
 export type SitemapClientLike = {
 	blog: {
 		listPosts(input?: { limit?: number }): Promise<{
-			posts: Array<{ slug: string; updatedAt?: string | null }>;
+			items: Array<{ slug: string; updatedAt?: string | null }>;
 		}>;
 	};
 	storefront: {
 		listProducts(input?: { limit?: number }): Promise<{
-			products: Array<{ id: string; updatedAt?: string | null }>;
+			items: Array<{ id: string; updatedAt?: string | null }>;
 		}>;
 	};
 	site(siteId: string): {
@@ -209,7 +217,7 @@ export async function collectSitemapEntries(
 	// 3. Blog — only when you told us where it lives.
 	if (routes.blog) {
 		try {
-			const { posts } = await client.blog.listPosts({
+			const { items: posts } = await client.blog.listPosts({
 				limit: options.blogLimit ?? 1000,
 			});
 			for (const post of posts) {
@@ -230,7 +238,7 @@ export async function collectSitemapEntries(
 	// 4. Products — same rule.
 	if (routes.product) {
 		try {
-			const { products } = await client.storefront.listProducts({
+			const { items: products } = await client.storefront.listProducts({
 				limit: options.productLimit ?? 1000,
 			});
 			for (const product of products) {