anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-22 18:25:32 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

[PR #860] [MERGED] fix(session): add fallback for undefined output token limit #9729

New Issue
Closed
opened 2026-02-16 18:14:04 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 18:14:04 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/anomalyco/opencode/pull/860
Author: @svkozak
Created: 7/10/2025
Status: ✅ Merged
Merged: 7/11/2025
Merged by: @thdxr

Base: dev ← Head: fix

📝 Commits (1)

  • 4ab12b4 fix(session): add fallback for undefined output token limit

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 packages/opencode/src/session/index.ts (+1 -1)

📄 Description

Summary

  • Add fallback value for output token limit calculation to prevent undefined values
  • Ensures outputLimit always has a valid numeric value when Math.min() returns undefined

Test plan

  • Verify the change handles edge cases where model.info.limit.output is undefined
  • Confirm existing functionality remains intact

🤖 Generated with opencode

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/anomalyco/opencode/pull/860 **Author:** [@svkozak](https://github.com/svkozak) **Created:** 7/10/2025 **Status:** ✅ Merged **Merged:** 7/11/2025 **Merged by:** [@thdxr](https://github.com/thdxr) **Base:** `dev` ← **Head:** `fix` --- ### 📝 Commits (1) - [`4ab12b4`](https://github.com/anomalyco/opencode/commit/4ab12b4d66711ac2462e2edbd3e2711557e2a2df) fix(session): add fallback for undefined output token limit ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `packages/opencode/src/session/index.ts` (+1 -1) </details> ### 📄 Description ## Summary - Add fallback value for output token limit calculation to prevent undefined values - Ensures `outputLimit` always has a valid numeric value when `Math.min()` returns undefined ## Test plan - [x] Verify the change handles edge cases where `model.info.limit.output` is undefined - [x] Confirm existing functionality remains intact 🤖 Generated with [opencode](https://opencode.ai) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 18:14:04 -05:00
yindo closed this issue 2026-02-16 18:14:04 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#9729
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?