@ultimat3/seo 2.0.0 → 3.0.0

package/CLAUDE.md

@@ -28,6 +28,17 @@ Tier 1. May import `@ultimat3/core`, `@ultimat3/schema`, `@ultimat3/i18n`. Nothi
   over the route manifest and the build's own stats, throwing `@ultimat3/render`'s
   `X_BUDGET_EXCEEDED`. seo is tier 1 and cannot see a build's bytes, so it was never the package
   that could answer. `errors.test.ts` pins the code set, so re-adding one is a failing test.
+- **A length bound with no enforcer does not ship.** `DESCRIPTION_MIN_LENGTH` (50) sat in
+  `meta.ts` under the comment "validate.ts enforces it", was re-exported from `index.ts`, and no
+  validator anywhere read it — a 10-character description passed the gate the constant claimed to
+  fail. Deleted `As of 2026-08`, comment included; `validateMeta` enforces maxima only. Adding a
+  minimum back means adding the check AND a new `X_SEO_*` code in the same change.
+  `meta.test.ts` pins the exported `*_LENGTH` set, so a bound with no enforcer is a failing test.
+- **The `<img src>` fallback is the LARGEST usable width, chosen with `Math.max`.**
+  `usableWidths` preserves the CALLER's order, so `widths[widths.length - 1]` was the largest only
+  because `DEFAULT_WIDTHS` happens to ascend — `widths: [1200, 640]` handed every browser without
+  `srcset` support the 640 variant of a 1200-wide image. Never re-derive it from position, and
+  never sort inside `usableWidths`: the `srcset` order is the caller's to choose.
 - **Errors name the file, not the URL.** `RouteRecord.file` is in every cause and every fix; an agent must be able to open the source without guessing.
 - **Fail closed, and core reads the key.** `isIndexable()` is `environment === 'production'` and
   nothing else — `staging`, a laptop, a typo and an unset variable all disallow. `ULTIMATE_ENV` has

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ultimat3/seo",
-  "version": "2.0.0",
+  "version": "3.0.0",
   "description": "Enforced SEO: typed meta, JSON-LD, sitemap, robots, feeds, responsive images, perf budgets",
   "license": "MIT",
   "type": "module",
@@ -31,6 +31,6 @@
     "test": "bun test"
   },
   "dependencies": {
-    "@ultimat3/core": "2.0.0"
+    "@ultimat3/core": "3.0.0"
   }
 }

package/src/images.ts

@@ -176,6 +176,11 @@ export function usableWidths(intrinsic: number, widths: readonly number[]): read
   return usable.includes(intrinsic) ? usable : [...usable, intrinsic];
 }
 
+/** The widest candidate, or `undefined` for an empty list — never `Math.max()`'s `-Infinity`. */
+function largestOf(widths: readonly number[]): number | undefined {
+  return widths.length === 0 ? undefined : Math.max(...widths);
+}
+
 export function srcsetFor(
   input: ImageInput,
   widths: readonly number[],
@@ -212,7 +217,10 @@ export function responsiveImage(
   return {
     sources,
     img: {
-      src: urlFor(input.src, widths[widths.length - 1] ?? input.width, undefined),
+      // `Math.max`, not the last element: `usableWidths` preserves the CALLER's order, so the
+      // tail was the largest only because `DEFAULT_WIDTHS` happens to ascend. `widths: [1200, 640]`
+      // handed every no-`srcset` browser the 640 variant of a 1200-wide image.
+      src: urlFor(input.src, largestOf(widths) ?? input.width, undefined),
       srcset: srcsetFor(input, widths, undefined, urlFor),
       sizes,
       alt: input.alt,

package/src/index.ts

@@ -84,7 +84,6 @@ export type {
 export {
   applyTitleTemplate,
   DESCRIPTION_MAX_LENGTH,
-  DESCRIPTION_MIN_LENGTH,
   hreflangSet,
   renderMeta,
   robotsContent,

package/src/meta.ts

@@ -4,9 +4,12 @@
 
 import { absoluteUrl } from './xml';
 
-/** Search results truncate past this; validate.ts enforces it. */
+/**
+ * Search results truncate past these; `validate.ts` enforces both, and only these two exist for
+ * that reason — a `DESCRIPTION_MIN_LENGTH` shipped here with no validator reading it, so the
+ * comment promised a gate that never ran. A bound with no enforcer does not ship.
+ */
 export const TITLE_MAX_LENGTH = 60;
-export const DESCRIPTION_MIN_LENGTH = 50;
 export const DESCRIPTION_MAX_LENGTH = 160;
 
 export interface RobotsDirectives {