- Implemented `install_pyghidra` function to set up PyGhidra in a Python virtual environment for Ghidra installations.
- Enhanced `install_ghidra` to call `install_pyghidra` after Ghidra installation.
- Updated daemon command handling to include program name in start, restart, and stop commands.
- Refactored project path resolution to streamline project management.
- Improved socket path handling to respect `GHIDRA_CLI_SOCKET` environment variable for testing.
- Modified tests to remove ignore flags, allowing for automated testing without Ghidra installation.
- Added analysis step in test project setup to ensure comments and other features work correctly.
HeadlessExecutor spawned a new Ghidra process per command, which was
slow. The daemon maintains a persistent connection and is now the only
way to execute queries.
Changes:
- Delete src/ghidra/headless.rs entirely
- Remove dead handler functions that used HeadlessExecutor:
- handle_query, handle_function_command, handle_decompile,
- handle_decompile_impl, handle_strings_command,
- handle_memory_command, handle_dump_command, handle_summary
- Update handle_quick to inform user about daemon requirement
- Query::execute replaced with Query::process_results for
post-processing of daemon results
All query commands (function list, decompile, strings, memory, etc.)
now require the daemon to be running. The CLI prints clear instructions
when daemon is not available.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add E2E test infrastructure:
- DaemonTestHarness for managing daemon lifecycle in tests
- Test fixtures and helpers in tests/common/
- Sample binary fixture for integration tests
Add test coverage:
- command_tests.rs: version, doctor, config commands
- project_tests.rs: project create/list/info/delete, import, analyze
- daemon_tests.rs: daemon start/status/ping/stop/clear-cache
- query_tests.rs: function list, strings, memory, decompile, xref
- unimplemented_tests.rs: 39 tests for graceful error messages
Fix CLI bugs:
- DisasmArgs: rename count to num_instructions (--instructions/-n)
to avoid conflict with QueryOptions.count
- GraphExportArgs: add unique arg id for format positional to avoid
conflict with QueryOptions.format
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit implements the daemon-only architecture where all query
operations (functions, strings, decompile, memory, summary, xrefs)
must go through the persistent daemon instead of spawning new Ghidra
processes per command.
Key changes:
- Wire IPC client in main.rs to route queries through daemon
- Add requires_daemon() to determine which commands need daemon
- Add execute_via_daemon() to translate CLI commands to IPC calls
- Deprecate HeadlessExecutor with migration notice
- Fix filter.pest hex number parsing order (hex before number)
- Add #[allow(dead_code)] to infrastructure modules for future use
- Mark E2E tests requiring daemon as #[ignore]
Architecture benefits:
- Faster queries: Ghidra stays loaded, no 5-30s startup per command
- Simpler code: One execution path instead of two
- Better UX: Clear daemon requirement with helpful error messages
When daemon is not running, users see:
Error: This command requires the daemon to be running.
Start the daemon with: ghidra daemon start --project <name>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Key changes:
- Remove HeadlessExecutor as execution path
- All query operations require daemon
- Wire IPC layer (already implemented) as primary client
- 9 milestones for complete implementation
Rationale: Binary analysis is slow enough that persistent
daemon is always preferable to per-command process spawning.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add docs/plan-prod.md with comprehensive release plan
- Include sample_binary test fixture
- Update e2e tests
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add true background daemonization that detaches from terminal and allows the process to continue after terminal closes.
Unix implementation:
- Use daemonize crate for proper fork/detach/setsid
- Redirect stdout/stderr to log file
- Create PID file for process tracking
- Change working directory to root
Windows implementation:
- Spawn detached child process with CREATE_NO_WINDOW flag
- Pass --foreground flag to child to run in foreground internally
- Parent process exits immediately after spawn
Resolves the TODO at main.rs:180 for proper daemonization.
