Skip to content

Remove remote code generation functionality#4417

Merged
kyleconroy merged 1 commit intomainfrom
claude/remove-sqlc-remote-execution-145oI
Apr 28, 2026
Merged

Remove remote code generation functionality#4417
kyleconroy merged 1 commit intomainfrom
claude/remove-sqlc-remote-execution-145oI

Conversation

@kyleconroy
Copy link
Copy Markdown
Collaborator

This PR removes the remote code generation feature that allowed sqlc to execute code generation on a remote server via gRPC.

Summary

The remote execution capability has been completely removed from the codebase, including all related infrastructure, configuration options, and protobuf definitions.

Key Changes

  • Removed protobuf definitions and generated gRPC code (gen.proto, gen.pb.go, gen_grpc.pb.go)
  • Removed remote RPC client implementation (internal/remote/rpc.go)
  • Removed remoteGenerate() function from the generate command that handled remote execution logic
  • Removed CLI flags --remote and --no-remote from the root command
  • Removed Remote and NoRemote fields from the Env struct
  • Removed related imports (grpc/status, remote, info, sqlpath)
  • Updated documentation to remove references to remote execution flags
  • Updated end-to-end tests to remove NoRemote configuration

Implementation Details

The remote generation feature was conditionally invoked when a cloud project was configured and remote execution was enabled. This entire code path has been removed, simplifying the generate command to only support local and plugin-based code generation.

https://claude.ai/code/session_01Fw2jJDuQ5mEU9d3rHS4uyo

Removes the --remote/--no-remote flags, the remoteGenerate() codegen
path, and the now-unused internal/remote gRPC client. The cloud.*
config block, SQLC_AUTH_TOKEN, QuickDB, and managed databases are
unaffected.
@kyleconroy kyleconroy merged commit 977ac6d into main Apr 28, 2026
13 checks passed
@kyleconroy kyleconroy deleted the claude/remove-sqlc-remote-execution-145oI branch April 28, 2026 05:09
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.

2 participants