Skip to content

chore(openshell): re-vendor proto to v0.0.71#147

Open
onsails wants to merge 1 commit into
masterfrom
automation/openshell-proto-revendor
Open

chore(openshell): re-vendor proto to v0.0.71#147
onsails wants to merge 1 commit into
masterfrom
automation/openshell-proto-revendor

Conversation

@onsails

@onsails onsails commented Jun 25, 2026

Copy link
Copy Markdown
Owner

Automated re-vendor of the OpenShell protos to v0.0.71.

static check result
buf breaking (PACKAGE) BREAKING
cargo check -p right-openshell FAILED
buf breaking output
crates/right-openshell/proto/openshell/openshell.proto:326:3:Field "9" with name "resource_requirements" on message "SandboxSpec" changed cardinality from "optional with implicit presence" to "optional with explicit presence".
crates/right-openshell/proto/openshell/openshell.proto:326:3:Field "9" with name "resource_requirements" on message "SandboxSpec" changed option "json_name" from "gpu" to "resourceRequirements".
crates/right-openshell/proto/openshell/openshell.proto:326:3:Field "9" with name "resource_requirements" on message "SandboxSpec" changed type from "bool" to "message".
crates/right-openshell/proto/openshell/openshell.proto:326:24:Field "9" on message "SandboxSpec" changed name from "gpu" to "resource_requirements".
cargo check (tail)
   Compiling tonic-build v0.14.6
    Checking futures-executor v0.3.32
   Compiling ref-cast-impl v1.0.25
   Compiling pin-project-internal v1.1.11
   Compiling serde_derive_internals v0.29.1
    Checking encoding_rs v0.8.35
    Checking arraydeque v0.5.1
    Checking anstyle v1.0.14
    Checking option-ext v0.2.0
    Checking const-oid v0.10.2
    Checking matchit v0.8.4
    Checking iri-string v0.7.11
    Checking axum v0.8.9
    Checking tower-http v0.6.8
    Checking digest v0.11.2
    Checking encoding_rs_io v0.1.7
    Checking dirs-sys v0.5.0
    Checking annotate-snippets v0.12.13
    Checking granit-parser v0.0.3
   Compiling schemars_derive v1.2.1
    Checking pin-project v1.1.11
   Compiling tonic-prost-build v0.14.6
    Checking futures v0.3.32
    Checking rand v0.10.1
    Checking url v2.5.8
    Checking mimalloc v0.1.52
    Checking serde_urlencoded v0.7.1
    Checking hyper-timeout v0.5.2
    Checking dyn-clone v1.0.20
   Compiling rmcp v1.7.0
    Checking nohash-hasher v0.2.0
    Checking serde-saphyr v0.0.27
    Checking schemars v1.2.1
   Compiling right-openshell v0.4.2 (/home/runner/work/right-agent/right-agent/crates/right-openshell)
    Checking turso_sync_engine v0.7.0-pre.8
   Compiling rmcp-macros v1.7.0
    Checking dirs v6.0.0
    Checking sse-stream v0.2.3
    Checking fs4 v1.1.0
    Checking same-file v1.0.6
    Checking walkdir v2.5.0
    Checking right-process v0.4.2 (/home/runner/work/right-agent/right-agent/crates/right-process)
    Checking sha2 v0.11.0
    Checking hmac v0.13.0
    Checking secrecy v0.10.3
    Checking which v8.0.4
   Compiling include_dir_macros v0.7.4
    Checking turso_sync_sdk_kit v0.7.0-pre.8
    Checking memo-map v0.3.3
    Checking shlex v2.0.1
    Checking minijinja v2.21.0
    Checking include_dir v0.7.4
    Checking right-config v0.4.2 (/home/runner/work/right-agent/right-agent/crates/right-config)
    Checking right-runtime-state v0.4.2 (/home/runner/work/right-agent/right-agent/crates/right-runtime-state)
    Checking right-agent-config v0.4.2 (/home/runner/work/right-agent/right-agent/crates/right-agent-config)
    Checking right-platform-knobs v0.4.2 (/home/runner/work/right-agent/right-agent/crates/right-platform-knobs)
    Checking rustls-webpki v0.103.10
    Checking tokio-rustls v0.26.4
    Checking rustls-platform-verifier v0.6.2
    Checking hyper-rustls v0.27.9
    Checking tonic v0.14.6
    Checking reqwest v0.13.4
    Checking turso v0.7.0-pre.8
    Checking right-db v0.4.2 (/home/runner/work/right-agent/right-agent/crates/right-db)
    Checking tonic-prost v0.14.6
error[E0063]: missing field `resource_version` in initializer of `openshell_proto::openshell::v1::ProviderProfile`
   --> crates/right-openshell/src/managed_profiles.rs:134:5
    |
134 |     proto_v1::ProviderProfile {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^ missing `resource_version`

error[E0063]: missing field `resource_version` in initializer of `openshell_proto::openshell::v1::ProviderProfile`
   --> crates/right-openshell/src/managed_profiles.rs:179:5
    |
179 |     proto_v1::ProviderProfile {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^ missing `resource_version`

For more information about this error, try `rustc --explain E0063`.
error: could not compile `right-openshell` (lib) due to 2 previous errors
warning: build failed, waiting for other jobs to finish...

A BREAKING or FAILED result means upstream changed the wire/API shape.
Audit SandboxReadiness and any code decoding moved/removed fields, then bump
MIN_OPENSHELL_VERSION and the CI OPENSHELL_VERSION pin as needed.
This bot surfaces drift only — it does not fix Rust. Full CI runs on this PR only
when BOT_PR_TOKEN (a fine-grained PAT) is configured as a repo secret;
otherwise bot-authored PRs don't trigger downstream workflows. Either way, the
openshell-proto-compat workflow run itself goes RED when cargo check fails,
so a failing proto bump is visible in the Actions tab even without full CI.

Copilot AI review requested due to automatic review settings June 25, 2026 07:09

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 re-vendors the OpenShell protobuf definitions in right-openshell to upstream tag v0.0.69, updating the API/wire surface that right-openshell compiles against and communicates with.

Changes:

  • Updated vendored upstream tracking metadata to v0.0.69.
  • Updated openshell.proto to include new provider profile update RPCs/messages, a new ProviderProfile.resource_version field, and replaced SandboxSpec.gpu with SandboxSpec.resource_requirements (plus new resource requirement messages).

Reviewed changes

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

File Description
crates/right-openshell/proto/UPSTREAM.md Bumps recorded upstream proto tag/fetch timestamp to v0.0.69.
crates/right-openshell/proto/openshell/openshell.proto Updates the vendored OpenShell API surface (new RPC/messages, new resource_version, and breaking SandboxSpec field-9 change).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 1094 to 1099
ProviderProfileDiscovery discovery = 9;
// Storage resource version for custom profiles. Built-in profiles and new
// profile files use 0. Gateway responses set this for stored custom profiles.
// Update calls use this for optimistic concurrency.
uint64 resource_version = 10;
}
Comment on lines 321 to +326
openshell.sandbox.v1.SandboxPolicy policy = 7;
// Provider names to attach to this sandbox.
repeated string providers = 8;
// Request NVIDIA GPU resources for this sandbox.
bool gpu = 9;
// Portable resource requirements used by the gateway for driver selection
// and by drivers for provisioning.
ResourceRequirements resource_requirements = 9;
@onsails onsails force-pushed the automation/openshell-proto-revendor branch from af4fc08 to 1e6a5d1 Compare June 26, 2026 07:14
@onsails onsails changed the title chore(openshell): re-vendor proto to v0.0.69 chore(openshell): re-vendor proto to v0.0.70 Jun 26, 2026
@onsails onsails changed the title chore(openshell): re-vendor proto to v0.0.70 chore(openshell): re-vendor proto to v0.0.71 Jun 27, 2026
@onsails onsails force-pushed the automation/openshell-proto-revendor branch 2 times, most recently from 09e0a0c to 05772f4 Compare June 28, 2026 07:12
@onsails onsails force-pushed the automation/openshell-proto-revendor branch from 05772f4 to 46e0d9f Compare June 29, 2026 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants