fix: remove problematic omit.js dependency (#6172)

* fix: don't reference `.default` imported ESM default I'm not sure why this even works, but it doesn't via bun, webpack, and many in other cases. Fixes #5613. * fix: remove faulty omit.js package See benjycui/omit.js#10 and #5613

Author: serhalp

package-lock.json

@@ -45,7 +45,6 @@
  "lodash-es": "^4.17.21",
  "micro-api-client": "^3.3.0",
  "node-fetch": "^3.0.0",
- "omit.js": "^2.0.2",
  "p-wait-for": "^5.0.0",
  "qs": "^6.9.6"
  },

packages/js-client/package.json

@@ -1,5 +1,6 @@
 import { JSONHTTPError, TextHTTPError } from 'micro-api-client'
-import omit from 'omit.js'
+
+import omit from '../omit.js'
 
 // Read and parse the HTTP response
 export const parseResponse = async function (response) {
@@ -44,7 +45,7 @@ const addFallbackErrorMessage = function (error, textResponse) {
 }
 
 export const getFetchError = function (error, url, opts) {
- const data = omit.default(opts, ['Authorization'])
+ const data = omit(opts, ['Authorization'])
  if (error.name !== 'FetchError') {
  error.name = 'FetchError'
  }

packages/js-client/src/methods/response.js

@@ -0,0 +1,16 @@
+import test from 'ava'
+
+import omit from './omit.js'
+
+test('creates a shallow copy', (t) => {
+ const obj = { name: 'Benjy' }
+ const copy = omit(obj, [])
+ t.not(obj, copy)
+ t.deepEqual(obj, copy)
+})
+
+test('returns an object without the specified fields', (t) => {
+ const obj = { name: 'Benjy', age: 18 }
+ t.deepEqual(omit(obj, ['age']), { name: 'Benjy' })
+ t.deepEqual(omit(obj, ['name', 'age']), {})
+})

packages/js-client/src/omit.test.ts

@@ -0,0 +1,10 @@
+export default function omit<T extends Record<string | number | symbol, unknown>, K extends keyof T>(
+ obj: T,
+ keys: K[],
+): Omit<T, K> {
+ const shallowCopy = { ...obj }
+ for (const key of keys) {
+ delete shallowCopy[key]
+ }
+ return shallowCopy
+}

packages/js-client/src/omit.ts

@@ -1,11 +1,10 @@
-import omit from 'omit.js'
-
+import omit from './omit.js'
 import { openApiSpec } from './open_api.js'
 
 // Retrieve all OpenAPI operations
 export const getOperations = function () {
  return Object.entries(openApiSpec.paths).flatMap(([path, pathItem]) => {
- const operations = omit.default(pathItem, ['parameters'])
+ const operations = omit(pathItem, ['parameters'])
  return Object.entries(operations).map(([method, operation]) => {
  const parameters = getParameters(pathItem.parameters, operation.parameters)
  return { ...operation, verb: method, path, parameters }