Skip to content

Migrate go-proxy implementation into dedicated aip-go repo for code#5

Merged
yungcero merged 7 commits intomainfrom
feat/remove-go-example
Feb 22, 2026
Merged

Migrate go-proxy implementation into dedicated aip-go repo for code#5
yungcero merged 7 commits intomainfrom
feat/remove-go-example

Conversation

@yungcero
Copy link
Member

Summary

Migrate over go implementation into dedicated repository

Related Issues

N/A

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📝 Documentation update
  • 🔧 Configuration change
  • [ X] ♻️ Refactoring (no functional changes)
  • 🧪 Test improvement
  • 🔒 Security fix

Changes Made

Testing

  • Unit tests added/updated
  • [] Manual testing performed
  • Tested with real MCP server
  • Tested policy enforcement

Test Commands

# Commands used to test
cd proxy
make test
make build
./bin/aip --policy examples/agent.yaml --target "python3 test/echo_server.py" --verbose

Policy Impact

  • No policy changes
  • New policy feature (describe below)
  • Policy behavior change (describe migration path)

Security Checklist

  • No new dependencies with known vulnerabilities
  • No secrets or credentials in code
  • Audit logging maintained for new operations
  • Input validation added for new parameters
  • Documentation updated for security implications

Documentation

  • [X ] README updated (if needed)
  • Code comments added for complex logic
  • Example configurations updated
  • CHANGELOG entry added (for user-facing changes)

Screenshots / Recordings

Checklist

  • My code follows the project's code style (make lint passes)
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • New and existing unit tests pass locally (make test)
  • Any dependent changes have been merged and published

/cc @ArangoGutierrez

@ArangoGutierrez
Copy link
Collaborator

Let's edit also the CI checks as we won't have the go code here. and move the CI Checks to the go-aip repo

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the Go proxy implementation out of the main Agent Identity Protocol repository into a dedicated aip-go repository. This refactoring removes all implementation-specific code while maintaining the core specification and documentation in the main repo.

Changes:

  • Removes entire implementations/go-proxy/ directory with all Go implementation code, tests, documentation, and examples
  • Updates README.md to add a new "SDKs & Implementations" section pointing to the external aip-go repository
  • Adds a reference link to the Go implementation at the top of the README

Reviewed changes

Copilot reviewed 51 out of 52 changed files in this pull request and generated no comments.

File Description
README.md Added SDK section with link to aip-go repo
implementations/go-proxy/* Complete removal of Go implementation (300+ files)

@yungcero
Copy link
Member Author

@ArangoGutierrez ill let you merge it if all looks good on your end

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 63 out of 64 changed files in this pull request and generated 1 comment.

@yungcero
Copy link
Member Author

I'll make my GPG key later for future commits. Merging this in after breakfast.

@yungcero yungcero merged commit 6b79b80 into main Feb 22, 2026
1 check passed
@yungcero yungcero deleted the feat/remove-go-example branch February 22, 2026 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants