fix: remove non-null assertions to fix lint warnings

Author: jonathanpeterwu

src/hooks/events.ts

@@ -75,7 +75,10 @@ export class HookEventEmitter extends EventEmitter {
     if (!this.handlers.has(eventType)) {
       this.handlers.set(eventType, new Set());
     }
-    this.handlers.get(eventType)!.add(handler);
+    const handlers = this.handlers.get(eventType);
+    if (handlers) {
+      handlers.add(handler);
+    }
     this.on(eventType, handler);
   }
 

src/skills/api-discovery.ts

@@ -274,8 +274,9 @@ export class APIDiscoverySkill {
     }
 
     // Check if already discovered
-    if (this.discoveredAPIs.has(discovered.name)) {
-      return this.discoveredAPIs.get(discovered.name)!;
+    const existing = this.discoveredAPIs.get(discovered.name);
+    if (existing) {
+      return existing;
     }
 
     // Only probe for spec if it's not a known service (known services already have spec URLs)