chore: Phase 6 cleanup - remove deprecated files and update docs #155
Conversation
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
Phase 6 cleanup for #136: Removed installation scripts (replaced by scripts/install.sh): - local-install.sh (1400+ lines) - web_install.sh - update.sh (900+ lines) - uninstall.sh - rebuild-watcher.sh - docker-up.sh - check-updates-on-start.sh Removed obsolete directories: - gemini-client/ (Gemini CLI now installed in unified image) - macos-installer/ (no longer needed with Docker-based install) Removed outdated documentation: - UPDATE_GUIDE.md - PHASE2_COMPLETION_REPORT.md, PHASE3_PLAN.md, PHASE4_PLAN.md - REFACTORING_SUMMARY.md, TEST_FIXES.md - DEPENDENCY_UPDATE_SUMMARY.md, FINAL_SUMMARY.md Removed old compose files: - docker-compose.old Part of #136
- Complete README rewrite for unified container architecture - Updated installation instructions (single curl command) - Added migration instructions for v1.x users - Simplified update/uninstall procedures - Added architecture diagram - Updated troubleshooting section - Bumped VERSION to 2.0.0 Part of #136
Contributor
There was a problem hiding this comment.
Pull request overview
Phase 6 cleanup for v2.0.0: removes deprecated installer/update/uninstall tooling and old documentation, and updates the README/versioning to match the new unified structure.
Changes:
- Deleted legacy scripts/directories (install/update/uninstall/watchers, macOS installer, gemini-client, old compose/docs).
- Updated
README.mdwith new install/migration/update/uninstall instructions for the v2.0 unified container approach. - Bumped
VERSIONto2.0.0and added anapps/mcp-servergitlink (subproject pointer).
Reviewed changes
Copilot reviewed 33 out of 33 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| web_install.sh | Removed deprecated remote installer script. |
| update.sh | Removed deprecated update manager script. |
| uninstall.sh | Removed deprecated uninstaller script. |
| rebuild-watcher.sh | Removed deprecated rebuild watcher script. |
| macos-installer/uninstall-wrapper.sh | Removed obsolete macOS GUI uninstaller wrapper. |
| macos-installer/install-wrapper.sh | Removed obsolete macOS GUI installer wrapper. |
| macos-installer/build-uninstaller-app.sh | Removed obsolete macOS uninstaller app builder. |
| macos-installer/build-macos-app.sh | Removed obsolete macOS installer app builder. |
| macos-installer/README.md | Removed obsolete macOS installer documentation. |
| macos-installer/QUICKSTART.md | Removed obsolete macOS quickstart guide. |
| macos-installer/IMPLEMENTATION_SUMMARY.md | Removed obsolete macOS implementation summary. |
| macos-installer/FUTURE_ENHANCEMENTS.md | Removed obsolete macOS enhancements backlog doc. |
| macos-installer/FULLY_GUI_SUMMARY.md | Removed obsolete macOS fully-GUI summary doc. |
| macos-installer/ENHANCEMENTS_SUMMARY.md | Removed obsolete macOS enhancements summary doc. |
| macos-installer/BUGFIX_SUMMARY.md | Removed obsolete macOS bugfix summary doc. |
| gemini-client/settings.json.old | Removed deprecated Gemini client config artifact. |
| gemini-client/settings.json | Removed deprecated Gemini client config. |
| gemini-client/Dockerfile | Removed deprecated Gemini client image definition. |
| docker-up.sh | Removed deprecated Docker Compose wrapper script. |
| docker-compose.old | Removed obsolete compose file. |
| check-updates-on-start.sh | Removed deprecated startup update check script. |
| apps/mcp-server | Added subproject pointer for new structure. |
| VERSION | Bumped version to 2.0.0. |
| UPDATE_GUIDE.md | Removed outdated update system documentation. |
| TEST_FIXES.md | Removed outdated testing fixes writeup. |
| REFACTORING_SUMMARY.md | Removed outdated refactoring summary doc. |
| README.md | Rewritten install/migration/update/uninstall docs for v2.0 unified container. |
| PHASE4_PLAN.md | Removed outdated planning document. |
| PHASE3_PLAN.md | Removed outdated planning document. |
| PHASE2_COMPLETION_REPORT.md | Removed outdated completion report. |
| FINAL_SUMMARY.md | Removed outdated final summary doc. |
| DEPENDENCY_UPDATE_SUMMARY.md | Removed outdated dependency update summary doc. |
README.md
Outdated
Comment on lines
66
to
68
| **One-Line Install:** | ||
| ```bash | ||
| curl -fsSL https://raw.githubusercontent.com/hessius/MeticAI/main/web_install.sh | bash | ||
| curl -fsSL https://raw.githubusercontent.com/hessius/MeticAI/main/scripts/install.sh | bash |
There was a problem hiding this comment.
The README recommends curl | bash from the moving main branch without any integrity/pinning guidance. Consider updating the docs to use a versioned URL (tagged release) and/or add a verification step (e.g., checksum/signature) so users can verify what they’re executing.
Owner
Author
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
| -H "Authorization: Bearer YOUR_WATCHTOWER_TOKEN" | ||
| ``` | ||
|
|
||
| ## 🗑️ Uninstalling MeticAI |
There was a problem hiding this comment.
The uninstall steps mix docker compose down with a hard-coded docker volume rm meticai-data. To make this more robust and consistent with the later “Reset everything” section, consider documenting docker compose down -v as the primary “remove all data” option (and optionally mention docker volume ls for verifying the actual volume name) to avoid users failing to remove data due to mismatched volume naming.
Owner
Author
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
Contributor
Contributor
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: hessius <1499030+hessius@users.noreply.github.com>
…y checks Address security feedback from PR #155: - Remove curl | bash from unstable 'main' branch - Replace with git clone as primary method (safer, more transparent) - Add alternative method with commit hash pinning - Include explicit security warnings - Add file verification steps (review file, checksum) - Fix incorrect filename (docker-compose.unified.yml → docker-compose.yml) - Document best practices for downloading configs Fixes #155 (comment) Co-authored-by: hessius <1499030+hessius@users.noreply.github.com>
docs: use `docker compose down -v` in uninstall section for robustness
- Add link to GitHub commits page for finding latest commit hash - Clarify that commit hash is an example that should be updated - Provide example values for METICULOUS_IP (hostname and IP address) - Make placeholders more helpful with inline comments Co-authored-by: hessius <1499030+hessius@users.noreply.github.com>
docs: replace insecure curl | bash with pinned versions and verification
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
Phase 6 cleanup for #136 - removes deprecated files and updates documentation.
Changes
Removed Files
Installation scripts (replaced by
scripts/install.sh):local-install.sh(1400+ lines)web_install.shupdate.sh(900+ lines)uninstall.shrebuild-watcher.shdocker-up.shcheck-updates-on-start.shObsolete directories:
gemini-client/(Gemini CLI now installed in unified image)macos-installer/(no longer needed with Docker-based install)Outdated documentation:
UPDATE_GUIDE.mdPHASE2_COMPLETION_REPORT.md,PHASE3_PLAN.md,PHASE4_PLAN.mdREFACTORING_SUMMARY.md,TEST_FIXES.mdDEPENDENCY_UPDATE_SUMMARY.md,FINAL_SUMMARY.mdOld compose files:
docker-compose.oldUpdated Files
README.md - Complete rewrite:
VERSION - Bumped to 2.0.0
Deleted Lines
~9,400 lines removed! 🎉
The installation process went from 1,400+ lines (local-install.sh) to ~200 lines (scripts/install.sh).
Related Issues