Tidy up formatting

babel.config.js

@@ -18,4 +18,4 @@ module.exports = {
     '@babel/preset-typescript',
   ],
   plugins: ['@babel/plugin-transform-runtime'],
-};
+}

design-system/packages/core/src/utils.ts

@@ -129,7 +129,6 @@ export const useId = (idFromProps?: string | null) => {
     if (id === null) {
       setId(genId())
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [])
 
   useEffect(() => {

design-system/packages/fields/src/DatePicker/utils/getContrastText.ts

@@ -4,7 +4,7 @@ export function getContrastText (color: string) {
   const [r, g, b] = hexToTriplet(color)
 
   // calculate contrast against grayscale
-  var contrast = (Math.round(r * 299) + Math.round(g * 587) + Math.round(b * 114)) / 1000
+  let contrast = (Math.round(r * 299) + Math.round(g * 587) + Math.round(b * 114)) / 1000
 
   return contrast >= 128 ? 'black' : 'white'
 }

design-system/packages/fields/src/DatePicker/utils/hexToRgb.ts

@@ -14,7 +14,7 @@ function parseHex (hex: string) {
   return result
 }
 
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
+ 
 export function hexToTriplet (dirtyHex: string, alpha?: number) {
   const cleanHex = parseHex(dirtyHex)
 

design-system/packages/icons/build-icons.js

@@ -1,7 +1,6 @@
 const path = require('node:path')
 const fs = require('node:fs/promises')
 
-/* eslint-disable import/no-extraneous-dependencies */
 const svgr = require('@svgr/core').default
 const { icons } = require('feather-icons')
 const toPascalCase = require('to-pascal-case')

design-system/packages/modals/src/drawer-context.tsx

@@ -47,7 +47,6 @@ export const useDrawerManager = (uniqueKey: string) => {
     return () => {
       modalState.popFromDrawerStack()
     }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [])
   // the last key in the array is the "top" modal visually, so the depth is the inverse index
   // be careful not to mutate the stack

design-system/packages/toast/src/Toast.tsx

@@ -127,7 +127,6 @@ export const ToastElement = forwardRef<HTMLDivElement, ToastElementProps>((props
     }
     // this is not like other components because the consumer cannot update the props once they `addToast()`
     // we intentionally only want this to be run when the toast element mounts/unmounts
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [])
 
   const iconElement = {

docs/components/docs/DocumentEditorDemo.tsx

@@ -90,6 +90,7 @@ const componentBlocks = {
   carousel: componentBlocksInSandboxProject.carousel,
 }
 
+// eslint-disable-next-line @typescript-eslint/consistent-type-imports
 type DocumentFieldConfig = Parameters<typeof import('@keystone-6/fields-document').document>[0]
 
 function documentFeaturesCodeExample (config: DocumentFieldConfig | DocumentFeatures) {
@@ -292,7 +293,6 @@ export const DocumentEditorDemo = () => {
     // slate looks like it's a controlled component but it actually isn't
     // so we need to re-mount it so that it looks at the updated value
     setKey(x => x + 1)
-    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [documentFeatures])
 
   return (

docs/components/primitives/Emoji.tsx

@@ -43,7 +43,6 @@ export function Emoji ({ symbol, alt, ...props }: EmojiProps) {
   const posRef = useRef<HTMLElement>(null)
   const [showOnTop, setShownTop] = useState(true)
 
-  // eslint-disable-next-line react-hooks/exhaustive-deps
   useEffect(() => {
     if (posRef.current && posRef.current.offsetTop - window.pageYOffset < 50) {
       setShownTop(false)

docs/next.config.mjs

@@ -1,6 +1,6 @@
 // you don't need this if you're building something outside of the Keystone repo
-import withPreconstruct from '@preconstruct/next';
-import redirects from './redirects.mjs';
+import withPreconstruct from '@preconstruct/next'
+import redirects from './redirects.mjs'
 
 export default withPreconstruct({
   env: {
@@ -15,4 +15,4 @@ export default withPreconstruct({
     ignoreBuildErrors: true,
   },
   redirects,
-});
+})

docs/pages/updates/index.tsx

@@ -560,9 +560,9 @@ export default function WhatsNew () {
         </Box> */}
         <Timeline date="16th September 2021" />
         <Box heading="create-keystone-app now uses SQLite">
-          Our CLI app now uses <InlineCode>SQLite</InlineCode> under the hood so you don’t have to
+          Our CLI app now uses <InlineCode>SQLite</InlineCode> under the hood so you don’t have to
           spend time on DB config when trying out new ideas. We also updated the{' '}
-          <Link href="/docs/getting-started">getting started walkthrough</Link> to reflect this
+          <Link href="/docs/getting-started">getting started walkthrough</Link> to reflect this
           improvement.
         </Box>
         <Timeline date="15th September 2021" />

docs/redirects.mjs

@@ -25,7 +25,7 @@ const KEYSTONE_4 = [
   //   destination: 'https://v4.keystonejs.com/getting-started',
   //   permanent: true,
   // },
-];
+]
 
 /* URLs from v5.keystonejs.com */
 const KEYSTONE_5 = [
@@ -56,7 +56,7 @@ const KEYSTONE_5 = [
     destination: '/docs',
     permanent: false,
   },
-];
+]
 
 /* URLs from the original next.keystonejs.com website */
 const ORIGINAL_NEXT = [
@@ -95,7 +95,7 @@ const ORIGINAL_NEXT = [
   },
   { source: '/roadmap', destination: '/updates/roadmap', permanent: true },
   { source: '/whats-new', destination: '/updates/whats-new-in-v6', permanent: true },
-];
+]
 
 /* Splitbee Proxy */
 const SPLITBEE = [
@@ -109,7 +109,7 @@ const SPLITBEE = [
     destination: 'https://hive.splitbee.io/:slug',
     permanent: false,
   },
-];
+]
 
 /* Current website redirections */
 const CURRENT = [
@@ -274,14 +274,14 @@ const CURRENT = [
     destination: '/blog/embedded-mode-with-sqlite-nextjs',
     permanent: true,
   },
-];
+]
 
-export default async function redirects() {
+export default async function redirects () {
   return [
     ...SPLITBEE,
     ...CURRENT,
     ...ORIGINAL_NEXT,
     ...KEYSTONE_5,
     ...KEYSTONE_4
-  ];
+  ]
 }

eslint.config.js

@@ -9,7 +9,8 @@ module.exports = tseslint.config(
       '**/.keystone/',
       '**/dist/',
       '**/node_modules/',
-      '**/syntax-error.js'
+      '**/syntax-error.js',
+      '**/public/'
     ],
   },
   eslint.configs.recommended,
@@ -19,6 +20,7 @@ module.exports = tseslint.config(
     rules: {
       // TODO: remove
       'no-empty': 'off',
+      'no-empty-pattern': ['error', { allowObjectPatternsAsParameters: true }],
       'no-extra-boolean-cast': 'off',
       'no-async-promise-executor': 'off',
       '@typescript-eslint/no-non-null-asserted-optional-chain': 'off',
@@ -37,6 +39,7 @@ module.exports = tseslint.config(
       // TODO: remove
 
       semi: ['error', 'never'],
+      quotes: ['error', 'single', { allowTemplateLiterals: true, avoidEscape: true }],
       'block-spacing': ['error', 'always'],
       'brace-style': ['error', '1tbs', { allowSingleLine: true }],
       'comma-spacing': ['error', { before: false, after: true }],

examples/custom-field/1-text-field/index.ts

@@ -26,7 +26,7 @@ export function text<ListTypeInfo extends BaseListTypeInfo> ({
       input: {
         create: {
           arg: graphql.arg({ type: graphql.String }),
-          // eslint-disable-next-line @typescript-eslint/no-unused-vars
+           
           resolve (value, context) {
             return value
           },
@@ -36,7 +36,7 @@ export function text<ListTypeInfo extends BaseListTypeInfo> ({
       },
       output: graphql.field({
         type: graphql.String,
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
+         
         resolve ({ value, item }, args, context, info) {
           return value
         },

examples/custom-field/2-stars-field/index.ts

@@ -52,7 +52,7 @@ export const stars =
           // this field type doesn't need to do anything special
           // but field types can specify resolvers for inputs like they can for their output GraphQL field
           // this function can be omitted, it is here purely to show how you could change it
-          // eslint-disable-next-line @typescript-eslint/no-unused-vars
+           
           resolve (val, context) {
             // if it's null, then the value will be set to null in the database
             if (val === null) {
@@ -77,7 +77,7 @@ export const stars =
         type: graphql.Int,
         // like the input resolvers, providing the resolver is unnecessary if you're just returning the value
         // it is shown here to show what you could do
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
+         
         resolve ({ value, item }, args, context, info) {
           return value
         },

examples/custom-field/4-conditional-field/index.tsx

@@ -31,7 +31,7 @@ export function feedback<ListTypeInfo extends BaseListTypeInfo> ({
       input: {
         create: {
           arg: graphql.arg({ type: graphql.String }),
-          // eslint-disable-next-line @typescript-eslint/no-unused-vars
+           
           resolve (value, context) {
             return value
           },
@@ -41,7 +41,7 @@ export function feedback<ListTypeInfo extends BaseListTypeInfo> ({
       },
       output: graphql.field({
         type: graphql.String,
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
+         
         resolve ({ value, item }, args, context, info) {
           return value
         },

examples/extend-graphql-schema-nexus/nexus-types.ts

@@ -4,7 +4,7 @@
  */
 
 
-import type { Context } from "./keystone-types"
+import type { Context } from './keystone-types'
 
 
 
@@ -164,15 +164,15 @@ export interface NexusGenInputs {
 }
 
 export interface NexusGenEnums {
-  KeystoneAdminUIFieldMetaCreateViewFieldMode: "edit" | "hidden"
-  KeystoneAdminUIFieldMetaIsNonNull: "create" | "read" | "update"
-  KeystoneAdminUIFieldMetaItemViewFieldMode: "edit" | "hidden" | "read"
-  KeystoneAdminUIFieldMetaItemViewFieldPosition: "form" | "sidebar"
-  KeystoneAdminUIFieldMetaListViewFieldMode: "hidden" | "read"
-  KeystoneAdminUISortDirection: "ASC" | "DESC"
-  OrderDirection: "asc" | "desc"
-  PostStatusType: "draft" | "published"
-  QueryMode: "default" | "insensitive"
+  KeystoneAdminUIFieldMetaCreateViewFieldMode: 'edit' | 'hidden'
+  KeystoneAdminUIFieldMetaIsNonNull: 'create' | 'read' | 'update'
+  KeystoneAdminUIFieldMetaItemViewFieldMode: 'edit' | 'hidden' | 'read'
+  KeystoneAdminUIFieldMetaItemViewFieldPosition: 'form' | 'sidebar'
+  KeystoneAdminUIFieldMetaListViewFieldMode: 'hidden' | 'read'
+  KeystoneAdminUISortDirection: 'ASC' | 'DESC'
+  OrderDirection: 'asc' | 'desc'
+  PostStatusType: 'draft' | 'published'
+  QueryMode: 'default' | 'insensitive'
 }
 
 export interface NexusGenScalars {

examples/extend-graphql-schema-nexus/schema.ts

@@ -95,6 +95,7 @@ export function extendGraphqlSchema (baseSchema: GraphQLSchema) {
 
     // Typescript output settings, probably something you might commit in dev
     //   TODO: remove false ??, it is not part of the example, but we are having monorepo issues
+    // eslint-disable-next-line no-constant-binary-expression
     shouldGenerateArtifacts: false ?? process.env.NODE_ENV !== 'production',
     outputs: {
       typegen: path.join(process.cwd(), 'nexus-types.ts'),

examples/extend-graphql-subscriptions/websocket.ts

@@ -8,7 +8,7 @@ import { type Context } from '.keystone/types'
 
 // Setup pubsub as a Global variable in dev so it survives Hot Reloads.
 declare global {
-  var graphqlSubscriptionPubSub: PubSub
+  let graphqlSubscriptionPubSub: PubSub
 }
 
 // The 'graphql-subscriptions' pubsub library is not recommended for production use, but can be useful as an example

examples/framework-astro/astro.config.mjs

@@ -1,4 +1,4 @@
-import { defineConfig } from 'astro/config';
+import { defineConfig } from 'astro/config'
 
 // https://astro.build/config
 export default defineConfig({
@@ -9,4 +9,4 @@ export default defineConfig({
       external: ['@keystone-6/core/context', '@keystone-6/core', '@keystone-6/core/fields'],
     },
   },
-});
+})

examples/framework-astro/src/keystone/context.ts

@@ -1,7 +1,6 @@
 import { getContext } from '@keystone-6/core/context'
 // Using myprisma/client as vite does not like .
 // This should be @prisma/client in a project outside of the Keystone monorepo
-// eslint-disable-next-line import/no-extraneous-dependencies
 import * as PrismaModule from 'myprisma/client'
 import config from '../../keystone'
 import type { Context } from '.keystone/types'

examples/framework-nextjs-app-directory/next.config.mjs

@@ -1,5 +1,5 @@
 // you don't need this if you're building something outside of the Keystone repo
-import withPreconstruct from '@preconstruct/next';
+import withPreconstruct from '@preconstruct/next'
 
 export default withPreconstruct({
   experimental: {
@@ -12,4 +12,4 @@ export default withPreconstruct({
   typescript: {
     ignoreBuildErrors: true,
   },
-});
+})

examples/graphql-ts-gql/schema.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/consistent-type-imports */
 import { list, graphql } from '@keystone-6/core'
 import { select, relationship, text, timestamp, virtual } from '@keystone-6/core/fields'
 import { allowAll } from '@keystone-6/core/access'

packages/fields-document/src/DocumentEditor/editor-shared.ts

@@ -136,10 +136,10 @@ export function createDocumentEditor (
   componentBlocks: Record<string, ComponentBlock>,
   relationships: Relationships,
   slate?: {
-    withReact: (editor: Editor) => any,
+    withReact: (editor: Editor) => any
     ReactEditor: {
-      focus: (editor: Editor) => void,
-    },
+      focus: (editor: Editor) => void
+    }
   },
 ) {
   return withPasting(

packages/fields-document/src/DocumentEditor/utils.ts

@@ -79,8 +79,8 @@ export function insertNodesButReplaceIfSelectionIsAtEmptyParagraphOrHeading (
   nodes: Node | Node[],
   slate?: {
     ReactEditor: {
-      focus: (editor: Editor) => void,
-    },
+      focus: (editor: Editor) => void
+    }
   },
 ) {
   let pathRefForEmptyNodeAtCursor: PathRef | undefined

packages/fields-document/src/index.ts

@@ -71,7 +71,7 @@ export type DocumentFieldConfig<ListTypeInfo extends BaseListTypeInfo> =
     db?: { map?: string, extendPrismaSchema?: (field: string) => string }
   }
 
-export function document <ListTypeInfo extends BaseListTypeInfo>({
+export function document <ListTypeInfo extends BaseListTypeInfo> ({
   componentBlocks = {},
   dividers,
   formatting,

scripts/prepare-release.js

@@ -1,8 +1,6 @@
 const { spawnSync } = require('child_process')
 const { readFileSync, writeFileSync } = require('fs')
-//  eslint-disable-next-line import/no-extraneous-dependencies
 const { default: getReleasePlan } = require('@changesets/get-release-plan')
-//  eslint-disable-next-line import/no-extraneous-dependencies
 const { getInfo } = require('@changesets/get-github-info')
 
 // TODO: move this to CI linting