Repository navigation
chore: backup #448
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -43,10 +43,10 @@ | |||||
| "name": "Claude Haiku 4.5 (via CLIProxyAPI)" | ||||||
| }, | ||||||
| "z-ai/glm-4.6": { | ||||||
| "name": "GLM-4.6 (via OpenRouter and Z.AI Coding Plan)" | ||||||
| }, | ||||||
| "z-ai/glm-4.7": { | ||||||
| "name": "GLM-4.7 (via OpenRouter and Z.AI Coding Plan)" | ||||||
|
Contributor
There was a problem hiding this comment. The model description here states that
Comment on lines
45
to
+49
There was a problem hiding this comment.
The new Useful? React with 👍 / 👎. There was a problem hiding this comment. The model name mentions "via OpenRouter and Z.AI Coding Plan", but z-ai/glm-4.7 was removed from the OpenRouter configuration in config/cliproxyapi/config.yaml. The name should be updated to reflect that this model is only available via Z.AI Coding Plan, not OpenRouter.
Suggested change
|
||||||
| } | ||||||
| } | ||||||
| }, | ||||||
|
|
@@ -85,7 +85,9 @@ | |||||
| "reasoningEffort": "medium", | ||||||
| "reasoningSummary": "auto", | ||||||
| "textVerbosity": "medium", | ||||||
| "include": [ | ||||||
| "reasoning.encrypted_content" | ||||||
| ] | ||||||
| }, | ||||||
| "models": { | ||||||
| "gpt-5-codex-low-oauth": { | ||||||
|
|
@@ -151,39 +153,49 @@ | |||||
| "deepseek/deepseek-chat-v3.1:free": { | ||||||
| "options": { | ||||||
| "provider": { | ||||||
| "order": [ | ||||||
| "baseten" | ||||||
| ], | ||||||
| "allow_fallbacks": false | ||||||
| } | ||||||
| } | ||||||
| }, | ||||||
| "moonshotai/kimi-k2": { | ||||||
| "options": { | ||||||
| "provider": { | ||||||
| "order": [ | ||||||
| "baseten" | ||||||
| ], | ||||||
| "allow_fallbacks": false | ||||||
| } | ||||||
| } | ||||||
| }, | ||||||
| "moonshotai/kimi-k2:free": { | ||||||
| "options": { | ||||||
| "provider": { | ||||||
| "order": [ | ||||||
| "baseten" | ||||||
| ], | ||||||
| "allow_fallbacks": false | ||||||
| } | ||||||
| } | ||||||
| }, | ||||||
| "openai/gpt-oss-120b:free": { | ||||||
| "options": { | ||||||
| "provider": { | ||||||
| "order": [ | ||||||
| "baseten" | ||||||
| ], | ||||||
| "allow_fallbacks": false | ||||||
| } | ||||||
| } | ||||||
| }, | ||||||
| "qwen/qwen3-coder:free": { | ||||||
| "options": { | ||||||
| "provider": { | ||||||
| "order": [ | ||||||
| "baseten" | ||||||
| ], | ||||||
| "allow_fallbacks": false | ||||||
| } | ||||||
| } | ||||||
|
|
@@ -198,7 +210,17 @@ | |||||
| "glm-4.6": { | ||||||
| "name": "GLM-4.6 (via Z-AI)", | ||||||
| }, | ||||||
|
There was a problem hiding this comment. Trailing comma after the last property in the JSON object. This will cause a JSON parsing error in most parsers, although JSONC typically allows trailing commas in arrays and objects. |
||||||
| }, | ||||||
|
There was a problem hiding this comment. Trailing comma after the last property in the models object. This will cause a JSON parsing error in most parsers, although JSONC typically allows trailing commas in arrays and objects. |
||||||
| "options": { | ||||||
| "apiKey": "{env:ZAI_API_KEY}", | ||||||
| } | ||||||
|
Comment on lines
+215
to
+216
There was a problem hiding this comment. Trailing comma after the last property in the options object. This will cause a JSON parsing error in most parsers, although JSONC typically allows trailing commas in arrays and objects. |
||||||
| }, | ||||||
| "zai-coding-plan": { | ||||||
| "models": { | ||||||
| "glm-4.6": { | ||||||
| "name": "GLM-4.6 Coding Plan (via Z-AI)" | ||||||
| }, | ||||||
| "glm-4.7": { | ||||||
| "name": "GLM-4.7 Coding Plan (via Z-AI)" | ||||||
| } | ||||||
| }, | ||||||
|
Comment on lines
+218
to
226
There was a problem hiding this comment. The zai-coding-plan provider is missing required configuration fields. Based on the pattern used by other providers like cliproxyapi, lmstudio, and ollama, it should include an npm package specification and a baseURL in the options section. Without these fields, the provider may not function correctly. |
||||||
|
|
@@ -210,4 +232,4 @@ | |||||
| "tui": { | ||||||
| "scroll_speed": 3 | ||||||
| } | ||||||
| } | ||||||
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.
Adding a timeout is a good way to prevent the command from hanging. However, hardcoding the timeout value
5in multiple places makes it harder to maintain. Consider defining a variable for the timeout at the top of theMakefile. This would allow for easy adjustment and ensure consistency across alllaunchctltargets.For example:
This change would apply to all similar
launchctlcommands modified in this PR.