Skip to content

Node.js: Add RESET command support#5945

Open
affonsov wants to merge 4 commits into
mainfrom
affonsov/reset-node
Open

Node.js: Add RESET command support#5945
affonsov wants to merge 4 commits into
mainfrom
affonsov/reset-node

Conversation

@affonsov
Copy link
Copy Markdown
Collaborator

@affonsov affonsov commented May 13, 2026

Summary

Adds support for the RESET command to the Node.js client. RESET resets the connection state.

Issue link

This Pull Request is linked to issue: Implement RESET command for Python, Node.js, Go, and Java clients
Closes #5942

Features / Behaviour Changes

  • createReset() factory added to Commands.ts
  • reset() added to BaseClient, available on both GlideClient and GlideClusterClient
  • reset() added to BaseBatch

Implementation

  • Sends RequestType.Reset with no arguments; returns "RESET" on success

Limitations

None.

Testing

Checklist

  • This Pull Request is related to one issue.
  • Commit message has a detailed description of what changed and why.
  • Tests are added or updated.
  • CHANGELOG.md and documentation files are updated.
  • Linters have been run and Prettier has been run.
  • Destination branch is correct - main or release
  • Create merge commit if merging release branch into main, squash otherwise.

@affonsov affonsov force-pushed the affonsov/reset-node branch 2 times, most recently from ec17782 to 77503a1 Compare May 13, 2026 20:40
@affonsov affonsov marked this pull request as ready for review May 14, 2026 22:01
@affonsov affonsov requested a review from a team as a code owner May 14, 2026 22:01
@alexr-bq
Copy link
Copy Markdown
Collaborator

Changelog?

Copy link
Copy Markdown
Collaborator

@xShinnRyuu xShinnRyuu left a comment

Choose a reason for hiding this comment

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

Everything looks good to me besides adding CHANGELOG entry. Please update change log before merging

affonsov added 3 commits May 21, 2026 17:17
Signed-off-by: affonsov <67347924+affonsov@users.noreply.github.com>
Signed-off-by: affonsov <67347924+affonsov@users.noreply.github.com>
Signed-off-by: affonsov <67347924+affonsov@users.noreply.github.com>
@affonsov affonsov force-pushed the affonsov/reset-node branch from 6803aca to 18db692 Compare May 22, 2026 00:17
Signed-off-by: affonsov <67347924+affonsov@users.noreply.github.com>
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.

Implement RESET command for Python, Node.js, Go, and Java clients

3 participants