fix: address code review feedback

- Remove void workaround in MemoryQueueAdapter, use underscore prefix
- Remove void workaround in RedisCacheAdapter
- Fix ROADMAP.md count inconsistency (15→12 spec-only contracts)

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

ROADMAP.md

@@ -61,7 +61,7 @@ the ecosystem for enterprise workloads.
 
 ### What Needs Building
 
-15 of 25 service contracts are specification-only (no runtime implementation).
+12 of 25 service contracts are specification-only (no runtime implementation).
 These are the backbone of ObjectStack's enterprise capabilities.
 
 ---

packages/services/service-cache/src/redis-cache-adapter.ts

@@ -42,7 +42,6 @@ export class RedisCacheAdapter implements ICacheService {
   }
 
   async set<T = unknown>(_key: string, _value: T, _ttl?: number): Promise<void> {
-    void this.defaultTtl;
     throw new Error('RedisCacheAdapter not yet implemented');
   }
 

packages/services/service-queue/src/memory-queue-adapter.ts

@@ -71,9 +71,8 @@ export class MemoryQueueAdapter implements IQueueService {
     this.handlers.delete(queue);
   }
 
-  async getQueueSize(queue: string): Promise<number> {
-    // In-memory: no persistent queue, count dead letters for the queue
-    void queue;
+  async getQueueSize(_queue: string): Promise<number> {
+    // In-memory: no persistent queue depth tracking
     return 0;
   }