demon, branch fix-tail-flag-logic

Refactor cat and tail flag logic for better readability

2025-06-26T15:23:49+00:00

Replace the cryptic boolean expressions with clearer if-else logic that
explicitly handles the flag combinations:

Old logic (functionally correct but hard to understand):
    let show_stdout = !args.stderr || args.stdout;
    let show_stderr = !args.stdout || args.stderr;

New logic (clear and self-documenting):
    let show_stdout = if args.stdout || args.stderr {
        args.stdout
    } else {
        true // Show both when no flags are specified
    };

This change improves code readability while maintaining identical behavior:
- No flags: show both stdout and stderr
- --stdout only: show only stdout
- --stderr only: show only stderr
- Both flags: show both stdout and stderr

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Fix critical process daemonization issue by replacing std::mem::forget with proper background thread management

2025-06-26T15:21:58+00:00

PROBLEM:
The previous implementation used std::mem::forget(child) to detach processes,
which caused several critical issues:
- Prevented Child destructor from running, leading to resource leaks
- Could result in zombie processes under certain conditions
- Violated Rust best practices for resource management
- No proper cleanup of OS handles and process resources

SOLUTION:
Replaced std::mem::forget(child) with a background thread approach:
- Spawn child process in background thread that owns the Child struct
- When thread completes, Child's Drop implementation runs automatically
- Ensures proper resource cleanup while maintaining process detachment
- Process becomes child of init (PID 1) which handles reaping
- Follows Rust idioms for resource management

VERIFICATION:
- Added test that verifies proper resource management
- All existing functionality preserved
- No breaking changes to CLI interface
- Improved system resource handling

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Add comprehensive root directory validation tests

2025-06-26T15:21:49+00:00

These tests cover edge cases including:
- Root directory specified as file instead of directory
- Non-existent root directories
- .demon directory exists as file in git root
- Permission denied scenarios
- Symlink handling (valid and broken)
- Invalid UTF-8 paths
- Deeply nested paths

Most validation is already working correctly, but these tests
document and verify the expected behavior for edge cases.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Fix all tests and add cargo test to pre-commit hook

2025-06-23T10:03:22+00:00

- Fixed all 19 failing tests by replacing --root-dir flag with DEMON_ROOT_DIR environment variable
- All 28 tests now pass consistently
- Added cargo test to pre-commit hook for comprehensive quality checks
- Pre-commit hook now runs: cargo fmt, cargo clippy, and cargo test

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Add clippy check to pre-commit hook and fix all clippy warnings

2025-06-23T09:44:45+00:00

- Enhanced pre-commit hook to run cargo clippy with -D warnings
- Fixed uninlined_format_args warnings by using variables directly in format strings
- Removed needless borrows in .args() calls
- Fixed print_literal warnings by moving literals out of format placeholders
- Replaced seek_from_current with stream_position for better semantics
- Fixed len_zero warning by using is_empty() instead of len() > 0

All code now passes both cargo fmt and cargo clippy checks

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Remove implementation plans and goal files

2025-06-23T09:37:37+00:00

Clean up project by removing internal planning documents

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Add pre-commit hook for cargo fmt and apply formatting

2025-06-23T09:36:24+00:00

- Created pre-commit hook that runs cargo fmt --check
- If formatting issues found, runs cargo fmt and asks for re-commit
- Applied cargo fmt to fix existing formatting issues

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Simplify README title

2025-06-23T09:34:39+00:00

Remove "Background Process Manager" subtitle for cleaner title

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Add GitHub Actions workflow for Rust testing

2025-06-23T09:33:21+00:00

- Runs tests on push and pull requests to main branch
- Includes formatting check, clippy linting, build, and test steps
- Uses cargo caching for improved performance

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude

Improve README introduction and add root directory information

2025-06-23T09:29:47+00:00

- Rewrite opening section to be clearer and more direct
- Better explain the tool's purpose for AI agents
- Add root directory discovery details to "How It Works" section
- Simplify content structure while maintaining essential information

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude