Skip to content

Merge train 29: Settings::Limits (#601) - #610

Merged
rmorozov merged 12 commits into
masterfrom
claude/project-thread-53pwid
Oct 19, 2026
Merged

rmorozov merged 12 commits into
masterfrom
claude/project-thread-53pwid

Conversation

@rmorozov

Copy link
Copy Markdown
Member

Train of one: #601 (0262 Settings::Limits) on master 93c0934, merged with --no-ff. Its base was stale (contained 9cf8586, before train 28's ABI namespace).

Conflicts resolved by the steward:

  • src/global_functions.cpp and src/internal_value.cpp: include lines (recursion_guard.h and template_dict.h both kept, sorted).
  • src/internal_value.cpp: ThrowSequenceTooLong now sits inside inline namespace v2. python3 scripts/abi_namespace.py re-run on the train: 0 files rewritten.

Task index regenerated (0262 done, 0282 added); docs/parity.md count unchanged (1995). Local: build clean, 4/4 ctest binaries pass, clang-format clean.

Roles: no subagents; no verifier round on the train itself (owner ran theirs). Codex on #601: 2 findings, both fixed, re-review clean. Perf track: approved at f3b25cd (Render/plain_text +0.76% named exception). Red pushes: none so far.

Known overlap for later: #597 (0253) also adds ResourceLimitExceeded = 18 and changes the catch blocks in src/template_impl.h Render; it will need to rebase over this train.


Generated by Claude Code

claude added 12 commits October 18, 2026 12:47
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
…Load, output checked per step

With every limit at its default a render takes no new per-step work: a render with
limits makes CheckStack take its slow path, which counts the step and checks the
output size. Parse limits live in a thread-local set for the Load.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
…nternal sequence cap

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
…9 loop in the defaults pass

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
…s uint64

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
…ds as uint64

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
…elector and the cycle collector); close 0262

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
…rce; parse limits in FindUndeclaredVariables

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01USAwpuAiFNpCMNpcRmwvsh
@rmorozov
rmorozov marked this pull request as ready for review October 19, 2026 12:40
@rmorozov
rmorozov merged commit 5fe00b9 into master Oct 19, 2026
41 checks passed
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