feat(golang): Go function and method declaration parsing (PR-03)

[shivasurya]

Summary

• Parse Go function declarations, method declarations, and init() into CodeGraph nodes
• Set Modifier (public/private) based on Go export rules (uppercase = public)
• Return function Node as currentContext for child call node linking
• Add FunctionInfo/MethodInfo structs in golang package (avoids circular dependency)
• Resolves PR-02 lint failures: setGoSourceLocation and isGoSourceFile now used

Test plan

• func Foo(a, b int) string {} → correct Node with params, return type, Modifier="public"
• func foo() {} → Node{Modifier: "private"}
• func init() {} → Node{Type: "init_function"}
• func Printf(format string, args ...interface{}) {} → variadic params handled
• func Read(p []byte) (int, error) {} → multiple return types
• func (s *Server) Start() error {} → Node{Type: "method", Interface: ["S"]}
• func (s Server) String() string {} → value receiver, same result
• func (s *S) stop() {} → Node{Modifier: "private"}
• Java method_declaration regression: still works with isJavaSourceFile guard
• go build ./... compiles without errors
• go test ./graph/golang/... — 100% coverage on declarations.go
• go test ./graph/... — no regressions
• go vet ./... reports zero findings

🤖 Generated with Claude Code

[safedep]

SafeDep Report Summary

No dependency changes detected. Nothing to scan.

_{This report is generated by SafeDep Github App}

[codecov]

Codecov Report

❌ Patch coverage is 97.46835% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.93%. Comparing base (c19850d) to head (e4d94c8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
sast-engine/graph/parser_golang.go	94.73%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #522      +/-   ##
==========================================
+ Coverage   82.83%   82.93%   +0.09%     
==========================================
  Files         124      125       +1     
  Lines       14462    14540      +78     
==========================================
+ Hits        11980    12059      +79     
+ Misses       2030     2027       -3     
- Partials      452      454       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[shivasurya]

• feat(mcp): Add foundational Go language support to MCP server #530
• feat(golang): Add Go security rules and rule manifest #529
• feat(golang): Scan & CI pipeline routing (PR-09) #528
• feat(golang): Implement Go call graph construction (PR-08) #527
• feat(golang): Go module registry and import resolution (PR-07) #526
• feat(golang): Implement calls, closures, and control flow statements (PR-06 complete) #525
• feat(golang): Go variable, constant, and assignment parsing (PR-05) #524
• feat(golang): Go type definitions parsing (PR-04) #523
• feat(golang): Go function and method declaration parsing (PR-03) #522 👈 (View in Graphite)
• feat(golang): Go AST helper functions and parser scaffold (PR-02) #521
• feat(golang): Go file detection and tree-sitter-go parsing (PR-01) #520
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[shivasurya]

Merge activity

• Feb 15, 4:00 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Feb 15, 4:03 AM UTC: Graphite rebased this pull request as part of a merge.
• Feb 15, 4:04 AM UTC: @shivasurya merged this pull request with Graphite.

[github-actions]

Code Pathfinder Security Scan

No security issues detected.

Metric	Value
Files Scanned	4
Rules	38

---

_{Powered by Code Pathfinder}