chore(deps): drop unused utoipa, utoipa-axum, utoipa-scalar by moonming · Pull Request #306 · api7/aisix

moonming · 2026-05-17T00:45:53Z

Summary

Removes three workspace dependencies that have zero in-tree usage:

utoipa = "5"
utoipa-axum = "0.1"
utoipa-scalar = "0.2"

Why

These were introduced in case the admin OpenAPI 3.1 doc would be derived via utoipa's #[derive(ToSchema)] macros, but crates/aisix-admin/src/openapi.rs:15-19 records the team's decision to keep the spec hand-written:

Small enough that maintaining it by hand is less effort than wiring utoipa derive macros across every handler; the surface is stable enough that drift is easy to spot in review.

The decision has been in effect for some time and the unused deps are pure noise in Cargo.lock (~59 lines of transitive crates). Removing them is a preparatory tidy-up before the upcoming canonical JSON Schema work (#304 item #1), which will follow a different code path (schemars derive on resource structs in aisix-core) and benefit from a clean dependency baseline.

Verification

grep -rE "utoipa|ToSchema|IntoParams" --include="*.rs" crates/ — only the doc-comment match in openapi.rs (kept as historical context)
cargo check --workspace
cargo clippy --workspace --all-targets -- -D warnings
cargo fmt --all -- --check

Scope

Zero behavior change. The hand-written OpenAPI doc and Scalar mount at /admin/openapi.json / /admin/openapi-scalar continue to serve exactly the same content; nothing about how requests flow through aisix-admin is touched.

Refs #304 (#1).

Summary by CodeRabbit

Chores
- Removed OpenAPI documentation-related dependencies from project configuration.

These three workspace dependencies (and their transitives) have zero in-tree usage: - 0 `use utoipa` / `use utoipa_axum` / `use utoipa_scalar` imports - 0 `#[derive(ToSchema)]` / `#[derive(IntoParams)]` derives - The only mention is a doc-comment in `crates/aisix-admin/src/openapi.rs:15-19` explaining why the OpenAPI 3.1 document is hand-written rather than derived — that rationale is preserved. Removing them shrinks the dependency tree (59 lines off Cargo.lock) without any behavior change. Verified with `cargo check --workspace`, `cargo clippy --workspace --all-targets -- -D warnings`, and `cargo fmt --all -- --check`.

coderabbitai · 2026-05-17T00:45:59Z

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: ff8405f2-61c4-4d68-90f6-9da262af5573

📥 Commits

Reviewing files that changed from the base of the PR and between 7e8dab2 and 4b3af76.

⛔ Files ignored due to path filters (1)

Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

💤 Files with no reviewable changes (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

📝 Walkthrough

Walkthrough

OpenAPI-related dependencies (utoipa, utoipa-axum, utoipa-scalar) are removed from the workspace-level dependency definitions and from the aisix-admin crate's dependency list.

Changes

OpenAPI Dependency Removal

Layer / File(s)	Summary
OpenAPI dependency removal `Cargo.toml`, `crates/aisix-admin/Cargo.toml`	OpenAPI crate dependencies are removed from the workspace `[workspace.dependencies]` block and the aisix-admin crate's `[dependencies]` section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Note

🎁 Summarized by CodeRabbit Free

Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

moonming · 2026-05-17T00:45:53Z

Summary

Removes three workspace dependencies that have zero in-tree usage:

utoipa = "5"
utoipa-axum = "0.1"
utoipa-scalar = "0.2"

Why

These were introduced in case the admin OpenAPI 3.1 doc would be derived via utoipa's #[derive(ToSchema)] macros, but crates/aisix-admin/src/openapi.rs:15-19 records the team's decision to keep the spec hand-written:

Small enough that maintaining it by hand is less effort than wiring utoipa derive macros across every handler; the surface is stable enough that drift is easy to spot in review.

The decision has been in effect for some time and the unused deps are pure noise in Cargo.lock (~59 lines of transitive crates). Removing them is a preparatory tidy-up before the upcoming canonical JSON Schema work (#304 item #1), which will follow a different code path (schemars derive on resource structs in aisix-core) and benefit from a clean dependency baseline.

Verification

grep -rE "utoipa|ToSchema|IntoParams" --include="*.rs" crates/ — only the doc-comment match in openapi.rs (kept as historical context)
cargo check --workspace
cargo clippy --workspace --all-targets -- -D warnings
cargo fmt --all -- --check

Scope

Zero behavior change. The hand-written OpenAPI doc and Scalar mount at /admin/openapi.json / /admin/openapi-scalar continue to serve exactly the same content; nothing about how requests flow through aisix-admin is touched.

Refs #304 (#1).

Summary by CodeRabbit

Chores
- Removed OpenAPI documentation-related dependencies from project configuration.

These three workspace dependencies (and their transitives) have zero in-tree usage: - 0 `use utoipa` / `use utoipa_axum` / `use utoipa_scalar` imports - 0 `#[derive(ToSchema)]` / `#[derive(IntoParams)]` derives - The only mention is a doc-comment in `crates/aisix-admin/src/openapi.rs:15-19` explaining why the OpenAPI 3.1 document is hand-written rather than derived — that rationale is preserved. Removing them shrinks the dependency tree (59 lines off Cargo.lock) without any behavior change. Verified with `cargo check --workspace`, `cargo clippy --workspace --all-targets -- -D warnings`, and `cargo fmt --all -- --check`.

coderabbitai · 2026-05-17T00:45:59Z

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: ff8405f2-61c4-4d68-90f6-9da262af5573

📥 Commits

Reviewing files that changed from the base of the PR and between 7e8dab2 and 4b3af76.

⛔ Files ignored due to path filters (1)

Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

💤 Files with no reviewable changes (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

📝 Walkthrough

Walkthrough

OpenAPI-related dependencies (utoipa, utoipa-axum, utoipa-scalar) are removed from the workspace-level dependency definitions and from the aisix-admin crate's dependency list.

Changes

OpenAPI Dependency Removal

Layer / File(s)	Summary
OpenAPI dependency removal `Cargo.toml`, `crates/aisix-admin/Cargo.toml`	OpenAPI crate dependencies are removed from the workspace `[workspace.dependencies]` block and the aisix-admin crate's `[dependencies]` section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Note

🎁 Summarized by CodeRabbit Free

Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

moonming · 2026-05-17T00:45:53Z

Summary

Removes three workspace dependencies that have zero in-tree usage:

utoipa = "5"
utoipa-axum = "0.1"
utoipa-scalar = "0.2"

Why

These were introduced in case the admin OpenAPI 3.1 doc would be derived via utoipa's #[derive(ToSchema)] macros, but crates/aisix-admin/src/openapi.rs:15-19 records the team's decision to keep the spec hand-written:

Small enough that maintaining it by hand is less effort than wiring utoipa derive macros across every handler; the surface is stable enough that drift is easy to spot in review.

The decision has been in effect for some time and the unused deps are pure noise in Cargo.lock (~59 lines of transitive crates). Removing them is a preparatory tidy-up before the upcoming canonical JSON Schema work (#304 item #1), which will follow a different code path (schemars derive on resource structs in aisix-core) and benefit from a clean dependency baseline.

Verification

grep -rE "utoipa|ToSchema|IntoParams" --include="*.rs" crates/ — only the doc-comment match in openapi.rs (kept as historical context)
cargo check --workspace
cargo clippy --workspace --all-targets -- -D warnings
cargo fmt --all -- --check

Scope

Zero behavior change. The hand-written OpenAPI doc and Scalar mount at /admin/openapi.json / /admin/openapi-scalar continue to serve exactly the same content; nothing about how requests flow through aisix-admin is touched.

Refs #304 (#1).

Summary by CodeRabbit

Chores
- Removed OpenAPI documentation-related dependencies from project configuration.

These three workspace dependencies (and their transitives) have zero in-tree usage: - 0 `use utoipa` / `use utoipa_axum` / `use utoipa_scalar` imports - 0 `#[derive(ToSchema)]` / `#[derive(IntoParams)]` derives - The only mention is a doc-comment in `crates/aisix-admin/src/openapi.rs:15-19` explaining why the OpenAPI 3.1 document is hand-written rather than derived — that rationale is preserved. Removing them shrinks the dependency tree (59 lines off Cargo.lock) without any behavior change. Verified with `cargo check --workspace`, `cargo clippy --workspace --all-targets -- -D warnings`, and `cargo fmt --all -- --check`.

coderabbitai · 2026-05-17T00:45:59Z

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: ff8405f2-61c4-4d68-90f6-9da262af5573

📥 Commits

Reviewing files that changed from the base of the PR and between 7e8dab2 and 4b3af76.

⛔ Files ignored due to path filters (1)

Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

💤 Files with no reviewable changes (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

📝 Walkthrough

Walkthrough

OpenAPI-related dependencies (utoipa, utoipa-axum, utoipa-scalar) are removed from the workspace-level dependency definitions and from the aisix-admin crate's dependency list.

Changes

OpenAPI Dependency Removal

Layer / File(s)	Summary
OpenAPI dependency removal `Cargo.toml`, `crates/aisix-admin/Cargo.toml`	OpenAPI crate dependencies are removed from the workspace `[workspace.dependencies]` block and the aisix-admin crate's `[dependencies]` section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Note

🎁 Summarized by CodeRabbit Free

Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

moonming · 2026-05-17T00:45:53Z

Summary

Removes three workspace dependencies that have zero in-tree usage:

utoipa = "5"
utoipa-axum = "0.1"
utoipa-scalar = "0.2"

Why

These were introduced in case the admin OpenAPI 3.1 doc would be derived via utoipa's #[derive(ToSchema)] macros, but crates/aisix-admin/src/openapi.rs:15-19 records the team's decision to keep the spec hand-written:

Small enough that maintaining it by hand is less effort than wiring utoipa derive macros across every handler; the surface is stable enough that drift is easy to spot in review.

The decision has been in effect for some time and the unused deps are pure noise in Cargo.lock (~59 lines of transitive crates). Removing them is a preparatory tidy-up before the upcoming canonical JSON Schema work (#304 item #1), which will follow a different code path (schemars derive on resource structs in aisix-core) and benefit from a clean dependency baseline.

Verification

grep -rE "utoipa|ToSchema|IntoParams" --include="*.rs" crates/ — only the doc-comment match in openapi.rs (kept as historical context)
cargo check --workspace
cargo clippy --workspace --all-targets -- -D warnings
cargo fmt --all -- --check

Scope

Zero behavior change. The hand-written OpenAPI doc and Scalar mount at /admin/openapi.json / /admin/openapi-scalar continue to serve exactly the same content; nothing about how requests flow through aisix-admin is touched.

Refs #304 (#1).

Summary by CodeRabbit

Chores
- Removed OpenAPI documentation-related dependencies from project configuration.

These three workspace dependencies (and their transitives) have zero in-tree usage: - 0 `use utoipa` / `use utoipa_axum` / `use utoipa_scalar` imports - 0 `#[derive(ToSchema)]` / `#[derive(IntoParams)]` derives - The only mention is a doc-comment in `crates/aisix-admin/src/openapi.rs:15-19` explaining why the OpenAPI 3.1 document is hand-written rather than derived — that rationale is preserved. Removing them shrinks the dependency tree (59 lines off Cargo.lock) without any behavior change. Verified with `cargo check --workspace`, `cargo clippy --workspace --all-targets -- -D warnings`, and `cargo fmt --all -- --check`.

coderabbitai · 2026-05-17T00:45:59Z

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: ff8405f2-61c4-4d68-90f6-9da262af5573

📥 Commits

Reviewing files that changed from the base of the PR and between 7e8dab2 and 4b3af76.

⛔ Files ignored due to path filters (1)

Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

💤 Files with no reviewable changes (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

📝 Walkthrough

Walkthrough

OpenAPI-related dependencies (utoipa, utoipa-axum, utoipa-scalar) are removed from the workspace-level dependency definitions and from the aisix-admin crate's dependency list.

Changes

OpenAPI Dependency Removal

Layer / File(s)	Summary
OpenAPI dependency removal `Cargo.toml`, `crates/aisix-admin/Cargo.toml`	OpenAPI crate dependencies are removed from the workspace `[workspace.dependencies]` block and the aisix-admin crate's `[dependencies]` section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Note

🎁 Summarized by CodeRabbit Free

Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

moonming · 2026-05-17T00:45:53Z

Summary

Removes three workspace dependencies that have zero in-tree usage:

utoipa = "5"
utoipa-axum = "0.1"
utoipa-scalar = "0.2"

Why

These were introduced in case the admin OpenAPI 3.1 doc would be derived via utoipa's #[derive(ToSchema)] macros, but crates/aisix-admin/src/openapi.rs:15-19 records the team's decision to keep the spec hand-written:

Small enough that maintaining it by hand is less effort than wiring utoipa derive macros across every handler; the surface is stable enough that drift is easy to spot in review.

The decision has been in effect for some time and the unused deps are pure noise in Cargo.lock (~59 lines of transitive crates). Removing them is a preparatory tidy-up before the upcoming canonical JSON Schema work (#304 item #1), which will follow a different code path (schemars derive on resource structs in aisix-core) and benefit from a clean dependency baseline.

Verification

grep -rE "utoipa|ToSchema|IntoParams" --include="*.rs" crates/ — only the doc-comment match in openapi.rs (kept as historical context)
cargo check --workspace
cargo clippy --workspace --all-targets -- -D warnings
cargo fmt --all -- --check

Scope

Zero behavior change. The hand-written OpenAPI doc and Scalar mount at /admin/openapi.json / /admin/openapi-scalar continue to serve exactly the same content; nothing about how requests flow through aisix-admin is touched.

Refs #304 (#1).

Summary by CodeRabbit

Chores
- Removed OpenAPI documentation-related dependencies from project configuration.

These three workspace dependencies (and their transitives) have zero in-tree usage: - 0 `use utoipa` / `use utoipa_axum` / `use utoipa_scalar` imports - 0 `#[derive(ToSchema)]` / `#[derive(IntoParams)]` derives - The only mention is a doc-comment in `crates/aisix-admin/src/openapi.rs:15-19` explaining why the OpenAPI 3.1 document is hand-written rather than derived — that rationale is preserved. Removing them shrinks the dependency tree (59 lines off Cargo.lock) without any behavior change. Verified with `cargo check --workspace`, `cargo clippy --workspace --all-targets -- -D warnings`, and `cargo fmt --all -- --check`.

coderabbitai · 2026-05-17T00:45:59Z

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: ff8405f2-61c4-4d68-90f6-9da262af5573

📥 Commits

Reviewing files that changed from the base of the PR and between 7e8dab2 and 4b3af76.

⛔ Files ignored due to path filters (1)

Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

💤 Files with no reviewable changes (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

📝 Walkthrough

Walkthrough

OpenAPI-related dependencies (utoipa, utoipa-axum, utoipa-scalar) are removed from the workspace-level dependency definitions and from the aisix-admin crate's dependency list.

Changes

OpenAPI Dependency Removal

Layer / File(s)	Summary
OpenAPI dependency removal `Cargo.toml`, `crates/aisix-admin/Cargo.toml`	OpenAPI crate dependencies are removed from the workspace `[workspace.dependencies]` block and the aisix-admin crate's `[dependencies]` section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Note

🎁 Summarized by CodeRabbit Free

Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

moonming · 2026-05-17T00:45:53Z

Summary

Removes three workspace dependencies that have zero in-tree usage:

utoipa = "5"
utoipa-axum = "0.1"
utoipa-scalar = "0.2"

Why

These were introduced in case the admin OpenAPI 3.1 doc would be derived via utoipa's #[derive(ToSchema)] macros, but crates/aisix-admin/src/openapi.rs:15-19 records the team's decision to keep the spec hand-written:

Small enough that maintaining it by hand is less effort than wiring utoipa derive macros across every handler; the surface is stable enough that drift is easy to spot in review.

The decision has been in effect for some time and the unused deps are pure noise in Cargo.lock (~59 lines of transitive crates). Removing them is a preparatory tidy-up before the upcoming canonical JSON Schema work (#304 item #1), which will follow a different code path (schemars derive on resource structs in aisix-core) and benefit from a clean dependency baseline.

Verification

grep -rE "utoipa|ToSchema|IntoParams" --include="*.rs" crates/ — only the doc-comment match in openapi.rs (kept as historical context)
cargo check --workspace
cargo clippy --workspace --all-targets -- -D warnings
cargo fmt --all -- --check

Scope

Zero behavior change. The hand-written OpenAPI doc and Scalar mount at /admin/openapi.json / /admin/openapi-scalar continue to serve exactly the same content; nothing about how requests flow through aisix-admin is touched.

Refs #304 (#1).

Summary by CodeRabbit

Chores
- Removed OpenAPI documentation-related dependencies from project configuration.

These three workspace dependencies (and their transitives) have zero in-tree usage: - 0 `use utoipa` / `use utoipa_axum` / `use utoipa_scalar` imports - 0 `#[derive(ToSchema)]` / `#[derive(IntoParams)]` derives - The only mention is a doc-comment in `crates/aisix-admin/src/openapi.rs:15-19` explaining why the OpenAPI 3.1 document is hand-written rather than derived — that rationale is preserved. Removing them shrinks the dependency tree (59 lines off Cargo.lock) without any behavior change. Verified with `cargo check --workspace`, `cargo clippy --workspace --all-targets -- -D warnings`, and `cargo fmt --all -- --check`.

coderabbitai · 2026-05-17T00:45:59Z

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: ff8405f2-61c4-4d68-90f6-9da262af5573

📥 Commits

Reviewing files that changed from the base of the PR and between 7e8dab2 and 4b3af76.

⛔ Files ignored due to path filters (1)

Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

💤 Files with no reviewable changes (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

📝 Walkthrough

Walkthrough

OpenAPI-related dependencies (utoipa, utoipa-axum, utoipa-scalar) are removed from the workspace-level dependency definitions and from the aisix-admin crate's dependency list.

Changes

OpenAPI Dependency Removal

Layer / File(s)	Summary
OpenAPI dependency removal `Cargo.toml`, `crates/aisix-admin/Cargo.toml`	OpenAPI crate dependencies are removed from the workspace `[workspace.dependencies]` block and the aisix-admin crate's `[dependencies]` section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Note

🎁 Summarized by CodeRabbit Free

Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

moonming · 2026-05-17T00:45:53Z

Summary

Removes three workspace dependencies that have zero in-tree usage:

utoipa = "5"
utoipa-axum = "0.1"
utoipa-scalar = "0.2"

Why

These were introduced in case the admin OpenAPI 3.1 doc would be derived via utoipa's #[derive(ToSchema)] macros, but crates/aisix-admin/src/openapi.rs:15-19 records the team's decision to keep the spec hand-written:

Small enough that maintaining it by hand is less effort than wiring utoipa derive macros across every handler; the surface is stable enough that drift is easy to spot in review.

The decision has been in effect for some time and the unused deps are pure noise in Cargo.lock (~59 lines of transitive crates). Removing them is a preparatory tidy-up before the upcoming canonical JSON Schema work (#304 item #1), which will follow a different code path (schemars derive on resource structs in aisix-core) and benefit from a clean dependency baseline.

Verification

grep -rE "utoipa|ToSchema|IntoParams" --include="*.rs" crates/ — only the doc-comment match in openapi.rs (kept as historical context)
cargo check --workspace
cargo clippy --workspace --all-targets -- -D warnings
cargo fmt --all -- --check

Scope

Zero behavior change. The hand-written OpenAPI doc and Scalar mount at /admin/openapi.json / /admin/openapi-scalar continue to serve exactly the same content; nothing about how requests flow through aisix-admin is touched.

Refs #304 (#1).

Summary by CodeRabbit

Chores
- Removed OpenAPI documentation-related dependencies from project configuration.

These three workspace dependencies (and their transitives) have zero in-tree usage: - 0 `use utoipa` / `use utoipa_axum` / `use utoipa_scalar` imports - 0 `#[derive(ToSchema)]` / `#[derive(IntoParams)]` derives - The only mention is a doc-comment in `crates/aisix-admin/src/openapi.rs:15-19` explaining why the OpenAPI 3.1 document is hand-written rather than derived — that rationale is preserved. Removing them shrinks the dependency tree (59 lines off Cargo.lock) without any behavior change. Verified with `cargo check --workspace`, `cargo clippy --workspace --all-targets -- -D warnings`, and `cargo fmt --all -- --check`.

coderabbitai · 2026-05-17T00:45:59Z

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: ff8405f2-61c4-4d68-90f6-9da262af5573

📥 Commits

Reviewing files that changed from the base of the PR and between 7e8dab2 and 4b3af76.

⛔ Files ignored due to path filters (1)

Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

💤 Files with no reviewable changes (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

📝 Walkthrough

Walkthrough

OpenAPI-related dependencies (utoipa, utoipa-axum, utoipa-scalar) are removed from the workspace-level dependency definitions and from the aisix-admin crate's dependency list.

Changes

OpenAPI Dependency Removal

Layer / File(s)	Summary
OpenAPI dependency removal `Cargo.toml`, `crates/aisix-admin/Cargo.toml`	OpenAPI crate dependencies are removed from the workspace `[workspace.dependencies]` block and the aisix-admin crate's `[dependencies]` section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Note

🎁 Summarized by CodeRabbit Free

Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

moonming · 2026-05-17T00:45:53Z

Summary

Removes three workspace dependencies that have zero in-tree usage:

utoipa = "5"
utoipa-axum = "0.1"
utoipa-scalar = "0.2"

Why

These were introduced in case the admin OpenAPI 3.1 doc would be derived via utoipa's #[derive(ToSchema)] macros, but crates/aisix-admin/src/openapi.rs:15-19 records the team's decision to keep the spec hand-written:

Small enough that maintaining it by hand is less effort than wiring utoipa derive macros across every handler; the surface is stable enough that drift is easy to spot in review.

The decision has been in effect for some time and the unused deps are pure noise in Cargo.lock (~59 lines of transitive crates). Removing them is a preparatory tidy-up before the upcoming canonical JSON Schema work (#304 item #1), which will follow a different code path (schemars derive on resource structs in aisix-core) and benefit from a clean dependency baseline.

Verification

grep -rE "utoipa|ToSchema|IntoParams" --include="*.rs" crates/ — only the doc-comment match in openapi.rs (kept as historical context)
cargo check --workspace
cargo clippy --workspace --all-targets -- -D warnings
cargo fmt --all -- --check

Scope

Zero behavior change. The hand-written OpenAPI doc and Scalar mount at /admin/openapi.json / /admin/openapi-scalar continue to serve exactly the same content; nothing about how requests flow through aisix-admin is touched.

Refs #304 (#1).

Summary by CodeRabbit

Chores
- Removed OpenAPI documentation-related dependencies from project configuration.

These three workspace dependencies (and their transitives) have zero in-tree usage: - 0 `use utoipa` / `use utoipa_axum` / `use utoipa_scalar` imports - 0 `#[derive(ToSchema)]` / `#[derive(IntoParams)]` derives - The only mention is a doc-comment in `crates/aisix-admin/src/openapi.rs:15-19` explaining why the OpenAPI 3.1 document is hand-written rather than derived — that rationale is preserved. Removing them shrinks the dependency tree (59 lines off Cargo.lock) without any behavior change. Verified with `cargo check --workspace`, `cargo clippy --workspace --all-targets -- -D warnings`, and `cargo fmt --all -- --check`.

coderabbitai · 2026-05-17T00:45:59Z

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: ff8405f2-61c4-4d68-90f6-9da262af5573

📥 Commits

Reviewing files that changed from the base of the PR and between 7e8dab2 and 4b3af76.

⛔ Files ignored due to path filters (1)

Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

💤 Files with no reviewable changes (2)

Cargo.toml
crates/aisix-admin/Cargo.toml

📝 Walkthrough

Walkthrough

OpenAPI-related dependencies (utoipa, utoipa-axum, utoipa-scalar) are removed from the workspace-level dependency definitions and from the aisix-admin crate's dependency list.

Changes

OpenAPI Dependency Removal

Layer / File(s)	Summary
OpenAPI dependency removal `Cargo.toml`, `crates/aisix-admin/Cargo.toml`	OpenAPI crate dependencies are removed from the workspace `[workspace.dependencies]` block and the aisix-admin crate's `[dependencies]` section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Note

🎁 Summarized by CodeRabbit Free

Your organization has reached its limit of developer seats under the Pro Plan. For new users, CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please add seats to your subscription by visiting https://app.coderabbit.ai/login.If you believe this is a mistake and have available seats, please assign one to the pull request author through the subscription management page using the link above.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

CopilotAI review requested due to automatic review settings May 17, 2026 00:45

Copilotstarted reviewing on behalf of moonming May 17, 2026 00:46 View session

CopilotAI reviewed May 17, 2026

moonming merged commit e9a1dca into mainMay 17, 2026
10 of 11 checks passed

moonming mentioned this pull request May 17, 2026
tracking: engineering & docs quality improvements — config schema / provider doc templates / architecture deep-dives / AGENTS.md / concept abstraction #304
Open

jarvis9443 deleted the chore/drop-utoipa-deps branch June 25, 2026 06:25

CopilotAI review requested due to automatic review settings May 17, 2026 00:45

Copilotstarted reviewing on behalf of moonming May 17, 2026 00:46 View session

CopilotAI reviewed May 17, 2026

moonming merged commit e9a1dca into mainMay 17, 2026
10 of 11 checks passed

moonming mentioned this pull request May 17, 2026
tracking: engineering & docs quality improvements — config schema / provider doc templates / architecture deep-dives / AGENTS.md / concept abstraction #304
Open

jarvis9443 deleted the chore/drop-utoipa-deps branch June 25, 2026 06:25

CopilotAI review requested due to automatic review settings May 17, 2026 00:45

Copilotstarted reviewing on behalf of moonming May 17, 2026 00:46 View session

CopilotAI reviewed May 17, 2026

moonming merged commit e9a1dca into mainMay 17, 2026
10 of 11 checks passed

moonming mentioned this pull request May 17, 2026
tracking: engineering & docs quality improvements — config schema / provider doc templates / architecture deep-dives / AGENTS.md / concept abstraction #304
Open

jarvis9443 deleted the chore/drop-utoipa-deps branch June 25, 2026 06:25

CopilotAI review requested due to automatic review settings May 17, 2026 00:45

Copilotstarted reviewing on behalf of moonming May 17, 2026 00:46 View session

CopilotAI reviewed May 17, 2026

moonming merged commit e9a1dca into mainMay 17, 2026
10 of 11 checks passed

moonming mentioned this pull request May 17, 2026
tracking: engineering & docs quality improvements — config schema / provider doc templates / architecture deep-dives / AGENTS.md / concept abstraction #304
Open

jarvis9443 deleted the chore/drop-utoipa-deps branch June 25, 2026 06:25

CopilotAI review requested due to automatic review settings May 17, 2026 00:45

Copilotstarted reviewing on behalf of moonming May 17, 2026 00:46 View session

CopilotAI reviewed May 17, 2026

moonming merged commit e9a1dca into mainMay 17, 2026
10 of 11 checks passed

moonming mentioned this pull request May 17, 2026
tracking: engineering & docs quality improvements — config schema / provider doc templates / architecture deep-dives / AGENTS.md / concept abstraction #304
Open

jarvis9443 deleted the chore/drop-utoipa-deps branch June 25, 2026 06:25

CopilotAI review requested due to automatic review settings May 17, 2026 00:45

Copilotstarted reviewing on behalf of moonming May 17, 2026 00:46 View session

CopilotAI reviewed May 17, 2026

moonming merged commit e9a1dca into mainMay 17, 2026
10 of 11 checks passed

moonming mentioned this pull request May 17, 2026
tracking: engineering & docs quality improvements — config schema / provider doc templates / architecture deep-dives / AGENTS.md / concept abstraction #304
Open

jarvis9443 deleted the chore/drop-utoipa-deps branch June 25, 2026 06:25

CopilotAI review requested due to automatic review settings May 17, 2026 00:45

Copilotstarted reviewing on behalf of moonming May 17, 2026 00:46 View session

CopilotAI reviewed May 17, 2026

moonming merged commit e9a1dca into mainMay 17, 2026
10 of 11 checks passed

moonming mentioned this pull request May 17, 2026
tracking: engineering & docs quality improvements — config schema / provider doc templates / architecture deep-dives / AGENTS.md / concept abstraction #304
Open

jarvis9443 deleted the chore/drop-utoipa-deps branch June 25, 2026 06:25

CopilotAI review requested due to automatic review settings May 17, 2026 00:45

Copilotstarted reviewing on behalf of moonming May 17, 2026 00:46 View session

CopilotAI reviewed May 17, 2026

moonming merged commit e9a1dca into mainMay 17, 2026
10 of 11 checks passed

moonming mentioned this pull request May 17, 2026
tracking: engineering & docs quality improvements — config schema / provider doc templates / architecture deep-dives / AGENTS.md / concept abstraction #304
Open

jarvis9443 deleted the chore/drop-utoipa-deps branch June 25, 2026 06:25

Uh oh!

Conversation

moonming commented May 17, 2026• edited by coderabbitai BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Why

Verification

Scope

Summary by CodeRabbit

Uh oh!

coderabbitaiBot commented May 17, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

moonming commented May 17, 2026• edited by coderabbitai BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Why

Verification

Scope

Summary by CodeRabbit

Uh oh!

coderabbitaiBot commented May 17, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

moonming commented May 17, 2026• edited by coderabbitai BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Why

Verification

Scope

Summary by CodeRabbit

Uh oh!

coderabbitaiBot commented May 17, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

moonming commented May 17, 2026•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented May 17, 2026•
edited
Loading

moonming commented May 17, 2026•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented May 17, 2026•
edited
Loading

moonming commented May 17, 2026•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented May 17, 2026•
edited
Loading

moonming commented May 17, 2026•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented May 17, 2026•
edited
Loading

moonming commented May 17, 2026•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented May 17, 2026•
edited
Loading