Skip to content

fix: Support bru-lang format and fix OpenAPI spec compliance#3

Merged
GyeongHoKim merged 9 commits intomainfrom
fix/bru-lang-parse
Jan 7, 2026
Merged

fix: Support bru-lang format and fix OpenAPI spec compliance#3
GyeongHoKim merged 9 commits intomainfrom
fix/bru-lang-parse

Conversation

@GyeongHoKim
Copy link
Copy Markdown
Owner

Changes

This PR includes fixes for bru-lang format support and OpenAPI 3.0 spec compliance.

Key Changes

  1. bru-lang Format Support

  2. OpenAPI 3.0 Spec Compliance

    • Changed response codes to string format ('200', '400', etc.) as required by OpenAPI spec
    • Unified version field to string format
    • Cleaned up description format
    • Added requestBody for form-urlencoded and multipart/form-data requests
  3. Documentation Improvements

    • Fixed README.md examples (result.openapiresult.spec)
    • Added Sync function usage examples
    • Fixed function name typo (convertBrunoCollectionToAPISyncconvertBrunoCollectionToOpenAPISync)
  4. Test Files Update

    • Updated all fixture expected-openapi.yml files to correct OpenAPI format

Testing

  • All existing tests pass
  • OpenAPI spec validation passes

Resolves #2

- 테스트에 사용되는 fixture들이 공식문서의 bru-lang을 따르지 않던 문제
- 모든 HTTP Method를 사용하도록 fixtures 추가
- bruno-parser.test.ts가 실제 bru-lang fixture들을 사용하도록 수정
- InvalidCollectionPathError for existance of collection path
- InvalidJsonError for json parse error
- InvalidBrunoJsonError for existing bruno.json but invalid json inside
- BruFileParseError for parse error
- OpenAPI 관련 타입을 openapi-types 라이브러리를 re-export하는 식으로 변경
- 기존에 *.bru 파일들 파싱을 INI 형식으로 착각한 점, 공식문서 참고하여 bru-lang을 지원하게 변경
- 변경된 타입을 테스트 코드에 적용
@gitguardian
Copy link
Copy Markdown

gitguardian bot commented Jan 7, 2026

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
24268479 Triggered Username Password a320c05 tests/fixtures/auth-test-collection/request-with-basic-auth.bru View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@GyeongHoKim GyeongHoKim merged commit 776158c into main Jan 7, 2026
2 checks passed
@GyeongHoKim GyeongHoKim deleted the fix/bru-lang-parse branch January 7, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skipping item 'register' - missing method or URL

1 participant