Skip to content

Commit 59232f8

Browse files
committed
fix(openapi): import the forbidden-code constants from their module, not the application barrel
The barrel also re-exports the authorized use-case layer, which loads @sim/db at import time. That pulled a database connection into the OpenAPI spec check, so check:audits failed wherever DATABASE_URL is absent, including CI.
1 parent ffb8306 commit 59232f8

2 files changed

Lines changed: 8 additions & 2 deletions

File tree

apps/sim/lib/api/contracts/v2/__tests__/cross-cutting.test.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,10 @@ import { v2ListAuditLogsContract } from '@/lib/api/contracts/v2/audit-logs'
77
import { ERROR_RESPONSES } from '@/lib/api/contracts/v2/openapi/shared'
88
import { v2CreateTableViewContract, v2QueryRowsBodySchema } from '@/lib/api/contracts/v2/tables'
99
import { v2GetWorkflowRunContract } from '@/lib/api/contracts/v2/workflows'
10-
import { FORBIDDEN_DETAIL_CODE_DESCRIPTIONS, FORBIDDEN_DETAIL_CODES } from '@/lib/core/application'
10+
import {
11+
FORBIDDEN_DETAIL_CODE_DESCRIPTIONS,
12+
FORBIDDEN_DETAIL_CODES,
13+
} from '@/lib/core/application/forbidden'
1114

1215
/**
1316
* The cross-cutting promises that no single resource family owns, and that

apps/sim/lib/api/contracts/v2/openapi/shared.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,10 @@ import type {
55
OpenApiHeader,
66
OpenApiSecurityScheme,
77
} from '@/lib/api/openapi/types'
8-
import { FORBIDDEN_DETAIL_CODE_DESCRIPTIONS, FORBIDDEN_DETAIL_CODES } from '@/lib/core/application'
8+
import {
9+
FORBIDDEN_DETAIL_CODE_DESCRIPTIONS,
10+
FORBIDDEN_DETAIL_CODES,
11+
} from '@/lib/core/application/forbidden'
912

1013
export const RATE_LIMIT_HEADERS = [
1114
'X-RateLimit-Limit',

0 commit comments

Comments
 (0)