Skip to content

Commit 8db8d9d

Browse files
authored
refactor: move web stuff into their own folder (nodejs#2793)
Refs: nodejs#2732
1 parent fdbc221 commit 8db8d9d

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

83 files changed

+109
-109
lines changed

benchmarks/cacheGetFieldValues.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { bench, group, run } from 'mitata'
2-
import { getFieldValues } from '../lib/cache/util.js'
2+
import { getFieldValues } from '../lib/web/cache/util.js'
33

44
const values = [
55
'',

benchmarks/headers-length32.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { bench, run } from 'mitata'
2-
import { Headers } from '../lib/fetch/headers.js'
2+
import { Headers } from '../lib/web/fetch/headers.js'
33

44
const headers = new Headers(
55
[

benchmarks/headers.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { bench, group, run } from 'mitata'
2-
import { Headers } from '../lib/fetch/headers.js'
2+
import { Headers } from '../lib/web/fetch/headers.js'
33

44
const characters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789'
55
const charactersLength = characters.length

benchmarks/sort.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { bench, group, run } from 'mitata'
2-
import { sort, heapSort, introSort } from '../lib/fetch/sort.js'
2+
import { sort, heapSort, introSort } from '../lib/web/fetch/sort.js'
33

44
function compare (a, b) {
55
return a < b ? -1 : 1

index-fetch.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict'
22

3-
const fetchImpl = require('./lib/fetch').fetch
3+
const fetchImpl = require('./lib/web/fetch').fetch
44

55
module.exports.fetch = function fetch (resource, init = undefined) {
66
return fetchImpl(resource, init).catch((err) => {
@@ -10,12 +10,12 @@ module.exports.fetch = function fetch (resource, init = undefined) {
1010
throw err
1111
})
1212
}
13-
module.exports.FormData = require('./lib/fetch/formdata').FormData
14-
module.exports.Headers = require('./lib/fetch/headers').Headers
15-
module.exports.Response = require('./lib/fetch/response').Response
16-
module.exports.Request = require('./lib/fetch/request').Request
13+
module.exports.FormData = require('./lib/web/fetch/formdata').FormData
14+
module.exports.Headers = require('./lib/web/fetch/headers').Headers
15+
module.exports.Response = require('./lib/web/fetch/response').Response
16+
module.exports.Request = require('./lib/web/fetch/request').Request
1717

18-
module.exports.WebSocket = require('./lib/websocket/websocket').WebSocket
19-
module.exports.MessageEvent = require('./lib/websocket/events').MessageEvent
18+
module.exports.WebSocket = require('./lib/web/websocket/websocket').WebSocket
19+
module.exports.MessageEvent = require('./lib/web/websocket/events').MessageEvent
2020

21-
module.exports.EventSource = require('./lib/eventsource/eventsource').EventSource
21+
module.exports.EventSource = require('./lib/web/eventsource/eventsource').EventSource

index.js

+15-15
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ function makeDispatcher (fn) {
9696
module.exports.setGlobalDispatcher = setGlobalDispatcher
9797
module.exports.getGlobalDispatcher = getGlobalDispatcher
9898

99-
const fetchImpl = require('./lib/fetch').fetch
99+
const fetchImpl = require('./lib/web/fetch').fetch
100100
module.exports.fetch = async function fetch (init, options = undefined) {
101101
try {
102102
return await fetchImpl(init, options)
@@ -108,39 +108,39 @@ module.exports.fetch = async function fetch (init, options = undefined) {
108108
throw err
109109
}
110110
}
111-
module.exports.Headers = require('./lib/fetch/headers').Headers
112-
module.exports.Response = require('./lib/fetch/response').Response
113-
module.exports.Request = require('./lib/fetch/request').Request
114-
module.exports.FormData = require('./lib/fetch/formdata').FormData
115-
module.exports.File = require('./lib/fetch/file').File
116-
module.exports.FileReader = require('./lib/fileapi/filereader').FileReader
111+
module.exports.Headers = require('./lib/web/fetch/headers').Headers
112+
module.exports.Response = require('./lib/web/fetch/response').Response
113+
module.exports.Request = require('./lib/web/fetch/request').Request
114+
module.exports.FormData = require('./lib/web/fetch/formdata').FormData
115+
module.exports.File = require('./lib/web/fetch/file').File
116+
module.exports.FileReader = require('./lib/web/fileapi/filereader').FileReader
117117

118-
const { setGlobalOrigin, getGlobalOrigin } = require('./lib/fetch/global')
118+
const { setGlobalOrigin, getGlobalOrigin } = require('./lib/web/fetch/global')
119119

120120
module.exports.setGlobalOrigin = setGlobalOrigin
121121
module.exports.getGlobalOrigin = getGlobalOrigin
122122

123-
const { CacheStorage } = require('./lib/cache/cachestorage')
124-
const { kConstruct } = require('./lib/cache/symbols')
123+
const { CacheStorage } = require('./lib/web/cache/cachestorage')
124+
const { kConstruct } = require('./lib/web/cache/symbols')
125125

126126
// Cache & CacheStorage are tightly coupled with fetch. Even if it may run
127127
// in an older version of Node, it doesn't have any use without fetch.
128128
module.exports.caches = new CacheStorage(kConstruct)
129129

130-
const { deleteCookie, getCookies, getSetCookies, setCookie } = require('./lib/cookies')
130+
const { deleteCookie, getCookies, getSetCookies, setCookie } = require('./lib/web/cookies')
131131

132132
module.exports.deleteCookie = deleteCookie
133133
module.exports.getCookies = getCookies
134134
module.exports.getSetCookies = getSetCookies
135135
module.exports.setCookie = setCookie
136136

137-
const { parseMIMEType, serializeAMimeType } = require('./lib/fetch/dataURL')
137+
const { parseMIMEType, serializeAMimeType } = require('./lib/web/fetch/dataURL')
138138

139139
module.exports.parseMIMEType = parseMIMEType
140140
module.exports.serializeAMimeType = serializeAMimeType
141141

142-
const { CloseEvent, ErrorEvent, MessageEvent } = require('./lib/websocket/events')
143-
module.exports.WebSocket = require('./lib/websocket/websocket').WebSocket
142+
const { CloseEvent, ErrorEvent, MessageEvent } = require('./lib/web/websocket/events')
143+
module.exports.WebSocket = require('./lib/web/websocket/websocket').WebSocket
144144
module.exports.CloseEvent = CloseEvent
145145
module.exports.ErrorEvent = ErrorEvent
146146
module.exports.MessageEvent = MessageEvent
@@ -156,6 +156,6 @@ module.exports.MockPool = MockPool
156156
module.exports.MockAgent = MockAgent
157157
module.exports.mockErrors = mockErrors
158158

159-
const { EventSource } = require('./lib/eventsource/eventsource')
159+
const { EventSource } = require('./lib/web/eventsource/eventsource')
160160

161161
module.exports.EventSource = EventSource

lib/cache/symbols.js

-5
This file was deleted.

lib/client.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1500,7 +1500,7 @@ function write (client, request) {
15001500

15011501
if (util.isFormDataLike(body)) {
15021502
if (!extractBody) {
1503-
extractBody = require('./fetch/body.js').extractBody
1503+
extractBody = require('./web/fetch/body.js').extractBody
15041504
}
15051505

15061506
const [bodyStream, contentType] = extractBody(body)

lib/mock/mock-interceptor.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ class MockInterceptor {
7474
if (opts.query) {
7575
opts.path = buildURL(opts.path, opts.query)
7676
} else {
77-
// Matches https://github.com/nodejs/undici/blob/main/lib/fetch/index.js#L1811
77+
// Matches https://github.com/nodejs/undici/blob/main/lib/web/fetch/index.js#L1811
7878
const parsedURL = new URL(opts.path, 'data://')
7979
opts.path = parsedURL.pathname + parsedURL.search
8080
}

lib/cache/cache.js renamed to lib/web/cache/cache.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,15 @@
22

33
const { kConstruct } = require('./symbols')
44
const { urlEquals, getFieldValues } = require('./util')
5-
const { kEnumerableProperty, isDisturbed } = require('../core/util')
5+
const { kEnumerableProperty, isDisturbed } = require('../../core/util')
66
const { webidl } = require('../fetch/webidl')
77
const { Response, cloneResponse, fromInnerResponse } = require('../fetch/response')
88
const { Request, fromInnerRequest } = require('../fetch/request')
99
const { kState } = require('../fetch/symbols')
1010
const { fetching } = require('../fetch/index')
1111
const { urlIsHttpHttpsScheme, createDeferredPromise, readAllBytes } = require('../fetch/util')
1212
const assert = require('node:assert')
13-
const { getGlobalDispatcher } = require('../global')
13+
const { getGlobalDispatcher } = require('../../global')
1414

1515
/**
1616
* @see https://w3c.github.io/ServiceWorker/#dfn-cache-batch-operation

lib/cache/cachestorage.js renamed to lib/web/cache/cachestorage.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
const { kConstruct } = require('./symbols')
44
const { Cache } = require('./cache')
55
const { webidl } = require('../fetch/webidl')
6-
const { kEnumerableProperty } = require('../core/util')
6+
const { kEnumerableProperty } = require('../../core/util')
77

88
class CacheStorage {
99
/**

lib/web/cache/symbols.js

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
'use strict'
2+
3+
module.exports = {
4+
kConstruct: require('../../core/symbols').kConstruct
5+
}
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

lib/cookies/util.js renamed to lib/web/cookies/util.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict'
22

33
const assert = require('node:assert')
4-
const { kHeadersList } = require('../core/symbols')
4+
const { kHeadersList } = require('../../core/symbols')
55

66
function isCTLExcludingHtab (value) {
77
if (value.length === 0) {

lib/eventsource/eventsource.js renamed to lib/web/eventsource/eventsource.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const { EventSourceStream } = require('./eventsource-stream')
99
const { parseMIMEType } = require('../fetch/dataURL')
1010
const { MessageEvent } = require('../websocket/events')
1111
const { isNetworkError } = require('../fetch/response')
12-
const { getGlobalDispatcher } = require('../global')
12+
const { getGlobalDispatcher } = require('../../global')
1313
const { delay } = require('./util')
1414

1515
let experimentalWarned = false
File renamed without changes.
File renamed without changes.

lib/fetch/body.js renamed to lib/web/fetch/body.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict'
22

33
const Busboy = require('@fastify/busboy')
4-
const util = require('../core/util')
4+
const util = require('../../core/util')
55
const {
66
ReadableStreamFrom,
77
isBlobLike,
@@ -16,7 +16,7 @@ const { kState } = require('./symbols')
1616
const { webidl } = require('./webidl')
1717
const { Blob, File: NativeFile } = require('node:buffer')
1818
const assert = require('node:assert')
19-
const { isErrored } = require('../core/util')
19+
const { isErrored } = require('../../core/util')
2020
const { isArrayBuffer } = require('node:util/types')
2121
const { File: UndiciFile } = require('./file')
2222
const { serializeAMimeType } = require('./dataURL')
File renamed without changes.
File renamed without changes.

lib/compat/dispatcher-weakref.js renamed to lib/web/fetch/dispatcher-weakref.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
'use strict'
22

3-
const { kConnected, kSize } = require('../core/symbols')
3+
const { kConnected, kSize } = require('../../core/symbols')
44

55
class CompatWeakRef {
66
constructor (value) {

lib/fetch/file.js renamed to lib/web/fetch/file.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const { kState } = require('./symbols')
66
const { isBlobLike } = require('./util')
77
const { webidl } = require('./webidl')
88
const { parseMIMEType, serializeAMimeType } = require('./dataURL')
9-
const { kEnumerableProperty } = require('../core/util')
9+
const { kEnumerableProperty } = require('../../core/util')
1010
const encoder = new TextEncoder()
1111

1212
class File extends Blob {

lib/fetch/formdata.js renamed to lib/web/fetch/formdata.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
const { isBlobLike, iteratorMixin } = require('./util')
44
const { kState } = require('./symbols')
5-
const { kEnumerableProperty } = require('../core/util')
5+
const { kEnumerableProperty } = require('../../core/util')
66
const { File: UndiciFile, FileLike, isFileLike } = require('./file')
77
const { webidl } = require('./webidl')
88
const { File: NativeFile } = require('node:buffer')
File renamed without changes.

lib/fetch/headers.js renamed to lib/web/fetch/headers.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22

33
'use strict'
44

5-
const { kHeadersList, kConstruct } = require('../core/symbols')
5+
const { kHeadersList, kConstruct } = require('../../core/symbols')
66
const { kGuard } = require('./symbols')
7-
const { kEnumerableProperty } = require('../core/util')
7+
const { kEnumerableProperty } = require('../../core/util')
88
const {
99
iteratorMixin,
1010
isValidHeaderName,

lib/fetch/index.js renamed to lib/web/fetch/index.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,9 @@ const {
5959
} = require('./constants')
6060
const EE = require('node:events')
6161
const { Readable, pipeline } = require('node:stream')
62-
const { addAbortListener, isErrored, isReadable, nodeMajor, nodeMinor, bufferToLowerCasedHeaderName } = require('../core/util')
62+
const { addAbortListener, isErrored, isReadable, nodeMajor, nodeMinor, bufferToLowerCasedHeaderName } = require('../../core/util')
6363
const { dataURLProcessor, serializeAMimeType, minimizeSupportedMimeType } = require('./dataURL')
64-
const { getGlobalDispatcher } = require('../global')
64+
const { getGlobalDispatcher } = require('../../global')
6565
const { webidl } = require('./webidl')
6666
const { STATUS_CODES } = require('node:http')
6767
const GET_OR_HEAD = ['GET', 'HEAD']

lib/fetch/request.js renamed to lib/web/fetch/request.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44

55
const { extractBody, mixinBody, cloneBody } = require('./body')
66
const { Headers, fill: fillHeaders, HeadersList } = require('./headers')
7-
const { FinalizationRegistry } = require('../compat/dispatcher-weakref')()
8-
const util = require('../core/util')
7+
const { FinalizationRegistry } = require('./dispatcher-weakref')()
8+
const util = require('../../core/util')
99
const {
1010
isValidHTTPToken,
1111
sameOrigin,
@@ -28,7 +28,7 @@ const { kHeaders, kSignal, kState, kGuard, kRealm } = require('./symbols')
2828
const { webidl } = require('./webidl')
2929
const { getGlobalOrigin } = require('./global')
3030
const { URLSerializer } = require('./dataURL')
31-
const { kHeadersList, kConstruct } = require('../core/symbols')
31+
const { kHeadersList, kConstruct } = require('../../core/symbols')
3232
const assert = require('node:assert')
3333
const { getMaxListeners, setMaxListeners, getEventListeners, defaultMaxListeners } = require('node:events')
3434

lib/fetch/response.js renamed to lib/web/fetch/response.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
const { Headers, HeadersList, fill } = require('./headers')
44
const { extractBody, cloneBody, mixinBody } = require('./body')
5-
const util = require('../core/util')
5+
const util = require('../../core/util')
66
const { kEnumerableProperty } = util
77
const {
88
isValidReasonPhrase,
@@ -22,7 +22,7 @@ const { webidl } = require('./webidl')
2222
const { FormData } = require('./formdata')
2323
const { getGlobalOrigin } = require('./global')
2424
const { URLSerializer } = require('./dataURL')
25-
const { kHeadersList, kConstruct } = require('../core/symbols')
25+
const { kHeadersList, kConstruct } = require('../../core/symbols')
2626
const assert = require('node:assert')
2727
const { types } = require('node:util')
2828

File renamed without changes.
File renamed without changes.

lib/fetch/util.js renamed to lib/web/fetch/util.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const { redirectStatusSet, referrerPolicySet: referrerPolicyTokens, badPortsSet
66
const { getGlobalOrigin } = require('./global')
77
const { collectASequenceOfCodePoints, collectAnHTTPQuotedString, removeChars, parseMIMEType } = require('./dataURL')
88
const { performance } = require('node:perf_hooks')
9-
const { isBlobLike, ReadableStreamFrom, isValidHTTPToken } = require('../core/util')
9+
const { isBlobLike, ReadableStreamFrom, isValidHTTPToken } = require('../../core/util')
1010
const assert = require('node:assert')
1111
const { isUint8Array } = require('node:util/types')
1212
const { webidl } = require('./webidl')

lib/fetch/webidl.js renamed to lib/web/fetch/webidl.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict'
22

33
const { types } = require('node:util')
4-
const { toUSVString } = require('../core/util')
4+
const { toUSVString } = require('../../core/util')
55

66
/** @type {import('../../types/webidl').Webidl} */
77
const webidl = {}
File renamed without changes.

lib/fileapi/filereader.js renamed to lib/web/fileapi/filereader.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ const {
1313
kAborted
1414
} = require('./symbols')
1515
const { webidl } = require('../fetch/webidl')
16-
const { kEnumerableProperty } = require('../core/util')
16+
const { kEnumerableProperty } = require('../../core/util')
1717

1818
class FileReader extends EventTarget {
1919
constructor () {
File renamed without changes.
File renamed without changes.
File renamed without changes.

lib/websocket/connection.js renamed to lib/web/websocket/connection.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@ const {
88
kReceivedClose
99
} = require('./symbols')
1010
const { fireEvent, failWebsocketConnection } = require('./util')
11-
const { channels } = require('../core/diagnostics')
11+
const { channels } = require('../../core/diagnostics')
1212
const { CloseEvent } = require('./events')
1313
const { makeRequest } = require('../fetch/request')
1414
const { fetching } = require('../fetch/index')
1515
const { Headers } = require('../fetch/headers')
16-
const { getGlobalDispatcher } = require('../global')
17-
const { kHeadersList } = require('../core/symbols')
16+
const { getGlobalDispatcher } = require('../../global')
17+
const { kHeadersList } = require('../../core/symbols')
1818

1919
/** @type {import('crypto')} */
2020
let crypto
File renamed without changes.

lib/websocket/events.js renamed to lib/web/websocket/events.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict'
22

33
const { webidl } = require('../fetch/webidl')
4-
const { kEnumerableProperty } = require('../core/util')
4+
const { kEnumerableProperty } = require('../../core/util')
55
const { MessagePort } = require('node:worker_threads')
66

77
/**
File renamed without changes.

lib/websocket/receiver.js renamed to lib/web/websocket/receiver.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
const { Writable } = require('node:stream')
44
const { parserStates, opcodes, states, emptyBuffer } = require('./constants')
55
const { kReadyState, kSentClose, kResponse, kReceivedClose } = require('./symbols')
6-
const { channels } = require('../core/diagnostics')
6+
const { channels } = require('../../core/diagnostics')
77
const { isValidStatusCode, failWebsocketConnection, websocketMessageReceived } = require('./util')
88
const { WebsocketFrameSend } = require('./frame')
99

File renamed without changes.
File renamed without changes.

lib/websocket/websocket.js renamed to lib/web/websocket/websocket.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ const { isEstablished, isClosing, isValidSubprotocol, failWebsocketConnection, f
1717
const { establishWebSocketConnection } = require('./connection')
1818
const { WebsocketFrameSend } = require('./frame')
1919
const { ByteParser } = require('./receiver')
20-
const { kEnumerableProperty, isBlobLike } = require('../core/util')
21-
const { getGlobalDispatcher } = require('../global')
20+
const { kEnumerableProperty, isBlobLike } = require('../../core/util')
21+
const { getGlobalDispatcher } = require('../../global')
2222
const { types } = require('node:util')
2323

2424
let experimentalWarned = false

test/cache/get-field-values.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
const { deepStrictEqual, throws } = require('node:assert')
44
const { test } = require('node:test')
5-
const { getFieldValues } = require('../../lib/cache/util')
5+
const { getFieldValues } = require('../../lib/web/cache/util')
66

77
test('getFieldValues', () => {
88
throws(() => getFieldValues(null), {

test/cookie/global-headers.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ const {
88
getSetCookies,
99
setCookie
1010
} = require('../..')
11-
const { getHeadersList } = require('../../lib/cookies/util')
11+
const { getHeadersList } = require('../../lib/web/cookies/util')
1212

1313
describe('Using global Headers', async () => {
1414
test('deleteCookies', () => {

0 commit comments

Comments
 (0)