create-ailk 0.11.5 → 0.12.0

package/templates/apps/web/app/[locale]/layout.tsx

@@ -63,6 +63,10 @@ import { siteLogo } from "../../lib/site-logo";
 // Registers the `_site.mdx` identity with page-renderer at module load, so
 // every page rendered under this layout carries the sitewide JSON-LD (#6597).
 import "../../lib/site-identity";
+// Registers the listicle directory with page-renderer at module load, so a
+// live listicle's landing page gets its shelves and each entry its page
+// (#7521). Inert without AILK_LISTING_PUBLIC_KEY or the paid package.
+import "../../lib/listings";
 import { loadSiteBrandTokens, loadSiteShapeCss } from "../../lib/site-theme";
 import { resolveBaseUrl } from "../../src/lib/routes";
 

package/templates/apps/web/app/[locale]/partners/[slug]/page.tsx

@@ -1,12 +1,8 @@
 // Partner detail page. Reads the Partner MDX for the given slug, renders via
-// next-mdx-remote/rsc, and emits a JSON-LD <script>. Unknown slugs trigger 404.
-import {
-  getMdxSource as defaultGetMdxSource,
-  renderPage,
-  renderPageMetadata,
-} from "@working-theory/page-renderer";
+// next-mdx-remote/rsc, and emits a JSON-LD <script>. Unknown slugs 404 through renderPage.
+import type { getMdxSource as defaultGetMdxSource } from "@working-theory/page-renderer";
+import { renderPage, renderPageMetadata } from "@working-theory/page-renderer";
 import type { Metadata } from "next";
-import { notFound } from "next/navigation";
 import type { ReactElement } from "react";
 
 interface Props {
@@ -23,21 +19,14 @@ export async function generateMetadata({
 
 export default async function Page({
   params,
-  getMdxSource = defaultGetMdxSource,
+  getMdxSource,
 }: Props): Promise<ReactElement> {
   const { locale, slug } = await params;
-  const source = await getMdxSource("Partner", slug, locale);
-
-  if (!source) {
-    notFound();
-  }
-
   return renderPage({
     type: "Partner",
     slug,
     locale,
-    translationsNamespace: "partners",
     bodyMode: "mdx-body",
-    getMdxSource: async () => source,
+    ...(getMdxSource !== undefined && { getMdxSource }),
   });
 }

package/templates/apps/web/app/[locale]/partners/__tests__/page-renderer-migration.test.tsx

@@ -32,6 +32,7 @@ jest.mock("next-mdx-remote/rsc", () => ({
 // ── Import page components ─────────────────────────────────────────────────────
 import PartnersPage from "../page";
 import PartnerSlugPage from "../[slug]/page";
+import * as pageRenderer from "@working-theory/page-renderer";
 
 // ── Shared stubs ───────────────────────────────────────────────────────────────
 const getMdxSourceEmpty = jest.fn().mockResolvedValue(null);
@@ -97,6 +98,7 @@ describe("partners/[slug]/page — snapshot regression", () => {
   });
 
   it("null source triggers notFound()", async () => {
+    const renderPageSpy = jest.spyOn(pageRenderer, "renderPage");
     const { notFound } = jest.requireMock("next/navigation") as {
       notFound: jest.MockedFunction<() => never>;
     };
@@ -108,6 +110,12 @@ describe("partners/[slug]/page — snapshot regression", () => {
         getMdxSource: getMdxSourceEmpty,
       }),
     ).rejects.toThrow("NEXT_NOT_FOUND");
+    // The 404 is renderPage's detail branch, not a route pre-check (#7537).
+    expect(renderPageSpy).toHaveBeenCalledTimes(1);
+    expect(
+      renderPageSpy.mock.calls[0]![0].translationsNamespace,
+    ).toBeUndefined();
+    renderPageSpy.mockRestore();
     expect(notFound).toHaveBeenCalled();
   });
 });

package/templates/apps/web/app/[locale]/portfolio/[slug]/page.tsx

@@ -1,5 +1,5 @@
 // Portfolio item page (#6687). Reads the PortfolioItem MDX for the given slug and emits a
-// JSON-LD <script>. Unknown slugs trigger 404. Renders `sections[]` when the
+// JSON-LD <script>. Unknown slugs 404 through renderPage. Renders `sections[]` when the
 // frontmatter carries them (the /for composed pattern), MDX prose otherwise.
 // NO_HAND_IMPLEMENT_OK: /build-starter operator-directed live pass (#6687).
 import {
@@ -8,7 +8,6 @@ import {
   renderPageMetadata,
 } from "@working-theory/page-renderer";
 import type { Metadata } from "next";
-import { notFound } from "next/navigation";
 import type { ReactElement } from "react";
 
 interface Props {
@@ -28,13 +27,12 @@ export default async function Page({
   getMdxSource = defaultGetMdxSource,
 }: Props): Promise<ReactElement> {
   const { locale, slug } = await params;
+  // Read once to pick the body mode. A null source still goes to renderPage,
+  // which owns the unknown-slug branch (#7537).
   const source = await getMdxSource("PortfolioItem", slug, locale);
 
-  if (!source) {
-    notFound();
-  }
-
   const composed =
+    source !== null &&
     Array.isArray((source.frontmatter as { sections?: unknown[] }).sections) &&
     ((source.frontmatter as { sections?: unknown[] }).sections?.length ?? 0) > 0;
 
@@ -42,7 +40,6 @@ export default async function Page({
     type: "PortfolioItem",
     slug,
     locale,
-    translationsNamespace: "portfolio",
     ...(composed ? {} : { bodyMode: "mdx-body" as const }),
     getMdxSource: async () => source,
   });

package/templates/apps/web/app/[locale]/products/[slug]/page.tsx

@@ -1,12 +1,8 @@
 // Product detail page. Reads the Product MDX for the given slug, renders via
-// next-mdx-remote/rsc, and emits a JSON-LD <script>. Unknown slugs trigger 404.
-import {
-  renderPage,
-  renderPageMetadata,
-  getMdxSource as defaultGetMdxSource,
-} from "@working-theory/page-renderer";
+// next-mdx-remote/rsc, and emits a JSON-LD <script>. Unknown slugs 404 through renderPage.
+import type { getMdxSource as defaultGetMdxSource } from "@working-theory/page-renderer";
+import { renderPage, renderPageMetadata } from "@working-theory/page-renderer";
 import type { Metadata } from "next";
-import { notFound } from "next/navigation";
 import type { ReactElement } from "react";
 
 interface Props {
@@ -23,20 +19,15 @@ export async function generateMetadata({
 
 export default async function Page({
   params,
-  getMdxSource = defaultGetMdxSource,
+  getMdxSource,
 }: Props): Promise<ReactElement> {
   const { locale, slug } = await params;
-  const source = await getMdxSource("Product", slug, locale);
-  if (!source) {
-    notFound();
-  }
   return renderPage({
     type: "Product",
     slug,
     locale,
-    translationsNamespace: "products",
     bodyMode: "mdx-body",
     populatedDescriptionTestId: "product-description",
-    getMdxSource: async () => source,
+    ...(getMdxSource !== undefined && { getMdxSource }),
   });
 }

package/templates/apps/web/app/[locale]/products/__tests__/page-renderer-migration.test.tsx

@@ -29,6 +29,7 @@ jest.mock("next/navigation", () => ({
 import { notFound } from "next/navigation";
 import ProductsPage from "../page";
 import ProductSlugPage from "../[slug]/page";
+import * as pageRenderer from "@working-theory/page-renderer";
 
 const mockNotFound = notFound as jest.MockedFunction<typeof notFound>;
 
@@ -117,6 +118,7 @@ describe("ProductSlugPage — snapshot regression", () => {
   });
 
   it("null source calls notFound()", async () => {
+    const renderPageSpy = jest.spyOn(pageRenderer, "renderPage");
     const getMdxSourceNull = jest.fn().mockResolvedValue(null);
     await expect(
       ProductSlugPage({
@@ -124,6 +126,12 @@ describe("ProductSlugPage — snapshot regression", () => {
         getMdxSource: getMdxSourceNull,
       }),
     ).rejects.toThrow("NEXT_NOT_FOUND");
+    // The 404 is renderPage's detail branch, not a route pre-check (#7537).
+    expect(renderPageSpy).toHaveBeenCalledTimes(1);
+    expect(
+      renderPageSpy.mock.calls[0]![0].translationsNamespace,
+    ).toBeUndefined();
+    renderPageSpy.mockRestore();
     expect(mockNotFound).toHaveBeenCalled();
   });
 });

package/templates/apps/web/app/[locale]/purchase/__tests__/module-scope.test.ts

@@ -18,8 +18,8 @@
  *
  * The fix — `../../../../src/lib/commerce-sections.ts`'s
  * `ensureCommerceSectionsRegistered()` — registers the module LAZILY, via a
- * `webpackIgnore`-annotated `import()` wrapped in try/catch, so the build
- * never needs the package to resolve. This test now pins the ABSENCE of the
+ * generated paid-sections registry (`lib/paid-sections.generated.ts`, #7536),
+ * so the build never needs the package to resolve. This test now pins the ABSENCE of the
  * static import (the regression this file exists to catch) and the
  * PRESENCE of the lazy-registration call site instead.
  */

package/templates/apps/web/app/[locale]/resources/[slug]/page.tsx

@@ -1,15 +1,8 @@
 // Resource detail page. Reads the ResourceDetail MDX for the given slug, renders
-// via renderPage (mdx-body mode), and emits a JSON-LD <script>. Unknown slugs trigger 404.
-import type {
-  getMdxSource as defaultGetMdxSource,
-} from "@working-theory/page-renderer";
-import {
-  getMdxSource as defaultGetMdxSourceImpl,
-  renderPage,
-  renderPageMetadata,
-} from "@working-theory/page-renderer";
+// via renderPage (mdx-body mode), and emits a JSON-LD <script>. Unknown slugs 404 through renderPage.
+import type { getMdxSource as defaultGetMdxSource } from "@working-theory/page-renderer";
+import { renderPage, renderPageMetadata } from "@working-theory/page-renderer";
 import type { Metadata } from "next";
-import { notFound } from "next/navigation";
 import type { ReactElement } from "react";
 
 interface Props {
@@ -26,25 +19,15 @@ export async function generateMetadata({
 
 export default async function Page({
   params,
-  getMdxSource = defaultGetMdxSourceImpl,
+  getMdxSource,
 }: Props): Promise<ReactElement> {
   const { locale, slug } = await params;
-
-  // Pre-check: verify slug exists; call notFound() on null (not empty-state JSX).
-  const source = await getMdxSource("ResourceDetail", slug, locale);
-  if (!source) {
-    notFound();
-  }
-
-  // translationsNamespace is required by RenderPageInput but unreachable here:
-  // notFound() above ensures renderPage's empty-state branch never executes.
   return renderPage({
     type: "ResourceDetail",
     slug,
     locale,
-    translationsNamespace: slug,
     bodyMode: "mdx-body",
     populatedDescriptionTestId: "resource-description",
-    getMdxSource,
+    ...(getMdxSource !== undefined && { getMdxSource }),
   });
 }

package/templates/apps/web/app/[locale]/resources/__tests__/page-renderer-migration.test.tsx

@@ -45,6 +45,7 @@ jest.mock("next/navigation", () => ({
 
 import ResourcesPage from "../page";
 import ResourceDetailPage from "../[slug]/page";
+import * as pageRenderer from "@working-theory/page-renderer";
 
 const getMdxSourceEmpty = jest.fn().mockResolvedValue(null);
 
@@ -115,12 +116,19 @@ describe("ResourceDetailPage — snapshot regression", () => {
   });
 
   it("notFound state calls notFound() when source is null", async () => {
+    const renderPageSpy = jest.spyOn(pageRenderer, "renderPage");
     await expect(
       ResourceDetailPage({
         params: Promise.resolve({ locale: "en", slug: "missing" }),
         getMdxSource: getMdxSourceEmpty,
       }),
     ).rejects.toThrow("NEXT_NOT_FOUND");
+    // The 404 is renderPage's detail branch, not a route pre-check (#7537).
+    expect(renderPageSpy).toHaveBeenCalledTimes(1);
+    expect(
+      renderPageSpy.mock.calls[0]![0].translationsNamespace,
+    ).toBeUndefined();
+    renderPageSpy.mockRestore();
     expect(mockNotFound).toHaveBeenCalledTimes(1);
   });
 });

package/templates/apps/web/app/[locale]/services/[slug]/page.tsx

@@ -1,12 +1,8 @@
 // Service detail page. Reads the Service MDX for the given slug, renders via
-// next-mdx-remote/rsc, and emits a JSON-LD <script>. Unknown slugs trigger 404.
-import {
-  renderPage,
-  renderPageMetadata,
-  getMdxSource as defaultGetMdxSource,
-} from "@working-theory/page-renderer";
+// next-mdx-remote/rsc, and emits a JSON-LD <script>. Unknown slugs 404 through renderPage.
+import type { getMdxSource as defaultGetMdxSource } from "@working-theory/page-renderer";
+import { renderPage, renderPageMetadata } from "@working-theory/page-renderer";
 import type { Metadata } from "next";
-import { notFound } from "next/navigation";
 import type { ReactElement } from "react";
 
 interface Props {
@@ -23,20 +19,15 @@ export async function generateMetadata({
 
 export default async function Page({
   params,
-  getMdxSource = defaultGetMdxSource,
+  getMdxSource,
 }: Props): Promise<ReactElement> {
   const { locale, slug } = await params;
-  const source = await getMdxSource("Service", slug, locale);
-  if (!source) {
-    notFound();
-  }
   return renderPage({
     type: "Service",
     slug,
     locale,
-    translationsNamespace: "services",
     bodyMode: "mdx-body",
     populatedDescriptionTestId: "service-description",
-    getMdxSource: async () => source,
+    ...(getMdxSource !== undefined && { getMdxSource }),
   });
 }

package/templates/apps/web/app/[locale]/services/__tests__/page-renderer-migration.test.tsx

@@ -29,6 +29,7 @@ jest.mock("next/navigation", () => ({
 import { notFound } from "next/navigation";
 import ServicesPage from "../page";
 import ServiceSlugPage from "../[slug]/page";
+import * as pageRenderer from "@working-theory/page-renderer";
 
 const mockNotFound = notFound as jest.MockedFunction<typeof notFound>;
 
@@ -117,6 +118,7 @@ describe("ServiceSlugPage — snapshot regression", () => {
   });
 
   it("null source calls notFound()", async () => {
+    const renderPageSpy = jest.spyOn(pageRenderer, "renderPage");
     const getMdxSourceNull = jest.fn().mockResolvedValue(null);
     await expect(
       ServiceSlugPage({
@@ -124,6 +126,12 @@ describe("ServiceSlugPage — snapshot regression", () => {
         getMdxSource: getMdxSourceNull,
       }),
     ).rejects.toThrow("NEXT_NOT_FOUND");
+    // The 404 is renderPage's detail branch, not a route pre-check (#7537).
+    expect(renderPageSpy).toHaveBeenCalledTimes(1);
+    expect(
+      renderPageSpy.mock.calls[0]![0].translationsNamespace,
+    ).toBeUndefined();
+    renderPageSpy.mockRestore();
     expect(mockNotFound).toHaveBeenCalled();
   });
 });

package/templates/apps/web/jest.config.cjs

@@ -117,7 +117,7 @@ module.exports = {
     // Collapse every copy onto the one this workspace resolves, so the
     // existing per-test mocks cover page-renderer's call too. Resolved at
     // config load — no hard-coded virtual-store path.
-    "^next/navigation$": require.resolve("next/navigation"),
+    "^next/navigation(\\.js)?$": require.resolve("next/navigation"),
     "^next-intl/server$": "<rootDir>/__tests__/__mocks__/next-intl-server.ts",
     "^next-intl$": "<rootDir>/__tests__/__mocks__/next-intl.ts",
     // next-mdx-remote/rsc is ESM-only; map to a CJS stub so ts-jest can load