Skip to content

Reshuffle files - #219

Closed
renal128 wants to merge 1 commit into
remove-global-configfrom
reshuffle-files
Closed

Reshuffle files#219
renal128 wants to merge 1 commit into
remove-global-configfrom
reshuffle-files

Conversation

@renal128

@renal128 renal128 commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Note

Low Risk
Visibility-only change to an internal auth helper; low risk unless external clients depended on the previously public TokenService API.

Overview
Narrows the SDK surface by making TokenService and its initializer, debugApiKey, and fetchToken module-internal instead of public.

Behavior of token resolution is unchanged; callers are expected to authenticate through the existing conversation APIs rather than using TokenService directly.

Reviewed by Cursor Bugbot for commit 744a9f9. Bugbot is set up for automated code reviews on this repo. Configure here.

@renal128
renal128 changed the base branch from main to remove-feedback-availability-callback July 25, 2026 12:51
@renal128
renal128 changed the base branch from remove-feedback-availability-callback to remove-global-config July 25, 2026 15:27
This was referenced Jul 25, 2026
@renal128
renal128 marked this pull request as ready for review July 25, 2026 18:17
@renal128
renal128 force-pushed the remove-global-config branch from aa03bf9 to b74665e Compare July 26, 2026 07:25
@renal128
renal128 force-pushed the reshuffle-files branch 2 times, most recently from e4ff330 to f45d809 Compare July 26, 2026 16:30
@renal128
renal128 force-pushed the remove-global-config branch from b74665e to 0d9f489 Compare July 26, 2026 16:30
@renal128
renal128 force-pushed the remove-global-config branch 2 times, most recently from 1077062 to 56af297 Compare July 26, 2026 16:59
Co-authored-by: Cursor <cursoragent@cursor.com>
@renal128

Copy link
Copy Markdown
Contributor Author

Folded into #223 (base retargeted to remove-global-config) so the intermediate file moves and the public-type layout cleanup review as one PR. Closing without merging.

@renal128 renal128 closed this Jul 26, 2026
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.

1 participant