Skip to content

fix: encode message content with split_special_tokens=True - #116

Open
hubert-marek wants to merge 2 commits into
mainfrom
fix/no-special-token-injection-from-content
Open

fix: encode message content with split_special_tokens=True#116
hubert-marek wants to merge 2 commits into
mainfrom
fix/no-special-token-injection-from-content

fix: exclude GLM-4.5-Air from split_special_tokens

23a7029
Select commit
Loading
Failed to load commit list.
MacroscopeApp / Macroscope - Approvability Check completed Aug 5, 2026 in 4m 59s

Not eligible (click on check for details)

  • Eligibility
  • Correctness

Note

More information about how approvability works can be found in our Help Center.

Details

Eligibility
Unresolved review comments identify that this fix is incomplete — at least two code paths (minimax_m2.py tool encoding and hy3.py segment encoding) still lack the split_special_tokens=True protection, leaving the security issue partially unaddressed.

Notes:
No code owners found in target merge base d470786