@mostlyrightmd/core 1.1.2 → 1.2.0

package/dist/internal/cache/index.browser.mjs

@@ -1,4 +1,5 @@
 import {
+  CACHE_SCHEMA_VERSION,
   DB_NAME,
   IndexedDBStore,
   MemoryStore,
@@ -10,18 +11,21 @@ import {
   isWritableYear,
   lockKeyFor,
   shouldSkipCacheForCurrentLstMonth,
-  shouldSkipCacheForCurrentLstYear
-} from "../chunk-PKJXHY27.mjs";
+  shouldSkipCacheForCurrentLstYear,
+  versionedCacheStore
+} from "../chunk-IPC4XUYW.mjs";
 
 // src/internal/cache/index.browser.ts
 async function defaultCacheStore() {
-  if (typeof indexedDB !== "undefined") return new IndexedDBStore();
-  return new MemoryStore();
+  const inner = typeof indexedDB !== "undefined" ? new IndexedDBStore() : new MemoryStore();
+  return versionedCacheStore(inner, CACHE_SCHEMA_VERSION);
 }
 export {
+  CACHE_SCHEMA_VERSION,
   DB_NAME as INDEXEDDB_DB_NAME,
   IndexedDBStore,
   MemoryStore,
+  CACHE_SCHEMA_VERSION as VERSIONED_CACHE_SCHEMA_VERSION,
   cacheKeyForClimate,
   cacheKeyForObservations,
   defaultCacheStore,
@@ -31,6 +35,7 @@ export {
   isWritableYear,
   lockKeyFor,
   shouldSkipCacheForCurrentLstMonth,
-  shouldSkipCacheForCurrentLstYear
+  shouldSkipCacheForCurrentLstYear,
+  versionedCacheStore
 };
 //# sourceMappingURL=index.browser.mjs.map

package/dist/internal/cache/index.browser.mjs.map

@@ -1 +1 @@
-{"version":3,"sources":["../../../src/internal/cache/index.browser.ts"],"sourcesContent":["// Browser/MV3 entry for @mostlyrightmd/core/internal/cache.\n//\n// Iter-8 H15: the iter-1/iter-2 fixes (dynamic `await import('./fs.js')`\n// behind a runtime feature-detect + dropping the FsStore re-export from\n// the barrel) eliminated STATIC references to FsStore from the cache\n// subbundle but NOT dynamic ones. esbuild, when bundling for browser/MV3\n// targets, still follows `await import(\"./fs.js\")` from `default.ts`\n// into the FsStore chunk and pulls `node:crypto`, `node:fs/promises`,\n// `node:os`, `node:path`, and `proper-lockfile` into the bundle —\n// breaking `pnpm size` for `packages-ts/meta/dist/index.mjs`.\n//\n// The architectural fix: package.json conditional exports route Node\n// consumers to `./index.ts` (this file's sibling — keeps FsStore via\n// dynamic import) and browser/MV3 consumers to THIS file. This file has\n// NO reference to `./fs.js` via ANY mechanism — static import, dynamic\n// import, or re-export. esbuild cannot follow what isn't there.\n//\n// Exports MUST mirror `index.ts` exactly, MINUS anything that references\n// FsStore. The runtime priority for `defaultCacheStore` here is:\n//   1. `typeof indexedDB !== \"undefined\"` → IndexedDBStore.\n//   2. else → MemoryStore.\n// (The FsStore branch is unreachable in browser/MV3 bundles by\n// construction — there's no `process.versions?.node` in a service\n// worker anyway, but more importantly the source code is absent so\n// esbuild's static analysis cannot drag the Node-only chunk in.)\n\nimport { IndexedDBStore } from \"./indexeddb.js\";\nimport { MemoryStore } from \"./memory.js\";\nimport type { CacheStore } from \"./types.js\";\n\nexport type { CacheStore, CacheSetOptions, CacheEntry } from \"./types.js\";\nexport { lockKeyFor } from \"./types.js\";\nexport { MemoryStore } from \"./memory.js\";\nexport { IndexedDBStore, DB_NAME as INDEXEDDB_DB_NAME } from \"./indexeddb.js\";\nexport type { IndexedDBStoreOptions } from \"./indexeddb.js\";\nexport {\n  shouldSkipCacheForCurrentLstMonth,\n  shouldSkipCacheForCurrentLstYear,\n  isLiveSource,\n  isWithinVolatileWindow,\n  isWritableMonth,\n  isWritableYear,\n} from \"./skip-rules.js\";\nexport { cacheKeyForObservations, cacheKeyForClimate } from \"./keys.js\";\n\n/**\n * Browser/MV3 variant of {@link defaultCacheStore}. Auto-detects the best\n * available CacheStore in a browser/edge environment:\n *\n *   1. IndexedDB present → {@link IndexedDBStore}\n *   2. else → {@link MemoryStore}\n *\n * Returns a NEW instance per call.\n *\n * Iter-8 H15: kept async so the signature matches the Node entry's\n * `defaultCacheStore` (which awaits a dynamic import). Callers that\n * `await defaultCacheStore()` work unchanged when the package.json\n * conditional exports flip them between entries.\n */\nexport async function defaultCacheStore(): Promise<CacheStore> {\n  if (typeof indexedDB !== \"undefined\") return new IndexedDBStore();\n  return new MemoryStore();\n}\n"],"mappings":";;;;;;;;;;;;;;;;AA2DA,eAAsB,oBAAyC;AAC7D,MAAI,OAAO,cAAc,YAAa,QAAO,IAAI,eAAe;AAChE,SAAO,IAAI,YAAY;AACzB;","names":[]}
+{"version":3,"sources":["../../../src/internal/cache/index.browser.ts"],"sourcesContent":["// Browser/MV3 entry for @mostlyrightmd/core/internal/cache.\n//\n// Iter-8 H15: the iter-1/iter-2 fixes (dynamic `await import('./fs.js')`\n// behind a runtime feature-detect + dropping the FsStore re-export from\n// the barrel) eliminated STATIC references to FsStore from the cache\n// subbundle but NOT dynamic ones. esbuild, when bundling for browser/MV3\n// targets, still follows `await import(\"./fs.js\")` from `default.ts`\n// into the FsStore chunk and pulls `node:crypto`, `node:fs/promises`,\n// `node:os`, `node:path`, and `proper-lockfile` into the bundle —\n// breaking `pnpm size` for `packages-ts/meta/dist/index.mjs`.\n//\n// The architectural fix: package.json conditional exports route Node\n// consumers to `./index.ts` (this file's sibling — keeps FsStore via\n// dynamic import) and browser/MV3 consumers to THIS file. This file has\n// NO reference to `./fs.js` via ANY mechanism — static import, dynamic\n// import, or re-export. esbuild cannot follow what isn't there.\n//\n// Exports MUST mirror `index.ts` exactly, MINUS anything that references\n// FsStore. The runtime priority for `defaultCacheStore` here is:\n//   1. `typeof indexedDB !== \"undefined\"` → IndexedDBStore.\n//   2. else → MemoryStore.\n// (The FsStore branch is unreachable in browser/MV3 bundles by\n// construction — there's no `process.versions?.node` in a service\n// worker anyway, but more importantly the source code is absent so\n// esbuild's static analysis cannot drag the Node-only chunk in.)\n\nimport { IndexedDBStore } from \"./indexeddb.js\";\nimport { MemoryStore } from \"./memory.js\";\nimport { CACHE_SCHEMA_VERSION, type CacheStore } from \"./types.js\";\nimport { versionedCacheStore } from \"./versionedCacheStore.js\";\n\nexport type { CacheStore, CacheSetOptions, CacheEntry } from \"./types.js\";\nexport { lockKeyFor } from \"./types.js\";\nexport { MemoryStore } from \"./memory.js\";\nexport { IndexedDBStore, DB_NAME as INDEXEDDB_DB_NAME } from \"./indexeddb.js\";\nexport type { IndexedDBStoreOptions } from \"./indexeddb.js\";\nexport {\n  shouldSkipCacheForCurrentLstMonth,\n  shouldSkipCacheForCurrentLstYear,\n  isLiveSource,\n  isWithinVolatileWindow,\n  isWritableMonth,\n  isWritableYear,\n} from \"./skip-rules.js\";\nexport { cacheKeyForObservations, cacheKeyForClimate } from \"./keys.js\";\n// Phase 21 21-03 fix-iter-2 (ts-architect CRITICAL): browser entry must\n// also re-export the version adapter + canonical version constant so\n// downstream code that imports them via this path resolves to the same\n// symbols as the Node entry.\nexport {\n  versionedCacheStore,\n  CACHE_SCHEMA_VERSION as VERSIONED_CACHE_SCHEMA_VERSION,\n} from \"./versionedCacheStore.js\";\nexport { CACHE_SCHEMA_VERSION } from \"./types.js\";\n\n/**\n * Browser/MV3 variant of {@link defaultCacheStore}. Auto-detects the best\n * available CacheStore in a browser/edge environment:\n *\n *   1. IndexedDB present → {@link IndexedDBStore}\n *   2. else → {@link MemoryStore}\n *\n * Phase 21 21-03 fix-iter-2 (ts-architect CRITICAL): the concrete store\n * is wrapped in `versionedCacheStore(CACHE_SCHEMA_VERSION)` so pre-\n * Phase-18 entries silently miss. The Node entry already does this; the\n * browser entry was missed in iter-1 — browser/MV3 consumers were still\n * served stale cache rows. Fix is symmetric.\n *\n * Returns a NEW instance per call.\n *\n * Iter-8 H15: kept async so the signature matches the Node entry's\n * `defaultCacheStore` (which awaits a dynamic import). Callers that\n * `await defaultCacheStore()` work unchanged when the package.json\n * conditional exports flip them between entries.\n */\nexport async function defaultCacheStore(): Promise<CacheStore> {\n  const inner = typeof indexedDB !== \"undefined\" ? new IndexedDBStore() : new MemoryStore();\n  return versionedCacheStore(inner, CACHE_SCHEMA_VERSION);\n}\n"],"mappings":";;;;;;;;;;;;;;;;;;AA2EA,eAAsB,oBAAyC;AAC7D,QAAM,QAAQ,OAAO,cAAc,cAAc,IAAI,eAAe,IAAI,IAAI,YAAY;AACxF,SAAO,oBAAoB,OAAO,oBAAoB;AACxD;","names":[]}

package/dist/internal/cache/index.cjs

@@ -223,9 +223,11 @@ var init_fs = __esm({
 // src/internal/cache/index.ts
 var cache_exports = {};
 __export(cache_exports, {
+  CACHE_SCHEMA_VERSION: () => CACHE_SCHEMA_VERSION,
   INDEXEDDB_DB_NAME: () => DB_NAME,
   IndexedDBStore: () => IndexedDBStore,
   MemoryStore: () => MemoryStore,
+  VERSIONED_CACHE_SCHEMA_VERSION: () => CACHE_SCHEMA_VERSION,
   cacheKeyForClimate: () => cacheKeyForClimate,
   cacheKeyForObservations: () => cacheKeyForObservations,
   defaultCacheStore: () => defaultCacheStore,
@@ -235,7 +237,8 @@ __export(cache_exports, {
   isWritableYear: () => isWritableYear,
   lockKeyFor: () => lockKeyFor,
   shouldSkipCacheForCurrentLstMonth: () => shouldSkipCacheForCurrentLstMonth,
-  shouldSkipCacheForCurrentLstYear: () => shouldSkipCacheForCurrentLstYear
+  shouldSkipCacheForCurrentLstYear: () => shouldSkipCacheForCurrentLstYear,
+  versionedCacheStore: () => versionedCacheStore
 });
 module.exports = __toCommonJS(cache_exports);
 
@@ -243,6 +246,7 @@ module.exports = __toCommonJS(cache_exports);
 function lockKeyFor(key) {
   return `mostlyright:cache:lock:${key}`;
 }
+var CACHE_SCHEMA_VERSION = "v2-phase18-integer-f";
 
 // src/internal/cache/memory.ts
 var MemoryStore = class {
@@ -395,8 +399,78 @@ var IndexedDBStore = class {
   }
 };
 
+// src/internal/cache/versionedCacheStore.ts
+var VERSION_FIELD = "_cache_schema_version";
+function isVersionedEntry(v) {
+  if (v === null || typeof v !== "object") return false;
+  if (!(VERSION_FIELD in v)) return false;
+  return typeof v[VERSION_FIELD] === "string";
+}
+function hasListKeys(s) {
+  return typeof s.listKeys === "function";
+}
+var VersionedCacheStore = class {
+  #inner;
+  #version;
+  constructor(inner, version) {
+    if (typeof version !== "string" || version.length === 0) {
+      throw new TypeError("versionedCacheStore: version must be a non-empty string");
+    }
+    this.#inner = inner;
+    this.#version = version;
+  }
+  /**
+   * Test/diagnostics seam: return the underlying store so tests can assert
+   * which concrete backend `defaultCacheStore()` selected. NOT a production
+   * API — production code MUST use the wrapped store so version
+   * invalidation fires on stale reads.
+   *
+   * @internal
+   */
+  __peekInner() {
+    return this.#inner;
+  }
+  async get(key) {
+    const raw = await this.#inner.get(key);
+    if (raw === null) return null;
+    if (!isVersionedEntry(raw)) {
+      return null;
+    }
+    if (raw._cache_schema_version !== this.#version) {
+      return null;
+    }
+    return raw.value;
+  }
+  async set(key, value, opts) {
+    const wrapped = {
+      value,
+      [VERSION_FIELD]: this.#version
+    };
+    await this.#inner.set(key, wrapped, opts);
+  }
+  async delete(key) {
+    await this.#inner.delete(key);
+  }
+  async withLock(key, fn) {
+    return this.#inner.withLock(key, fn);
+  }
+  async listKeys(prefix) {
+    if (hasListKeys(this.#inner)) {
+      return this.#inner.listKeys(prefix);
+    }
+    return Object.freeze([]);
+  }
+};
+function versionedCacheStore(inner, version) {
+  return new VersionedCacheStore(inner, version);
+}
+
 // src/internal/cache/default.ts
 async function defaultCacheStore() {
+  const inner = await pickConcreteStore();
+  return versionedCacheStore(inner, CACHE_SCHEMA_VERSION);
+}
+async function pickConcreteStore() {
   if (typeof indexedDB !== "undefined") return new IndexedDBStore();
   if (typeof process !== "undefined" && typeof process.versions === "object" && process.versions !== null && typeof process.versions.node === "string") {
     const { FsStore: FsStore2 } = await Promise.resolve().then(() => (init_fs(), fs_exports));
@@ -1372,9 +1446,11 @@ function cacheKeyForClimate(station, year) {
 }
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
+  CACHE_SCHEMA_VERSION,
   INDEXEDDB_DB_NAME,
   IndexedDBStore,
   MemoryStore,
+  VERSIONED_CACHE_SCHEMA_VERSION,
   cacheKeyForClimate,
   cacheKeyForObservations,
   defaultCacheStore,
@@ -1384,6 +1460,7 @@ function cacheKeyForClimate(station, year) {
   isWritableYear,
   lockKeyFor,
   shouldSkipCacheForCurrentLstMonth,
-  shouldSkipCacheForCurrentLstYear
+  shouldSkipCacheForCurrentLstYear,
+  versionedCacheStore
 });
 //# sourceMappingURL=index.cjs.map