Repository navigation
docker: name the memory and scratch controls in the entrypoint's usage text - #543
Merged
Merged
Conversation
…e text The release entrypoint's usage text is its header comment. Add one paragraph that names the memory and scratch environment variables an operator can set (PINEFORGE_COMMAND_SCRATCH_MAX_BYTES, PINEFORGE_COMMAND_SCRATCH_DIR or TMPDIR, PF_SCRATCH_ARENA_FD with PF_SCRATCH_ARENA_BYTES, PF_SCRATCH_ARENA_JAILED, PF_SCRATCH_WINDOW_BYTES, PINEFORGE_RUN_JSON_PROFILE) and points to docs/pages/memory-management.md for defaults, units and limits. All of them reach the engine by inheritance: the entrypoint takes no command-line flags and does not scrub the environment. Comment-only: every code line of docker/entrypoint.sh is unchanged. The nine inserted lines shift the later line numbers, so the citations of docker/entrypoint.sh by line number in tests/ownership/driver.py and docs/design/BARS-BIN-INPUT.md are re-pinned to the same lines of code. Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Takes the docs-only change of #542 (docs/design/arena-bound-corpus-census.md, docs/design/arena-bound.md, docs/pages/memory-management.md, docs/scratch-store.md). No conflict; none of those files is touched by this branch, and the entrypoint comment block and the re-pinned citations are unchanged. Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
luisleo526
marked this pull request as ready for review
October 26, 2026 21:50
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.
What
The release entrypoint's usage text is its header comment. This adds one paragraph that names the memory and scratch environment variables an operator can set (
PINEFORGE_COMMAND_SCRATCH_MAX_BYTES,PINEFORGE_COMMAND_SCRATCH_DIRorTMPDIR,PF_SCRATCH_ARENA_FDwithPF_SCRATCH_ARENA_BYTES,PF_SCRATCH_ARENA_JAILED,PF_SCRATCH_WINDOW_BYTES,PINEFORGE_RUN_JSON_PROFILE) and points todocs/pages/memory-management.mdfor defaults, units and limits.Comment-only: every code line of
docker/entrypoint.shis unchanged (nine comment lines are added beforeset -euo pipefail; no line is removed).Because the block moves the lines below it by nine, the
docker/entrypoint.sh:A-Bcitations intests/ownership/driver.pyanddocs/design/BARS-BIN-INPUT.mdare re-pinned to the same lines of code (found by the text of the first and last line, not by adding nine blindly). Thedriver.pyruntime-command citation was already one line stale on main (it named 287-300; therun_json.pycall is 288-301 since the report-descriptor line was added at 263) and now names the right lines, so the five-linedriver.pychange is not a pure shift.Checks
git diff --summaryagainst the base is empty (no mode change); no code line of the entrypoint changes.🤖 Generated with Claude Code