Bump the dev-dependencies group with 8 updates (#551)

* Bump the dev-dependencies group with 8 updates

Bumps the dev-dependencies group with 8 updates:

| Package | From | To |
| --- | --- | --- |
| [@microsoft/api-documenter](https://github.com/microsoft/rushstack/tree/HEAD/apps/api-documenter) | `7.27.3` | `7.28.1` |
| [@microsoft/api-extractor](https://github.com/microsoft/rushstack/tree/HEAD/apps/api-extractor) | `7.53.3` | `7.55.1` |
| [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) | `20.19.24` | `20.19.25` |
| [@vitest/eslint-plugin](https://github.com/vitest-dev/eslint-plugin-vitest) | `1.4.0` | `1.5.1` |
| [prettier](https://github.com/prettier/prettier) | `3.6.2` | `3.7.3` |
| [tsup](https://github.com/egoist/tsup) | `8.5.0` | `8.5.1` |
| [typescript-eslint](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/typescript-eslint) | `8.46.3` | `8.48.0` |
| [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) | `4.0.7` | `4.0.14` |


Updates `@microsoft/api-documenter` from 7.27.3 to 7.28.1
- [Changelog](https://github.com/microsoft/rushstack/blob/main/apps/api-documenter/CHANGELOG.md)
- [Commits](https://github.com/microsoft/rushstack/commits/@microsoft/api-documenter_v7.28.1/apps/api-documenter)

Updates `@microsoft/api-extractor` from 7.53.3 to 7.55.1
- [Changelog](https://github.com/microsoft/rushstack/blob/main/apps/api-extractor/CHANGELOG.md)
- [Commits](https://github.com/microsoft/rushstack/commits/@microsoft/api-extractor_v7.55.1/apps/api-extractor)

Updates `@types/node` from 20.19.24 to 20.19.25
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Updates `@vitest/eslint-plugin` from 1.4.0 to 1.5.1
- [Release notes](https://github.com/vitest-dev/eslint-plugin-vitest/releases)
- [Commits](vitest-dev/eslint-plugin-vitest@v1.4.0...v1.5.1)

Updates `prettier` from 3.6.2 to 3.7.3
- [Release notes](https://github.com/prettier/prettier/releases)
- [Changelog](https://github.com/prettier/prettier/blob/main/CHANGELOG.md)
- [Commits](prettier/prettier@3.6.2...3.7.3)

Updates `tsup` from 8.5.0 to 8.5.1
- [Release notes](https://github.com/egoist/tsup/releases)
- [Commits](egoist/tsup@v8.5.0...v8.5.1)

Updates `typescript-eslint` from 8.46.3 to 8.48.0
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/typescript-eslint/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.48.0/packages/typescript-eslint)

Updates `vitest` from 4.0.7 to 4.0.14
- [Release notes](https://github.com/vitest-dev/vitest/releases)
- [Commits](https://github.com/vitest-dev/vitest/commits/v4.0.14/packages/vitest)

---
updated-dependencies:
- dependency-name: "@microsoft/api-documenter"
  dependency-version: 7.28.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: "@microsoft/api-extractor"
  dependency-version: 7.55.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: "@types/node"
  dependency-version: 20.19.25
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dev-dependencies
- dependency-name: "@vitest/eslint-plugin"
  dependency-version: 1.5.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: prettier
  dependency-version: 3.7.3
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: tsup
  dependency-version: 8.5.1
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dev-dependencies
- dependency-name: typescript-eslint
  dependency-version: 8.48.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: dev-dependencies
- dependency-name: vitest
  dependency-version: 4.0.14
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: dev-dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>

* prettier

* chore: linter fixes

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Will Harney <[email protected]>
Co-authored-by: Colin Casey <[email protected]>

Author: 3 people

lib/__tests__/cookieJar.spec.ts

@@ -30,6 +30,7 @@
  */
 
 /* eslint vitest/expect-expect: [error, {assertFunctionNames: [expect, assertions]}] */
+/* eslint vitest/no-standalone-expect: [error, {additionalTestBlockFunctions: [apiVariants, beforeEach]}]*/
 
 import {
   afterAll,
@@ -1539,19 +1540,16 @@ describe.each(['local', 'example', 'invalid', 'localhost', 'test'])(
         rejectPublicSuffixes: true,
         allowSpecialUseDomain: false,
       })
-      try {
-        await cookieJar.setCookie(
+      await expect(
+        cookieJar.setCookie(
           `settingThisShouldPass=true; Domain=dev.${specialUseDomain}; Path=/;`,
           `http://dev.${specialUseDomain}`,
-        )
-      } catch (e) {
-        if (!(e instanceof Error)) {
-          throw new Error('This should be an error instance')
-        }
-        expect(e.message).toBe(
-          `Cookie has domain set to the public suffix "${specialUseDomain}" which is a special use domain. To allow this, configure your CookieJar with {allowSpecialUseDomain: true, rejectPublicSuffixes: false}.`,
-        )
-      }
+        ),
+      ).rejects.toThrow(
+        `Cookie has domain set to the public suffix "${
+          specialUseDomain
+        }" which is a special use domain. To allow this, configure your CookieJar with {allowSpecialUseDomain: true, rejectPublicSuffixes: false}.`,
+      )
     })
   },
 )
@@ -1590,19 +1588,16 @@ describe.each(['local', 'example', 'invalid', 'localhost', 'test'])(
       it('should reject special domain cookies if allowSpecialUseDomain is set to the default value', async () => {
         expect.assertions(1)
         const cookieJar = new CookieJar()
-        try {
-          await cookieJar.setCookie(
+        await expect(
+          cookieJar.setCookie(
             `settingThisShouldPass=true; Domain=${specialUseDomain}; Path=/;`,
             `http://${specialUseDomain}`,
-          )
-        } catch (e) {
-          if (!(e instanceof Error)) {
-            throw new Error('This should be an error instance')
-          }
-          expect(e.message).toBe(
-            `Cookie has domain set to the public suffix "${specialUseDomain}" which is a special use domain. To allow this, configure your CookieJar with {allowSpecialUseDomain: true, rejectPublicSuffixes: false}.`,
-          )
-        }
+          ),
+        ).rejects.toThrow(
+          `Cookie has domain set to the public suffix "${
+            specialUseDomain
+          }" which is a special use domain. To allow this, configure your CookieJar with {allowSpecialUseDomain: true, rejectPublicSuffixes: false}.`,
+        )
       })
     }
 
@@ -1638,19 +1633,16 @@ describe.each(['local', 'example', 'invalid', 'localhost', 'test'])(
           rejectPublicSuffixes: true,
           allowSpecialUseDomain: true,
         })
-        try {
-          await cookieJar.setCookie(
+        await expect(
+          cookieJar.setCookie(
             `settingThisShouldPass=true; Domain=${specialUseDomain}; Path=/;`,
             `http://${specialUseDomain}`,
-          )
-        } catch (e) {
-          if (!(e instanceof Error)) {
-            throw new Error('This should be an error instance')
-          }
-          expect(e.message).toBe(
-            `Cookie has domain set to the public suffix "${specialUseDomain}" which is a special use domain. To allow this, configure your CookieJar with {allowSpecialUseDomain: true, rejectPublicSuffixes: false}.`,
-          )
-        }
+          ),
+        ).rejects.toThrow(
+          `Cookie has domain set to the public suffix "${
+            specialUseDomain
+          }" which is a special use domain. To allow this, configure your CookieJar with {allowSpecialUseDomain: true, rejectPublicSuffixes: false}.`,
+        )
       })
     }
 
@@ -1660,19 +1652,16 @@ describe.each(['local', 'example', 'invalid', 'localhost', 'test'])(
         rejectPublicSuffixes: true,
         allowSpecialUseDomain: false,
       })
-      try {
-        await cookieJar.setCookie(
+      await expect(
+        cookieJar.setCookie(
           `settingThisShouldPass=true; Domain=${specialUseDomain}; Path=/;`,
           `http://${specialUseDomain}`,
-        )
-      } catch (e) {
-        if (!(e instanceof Error)) {
-          throw new Error('This should be an error instance')
-        }
-        expect(e.message).toBe(
-          `Cookie has domain set to the public suffix "${specialUseDomain}" which is a special use domain. To allow this, configure your CookieJar with {allowSpecialUseDomain: true, rejectPublicSuffixes: false}.`,
-        )
-      }
+        ),
+      ).rejects.toThrow(
+        `Cookie has domain set to the public suffix "${
+          specialUseDomain
+        }" which is a special use domain. To allow this, configure your CookieJar with {allowSpecialUseDomain: true, rejectPublicSuffixes: false}.`,
+      )
     })
   },
 )

lib/__tests__/cookiePrefixes.spec.ts

@@ -6,6 +6,8 @@ let cookieJar: CookieJar
 const insecureUrl = 'http://www.example.com'
 const secureUrl = 'https://www.example.com'
 
+/* eslint vitest/no-standalone-expect: [error, {additionalTestBlockFunctions: [beforeEach]}]*/
+
 describe('When `prefixSecurity` is enabled for `CookieJar`', () => {
   describe('silent', () => {
     beforeEach(() => {

lib/__tests__/ietf.spec.ts

@@ -73,30 +73,14 @@ describe('IETF http state tests', () => {
     it.each(exampleDates)(
       `ietf_data/dates/examples: $test`,
       ({ test, expected }) => {
-        if (expected) {
-          const date = parseDate(test)
-          if (!date) {
-            throw new Error('This should have parsed')
-          }
-          expect(date.toUTCString()).toBe(expected)
-        } else {
-          expect(parseDate(test)).toBeUndefined()
-        }
+        expect(parseDate(test)?.toUTCString()).toBe(expected || undefined)
       },
     )
 
     it.each(bsdExampleDates)(
       `ietf_data/dates/bsd_examples: $test`,
       ({ test, expected }) => {
-        if (expected) {
-          const date = parseDate(test)
-          if (!date) {
-            throw new Error('This should have parsed')
-          }
-          expect(date.toUTCString()).toBe(expected)
-        } else {
-          expect(parseDate(test)).toBeUndefined()
-        }
+        expect(parseDate(test)?.toUTCString()).toBe(expected || undefined)
       },
     )
   })

lib/__tests__/jarSerialization.spec.ts

@@ -253,32 +253,24 @@ describe('cookieJar serialization', () => {
             domain?: string
             path?: string
           }
-          expect(typeof parsedValue.domain).toBe('string')
-          expect(typeof parsedValue.path).toBe('string')
+          // eslint-disable-next-line vitest/no-conditional-expect
+          expect(parsedValue.domain).toBeTypeOf('string')
+          // eslint-disable-next-line vitest/no-conditional-expect
+          expect(parsedValue.path).toBeTypeOf('string')
         }
 
-        if (
-          serializedCookie.key === 'infExp' ||
-          serializedCookie.key === 'max'
-        ) {
-          expect(serializedCookie.expires).toBeFalsy()
-        } else {
-          expect(serializedCookie.expires).toBe(expires.toISOString())
-        }
+        const expiry =
+          serializedCookie.key === 'infExp' || serializedCookie.key === 'max'
+            ? undefined
+            : expires.toISOString()
+        expect(serializedCookie.expires).toBe(expiry)
 
-        if (serializedCookie.key === 'max') {
-          expect(serializedCookie.maxAge).toBe(3600)
-        } else {
-          expect(serializedCookie.maxAge).toBeUndefined()
-        }
+        const maxAge = serializedCookie.key === 'max' ? 3600 : undefined
+        expect(serializedCookie.maxAge).toBe(maxAge)
 
-        if (serializedCookie.key === 'flags') {
-          expect(serializedCookie.secure).toBe(true)
-          expect(serializedCookie.httpOnly).toBe(true)
-        } else {
-          expect(serializedCookie.secure).toBeUndefined()
-          expect(serializedCookie.httpOnly).toBeUndefined()
-        }
+        const flag = serializedCookie.key === 'flags' ? true : undefined
+        expect(serializedCookie.secure).toBe(flag)
+        expect(serializedCookie.httpOnly).toBe(flag)
 
         expect(serializedCookie.hostOnly).toBe(serializedCookie.key === 'honly')
 

lib/__tests__/parse.spec.ts

@@ -407,9 +407,11 @@ describe('Cookie.parse', () => {
 
     const value = input === undefined ? undefined : input.valueOf()
     const cookie = Cookie.parse(value as string, parseOptions)
+    // eslint-disable-next-line vitest/no-conditional-expect
     expect(cookie).toEqual(output && expect.objectContaining(output))
 
     if (cookie && typeof assertValidateReturns === 'boolean') {
+      // eslint-disable-next-line vitest/no-conditional-expect
       expect(cookie.validate()).toBe(assertValidateReturns)
     }
   })

lib/cookie/cookieJar.ts

@@ -276,9 +276,9 @@ function isHostPrefixConditionMet(cookie: Cookie): boolean {
     !startsWithHostPrefix ||
     Boolean(
       cookie.secure &&
-        cookie.hostOnly &&
-        cookie.path != null &&
-        cookie.path === '/',
+      cookie.hostOnly &&
+      cookie.path != null &&
+      cookie.path === '/',
     )
   )
 }