Skip to content

Conversation

@shivasurya
Copy link
Owner

This PR establishes the infrastructure core by creating three new packages that resolve circular dependencies and improve code organization. The registry package handles module discovery, builtin type information, and attribute tracking. The cfg package manages control flow graph construction. The resolution package defines type resolution structures for variable bindings and function scopes. All original files have been updated with type aliases to maintain complete backward compatibility.

Verification completed with gradle buildGo showing successful compilation with no circular import errors, gradle testGo confirming all test suites pass across callgraph, registry, cfg, and resolution packages, and gradle lintGo reporting zero issues. Test coverage ranges from 95.9% to 100% across the new packages.

🤖 Generated with Claude Code

@safedep
Copy link

safedep bot commented Nov 15, 2025

SafeDep Report Summary

Green Malicious Packages Badge Green Vulnerable Packages Badge Green Risky License Badge

No dependency changes detected. Nothing to scan.

This report is generated by SafeDep Github App

@codecov
Copy link

codecov bot commented Nov 15, 2025

Codecov Report

❌ Patch coverage is 95.92199% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.07%. Comparing base (6b53218) to head (704cadc).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...urcecode-parser/graph/callgraph/registry/module.go 76.19% 5 Missing and 5 partials ⚠️
sourcecode-parser/graph/callgraph/cfg/cfg.go 93.49% 4 Missing and 4 partials ⚠️
...rcecode-parser/graph/callgraph/registry/builtin.go 98.75% 2 Missing and 2 partials ⚠️
sourcecode-parser/graph/callgraph/cfg.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #373   +/-   ##
=======================================
  Coverage   80.07%   80.07%           
=======================================
  Files          69       74    +5     
  Lines        6836     6857   +21     
=======================================
+ Hits         5474     5491   +17     
- Misses       1139     1142    +3     
- Partials      223      224    +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.

Copy link
Owner Author

shivasurya commented Nov 16, 2025

Merge activity

  • Nov 16, 12:00 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Nov 16, 12:01 AM UTC: Graphite rebased this pull request as part of a merge.
  • Nov 16, 12:02 AM UTC: @shivasurya merged this pull request with Graphite.

@shivasurya shivasurya changed the base branch from refactor/01-foundation-types to graphite-base/373 November 16, 2025 00:01
@shivasurya shivasurya changed the base branch from graphite-base/373 to main November 16, 2025 00:01
This PR establishes the infrastructure core by creating three new packages
that resolve circular dependencies and improve code organization. The
registry package handles module/type/attribute registries, cfg package
manages control flow graphs, and resolution package defines type resolution
structures. All changes maintain backward compatibility through type aliases.

Verification: gradle buildGo (success), gradle testGo (all pass), gradle
lintGo (0 issues), coverage 95.9-100% across new packages.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@shivasurya shivasurya force-pushed the refactor/02-infrastructure-core branch from 4a12acf to 704cadc Compare November 16, 2025 00:01
@shivasurya shivasurya merged commit 64f8c7a into main Nov 16, 2025
3 checks passed
@shivasurya shivasurya deleted the refactor/02-infrastructure-core branch November 16, 2025 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request go Pull requests that update go code refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants