You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
chore: resolve merge conflicts between daemon_mode_b_main and main
Resolve all conflicts from merging origin/main into daemon_mode_b_main
using 84f4080 as the virtual merge base (last sync point from #4469).
Key resolution decisions:
- httpAcpBridge.ts: keep daemon's 102-line re-export shim (code lifted
to @qwen-code/acp-bridge per F1)
- httpAcpBridge.test.ts: removed (tests moved to acp-bridge/bridge.test.ts)
- TelemetryRuntimeConfig: added isInteractive() and
getOutboundCorrelationPropagateTraceContext() from main
- btwCommand.ts: keep daemon's clean buildBtwCacheSafeParams delegation
- sessionRecap.ts: main's getHistoryShallow() API + daemon's debug logging
- session-tracing.ts: daemon's ROOT_CONTEXT parent for bounded traces
- Session.ts: daemon's runInToolSpanContext + main's skipClassifierReason rename
Maintainers prioritize PRs with a clear reviewer test plan — without it, review may be delayed.
3
3
4
-
Maintainers prioritize PRs that include clear proof of work.
5
-
If a PR does not include enough validation detail to reproduce and verify the change efficiently, review may be delayed.
4
+
Don't hard-wrap paragraphs: GitHub renders single newlines as <br>, so wrapped text shows as a narrow column. Write each paragraph or list item as one long line.
6
5
-->
7
6
8
-
## Summary
7
+
## What this PR does
9
8
10
-
- What changed:
11
-
- Why it changed:
12
-
- Reviewer focus:
9
+
<!-- What this PR does. Describe the change in prose, not by file or function names. -->
13
10
14
-
## Validation
11
+
## Why it's needed
15
12
16
-
<!--
17
-
Be concrete. Do not write only "tested locally".
18
-
Include the exact commands, prompts, outputs, logs, screenshots, or videos that prove the change was actually run and observed.
13
+
<!-- Why it's needed: the motivation, the problem being solved, or the user-facing benefit. -->
14
+
15
+
## Reviewer Test Plan
19
16
20
-
For user-visible changes, bug fixes, CLI / TUI behavior changes, or interaction changes, include key screenshots or a short video.
21
-
When possible, show before/after behavior.
17
+
<!--
18
+
How a reviewer can confirm this PR: reproduction steps, expected vs observed behavior, and evidence. CI runs on macOS, Windows, and Linux — Tested on is what you verified locally.
22
19
23
-
If helpful, use the `e2e-testing` skill to gather stronger end-to-end validation evidence.
20
+
User-visible / TUI: Before/After with tmux-real-user-testing skill, screenshots, or a short recording.
21
+
Non–user-visible (refactor, types, docs): commands and output below; write N/A under Before/After.
<!-- How you reproduced it and what a reviewer should confirm — steps if needed, expected vs observed behavior. Focus on outcomes; paste logs or test output when helpful. -->
37
27
38
-
- Main risk or tradeoff:
39
-
- Not covered / not validated:
40
-
- Breaking changes / migration notes:
28
+
### Evidence (Before & After)
41
29
42
-
## Testing Matrix
30
+
<!-- User-visible / TUI changes: paste before-and-after screenshots, tmux logs, or video side by side. Non-UI changes (docs, refactor, types): N/A -->
43
31
44
-
<!--
45
-
Use:
46
-
- ✅ tested
47
-
- ⚠️ not tested
48
-
- N/A
49
-
If anything is ⚠️, explain why briefly below.
50
-
-->
32
+
### Tested on
33
+
34
+
| OS | Status |
35
+
| :--------: | :----: |
36
+
| 🍏 macOS ||
37
+
| 🪟 Windows ||
38
+
| 🐧 Linux ||
51
39
52
-
|| 🍏 | 🪟 | 🐧 |
53
-
| -------- | --- | --- | --- |
54
-
| npm run | ⚠️ | ⚠️ | ⚠️ |
55
-
| npx | ⚠️ | ⚠️ | ⚠️ |
56
-
| Docker | ⚠️ | ⚠️ | ⚠️ |
57
-
| Podman | ⚠️ | N/A | N/A |
58
-
| Seatbelt | ⚠️ | N/A | N/A |
40
+
<!-- ✅ tested · ⚠️ not tested · N/A -->
59
41
60
-
Testing matrix notes:
42
+
### Environment (optional)
61
43
62
-
-
44
+
<!-- Local runtime: e.g. npm run dev, Docker/Podman sandbox, seatbelt. N/A if only unit tests. -->
63
45
64
-
## Linked Issues / Bugs
46
+
## Risk & Scope
47
+
48
+
- Main risk or tradeoff:
49
+
- Not validated / out of scope:
50
+
- Breaking changes / migration notes:
51
+
52
+
## Linked Issues
65
53
66
54
<!--
67
-
If this PR fully resolves an issue, use one of:
68
-
- Closes #<issue_number>
69
-
- Fixes #<issue_number>
70
-
- Resolves #<issue_number>
55
+
Closes #N / Fixes #N / Resolves #N to auto-close.
56
+
Otherwise reference without a closing keyword.
57
+
-->
58
+
59
+
<details>
60
+
<summary>中文说明</summary>
71
61
72
-
Otherwise reference related issues without a closing keyword.
if [[ -z "${ALIYUN_OSS_ACCESS_KEY_ID}" || -z "${ALIYUN_OSS_ACCESS_KEY_SECRET}" ]]; then
435
-
echo "::error::Missing Aliyun OSS credentials. Set ALIYUN_OSS_ACCESS_KEY_ID and ALIYUN_OSS_ACCESS_KEY_SECRET in the production-release environment secrets."
436
-
exit 1
437
-
fi
438
-
439
-
ossutil config \
440
-
-e "${ALIYUN_OSS_ENDPOINT}" \
441
-
-i "${ALIYUN_OSS_ACCESS_KEY_ID}" \
442
-
-k "${ALIYUN_OSS_ACCESS_KEY_SECRET}" \
443
-
-L EN \
444
-
-c "${RUNNER_TEMP}/.ossutilconfig"
445
-
446
389
- name: 'Publish @qwen-code/qwen-code'
447
390
working-directory: 'dist'
448
391
run: |-
@@ -457,150 +400,37 @@ jobs:
457
400
env:
458
401
NODE_AUTH_TOKEN: '${{ secrets.NPM_TOKEN }}'
459
402
403
+
- name: 'Verify Standalone Archives'
404
+
run: |-
405
+
npm run verify:installation-release -- --dir dist/standalone
for asset in install-qwen-standalone.sh install-qwen-standalone.ps1 install-qwen-standalone.bat uninstall-qwen-standalone.sh uninstall-qwen-standalone.ps1 SHA256SUMS; do
0 commit comments