Skip to content

chore: retirer la sonde live des stops T212 - #102

Merged
laurentvv merged 1 commit into
mainfrom
chore/remove-t212-stops-probe
Sep 29, 2026
Merged

laurentvv merged 1 commit into
mainfrom
chore/remove-t212-stops-probe

Conversation

@laurentvv

Copy link
Copy Markdown
Owner

Retire la sonde live tests/check_t212_stops.py (jamais exécutée, consentement en attente) et ses références dans AGENTS.md et TRADING212_API_GUIDE.md. Décision de l'utilisateur : le cœur moyen/long terme n'aura pas de stop broker. Les mentions historiques (CHANGELOG, memory-bank/log.md, plans) restent, ce sont des archives.

🤖 Generated with Claude Code

Décision de l'utilisateur (2026-09-29) : le cœur moyen/long terme n'aura plus de stop broker,
la sonde n'a plus d'objet. Références retirées d'AGENTS.md et du guide API.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@kilo-code-bot

kilo-code-bot Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The PR is deletion-only (1 file removed, 2 doc references removed). No added or context lines exist in the diff, so no inline findings could be anchored.

Observations (non-blocking, outside the diff scope):

  • docs/PLAN_RUN_DEMO_30J.md:30, memory-bank/contract.md:63 and memory-bank/progress.md:29 still reference tests/check_t212_stops.py, which no longer exists. These files are unchanged by this PR; the PR description states plans/memory-bank are kept as archives, so this is intentional, but the dangling command would fail if ever copy-pasted.
  • No code imports or references the deleted probe, so no runtime breakage.
Files Reviewed (3 files)
  • AGENTS.md - 0 issues
  • TRADING212_API_GUIDE.md - 0 issues
  • tests/check_t212_stops.py (deleted) - 0 issues

Reviewed by free · Input: 0 · Output: 0 · Cached: 0

@laurentvv
laurentvv merged commit e03d58c into main Sep 29, 2026
2 checks passed
@laurentvv
laurentvv deleted the chore/remove-t212-stops-probe branch September 29, 2026 21:47
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