Repository navigation
Train: #737 (0360 phase 2) on master c0021a5 - #741
Merged
Merged
Conversation
… tojson string separators
…C1083; glibc has a features.h)
…ts its own cases (CI and preflight)
…(Codex review) - strftime_now checks the sequence cap as its result grows, so a format of repeated directives fails with ResourceLimitExceeded instead of building a string past maxSequenceSize. - A string given as tojson separators stops unpacking at the third character. - tojson evaluates its indent argument for string values too, as Python does, and only then ignores it. - strftime_now and raise_exception accept a string that host data passes as a view; such a format used to fail as a non-string. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AWNL7nyP1j6LYCMdbnWsHk
…separators (Codex review) - PythonJsonWriter checks the document size before it appends an item or key separator or an indent, and once more at the end, so host strings of any length cannot grow it past maxSequenceSize. - separators accepts a dict and unpacks its keys, as json.dumps does; a dict of more than two keys fails before they are copied. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AWNL7nyP1j6LYCMdbnWsHk
rmorozov
force-pushed
the
claude/project-thread-53pwid
branch
from
November 2, 2026 14:53
b32815c to
bcd5728
Compare
rmorozov
force-pushed
the
claude/project-thread-53pwid
branch
from
November 2, 2026 15:48
bcd5728 to
0dfaa72
Compare
rmorozov
changed the title
rmorozov
marked this pull request as ready for review
November 2, 2026 16:38
This was referenced Nov 2, 2026
Merged
rmorozov
pushed a commit
that referenced
this pull request
Nov 2, 2026
…_methods.cpp includes Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AWNL7m7VMb31R2aVZD7WkR
This was referenced Nov 2, 2026
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
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.
Integration train with one PR: #737 (6682416) merged onto master c0021a5, so CI and the perf verdict run on the head that will land (master moved by #735 and #736 since #737's own CI).
#733 (group C) is out of this train: the perf track held it (Render/config_file +1.46%, Render/many_tags +1.37%, Load/many_tags +1.28%, a fixed +5 Ir per render). Its owner reworks the rises; it restacks afterwards. #726 (group D) follows both and conflicts with #737 in src/value_methods.cpp.
Merging this marks #737 merged. Codex: the security review never ran on #737.
🤖 Generated with Claude Code
https://claude.ai/code/session_01DTQTriEsFrrBgcYNKDUUF4