react-obsidian 2.10.2 → 2.11.0-alpha.1

package/src/injectors/components/ComponentInjector.tsx

@@ -32,7 +32,7 @@ export default class ComponentInjector {
       const proxiedProps = new PropsInjector(graph).inject(passedProps);
 
       return (
-        <GraphContext.Provider value={{injectionToken}}>
+        <GraphContext.Provider value={{ injectionToken }}>
           {Target(proxiedProps as unknown as PropsWithChildren<P>)}
         </GraphContext.Provider>
       );

package/src/injectors/components/InjectComponent.ts

@@ -18,9 +18,9 @@ export const injectComponent = <OwnProps = Discriminator, InjectedProps = Discri
   assertGraph(Graph, Target);
 
   return componentInjector.inject(Target, Graph) as React.FunctionComponent<
-  InjectedProps extends Discriminator ?
-    OwnProps extends Discriminator ? Partial<OwnProps> : OwnProps :
-    OwnProps extends InjectedProps ? Partial<OwnProps> : OwnProps & Partial<InjectedProps>
+    InjectedProps extends Discriminator ?
+      OwnProps extends Discriminator ? Partial<OwnProps> : OwnProps :
+      OwnProps extends InjectedProps ? Partial<OwnProps> : OwnProps & Partial<InjectedProps>
   >;
 };
 function assertGraph(Graph: Constructable<ObjectGraph<unknown>>, Target: any) {

package/src/injectors/components/PropsInjector.ts

@@ -4,7 +4,6 @@ export default class PropsInjector<Props> {
   constructor(private graph: ObjectGraph) {}
 
   inject(passedProps: Props): Partial<Props> {
-    // eslint-disable-next-line prefer-object-spread
     return new Proxy(Object.assign({}, passedProps), {
       get: (target: object, p: string, receiver: any): any => {
         return p in target ? Reflect.get(target, p, receiver) : this.graph.retrieve(p, receiver);

package/src/injectors/components/useGraph.ts

@@ -10,7 +10,6 @@ export default <P>(
   props?: Partial<P>,
   injectionToken?: string,
 ) => {
-
   const [graph] = useState(() => {
     const resolvedGraph = graphRegistry.resolve(Graph, 'lifecycleOwner', props, injectionToken);
     resolvedGraph.onBind(target);
@@ -18,7 +17,7 @@ export default <P>(
   });
   useEffect(() => {
     referenceCounter.retain(graph);
-    return () => referenceCounter.release(graph, (g) => graphRegistry.clear(g));
+    return () => referenceCounter.release(graph, g => graphRegistry.clear(g));
   }, [graph]);
   return graph;
 };

package/src/injectors/hooks/InjectHook.ts

@@ -1,4 +1,3 @@
-/* eslint-disable max-len */
 import { ObjectGraph } from '../../graph/ObjectGraph';
 import { Constructable } from '../../types';
 import HookInjector from './HookInjector';

package/src/observable/Observable.ts

@@ -14,7 +14,7 @@ export class Observable<T> implements IObservable<T> {
 
   public set value(value: T) {
     this.currentValue = value;
-    this.subscribers.forEach((subscriber) => subscriber(value));
+    this.subscribers.forEach(subscriber => subscriber(value));
   }
 
   async first(): Promise<T> {
@@ -35,7 +35,7 @@ export class Observable<T> implements IObservable<T> {
     return () => this.subscribers.delete(onNext);
   }
 
-  public unsubscribe(onNext:OnNext<T>) {
+  public unsubscribe(onNext: OnNext<T>) {
     if (!this.subscribers.has(onNext)) {
       throw new Error(`Can't unsubscribe, subscriber doesn't exist`);
     }

package/src/observable/cold/ColdMediatorObservable.ts

@@ -1,7 +1,6 @@
 import { Observable } from '../Observable';
 import { MediatorObservable } from '../mediator/MediatorObservable';
 import { OnNext } from '../types';
-
 export class ColdMediatorObservable<T extends object> extends MediatorObservable<T> {
   constructor(obj: T, private readonly handler = new PropertyAccessTrackingProxy<T>()) {
     super(new Proxy(obj, handler));

package/src/observable/cold/useColdObservers.ts

@@ -10,7 +10,7 @@ export function useColdObservables<T extends Record<string, any>>(observables: T
   const [values, setValues] = useState(() => mediator.value as ObservedValues<T>);
 
   useEffect(() => {
-    Object.keys(observables as {}).forEach((key) => {
+    Object.keys(observables).forEach((key) => {
       mediator.addSource(observables[key], (value) => {
         mediator.setValue(key, value);
       });

package/src/observable/mediator/MediatorObservable.ts

@@ -67,7 +67,7 @@ export class MediatorObservable<T> extends Observable<T> {
             values[index] = next;
           } else {
             values[index] = next;
-            this.value = mapNext(values, this.value) as T;
+            this.value = mapNext(values, this.value);
           }
         });
       });

package/src/observable/useObserver.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-param-reassign */
 import {
   useCallback,
   useEffect,
@@ -26,6 +25,6 @@ export function useObserver<T>(observableOrGenerator: ObservableOrGenerator<T>):
   return [value, onNext];
 }
 
-function getOrGenerateObservable(observableOrGenerator: ObservableOrGenerator<any>) {
+function getOrGenerateObservable<T>(observableOrGenerator: ObservableOrGenerator<T>): Observable<T> {
   return observableOrGenerator instanceof Observable ? observableOrGenerator : observableOrGenerator();
 }

package/src/observable/useObservers.ts

@@ -2,15 +2,16 @@ import { useEffect, useState } from 'react';
 import { MediatorObservable } from './mediator/MediatorObservable';
 import { ObservedValues, Unsubscribe } from './types';
 import { mapObservablesToValues } from './mapObservablesToValues';
+import type { Observable } from './Observable';
 
-export function useObservers<T extends Record<string, any>>(observables: T): ObservedValues<T> {
+export function useObservers<T extends Record<string, Observable<any>>>(observables: T): ObservedValues<T> {
   const [values, setValues] = useState(() => mapObservablesToValues(observables));
 
   useEffect(() => {
     const mediator = new MediatorObservable();
     const unsubscribers: Unsubscribe[] = [];
 
-    Object.keys(observables as {}).forEach((key) => {
+    Object.keys(observables).forEach((key) => {
       const onNext = (value: any) => setValues({ ...values, [key]: value });
       mediator.addSource(observables[key], onNext);
 
@@ -19,7 +20,7 @@ export function useObservers<T extends Record<string, any>>(observables: T): Obs
       });
     });
 
-    return () => unsubscribers.forEach((unsubscribe) => unsubscribe());
+    return () => unsubscribers.forEach(unsubscribe => unsubscribe());
   }, []);
 
   return values;

package/src/utils/isDev.ts

@@ -3,11 +3,10 @@ export function isDev(): boolean {
 }
 
 function isNodeDev(): boolean {
-  // eslint-disable-next-line @typescript-eslint/dot-notation
   return ['test', 'development'].includes(process.env['NODE_ENV'] ?? '');
 }
 
 function isReactNativeDev(): boolean {
-  // @ts-ignore
+  // @ts-expect-error __DEV__ is a global variable in React Native
   return __DEV__ as boolean ?? false;
 }

package/testkit/index.ts

@@ -7,7 +7,6 @@ class TestKit {
    * @deprecated testKit.mockGraphs is deprecated, use mockGraphs instead
    */
   public mockGraphs(graphNameToGraph: Record<string, Constructable<ObjectGraph> | ((props: any) => ObjectGraph)>) {
-    // eslint-disable-next-line no-console
     console.warn('testKit.mockGraphs is deprecated, use mockGraphs instead');
     return mockGraphs(graphNameToGraph);
   }

package/transformers/babel-plugin-obsidian/helpers/index.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-param-reassign */
 import { types as t } from '@babel/core';
 import {
   CallExpression,
@@ -9,6 +8,7 @@ import {
   ObjectExpression,
   ObjectPattern,
   TSParameterProperty,
+  Node,
 } from '@babel/types';
 
 const never = '';
@@ -39,7 +39,7 @@ export function addNameToProviderArguments(node: ClassMethod, decorator: Decorat
 
 export function getDecoratorArgument(decorator: Decorator): ObjectExpression | undefined {
   if (t.isCallExpression(decorator.expression)) {
-    return decorator.expression.arguments.find((a) => t.isObjectExpression(a)) as ObjectExpression;
+    return decorator.expression.arguments.find(a => t.isObjectExpression(a)) as ObjectExpression;
   }
   return undefined;
 }
@@ -53,17 +53,17 @@ export function getDecoratorByName(
   decorators: Array<Decorator> | undefined | null,
   decoratorName: string,
 ): Decorator | undefined {
-  return decorators?.find((decorator) => get(decorator, 'expression.callee.name') === decoratorName);
+  return decorators?.find(decorator => get(decorator, 'expression.callee.name') === decoratorName);
 }
 
 export function getDecoratorName(decorator?: Decorator): string | undefined {
   return get(decorator, 'expression.callee.name');
 }
 
-export function paramsToDestructuringAssignment(params: (Identifier | any)[]): ObjectPattern {
+export function paramsToDestructuringAssignment(params: Node[]): ObjectPattern {
   return t.objectPattern(params
-    .filter((p) => t.isIdentifier(p))
-    .map((p) => t.objectProperty(t.identifier(p.name), t.identifier(p.name))));
+    .filter(p => t.isIdentifier(p))
+    .map(p => t.objectProperty(t.identifier(p.name), t.identifier(p.name))));
 }
 
 export function passParamNameAsInjectArgument(
@@ -93,9 +93,9 @@ function getNodeName(node: AcceptedNodeType): string {
   return node.name;
 }
 
-function get(node: any, path: string): any {
+function get<T>(node: any, path: string): T | undefined {
   if (node === undefined || node === null) return undefined;
   const [key, ...rest] = path.split('.');
-  if (rest.length === 0) return node[key];
+  if (rest.length === 0) return node[key] as T;
   return get(node[key], rest.join('.'));
 }

package/transformers/babel-plugin-obsidian/index.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-param-reassign */
 import {
   ClassMethod,
   ClassProperty,

package/transformers/babel-plugin-obsidian/unmagler/method.ts

@@ -16,9 +16,8 @@ function saveMethod(name: string, node: ClassMethod) {
 }
 
 function convertProviderParamsToDestructuringAssignment(node: ClassMethod) {
-  if (node.params.length === 0) { return; }
+  if (node.params.length === 0) return;
   const destructuredParams = paramsToDestructuringAssignment(node.params);
-  // eslint-disable-next-line no-param-reassign
   node.params.length = 0;
   node.params.push(destructuredParams);
 }