Skip to content

feat: add proxy job#15

Merged
davideimola merged 13 commits into
mainfrom
feature/proxy
Aug 1, 2025
Merged

feat: add proxy job#15
davideimola merged 13 commits into
mainfrom
feature/proxy

Conversation

@davideimola

Copy link
Copy Markdown
Member

No description provided.

@davideimola davideimola requested a review from ludusrusso July 21, 2025 15:34
@ludusrusso ludusrusso requested a review from Copilot July 21, 2025 17:07

Copilot AI left a comment

Copy link
Copy Markdown

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 adds a proxy job functionality to the RedCarbon agent. The primary purpose is to enable the agent to fetch HTTP requests from the server and execute them as a proxy, then submit the responses back.

  • Added new protobuf message types for proxy requests and responses
  • Implemented new gRPC methods for fetching agent requests and submitting responses
  • Created a background proxy routine that polls for requests and processes them concurrently

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.proto/redcarbon/agents_public/v1/v1.proto Added gRPC service methods for proxy functionality
.proto/redcarbon/agents_public/v1/types.proto Added AgentRequest and AgentResponse message types
proto/redcarbon/agents_public/v1/v1.pb.go Generated Go code with new protobuf structures and methods
proto/redcarbon/agents_public/v1/types.pb.go Generated Go code with new request/response types
proto/redcarbon/agents_public/v1/agents_publicv1connect/v1.connect.go Generated Connect protocol client and handler code
internal/routines/proxy.go Main proxy routine implementation for handling HTTP requests
cmd/main.go Added proxy routine to the scheduler
mise.toml Added development environment tool configuration
Comments suppressed due to low confidence (1)

internal/routines/proxy.go:31

  • [nitpick] A 200ms polling interval could generate excessive load on the server. Consider increasing the interval or implementing a push-based mechanism for better performance.
	res, err := r.agentsCli.FetchAgentRequests(ctxWithTimeout, req)

Comment thread internal/routines/proxy.go Outdated
Comment thread internal/routines/proxy.go
@davideimola davideimola merged commit f08457c into main Aug 1, 2025
2 checks passed
@davideimola davideimola deleted the feature/proxy branch August 1, 2025 09:08
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