@se-studio/skills 1.8.0 → 1.8.2

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @se-studio/skills
 
+## 1.8.2
+
+### Patch Changes
+
+- Map page-test homepage canonical paths (`/`) to CMS slug `index` so control `indexed` is used instead of the variant's `noindex`.
+
+## 1.8.1
+
+### Patch Changes
+
+- 5c18a78: Document that `docs/schema-export.json` is the committed Pedestal golden schema, and that every core migration refreshes and commits that export in the same PR.
+
 ## 1.8.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@se-studio/skills",
-  "version": "1.8.0",
+  "version": "1.8.2",
   "description": "SE Studio agent skills for marketing site development with Contentful CMS",
   "repository": {
     "type": "git",

package/skills/contentful-cms-sync-schema/SKILL.md

@@ -11,9 +11,11 @@ Use when a customer Contentful space should match the **structural** core schema
 
 | Artifact | Purpose |
 |----------|---------|
-| `docs/schema-export.json` | Structural golden (Pedestal `h4s3ip99qawo` / `master`) |
+| `docs/schema-export.json` | **Committed** structural golden (Pedestal `h4s3ip99qawo` / `master`) |
 | `docs/blank-schema.json` | New-project **bootstrap only** — not the sync target |
 
+`docs/schema-export.json` is committed. It is not a local-only artifact.
+
 Refresh golden:
 
 ```bash
@@ -23,8 +25,11 @@ contentful space export \
   --management-token "$CONTENTFUL_MANAGEMENT_TOKEN" \
   --skip-content --skip-webhooks --skip-roles --skip-tags \
   --content-file docs/schema-export.json
+pnpm exec biome format --write docs/schema-export.json
 ```
 
+**Required after a core migration.** After any core migration is applied to the golden space (Pedestal `master`), re-run the Refresh golden export above and commit the updated `docs/schema-export.json` in the same PR as the migration script. The Biome format step keeps `pnpm format:check` green; the CLI export is valid JSON but not Biome-formatted.
+
 ## Workflow
 
 ### 1. Resolve target space

package/skills/se-marketing-sites-create-page/SKILL.md

@@ -138,7 +138,7 @@ If the slug is not a known variant in `testsByPath`, **return 404** (`notFound()
 Direct visits to a variant's own CMS URL (normal `[...slugs]` route) continue to use `generatePage` and show variant breadcrumbs.
 
 ```typescript
-import { findCanonicalPath } from '@se-studio/ab-testing';
+import { controlSlugFromCanonicalPath, findCanonicalPath } from '@se-studio/ab-testing';
 import { joinCmsPathSegments } from '@se-studio/core-ui';
 import type { ResolvingMetadata } from 'next';
 import { notFound } from 'next/navigation';
@@ -174,7 +174,10 @@ export async function generateMetadata(
 ) {
   const { slug } = await extractDetails(props);
   const canonicalPath = requireCanonicalPath(slug);
-  const canonicalSlug = canonicalPath.replace(/^\/|\/$/g, '') || slug;
+  // `/` → CMS slug `index`. Never `replace(...) || slug` — that uses the variant
+  // (indexed: false) and flip-flops robots on the homepage. generatePageTest*
+  // also re-resolves so a stale `|| slug` cannot noindex `/`.
+  const canonicalSlug = controlSlugFromCanonicalPath(canonicalPath);
   // Use generatePageTestMetadata: merges variant title/description with
   // canonical page's indexed flag so robots directives are always correct.
   return generatePageTestMetadata(slug, canonicalSlug, canonicalPath, getPageRouteConfig(canonicalSlug), parent);
@@ -183,12 +186,14 @@ export async function generateMetadata(
 export default async function (props: PageProps<'/page-test/[...slugs]'>) {
   const { slug } = await extractDetails(props);
   const canonicalPath = requireCanonicalPath(slug);
-  const canonicalSlug = canonicalPath.replace(/^\/|\/$/g, '') || slug;
+  const canonicalSlug = controlSlugFromCanonicalPath(canonicalPath);
   // Use generatePageTest: variant contents with control identity (href, breadcrumbs, JSON-LD).
   return generatePageTest(slug, canonicalSlug, canonicalPath, getPageRouteConfig(canonicalSlug));
 }
 ```
 
+If the site homepage CMS slug is not `index` (e.g. Pedestal `home`), pass it: `controlSlugFromCanonicalPath(canonicalPath, 'home')`.
+
 Also export `generatePageTestMetadata` and `generatePageTest` from the app's `src/lib/route-handlers.ts` destructuring alongside `generatePageMetadata`.
 
 ## See Also