Skip to content

docs: clarify mt5api operational scope#44

Merged
dceoy merged 2 commits into
mainfrom
docs/readme-scope-trading-operations
Jun 27, 2026
Merged

docs: clarify mt5api operational scope#44
dceoy merged 2 commits into
mainfrom
docs/readme-scope-trading-operations

Conversation

@dceoy

@dceoy dceoy commented Jun 27, 2026

Copy link
Copy Markdown
Owner

Summary

  • Clarify that mt5api exposes market data, account state, history, order validation, and narrow operational terminal endpoints over HTTP.
  • Avoid implying that mt5api exposes live order-send endpoints or strategy orchestration.
  • Align the feature list with the documented endpoint surface.

Testing

  • Not run; documentation-only change.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 456ec14c7f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
@claude

claude Bot commented Jun 27, 2026

Copy link
Copy Markdown

The scope clarification and feature list expansion are a clear improvement over the previous description.

One pre-existing gap becomes more visible with this PR: the Mermaid architecture diagram (listing health · symbols · market · account · history · calc · trading) still omits the connection router. With this PR explicitly framing the operational surface, the diagram now reads as a complete inventory when it isn't. Worth adding connection to that list in a follow-up (or here).

…rding

Address review feedback: mention POST /connection/login in the operational
surface paragraph (it reconnects the terminal to a different MT5 account,
which reviewers flagged as missing), and replace "or" with "and" for
terminal-side subscriptions and MarketWatch visibility.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@dceoy dceoy merged commit a7e94bb into main Jun 27, 2026
14 checks passed
@dceoy dceoy deleted the docs/readme-scope-trading-operations branch June 27, 2026 15:20
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