Skip to content

chore(contracts): sync the worker's steering v2 contract - #9727

Merged
waleedlatif1 merged 1 commit into
stagingfrom
chore/sync-steering-v2-contract
Nov 6, 2026
Merged

chore(contracts): sync the worker's steering v2 contract#9727
waleedlatif1 merged 1 commit into
stagingfrom
chore/sync-steering-v2-contract

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Nov 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Syncs the worker's steering v2 contract into apps/sim/lib/mothership/generated/ with the repo's sync scripts: SteerV2Request, SteerV2Response, the MAX_STEER_CONTEXTS and MAX_RUN_STEERED_IMAGES limits, StreamSteeringApplied, and the steering_applied stream frame.
  • Additive only. Nothing in Sim uses these yet. Sim's side of a paired contract merges first, per the mothership sim-companion process, so sim-sync can pass on the worker PR.

Companion: simstudioai/mothership#929

Type of Change

  • Chore

Testing

bun run type-check in apps/sim, biome on both files, and the worker's sync-protocol.ts --check against this branch ("contract copies are in sync").

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@waleedlatif1
chore(contracts): sync the worker's steering v2 contract
c85533a 
Additive copies of the worker's SteerV2Request, SteerV2Response and steering_applied frame shape, generated from simstudioai/mothership#929. Nothing in Sim uses them yet; Sim's side of a paired contract merges first.
@vercel

vercel Bot commented Nov 6, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Nov 6, 2026 3:48am UTC

Request Review

@github-actions github-actions Bot added the requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep label Nov 6, 2026
@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

⚠️ Cross-repo companion check

One or more companion PRs aren't merged into staging yet. Merging this without them will leave copilot and sim out of sync — merge them in lockstep.

  • ❌ simstudioai/mothership#929 — OPEN, not merged (targets staging) — feat(worker): answer each steer with a durable receipt at /api/streams/steer/v2

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Nov 6, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Turn on auto-fix | Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Nov 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] This contract-only change appears safe to merge.

Summary

Adds the worker’s steering v2 contracts to Sim’s generated files.

  • Steering v2 defines bounded input and durable outcomes.
  • Run streams can identify when a steer reaches model input.

Tests were not run during this review.

Reviews (1) · Last reviewed commit: "chore(contracts): sync the worker's stee..." · Reviewed by Greptile

@waleedlatif1
waleedlatif1 merged commit bba90ab into staging Nov 6, 2026
61 checks passed
@waleedlatif1
waleedlatif1 deleted the chore/sync-steering-v2-contract branch November 6, 2026 03:50

This branch was previously deployed

1 inactive deployment
Preview — c85533a0 Deployed Nov 6, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees

No one assigned

Labels

requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@waleedlatif1