Skip to content

Commit 3854370

Browse files
committed
chore: rename again
1 parent 5215688 commit 3854370

File tree

5 files changed

+11
-11
lines changed

5 files changed

+11
-11
lines changed

plugin/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353
"publish:pull": "git pull",
5454
"publish:install": "npm ci",
5555
"publish:test": "cd .. && npm ci && npm test",
56-
"clean": "rimraf lib",
56+
"clean": "rimraf lib dist-types",
5757
"build": "tsc",
5858
"watch": "tsc --watch",
5959
"prepare": "npm run build"

plugin/src/middleware/index.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
export * from './next-origin-response'
2-
export * from './middleware-request'
1+
export * from './response'
2+
export * from './request'
33
export * from './html-rewriter'

plugin/src/middleware/middleware-request.ts renamed to plugin/src/middleware/request.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { NextURL } from 'next/dist/server/web/next-url'
33
import { NextResponse } from 'next/server'
44
import type { NextRequest } from 'next/server'
55

6-
import { NextOriginResponse } from './next-origin-response'
6+
import { MiddlewareResponse } from './response'
77

88
// TODO: add Context type
99
type Context = {
@@ -44,10 +44,10 @@ export class MiddlewareRequest {
4444
})
4545
}
4646

47-
async next(): Promise<NextOriginResponse> {
47+
async next(): Promise<MiddlewareResponse> {
4848
this.applyHeaders()
4949
const response = await this.context.next()
50-
return new NextOriginResponse(response)
50+
return new MiddlewareResponse(response)
5151
}
5252

5353
rewrite(destination: string | URL | NextURL, init?: ResponseInit): NextResponse {

plugin/src/middleware/next-origin-response.ts renamed to plugin/src/middleware/response.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export type NextDataTransform = <T extends Record<string, any>>(props: T) => T
77

88
// A NextReponse that wraps the Netlify origin response
99
// We can't pass it through directly, because Next disallows returning a response body
10-
export class NextOriginResponse extends NextResponse {
10+
export class MiddlewareResponse extends NextResponse {
1111
private readonly dataTransforms: NextDataTransform[]
1212
private readonly elementHandlers: Array<[selector: string, handlers: ElementHandlers]>
1313
constructor(public originResponse: Response) {

plugin/src/templates/edge/utils.ts

+4-4
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ export const addMiddlewareHeaders = async (
3737
return response
3838
}
3939

40-
interface NextOriginResponse extends Response {
40+
interface MiddlewareResponse extends Response {
4141
originResponse: Response
4242
dataTransforms: NextDataTransform[]
4343
elementHandlers: Array<[selector: string, handlers: ElementHandlers]>
@@ -47,15 +47,15 @@ interface MiddlewareRequest {
4747
request: Request
4848
context: Context
4949
originalRequest: Request
50-
next(): Promise<NextOriginResponse>
50+
next(): Promise<MiddlewareResponse>
5151
rewrite(destination: string | URL, init?: ResponseInit): Response
5252
}
5353

5454
function isMiddlewareRequest(response: Response | MiddlewareRequest): response is MiddlewareRequest {
5555
return 'originalRequest' in response
5656
}
5757

58-
function isNextOriginResponse(response: Response | NextOriginResponse): response is NextOriginResponse {
58+
function isMiddlewareResponse(response: Response | MiddlewareResponse): response is MiddlewareResponse {
5959
return 'dataTransforms' in response
6060
}
6161

@@ -72,7 +72,7 @@ export const buildResponse = async ({
7272
if (isMiddlewareRequest(result.response)) {
7373
result.response = await result.response.next()
7474
}
75-
if (isNextOriginResponse(result.response)) {
75+
if (isMiddlewareResponse(result.response)) {
7676
const { response } = result
7777
// If it's JSON we don't need to use the rewriter, we can just parse it
7878
if (response.originResponse.headers.get('content-type')?.includes('application/json')) {

0 commit comments

Comments
 (0)