From 795616fab5aed420d7ad001075b8c1547fda6705 Mon Sep 17 00:00:00 2001 From: Travis Fischer Date: Tue, 27 May 2025 00:43:53 +0700 Subject: [PATCH] chore: knip; remove unused files/deps/exports --- apps/api/package.json | 1 - apps/api/src/api-v1/oauth-redirect.ts | 42 ---------------------- apps/api/src/db/index.ts | 34 +++++++++--------- apps/api/src/lib/test/index.ts | 2 -- apps/api/src/lib/test/mock-sentry-node.ts | 25 ------------- apps/api/src/lib/test/setup-mock-logger.ts | 12 ------- packages/api-client/package.json | 3 +- packages/cli/package.json | 3 +- 8 files changed, 19 insertions(+), 103 deletions(-) delete mode 100644 apps/api/src/api-v1/oauth-redirect.ts delete mode 100644 apps/api/src/lib/test/index.ts delete mode 100644 apps/api/src/lib/test/mock-sentry-node.ts delete mode 100644 apps/api/src/lib/test/setup-mock-logger.ts diff --git a/apps/api/package.json b/apps/api/package.json index 17ec24b2..d5bb1e75 100644 --- a/apps/api/package.json +++ b/apps/api/package.json @@ -27,7 +27,6 @@ }, "dependencies": { "@agentic/platform-core": "workspace:*", - "@agentic/platform-openapi": "workspace:*", "@agentic/platform-schemas": "workspace:*", "@agentic/platform-validators": "workspace:*", "@fisch0920/drizzle-orm": "^0.43.7", diff --git a/apps/api/src/api-v1/oauth-redirect.ts b/apps/api/src/api-v1/oauth-redirect.ts deleted file mode 100644 index eea2b5b6..00000000 --- a/apps/api/src/api-v1/oauth-redirect.ts +++ /dev/null @@ -1,42 +0,0 @@ -import type { OpenAPIHono } from '@hono/zod-openapi' -import { assert } from '@agentic/platform-core' - -// TODO: Unused in favor of `better-auth` -export function registerV1OAuthRedirect(app: OpenAPIHono) { - return app.all('oauth', async (ctx) => { - if (ctx.req.query('state')) { - const { state: state64, ...query } = ctx.req.query() - - // google oauth + others - const { uri, ...state } = JSON.parse( - Buffer.from(state64!, 'base64').toString() - ) as any - - assert( - uri, - 404, - `Error oauth redirect not found "${new URLSearchParams(ctx.req.query()).toString()}"` - ) - - const searchParams = new URLSearchParams({ - ...state, - ...query - }) - - ctx.redirect(`${uri}?${searchParams.toString()}`) - } else { - // github oauth - // https://developer.github.com/apps/building-oauth-apps/authorizing-oauth-apps/#redirect-urls - const { uri, ...params } = ctx.req.query() - - assert( - uri, - 404, - `Error oauth redirect not found "${new URLSearchParams(ctx.req.query()).toString()}"` - ) - - const searchParams = new URLSearchParams(params) - ctx.redirect(`${uri}?${searchParams.toString()}`) - } - }) -} diff --git a/apps/api/src/db/index.ts b/apps/api/src/db/index.ts index 663ca99b..102ff108 100644 --- a/apps/api/src/db/index.ts +++ b/apps/api/src/db/index.ts @@ -25,29 +25,29 @@ export type * from './types' export * from './utils' export { and, - arrayContained, - arrayContains, - arrayOverlaps, + // arrayContained, + // arrayContains, + // arrayOverlaps, asc, - between, + // between, desc, eq, - exists, + // exists, gt, - gte, - ilike, - inArray, - isNotNull, + // gte, + // ilike, + // inArray, + // isNotNull, isNull, like, - lt, - lte, - ne, + // lt, + // lte, + // ne, not, - notBetween, - notExists, - notIlike, - notInArray, - notLike, + // notBetween, + // notExists, + // notIlike, + // notInArray, + // notLike, or } from '@fisch0920/drizzle-orm' diff --git a/apps/api/src/lib/test/index.ts b/apps/api/src/lib/test/index.ts deleted file mode 100644 index 74ec6e0e..00000000 --- a/apps/api/src/lib/test/index.ts +++ /dev/null @@ -1,2 +0,0 @@ -export * from './mock-sentry-node' -export * from './setup-mock-logger' diff --git a/apps/api/src/lib/test/mock-sentry-node.ts b/apps/api/src/lib/test/mock-sentry-node.ts deleted file mode 100644 index 8e4b6904..00000000 --- a/apps/api/src/lib/test/mock-sentry-node.ts +++ /dev/null @@ -1,25 +0,0 @@ -import { vi } from 'vitest' - -/** - * Mocks the entire @sentry/node module so that captureException is a spy. - */ -export function mockSentryNode(): void { - vi.mock('@sentry/node', async () => { - const actual = await vi.importActual('@sentry/node') - return { - ...actual, - captureException: vi.fn(), - setTags: vi.fn(), - setTag: vi.fn(), - withIsolationScope: vi.fn((fn) => fn()), - startSpan: vi.fn((_, fn) => { - const fakeSpan = { - setAttributes: vi.fn(), - end: vi.fn() - } - const callbackResult = fn(fakeSpan) - return callbackResult - }) - } - }) -} diff --git a/apps/api/src/lib/test/setup-mock-logger.ts b/apps/api/src/lib/test/setup-mock-logger.ts deleted file mode 100644 index 81761f05..00000000 --- a/apps/api/src/lib/test/setup-mock-logger.ts +++ /dev/null @@ -1,12 +0,0 @@ -import type { Logger } from '@agentic/platform-core' -import { vi } from 'vitest' - -export function setupMockLogger() { - return { - trace: vi.fn(), - debug: vi.fn(), - info: vi.fn(), - warn: vi.fn(), - error: vi.fn() - } as Logger -} diff --git a/packages/api-client/package.json b/packages/api-client/package.json index 16d4ed4a..fa59aa66 100644 --- a/packages/api-client/package.json +++ b/packages/api-client/package.json @@ -27,8 +27,7 @@ "@agentic/platform-schemas": "workspace:*", "@openauthjs/openauth": "^0.4.3", "ky": "catalog:", - "type-fest": "catalog:", - "zod": "catalog:" + "type-fest": "catalog:" }, "devDependencies": { "openapi-typescript": "^7.8.0" diff --git a/packages/cli/package.json b/packages/cli/package.json index a1d6ddc0..3f8b0a75 100644 --- a/packages/cli/package.json +++ b/packages/cli/package.json @@ -42,8 +42,7 @@ "open": "^10.1.2", "ora": "^8.2.0", "restore-cursor": "catalog:", - "unconfig": "^7.3.2", - "zod": "catalog:" + "unconfig": "^7.3.2" }, "devDependencies": { "@agentic/platform-fixtures": "workspace:*",