Yeah this looks like the right approach to me! I think the most react-y way to implement usePromise to actually pay attention to the refetch param would be to use that refetch object in a dependencies array of a useEffect or similar. I didn't try it but I think what you have would work as well.

The current approach won't work, from facebook/react#17526

The result is then cached. This cache is “append-only”: once we put the fetched data into the cache, we never mutate or remove it. (There is a separate mechanism for invalidation in React itself, which amounts to replacing the cache with an empty one — see https://github.com/reactwg/react-18/discussions/25 for an early look.)