Also handle type assertions in channel send (PR feedback)

rule/unchecked-type-assertion.go

@@ -160,6 +160,10 @@ func (w *lintUnchekedTypeAssertion) handleRange(n *ast.RangeStmt) {
 	w.requireNoTypeAssert(n.X)
 }
 
+func (w *lintUnchekedTypeAssertion) handleChannelSend(n *ast.SendStmt) {
+	w.requireNoTypeAssert(n.Value)
+}
+
 func (w *lintUnchekedTypeAssertion) Visit(node ast.Node) ast.Visitor {
 	switch n := node.(type) {
 	case *ast.RangeStmt:
@@ -174,6 +178,8 @@ func (w *lintUnchekedTypeAssertion) Visit(node ast.Node) ast.Visitor {
 		w.handleIfStmt(n)
 	case *ast.CaseClause:
 		w.handleCaseClause(n)
+	case *ast.SendStmt:
+		w.handleChannelSend(n)
 	}
 
 	return w

testdata/unchecked-type-assertion.go

@@ -59,7 +59,7 @@ func handleTypeComparison() {
 }
 
 func handleTypeComparisonReverse() {
-	if foo.(int) == 1 { // MATCH /type cast result is unchecked in foo.(int) - type assertion will panic if not matched/
+	if 1 == foo.(int) { // MATCH /type cast result is unchecked in foo.(int) - type assertion will panic if not matched/
 		return
 	}
 }
@@ -100,3 +100,9 @@ func handleInnerSwitchAssertionReverse() {
 	case 1 == bar.(int): // MATCH /type cast result is unchecked in bar.(int) - type assertion will panic if not matched/
 	}
 }
+
+func handleChannelWrite() {
+	c := make(chan any)
+	var a any = "foo"
+	c <- a.(int) // MATCH /type cast result is unchecked in a.(int) - type assertion will panic if not matched/
+}