Add project rule: allow all tv CLI commands without prompting - #76
Draft
keag00 wants to merge 5 commits into
Draft
Add project rule: allow all tv CLI commands without prompting#76keag00 wants to merge 5 commits into
keag00 wants to merge 5 commits into
Conversation
… expand CLAUDE.md - New Pine Script indicator (ICT Concepts + Doji Scanner) implementing swing-based trend bias, BOS, liquidity sweeps, FVGs, order blocks, equilibrium zones, and the 8 requested doji pattern variants, with a living spec in ICT_STRATEGY_SPEC.md - Fix pine_push.js/pine_pull.js picking a stale hidden Monaco editor instance instead of the live one, which caused silent "Could not inject" failures - Expand CLAUDE.md with a Development section (test commands, core/tools/cli architecture, known fragility) alongside the existing tool-usage decision tree - Update rules.json to the current 4H EMA-ribbon crypto strategy Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
for j = array.size(x) - 1 to 0 iterates ascending (not descending) when the array is empty, since -1 <= 0 — causing array.get(x, -1) on a size-0 array. Guard each of the 4 mitigation loops with array.size(x) > 0 before entering. Was silently killing all indicator output (bias table, BOS/FVG/OB/doji markers) whenever any of the four box arrays emptied out. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…repaint fix Confirmed swing/BOS/FVG detection is structurally lagged (pivots need swingLen bars of confirmation) or had a latent bug where live intrabar price could fire a one-shot line/box-creation condition more than once per bar, stacking duplicates. Added bar_index one-shot guards and a lag-free live (unconfirmed) swing high/low + live BOS layer that reacts every tick.
Non-repainting previous day/week high-low via request.security with a [1]-lagged D/W series, plus running session highs/lows that freeze and extend right once each session ends until its next occurrence resets them. All toggleable, all on by default per Keagan's "always on chart" ask.
Bash(tv *) prefix-matches every tv subcommand, so the CLI's TradingView tool calls no longer need one-off allow entries in settings.local.json. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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
1 participant
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.
Summary
.claude/settings.jsonwithpermissions.allow: ["Bash(tv *)"]so anytvCLI subcommand runs without a permission prompt for everyone on this repo.mcp__tradingview__*MCP tools connected — the MCP server code insrc/isn't registered as an active MCP server here, so "TradingView tools" in practice meanstvCLI calls via Bash. This rule covers those.Test plan
jq -eBash(tv *)prefix-matches all existing individualtv ...allow entries in.claude/settings.local.json🤖 Generated with Claude Code