Skip to content

fix: refresh active sessions after revocation - #915

Merged
Topmatrixmor2014 merged 1 commit into
FinChippay:mainfrom
dollarop:fix/session-list-refresh-908
Aug 26, 2026
Merged

fix: refresh active sessions after revocation#915
Topmatrixmor2014 merged 1 commit into
FinChippay:mainfrom
dollarop:fix/session-list-refresh-908

Conversation

@dollarop

Copy link
Copy Markdown
Contributor

Closes #908

Summary

  • re-fetch active sessions after successful single-session revocation
  • re-fetch after revoking all other sessions
  • keep the affected action disabled while the authoritative refresh is in flight
  • add regression coverage for both revoke flows

Validation

  • npm test -- --runTestsByPath __tests__/SessionManager.test.tsx (2 tests passed)
  • npx eslint components/SessionManager.tsx
  • git diff --check

The repository-wide type-check currently reports numerous unrelated errors already present on upstream main; the new test's local SessionInfo fixtures were corrected and the focused suite passes.

@github-actions github-actions Bot added the needs-review PR ready for Greptile AI code review label Aug 26, 2026
@github-actions

Copy link
Copy Markdown

🤖 Greptile AI Code Review

Greptile will automatically review this PR (2 file(s) changed).

Review gates:

  • ✅ CodeQL Security Scan
  • ✅ Custom rules (.greptile/config.json)
  • ✅ Architecture guidelines (.greptile/rules.md)

To manually trigger a re-review, comment @greptileai on this PR.
To skip review, add the skip-review label.

@Topmatrixmor2014
Topmatrixmor2014 merged commit 3475314 into FinChippay:main Aug 26, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review PR ready for Greptile AI code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

#131 — / Don't Refresh the Session List After Revoking a Session

2 participants