Skip to content

refactor: Rename ListCurrentUserPATs to SearchCurrentUserPATs#1476

Merged
AmanGIT07 merged 1 commit intomainfrom
refactor/rename-list-to-search-pat
Mar 25, 2026
Merged

refactor: Rename ListCurrentUserPATs to SearchCurrentUserPATs#1476
AmanGIT07 merged 1 commit intomainfrom
refactor/rename-list-to-search-pat

Conversation

@AmanGIT07
Copy link
Copy Markdown
Contributor

No description provided.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
frontier Ready Ready Preview, Comment Mar 24, 2026 7:25pm

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 586d36dc-431c-4eb1-ba8e-462122eb3624

📥 Commits

Reviewing files that changed from the base of the PR and between 9dba632 and 79b8c62.

⛔ Files ignored due to path filters (3)
  • proto/v1beta1/frontier.pb.go is excluded by !**/*.pb.go, !proto/**
  • proto/v1beta1/frontier.pb.validate.go is excluded by !proto/**
  • proto/v1beta1/frontierv1beta1connect/frontier.connect.go is excluded by !proto/**
📒 Files selected for processing (3)
  • Makefile
  • internal/api/v1beta1connect/user_pat.go
  • pkg/server/connect_interceptors/authorization.go

📝 Walkthrough

Summary by CodeRabbit

  • New Features

    • Personal access token management improved with enhanced search and filtering capabilities for better token discovery and organization.
  • Chores

    • Updated Proton build dependency version for infrastructure improvements.

Walkthrough

The pull request updates the Proton commit version in the Makefile and renames a personal access token handler from ListCurrentUserPATs to SearchCurrentUserPATs, including corresponding updates to request/response types and authorization validation mappings to maintain consistency across the API layer.

Changes

Cohort / File(s) Summary
Build Configuration
Makefile
Updated PROTON_COMMIT variable to reference a new commit hash for proto code generation.
API Handler
internal/api/v1beta1connect/user_pat.go
Renamed ListCurrentUserPATs method to SearchCurrentUserPATs with corresponding request/response type updates (ListCurrentUserPATsRequest/ListCurrentUserPATsResponseSearchCurrentUserPATsRequest/SearchCurrentUserPATsResponse).
Authorization Configuration
pkg/server/connect_interceptors/authorization.go
Updated authorization validation map to replace FrontierServiceListCurrentUserPATsProcedure with FrontierServiceSearchCurrentUserPATsProcedure and updated request type assertion accordingly.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested reviewers

  • rohilsurana

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.

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 23508244632

Details

  • 0 of 5 (0.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 41.058%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/server/connect_interceptors/authorization.go 0 2 0.0%
internal/api/v1beta1connect/user_pat.go 0 3 0.0%
Totals Coverage Status
Change from base Build 23482039206: 0.0%
Covered Lines: 14580
Relevant Lines: 35511

💛 - Coveralls

@AmanGIT07 AmanGIT07 merged commit 993467b into main Mar 25, 2026
8 checks passed
@AmanGIT07 AmanGIT07 deleted the refactor/rename-list-to-search-pat branch March 25, 2026 08:03
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