Skip to content

fix(cortex-cli): display feature descriptions in features list#124

Closed
echobt wants to merge 1 commit intomasterfrom
fix/bounty-issue-1355
Closed

fix(cortex-cli): display feature descriptions in features list#124
echobt wants to merge 1 commit intomasterfrom
fix/bounty-issue-1355

Conversation

@echobt
Copy link
Contributor

@echobt echobt commented Jan 26, 2026

Summary

This PR fixes the issue where cortex features list command displayed only the feature name, stage, and enabled status without showing what each feature actually does.

Changes

  • Modified list_features() function in cortex-cli/src/main.rs to use the actual feature registry from cortex-experimental
  • Added a Description column to the features list output
  • Features are now displayed with their descriptions, making the output more informative for users

Before

Feature                        Stage        Enabled
----------------------------------------------------
unified_exec                   stable       true
web_search                     beta         false

After

Feature              Stage          Enabled  Description
--------------------------------------------------------------------------------
unified_exec         Experimental   false    Use PTY-backed command execution
web_search           Stable         false    Allow agent to search the web

Related

Fixes PlatformNetwork/bounty-challenge#1355

Fixes bounty issue #1355

The 'cortex features list' command was displaying features with only
name, stage, and enabled status without showing descriptions. This
change uses the actual feature registry from cortex-experimental to
display each feature's description, making the output more informative
for users.
echobt pushed a commit that referenced this pull request Jan 27, 2026
This commit implements the following 10 open PRs for cortex-cli:

1. PR #155 - Prompt for tab completion setup on first run
   - Added completion_setup module for first-run completion detection
   - On first interactive run, prompts user to enable tab completion
   - Automatically detects shell (bash, zsh, fish, PowerShell, elvish)
   - Creates marker file to avoid repeated prompts

2. PR #153 - Emit valid JSONL with full event data in streaming mode
   - Already implemented in previous work

3. PR #151 - Add man page generation command
   - Added clap_mangen dependency
   - Added 'man' command with optional output directory
   - Generates roff-format man pages

4. PR #137 - Use consistent provider name casing in models output
   - Already using lowercase provider names (no changes needed)

5. PR #134 - Check actual write permission for current user in debug file
   - Added is_writable_by_current_user() helper function
   - Uses actual file open test instead of permission bits

6. PR #133 - Detect actual binary location from PATH for uninstall dry-run
   - Added 'which' dependency for PATH lookup
   - Updated collect_binary_locations() to use PATH search first

7. PR #130 - Show searched paths in debug ripgrep output
   - Added searched_paths field to RipgrepDebugOutput
   - Added get_path_directories() helper function
   - Shows PATH directories when ripgrep is not found

8. PR #129 - Output valid JSON on errors when --json flag is set
   - Updated run_servers() to handle MdnsBrowser errors as JSON
   - Move discovery banner inside conditional for non-JSON mode

9. PR #126 - Add batch export for sessions
   - Added --all (-a) flag for batch export mode
   - Added --output-dir option for batch exports
   - Each session exported to separate JSON file

10. PR #124 - Display feature descriptions in features list
    - Updated list_features() to use actual feature registry
    - Added Description column to features list output

11. PR #122 - Add debug system command for system information
    - Added System subcommand to debug CLI
    - Gathers OS, hardware, environment, and Cortex info
    - Supports JSON output for scripts/automation
echobt added a commit that referenced this pull request Jan 27, 2026
This commit implements the following 10 open PRs for cortex-cli:

1. PR #155 - Prompt for tab completion setup on first run
   - Added completion_setup module for first-run completion detection
   - On first interactive run, prompts user to enable tab completion
   - Automatically detects shell (bash, zsh, fish, PowerShell, elvish)
   - Creates marker file to avoid repeated prompts

2. PR #153 - Emit valid JSONL with full event data in streaming mode
   - Already implemented in previous work

3. PR #151 - Add man page generation command
   - Added clap_mangen dependency
   - Added 'man' command with optional output directory
   - Generates roff-format man pages

4. PR #137 - Use consistent provider name casing in models output
   - Already using lowercase provider names (no changes needed)

5. PR #134 - Check actual write permission for current user in debug file
   - Added is_writable_by_current_user() helper function
   - Uses actual file open test instead of permission bits

6. PR #133 - Detect actual binary location from PATH for uninstall dry-run
   - Added 'which' dependency for PATH lookup
   - Updated collect_binary_locations() to use PATH search first

7. PR #130 - Show searched paths in debug ripgrep output
   - Added searched_paths field to RipgrepDebugOutput
   - Added get_path_directories() helper function
   - Shows PATH directories when ripgrep is not found

8. PR #129 - Output valid JSON on errors when --json flag is set
   - Updated run_servers() to handle MdnsBrowser errors as JSON
   - Move discovery banner inside conditional for non-JSON mode

9. PR #126 - Add batch export for sessions
   - Added --all (-a) flag for batch export mode
   - Added --output-dir option for batch exports
   - Each session exported to separate JSON file

10. PR #124 - Display feature descriptions in features list
    - Updated list_features() to use actual feature registry
    - Added Description column to features list output

11. PR #122 - Add debug system command for system information
    - Added System subcommand to debug CLI
    - Gathers OS, hardware, environment, and Cortex info
    - Supports JSON output for scripts/automation

Co-authored-by: Droid Agent <droid@factory.ai>
@echobt
Copy link
Contributor Author

echobt commented Jan 27, 2026

Closing: this feature has already been implemented in commit 6c8cccf (PR #182).

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.

[BUG] features list doesn't show feature descriptions

2 participants