Rename file-close AST walker helper for clearer intent

[Copilot]

This update addresses the function-namer recommendation in pkg/linters/fileclosenotdeferred by replacing an ambiguous internal helper name with one that explicitly describes its role in AST-based file-close pattern analysis. The goal is better readability and discoverability without changing analyzer behavior.

• Scope

  • Renamed inspectFileNode to analyzeASTNodeForFileClosePatterns in pkg/linters/fileclosenotdeferred/fileclosenotdeferred.go.
  • Updated the single in-package call site in inspectFileFuncDecl.

• Behavioral impact

  • No logic, control-flow, or diagnostic-message changes.
  • Internal naming-only refactor; analyzer semantics remain unchanged.

• Representative diff

  ast.Inspect(fn.Body, func(node ast.Node) bool {
      return analyzeASTNodeForFileClosePatterns(pass, fileVars, node)
  })
  
  func analyzeASTNodeForFileClosePatterns(
      pass *analysis.Pass,
      fileVars map[types.Object]*fileVarState,
      node ast.Node,
  ) bool { ... }

[Copilot]

Pull request overview

This PR performs a naming-only refactor in the fileclosenotdeferred analyzer to make an internal AST-walker helper’s intent more explicit, improving readability and discoverability without changing diagnostics or control flow.

Changes:

• Renamed the internal helper inspectFileNode to analyzeASTNodeForFileClosePatterns.
• Updated the single in-package call site used by ast.Inspect to call the renamed helper.

Show a summary per file

File	Description
pkg/linters/fileclosenotdeferred/fileclosenotdeferred.go	Renames the AST inspection callback helper and updates its call site to match.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 1/1 changed files
• Comments generated: 0