Skip to content

SNOW-3055677 Implement SESSION_ID_AS_STRING capability#1345

Closed
sfc-gh-rsavenok wants to merge 1 commit into
rsavenok/SNOW-3055676-remove-smkids-hacksfrom
rsavenok/SNOW-3055677-session-id-as-string
Closed

SNOW-3055677 Implement SESSION_ID_AS_STRING capability#1345
sfc-gh-rsavenok wants to merge 1 commit into
rsavenok/SNOW-3055676-remove-smkids-hacksfrom
rsavenok/SNOW-3055677-session-id-as-string

Conversation

@sfc-gh-rsavenok
Copy link
Copy Markdown
Collaborator

@sfc-gh-rsavenok sfc-gh-rsavenok commented Mar 19, 2026

Description

Send SESSION_ID_AS_STRING: true in CLIENT_CAPABILITIES during login so the server returns sessionId as a string, avoiding precision loss with large numeric IDs. Updates JSDoc type annotations in sf.js and queryContextCache.js to reflect the new string type.

sessionId is only used for logging—no functional changes.

Checklist

  • Create tests which fail without the change (if possible)
  • Make all tests (unit and integration) pass (npm run test:unit and npm run test:integration)
  • Extend the types in index.d.ts file (if necessary)
  • Extend the README / documentation and ensure is properly displayed (if necessary)
  • Provide JIRA issue id (if possible) or GitHub issue id in commit message

Request sessionId as a string from the server to avoid precision loss
with large numeric session IDs. Update JSDoc annotations accordingly.
Copy link
Copy Markdown
Collaborator Author

sfc-gh-rsavenok commented Mar 19, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions Bot locked and limited conversation to collaborators Apr 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant