Skip to content
This repository was archived by the owner on Aug 23, 2026. It is now read-only.

refactor: remove GetAgentManifest RPC and related types from LinkService - #149

Merged
iasthc merged 1 commit into
mainfrom
remove-agent-manifest
Aug 23, 2026
Merged

refactor: remove GetAgentManifest RPC and related types from LinkService#149
iasthc merged 1 commit into
mainfrom
remove-agent-manifest

Conversation

@iasthc

@iasthc iasthc commented Aug 23, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings August 23, 2026 04:56
@iasthc
iasthc merged commit 2d767bb into main Aug 23, 2026
5 checks passed
@iasthc
iasthc deleted the remove-agent-manifest branch August 23, 2026 04:56

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 removes the GetAgentManifest RPC from otterscale.link.v1.LinkService and cleans up all generated artifacts (TypeScript, Go, and OpenAPI) so the API surface is consistent across protobuf, Connect, and OpenAPI outputs.

Changes:

  • Removed GetAgentManifest RPC from proto/link/v1/link.proto.
  • Removed GetAgentManifest{Request,Response} message types from generated TypeScript and Go protobuf outputs.
  • Removed GetAgentManifest endpoints and schemas from the generated OpenAPI spec, and removed the RPC from the generated Go Connect client/handler.

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
ts/src/link/v1/link_pb.js Updates generated TS protobuf descriptors to remove GetAgentManifest types/service method metadata.
ts/src/link/v1/link_pb.d.ts Removes the exported TS types and service method signature for GetAgentManifest.
proto/link/v1/link.proto Deletes the GetAgentManifest RPC and its request/response messages from the source proto.
openapi/openapi.yaml Removes the /GetAgentManifest path entries and related request/response component schemas.
link/v1/link.pb.go Updates generated Go protobuf code: removes GetAgentManifest messages and service descriptor entries.
link/v1/link.connect.go Updates generated Go Connect client/handler code to remove the GetAgentManifest procedure and wiring.
Files not reviewed (2)
  • link/v1/link.connect.go: Generated file
  • link/v1/link.pb.go: Generated file

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants