@sarimarcus/content-sites-core 0.28.0 → 0.29.0

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 All packages in the platform release in lockstep; entries are per release version.
 
+## 0.29.0 (2026-10-06)
+
+- No changes; lockstep release with ui and tourism (SLA-2993, SLA-2995).
+
+## 0.28.1 (2026-10-06)
+
+- `./schema` structured data no longer publishes false facts (SLA-2983):
+  - `isFreeEntry` / `parsePriceFromString`: price or fee text is free only when it leads with "free" and names no amount and no condition. Paid text with a free concession ("€15 general … free under 10") gives `isAccessibleForFree: false` and its first price; a conditional leading "free" ("Free for EU citizens; €4.50 for others") omits `isAccessibleForFree` and gives no price, so an event emits no Offer priced at a side cost. A `free` tag still wins.
+  - `aggregateRating` returns nothing without a real `ratingCount`, instead of inventing `reviewCount: 1`.
+  - `itemListBuilder` and touristTrip stop URLs resolve through the RouteMap (`ctx.route`), so Dark Skies lists link `/lodges/` and `/events/` instead of the redirecting `/hotel/` and `/event/`.
+  Output changes on the affected pages only; no API change, no migration.
+
 ## 0.28.0 (2026-10-05)
 
 - `./articles`: `selectFeaturedArticles(sorted, snapshot, { slugOf, now, shelfSize = 3, maxAgeDays = 21 })` splits a newest-first article list into a hero, a trending shelf and the rest, from the committed `src/data/featured-articles.json` snapshot the factory writes from GSC data. A snapshot older than `maxAgeDays`, or naming no current article, falls back to the newest article as hero with no shelf. It replaces the sites' `getFeaturedAndRest` copies (SLA-2972).

package/dist/schema/pages.d.ts

@@ -28,7 +28,7 @@ export interface ItemListOptions {
     /** `data.itemType` → schema type. Given: each item is a typed nested entity; absent: name/url on the ListItem. */
     itemTypes?: Record<string, string>;
 }
-/** ItemList of `data.items` linked at `/${data.itemType}/${slug}`. */
+/** ItemList of `data.items` linked at the `data.itemType` route. */
 export declare function itemListBuilder(options?: ItemListOptions): SchemaBuilder;
 export declare function howToBuilder(): SchemaBuilder;
 export interface AboutPageOptions {

package/dist/schema/pages.js

@@ -108,7 +108,7 @@ export function collectionPageBuilder(options = {}) {
         };
     };
 }
-/** ItemList of `data.items` linked at `/${data.itemType}/${slug}`. */
+/** ItemList of `data.items` linked at the `data.itemType` route. */
 export function itemListBuilder(options = {}) {
     const { itemTypes } = options;
     return (data, ctx) => ({
@@ -117,7 +117,7 @@ export function itemListBuilder(options = {}) {
         description: data.description,
         numberOfItems: data.items?.length || 0,
         itemListElement: data.items?.map((item, index) => {
-            const url = ctx.abs(`/${data.itemType}/${item.slug}`);
+            const url = ctx.route(data.itemType, item.slug);
             return itemTypes
                 ? { '@type': 'ListItem', position: index + 1, item: { '@type': itemTypes[data.itemType] ?? 'WebPage', name: item.name, url } }
                 : { '@type': 'ListItem', position: index + 1, name: item.name, url };

package/dist/schema/places.js

@@ -61,7 +61,7 @@ export function touristTripBuilder(options = {}) {
                     name: stop.name,
                     description: stop.description,
                     geo: geoCoordinates(stop.coordinates),
-                    ...(stop.slug && stop.type !== 'landmark' ? { url: ctx.abs(`/${stop.type}/${stop.slug}`) } : {}),
+                    ...(stop.slug && stop.type !== 'landmark' ? { url: ctx.route(stop.type, stop.slug) } : {}),
                 },
             })),
         },

package/dist/schema/values.d.ts

@@ -1,9 +1,13 @@
 import type { JsonLdNode } from './types.ts';
-/** First integer in a price string; any "free" reads as 0; null when there is no number. */
+/**
+ * First integer in a price string; 0 when `isFreeText`; null when there is no number, or when the text leads with a
+ * conditional "free", whose first number is a side cost ("Free (…). Shuttle: €2"), not the price.
+ */
 export declare function parsePriceFromString(priceStr: string | undefined): number | null;
 /**
- * Free entry: a `free` tag, or "free" in the price text (`price` for events, `fees` for places). `undefined` when
- * neither is present, so the key drops out of the JSON-LD.
+ * Free entry: a `free` tag, or price text that `isFreeText` accepts (`price` for events, `fees` for places).
+ * `undefined`, so the key drops out of the JSON-LD, when neither is present or the text leads with a conditional
+ * "free" ("Free for EU citizens; …"), which proves neither.
  */
 export declare function isFreeEntry(data: Record<string, any>, field?: 'price' | 'fees'): boolean | undefined;
 /** Street and number only: drops postal code, city and country, and a second part that is a district or postcode. */
@@ -41,6 +45,7 @@ export declare function geoCoordinates(coordinates: {
     lat: number;
     lng: number;
 } | undefined): JsonLdNode | undefined;
+/** Only with a real `ratingCount`: a rating without a count is an invalid AggregateRating, and inventing one is false. */
 export declare function aggregateRating(data: {
     rating?: number;
     ratingCount?: number;

package/dist/schema/values.js

@@ -1,18 +1,42 @@
-/** First integer in a price string; any "free" reads as 0; null when there is no number. */
+const LEADING_FREE = /^\s*free\b/i;
+const FREE_QUALIFIER = /\b(or|for|with|at|except|paid|fees?|charges?|priced|included|rates?|applies|apply)\b/i;
+/**
+ * Price text that says the whole thing is free: it starts with "free" and names no amount and no condition.
+ * "€15 … free under 10" and "Free for EU citizens; €4.50 for others" are not free.
+ */
+function isFreeText(priceStr) {
+    if (!priceStr || !LEADING_FREE.test(priceStr))
+        return false;
+    return !/\d/.test(priceStr) && !FREE_QUALIFIER.test(priceStr);
+}
+/**
+ * First integer in a price string; 0 when `isFreeText`; null when there is no number, or when the text leads with a
+ * conditional "free", whose first number is a side cost ("Free (…). Shuttle: €2"), not the price.
+ */
 export function parsePriceFromString(priceStr) {
     if (!priceStr)
         return null;
-    if (priceStr.toLowerCase().includes('free'))
+    if (isFreeText(priceStr))
         return 0;
+    if (LEADING_FREE.test(priceStr))
+        return null;
     const match = priceStr.match(/(\d+)/);
     return match ? parseFloat(match[1]) : null;
 }
 /**
- * Free entry: a `free` tag, or "free" in the price text (`price` for events, `fees` for places). `undefined` when
- * neither is present, so the key drops out of the JSON-LD.
+ * Free entry: a `free` tag, or price text that `isFreeText` accepts (`price` for events, `fees` for places).
+ * `undefined`, so the key drops out of the JSON-LD, when neither is present or the text leads with a conditional
+ * "free" ("Free for EU citizens; …"), which proves neither.
  */
 export function isFreeEntry(data, field = 'price') {
-    return data.tags?.includes('free') || data[field]?.toLowerCase().includes('free');
+    if (data.tags?.includes('free'))
+        return true;
+    const text = data[field];
+    if (!text)
+        return undefined;
+    if (isFreeText(text))
+        return true;
+    return LEADING_FREE.test(text) ? undefined : false;
 }
 /** Street and number only: drops postal code, city and country, and a second part that is a district or postcode. */
 export function cleanStreetAddress(fullAddress) {
@@ -82,12 +106,13 @@ export function postalAddress(policy, data, parts = {}) {
 export function geoCoordinates(coordinates) {
     return coordinates ? { '@type': 'GeoCoordinates', latitude: coordinates.lat, longitude: coordinates.lng } : undefined;
 }
+/** Only with a real `ratingCount`: a rating without a count is an invalid AggregateRating, and inventing one is false. */
 export function aggregateRating(data) {
-    return data.rating ? {
+    return data.rating && data.ratingCount ? {
         '@type': 'AggregateRating',
         ratingValue: data.rating,
         bestRating: 5,
         worstRating: 1,
-        reviewCount: data.ratingCount ?? 1,
+        reviewCount: data.ratingCount,
     } : undefined;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sarimarcus/content-sites-core",
-  "version": "0.28.0",
+  "version": "0.29.0",
   "description": "Non-visual utilities, Astro config builders and SEO schema assembly shared by the content sites.",
   "type": "module",
   "license": "MIT",

package/src/schema/pages.ts

@@ -143,7 +143,7 @@ export interface ItemListOptions {
   itemTypes?: Record<string, string>;
 }
 
-/** ItemList of `data.items` linked at `/${data.itemType}/${slug}`. */
+/** ItemList of `data.items` linked at the `data.itemType` route. */
 export function itemListBuilder(options: ItemListOptions = {}): SchemaBuilder {
   const { itemTypes } = options;
   return (data, ctx) => ({
@@ -152,7 +152,7 @@ export function itemListBuilder(options: ItemListOptions = {}): SchemaBuilder {
     description: data.description,
     numberOfItems: data.items?.length || 0,
     itemListElement: data.items?.map((item: any, index: number) => {
-      const url = ctx.abs(`/${data.itemType}/${item.slug}`);
+      const url = ctx.route(data.itemType, item.slug);
       return itemTypes
         ? { '@type': 'ListItem', position: index + 1, item: { '@type': itemTypes[data.itemType] ?? 'WebPage', name: item.name, url } }
         : { '@type': 'ListItem', position: index + 1, name: item.name, url };

package/src/schema/places.ts

@@ -90,7 +90,7 @@ export function touristTripBuilder(options: TouristTripOptions = {}): SchemaBuil
           name: stop.name,
           description: stop.description,
           geo: geoCoordinates(stop.coordinates),
-          ...(stop.slug && stop.type !== 'landmark' ? { url: ctx.abs(`/${stop.type}/${stop.slug}`) } : {}),
+          ...(stop.slug && stop.type !== 'landmark' ? { url: ctx.route(stop.type, stop.slug) } : {}),
         },
       })),
     },

package/src/schema/values.ts

@@ -1,19 +1,40 @@
 import type { JsonLdNode } from './types.ts';
 
-/** First integer in a price string; any "free" reads as 0; null when there is no number. */
+const LEADING_FREE = /^\s*free\b/i;
+const FREE_QUALIFIER = /\b(or|for|with|at|except|paid|fees?|charges?|priced|included|rates?|applies|apply)\b/i;
+
+/**
+ * Price text that says the whole thing is free: it starts with "free" and names no amount and no condition.
+ * "€15 … free under 10" and "Free for EU citizens; €4.50 for others" are not free.
+ */
+function isFreeText(priceStr: string | undefined): boolean {
+  if (!priceStr || !LEADING_FREE.test(priceStr)) return false;
+  return !/\d/.test(priceStr) && !FREE_QUALIFIER.test(priceStr);
+}
+
+/**
+ * First integer in a price string; 0 when `isFreeText`; null when there is no number, or when the text leads with a
+ * conditional "free", whose first number is a side cost ("Free (…). Shuttle: €2"), not the price.
+ */
 export function parsePriceFromString(priceStr: string | undefined): number | null {
   if (!priceStr) return null;
-  if (priceStr.toLowerCase().includes('free')) return 0;
+  if (isFreeText(priceStr)) return 0;
+  if (LEADING_FREE.test(priceStr)) return null;
   const match = priceStr.match(/(\d+)/);
   return match ? parseFloat(match[1]) : null;
 }
 
 /**
- * Free entry: a `free` tag, or "free" in the price text (`price` for events, `fees` for places). `undefined` when
- * neither is present, so the key drops out of the JSON-LD.
+ * Free entry: a `free` tag, or price text that `isFreeText` accepts (`price` for events, `fees` for places).
+ * `undefined`, so the key drops out of the JSON-LD, when neither is present or the text leads with a conditional
+ * "free" ("Free for EU citizens; …"), which proves neither.
  */
 export function isFreeEntry(data: Record<string, any>, field: 'price' | 'fees' = 'price'): boolean | undefined {
-  return data.tags?.includes('free') || data[field]?.toLowerCase().includes('free');
+  if (data.tags?.includes('free')) return true;
+  const text = data[field];
+  if (!text) return undefined;
+  if (isFreeText(text)) return true;
+  return LEADING_FREE.test(text) ? undefined : false;
 }
 
 /** Street and number only: drops postal code, city and country, and a second part that is a district or postcode. */
@@ -99,12 +120,13 @@ export function geoCoordinates(coordinates: { lat: number; lng: number } | undef
   return coordinates ? { '@type': 'GeoCoordinates', latitude: coordinates.lat, longitude: coordinates.lng } : undefined;
 }
 
+/** Only with a real `ratingCount`: a rating without a count is an invalid AggregateRating, and inventing one is false. */
 export function aggregateRating(data: { rating?: number; ratingCount?: number }): JsonLdNode | undefined {
-  return data.rating ? {
+  return data.rating && data.ratingCount ? {
     '@type': 'AggregateRating',
     ratingValue: data.rating,
     bestRating: 5,
     worstRating: 1,
-    reviewCount: data.ratingCount ?? 1,
+    reviewCount: data.ratingCount,
   } : undefined;
 }