Skip to content

combine auth apis#285

Merged
nikita-seedlabs merged 3 commits intomainfrom
nikita/combine_auth_apis
Mar 19, 2026
Merged

combine auth apis#285
nikita-seedlabs merged 3 commits intomainfrom
nikita/combine_auth_apis

Conversation

@nikita-seedlabs
Copy link
Contributor

  • combines the auth api yamls back together
  • removes version bump python script
  • fixes rust package version inconsistencies

Copilot AI review requested due to automatic review settings March 19, 2026 18:30
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR consolidates the Auth OpenAPI specs back into a single YAML, removes the legacy Python-based version bump workflow, and aligns Rust crate versions to the current SDK versioning used elsewhere in the repo.

Changes:

  • Merge auth-common.yaml components (schemas + default error response) into auth-api.yaml and update refs accordingly.
  • Remove scripts/version_bump.py and drop the version-bump Makefile target.
  • Fix Rust version inconsistencies by bumping rust/Cargo.toml and updating rust/Cargo.lock to 1.16.0.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
scripts/version_bump.py Deletes the legacy spec-hash/version bump Python script.
Makefile Removes version-bump target; generate now only regenerates clients.
resources/auth-common.yaml Removes the separate “auth common” spec file (now merged).
resources/auth-api.yaml Adds DefaultError + LoginResponse and updates $refs to be self-contained.
rust/Cargo.toml Bumps bluefin-pro version to 1.16.0 to match the workspace/dependency versions.
rust/Cargo.lock Updates locked package versions for bluefin-pro and bluefin_api to 1.16.0.
rust/gen/bluefin_api/.openapi-generator/FILES Updates generated-file manifest (drops .openapi-generator-ignore entry).
.apigen-state Updates stored spec hashes / combined hash after auth spec consolidation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nikita-seedlabs nikita-seedlabs merged commit 21e324d into main Mar 19, 2026
12 checks passed
@nikita-seedlabs nikita-seedlabs deleted the nikita/combine_auth_apis branch March 19, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants