Merge pull request #26 from lfantone/main

fix(network-first): convert headers to an object first

src/strategy/networkFirst.ts

@@ -36,7 +36,11 @@ export class NetworkFirst extends CacheStrategy {
 
  // Safari throws an error if we try to mutate the headers directly
  const newResponse = new Response(body, {
- headers: { ...headers, 'X-Remix-Worker': 'yes' },
+ headers: {
+ // Headers are a Map so we need to convert to an object first
+ ...Object.fromEntries(headers.entries()),
+ 'X-Remix-Worker': 'yes'
+ },
  status: cachedResponse.status,
  statusText: cachedResponse.statusText
  });

tsconfig.json

@@ -1,14 +1,14 @@
 {
  "compilerOptions": {
  "incremental": true,
- "target": "es2016", 
+ "target": "es2016",
  "useDefineForClassFields": true,
  "moduleResolution": "node",
- "resolveJsonModule": true, 
+ "resolveJsonModule": true,
  "allowJs": true,
  "module": "ES2022",
  "outDir": "./lib",
- "lib": ["ES2022", "DOM", "WebWorker"],
+ "lib": ["ES2022", "DOM", "WebWorker", "DOM.Iterable"],
  "checkJs": true,
  "declaration": true,
  "declarationMap": true,
@@ -17,11 +17,11 @@
  "esModuleInterop": true,
  "forceConsistentCasingInFileNames": true,
  "strict": true,
- "noUnusedParameters": true, 
+ "noUnusedParameters": true,
  "noImplicitReturns": true,
  "noFallthroughCasesInSwitch": true,
  "noImplicitOverride": true,
- "skipLibCheck": true 
+ "skipLibCheck": true
  },
  "include": ["src/**/*"],
  "exclude": ["node_modules", "lib", "types"]