refactor: revisit prettier config to be minimal (#816)

* update config

* update snapshot

docs/api/advanced/subscribe.mdx

@@ -18,7 +18,7 @@ const state = proxy({ count: 0 })
 
 // Subscribe to all changes to the state proxy (and its child proxies)
 const unsubscribe = subscribe(state, () =>
-  console.log('state has changed to', state)
+  console.log('state has changed to', state),
 )
 // Unsubscribe by calling the result
 unsubscribe()

docs/api/basic/useSnapshot.mdx

@@ -33,7 +33,8 @@ function Counter() {
           if (state.count < 10) {
             ++state.count
           }
-        }}>
+        }}
+      >
         +1
       </button>
     </div>

docs/api/utils/derive.mdx

@@ -37,7 +37,7 @@ derive(
   },
   {
     proxy: state,
-  }
+  },
 )
 ```
 

docs/api/utils/subscribeKey.mdx

@@ -14,7 +14,7 @@ import { subscribeKey } from 'valtio/utils'
 
 const state = proxy({ count: 0, text: 'hello' })
 subscribeKey(state, 'count', (v) =>
-  console.log('state.count has changed to', v)
+  console.log('state.count has changed to', v),
 )
 ```
 

docs/how-tos/how-to-avoid-rerenders-manually.mdx

@@ -39,7 +39,7 @@ const Component = () => {
           setCount(state.count)
         }
       }),
-    []
+    [],
   )
   return <>{count}</>
 }

docs/how-tos/how-to-persist-states.mdx

@@ -13,7 +13,7 @@ const state = proxy(
   JSON.parse(localStorage.getItem('foo')) || {
     count: 0,
     text: 'hello',
-  }
+  },
 )
 
 subscribe(state, () => {

docs/how-tos/some-gotchas.mdx

@@ -76,7 +76,7 @@ const ChildComponent = React.memo(
     <div>
       {title} - {description}
     </div>
-  )
+  ),
 )
 
 const ParentComponent = () => {

package.json

@@ -81,11 +81,7 @@
   },
   "prettier": {
     "semi": false,
-    "trailingComma": "es5",
-    "singleQuote": true,
-    "bracketSameLine": true,
-    "tabWidth": 2,
-    "printWidth": 80
+    "singleQuote": true
   },
   "repository": {
     "type": "git",

readme.md

@@ -121,7 +121,7 @@ import { subscribe } from 'valtio'
 
 // Subscribe to all state changes
 const unsubscribe = subscribe(state, () =>
-  console.log('state has changed to', state)
+  console.log('state has changed to', state),
 )
 // Unsubscribe by calling the result
 unsubscribe()
@@ -146,7 +146,7 @@ import { subscribeKey } from 'valtio/utils'
 
 const state = proxy({ count: 0, text: 'hello' })
 subscribeKey(state, 'count', (v) =>
-  console.log('state.count has changed to', v)
+  console.log('state.count has changed to', v),
 )
 ```
 

src/macro.ts

@@ -17,7 +17,7 @@ const macro = ({ references }: any) => {
     path.parentPath?.parentPath?.replaceWith(
       t.variableDeclaration('const', [
         t.variableDeclarator(snap, t.callExpression(hook, [proxy])),
-      ])
+      ]),
     )
     let inFunction = 0
     path.parentPath?.getFunctionParent()?.traverse({

src/macro/vite.ts

@@ -26,7 +26,7 @@ export const valtioMacro = defineMacro(`useProxy`)
     path.parentPath?.replaceWith(
       t.variableDeclaration('const', [
         t.variableDeclarator(snap, t.callExpression(hook, [proxy])),
-      ])
+      ]),
     )
 
     let inFunction = 0

src/react.ts

@@ -26,7 +26,7 @@ const { useSyncExternalStore } = useSyncExternalStoreExports
 
 const useAffectedDebugValue = (
   state: object,
-  affected: WeakMap<object, unknown>
+  affected: WeakMap<object, unknown>,
 ) => {
   const pathList = useRef<(string | number | symbol)[][]>()
   useEffect(() => {
@@ -117,7 +117,7 @@ type Options = {
  */
 export function useSnapshot<T extends object>(
   proxyObject: T,
-  options?: Options
+  options?: Options,
 ): Snapshot<T> {
   const notifyInSync = options?.sync
   const lastSnapshot = useRef<Snapshot<T>>()
@@ -130,7 +130,7 @@ export function useSnapshot<T extends object>(
         callback() // Note: do we really need this?
         return unsub
       },
-      [proxyObject, notifyInSync]
+      [proxyObject, notifyInSync],
     ),
     () => {
       const nextSnapshot = snapshot(proxyObject, use)
@@ -143,7 +143,7 @@ export function useSnapshot<T extends object>(
             lastSnapshot.current,
             nextSnapshot,
             lastAffected.current,
-            new WeakMap()
+            new WeakMap(),
           )
         ) {
           // not changed
@@ -154,7 +154,7 @@ export function useSnapshot<T extends object>(
       }
       return nextSnapshot
     },
-    () => snapshot(proxyObject, use)
+    () => snapshot(proxyObject, use),
   )
   inRender = false
   const currAffected = new WeakMap()
@@ -171,6 +171,6 @@ export function useSnapshot<T extends object>(
     currSnapshot,
     currAffected,
     proxyCache,
-    targetCache
+    targetCache,
   )
 }

src/react/utils/useProxy.ts

@@ -24,7 +24,7 @@ import { useSnapshot } from '../../react.ts'
  */
 export function useProxy<T extends object>(
   proxy: T,
-  options?: NonNullable<Parameters<typeof useSnapshot>[1]>
+  options?: NonNullable<Parameters<typeof useSnapshot>[1]>,
 ): T {
   const snapshot = useSnapshot(proxy, options) as T
 

src/vanilla.ts

@@ -50,7 +50,7 @@ type HandlePromise = <P extends Promise<any>>(promise: P) => Awaited<P>
 type CreateSnapshot = <T extends object>(
   target: T,
   version: number,
-  handlePromise?: HandlePromise
+  handlePromise?: HandlePromise,
 ) => T
 
 type RemoveListener = () => void
@@ -91,7 +91,7 @@ const buildProxyFunction = (
       status?: 'pending' | 'fulfilled' | 'rejected'
       value?: Awaited<P>
       reason?: unknown
-    }
+    },
   ) => {
     switch (promise.status) {
       case 'fulfilled':
@@ -108,7 +108,7 @@ const buildProxyFunction = (
   createSnapshot: CreateSnapshot = <T extends object>(
     target: T,
     version: number,
-    handlePromise: HandlePromise = defaultHandlePromise
+    handlePromise: HandlePromise = defaultHandlePromise,
   ): T => {
     const cache = snapCache.get(target)
     if (cache?.[0] === version) {
@@ -127,7 +127,7 @@ const buildProxyFunction = (
       const value = Reflect.get(target, key)
       const { enumerable } = Reflect.getOwnPropertyDescriptor(
         target,
-        key
+        key,
       ) as PropertyDescriptor
       const desc: PropertyDescriptor = {
         value,
@@ -143,12 +143,12 @@ const buildProxyFunction = (
         desc.get = () => handlePromise(value)
       } else if (proxyStateMap.has(value as object)) {
         const [target, ensureVersion] = proxyStateMap.get(
-          value as object
+          value as object,
         ) as ProxyState
         desc.value = createSnapshot(
           target,
           ensureVersion(),
-          handlePromise
+          handlePromise,
         ) as Snapshot<T>
       }
       Object.defineProperty(snap, key, desc)
@@ -202,7 +202,7 @@ const buildProxyFunction = (
     >()
     const addPropListener = (
       prop: string | symbol,
-      propProxyState: ProxyState
+      propProxyState: ProxyState,
     ) => {
       if (import.meta.env?.MODE !== 'production' && propProxyStates.has(prop)) {
         throw new Error('prop listener already exists')
@@ -313,7 +313,7 @@ const buildProxyFunction = (
     Reflect.ownKeys(initialObject).forEach((key) => {
       const desc = Object.getOwnPropertyDescriptor(
         initialObject,
-        key
+        key,
       ) as PropertyDescriptor
       if ('value' in desc) {
         proxyObject[key as keyof T] = initialObject[key as keyof T]
@@ -325,7 +325,7 @@ const buildProxyFunction = (
       Object.defineProperty(baseObject, key, desc)
     })
     return proxyObject
-  }
+  },
 ) =>
   [
     // public functions
@@ -358,7 +358,7 @@ export function getVersion(proxyObject: unknown): number | undefined {
 export function subscribe<T extends object>(
   proxyObject: T,
   callback: (ops: Op[]) => void,
-  notifyInSync?: boolean
+  notifyInSync?: boolean,
 ): () => void {
   const proxyState = proxyStateMap.get(proxyObject as object)
   if (import.meta.env?.MODE !== 'production' && !proxyState) {
@@ -393,7 +393,7 @@ export function subscribe<T extends object>(
 
 export function snapshot<T extends object>(
   proxyObject: T,
-  handlePromise?: HandlePromise
+  handlePromise?: HandlePromise,
 ): Snapshot<T> {
   const proxyState = proxyStateMap.get(proxyObject as object)
   if (import.meta.env?.MODE !== 'production' && !proxyState) {

src/vanilla/utils/addComputed.ts

@@ -10,11 +10,11 @@ export function addComputed_DEPRECATED<T extends object, U extends object>(
   computedFns_FAKE: {
     [K in keyof U]: (snap_FAKE: T) => U[K]
   },
-  targetObject: any = proxyObject
+  targetObject: any = proxyObject,
 ) {
   if (import.meta.env?.MODE !== 'production') {
     console.warn(
-      'addComputed is deprecated. Please consider using `derive`. Falling back to emulation with derive. https://github.com/pmndrs/valtio/pull/201'
+      'addComputed is deprecated. Please consider using `derive`. Falling back to emulation with derive. https://github.com/pmndrs/valtio/pull/201',
     )
   }
   const derivedFns: {

src/vanilla/utils/devtools.ts

@@ -23,15 +23,15 @@ type Options = {
 
 export function devtools<T extends object>(
   proxyObject: T,
-  options?: Options
+  options?: Options,
 ): (() => void) | undefined
 
 /**
  * @deprecated Please use { name } option
  */
 export function devtools<T extends object>(
   proxyObject: T,
-  name?: string
+  name?: string,
 ): (() => void) | undefined
 
 /**
@@ -47,11 +47,11 @@ export function devtools<T extends object>(
  */
 export function devtools<T extends object>(
   proxyObject: T,
-  options?: Options | string
+  options?: Options | string,
 ) {
   if (typeof options === 'string') {
     console.warn(
-      'string name option is deprecated, use { name }. https://github.com/pmndrs/valtio/pull/400'
+      'string name option is deprecated, use { name }. https://github.com/pmndrs/valtio/pull/400',
     )
     options = { name: options }
   }
@@ -94,15 +94,15 @@ export function devtools<T extends object>(
           type: action,
           updatedAt: new Date().toLocaleString(),
         } as any,
-        snapWithoutDevtools
+        snapWithoutDevtools,
       )
     }
   })
   const unsub2 = (
     devtools as unknown as {
       // FIXME https://github.com/reduxjs/redux-devtools/issues/1097
       subscribe: (
-        listener: (message: Message) => void
+        listener: (message: Message) => void,
       ) => (() => void) | undefined
     }
   ).subscribe((message) => {
@@ -112,7 +112,7 @@ export function devtools<T extends object>(
       } catch (e) {
         console.error(
           'please dispatch a serializable value that JSON.parse() and proxy() support\n',
-          e
+          e,
         )
       }
     }

src/vanilla/utils/proxyMap.ts

@@ -36,7 +36,7 @@ type InternalProxyMap<K, V> = Map<K, V> & {
  * state.get(key) //undefined
  */
 export function proxyMap<K, V>(
-  entries?: Iterable<readonly [K, V]> | null
+  entries?: Iterable<readonly [K, V]> | null,
 ): Map<K, V> {
   const map: InternalProxyMap<K, V> = proxy({
     data: Array.from(entries || []) as KeyValRecord<K, V>[],

src/vanilla/utils/proxyWithComputed.ts

@@ -18,11 +18,11 @@ export function proxyWithComputed_DEPRECATED<
           get: (snap: Snapshot<T>) => U[K]
           set?: (state: T, newValue: U[K]) => void
         }
-  }
+  },
 ) {
   if (import.meta.env?.MODE !== 'production') {
     console.warn(
-      'proxyWithComputed is deprecated. Please follow "Computed Properties" guide in docs.'
+      'proxyWithComputed is deprecated. Please follow "Computed Properties" guide in docs.',
     )
   }
   ;(Object.keys(computedFns) as (keyof U)[]).forEach((key) => {

src/vanilla/utils/proxyWithHistory.ts

@@ -67,7 +67,7 @@ export function proxyWithHistory<V>(initialValue: V, skipSubscribe = false) {
     undo: () => {
       if (proxyObject.canUndo()) {
         proxyObject.value = (proxyObject.history.wip = proxyObject.clone(
-          proxyObject.history.snapshots[--proxyObject.history.index]
+          proxyObject.history.snapshots[--proxyObject.history.index],
         ) as Snapshot<V>) as V
       }
     },
@@ -76,7 +76,7 @@ export function proxyWithHistory<V>(initialValue: V, skipSubscribe = false) {
     redo: () => {
       if (proxyObject.canRedo()) {
         proxyObject.value = (proxyObject.history.wip = proxyObject.clone(
-          proxyObject.history.snapshots[++proxyObject.history.index]
+          proxyObject.history.snapshots[++proxyObject.history.index],
         ) as Snapshot<V>) as V
       }
     },
@@ -91,7 +91,7 @@ export function proxyWithHistory<V>(initialValue: V, skipSubscribe = false) {
           ops.every(
             (op) =>
               op[1][0] === 'value' &&
-              (op[0] !== 'set' || op[2] !== proxyObject.history.wip)
+              (op[0] !== 'set' || op[2] !== proxyObject.history.wip),
           )
         ) {
           proxyObject.saveHistory()