-
Notifications
You must be signed in to change notification settings - Fork 2
sync: merge upstream v1.0.131 #84
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
…ould only be available to primary agents (sst#4913) Co-authored-by: GitHub Action <[email protected]>
…ge (implements sst#4847) (sst#4855) Co-authored-by: GitHub Action <[email protected]> Co-authored-by: Aiden Cline <[email protected]>
Co-authored-by: Aiden Cline <[email protected]>
Add comprehensive test suite for Provider module to ensure safe refactoring of provider internals. Tests cover: - Provider loading from env vars and config - Provider filtering (disabled_providers, enabled_providers) - Model whitelist/blacklist - Model aliasing and custom providers - getModel, getProvider, closest, defaultModel functions Also adds Env module for instance-scoped environment variable access, enabling isolated test environments without global state pollution.
Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: thdxr <[email protected]>
Co-authored-by: Aiden Cline <[email protected]>
Resolved conflicts: - AGENTS.md: keep ours (fork-specific instructions) - bun.lock: regenerated with bun install - packages/opencode/script/build.ts: keep ours (shuvcode naming + OPENCODE_BASE_VERSION) - packages/opencode/src/session/index.ts: accept upstream (safer cache?.read optional chaining) - packages/opencode/src/session/processor.ts: keep ours (Token import) - packages/opencode/src/tool/task.ts: keep ours (Wildcard + filterSubagents) - packages/console/app/src/routes/workspace/[id]/graph-section.tsx: accept upstream pattern - package.json files: accept upstream versions (1.0.131) - packages/extensions/zed/extension.toml: accept upstream versions (1.0.131)
|
Caution Review failedThe pull request is closed. WalkthroughRelease version bump from 1.0.130 to 1.0.131 across multiple packages and configuration files. Includes refactoring of graph-section component data loading to use effect-based reactive flow, optional chaining safety improvements, and removal of an unused dependency. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes
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 (20)
Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
https://github.com/Latitudes-Dev/shuvcode/blob/ed09d7236df328f396e41112561e52f01c864482/packages/console/app/src/routes/workspace/[id]/graph-section.tsx#L206-L210
Restore SSR guard around DOM access
The chart config memo now hits getComputedStyle(document.documentElement) without the previous if (typeof window === "undefined") return null guard, so server-side rendering this route will throw ReferenceError: document is not defined before any data loads. The console app renders on the server, so any SSR render or test that imports this component will crash until the DOM access is gated to the client.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Summary
Resolved Conflicts
AGENTS.mdbun.lockbun installpackages/opencode/script/build.tspackages/opencode/src/session/index.tscache?.readoptional chaining)packages/opencode/src/session/processor.tspackages/opencode/src/tool/task.tspackages/console/app/src/routes/workspace/[id]/graph-section.tsxpackage.jsonfilespackages/extensions/zed/extension.tomlValidation
Closes #83
Summary by CodeRabbit
Chores
Bug Fixes
Improvements
✏️ Tip: You can customize this high-level summary in your review settings.