refactor: fix unused method receiver

deepsource-autofix[bot] · web-flow · commit d6a9b2ff0f5f · 2025-05-01T20:45:39.000Z
Methods with unused receivers can be a symptom of unfinished refactoring or a bug. To keep 
the same method signature, omit the receiver name or '_' as it is unused.
diff --git a/analysis/ts_scope.go b/analysis/ts_scope.go
@@ -19,7 +19,7 @@ type TsScopeBuilder struct {
 	unresolvedRefs []UnresolvedRef
 }
 
-func (j *TsScopeBuilder) GetLanguage() Language {
+func (*TsScopeBuilder) GetLanguage() Language {
 	return LangJs
 }
 
@@ -37,11 +37,11 @@ var ScopeNodes = []string{
 	"method_definition",
 }
 
-func (ts *TsScopeBuilder) NodeCreatesScope(node *sitter.Node) bool {
+func (*TsScopeBuilder) NodeCreatesScope(node *sitter.Node) bool {
 	return slices.Contains(ScopeNodes, node.Type())
 }
 
-func (ts *TsScopeBuilder) DeclaresVariable(node *sitter.Node) bool {
+func (*TsScopeBuilder) DeclaresVariable(node *sitter.Node) bool {
 	typ := node.Type()
 	return typ == "variable_declarator" ||
 		typ == "import_clause" ||
diff --git a/pkg/analysis/analyze.go b/pkg/analysis/analyze.go
@@ -405,7 +405,7 @@ func GatherSkipInfo(fileContext *ParseResult) []*SkipComment {
 	return skipLines
 }
 
-func (ana *Analyzer) ContainsSkipcq(skipLines []*SkipComment, issue *Issue) bool {
+func (*Analyzer) ContainsSkipcq(skipLines []*SkipComment, issue *Issue) bool {
 	if len(skipLines) == 0 {
 		return false
 	}
diff --git a/pkg/analysis/scope_ts.go b/pkg/analysis/scope_ts.go
@@ -19,7 +19,7 @@ type TsScopeBuilder struct {
 	unresolvedRefs []UnresolvedRef
 }
 
-func (j *TsScopeBuilder) GetLanguage() Language {
+func (*TsScopeBuilder) GetLanguage() Language {
 	return LangJs
 }
 
@@ -33,11 +33,11 @@ var ScopeNodes = []string{
 	"program",
 }
 
-func (ts *TsScopeBuilder) NodeCreatesScope(node *sitter.Node) bool {
+func (*TsScopeBuilder) NodeCreatesScope(node *sitter.Node) bool {
 	return slices.Contains(ScopeNodes, node.Type())
 }
 
-func (ts *TsScopeBuilder) DeclaresVariable(node *sitter.Node) bool {
+func (*TsScopeBuilder) DeclaresVariable(node *sitter.Node) bool {
 	typ := node.Type()
 	// addition of function_declaration and formal_parameters necessary for functional scope handling.
 	return typ == "variable_declarator" || typ == "import_clause" || typ == "import_specifier" || typ == "formal_parameters" || typ == "function_declaration"

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ type TsScopeBuilder struct {`
`19`	`19`	`unresolvedRefs []UnresolvedRef`
`20`	`20`	`}`
`21`	`21`
`22`		`-func (j *TsScopeBuilder) GetLanguage() Language {`
	`22`	`+func (*TsScopeBuilder) GetLanguage() Language {`
`23`	`23`	`return LangJs`
`24`	`24`	`}`
`25`	`25`
`@@ -37,11 +37,11 @@ var ScopeNodes = []string{`
`37`	`37`	`"method_definition",`
`38`	`38`	`}`
`39`	`39`
`40`		`-func (ts TsScopeBuilder) NodeCreatesScope(node sitter.Node) bool {`
	`40`	`+func (TsScopeBuilder) NodeCreatesScope(node sitter.Node) bool {`
`41`	`41`	`return slices.Contains(ScopeNodes, node.Type())`
`42`	`42`	`}`
`43`	`43`
`44`		`-func (ts TsScopeBuilder) DeclaresVariable(node sitter.Node) bool {`
	`44`	`+func (TsScopeBuilder) DeclaresVariable(node sitter.Node) bool {`
`45`	`45`	`typ := node.Type()`
`46`	`46`	`return typ == "variable_declarator" \|\|`
`47`	`47`	`typ == "import_clause" \|\|`
Original file line number	Diff line number	Diff line change
`@@ -405,7 +405,7 @@ func GatherSkipInfo(fileContext ParseResult) []SkipComment {`
`405`	`405`	`return skipLines`
`406`	`406`	`}`
`407`	`407`
`408`		`-func (ana Analyzer) ContainsSkipcq(skipLines []SkipComment, issue *Issue) bool {`
	`408`	`+func (Analyzer) ContainsSkipcq(skipLines []SkipComment, issue *Issue) bool {`
`409`	`409`	`if len(skipLines) == 0 {`
`410`	`410`	`return false`
`411`	`411`	`}`