fix(types): Remove usage of allowSyntheticDefaultImports (#11073)

The TypeScript docs say
> `allowSyntheticDefaultImports` without `esModuleInterop` should be
avoided. It changes the compiler’s checking behavior without changing
the code emitted by tsc, allowing potentially unsafe JavaScript to be
emitted. Additionally, the checking changes it introduces are an
incomplete version of the ones introduced by `esModuleInterop`.

Author: timfish

dev-packages/browser-integration-tests/tsconfig.json

@@ -5,7 +5,8 @@
     "lib": ["dom", "es2019"],
     "moduleResolution": "node",
     "noEmit": true,
-    "strict": true
+    "strict": true,
+    "allowSyntheticDefaultImports": true
   },
   "include": ["**/*.ts"],
   "exclude": ["node_modules"]

packages/nextjs/src/config/loaders/types.ts

@@ -1,4 +1,4 @@
-import type webpack from 'webpack';
+import type * as webpack from 'webpack';
 
 export type LoaderThis<Options> = {
   /**

packages/node/src/proxy/index.ts

@@ -28,7 +28,6 @@
 
 /* eslint-disable @typescript-eslint/explicit-member-accessibility */
 /* eslint-disable @typescript-eslint/no-unused-vars */
-import assert from 'assert';
 import type * as http from 'http';
 import type { OutgoingHttpHeaders } from 'http';
 import * as net from 'net';
@@ -190,8 +189,6 @@ export class HttpsProxyAgent<Uri extends string> extends Agent {
     // Need to wait for the "socket" event to re-play the "data" events.
     req.once('socket', (s: net.Socket) => {
       debug('Replaying proxy buffer for failed request');
-      assert(s.listenerCount('data') > 0);
-
       // Replay the "buffered" Buffer onto the fake `socket`, since at
       // this point the HTTP module machinery has been hooked up for
       // the user.

tsconfig.json

@@ -2,7 +2,6 @@
   "extends": "./packages/typescript/tsconfig.json",
 
   "compilerOptions": {
-    "allowSyntheticDefaultImports": true,
     "declaration": false,
     "declarationMap": false,
     "skipLibCheck": true,