--- phase: 07-seo-blog plan: 04 type: execute wave: 2 depends_on: [07-01] files_modified: - server/api/__sitemap__/urls.ts autonomous: true requirements: [SEO-12] must_haves: truths: - "curl /sitemap.xml contient les URLs /fr/blog/{slug} ET /en/blog/{slug} pour chaque article non-draft" - "Chaque entrée d'un article bilingue contient xhtml:link alternate hreflang=fr, hreflang=en, et hreflang=x-default pointant vers la version FR" - "Articles draft:true sont ABSENTS du sitemap" - "lastmod = updated frontmatter si présent, sinon date" artifacts: - path: "server/api/__sitemap__/urls.ts" provides: "defineSitemapEventHandler retournant SitemapUrl[] bilingue" contains: "defineSitemapEventHandler" key_links: - from: "nuxt.config.ts > sitemap.sources" to: "server/api/__sitemap__/urls.ts" via: "/api/__sitemap__/urls HTTP route" pattern: "__sitemap__/urls" --- Créer l'endpoint Nitro `server/api/__sitemap__/urls.ts` qui alimente `@nuxtjs/sitemap` avec les URLs /blog/{slug} bilingues + alternates hreflang, filtrées sur `draft=false`, avec `lastmod` dérivé de `updated ?? date` (D-08, D-09, D-10, D-11, SEO-12). Purpose: Sans ce feed, le sitemap dynamique ne référence pas les articles → Google ne découvre pas les pages blog. Output: 1 endpoint Nitro créé. @$HOME/.claude/get-shit-done/workflows/execute-plan.md @$HOME/.claude/get-shit-done/templates/summary.md @.planning/phases/07-seo-blog/07-CONTEXT.md @.planning/phases/07-seo-blog/07-RESEARCH.md @.planning/phases/07-seo-blog/07-PATTERNS.md @.planning/phases/07-seo-blog/07-01-SUMMARY.md @server/plugins/reading-time.ts @server/api/contact.post.ts **Critique (Pitfall 1 RESEARCH)** : Dans les routes Nitro, `queryCollection` prend `event` en PREMIER argument (contrairement au context client/SSR page). **Critique (Pitfall 2)** : Toujours strings littérales — `queryCollection(event, 'blog_fr')` puis `queryCollection(event, 'blog_en')`, JAMAIS `queryCollection(event, 'blog_' + locale)`. Import canonique : `import { defineSitemapEventHandler } from '#imports'` et `import type { SitemapUrl } from '#sitemap/types'` (fournis par `@nuxtjs/sitemap` v8). Le schema blog (après 07-01) expose : `path`, `date`, `updated?`, `draft`, `title`, `description`, `image?`, `tags?`. Convention paths @nuxt/content : `/fr/blog/{slug}` et `/en/blog/{slug}` — même slug = paire bilingue (Phase 5/6 convention). Task 1: Créer server/api/__sitemap__/urls.ts — feed sitemap bilingue avec alternates hreflang server/api/__sitemap__/urls.ts - server/plugins/reading-time.ts (pattern Nitro ctx repo) - server/api/contact.post.ts (pattern defineEventHandler) - .planning/phases/07-seo-blog/07-RESEARCH.md §Pattern 3 (Nitro Sitemap Source Endpoint), Pitfalls 1, 2, 5, 6 - .planning/phases/07-seo-blog/07-PATTERNS.md §server/api/__sitemap__/urls.ts (new) - .planning/phases/07-seo-blog/07-CONTEXT.md D-08, D-09, D-10, D-11 Créer le dossier `server/api/__sitemap__/` (s'il n'existe pas) puis le fichier `server/api/__sitemap__/urls.ts` avec le contenu exact ci-dessous : ```ts /** * Dynamic sitemap URL feed for @nuxtjs/sitemap. * Referenced via nuxt.config.ts > sitemap.sources: ['/api/__sitemap__/urls']. * Emits /fr/blog/{slug} + /en/blog/{slug} with hreflang alternates for bilingual pairs. * Excludes drafts (D-10). lastmod = updated ?? date (D-09). See Pitfalls 1, 2, 5, 6 in RESEARCH. */ import { defineSitemapEventHandler } from '#imports' import type { SitemapUrl } from '#sitemap/types' const SITE_URL = 'https://killiandalcin.fr' type BlogRow = { path: string date: string updated?: string } export default defineSitemapEventHandler(async (event) => { // Literal collection strings (Pitfall 2). Pass event first (Pitfall 1). const [frArticles, enArticles] = await Promise.all([ queryCollection(event, 'blog_fr') .where('draft', '=', false) .order('date', 'DESC') .select('path', 'date', 'updated') .all() as unknown as Promise, queryCollection(event, 'blog_en') .where('draft', '=', false) .order('date', 'DESC') .select('path', 'date', 'updated') .all() as unknown as Promise, ]) // Build slug → { fr?, en? } index for pair detection (D-11) const extractSlug = (p: string) => p.split('/').filter(Boolean).pop()! const index = new Map() for (const a of frArticles) { const s = extractSlug(a.path) const e = index.get(s) ?? {} e.fr = a index.set(s, e) } for (const a of enArticles) { const s = extractSlug(a.path) const e = index.get(s) ?? {} e.en = a index.set(s, e) } const urls: SitemapUrl[] = [] for (const [slug, pair] of index) { const bilingual = !!(pair.fr && pair.en) const alternatives = bilingual ? [ { hreflang: 'fr', href: `${SITE_URL}/fr/blog/${slug}` }, { hreflang: 'en', href: `${SITE_URL}/en/blog/${slug}` }, { hreflang: 'x-default', href: `${SITE_URL}/fr/blog/${slug}` }, ] : [] if (pair.fr) { urls.push({ loc: `/fr/blog/${slug}`, lastmod: pair.fr.updated ?? pair.fr.date, alternatives, }) } if (pair.en) { urls.push({ loc: `/en/blog/${slug}`, lastmod: pair.en.updated ?? pair.en.date, alternatives, }) } } return urls }) ``` Ne PAS toucher aux autres fichiers server/. Ne PAS re-créer `public/sitemap.xml` (FIX-01 supprimé). test -f server/api/__sitemap__/urls.ts && grep -q "defineSitemapEventHandler" server/api/__sitemap__/urls.ts && grep -q "queryCollection(event, 'blog_fr')" server/api/__sitemap__/urls.ts && grep -q "queryCollection(event, 'blog_en')" server/api/__sitemap__/urls.ts && grep -q "'x-default'" server/api/__sitemap__/urls.ts && pnpm typecheck && pnpm dev --port 3000 & sleep 12 && curl -s http://localhost:3000/sitemap.xml | tee /tmp/sitemap.xml | grep -q '/fr/blog/' && grep -q '/en/blog/' /tmp/sitemap.xml && grep -q 'hreflang="x-default"' /tmp/sitemap.xml && ! grep -q 'test-kotlin-syntax' /tmp/sitemap.xml && kill %1 curl /sitemap.xml contient : au moins une URL /fr/blog/... ET /en/blog/..., xhtml:link hreflang=fr/en/x-default pour paires bilingues, les articles draft (ex: test-kotlin-syntax) SONT ABSENTS. typecheck vert. ## Trust Boundaries | Boundary | Description | |----------|-------------| | client (crawler) → /sitemap.xml | Endpoint public lecture seule, agrégation d'URLs publiques | ## STRIDE Threat Register | Threat ID | Category | Component | Disposition | Mitigation Plan | |-----------|----------|-----------|-------------|-----------------| | T-07-06 | Information Disclosure | Drafts (contenu non publié) | mitigate | Filtre obligatoire `.where('draft', '=', false)` — testé dans verify (absence `test-kotlin-syntax`) | | T-07-07 | DoS | Endpoint sitemap (query SQLite à chaque hit) | accept | @nuxtjs/sitemap v8 met en cache ; volume d'articles petit (<100) | | T-07-08 | Tampering | `extractSlug` parse path | mitigate | `path` est trusté (généré par @nuxt/content depuis le filesystem, pas user input) | - Endpoint en place : `test -f server/api/__sitemap__/urls.ts` - event first-arg (Pitfall 1) : `grep "queryCollection(event, 'blog_" server/api/__sitemap__/urls.ts` (2 matchs attendus) - Drafts exclus (Pitfall 5) : `grep "draft.*false" server/api/__sitemap__/urls.ts` - Sitemap HTTP : `curl /sitemap.xml | grep '/fr/blog/'` et `/en/blog/` - hreflang : `curl /sitemap.xml | grep 'hreflang="x-default"'` - Drafts filtrés en runtime : `curl /sitemap.xml | grep test-kotlin-syntax` DOIT retourner exit 1 1. SEO-12 : `curl /sitemap.xml` contient `/fr/blog/{slug}` ET `/en/blog/{slug}` pour chaque article non-draft 2. D-10 respecté : drafts absents du sitemap 3. D-11 respecté : paires bilingues portent les 3 alternates (fr, en, x-default); articles mono-langue pas d'alternate 4. D-09 respecté : `lastmod` reflète `updated ?? date` Après complétion, créer `.planning/phases/07-seo-blog/07-04-SUMMARY.md`.