[WB-2281] Address feedback

Author: tony-dinh

__docs__/wonder-blocks-timing/use-action-scheduler.mdx

@@ -7,7 +7,7 @@ import {Meta, Canvas} from "@storybook/addon-docs/blocks";
 # `useActionScheduler`
 
 `useActionScheduler` is a hook-based alternative to the `withActionScheduler`
-higher-order component. It returns an [`IScheduleActions`](#ischeduleactions)
+higher-order component. It returns an `IScheduleActions`
 instance that automatically clears all pending actions when the component
 unmounts.
 

__docs__/wonder-blocks-timing/use-animation-frame.mdx

@@ -11,7 +11,7 @@ cancelling animation frames. It is defined as follows:
 
 ```ts
 function useAnimationFrame(
-    action: () => unknown,
+    action: (time: DOMHighResTimeStamp) => unknown,
     options?: {
         schedulePolicy?: SchedulePolicy,
         clearPolicy?: ClearPolicy,

packages/wonder-blocks-timing/src/hooks/__tests__/use-action-scheduler.test.ts

@@ -5,10 +5,10 @@ import {useActionScheduler} from "../use-action-scheduler";
 
 describe("useActionScheduler", () => {
     beforeEach(() => {
-        // Exclude rAF/cAF from sinon's fake timers so our spies wrap jsdom's
+        // Exclude rAF/cAF from jest's fake timers so our spies wrap jsdom's
         // originals. After restoreAllMocks(), RTL auto-cleanup calls jsdom's
         // cancelAnimationFrame (which safely ignores unknown IDs) rather than
-        // sinon's fake (which throws when given a setTimeout ID).
+        // jest's fake (which throws when given a setTimeout ID).
         jest.useFakeTimers({
             doNotFake: ["requestAnimationFrame", "cancelAnimationFrame"],
         });

packages/wonder-blocks-timing/src/hooks/__tests__/use-animation-frame.test.ts

@@ -6,10 +6,10 @@ import {useAnimationFrame} from "../use-animation-frame";
 
 describe("useAnimationFrame", () => {
     beforeEach(() => {
-        // Exclude rAF/cAF from sinon's fake timers so our spies wrap jsdom's
+        // Exclude rAF/cAF from jest's fake timers so our spies wrap jsdom's
         // originals. After restoreAllMocks(), RTL auto-cleanup calls jsdom's
         // cancelAnimationFrame (which safely ignores unknown IDs) rather than
-        // sinon's fake (which throws when given a setTimeout ID).
+        // jest's fake (which throws when given a setTimeout ID).
         jest.useFakeTimers({
             doNotFake: ["requestAnimationFrame", "cancelAnimationFrame"],
         });

packages/wonder-blocks-timing/src/hooks/use-action-scheduler.ts

@@ -29,11 +29,8 @@ import ActionScheduler from "../util/action-scheduler";
 export function useActionScheduler(): IScheduleActions {
     const schedulerRef = useRef<ActionScheduler | null>(null);
 
-    if (schedulerRef.current === null) {
-        schedulerRef.current = new ActionScheduler();
-    }
-
     useEffect(() => {
+        schedulerRef.current = new ActionScheduler();
         return () => {
             schedulerRef.current?.disable();
             schedulerRef.current = null;

packages/wonder-blocks-timing/src/hooks/use-animation-frame.ts

@@ -22,8 +22,9 @@ import AnimationFrame from "../util/animation-frame";
  * @param options.clearPolicy Determines how the request is cleared when the
  * component is unmounted or the request is recreated. By default, the
  * request is cancelled immediately. If you want to let the request run to
- * completion, use `ClearPolicy.Resolve`. This is NOT applied if the request
- * is cleared manually via the `clear()` method on the returned API.
+ * completion, use `ClearPolicy.Resolve`. This policy is also used as the
+ * default when calling `clear()` manually with no argument. Pass an explicit
+ * `ClearPolicy` to `clear(policy)` to override it for a specific call.
  * @param options.schedulePolicy Determines when the request is scheduled.
  * By default, the request is made immediately. If you want to delay
  * scheduling the request, use `SchedulePolicy.OnDemand`.