feat: ログインユーザーのセッション情報を取得する GET /me エンドポイントを新設#35
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughOpenAPI 3.1.0仕様を更新し、ユーザーセッション情報を取得する ChangesAPI仕様とサーバーコード生成の統一化
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
openapi.yaml (1)
16-17: ⚡ Quick winoperation の
descriptionを追加して lint 警告を解消してくださいSpectral 警告どおり、主要 operation の説明が欠けています。API ドキュメント品質とレビュー効率のため、
descriptionを追加するのがよいです。Also applies to: 42-43, 133-134, 227-228
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@openapi.yaml` around lines 16 - 17, Add a descriptive "description" field to each OpenAPI operation that only has a "summary" (e.g., the POST operation with summary "isct メールアドレスの確認メールを送信" and the other operations referenced at 42-43, 133-134, 227-228) to satisfy Spectral linting; the description should be a short sentence (in Japanese to match existing text) explaining the operation’s intent, expected input and key side effects (e.g., "指定したメールアドレスに確認メールを送信します。リクエストはメールアドレスを含む JSON ボディを期待し、成功時に 202 を返します。"), and add similar concise descriptions to the other operations with missing descriptions so each operation object includes both summary and description.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@openapi.yaml`:
- Around line 715-730: The schema for the /me response uses OpenAPI 3.1 where
nullable: true is not supported; update the properties email, traq_id, and
customer_id in the /me schema to use JSON Schema union types instead of nullable
(e.g., set their type to ["string","null"]), leaving is_admin as a plain
boolean; ensure you modify the properties named email, traq_id, and customer_id
in the /me response object so generated clients will correctly accept null
values.
---
Nitpick comments:
In `@openapi.yaml`:
- Around line 16-17: Add a descriptive "description" field to each OpenAPI
operation that only has a "summary" (e.g., the POST operation with summary "isct
メールアドレスの確認メールを送信" and the other operations referenced at 42-43, 133-134,
227-228) to satisfy Spectral linting; the description should be a short sentence
(in Japanese to match existing text) explaining the operation’s intent, expected
input and key side effects (e.g., "指定したメールアドレスに確認メールを送信します。リクエストはメールアドレスを含む JSON
ボディを期待し、成功時に 202 を返します。"), and add similar concise descriptions to the other
operations with missing descriptions so each operation object includes both
summary and description.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 5e4817c8-d3a6-461a-903f-e599ccc58de9
📒 Files selected for processing (2)
openapi.yamlserver/gen.go
Close #34
Summary by CodeRabbit
リリースノート
New Features
/meエンドポイントを追加しました。Bug Fixes
__Host-checkin_session→__Host-checkin_email_verified_session)。Changes
/customer→/customers、/invoices、/checkout/sessions)。