1. 24 7月, 2020 1 次提交
  2. 22 7月, 2020 3 次提交
  3. 21 7月, 2020 6 次提交
  4. 20 7月, 2020 6 次提交
  5. 19 7月, 2020 6 次提交
    • J
      v9.4.5-canary.38 · b1cfa51b
      Joe Haddad 提交于
      b1cfa51b
    • N
    • J
      Fix static data fetching when using absolute assetprefix (#15287) · 381e44f3
      Jan Potoms 提交于
      Fixes https://github.com/vercel/next.js/issues/15188
      
      `parseRelativeUrl` was used on urls that weren't always relative. It was used to generate a cache key, but we actually don't need these cache keys to be relative if the urls aren't relative.
      
      Also took a look at the overall static data fetching logic and found a few things:
      
      - [x] cache key is unnecessarily transformed through `prepareRoute`, we can just cache by resolved `dataHref` and remove that function. Pretty sure that `prepareRoute` was also introducing edge cases with `assetPath` and `delBasePath`
      - [x] there is [a bug in the caching logic](https://github.com/vercel/next.js/blob/ebdfa2e7a3f8e22e03b94dfb5f00481bf06254b6/packages/next/next-server/lib/router/router.ts#L898) that made it fail on the second visit: it should be `Promise.resolve(this.sdc[pathname])` instead of `Promise.resolve(this.sdc[dataHref])`. Also added a test for this
      - [x] ~converted to async await to improve stacktraces and readability.~ I assumed this was fine since I saw some async/awaits in that file already but it seems to just blow up the size of the non-modern bundle.
      - [x] extracted nested `getResponse` function and define it top level. this should improve runtime performance
      - [x] convert `_getStaticData` and `_getServerData` to class methods instead of properties. Not sure why they were defined as properties but I think they belong on the prototype instead.
      - [x] remove `cb` property from `fetchNextData`, it's unnecessary and makes the async flow hard to understand.  The exact same logic can go in the `.then` instead.
      - [ ] data fetching logic [retries on 5xx errors](https://github.com/vercel/next.js/blob/ebdfa2e7a3f8e22e03b94dfb5f00481bf06254b6/packages/next/next-server/lib/router/router.ts#L157), but not on network level errors. It should also retry on those. It should also not retry on every 5xx, probably only makes sense on 502, 503 and 504. (e.g. 500 is a server error that I wouldn't expect to succeed on a retry)
      
      The overall result also is a few bytes smaller in size
      381e44f3
    • P
      Only required polyfill.io lint rule (#15277) · 4422be32
      Prateek Bhatnagar 提交于
      - Introduces a lint rule which points towards the unwanted polyfill.io features.
      - Aim here is to make the user aware that which of the requested features are actually required vs which are already covered under `next-polyfills`
      
      Next step: If the remaining required polyfills amounts to a only a few KBs then its better to include them in 1P javascript and remove the third party render blocking script tag.
      4422be32
    • J
      Drop `module: esnext` requirement in tsconfig.json (#15276) · f5b186cb
      Joe Haddad 提交于
      Next.js forcibly setting `module: 'esnext'` in `tsconfig.json` is necessary to prevent TypeScript from erroring on the following code:
      
      ```tsx
      import dynamic from 'next/dynamic';
      
      const A = dynamic(() => import('../A'));
      ```
      
      ```
      ERROR in /Users/joe/Desktop/scratch/test-cjs/pages/index.tsx(5,25):
      5:25 Dynamic imports are only supported when the '--module' flag is set to 'es2020', 'esnext', 'commonjs', 'amd', 'system', or 'umd'.
        > 5 | const A = dynamic(() => import("../test"));
      ```
      
      However, users may want to use one of the many other targets for better interoperability with projects that co-exist with their Next.js project (like `commonjs`).
      
      When cross referenced with:
      ```
      Option '--resolveJsonModule' can only be specified when module code generation is 'commonjs', 'amd', 'es2015' or 'esNext'.ts
      ```
      
      That means we can permit any of these values:
      
      ```json5
            parsedValues: [
              ts.ModuleKind.ES2020,
              ts.ModuleKind.ESNext,
              ts.ModuleKind.CommonJS,
              ts.ModuleKind.AMD,
            ],
      ```
      
      This PR updates Next.js to allow those!
      
      ---
      
      Fixes #15275
      f5b186cb
    • T
  6. 18 7月, 2020 1 次提交
  7. 17 7月, 2020 4 次提交
  8. 16 7月, 2020 7 次提交
  9. 15 7月, 2020 5 次提交
  10. 14 7月, 2020 1 次提交