Skip to content

fix(frontier): remove leftover grpc-gateway annotations from PAT model#445

Merged
AmanGIT07 merged 1 commit intomainfrom
fix/remove-gateway-annotations-pat
Mar 6, 2026
Merged

fix(frontier): remove leftover grpc-gateway annotations from PAT model#445
AmanGIT07 merged 1 commit intomainfrom
fix/remove-gateway-annotations-pat

Conversation

@AmanGIT07
Copy link
Contributor

@AmanGIT07 AmanGIT07 commented Mar 6, 2026

Summary

Test plan

  • Verify proto files compile without errors
  • Confirm no remaining grpc-gateway references in frontier proto files

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Refactor
    • Simplified Personal Access Token timestamp field declarations by removing OpenAPI gateway metadata annotations while preserving core field functionality.

The CreateCurrentUserPAT PR (#443) was merged with openapiv2 annotations
that were removed in #444. This cleans up the remaining annotations.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Mar 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f708be6b-f093-4147-a8c2-cc72b53bea9b

📥 Commits

Reviewing files that changed from the base of the PR and between e7f6f21 and 3d48a81.

📒 Files selected for processing (1)
  • raystack/frontier/v1beta1/models.proto

📝 Walkthrough

Walkthrough

This PR simplifies timestamp field declarations in the PAT message by removing grpc-gateway OpenAPI metadata annotations from four fields (expires_at, last_used_at, created_at, updated_at) while preserving OUTPUT_ONLY field behavior annotations where applicable.

Changes

Cohort / File(s) Summary
Timestamp Fields Cleanup
raystack/frontier/v1beta1/models.proto
Removed grpc-gateway OpenAPI field annotations from four timestamp fields (expires_at, last_used_at, created_at, updated_at) in PAT message, reducing verbose descriptive metadata while preserving OUTPUT_ONLY field behavior annotations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • rohilsurana

Poem

🐰 hops with delight
Timestamps trimmed of their metadata dress,
Simpler declarations, nothing less,
Four fields cleaned with a gentle touch,
Protobuf files thank you so much! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: removing grpc-gateway annotations from the PAT model in the frontier package.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/remove-gateway-annotations-pat

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.

❤️ Share

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

@AmanGIT07 AmanGIT07 merged commit e5878c3 into main Mar 6, 2026
1 of 4 checks passed
@AmanGIT07 AmanGIT07 deleted the fix/remove-gateway-annotations-pat branch March 6, 2026 08:22
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.

2 participants