feat: improve event listeners, expose more types

Author: roncohen

packages/browser-sdk/src/client.ts

@@ -482,6 +482,7 @@ export class BucketClient {
     handler: (args0: HookArgs[THookType]) => void,
   ) {
     this.hooks.addHook(type, handler);
+    return () => this.hooks.removeHook(type, handler);
   }
 
   /**

packages/browser-sdk/src/hooksManager.ts

@@ -13,7 +13,7 @@ export interface HookArgs {
   track: TrackEvent;
 }
 
-type TrackEvent = {
+export type TrackEvent = {
   user: UserContext;
   company?: CompanyContext;
   eventName: string;

packages/browser-sdk/src/index.ts

@@ -1,6 +1,7 @@
 export type { Feature, InitOptions, ToolbarOptions } from "./client";
 export { BucketClient } from "./client";
 export type { BucketContext, CompanyContext, UserContext } from "./context";
+export type { TrackEvent } from "./hooksManager";
 export type {
   CheckEvent,
   FallbackFeatureOverride,

packages/react-sdk/src/index.tsx

@@ -17,8 +17,13 @@ import {
   RawFeatures,
   RequestFeedbackData,
   UnassignedFeedback,
+  TrackEvent,
+  UserContext,
+  CompanyContext,
+  CheckEvent,
 } from "@bucketco/browser-sdk";
 
+export { TrackEvent, UserContext, CompanyContext, CheckEvent, RawFeatures };
 import { version } from "../package.json";
 
 // eslint-disable-next-line @typescript-eslint/no-empty-object-type
@@ -349,9 +354,11 @@ export function useUpdateOtherContext() {
  *
  * ```ts
  * const client = useClient();
- * client.on("configCheck", () => {
- *   console.log("configCheck hook called");
- * });
+ * useEffect(() => {
+ *   return client?.on("enabledCheck", () => {
+ *     console.log("enabledCheck hook called");
+ *   });
+ * }, [client]);
  * ```
  */
 export function useClient() {