fix(react-keytips): handle KEYTIP_REMOVED/UPDATED in useKeytipsManager

change/@fluentui-contrib-react-keytips-e09c85ce-05b9-4911-9379-5a37ee4d4f68.json

@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "fix(react-keytips): handle KEYTIP_REMOVED/UPDATED in useKeytipsManager",
+  "packageName": "@fluentui-contrib/react-keytips",
+  "email": "rwinkelman@users.noreply.github.com",
+  "dependentChangeType": "patch"
+}

packages/react-keytips/src/hooks/useKeytipsManager.test.ts

@@ -0,0 +1,49 @@
+import { useKeytipsManager } from './useKeytipsManager';
+import { renderHook } from '@testing-library/react';
+
+jest.mock('@fluentui/react-components', () => {
+  return {
+    useId: () => 'keytip-id',
+    useFluent: jest.fn().mockReturnValue({
+      targetDocument: document,
+    }),
+  };
+});
+
+describe('useKeytipsManager', () => {
+  it('keeps registered keytips in sync through update and removal', () => {
+    const { result } = renderHook(() => useKeytipsManager());
+
+    const uniqueId = result.current.register({
+      content: 'A',
+      keySequences: ['a'],
+    });
+
+    expect(result.current.getKeytips()[uniqueId]).toEqual(
+      expect.objectContaining({
+        uniqueId,
+        content: 'A',
+        keySequences: ['a'],
+      })
+    );
+
+    result.current.update({
+      uniqueId,
+      content: 'B',
+      keySequences: ['b'],
+    });
+
+    expect(result.current.getKeytips()[uniqueId]).toEqual(
+      expect.objectContaining({
+        uniqueId,
+        content: 'B',
+        keySequences: ['b'],
+      })
+    );
+    expect(result.current.getKeytips()[uniqueId].content).toBe('B');
+
+    result.current.unregister(uniqueId);
+
+    expect(result.current.getKeytips()[uniqueId]).toBeUndefined();
+  });
+});

packages/react-keytips/src/hooks/useKeytipsManager.ts

@@ -23,6 +23,14 @@ export function useKeytipsManager(): {
       keytips.current[keytip.uniqueId] = keytip;
     });
 
+    subscribe(EVENTS.KEYTIP_UPDATED, (keytip) => {
+      keytips.current[keytip.uniqueId] = keytip;
+    });
+
+    subscribe(EVENTS.KEYTIP_REMOVED, (uniqueId) => {
+      delete keytips.current[uniqueId];
+    });
+
     subscribe(EVENTS.ENTER_KEYTIP_MODE, () => {
       isKeytipModeEnabled.current = true;
     });