Skip to content

Commit 06c781b

Browse files
iRoninsudosubin
andauthored
fix(pi-kilocode): let detectCompat auto-detect supportsDeveloperRole for kilocode models (#16)
* fix(pi-kilocode): let detectCompat auto-detect supportsDeveloperRole for kilocode models Remove the hardcoded supportsDeveloperRole: false override from toPiModel() so pi-ai's detectCompat() handles it automatically. kilo.ai is not in detectCompat's isNonStandard list, so the developer role is enabled, matching direct OpenRouter behaviour. Closes #15 * Apply suggestion from @sudosubin --------- Co-authored-by: Subin Kim <sudosubin@gmail.com>
1 parent 6bc9d5f commit 06c781b

2 files changed

Lines changed: 1 addition & 2 deletions

File tree

pi-kilocode/src/provider/models.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ function toPiModel(m: KiloModel): ProviderModelConfig {
134134
contextWindow: m.context_length,
135135
maxTokens: maxOut ?? 8192,
136136
compat: {
137-
supportsDeveloperRole: false,
137+
// supportsDeveloperRole should be automatically inferred as true.
138138
supportsStore: false,
139139
thinkingFormat: supportsReasoning ? "openrouter" : undefined,
140140
},

pi-kilocode/test/models.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,6 @@ test("convertToPiModels drops image-only models and maps capabilities/pricing",
6969
cacheWrite: 3.5,
7070
});
7171
assert.deepEqual(model.compat, {
72-
supportsDeveloperRole: false,
7372
supportsStore: false,
7473
thinkingFormat: "openrouter",
7574
});

0 commit comments

Comments
 (0)