-
Notifications
You must be signed in to change notification settings - Fork 3
sync: merge upstream v1.0.133 #87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: Github Action <action@github.com> Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com>
Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
…ot explicitly defined
Co-authored-by: Github Action <action@github.com> Co-authored-by: Dax Raad <d@ironbay.co>
This reverts commit d763c11.
Resolved conflicts: - github/action.yml: merged both - kept fork-specific OPENCODE_BIN_PATH and added upstream PROMPT env var - packages/opencode/src/cli/cmd/tui/component/prompt/index.tsx: merged both - kept our 'text' getter and added upstream 'current' getter to PromptRef - packages/opencode/src/cli/cmd/tui/routes/session/index.tsx: merged both - kept bash output view, search mode, initialValue prop, and integrated upstream promptRef.set()
|
Caution Review failedThe pull request is closed. WalkthroughThis release bumps versions from v1.0.132 to v1.0.133 and introduces sticky provider session persistence, custom prompt support via environment variables and GitHub Actions configuration, refactors TUI prompt state preservation across routes, and updates provider/model data assembly with explicit aliasing and async loader handling. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Handler
participant StickyTracker as Sticky<br/>Tracker
participant KV as Gateway KV
participant Provider as Provider<br/>Selection
User->>Handler: Request with session ID
Handler->>StickyTracker: createStickyTracker(enabled, sessionId)
StickyTracker->>KV: Get sticky:${sessionId}
KV-->>StickyTracker: Return cached provider ID (or null)
StickyTracker-->>Handler: Return tracker instance
alt Cached Provider Available
Handler->>Provider: selectProvider(..., stickyProvider)
Provider-->>Handler: Use cached provider
else No Cache or Retry
Handler->>Provider: selectProvider with normal logic
Provider-->>Handler: Selected provider ID
Handler->>StickyTracker: Store provider selection
StickyTracker->>KV: Put sticky:${sessionId} → providerId (TTL: 24h)
KV-->>StickyTracker: Stored
end
Handler-->>User: Response with provider
sequenceDiagram
participant User
participant Session as Session<br/>Action
participant PromptRef as PromptRef<br/>Context
participant Route as Router
participant Home as Home Route
User->>Session: Trigger "New session"
Session->>PromptRef: usePromptRef()
PromptRef-->>Session: promptRef instance
Session->>PromptRef: Read promptRef.current.current.input
PromptRef-->>Session: Current prompt text
alt Prompt Input Exists
Session->>Route: Navigate to home with initialPrompt
Route->>Home: Provide route data
Home->>Home: onMount: prefill from route.initialPrompt
else Empty Prompt
Session->>Route: Navigate to home (no initialPrompt)
Route->>Home: Provide route data
Home->>Home: onMount: fallback to args.prompt
end
Home-->>User: Display home with prefilled prompt
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Areas requiring extra attention:
Possibly related PRs
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (44)
Comment |
Summary
Conflict Resolutions
OPENCODE_BIN_PATHand added upstreamPROMPTenv vartextgetter and added upstreamcurrentgetter to PromptRef typeinitialValueprop, and integrated upstreampromptRef.set()callCloses #86
Summary by CodeRabbit
Release Notes v1.0.133
New Features
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.