Skip to content

DeepSeek flash thinking off + lean 15s bash (ADR 0050 / 0051) - #675

Closed
justrach wants to merge 9 commits into
mainfrom
cursor/deepseek-fake-done-4ffc
Closed

DeepSeek flash thinking off + lean 15s bash (ADR 0050 / 0051)#675
justrach wants to merge 9 commits into
mainfrom
cursor/deepseek-fake-done-4ffc

Conversation

@justrach

@justrach justrach commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Hillclimb of DeepSeek flash SWE on the Codegraff seat. Make graff the harder DeepSeek harness than OpenCode without stealing Pi’s four-tool catalog or OpenCode’s ~1G heap.

Change

  1. Lean -p bounces a first-turn prose-only “done” (ADR 0048).
  2. Short Codegraff follow-up flakes retry twice (ADR 0049), including the 110-byte / ~450ms invalid_request_error / “Body must be valid JSON”. Auth / quota / a real invalid prompt stay fail-fast.
  3. graff-evals gains opencode-codegraff on the same CODEGRAFF_API_KEY seat.
  4. DeepSeek V4 flash default is thinking.type=disabled (ADR 0050). Official Chat Completions off switch. reasoning_effort=low still thinks (cookie-store once dumped 1.9MB / 75s). Live pong: omit/low ~20–30 reasoning_tokens; thinking.disabled bills 0. /effort high still thinks. GLM stays low-only.
  5. Lean -p bash auto-backgrounds at 15s (ADR 0051). Interactive stays 120s (ADR 0026). After thinking-off, wall was hung tests, not CoT.

Confirm — deepseek-v4-flash, --suite swe

harness pass wall RSS notes
graff (0047) 2/6 199s 91M one-shot no-edits
graff (0049, 072758) 5/6 362s 91M CoT dump on cookie-store
OpenCode (072310) 5/6 261s 1064M same seat
Pi (0047) 5/6 492s 186M reasoning: false
graff thinking off (074142) 4/6 210s 91M no 1.9MB dump; config-parse JSON-body flake
graff + flake retry (074659) 5/6 353s 91M json-stream 120s bash, cookie-store 71s bash
graff + 15s bash (075402) 4/6 143s 91M cookie 30s, json-stream 22s; validated died on balance reservation

Same label-sort miss on every harness. Do not stitch suite scores. 0-token follow-ups after 075402 are burnt gateway (reservation ~4655 µUSD), not a model result.

OpenCode first is the first JSONL event (~3.6s). Graff first is the stderr calling line. Do not treat 0.05s as a TUI steal.

Tests

Suite 1782 (baseline 1758). Tier 1 green.

Base is release/v0.0.281 (fold into #670). Do not merge to main / do not tag.

Open in Web Open in Cursor 

DeepSeek flash SWE one-shot 2/6 because three tasks described a
patch and never called a tool. Empty-completion retry only covers
whitespace. On lean -p, bounce once instead of stealing Pi's catalog.
Suite run-20260829-063114: 4/6 in 250s (was 2/6 in 199s). The three
4s no-edits now pass. Isolated serial retry recovered config-parse;
do not stitch to 5/6. Same label-sort miss.
@justrach
justrach marked this pull request as ready for review August 29, 2026 06:41
DeepSeek flash SWE 4/6 died on a ~110-byte / ~450ms api_error after
the first tool batch under -j 6. Isolated serial retry passed. Treat
internal/empty generic envelopes as bounded retries; leave
invalid_request / auth / quota fail-fast.

graff-evals gains opencode-codegraff on the same CODEGRAFF_API_KEY
gateway as graff-dev / pi-codegraff.
The -j 6 follow-up is still ~110 B / ~450 ms and was not matching
overloaded/internal. apiErrorMessage turns a missing message into
\"unknown error\"; retry that and truncated unparseable bodies.
Eval failures now keep a stderr tail so the next envelope is visible.
Graff 5/6 in 362s after short-flake retry (was 2/6, then 4/6).
OpenCode on the same seat is 5/6 in 261s / 1.0G RSS. Same
label-sort miss. Do not steal OpenCode's heap.
@cursor cursor Bot changed the title Bounce lean -p prose-only first completions (ADR 0048) DeepSeek flash SWE 5/6 + OpenCode A/B (ADR 0048 / 0049) Aug 29, 2026
reasoning_effort=low still emits reasoning_content (cookie-store dumped
1.9MB / 75s). Official off switch is thinking.type=disabled. /effort high
still thinks. GLM stays low-only. Do not steal Pi's catalog or OpenCode's heap.
@cursor cursor Bot changed the title DeepSeek flash SWE 5/6 + OpenCode A/B (ADR 0048 / 0049) DeepSeek flash thinking off at default low (ADR 0050) Aug 29, 2026
The DeepSeek -j 6 follow-up is still 110 bytes / ~450 ms, but the
envelope is now invalid_request_error / Body must be valid JSON. The
invalid needle fail-fasted it. Our stringify just succeeded on call 1;
auth/quota/a real invalid prompt stay fail-fast.
After thinking-off, DeepSeek SWE wall was hung foreground tests
(json-stream 120s, cookie-store 71s). Interactive stays 120s (ADR 0026).
The process is not killed. Do not steal Pi's catalog.
@cursor cursor Bot changed the title DeepSeek flash thinking off at default low (ADR 0050) DeepSeek flash thinking off + lean 15s bash (ADR 0050 / 0051) Aug 29, 2026
Gateway pong: low still thinks; thinking.disabled bills 0. SWE 5/6
in 353s was hung bash; 15s oneshot wait made cookie-store 30s /
json-stream 22s. Later 0-token runs are burnt balance, not a result.
Base automatically changed from release/v0.0.281 to main August 31, 2026 00:49
cursor Bot pushed a commit that referenced this pull request Aug 31, 2026
The parked Smolify and idle-localhost PRs are closed as abandoned vs tip;
issues stay inventory. Live leftovers remain #675 and #677.
cursor Bot pushed a commit that referenced this pull request Aug 31, 2026
#677 leftover of #554: /teleport restores a snapshot tar onto another
CLI backend; /snapshot gc keeps the newest n trees. ADR 0051 (their
0042 collided with TUI claims).

#675: lean -p bounces prose-only first "done"; short Codegraff flakes
retry twice; DeepSeek thinking off at default low; lean bash
auto-backgrounds at 15s. ADRs remapped 0048-0051 -> 0052-0055.

Suite floor 1861. Keep-alive-only bodies share the short-flake retry
so the request-scratch split stays.

Co-authored-by: yxlyx <85774423+yxlyx@users.noreply.github.com>
@justrach

Copy link
Copy Markdown
Owner Author

Folded into #696 (release/v0.0.282). Unique work kept; ADRs remapped 0048–0551 → 0052–0555 because 282 already owns 0048–051 (TLS generations, session branches, MCP TLS reuse, teleport/GC).

Did not take this head's src/agent_request.zig or graff-evals/run.py — 282 already split keep-alive into agent_request_scratch.zig and the runner already has {sandbox} / opencode-json.

Tier 1 green: suite 1861. Closing as superseded.

@justrach justrach closed this Aug 31, 2026
@justrach

Copy link
Copy Markdown
Owner Author

ADR remap correction: 0048–0051 → 0052–0055 (typo in the previous comment).

@justrach
justrach deleted the cursor/deepseek-fake-done-4ffc branch August 31, 2026 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants