@uniweb/projections 0.3.1 → 0.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uniweb/projections",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "Projections of a Uniweb site's content — agent index, per-page markdown, search index. Pure JS, runs anywhere.",
   "type": "module",
   "exports": {
@@ -31,8 +31,8 @@
     "node": ">=20.19"
   },
   "dependencies": {
-    "@uniweb/content-writer": "^0.3.3",
-    "@uniweb/core": "^0.8.5"
+    "@uniweb/core": "^0.9.0",
+    "@uniweb/content-writer": "^0.3.3"
   },
   "devDependencies": {
     "vitest": "^4.1.7",

package/src/search/collections.js

@@ -6,26 +6,57 @@
  * per-record detail files into each item before calling this function.
  */
 
+/**
+ * Compose a record's route the way the build does, for the sources that have
+ * not already done it.
+ *
+ * `collections[name].route` is authored in `site.yml` (see the blog and
+ * international templates). When it is set, the build's collection processor
+ * already stamps `item.route` on every record — so the route below is a
+ * *fallback* for records that arrived without one (an API-backed collection a
+ * host assembled itself), never a second opinion about records that have one.
+ *
+ * The trailing-slash strip matches `collectItems` in
+ * `@uniweb/build`'s `site/collection-processor.js`. Without it a `route:
+ * /blog/` authored with a slash yields `/blog//my-post` here and
+ * `/blog/my-post` there — two answers to one question, on a value nobody
+ * checks until a visitor clicks it.
+ */
+function composeRoute(configRoute, slug) {
+  if (typeof configRoute !== 'string' || configRoute === '') return undefined
+  return `${configRoute.replace(/\/$/, '')}/${slug}`
+}
+
 /**
  * @param {string} name - Collection name (e.g. "articles")
  * @param {Object} config - Collection config from site.yml (config.collections[name])
- * @param {Object} collectionData - Parsed cascade JSON (`data/{name}.json`)
+ * @param {Object[]|Object} collectionData - Parsed cascade JSON (`data/{name}.json`),
+ *   which the build writes as a bare array. The `{ items: [...] }` envelope is
+ *   accepted too, since a host fetching the collection from a backend may carry one.
  * @param {string} locale - Locale code (e.g. "en")
  * @returns {Object} Collection search index
  */
 export function generateCollectionIndex(name, config, collectionData, locale) {
   const fields = config.search?.fields || ['title']
   const weight = config.search?.weight ?? 0.7
-  const items = collectionData?.items || []
+  const items = Array.isArray(collectionData)
+    ? collectionData
+    : collectionData?.items || []
 
   const entries = items.map(item => {
     const content = fields.map(f => item[f] || '').filter(Boolean).join(' ')
     const slug = item.slug || item.id || String(item.title || '').toLowerCase().replace(/\s+/g, '-')
+    // The record's own route wins: the build already resolved it against the
+    // same config, so recomputing here could only disagree. `route` is omitted
+    // entirely when neither source can supply one — a missing key is detectable
+    // by a consumer, where the string "undefined/my-post" is a link that ranks
+    // correctly, looks plausible, and 404s on click.
+    const route = item.route || composeRoute(config.route, slug)
     return {
       id: `collection:${name}:${slug}`,
       type: 'collection',
       collection: name,
-      route: `${config.route}/${slug}`,
+      ...(route ? { route } : {}),
       title: item.title || item.name || slug,
       content,
       excerpt: content.length > 160