feat(background-agent): support 0 as unlimited concurrency

code-yeongyu · code-yeongyu · commit 1267a1bac50f · 2026-01-08T23:05:12.000+09:00
Setting concurrency to 0 means unlimited (Infinity). Works for defaultConcurrency, providerConcurrency, and modelConcurrency. 🤖 Generated with [OhMyOpenCode](https://github.com/code-yeongyu/oh-my-opencode)
diff --git a/src/features/background-agent/concurrency.ts b/src/features/background-agent/concurrency.ts
@@ -10,14 +10,20 @@ export class ConcurrencyManager {
   }
 
   getConcurrencyLimit(model: string): number {
-    if (this.config?.modelConcurrency?.[model]) {
-      return this.config.modelConcurrency[model]
+    const modelLimit = this.config?.modelConcurrency?.[model]
+    if (modelLimit !== undefined) {
+      return modelLimit === 0 ? Infinity : modelLimit
     }
     const provider = model.split('/')[0]
-    if (this.config?.providerConcurrency?.[provider]) {
-      return this.config.providerConcurrency[provider]
+    const providerLimit = this.config?.providerConcurrency?.[provider]
+    if (providerLimit !== undefined) {
+      return providerLimit === 0 ? Infinity : providerLimit
     }
-    return this.config?.defaultConcurrency ?? 5
+    const defaultLimit = this.config?.defaultConcurrency
+    if (defaultLimit !== undefined) {
+      return defaultLimit === 0 ? Infinity : defaultLimit
+    }
+    return 5
   }
 
   async acquire(model: string): Promise<void> {

Original file line number	Diff line number	Diff line change
`@@ -10,14 +10,20 @@ export class ConcurrencyManager {`
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`getConcurrencyLimit(model: string): number {`
`13`		`- if (this.config?.modelConcurrency?.[model]) {`
`14`		`- return this.config.modelConcurrency[model]`
	`13`	`+ const modelLimit = this.config?.modelConcurrency?.[model]`
	`14`	`+ if (modelLimit !== undefined) {`
	`15`	`+ return modelLimit === 0 ? Infinity : modelLimit`
`15`	`16`	`}`
`16`	`17`	`const provider = model.split('/')[0]`
`17`		`- if (this.config?.providerConcurrency?.[provider]) {`
`18`		`- return this.config.providerConcurrency[provider]`
	`18`	`+ const providerLimit = this.config?.providerConcurrency?.[provider]`
	`19`	`+ if (providerLimit !== undefined) {`
	`20`	`+ return providerLimit === 0 ? Infinity : providerLimit`
`19`	`21`	`}`
`20`		`- return this.config?.defaultConcurrency ?? 5`
	`22`	`+ const defaultLimit = this.config?.defaultConcurrency`
	`23`	`+ if (defaultLimit !== undefined) {`
	`24`	`+ return defaultLimit === 0 ? Infinity : defaultLimit`
	`25`	`+ }`
	`26`	`+ return 5`
`21`	`27`	`}`
`22`	`28`
`23`	`29`	`async acquire(model: string): Promise<void> {`