Conversation
…ould only be available to primary agents (anomalyco#4913) Co-authored-by: GitHub Action <action@github.com>
…ge (implements anomalyco#4847) (anomalyco#4855) Co-authored-by: GitHub Action <action@github.com> Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
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 <thdxr@users.noreply.github.com>
Co-authored-by: Aiden Cline <aidenpcline@gmail.com>
Resolved conflicts: - AGENTS.md: kept fork-specific IMPORTANT section + upstream Debugging section - packages/opencode/script/build.ts: kept shuvcode naming + upstream Windows path fix - packages/opencode/src/session/processor.ts: kept both Token import and Provider type import - packages/opencode/src/tool/task.ts: kept filterSubagents export + upstream Config import
Fixed type error in prompt.ts: model.modelID -> model.id
|
Caution Review failedThe pull request is closed. WalkthroughThis release (v1.0.130) refactors provider and session management systems, transitioning from ModelsDev types to new Provider.Model and Provider.Info abstractions. A GitHub Actions guidelines workflow is added, style guide documentation introduced, UI theming enhanced with color-scheme awareness, and comprehensive provider system tests added. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 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 (3)
📒 Files selected for processing (54)
Comment |
Summary
Conflicts Resolved
Additional Fix
model.modelID->model.id(Provider.Model usesidnotmodelID)Closes #80
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Chores
✏️ Tip: You can customize this high-level summary in your review settings.