feat(golang): Go type definitions parsing (PR-04)

[shivasurya]

Summary

• Add TypeInfo struct and ParseTypeDeclaration in graph/golang/types.go to extract type information from Go type_declaration AST nodes
• Handle struct definitions (with fields via ExtractStructFields), interface definitions (with methods via ExtractInterfaceMethods), type aliases/named types, and grouped declarations (type ( A int; B string ))
• Add dispatcher parseGoTypeDeclaration in parser_golang.go that converts TypeInfo into graph.Node with correct types (struct_definition, interface, type_alias) and SourceLocation byte ranges
• Fill type_declaration stub in parser.go

Test plan

• go build ./... compiles without errors
• go vet ./... clean
• All existing tests pass (go test ./graph/...) — no regressions
• New tests in types_test.go: struct with fields, embedded types, empty struct, unexported struct, struct with tags, interfaces with methods, interfaces with embedded, unexported interface, named type, function type alias, slice type, true alias (=), unexported alias, grouped declarations, grouped mixed types
• ParseTypeDeclaration 100% coverage, parseTypeAlias 100% coverage
• parseTypeSpec 87.5% coverage (remaining gap is defensive nil guard unreachable via tree-sitter)

🤖 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 90.36145% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.97%. Comparing base (b2882ff) to head (e87e9c4).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
sast-engine/graph/parser_golang.go	83.33%	5 Missing ⚠️
sast-engine/graph/golang/types.go	94.11%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #523      +/-   ##
==========================================
+ Coverage   82.93%   82.97%   +0.04%     
==========================================
  Files         125      126       +1     
  Lines       14540    14623      +83     
==========================================
+ Hits        12059    12134      +75     
- Misses       2027     2034       +7     
- Partials      454      455       +1     

☔ 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 👈 (View in Graphite)
• feat(golang): Go function and method declaration parsing (PR-03) #522
• 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:05 AM UTC: Graphite rebased this pull request as part of a merge.
• Feb 15, 4:06 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}