chore: repo cleanup — remove runtime files, fix docs, simplify TODO#111
Merged
Conversation
- .gitignore: add patterns for runtime TXT files (memories, signals, DCA, neural weights, trainer timestamps) and coin subdirectories (BTC/ETH/XRP/DOGE/BNB) - Remove 38 runtime-generated TXT/DB files from git tracking (git rm --cached) - Consolidate DB files: move market_data.db and portfolio_optimization.db into app/ - Remove stale root-level order_management.db-shm / .db-wal WAL artefacts - README.md, QUICK_START.md, DEVELOPMENT_WORKFLOW.md: fix repo names, stale status references, phantom file paths, Windows command syntax - TODO.md: replace 281-line inline roadmap with a concise pointer to the GitHub project board (https://github.com/users/sjackson0109/projects/11)
There was a problem hiding this comment.
Pull request overview
Housekeeping PR to make the repo cleaner and less noisy by ignoring/removing runtime-generated artifacts, and updating documentation/TODOs to reflect current project tracking on GitHub.
Changes:
- Expanded
.gitignoreto exclude additional runtime-generated artifacts (signals/memories/training time, sqlite WAL/SHM, etc.). - Updated docs (
README.md,QUICK_START.md,.github/DEVELOPMENT_WORKFLOW.md) to reflect the current repo name/paths and updated setup commands. - Replaced the long inline
TODO.mdroadmap with links to the GitHub Project board.
Reviewed changes
Copilot reviewed 42 out of 47 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| TODO.md | Replaced long backlog with links to GitHub Project views |
| README.md | Updated clone/path instructions, launch guidance, and requirements |
| QUICK_START.md | Updated status summary and Windows-friendly commands |
| .gitignore | Added ignores for additional runtime artifacts and local-only folders |
| .github/DEVELOPMENT_WORKFLOW.md | Updated repo references and clone instructions |
| app/order_management.db-wal | Removed tracked SQLite WAL artifact |
| app/signals_dca_single.txt | Removed tracked runtime-generated signal file |
| app/signals_dca_spread.txt | Removed tracked runtime-generated signal file |
| app/trainer_last_training_time.txt | Removed tracked runtime-generated training timestamp |
| app/memories_1hour.txt | Removed tracked runtime-generated memory vector |
| app/memories_2hour.txt | Removed tracked runtime-generated memory vector |
| app/memories_4hour.txt | Removed tracked runtime-generated memory vector |
| app/memories_8hour.txt | Removed tracked runtime-generated memory vector |
| app/memories_12hour.txt | Removed tracked runtime-generated memory vector |
| app/memories_1day.txt | Removed tracked runtime-generated memory vector |
| app/memories_1week.txt | Removed tracked runtime-generated memory vector |
| app/BNB/alerts_version.txt | Removed tracked runtime-generated coin artifact |
| app/BNB/futures_long_onoff.txt | Removed tracked runtime-generated coin config artifact |
| app/BNB/futures_long_profit_margin.txt | Removed tracked runtime-generated coin config artifact |
| app/BNB/futures_short_onoff.txt | Removed tracked runtime-generated coin config artifact |
| app/BNB/futures_short_profit_margin.txt | Removed tracked runtime-generated coin config artifact |
| app/BNB/long_dca_signal.txt | Removed tracked runtime-generated coin signal artifact |
| app/BNB/short_dca_signal.txt | Removed tracked runtime-generated coin signal artifact |
| app/DOGE/alerts_version.txt | Removed tracked runtime-generated coin artifact |
| app/DOGE/futures_long_onoff.txt | Removed tracked runtime-generated coin config artifact |
| app/DOGE/futures_long_profit_margin.txt | Removed tracked runtime-generated coin config artifact |
| app/DOGE/futures_short_onoff.txt | Removed tracked runtime-generated coin config artifact |
| app/DOGE/futures_short_profit_margin.txt | Removed tracked runtime-generated coin config artifact |
| app/DOGE/long_dca_signal.txt | Removed tracked runtime-generated coin signal artifact |
| app/DOGE/short_dca_signal.txt | Removed tracked runtime-generated coin signal artifact |
| app/ETH/alerts_version.txt | Removed tracked runtime-generated coin artifact |
| app/ETH/futures_long_onoff.txt | Removed tracked runtime-generated coin config artifact |
| app/ETH/futures_long_profit_margin.txt | Removed tracked runtime-generated coin config artifact |
| app/ETH/futures_short_onoff.txt | Removed tracked runtime-generated coin config artifact |
| app/ETH/futures_short_profit_margin.txt | Removed tracked runtime-generated coin config artifact |
| app/ETH/long_dca_signal.txt | Removed tracked runtime-generated coin signal artifact |
| app/ETH/short_dca_signal.txt | Removed tracked runtime-generated coin signal artifact |
| app/XRP/alerts_version.txt | Removed tracked runtime-generated coin artifact |
| app/XRP/futures_long_onoff.txt | Removed tracked runtime-generated coin config artifact |
| app/XRP/futures_long_profit_margin.txt | Removed tracked runtime-generated coin config artifact |
| app/XRP/futures_short_onoff.txt | Removed tracked runtime-generated coin config artifact |
| app/XRP/futures_short_profit_margin.txt | Removed tracked runtime-generated coin config artifact |
| app/XRP/long_dca_signal.txt | Removed tracked runtime-generated coin signal artifact |
| app/XRP/short_dca_signal.txt | Removed tracked runtime-generated coin signal artifact |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- README.md: replace phantom pt_server.py with actual pt_api_server.py - QUICK_START.md: remove contradictory 'Phase 1 not complete' safety warning (Phase 1 is substantially complete per status summary) - QUICK_START.md: replace stale TODO.md grep with link to GitHub Project board sprint view
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
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
Housekeeping pass to bring the repository to a clean, production-ready state.
Changes
.gitignore
Runtime files removed from tracking (git rm --cached)
Documentation fixes
TODO.md
Testing
No functional code changes — documentation, .gitignore, and git-tracked file list only.