FIX: manage_libraryがfalseのときOpenAPIスキーマーの参照がエラーになる問題を修正 (VOICEVOX…

…#1374)

* FIX: `manage_library`が`false`のときOpenAPIスキーマーの参照がエラーになる問題を修正

* FIX: `manage_library`が`False`の場合のみ`BaseLibraryInfo`と`LibrarySpeaker`を足す

* FIX: `manage_library`がTrueの場合のみ追加する

voicevox_engine/app/application.py

@@ -81,6 +81,8 @@ def generate_app(
     )
     app.include_router(generate_portal_page_router(engine_manifest.name))
 
-    app = configure_openapi_schema(app)
+    app = configure_openapi_schema(
+        app, engine_manifest.supported_features.manage_library
+    )
 
     return app

voicevox_engine/app/openapi_schema.py

@@ -4,11 +4,12 @@
 
 from fastapi import FastAPI
 from fastapi.openapi.utils import get_openapi
+from pydantic import BaseModel
 
 from voicevox_engine.library.model import BaseLibraryInfo, VvlibManifest
 
 
-def configure_openapi_schema(app: FastAPI) -> FastAPI:
+def configure_openapi_schema(app: FastAPI, manage_library: bool | None) -> FastAPI:
     """自動生成された OpenAPI schema へカスタム属性を追加する。"""
 
     # BaseLibraryInfo/VvlibManifestモデルはAPIとして表には出ないが、エディタ側で利用したいので、手動で追加する
@@ -27,16 +28,18 @@ def custom_openapi() -> Any:
             contact=app.contact,
             license_info=app.license_info,
         )
-        openapi_schema["components"]["schemas"][
-            "VvlibManifest"
-        ] = VvlibManifest.schema()
-        # ref_templateを指定しない場合、definitionsを参照してしまうので、手動で指定する
-        base_library_info = BaseLibraryInfo.schema(
-            ref_template="#/components/schemas/{model}"
-        )
-        # definitionsは既存のモデルを重複して定義するため、不要なので削除
-        del base_library_info["definitions"]
-        openapi_schema["components"]["schemas"]["BaseLibraryInfo"] = base_library_info
+        if manage_library:
+            additional_models: list[type[BaseModel]] = [
+                BaseLibraryInfo,
+                VvlibManifest,
+            ]
+            for model in additional_models:
+                # ref_templateを指定しない場合、definitionsを参照してしまうので、手動で指定する
+                schema = model.schema(ref_template="#/components/schemas/{model}")
+                # definitionsは既存のモデルを重複して定義するため、不要なので削除
+                if "definitions" in schema:
+                    del schema["definitions"]
+                openapi_schema["components"]["schemas"][schema["title"]] = schema
         app.openapi_schema = openapi_schema
         return openapi_schema