@ttsc/lint 0.7.1 → 0.7.3

package/plugin/rules_gap.go

@@ -8,17 +8,17 @@ type noEmptyStaticBlock struct{}
 
 func (noEmptyStaticBlock) Name() string { return "no-empty-static-block" }
 func (noEmptyStaticBlock) Visits() []shimast.Kind {
-	return []shimast.Kind{shimast.KindClassStaticBlockDeclaration}
+  return []shimast.Kind{shimast.KindClassStaticBlockDeclaration}
 }
 func (noEmptyStaticBlock) Check(ctx *Context, node *shimast.Node) {
-	block := node.AsClassStaticBlockDeclaration()
-	if block == nil || block.Body == nil {
-		return
-	}
-	body := block.Body.AsBlock()
-	if body == nil || body.Statements == nil || len(body.Statements.Nodes) == 0 {
-		ctx.Report(node, "Unexpected empty static block.")
-	}
+  block := node.AsClassStaticBlockDeclaration()
+  if block == nil || block.Body == nil {
+    return
+  }
+  body := block.Body.AsBlock()
+  if body == nil || body.Statements == nil || len(body.Statements.Nodes) == 0 {
+    ctx.Report(node, "Unexpected empty static block.")
+  }
 }
 
 // no-setter-return: setters must not return a value.
@@ -28,23 +28,23 @@ type noSetterReturn struct{}
 func (noSetterReturn) Name() string           { return "no-setter-return" }
 func (noSetterReturn) Visits() []shimast.Kind { return []shimast.Kind{shimast.KindReturnStatement} }
 func (noSetterReturn) Check(ctx *Context, node *shimast.Node) {
-	ret := node.AsReturnStatement()
-	if ret == nil || ret.Expression == nil || !isInsideDirectSetter(node) {
-		return
-	}
-	ctx.Report(node, "Setter should not return a value.")
+  ret := node.AsReturnStatement()
+  if ret == nil || ret.Expression == nil || !isInsideDirectSetter(node) {
+    return
+  }
+  ctx.Report(node, "Setter should not return a value.")
 }
 
 func isInsideDirectSetter(node *shimast.Node) bool {
-	for p := node.Parent; p != nil; p = p.Parent {
-		if p.Kind == shimast.KindSetAccessor {
-			return true
-		}
-		if isFunctionLikeKind(p) {
-			return false
-		}
-	}
-	return false
+  for p := node.Parent; p != nil; p = p.Parent {
+    if p.Kind == shimast.KindSetAccessor {
+      return true
+    }
+    if isFunctionLikeKind(p) {
+      return false
+    }
+  }
+  return false
 }
 
 // no-unused-labels: a label is useful only when a break/continue targets it.
@@ -54,31 +54,31 @@ type noUnusedLabels struct{}
 func (noUnusedLabels) Name() string           { return "no-unused-labels" }
 func (noUnusedLabels) Visits() []shimast.Kind { return []shimast.Kind{shimast.KindLabeledStatement} }
 func (noUnusedLabels) Check(ctx *Context, node *shimast.Node) {
-	stmt := node.AsLabeledStatement()
-	if stmt == nil || stmt.Label == nil {
-		return
-	}
-	label := identifierText(stmt.Label)
-	if label == "" {
-		return
-	}
-	used := false
-	walkDescendants(stmt.Statement, func(child *shimast.Node) {
-		if used || child == nil {
-			return
-		}
-		switch child.Kind {
-		case shimast.KindBreakStatement:
-			br := child.AsBreakStatement()
-			used = br != nil && identifierText(br.Label) == label
-		case shimast.KindContinueStatement:
-			cont := child.AsContinueStatement()
-			used = cont != nil && identifierText(cont.Label) == label
-		}
-	})
-	if !used {
-		ctx.Report(stmt.Label, "Label '"+label+"' is defined but never used.")
-	}
+  stmt := node.AsLabeledStatement()
+  if stmt == nil || stmt.Label == nil {
+    return
+  }
+  label := identifierText(stmt.Label)
+  if label == "" {
+    return
+  }
+  used := false
+  walkDescendants(stmt.Statement, func(child *shimast.Node) {
+    if used || child == nil {
+      return
+    }
+    switch child.Kind {
+    case shimast.KindBreakStatement:
+      br := child.AsBreakStatement()
+      used = br != nil && identifierText(br.Label) == label
+    case shimast.KindContinueStatement:
+      cont := child.AsContinueStatement()
+      used = cont != nil && identifierText(cont.Label) == label
+    }
+  })
+  if !used {
+    ctx.Report(stmt.Label, "Label '"+label+"' is defined but never used.")
+  }
 }
 
 // no-dynamic-delete: avoid deleting properties through dynamic keys.
@@ -88,29 +88,29 @@ type noDynamicDelete struct{}
 func (noDynamicDelete) Name() string           { return "no-dynamic-delete" }
 func (noDynamicDelete) Visits() []shimast.Kind { return []shimast.Kind{shimast.KindDeleteExpression} }
 func (noDynamicDelete) Check(ctx *Context, node *shimast.Node) {
-	del := node.AsDeleteExpression()
-	if del == nil || del.Expression == nil || del.Expression.Kind != shimast.KindElementAccessExpression {
-		return
-	}
-	access := del.Expression.AsElementAccessExpression()
-	if access == nil || isStaticPropertyKey(access.ArgumentExpression) {
-		return
-	}
-	ctx.Report(node, "Do not delete dynamically computed property keys.")
+  del := node.AsDeleteExpression()
+  if del == nil || del.Expression == nil || del.Expression.Kind != shimast.KindElementAccessExpression {
+    return
+  }
+  access := del.Expression.AsElementAccessExpression()
+  if access == nil || isStaticPropertyKey(access.ArgumentExpression) {
+    return
+  }
+  ctx.Report(node, "Do not delete dynamically computed property keys.")
 }
 
 func isStaticPropertyKey(node *shimast.Node) bool {
-	node = stripParens(node)
-	if node == nil {
-		return false
-	}
-	switch node.Kind {
-	case shimast.KindStringLiteral,
-		shimast.KindNoSubstitutionTemplateLiteral,
-		shimast.KindNumericLiteral:
-		return true
-	}
-	return false
+  node = stripParens(node)
+  if node == nil {
+    return false
+  }
+  switch node.Kind {
+  case shimast.KindStringLiteral,
+    shimast.KindNoSubstitutionTemplateLiteral,
+    shimast.KindNumericLiteral:
+    return true
+  }
+  return false
 }
 
 // no-non-null-asserted-nullish-coalescing: `foo! ?? bar` is contradictory.
@@ -118,22 +118,22 @@ func isStaticPropertyKey(node *shimast.Node) bool {
 type noNonNullAssertedNullishCoalescing struct{}
 
 func (noNonNullAssertedNullishCoalescing) Name() string {
-	return "no-non-null-asserted-nullish-coalescing"
+  return "no-non-null-asserted-nullish-coalescing"
 }
 func (noNonNullAssertedNullishCoalescing) Visits() []shimast.Kind {
-	return []shimast.Kind{shimast.KindBinaryExpression}
+  return []shimast.Kind{shimast.KindBinaryExpression}
 }
 func (noNonNullAssertedNullishCoalescing) Check(ctx *Context, node *shimast.Node) {
-	expr := node.AsBinaryExpression()
-	if expr == nil || expr.OperatorToken == nil || expr.OperatorToken.Kind != shimast.KindQuestionQuestionToken {
-		return
-	}
-	if left := stripParens(expr.Left); left != nil && left.Kind == shimast.KindNonNullExpression {
-		ctx.Report(left, "Nullish coalescing is unnecessary after a non-null assertion.")
-	}
-	if right := stripParens(expr.Right); right != nil && right.Kind == shimast.KindNonNullExpression {
-		ctx.Report(right, "Nullish coalescing is unnecessary before a non-null assertion.")
-	}
+  expr := node.AsBinaryExpression()
+  if expr == nil || expr.OperatorToken == nil || expr.OperatorToken.Kind != shimast.KindQuestionQuestionToken {
+    return
+  }
+  if left := stripParens(expr.Left); left != nil && left.Kind == shimast.KindNonNullExpression {
+    ctx.Report(left, "Nullish coalescing is unnecessary after a non-null assertion.")
+  }
+  if right := stripParens(expr.Right); right != nil && right.Kind == shimast.KindNonNullExpression {
+    ctx.Report(right, "Nullish coalescing is unnecessary before a non-null assertion.")
+  }
 }
 
 // no-unnecessary-type-constraint: `<T extends any>` / `<T extends unknown>`.
@@ -142,16 +142,16 @@ type noUnnecessaryTypeConstraint struct{}
 
 func (noUnnecessaryTypeConstraint) Name() string { return "no-unnecessary-type-constraint" }
 func (noUnnecessaryTypeConstraint) Visits() []shimast.Kind {
-	return []shimast.Kind{shimast.KindTypeParameter}
+  return []shimast.Kind{shimast.KindTypeParameter}
 }
 func (noUnnecessaryTypeConstraint) Check(ctx *Context, node *shimast.Node) {
-	param := node.AsTypeParameterDeclaration()
-	if param == nil || param.Constraint == nil {
-		return
-	}
-	if param.Constraint.Kind == shimast.KindAnyKeyword || param.Constraint.Kind == shimast.KindUnknownKeyword {
-		ctx.Report(param.Constraint, "Constraining a type parameter to any or unknown is unnecessary.")
-	}
+  param := node.AsTypeParameterDeclaration()
+  if param == nil || param.Constraint == nil {
+    return
+  }
+  if param.Constraint.Kind == shimast.KindAnyKeyword || param.Constraint.Kind == shimast.KindUnknownKeyword {
+    ctx.Report(param.Constraint, "Constraining a type parameter to any or unknown is unnecessary.")
+  }
 }
 
 // no-unsafe-function-type: `Function` accepts any callable shape.
@@ -161,11 +161,11 @@ type noUnsafeFunctionType struct{}
 func (noUnsafeFunctionType) Name() string           { return "no-unsafe-function-type" }
 func (noUnsafeFunctionType) Visits() []shimast.Kind { return []shimast.Kind{shimast.KindTypeReference} }
 func (noUnsafeFunctionType) Check(ctx *Context, node *shimast.Node) {
-	ref := node.AsTypeReferenceNode()
-	if ref == nil || identifierText(ref.TypeName) != "Function" {
-		return
-	}
-	ctx.Report(node, "The Function type is unsafe. Use a specific function type instead.")
+  ref := node.AsTypeReferenceNode()
+  if ref == nil || identifierText(ref.TypeName) != "Function" {
+    return
+  }
+  ctx.Report(node, "The Function type is unsafe. Use a specific function type instead.")
 }
 
 // no-wrapper-object-types: prefer primitive type keywords over boxed object
@@ -176,14 +176,14 @@ type noWrapperObjectTypes struct{}
 func (noWrapperObjectTypes) Name() string           { return "no-wrapper-object-types" }
 func (noWrapperObjectTypes) Visits() []shimast.Kind { return []shimast.Kind{shimast.KindTypeReference} }
 func (noWrapperObjectTypes) Check(ctx *Context, node *shimast.Node) {
-	ref := node.AsTypeReferenceNode()
-	if ref == nil {
-		return
-	}
-	switch identifierText(ref.TypeName) {
-	case "String", "Number", "Boolean", "Symbol", "BigInt", "Object":
-		ctx.Report(node, "Use primitive type keywords instead of wrapper object types.")
-	}
+  ref := node.AsTypeReferenceNode()
+  if ref == nil {
+    return
+  }
+  switch identifierText(ref.TypeName) {
+  case "String", "Number", "Boolean", "Symbol", "BigInt", "Object":
+    ctx.Report(node, "Use primitive type keywords instead of wrapper object types.")
+  }
 }
 
 // no-useless-constructor: an empty constructor with no parameters is noise.
@@ -193,17 +193,17 @@ type noUselessConstructor struct{}
 func (noUselessConstructor) Name() string           { return "no-useless-constructor" }
 func (noUselessConstructor) Visits() []shimast.Kind { return []shimast.Kind{shimast.KindConstructor} }
 func (noUselessConstructor) Check(ctx *Context, node *shimast.Node) {
-	ctor := node.AsConstructorDeclaration()
-	if ctor == nil || ctor.Body == nil {
-		return
-	}
-	if len(node.Parameters()) != 0 {
-		return
-	}
-	body := ctor.Body.AsBlock()
-	if body == nil || body.Statements == nil || len(body.Statements.Nodes) == 0 {
-		ctx.Report(node, "Useless empty constructor.")
-	}
+  ctor := node.AsConstructorDeclaration()
+  if ctor == nil || ctor.Body == nil {
+    return
+  }
+  if len(node.Parameters()) != 0 {
+    return
+  }
+  body := ctor.Body.AsBlock()
+  if body == nil || body.Statements == nil || len(body.Statements.Nodes) == 0 {
+    ctx.Report(node, "Useless empty constructor.")
+  }
 }
 
 // prefer-literal-enum-member: computed enum members are harder to inspect.
@@ -213,11 +213,11 @@ type preferLiteralEnumMember struct{}
 func (preferLiteralEnumMember) Name() string           { return "prefer-literal-enum-member" }
 func (preferLiteralEnumMember) Visits() []shimast.Kind { return []shimast.Kind{shimast.KindEnumMember} }
 func (preferLiteralEnumMember) Check(ctx *Context, node *shimast.Node) {
-	member := node.AsEnumMember()
-	if member == nil || member.Initializer == nil || isLiteralLike(member.Initializer) {
-		return
-	}
-	ctx.Report(member.Initializer, "Enum member initializer should be a literal value.")
+  member := node.AsEnumMember()
+  if member == nil || member.Initializer == nil || isLiteralLike(member.Initializer) {
+    return
+  }
+  ctx.Report(member.Initializer, "Enum member initializer should be a literal value.")
 }
 
 // consistent-type-assertions: prefer `value as Type` over `<Type>value`.
@@ -226,10 +226,10 @@ type consistentTypeAssertions struct{}
 
 func (consistentTypeAssertions) Name() string { return "consistent-type-assertions" }
 func (consistentTypeAssertions) Visits() []shimast.Kind {
-	return []shimast.Kind{shimast.KindTypeAssertionExpression}
+  return []shimast.Kind{shimast.KindTypeAssertionExpression}
 }
 func (consistentTypeAssertions) Check(ctx *Context, node *shimast.Node) {
-	ctx.Report(node, "Use `as` type assertions instead of angle-bracket assertions.")
+  ctx.Report(node, "Use `as` type assertions instead of angle-bracket assertions.")
 }
 
 // consistent-type-definitions: prefer interfaces for object-shaped public
@@ -239,14 +239,14 @@ type consistentTypeDefinitions struct{}
 
 func (consistentTypeDefinitions) Name() string { return "consistent-type-definitions" }
 func (consistentTypeDefinitions) Visits() []shimast.Kind {
-	return []shimast.Kind{shimast.KindTypeAliasDeclaration}
+  return []shimast.Kind{shimast.KindTypeAliasDeclaration}
 }
 func (consistentTypeDefinitions) Check(ctx *Context, node *shimast.Node) {
-	alias := node.AsTypeAliasDeclaration()
-	if alias == nil || alias.Type == nil || alias.Type.Kind != shimast.KindTypeLiteral {
-		return
-	}
-	ctx.Report(node, "Use an interface instead of a type literal alias.")
+  alias := node.AsTypeAliasDeclaration()
+  if alias == nil || alias.Type == nil || alias.Type.Kind != shimast.KindTypeLiteral {
+    return
+  }
+  ctx.Report(node, "Use an interface instead of a type literal alias.")
 }
 
 // dot-notation: `obj["prop"]` should be `obj.prop` when the key is a valid
@@ -256,34 +256,34 @@ type dotNotation struct{}
 
 func (dotNotation) Name() string { return "dot-notation" }
 func (dotNotation) Visits() []shimast.Kind {
-	return []shimast.Kind{shimast.KindElementAccessExpression}
+  return []shimast.Kind{shimast.KindElementAccessExpression}
 }
 func (dotNotation) Check(ctx *Context, node *shimast.Node) {
-	access := node.AsElementAccessExpression()
-	if access == nil {
-		return
-	}
-	key := stringLiteralText(access.ArgumentExpression)
-	if key == "" || !isSimpleIdentifierName(key) {
-		return
-	}
-	ctx.Report(node, "Use dot notation instead of a string literal property access.")
+  access := node.AsElementAccessExpression()
+  if access == nil {
+    return
+  }
+  key := stringLiteralText(access.ArgumentExpression)
+  if key == "" || !isSimpleIdentifierName(key) {
+    return
+  }
+  ctx.Report(node, "Use dot notation instead of a string literal property access.")
 }
 
 func isSimpleIdentifierName(value string) bool {
-	if value == "" {
-		return false
-	}
-	for i, r := range value {
-		if r == '_' || r == '$' || r >= 'a' && r <= 'z' || r >= 'A' && r <= 'Z' {
-			continue
-		}
-		if i > 0 && r >= '0' && r <= '9' {
-			continue
-		}
-		return false
-	}
-	return true
+  if value == "" {
+    return false
+  }
+  for i, r := range value {
+    if r == '_' || r == '$' || r >= 'a' && r <= 'z' || r >= 'A' && r <= 'Z' {
+      continue
+    }
+    if i > 0 && r >= '0' && r <= '9' {
+      continue
+    }
+    return false
+  }
+  return true
 }
 
 // no-unsafe-declaration-merging: class/interface merging hides runtime vs type
@@ -293,49 +293,49 @@ type noUnsafeDeclarationMerging struct{}
 
 func (noUnsafeDeclarationMerging) Name() string { return "no-unsafe-declaration-merging" }
 func (noUnsafeDeclarationMerging) Visits() []shimast.Kind {
-	return []shimast.Kind{shimast.KindSourceFile}
+  return []shimast.Kind{shimast.KindSourceFile}
 }
 func (noUnsafeDeclarationMerging) Check(ctx *Context, node *shimast.Node) {
-	classes := map[string]bool{}
-	var interfaces []*shimast.Node
-	walkDescendants(node, func(child *shimast.Node) {
-		switch child.Kind {
-		case shimast.KindClassDeclaration:
-			decl := child.AsClassDeclaration()
-			if decl != nil {
-				if name := identifierText(decl.Name()); name != "" {
-					classes[name] = true
-				}
-			}
-		case shimast.KindInterfaceDeclaration:
-			interfaces = append(interfaces, child)
-		}
-	})
-	for _, ifaceNode := range interfaces {
-		decl := ifaceNode.AsInterfaceDeclaration()
-		if decl == nil {
-			continue
-		}
-		name := identifierText(decl.Name())
-		if name != "" && classes[name] {
-			ctx.Report(ifaceNode, "Unsafe declaration merging between class and interface '"+name+"'.")
-		}
-	}
+  classes := map[string]bool{}
+  var interfaces []*shimast.Node
+  walkDescendants(node, func(child *shimast.Node) {
+    switch child.Kind {
+    case shimast.KindClassDeclaration:
+      decl := child.AsClassDeclaration()
+      if decl != nil {
+        if name := identifierText(decl.Name()); name != "" {
+          classes[name] = true
+        }
+      }
+    case shimast.KindInterfaceDeclaration:
+      interfaces = append(interfaces, child)
+    }
+  })
+  for _, ifaceNode := range interfaces {
+    decl := ifaceNode.AsInterfaceDeclaration()
+    if decl == nil {
+      continue
+    }
+    name := identifierText(decl.Name())
+    if name != "" && classes[name] {
+      ctx.Report(ifaceNode, "Unsafe declaration merging between class and interface '"+name+"'.")
+    }
+  }
 }
 
 func init() {
-	Register(noEmptyStaticBlock{})
-	Register(noSetterReturn{})
-	Register(noUnusedLabels{})
-	Register(noDynamicDelete{})
-	Register(noNonNullAssertedNullishCoalescing{})
-	Register(noUnnecessaryTypeConstraint{})
-	Register(noUnsafeDeclarationMerging{})
-	Register(noUnsafeFunctionType{})
-	Register(noWrapperObjectTypes{})
-	Register(noUselessConstructor{})
-	Register(preferLiteralEnumMember{})
-	Register(consistentTypeAssertions{})
-	Register(consistentTypeDefinitions{})
-	Register(dotNotation{})
+  Register(noEmptyStaticBlock{})
+  Register(noSetterReturn{})
+  Register(noUnusedLabels{})
+  Register(noDynamicDelete{})
+  Register(noNonNullAssertedNullishCoalescing{})
+  Register(noUnnecessaryTypeConstraint{})
+  Register(noUnsafeDeclarationMerging{})
+  Register(noUnsafeFunctionType{})
+  Register(noWrapperObjectTypes{})
+  Register(noUselessConstructor{})
+  Register(preferLiteralEnumMember{})
+  Register(consistentTypeAssertions{})
+  Register(consistentTypeDefinitions{})
+  Register(dotNotation{})
 }