#14 Fixing the issue for key encoding

sdk/node/CHANGELOG.md

@@ -5,6 +5,9 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.0.1] - 2025-04-10
+### Added
+- Fixed the issue with encoding keys for range and delete queries
 
 ## [1.0.0] - 2025-04-05
 - First stable release

sdk/node/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hpkv/rest-client",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "A NodeJS REST client for high-performance key-value store (HPKV)",
   "main": "dist/src/index.js",
   "types": "dist/src/index.d.ts",

sdk/node/src/client.ts

@@ -178,7 +178,10 @@ export class HPKVRestClient {
    */
   async delete(key: string): Promise<RecordResponse> {
     try {
-      return await this._request<RecordResponse>("DELETE", `/record/${key}`);
+      return await this._request<RecordResponse>(
+        "DELETE",
+        `/record/${encodeURIComponent(key)}`,
+      );
     } catch (error) {
       this._handleError(error as ApiError);
     }
@@ -242,8 +245,8 @@ export class HPKVRestClient {
   ): Promise<RangeResponse> {
     try {
       const params = new URLSearchParams({
-        startKey,
-        endKey,
+        startKey: encodeURIComponent(startKey),
+        endKey: encodeURIComponent(endKey),
         limit: limit.toString(),
       });
       return await this._request<RangeResponse>("GET", `/records?${params}`);

sdk/node/tests/integration.test.ts

@@ -196,6 +196,35 @@ describe("HPKVClient Integration Tests", () => {
       expect(result.value).toBe(value);
     });
 
+    test("should delete keys with special characters", async () => {
+      const key = getTestKey("special-delete:!@#$%^&*()_+");
+      testKeys.push(key);
+      const value = "Value to be deleted";
+
+      await client.set(key, value);
+      await client.delete(key);
+      await expect(client.get(key)).rejects.toThrow();
+    });
+
+    test("should handle range queries with special characters in keys", async () => {
+      const specialKeys = [
+        getTestKey("special-range:!@#$%^&*()_+:1"),
+        getTestKey("special-range:!@#$%^&*()_+:2"),
+        getTestKey("special-range:!@#$%^&*()_+:3"),
+      ];
+
+      for (const key of specialKeys) {
+        testKeys.push(key);
+        await client.set(key, `Value for ${key}`);
+      }
+
+      const results = await client.range(specialKeys[0], specialKeys[2], 10);
+
+      expect(results.records).toHaveLength(3);
+      expect(results.truncated).toBe(false);
+      expect(results.records.map((r) => r.key)).toEqual(specialKeys);
+    });
+
     test("should throw when value size exceeded", async () => {
       const key = getTestKey("large");
       testKeys.push(key);