Merge pull request #943 from pmcelhaney/server-context-registry-find-fix

fix(server): Update context find logic in context-registry

.changeset/wet-pumas-wash.md

@@ -0,0 +1,5 @@
+---
+"counterfact": patch
+---
+
+Update context-registry find logic for bug with mixed path casing

src/server/context-registry.ts

@@ -21,13 +21,26 @@ export class ContextRegistry {
     this.add("/", {});
   }
 
+  private getContextIgnoreCase(map: Map<string, Context>, key: string) {
+    const lowerCaseKey = key.toLowerCase();
+    for (const currentKey of map.keys()) {
+      if (currentKey.toLowerCase() === lowerCaseKey) {
+        return map.get(currentKey);
+      }
+    }
+    return undefined;
+  }
+
   public add(path: string, context: Context): void {
     this.entries.set(path, context);
     this.cache.set(path, structuredClone(context));
   }
 
   public find(path: string): Context {
-    return this.entries.get(path) ?? this.find(parentPath(path));
+    return (
+      this.getContextIgnoreCase(this.entries, path) ??
+      this.find(parentPath(path))
+    );
   }
 
   // eslint-disable-next-line max-statements

test/server/context-registry.test.ts

@@ -25,6 +25,16 @@ describe("a context registry", () => {
     expect(registry.find("/hello/world")).toBe(helloContext);
   });
 
+  it("finds a context at a parent path with alternate casing", () => {
+    const helloContext = { name: "hello" };
+
+    const registry = new ContextRegistry();
+
+    registry.add("/hello", helloContext);
+
+    expect(registry.find("/Hello/world")).toBe(helloContext);
+  });
+
   it("returns an empty object when there is no matching context", () => {
     const helloContext = { name: "hello" };