refactor: remove tidewaveMiddleware in favor of explicit setup (#26)

Author: zoey

README.md

@@ -84,37 +84,22 @@ export const config = {
 **Middleware** - Then create (or modify) `middleware.ts` with:
 
 ```typescript
-import { tidewaveMiddleware } from 'tidewave/next-js';
+import { NextRequest, NextResponse } from 'next/server';
 
-export const middleware = tidewaveMiddleware();
+export function middleware(req: NextRequest): NextResponse {
+  const { pathname } = req.nextUrl;
 
-export const config = {
-  matcher: '/tidewave/(.*)',
-};
-```
+  if (pathname.startsWith('/tidewave')) {
+    return NextResponse.rewrite(new URL(`/api${pathname}`, req.url));
+  }
 
-In case you already have an existing `middleware.ts`:
-
-```typescript
-import { tidewaveMiddleware } from 'tidewave/next-js';
-
-const withTidewave = tidewaveMiddleware();
-
-export function middleware(req: NextRequest): Promise<NextResponse> {
-  // This will return a unchanged NextResponse.next()
-  // if path doesn't match with `/tidewave`
-  // if does match it will rewrite to `/api/tidewave` instead
-  withTidewave(req, (req: NextRequest) => {
-    // your own logic
-    return NextResponse.json({ message: 'hello, world!' });
-  });
+  // here you could add your own logic or different middlewares
+  // while changing the config.matcher to a string[]
+  return NextResponse.next();
 }
 
 export const config = {
-  matcher: [
-    '/tidewave/(.*)', // tidewave specific matcher
-    '/your/route/', // your specific matcher
-  ],
+  matcher: '/tidewave/(.*)',
 };
 ```
 

src/next-js.ts

@@ -1,5 +1,4 @@
 import type { NextApiRequest, NextApiResponse } from 'next';
-import { NextResponse, type NextRequest } from 'next/server';
 import { checkSecurity, HANDLERS, methodNotAllowed, type Request, type Response } from './http';
 import bodyParser from 'body-parser';
 import type { TidewaveConfig } from './core';
@@ -12,7 +11,6 @@ const DEFAULT_CONFIG: TidewaveConfig = {
 };
 
 type NextJsHandler = (_req: NextApiRequest, _res: NextApiResponse) => Promise<void>;
-type NextJsMiddleware = (_req: NextRequest) => Promise<NextResponse>;
 
 type NextHandler = () => ValueOrPromise<unknown>;
 
@@ -83,30 +81,3 @@ export async function tidewaveHandler(
     return res.status(404).json({ message: `Route not found: ${req.method} ${req.url}` });
   };
 }
-
-export function tidewaveMiddleware(): NextJsMiddleware {
-  const env = process.env.NODE_ENV;
-
-  if (!(env === 'development')) {
-    throw Error(
-      `[Tidewave] tidewave is designed to only work on development environment, got: ${env}`,
-    );
-  }
-  return async function middleware(
-    req: NextRequest,
-    next?: NextJsMiddleware,
-  ): Promise<NextResponse> {
-    const { pathname } = req.nextUrl;
-
-    if (!isTidewaveRoute(pathname) && next) return next(req);
-    if (!isTidewaveRoute(pathname)) return NextResponse.next();
-
-    // since next.js v12, middleware doesn't
-    // accept relative URLs
-    return NextResponse.rewrite(new URL(`/api${pathname}`, req.url));
-  };
-}
-
-export function isTidewaveRoute(pathname: string): boolean {
-  return pathname.startsWith('/tidewave');
-}