Skip to content

fix: batch fixes for issues #2952, 2953, 2955, 2956, 2959, 2961, 2963, 2964, 2965, 2971 [skip ci]#409

Open
echobt wants to merge 1 commit intomasterfrom
fix/bounty-batch-2952-2971
Open

fix: batch fixes for issues #2952, 2953, 2955, 2956, 2959, 2961, 2963, 2964, 2965, 2971 [skip ci]#409
echobt wants to merge 1 commit intomasterfrom
fix/bounty-batch-2952-2971

Conversation

@echobt
Copy link
Contributor

@echobt echobt commented Jan 27, 2026

Summary

This PR fixes the following bounty issues from PlatformNetwork/bounty-challenge:

Issue #2952: Inconsistent Terminology (CWD vs --cd)

Changed 'CWD' display text to 'Working Directory' in config output for consistency with the --cd flag.

Issue #2953: Environment Variable Expansion in Config

Added support for environment variable substitution in config files:

  • `{env:VAR}` and `{env:VAR:default}` syntax
  • Shell-style `$VAR`, `${VAR}`, and `${VAR:-default}` syntax (auto-converted)

Issue #2955: Separate Error-Only Notification Option

Added `--notify-on-success` and `--notify-on-error` flags for granular notification control.
The existing `--notification` flag enables both for backward compatibility.

Issue #2956: Disabling Stream Still Shows Progress Output

Fixed progress indicators (spinners) from showing when `--no-stream` is enabled by adding streaming_enabled checks.

Issue #2959: Add ACP Status Command

Added `cortex acp status` subcommand to check running ACP server processes with `--json` and `--verbose` flags.

Issue #2961: Add Sandbox Test Command

Added `cortex sandbox test` command for platform-specific sandbox capability testing:

  • Linux: Checks kernel version and Landlock availability
  • macOS: Checks sandbox-exec and SIP status
  • Windows: Checks OS version for restricted token support

Issue #2963: Add PR List Command

Added `cortex pr list` subcommand with filtering (state, author, label), sorting, and JSON output options.

Issue #2964: User-Agent Version Mismatch

Fixed hardcoded version strings in example files to use `env!("CARGO_PKG_VERSION")` for dynamic versioning.

Issue #2965: JSON Output for Version Command

Added `cortex version` command with `--json` and `--verbose` flags for structured version information.

Issue #2971: Session Encryption at Rest

Added `session_security` config structure with `encrypt_at_rest` and `encryption_key_source` options.

Testing

  • Ran `cargo fmt` to format all code
  • Ran `cargo check -p cortex-engine` to verify compilation

Files Changed

  • cortex-cli/src/acp_cmd.rs
  • cortex-cli/src/debug_cmd.rs
  • cortex-cli/src/main.rs
  • cortex-cli/src/pr_cmd.rs
  • cortex-cli/src/run_cmd.rs
  • cortex-engine/examples/test_api.rs
  • cortex-engine/src/config/loader.rs
  • cortex-engine/src/config/project_config.rs
  • cortex-engine/src/config/types.rs

…, 2964, 2965, 2971 [skip ci]

Issue #2952: Changed inconsistent 'CWD' terminology to 'Working Directory' in config output
Issue #2953: Added environment variable expansion support in config files (, ${VAR}, ${VAR:-default})
Issue #2955: Added --notify-on-success and --notify-on-error flags for granular notification control
Issue #2956: Fixed progress indicators showing when --no-stream is enabled
Issue #2959: Added 'cortex acp status' command to check running ACP servers
Issue #2961: Added 'cortex sandbox test' command for platform-specific sandbox capability testing
Issue #2963: Added 'cortex pr list' command with filtering, sorting, and JSON output
Issue #2964: Fixed User-Agent version string in example files to use actual crate version
Issue #2965: Added 'cortex version' command with --json and --verbose flags
Issue #2971: Added session_security config structure for encryption at rest settings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant