Skip to content

[v3.8.50] fix(claude): preserve signed thinking turns during obfuscation - #8629

Merged
diegosouzapw merged 2 commits into
cheaper-inference:release/v3.8.50from
costaeder:fix/claude-signed-thinking-obfuscation
Aug 6, 2026
Merged

diegosouzapw merged 2 commits into
cheaper-inference:release/v3.8.50from
costaeder:fix/claude-signed-thinking-obfuscation

Conversation

@costaeder

Copy link
Copy Markdown
Contributor

Anthropic signatures cover a thinking turn and its text siblings. Obfuscating a sibling text block invalidates the signature and causes the next request to fail.\n\nSkip message-level obfuscation for turns containing thinking or redacted_thinking blocks, while keeping normal text-turn obfuscation unchanged.\n\nTests: targeted obfuscation and system-transform suites (28 passing).

@costaeder
costaeder requested a review from diegosouzapw as a code owner July 26, 2026 02:32
@diegosouzapw
diegosouzapw changed the base branch from main to release/v3.8.49 July 26, 2026 23:15
@diegosouzapw diegosouzapw added the deferred-v3.8.50 Adiada para o ciclo v3.8.50 (validacao VPS, refactor, ou escopo grande) label Jul 27, 2026
@diegosouzapw diegosouzapw changed the title fix(claude): preserve signed thinking turns during obfuscation [v3.8.50] fix(claude): preserve signed thinking turns during obfuscation Jul 27, 2026
@diegosouzapw

Copy link
Copy Markdown
Collaborator

Heads-up: this PR is running against a stale base

release/v3.8.49 has moved 116 commits ahead of the base this PR was last built against, and several gate failures that were red on the older base have since been repaired on the branch itself:

Gate class Repaired by
No new ESLint warnings (exit 2 — orphaned suppressions, not a lint violation) #8706, #8831
Fast Quality Gates / check:file-size / Quality Ratchet #8585, #8767, plus the post-merge-train re-pins
Docs Gates (fast-path) (missing Polish API_REFERENCE) #8831
backoff-clamp + compressionDetailNormalizers unit tests #8706

I verified this on a clean checkout of the current tip (3b515d90b3): lint:json --max-warnings 0, check:file-size, check:docs-sync, error-classification.test.ts (24/24) and check-db-rules.test.ts (22/22) all pass with no changes.

Currently failing here:

  • Build
  • Lint
  • Quality Gates (Extended)
  • Quality Ratchet

Please update the branch (the Update branch button, or rebase onto origin/release/v3.8.49) and let CI re-run. That should clear any failure caused by the stale base.

Note this is not a promise that every red disappears — a failure specific to the changes in this PR will survive the rebase, and Build (advisory) is advisory and does not block. Rebasing just removes the noise so what is left is actually yours.

No action was taken on this PR beyond this comment.

@diegosouzapw
diegosouzapw changed the base branch from release/v3.8.49 to release/v3.8.50 July 28, 2026 18:42
@diegosouzapw

Copy link
Copy Markdown
Collaborator

Re-homed to release/v3.8.50: v3.8.49 entered its release freeze, so the branch now belongs to the release captain and development continues on the next cycle. Nothing is wrong with this PR — it just needed a live base. No action needed from you; CI will re-run against the new base.

Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com>
@diegosouzapw
diegosouzapw merged commit aa7b391 into cheaper-inference:release/v3.8.50 Aug 6, 2026
4 of 5 checks passed
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deferred-v3.8.50 Adiada para o ciclo v3.8.50 (validacao VPS, refactor, ou escopo grande)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants