abbyswag/qwen-code
1
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2025-12-19 09:33:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(acp): replace EOL with newline for content splitting

Browse Source 
- Replace `EOL` from `node:os` with `\n` for consistent line splitting in ACP connection output processing
- This ensures cross-platform compatibility since `EOL` is platform-specific while `\n` is universally used in text decoding
- The change maintains the same behavior on all platforms by using standard newline characters
This commit is contained in:
Matthieu Beaumont
2025-11-08 14:54:43 +01:00
parent 3c01c7153b
commit 6aaac12d70
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/cli/src/zed-integration/acp.ts
View File

@@ -7,7 +7,6 @@
/* ACP defines a schema for a simple (experimental) JSON-RPC protocol that allows GUI applications to interact with agents. */ /* ACP defines a schema for a simple (experimental) JSON-RPC protocol that allows GUI applications to interact with agents. */
import { z } from 'zod'; import { z } from 'zod';
import { EOL } from 'node:os';
import * as schema from './schema.js'; import * as schema from './schema.js';
export * from './schema.js'; export * from './schema.js';
@@ -173,7 +172,7 @@ class Connection {
const decoder = new TextDecoder(); const decoder = new TextDecoder();
for await (const chunk of output) { for await (const chunk of output) {
content += decoder.decode(chunk, { stream: true }); content += decoder.decode(chunk, { stream: true });
const lines = content.split(EOL); const lines = content.split('\n');
content = lines.pop() || ''; content = lines.pop() || '';
for (const line of lines) { for (const line of lines) {