chore: add no-debugger eslint rule (#5906)

diff --git a/.eslintrc.js b/.eslintrc.js
index 71acd8dd898da99a6cb18c15f2456b2dc15a3ad8..5528024779601a451a8834f51f067d397e2aa0e7 100644 (file)
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -7,6 +7,7 @@ module.exports = {
     sourceType: 'module'
   },
   rules: {
+    'no-debugger': 'error',
     'no-unused-vars': [
       'error',
       // we are only using this rule to check for unused arguments since TS

diff --git a/packages/compiler-dom/src/index.ts b/packages/compiler-dom/src/index.ts
index fc1fa040736b854860290cc1b043da00dd57e7b1..2c6f71cefbbb259a81407108599bb10ea1025662 100644 (file)
--- a/packages/compiler-dom/src/index.ts
+++ b/packages/compiler-dom/src/index.ts
@@ -29,7 +29,7 @@ export const DOMNodeTransforms: NodeTransform[] = [
 ]
 
 export const DOMDirectiveTransforms: Record<string, DirectiveTransform> = {
-cloak: noopDirectiveTransform,
+  cloak: noopDirectiveTransform,
   html: transformVHtml,
   text: transformVText,
   model: transformModel, // override compiler-core

diff --git a/packages/runtime-core/__tests__/componentPublicInstance.spec.ts b/packages/runtime-core/__tests__/componentPublicInstance.spec.ts
index 2bea4e0eedc6bf42d3de7212d398e4bf05e00fbb..47dcb03c3ab650bfe5ce34ee2ee76f73ab05afcc 100644 (file)
--- a/packages/runtime-core/__tests__/componentPublicInstance.spec.ts
+++ b/packages/runtime-core/__tests__/componentPublicInstance.spec.ts
@@ -257,7 +257,6 @@ describe('component: proxy', () => {
     expect(instanceProxy.isDisplayed).toBe(true)
   })
 
- 
   test('allow jest spying on proxy methods with Object.defineProperty', () => {
     // #5417
     let instanceProxy: any
@@ -426,7 +425,6 @@ describe('component: proxy', () => {
     expect(instanceProxy.fromProp).toBe(false)
   })
 
-
   // #864
   test('should not warn declared but absent props', () => {
     const Comp = {

diff --git a/packages/runtime-core/__tests__/rendererTemplateRef.spec.ts b/packages/runtime-core/__tests__/rendererTemplateRef.spec.ts
index 4aad6f7edbc705e74eb98143514f5b267d695d36..6a03e7a8eb5fe11bb7f33462e917003a682472a0 100644 (file)
--- a/packages/runtime-core/__tests__/rendererTemplateRef.spec.ts
+++ b/packages/runtime-core/__tests__/rendererTemplateRef.spec.ts
@@ -443,10 +443,8 @@ describe('api: template refs', () => {
     expect(mapRefs()).toMatchObject(['2', '3', '4'])
   })
 
-   
-
   test('named ref in v-for', async () => {
-    const show = ref(true);
+    const show = ref(true)
     const list = reactive([1, 2, 3])
     const listRefs = ref([])
     const mapRefs = () => listRefs.value.map(n => serializeInner(n))
@@ -495,6 +493,4 @@ describe('api: template refs', () => {
     await nextTick()
     expect(mapRefs()).toMatchObject(['2', '3', '4'])
   })
-
-
 })

diff --git a/packages/runtime-core/src/componentProps.ts b/packages/runtime-core/src/componentProps.ts
index 2f3a2fe1452d6a2e816066c91b40d329c6999be4..e046342db3566214efb928593ce56873fc399205 100644 (file)
--- a/packages/runtime-core/src/componentProps.ts
+++ b/packages/runtime-core/src/componentProps.ts
@@ -135,8 +135,8 @@ const enum BooleanFlags {
 
 // extract props which defined with default from prop options
 export type ExtractDefaultPropTypes<O> = O extends object
-  // use `keyof Pick<O, DefaultKeys<O>>` instead of `DefaultKeys<O>` to support IDE features
-  ? { [K in keyof Pick<O, DefaultKeys<O>>]: InferPropType<O[K]> }
+  ? // use `keyof Pick<O, DefaultKeys<O>>` instead of `DefaultKeys<O>` to support IDE features
+    { [K in keyof Pick<O, DefaultKeys<O>>]: InferPropType<O[K]> }
   : {}
 
 type NormalizedProp =
@@ -226,7 +226,7 @@ export function updateProps(
       for (let i = 0; i < propsToUpdate.length; i++) {
         let key = propsToUpdate[i]
         // skip if the prop key is a declared emit event listener
-        if (isEmitListener(instance.emitsOptions, key)){
+        if (isEmitListener(instance.emitsOptions, key)) {
           continue
         }
         // PROPS flag guarantees rawProps to be non-null

diff --git a/packages/runtime-core/src/hydration.ts b/packages/runtime-core/src/hydration.ts
index ae0415aa82aea0f7086c51e3400b34ad1a4938ed..3d86f0f2494b66be5664fbbe19fff484de5e544b 100644 (file)
--- a/packages/runtime-core/src/hydration.ts
+++ b/packages/runtime-core/src/hydration.ts
@@ -113,7 +113,7 @@ export function createHydrationFunctions(
           nextNode = onMismatch()
         } else {
           if ((node as Text).data !== vnode.children) {
-            hasMismatch = true; debugger
+            hasMismatch = true
             __DEV__ &&
               warn(
                 `Hydration text mismatch:` +
@@ -351,7 +351,7 @@ export function createHydrationFunctions(
         )
         let hasWarned = false
         while (next) {
-          hasMismatch = true; debugger
+          hasMismatch = true
           if (__DEV__ && !hasWarned) {
             warn(
               `Hydration children mismatch in <${vnode.type as string}>: ` +
@@ -366,7 +366,7 @@ export function createHydrationFunctions(
         }
       } else if (shapeFlag & ShapeFlags.TEXT_CHILDREN) {
         if (el.textContent !== vnode.children) {
-          hasMismatch = true; debugger
+          hasMismatch = true
           __DEV__ &&
             warn(
               `Hydration text content mismatch in <${
@@ -411,7 +411,7 @@ export function createHydrationFunctions(
       } else if (vnode.type === Text && !vnode.children) {
         continue
       } else {
-        hasMismatch = true; debugger
+        hasMismatch = true
         if (__DEV__ && !hasWarned) {
           warn(
             `Hydration children mismatch in <${container.tagName.toLowerCase()}>: ` +
@@ -465,7 +465,7 @@ export function createHydrationFunctions(
     } else {
       // fragment didn't hydrate successfully, since we didn't get a end anchor
       // back. This should have led to node/children mismatch warnings.
-      hasMismatch = true; debugger
+      hasMismatch = true
       // since the anchor is missing, we need to create one and insert it
       insert((vnode.anchor = createComment(`]`)), container, next)
       return next
@@ -480,7 +480,7 @@ export function createHydrationFunctions(
     slotScopeIds: string[] | null,
     isFragment: boolean
   ): Node | null => {
-    hasMismatch = true; debugger
+    hasMismatch = true
     __DEV__ &&
       warn(
         `Hydration node mismatch:\n- Client vnode:`,

diff --git a/packages/runtime-dom/__tests__/createApp.spec.ts b/packages/runtime-dom/__tests__/createApp.spec.ts
index 86b25f5c21166da1d015e13e1b787fdc36e5d9c4..25ab5c67321126b94dd21ede2d66b2f87f1fcbce 100644 (file)
--- a/packages/runtime-dom/__tests__/createApp.spec.ts
+++ b/packages/runtime-dom/__tests__/createApp.spec.ts
@@ -15,8 +15,7 @@ describe('createApp for dom', () => {
 
   // #4398
   test('should not mutate original root component options object', () => {
-    
-    const originalObj =  {
+    const originalObj = {
       data() {
         return {
           counter: 0
@@ -28,17 +27,16 @@ describe('createApp for dom', () => {
       expect(msg).toMatch(`Component is missing template or render function`)
     })
 
-    const Root = { ...originalObj}
-    
+    const Root = { ...originalObj }
+
     const app = createApp(Root)
     app.config.warnHandler = handler
-    app.mount(document.createElement('div')) 
- 
-    // ensure mount is based on a copy of Root object rather than Root object itself 
+    app.mount(document.createElement('div'))
+
+    // ensure mount is based on a copy of Root object rather than Root object itself
     expect(app._component).not.toBe(Root)
-    
+
     // ensure no mutation happened to Root object
     expect(originalObj).toMatchObject(Root)
-    
   })
 })

diff --git a/packages/server-renderer/src/helpers/ssrRenderAttrs.ts b/packages/server-renderer/src/helpers/ssrRenderAttrs.ts
index 147d08f0588da63de018f28fb9060faa7fa75fb8..746ffa10729db81ac3e18f34d786d35e5ce7fd5c 100644 (file)
--- a/packages/server-renderer/src/helpers/ssrRenderAttrs.ts
+++ b/packages/server-renderer/src/helpers/ssrRenderAttrs.ts
@@ -12,7 +12,9 @@ import {
 } from '@vue/shared'
 
 // leading comma for empty string ""
-const shouldIgnoreProp = makeMap(`,key,ref,innerHTML,textContent,ref_key,ref_for`)
+const shouldIgnoreProp = makeMap(
+  `,key,ref,innerHTML,textContent,ref_key,ref_for`
+)
 
 export function ssrRenderAttrs(
   props: Record<string, unknown>,

diff --git a/packages/shared/src/typeUtils.ts b/packages/shared/src/typeUtils.ts
index 8a56abaa2c9ecd37c812b34ea321afb61e9845b8..8730d7f38bf545c4ef108d2d0ef638f8e252a0e3 100644 (file)
--- a/packages/shared/src/typeUtils.ts
+++ b/packages/shared/src/typeUtils.ts
@@ -7,7 +7,6 @@ export type UnionToIntersection<U> = (
 // make keys required but keep undefined values
 export type LooseRequired<T> = { [P in string & keyof T]: T[P] }
 
-
 // If the the type T accepts type "any", output type Y, otherwise output type N.
 // https://stackoverflow.com/questions/49927523/disallow-call-with-any/49928360#49928360
-export type IfAny<T, Y, N> = 0 extends (1 & T) ? Y : N
+export type IfAny<T, Y, N> = 0 extends 1 & T ? Y : N