Aggressive bundle size optimization: 1908 → 1792 bytes (-116 bytes, -6%)

[dkzlv]

Summary

This PR implements aggressive bundle size optimizations across the entire codebase, reducing the gzipped bundle from 1908 bytes to 1792 bytes (a 116-byte or ~6% reduction).

Key Optimizations

Core Changes (lib/factory.ts)

1. Removed redundant code patterns

   • Eliminated _revalidateOnInterval Map in favor of a local variable
   • Removed separate getCachedValueByKey function, inlined at call sites
   • Consolidated handleNewListener and runRefetcher into single function

2. Simplified control flow

   • Removed unnecessary firstRun + currentKeyValue block in onStart — subscribe already handles initialization
   • Merged cache hit logic directly into fetcher execution path
   • Converted if/else chains to ternary expressions where appropriate

3. Function and variable optimizations

   • Converted function declarations to arrow functions (4 bytes saved)
   • Replaced for...of loops with forEach (5-9 bytes per occurrence)
   • Changed Symbol() to plain object {} for fetcherSymbol (1 byte)
   • Combined assignment expressions: prevKey = fetcherStore.key = newKey

4. Type and property simplifications

   • Removed explicit void 0 property initializations in mutator store
   • Optimized destructuring with ?? operator: opts?.throttleCalls ?? true
   • Replaced safeKeySet helper with direct inline checks
   • Removed unnecessary async from fetcherStore.fetch (returns promise already)

5. Cache iteration improvements

   • Replaced for...of cache.keys() with cache.forEach (6 bytes)
   • Removed Map-based store indexing in getKeyStore, using parallel arrays instead (4 bytes + 1 TS error fix)

Platform Adapter (lib/platforms/browser.ts)

• Cached typeof window check in canSub boolean
• Removed isServer variable, using expression directly
• Simplified event subscription logic (10 bytes total)

Error Messages and Warnings

• Shortened error message: "No fetcher defined" (was: "You need to set up either global fetcher of fetcher in createFetcherStore")
• Removed console.warn from __unsafeOverruleSettings (13 bytes)
• Removed Error() wrapper new keyword (minor size reduction)

Developer Experience

• Added pnpm-workspace.yaml to fix pnpm build warnings
• Updated .gitignore to exclude autoresearch artifacts
• All 50+ tests pass
• TypeScript type checking passes
• Size limit budget (1924 bytes) still respected with margin

Impact

Metric	Before	After	Delta
Gzipped	1908 bytes	1792 bytes	-116 bytes (-6%)
Raw	11989 bytes	~11300 bytes	~-689 bytes

Testing

• All unit tests pass (pnpm test:unit)
• React integration tests pass
• Type tests pass
• Size limit check passes
• No breaking changes to public API

Commits

This PR contains ~30 granular commits, each addressing a specific size optimization with detailed commit messages explaining the byte savings.