Thanks to visit codestin.com
Credit goes to github.com

Skip to content

docs: correct stale lookahead_on hard-reject claim in CLAUDE.md#48

Merged
luisleo526 merged 1 commit into
mainfrom
docs/lookahead-on-claude-md-correction
Jul 7, 2026
Merged

docs: correct stale lookahead_on hard-reject claim in CLAUDE.md#48
luisleo526 merged 1 commit into
mainfrom
docs/lookahead-on-claude-md-correction

Conversation

@luisleo526

Copy link
Copy Markdown
Contributor

CLAUDE.md invariant 6 still said barmerge.lookahead_on is hard-rejected. Verified against the code on 2026-07-07: _check_request_security accepts it with a repaint warning, the engine implements the semantics (first-intrabar publication; script-tf publish gating for finer-than-chart targets), and test_request_security_lookahead_on_kwarg_warns + test_request_security_lookahead_on_now_supported pin the behavior. Also documents ignore_invalid_symbol as accepted-but-inert and the chart-symbol alias forms the checker actually admits.

Docs-only; no code changes. This correction was twice overwritten by concurrent CLAUDE.md rewrites from stale bases — the new wording flags the staleness explicitly.

🤖 Generated with Claude Code

barmerge.lookahead_on is accepted with a repaint warning and is
engine-supported (first-intrabar publication; script-tf publish gating
for finer-than-chart targets) — verified against _check_request_security
and test_request_security_lookahead_on_kwarg_warns on 2026-07-07. Also
documents ignore_invalid_symbol as accepted-but-inert and the
chart-symbol alias forms. This correction was twice lost to concurrent
CLAUDE.md rewrites authored from stale bases; the wording now flags the
staleness explicitly so it is not restored a third time.

Co-Authored-By: Claude Fable 5 <[email protected]>
@luisleo526 luisleo526 merged commit a5c94a7 into main Jul 7, 2026
4 checks passed
@luisleo526 luisleo526 deleted the docs/lookahead-on-claude-md-correction branch July 7, 2026 14:35
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.

1 participant