Skip to content

chore(sdk): upgrade apsara to 1.0.0-rc.9, PAT table fix#1651

Merged
paanSinghCoder merged 4 commits into
mainfrom
pat-sdk-fixes-2
May 27, 2026
Merged

chore(sdk): upgrade apsara to 1.0.0-rc.9, PAT table fix#1651
paanSinghCoder merged 4 commits into
mainfrom
pat-sdk-fixes-2

Conversation

@rohilsurana
Copy link
Copy Markdown
Member

@rohilsurana rohilsurana commented May 26, 2026

Summary

Test plan

  • Navigate to any detail page (PAT, project, team, service account) → verify breadcrumb shows pointer cursor on hover
  • Click breadcrumb → verify correct navigation back to list page
  • Verify unused PATs show NA in the "Last used" column
  • Verify no regressions from apsara upgrade

@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

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

Project Deployment Actions Updated (UTC)
frontier Ready Ready Preview, Comment May 27, 2026 7:00am

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 26, 2026

Review Change Stack

Warning

Review limit reached

@rohilsurana, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 11 minutes and 56 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 1f8d9a1f-2931-45a0-917f-79967d76d2ad

📥 Commits

Reviewing files that changed from the base of the PR and between b05dcdc and eb3dfa1.

📒 Files selected for processing (1)
  • web/sdk/react/views-new/pat/components/pat-columns.tsx
📝 Walkthrough

Walkthrough

Updates the @raystack/apsara-v1 dependency in web/sdk/package.json from version 1.0.0-rc.8 to 1.0.0-rc.9. No functional code changes.

Changes

web/sdk dependency update

Layer / File(s) Summary
Update @raystack/apsara-v1 dependency
web/sdk/package.json
The @raystack/apsara-v1 dependency version is bumped from 1.0.0-rc.8 to 1.0.0-rc.9.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • raystack/frontier#1428: Both PRs only bump the @raystack/apsara-related dependency version in web/.../package.json with no other code changes.
  • raystack/frontier#1433: Both PRs update the @raystack/apsara dependency in web/sdk/package.json version.

Suggested reviewers

  • paanSinghCoder
  • rohanchkrabrty
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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


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

coveralls commented May 26, 2026

Coverage Report for CI Build 26496052436

Coverage remained the same at 42.847%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 37846
Covered Lines: 16216
Line Coverage: 42.85%
Coverage Strength: 12.0 hits per line

💛 - Coveralls

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
web/sdk/react/views-new/teams/team-details-view.tsx (1)

272-272: 💤 Low value

Consider consolidating repeated inline styles.

The style={{ cursor: 'pointer' }} pattern is repeated identically across four detail views (PAT, Project, Service Account, and Team). While the current approach works, consider these alternatives if this pattern grows:

  • Extract to a shared CSS class (e.g., .breadcrumb-clickable)
  • Create a reusable ClickableBreadcrumbItem wrapper component
  • Request the apsara library to handle cursor styling internally for breadcrumb items with onClick

For now, the inline style is acceptable for this small change.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 081bef8c-6819-49b4-8c62-ab6e413807a0

📥 Commits

Reviewing files that changed from the base of the PR and between f1d968b and 5c9dd53.

📒 Files selected for processing (4)
  • web/sdk/react/views-new/pat/pat-details-view.tsx
  • web/sdk/react/views-new/projects/project-details-view.tsx
  • web/sdk/react/views-new/service-accounts/service-account-details-view.tsx
  • web/sdk/react/views-new/teams/team-details-view.tsx

@rohilsurana rohilsurana changed the title fix(sdk): add cursor pointer to breadcrumb items without href fix(sdk): add cursor pointer to breadcrumb items, upgrade apsara to 1.0.0-rc.9 May 27, 2026
@rohilsurana rohilsurana changed the title fix(sdk): add cursor pointer to breadcrumb items, upgrade apsara to 1.0.0-rc.9 chore(sdk): upgrade apsara to 1.0.0-rc.9 May 27, 2026
@rohilsurana rohilsurana changed the title chore(sdk): upgrade apsara to 1.0.0-rc.9 chore(sdk): upgrade apsara to 1.0.0-rc.9, PAT table fix May 27, 2026
@paanSinghCoder paanSinghCoder merged commit e399d9c into main May 27, 2026
8 checks passed
@paanSinghCoder paanSinghCoder deleted the pat-sdk-fixes-2 branch May 27, 2026 07:06
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