feat: drop Upbit/Bybit and reduce to Binance-only foundation#94
Merged
fray-cloud merged 2 commits intodevfrom May 2, 2026
Merged
feat: drop Upbit/Bybit and reduce to Binance-only foundation#94fray-cloud merged 2 commits intodevfrom
fray-cloud merged 2 commits intodevfrom
Conversation
Adds a four-principle "Coding behavior" section (Think before coding, Simplicity first, Surgical changes, Goal-driven execution) above the existing Skill routing rules. Surfacing these rules in CLAUDE.md keeps implicit guidance explicit and actionable across future sessions. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…only foundation Foundation cleanup PR before introducing Binance Futures + Claude CLI driven trading. Removes everything not needed for the new direction. What's removed: - Upbit and Bybit exchange adapters (REST + WS + tests, ~1300 lines) - worker-service strategies/, backtesting/, backtests/, flows/, paper-engine.service.ts - api-server strategies/, flows/, e2e/strategies tests - web app/strategies, app/flows, app/orders, app/backtests routes + components - web hooks for strategies/flows/orders/backtests, lib/indicators - Strategy factory + indicator constants from test-utils and lib - Strategy/Flow/Backtest/StrategyLog/BacktestTrace Prisma models - StrategySignalEvent/BacktestRequested/Completed Kafka events and topics What's reshaped: - ExchangeId union narrowed to 'binance'; all DTOs, factories, REST_ADAPTERS, EXCHANGES constants updated accordingly - Activity service drops strategy_signal/strategy_order/risk_blocked types - Notification settings/event types reduced to order_filled/order_failed - Markets service/gateway drops strategy:signal and backtest:completed listeners - Prisma migrations folder cleared and a single fresh init migration is applied by `prisma migrate dev` against the empty volume What's added (infra): - Docker Compose data volumes converted to named volumes (postgres_data, redis_data, kafka_data, zookeeper_data/log) to bypass WSL2 bind-mount permission collisions - API/worker/nginx logs moved to named volumes (stdout logging in services) - Web container's anonymous /app/node_modules volume removed so pnpm symlinks resolve correctly inside the container - Postgres logging_collector and Redis --logfile flags removed (use stdout) Verified end-to-end: - pnpm build green across all 9 workspace packages - docker compose up: postgres/redis/kafka healthy, api-server /api/health 200, worker connects to Kafka and joins consumer group, web renders /markets - Prisma init migration generated and applied against fresh volume Plan: PR2 will bring Binance Futures (fapi.binance.com) adapter + futures fields on OrderRequest/Result/Order; PR3 will add Claude CLI driven LLM trade flow + per-user OAuth token storage. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This was referenced Apr 30, 2026
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
PR1 of 3 in the LLM-driven Binance Futures pivot. Cleanup only — removes everything not needed for the new direction so the next two PRs can build on a clean base.
Why
Existing platform had Upbit/Binance/Bybit + indicator strategies + flow builder + backtest engine — too dispersed for the new goal of LLM-driven Binance Futures trading. Approved plan at `~/.claude/plans/sharded-dreaming-map.md` splits the pivot into three PRs:
Test plan
Out of scope
🤖 Generated with Claude Code