Skip to content

fix: validate generated OpenAPI spec against OAS 3.1 #1277

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
grdsdev wants to merge 8 commits into
base: master
Choose a base branch
from
Open

fix: validate generated OpenAPI spec against OAS 3.1 #1277

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
618b50c
feat: bump OpenAPI spec to 3.2.0
grdsdev Jul 28, 2026
4e37bde
chore: drop comments above openapi version field
grdsdev Jul 28, 2026
6b5be1a
ci: validate generated OpenAPI spec is valid 3.2
grdsdev Jul 28, 2026
0ec49f4
fix: validate embedded schemas in OpenAPI spec validation
grdsdev Jul 28, 2026
ac2699d
fix: convert OpenAPI 3.0 nullable idiom to JSON Schema 2020-12
grdsdev Jul 28, 2026
895761a
fix: enable vector and iceberg env vars for OpenAPI spec validation i…
grdsdev Jul 29, 2026
5d1e0ec
fix: downgrade OpenAPI spec target from 3.2.0 to 3.1.0
grdsdev Jul 29, 2026
f514abb
fix: catch stray OAS 3.0 nullable idiom in OpenAPI spec validation
grdsdev Jul 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,11 @@ jobs:
cp .env.test.sample .env.test
- name: Unit tests pass
run: npm run test:unit:coverage
- name: Validate generated OpenAPI spec
run: npm run docs:validate
Comment thread
grdsdev marked this conversation as resolved.
env:
VECTOR_ENABLED: true
ICEBERG_ENABLED: true
- name: Upload coverage results to Coveralls
uses: coverallsapp/github-action@5cbfd81b66ca5d10c19b062c04de0199c215fb6e # v2.3.7
with:
Expand Down
251 changes: 251 additions & 0 deletions package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
"migrations:types": "tsx src/scripts/migrations-types.ts",
"docs:export": "tsx ./src/scripts/export-docs.ts",
"docs:export-ui": "tsx ./src/scripts/export-docs-ui.ts",
"docs:validate": "tsx ./src/scripts/validate-openapi.ts",
"pprof:capture": "tsx src/scripts/pprof-client.ts",
"test:dummy-data": "tsx -r dotenv/config ./src/test/db/import-dummy-data.ts",
"test:unit": "vitest run --config vitest.unit.config.ts",
Expand Down Expand Up @@ -117,6 +118,7 @@
"devDependencies": {
"@aws-sdk/s3-presigned-post": "^3.1023.0",
"@biomejs/biome": "2.5.1",
"@hyperjump/json-schema": "^1.17.7",
Comment thread
ferhatelmas marked this conversation as resolved.
"@platformatic/runtime": "^3.63.0",
"@types/js-yaml": "^4.0.5",
"@types/json-bigint": "^1.0.4",
Expand Down
Loading