medplum / medplum / 23865486177

92%

main: 92%

push

github

Add `enabled` option to `useSearch` hooks (#8838)

* Add `enabled` option to useSearch hooks

This allows callers to defer a search until a dependency is ready (e.g.
waiting on another query's result), working around React's rules of
hooks which prohibit conditional hook calls.

Signed-off-by: Noah Silas <noah@medplum.com>

* Add useDebouncedValue tests

This hook only had implicit test coverage. Adding explicit test coverage
before I make changes.

Signed-off-by: Noah Silas <noah@medplum.com>

* Reset useDebouncedValue `leading` trigger more consistently

When the "leading" edge is fired, it sets the cooldown ref but doesn't
start a timer to clear it. This means that the cooldown after the first
"leading" edge was effectively infinite.

This meant that there were some scenarios, such as our "useSearch"
hooks, where updates can be less frequent than expected, which is
especially unfortunate in the context of the medplum client
auto-batcher; if some searches don't fire at the right time, they won't
get batched with other queries as expected, leading to extra network
traffic.

Signed-off-by: Noah Silas <noah@medplum.com>

* [autofix.ci] apply automated fixes

---------

Signed-off-by: Noah Silas <noah@medplum.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

17980 of 20415 branches covered (88.07%)

Branch coverage included in aggregate %.

7 of 7 new or added lines in 2 files covered. (100.0%)

32665 of 34505 relevant lines covered (94.67%)

14527.81 hits per line