@rudra-js/core 0.1.0 → 0.2.0

package/src/signal-digest.ts

@@ -230,14 +230,17 @@ function mergeViewsBySku(views: ViewSignal[]): MergedView[] {
 
 /** The most-viewed few, as the digest reports them. */
 function mostViewedProducts(views: ViewSignal[]): ViewedProduct[] {
-  return mergeViewsBySku(views)
+  const top = mergeViewsBySku(views)
     .toSorted((left, right) => right.views - left.views)
-    .slice(0, DIGEST_LIMITS.viewed)
-    .map(({ sku, views: viewCount, dwellMs }) => ({
-      sku,
-      views: viewCount,
-      ...(dwellMs !== undefined ? { dwellMs } : {}),
-    }));
+    .slice(0, DIGEST_LIMITS.viewed);
+
+  const products: ViewedProduct[] = [];
+  for (const merged of top) {
+    const product: ViewedProduct = { sku: merged.sku, views: merged.views };
+    if (merged.dwellMs !== undefined) product.dwellMs = merged.dwellMs;
+    products.push(product);
+  }
+  return products;
 }
 
 /** The open-vocabulary long tail, reduced to "which kinds, and how often". */

package/src/spec-cache.ts

@@ -1,5 +1,6 @@
 import { createHash } from 'node:crypto';
 import { SPEC_VERSION, type GeneratedSpec } from './component-spec.js';
+import { SYSTEM_PROMPT } from './model-prompt.js';
 import type { SignalDigest } from './signal-digest.js';
 
 /**
@@ -15,11 +16,6 @@ import type { SignalDigest } from './signal-digest.js';
  * instance count.
  */
 
-/**
- * Asynchronous on purpose. A synchronous `get` would look simpler and would
- * make the shared store the docs recommend impossible to write, since no Redis
- * or Memcached client can return a value without awaiting.
- */
 /**
  * A stored generation, with the moment it was produced.
  *
@@ -33,9 +29,15 @@ export interface CachedSpec {
   generatedAt: number;
 }
 
+/**
+ * Asynchronous on purpose. A synchronous `get` would look simpler and would
+ * make the shared store the docs recommend impossible to write, since no Redis
+ * or Memcached client can return a value without awaiting.
+ */
 export interface SpecCache {
   get(key: string): Promise<CachedSpec | undefined>;
   set(key: string, cached: CachedSpec): Promise<void>;
+  delete?(key: string): Promise<void>;
 }
 
 export interface MemorySpecCacheOptions {
@@ -115,6 +117,10 @@ export function createMemorySpecCache(options: MemorySpecCacheOptions = {}): Spe
         entries.delete(oldest.value);
       }
     },
+
+    async delete(key) {
+      entries.delete(key);
+    },
   };
 }
 
@@ -148,6 +154,8 @@ function canonicalise(value: unknown): string {
   return `{${entries.map(([name, fieldValue]) => `${JSON.stringify(name)}:${canonicalise(fieldValue)}`).join(',')}}`;
 }
 
+const PROMPT_FINGERPRINT = createHash('sha256').update(SYSTEM_PROMPT).digest('hex').slice(0, 16);
+
 /**
  * Derives the cache key.
  *
@@ -182,6 +190,7 @@ export function specCacheKey(
     // The spec's own version, so a shape change cannot read entries written by
     // the previous shape out of a shared store that outlives a deploy.
     specVersion: SPEC_VERSION,
+    prompt: PROMPT_FINGERPRINT,
     provider: providerId,
     digest,
     candidates: candidateSkus.toSorted(),
@@ -201,6 +210,7 @@ export function cohortCacheKey(
 ): string {
   const material = canonicalise({
     specVersion: SPEC_VERSION,
+    prompt: PROMPT_FINGERPRINT,
     provider: providerId,
     segment: digest.segment ?? null,
     surface: digest.surface,

package/src/tracking-input.ts

@@ -42,6 +42,8 @@ export const FIELD_LIMITS = {
   candidates: 200,
   productsPerBundle: 5,
   bundles: 20,
+  localeTag: 35,
+  maxItems: 12,
 } as const;
 
 /** Assigning this as an object key mutates the prototype instead of the object. */
@@ -179,9 +181,15 @@ export const renderContextSchema = z.strictObject({
   currentSku: optionalIdentifier(),
   currentCategory: optionalIdentifier(),
   searchQuery: z.string().max(FIELD_LIMITS.searchQuery).optional(),
-  locale: z.string().min(2).max(35).default('en-US'),
+  // One language tag, not the Accept-Language header it is often copied from:
+  // the locale is part of the cohort cache key, so a list makes its own cohort.
+  locale: z
+    .string()
+    .max(FIELD_LIMITS.localeTag)
+    .regex(/^[A-Za-z]{2,8}(?:-[A-Za-z0-9]{1,8})*$/, 'expected one language tag, such as en-US')
+    .default('en-US'),
   /** Upper bound on products across the whole generated component. */
-  maxItems: z.number().int().min(1).max(12).default(4),
+  maxItems: z.number().int().min(1).max(FIELD_LIMITS.maxItems).default(4),
 });
 export type RenderContext = z.infer<typeof renderContextSchema>;