Add agent tooling config and MSIX TradingView launch path - #71
Open
zli69 wants to merge 1 commit into
Open
Conversation
- .mcp.json / .codex/config.toml: register the TradingView MCP server for Claude Code and Codex so both agents pick it up per-project - AGENTS.md: Codex-facing mirror of the CLAUDE.md tool guide - .claude/settings.json: enable the frontend-design plugin - scripts/launch_tv_debug.bat: also detect the MSIX (Windows Store) TradingView install under Program Files\WindowsApps .claude/settings.local.json is intentionally left uncommitted (personal permission allowlist). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
devanshjainms
pushed a commit
to devanshjainms/tradingview-mcp
that referenced
this pull request
Jul 25, 2026
Pine graphics tools (lines/labels/boxes) and the OHLCV summary rounded every price to 2 decimals, flattening forex/crypto sub-cent levels to 0.00 and corrupting fractional futures levels (issue LewisWJackson#77 — e.g. a real level of 29941.9375 was reported as 29941.94). Replaced with a shared roundPrice() at 8 dp, which still removes float noise. Verified live: Daily Profiler levels now read back exact (29941.9375, 29890.82925). npm audit fix resolves all 7 advisories (3 high, 4 moderate: ws, qs, fast-uri, ip-address, hono chain). Lint 0 errors, tests 112/112. Fixes LewisWJackson#77. Supersedes the audit half of PRs tradesdontlie#267/LewisWJackson#71. Co-Authored-By: Claude Fable 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.
What changed
.mcp.json/.codex/config.toml— register the TradingView MCP server (node src/server.js) at the project level for Claude Code and Codex respectively, so either agent picks it up on open without manual setup.AGENTS.md— Codex-facing mirror of theCLAUDE.mdtool guide (decision tree, context-management rules, tool conventions)..claude/settings.json— enables thefrontend-designplugin.scripts/launch_tv_debug.bat— adds detection of the MSIX (Windows Store) TradingView install underProgram Files\WindowsApps.Why
Makes the repo self-configuring for agent workflows on a fresh clone, and fixes
launch_tv_debug.batfailing to find TradingView when it was installed from the Microsoft Store rather than the standalone installer.Reviewer notes
2.9.4.6649— it will stop matching when TradingView updates. The generic MSIX fallback block just below it may already cover that case; consider replacing the pinned path with a wildcard/dir-based lookup if this bites..claude/settings.local.json(personal permission allowlist) was deliberately left uncommitted..gitattributescould normalize this repo-wide if desired.🤖 Generated with Claude Code