fix: various typos

GOVERNANCE.md

@@ -50,7 +50,7 @@ request or issue. The WG should serve as the final arbiter where
 required.
 
 For the current list of Collaborators, see the project
-[README.md](./README.md#collaborators). The list shall be in an
+[README.md](./README.md#collaborators). The list should be in
 alphabetical order.
 
 ### WG Membership

README.md

@@ -230,7 +230,7 @@ A body can be of the following types:
 - URLSearchParams
 - FormData
 
-In this implementation of fetch, ```request.body``` now accepts ```Async Iterables```. It is not present in the [Fetch Standard.](https://fetch.spec.whatwg.org)
+In this implementation of fetch, ```request.body``` now accepts ```Async Iterables```. It is not present in the [Fetch Standard](https://fetch.spec.whatwg.org).
 
 ```js
 import { fetch } from 'undici'
@@ -261,7 +261,7 @@ await fetch('http://example.com', { method: 'POST', body })
 
 - `'half'`
 
-In this implementation of fetch, `request.duplex` must be set if `request.body` is `ReadableStream` or `Async Iterables`, however, even though the value must be set to `'half'`, it is actually a _full_ duplex. For more detail refer to the [Fetch Standard.](https://fetch.spec.whatwg.org/#dom-requestinit-duplex).
+In this implementation of fetch, `request.duplex` must be set if `request.body` is `ReadableStream` or `Async Iterables`, however, even though the value must be set to `'half'`, it is actually a _full_ duplex. For more detail refer to the [Fetch Standard](https://fetch.spec.whatwg.org/#dom-requestinit-duplex).
 
 #### `response.body`
 

docs/docs/api/Debug.md

@@ -2,7 +2,7 @@
 
 Undici (and subsenquently `fetch` and `websocket`) exposes a debug statement that can be enabled by setting `NODE_DEBUG` within the environment.
 
-The flags availabile are:
+The flags available are:
 
 ## `undici`
 

docs/docs/api/Dispatcher.md

@@ -478,7 +478,7 @@ The `RequestOptions.method` property should not be value `'CONNECT'`.
 #### Parameter: `ResponseData`
 
 * **statusCode** `number`
-* **headers** `Record<string, string | string[]>` - Note that all header keys are lower-cased, e. g. `content-type`.
+* **headers** `Record<string, string | string[]>` - Note that all header keys are lower-cased, e.g. `content-type`.
 * **body** `stream.Readable` which also implements [the body mixin from the Fetch Standard](https://fetch.spec.whatwg.org/#body-mixin).
 * **trailers** `Record<string, string>` - This object starts out
   as empty and will be mutated to contain trailers after `body` has emitted `'end'`.
@@ -974,7 +974,7 @@ const client = new Client("http://example.com").compose(
   })
 );
 
-// or 
+// or
 client.dispatch(
   {
     path: "/",

docs/docs/api/MockPool.md

@@ -81,7 +81,7 @@ By default, `reply` and `replyWithError` define the behaviour for the first matc
 
 ### Return: `MockScope`
 
-A `MockScope` is associated with a single `MockInterceptor`. With this, we can configure the default behaviour of a intercepted reply.
+A `MockScope` is associated with a single `MockInterceptor`. With this, we can configure the default behaviour of an intercepted reply.
 
 * **delay** `(waitInMs: number) => MockScope` - delay the associated reply by a set amount in ms.
 * **persist** `() => MockScope` - any matching request will always reply with the defined response indefinitely.

docs/docs/api/RetryAgent.md

@@ -40,6 +40,6 @@ import { Agent, RetryAgent } from 'undici'
 const agent = new RetryAgent(new Agent())
 
 const res = await agent.request('http://example.com')
-console.log(res.statuCode)
+console.log(res.statusCode)
 console.log(await res.body.text())
 ```

docs/docs/api/RetryHandler.md

@@ -46,7 +46,7 @@ It represents the retry state for a given request.
 - **dispatch** `(options: Dispatch.DispatchOptions, handlers: Dispatch.DispatchHandlers) => Promise<Dispatch.DispatchResponse>` (required) - Dispatch function to be called after every retry.
 - **handler** Extends [`Dispatch.DispatchHandlers`](Dispatcher.md#dispatcherdispatchoptions-handler) (required) - Handler function to be called after the request is successful or the retries are exhausted.
 
->__Note__: The `RetryHandler` does not retry over stateful bodies (e.g. streams, AsyncIterable) as those, once consumed, are left in an state that cannot be reutilized. For these situations the `RetryHandler` will identify
+>__Note__: The `RetryHandler` does not retry over stateful bodies (e.g. streams, AsyncIterable) as those, once consumed, are left in a state that cannot be reutilized. For these situations the `RetryHandler` will identify
 >the body as stateful and will not retry the request rejecting with the error `UND_ERR_REQ_RETRY`.
 
 Examples:

lib/api/readable.js

@@ -86,7 +86,7 @@ class BodyReadable extends Readable {
 
     // Workaround for Node "bug". If the stream is destroyed in same
     // tick as it is created, then a user who is waiting for a
-    // promise (i.e micro tick) for installing a 'error' listener will
+    // promise (i.e micro tick) for installing an 'error' listener will
     // never get a chance and will always encounter an unhandled exception.
     if (!this[kUsed]) {
       setImmediate(() => {

lib/dispatcher/client-h2.js

@@ -281,7 +281,7 @@ function writeH2 (client, request) {
     }
 
     // We do not destroy the socket as we can continue using the session
-    // the stream get's destroyed and the session remains to create new streams
+    // the stream gets destroyed and the session remains to create new streams
     util.destroy(body, err)
   }
 

lib/handler/redirect-handler.js

@@ -146,7 +146,7 @@ class RedirectHandler {
 
         TLDR: undici always ignores 3xx response bodies.
 
-        Redirection is used to serve the requested resource from another URL, so it is assumes that
+        Redirection is used to serve the requested resource from another URL, so it assumes that
         no body is generated (and thus can be ignored). Even though generating a body is not prohibited.
 
         For status 301, 302, 303, 307 and 308 (the latter from RFC 7238), the specs mention that the body usually

lib/handler/retry-handler.js

@@ -194,8 +194,8 @@ class RetryHandler {
 
       // Only Partial Content 206 supposed to provide Content-Range,
       // any other status code that partially consumed the payload
-      // should not be retry because it would result in downstream
-      // wrongly concatanete multiple responses.
+      // should not be retried because it would result in downstream
+      // wrongly concatenate multiple responses.
       if (statusCode !== 206 && (this.start > 0 || statusCode !== 200)) {
         this.abort(
           new RequestRetryError('server does not support the range header and the payload was partially consumed', statusCode, {

test/interceptors/retry.js

@@ -136,7 +136,7 @@ test('Should use retry-after header for retries (date)', async t => {
   let counter = 0
   const server = createServer()
   let checkpoint
-  const reuestOptions = {
+  const requestOptions = {
     method: 'PUT',
     path: '/',
     headers: {
@@ -182,7 +182,7 @@ test('Should use retry-after header for retries (date)', async t => {
     await once(server, 'close')
   })
 
-  const response = await client.request(reuestOptions)
+  const response = await client.request(requestOptions)
 
   t.equal(response.statusCode, 200)
   t.equal(await response.body.text(), 'hello world!')

test/mock-agent.js

@@ -2357,7 +2357,7 @@ test('MockAgent - clients are not garbage collected', async (t) => {
 
   const baseUrl = `http://localhost:${server.address().port}`
 
-  // Create the dispatcher and isable net connect so we can make sure it matches properly
+  // Create the dispatcher and disable net connect so we can make sure it matches properly
   const dispatcher = new MockAgent()
   dispatcher.disableNetConnect()
 

test/mock-interceptor-unused-assertions.js

@@ -224,7 +224,7 @@ test('defaults to rendering output with terminal color when process.env.CI is un
 
   try {
     mockAgentWithOneInterceptor().assertNoPendingInterceptors()
-    t.fail('Shoudl have thrown')
+    t.fail('Should have thrown')
   } catch (err) {
     t.deepStrictEqual(err.message, tableRowsAlignedToLeft
       ? `

test/node-fetch/main.js

@@ -735,7 +735,7 @@ describe('node-fetch', () => {
       })
       .catch(() => { })
       .then(new Promise((resolve) => {
-        // Wait a few ms to see if a uncaught error occurs
+        // Wait a few ms to see if an uncaught error occurs
         setTimeout(() => {
           resolve()
         }, 20)
@@ -1231,7 +1231,7 @@ describe('node-fetch', () => {
     })
   })
 
-  // Body should been cloned...
+  // Body should be cloned...
   it('constructing a Request/Response with URLSearchParams and mutating it should not affected body', () => {
     const parameters = new URLSearchParams()
     const request = new Request(`${base}inspect`, { method: 'POST', body: parameters })

test/node-test/autoselectfamily.js

@@ -36,7 +36,7 @@ function _lookup (resolver, hostname, options, cb) {
 }
 
 function createDnsServer (ipv6Addr, ipv4Addr, cb) {
-  // Create a DNS server which replies with a AAAA and a A record for the same host
+  // Create a DNS server which replies with an AAAA and an A record for the same host
   const socket = dgram.createSocket('udp4')
 
   socket.on('message', (msg, { address, port }) => {

test/node-test/balanced-pool.js

@@ -501,7 +501,7 @@ describe('weighted round robin', () => {
 
         await Promise.all(servers.map(server => server.prepareForIteration(i)))
 
-        // send a request using undinci
+        // send a request using undici
         try {
           await client.request({ path: '/', method: 'GET' })
         } catch (e) {

test/node-test/tree.js

@@ -16,7 +16,7 @@ describe('Ternary Search Tree', () => {
   test('looking up not inserted key returns null', () => {
     const tst = new TernarySearchTree()
     tst.insert('a', 'a')
-    assert.strictEqual(tst.lookup(Buffer.from('non-existant')), null)
+    assert.strictEqual(tst.lookup(Buffer.from('non-existent')), null)
   })
 
   test('not ascii string', () => {