added ci

[timbrinded]

No description provided.

[Copilot]

Pull request overview

This pull request adds comprehensive CI/CD infrastructure and test coverage to the simple-ralph project. The PR introduces GitHub Actions workflows for automated testing, linting, and building, along with extensive unit and integration tests across all major modules.

Changes:

• Added GitHub Actions CI workflow with jobs for checking, testing, linting, formatting, and multi-platform builds
• Added dev dependencies for testing: tempfile, assert_cmd, and predicates
• Implemented comprehensive test coverage for core functionality including session management, PRD handling, protocol serialization, CLI commands, and app state management

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
.github/workflows/ci.yml	Defines CI pipeline with check, test, lint, format, and build jobs using nightly Rust toolchain
Cargo.toml	Adds test dependencies (tempfile, assert_cmd, predicates) to dev-dependencies
Cargo.lock	Updates lock file with new test dependencies and their transitive dependencies
tests/session_workflow.rs	Integration tests for session lifecycle, JSON structure, persistence, context merging, and cleanup
tests/prd_workflow.rs	Integration tests for PRD file creation, loading, validation, and completed tasks tracking
tests/cli_integration.rs	CLI integration tests verifying help output, flags, error handling, and subcommands
src/prompt.rs	Unit tests for prompt generation functions and constant validation
src/prd.rs	Unit tests for PRD and completed task loading with error cases
src/plan/session.rs	Unit tests for session creation, persistence, context merging, and lifecycle management
src/plan/protocol.rs	Unit tests for JSON serialization/deserialization of protocol structures
src/plan/prompts.rs	Unit tests for prompt building functions with various inputs
src/plan/phases.rs	Unit tests for phase enum display and serialization
src/plan/app.rs	Comprehensive unit tests for TUI app state management, navigation, and user input
src/app.rs	Unit tests for build app state, log management, and progress tracking

Comments suppressed due to low confidence (1)

.github/workflows/ci.yml:95

• The build job uploads artifacts but only specifies target/release/ralph as the path. This won't work correctly for Windows builds if you plan to add them later, as Windows executables have a .exe extension. Consider using a conditional path or glob pattern like target/release/ralph* to handle cross-platform builds.

        with:
          name: ralph-${{ matrix.os }}
          path: target/release/ralph

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The project specifies edition = "2024" in Cargo.toml which requires the nightly Rust toolchain. As of January 2025, edition 2024 was not yet stable. If edition 2024 has since stabilized (we're now in January 2026), consider switching to the stable toolchain for broader compatibility and more reliable CI builds. If nightly features are actually needed, this is fine, but if they're not, using stable would be preferable.

[Copilot]

The cache configuration is missing a restore-keys fallback. If Cargo.lock changes, the cache won't be reused at all. Consider adding a restore-keys pattern like ${{ runner.os }}-cargo- to allow partial cache hits when dependencies change slightly. This can significantly speed up CI when dependencies are updated.