Address review comment

Author: mattrunyon

packages/jsapi-bootstrap/src/useDeferredApi.ts

@@ -26,9 +26,9 @@ export const DeferredApiContext = createContext<
  */
 export function useDeferredApi(
   widget: DhType.ide.VariableDescriptor | UriVariableDescriptor | null
-): [dh: typeof DhType | null, error: unknown | null] {
+): [dh: typeof DhType | null, error: NonNullable<unknown> | null] {
   const [api, setApi] = useState<typeof DhType | null>(null);
-  const [error, setError] = useState<unknown | null>(null);
+  const [error, setError] = useState<NonNullable<unknown> | null>(null);
   const deferredApi = useContext(DeferredApiContext);
   const contextApi = useContext(ApiContext);
 
@@ -63,7 +63,7 @@ export function useDeferredApi(
             setError(null);
           }
         } catch (e) {
-          if (!isCancelled) {
+          if (!isCancelled && e != null) {
             setApi(null);
             setError(e);
           }

packages/jsapi-bootstrap/src/useWidget.ts

@@ -112,8 +112,7 @@ export function useWidget<T extends WidgetTypes = dh.Widget>(
   const wrapperWithApi = useMemo(
     () => ({
       ...wrapper,
-      error:
-        wrapper.error ?? (descriptorApiError as NonNullable<unknown> | null),
+      error: wrapper.error ?? descriptorApiError,
       api: wrapper.widget != null ? api : null,
     }),
     [wrapper, api, descriptorApiError]