add CLI installation support and interactive graph visualization in UI#14
add CLI installation support and interactive graph visualization in UI#14AxelDlv00 wants to merge 31 commits intofrenzymath:mainfrom
Conversation
|
Review about setup.py:
setup.py runs sudo apt-get install -y / sudo pacman -S --noconfirm without user confirmation when dependencies are missing. Consider either printing manual install instructions (as the old setup.sh did) or prompting the user before escalating to sudo.
tmux is checked and installed in setup.py as "required for parallel agent teams," but the actual parallelism in both the old archon-loop.sh (bash background processes) and the new loop.py (ProcessPoolExecutor) does not use tmux. It can be safely removed from the setup flow. |
|
Review about init.py Review about loop.py |
After chating with leheng, we have made some new discoveries and ideas.
|
These two issues have been solved in commit #6bff60ab6daf5950c921b4cbb42e27e3ef53abef. |
Since the last commit (6bff60a) :
|
In commit 6bff60a, I also added two minor changes.
|
There's one detail I'd like to confirm: regarding the MCP reinstallation, if the user has already registered for MCP, I think we should cancel the previous registration and use the new one. This is because the expectation is that the user should be able to delete the old Archon without data loss after installing the new version. If the old MCP is retained, it might cause the program to malfunction after the user deletes the old Archon folder. |
I think a reasonable approach is for you to keep a copy of the PR that is exactly the same as your own fork, except for the download link. This way, we can use your code during testing, and once it passes review, we can directly approve the PR without needing to modify anything. |
You are correct, keeping the old registration would leave a dangling file path if the user decides to delete the old Archon folder. I've just updated the code to fix this. Now, if the script detects an existing archon-lean-lsp, it explicitly runs claude mcp remove to clear out the old path before re-registering it with the new one. Regarding my last "⚠" about "https://github.com/frenzymath/Archon/archive/refs/heads/main.tar.gz", I eventually decided to set "frenzymath" because the previous installation file installs the main branch (on which I added other functionnalities), but this PR is on a distinct branch, sorry for the confusion. If you want to test the installation, just run "python3 -m pip install ." in the root directory if you cloned my code. |
|
Remaining issues:
|
re-initialization of existing projects and the auto-launched dashboard
ChangesI modified the files to answer your catches.
|
|
The rollback instruction git checkout .archon/ in MIGRATION.md is invalid because .archon/ is in .gitignore and was never tracked by git; maybe it should be replaced with cp -r .archon .archon.bak for manual backup. |
Indeed, I modified with a copy from a backup as suggested. |
Overview
This PR introduces two major enhancements to the Archon project: a dedicated CLI installation workflow and a new graph visualization component within the user interface.
Changes
src/archonshfiles from the base repository were replaced with python scriptscurl ... | bashcommandinitloopdoctordashboardprovesetupui/directory.