fix: correct improvement report accuracy (reactiveCompact + speculation)#35
Closed
fix: correct improvement report accuracy (reactiveCompact + speculation)#35
Conversation
1. reactiveCompact.ts does NOT exist — changed to apiMicrocompact.ts (API-native context management: clear_tool_uses / clear_thinking) 2. Qwen Code speculation is NOT incomplete — v0.15.0 (PR #2525) has full system: 563-line speculation.ts + overlayFs + speculationToolGate + acceptSpeculation + pipelined suggestions. Only default-disabled. 3. Claude Code speculation.ts: 992 → 991 lines (wc -l verified) 4. Updated summary table: speculation gap from "中等差距" to "小差距（仅默认值）" Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Owner
Author
wenshao
added a commit
that referenced
this pull request
Apr 4, 2026
p2-stability #35: System prompt behavioral guidance for dangerous operations — 4 categories of risky actions, behavioral principles (investigate before deleting, resolve conflicts not discard), and approval scope limitation (one-time approval != permanent). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
wenshao
added a commit
that referenced
this pull request
Apr 4, 2026
p0-p1 #47: System prompt content gaps — OWASP security types, prompt injection detection guidance, code style constraints (no extra features/ abstractions/docs), output format rules (file_path:line_number, owner/repo#123). Also added p2-stability #35: dangerous action behavioral guidance. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
wenshao
added a commit
that referenced
this pull request
Apr 5, 2026
Closed PR #75 — 10 of 16 items duplicated existing 187. Source code references all verified correct (glm-5.1 self-corrected 8 errors). Extracted 4 new items to p2-tools: - #35 preserveQuoteStyle (quote style preservation in edits) - #36 areFileEditsInputsEquivalent (skip duplicate edits) - #37 MCP channel plugin allowlist - #38 SDK message type enrichment (11 → 30+ types) Total: 191 items. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
wenshao
added a commit
that referenced
this pull request
Apr 5, 2026
PR #105: 0/5 new (all duplicated). PR #108: 5/5 new (useSSHSession, useSwarmPermissionPoller, useTasksV2, useArrowKeyHistory, useCanUseTool). PR #109: 3/5 new (useSearchInput, usePasteHandler, useScheduledTasks). New P3 items: #35-#42. Total: 225 items. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes 2 factual errors in
qwen-code-improvement-report.md(written by qwen3.6-plus in PR #34):1.
reactiveCompact.tsdoes not existls services/compact/shows noreactiveCompact.ts;apiMicrocompact.ts(154 lines) exists2. Qwen Code speculation is complete, not "有骨架但功能不完整"
enableSpeculation: falseis the only gapTest plan
wc -lverified: speculation.ts=991, apiMicrocompact.ts=154, reactiveCompact.ts=DNE🤖 Generated with Claude Code