Revert "chore: use isbot version 4" (#8414)

remix-run · Jan 2, 2024 · 9748fde · 9748fde
1 parent b12b69a
commit 9748fde
Show file tree

Hide file tree

Showing 11 changed files with 15 additions and 15 deletions.
diff --git a/integration/defer-test.ts b/integration/defer-test.ts
@@ -985,7 +985,7 @@ test.describe("aborted", () => {
           import type { AppLoadContext, EntryContext } from "@remix-run/node";
           import { createReadableStreamFromReadable } from "@remix-run/node";
           import { RemixServer } from "@remix-run/react";
-          import { isbot } from "isbot";
+          import isbot from "isbot";
           import { renderToPipeableStream } from "react-dom/server";
 
           const ABORT_DELAY = 1;
@@ -997,7 +997,7 @@ test.describe("aborted", () => {
             remixContext: EntryContext,
             loadContext: AppLoadContext
           ) {
-            return isbot(request.headers.get("user-agent") || "")
+            return isbot(request.headers.get("user-agent"))
               ? handleBotRequest(
                   request,
                   responseStatusCode,

diff --git a/integration/helpers/vite-template/package.json b/integration/helpers/vite-template/package.json
@@ -12,7 +12,7 @@
     "@remix-run/node": "*",
     "@remix-run/react": "*",
     "@remix-run/serve": "*",
-    "isbot": "^4.1.0",
+    "isbot": "^3.6.8",
     "react": "^18.2.0",
     "react-dom": "^18.2.0"
   },

diff --git a/integration/package.json b/integration/package.json
@@ -21,7 +21,7 @@
     "get-port": "^5.1.1",
     "glob": "8.0.3",
     "globby": "^11.1.0",
-    "isbot": "^4.1.0",
+    "isbot": "^3.6.8",
     "npm-run-all": "^4.1.5",
     "pidtree": "^0.6.0",
     "postcss": "^8.4.19",

diff --git a/packages/remix-dev/__tests__/fixtures/cloudflare/package.json b/packages/remix-dev/__tests__/fixtures/cloudflare/package.json
@@ -13,7 +13,7 @@
     "@remix-run/cloudflare-pages": "*",
     "@remix-run/css-bundle": "*",
     "@remix-run/react": "*",
-    "isbot": "^4.1.0",
+    "isbot": "^3.6.8",
     "react": "^18.2.0",
     "react-dom": "^18.2.0"
   },

diff --git a/packages/remix-dev/__tests__/fixtures/deno/.vscode/resolve_npm_imports.json b/packages/remix-dev/__tests__/fixtures/deno/.vscode/resolve_npm_imports.json
@@ -13,7 +13,7 @@
     "@remix-run/dev/server-build": "https://esm.sh/@remix-run/dev@1.16.0/server-build",
     "@remix-run/react": "https://esm.sh/@remix-run/react@1.16.0",
     "@remix-run/server-runtime": "https://esm.sh/@remix-run/server-runtime@1.16.0",
-    "isbot": "https://esm.sh/isbot@^4.1.0",
+    "isbot": "https://esm.sh/isbot@^3.6.8",
     "react": "https://esm.sh/react@^18.2.0",
     "react-dom": "https://esm.sh/react-dom@^18.2.0",
     "react-dom/client": "https://esm.sh/react-dom@^18.2.0/client",

diff --git a/packages/remix-dev/__tests__/fixtures/deno/package.json b/packages/remix-dev/__tests__/fixtures/deno/package.json
@@ -16,7 +16,7 @@
     "@remix-run/css-bundle": "*",
     "@remix-run/deno": "*",
     "@remix-run/react": "*",
-    "isbot": "^4.1.0",
+    "isbot": "^3.6.8",
     "react": "^18.2.0",
     "react-dom": "^18.2.0"
   },

diff --git a/packages/remix-dev/__tests__/fixtures/node/package.json b/packages/remix-dev/__tests__/fixtures/node/package.json
@@ -13,7 +13,7 @@
     "@remix-run/node": "*",
     "@remix-run/react": "*",
     "@remix-run/serve": "*",
-    "isbot": "^4.1.0",
+    "isbot": "^3.6.8",
     "react": "^18.2.0",
     "react-dom": "^18.2.0"
   },

diff --git a/packages/remix-dev/config.ts b/packages/remix-dev/config.ts
@@ -493,7 +493,7 @@ export async function resolveConfig(
       pkgJson.update({
         dependencies: {
           ...pkgJson.content.dependencies,
-          isbot: "^4",
+          isbot: "latest",
         },
       });
 

diff --git a/packages/remix-dev/config/defaults/entry.server.cloudflare.tsx b/packages/remix-dev/config/defaults/entry.server.cloudflare.tsx
@@ -1,6 +1,6 @@
 import type { AppLoadContext, EntryContext } from "@remix-run/cloudflare";
 import { RemixServer } from "@remix-run/react";
-import { isbot } from "isbot";
+import isbot from "isbot";
 import { renderToReadableStream } from "react-dom/server";
 
 export default async function handleRequest(
@@ -22,7 +22,7 @@ export default async function handleRequest(
     }
   );
 
-  if (isbot(request.headers.get("user-agent") || "")) {
+  if (isbot(request.headers.get("user-agent"))) {
     await body.allReady;
   }
 

diff --git a/packages/remix-dev/config/defaults/entry.server.deno.tsx b/packages/remix-dev/config/defaults/entry.server.deno.tsx
@@ -1,6 +1,6 @@
 import type { AppLoadContext, EntryContext } from "@remix-run/deno";
 import { RemixServer } from "@remix-run/react";
-import { isbot } from "isbot";
+import isbot from "isbot";
 import { renderToReadableStream } from "react-dom/server";
 
 export default async function handleRequest(
@@ -22,7 +22,7 @@ export default async function handleRequest(
     }
   );
 
-  if (isbot(request.headers.get("user-agent") || "")) {
+  if (isbot(request.headers.get("user-agent"))) {
     await body.allReady;
   }
 

diff --git a/packages/remix-dev/config/defaults/entry.server.node.tsx b/packages/remix-dev/config/defaults/entry.server.node.tsx
@@ -3,7 +3,7 @@ import { PassThrough } from "node:stream";
 import type { AppLoadContext, EntryContext } from "@remix-run/node";
 import { createReadableStreamFromReadable } from "@remix-run/node";
 import { RemixServer } from "@remix-run/react";
-import { isbot } from "isbot";
+import isbot from "isbot";
 import { renderToPipeableStream } from "react-dom/server";
 
 const ABORT_DELAY = 5_000;
@@ -15,7 +15,7 @@ export default function handleRequest(
   remixContext: EntryContext,
   loadContext: AppLoadContext
 ) {
-  return isbot(request.headers.get("user-agent") || "")
+  return isbot(request.headers.get("user-agent"))
     ? handleBotRequest(
         request,
         responseStatusCode,