Skip to content

refactor(bindings/go): Restructure FFI system with type-safe wrapper#6268

Merged
Xuanwo merged 2 commits intomainfrom
refactor-go-binding
Jun 10, 2025
Merged

refactor(bindings/go): Restructure FFI system with type-safe wrapper#6268
Xuanwo merged 2 commits intomainfrom
refactor-go-binding

Conversation

@yuchanns
Copy link
Copy Markdown
Member

@yuchanns yuchanns commented Jun 8, 2025

Replace global context-based FFI approach with modular FFI[T] type system. This improves code organization, eliminates scattered global symbols, and provides better type safety for FFI operations across all Go binding files.

@yuchanns yuchanns requested a review from Xuanwo as a code owner June 8, 2025 11:34
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" labels Jun 8, 2025
yuchanns added 2 commits June 10, 2025 22:07
Replace global context-based FFI approach with modular FFI[T] type system.
This improves code organization, eliminates scattered global symbols, and
provides better type safety for FFI operations across all Go binding files.
- Add cross-platform Makefile with OS/architecture detection (Linux, macOS, Windows)
- Automate C library building, compression, and Go service generation
- Replace complex shell scripts in README with simple Makefile commands
- Improve developer experience with standardized build targets and error handling
@yuchanns yuchanns force-pushed the refactor-go-binding branch from 60cc47e to b0e5114 Compare June 10, 2025 14:07
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 10, 2025
@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Jun 10, 2025

Nice work!

@Xuanwo Xuanwo merged commit ff2a646 into main Jun 10, 2025
124 of 125 checks passed
@Xuanwo Xuanwo deleted the refactor-go-binding branch June 10, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants