feat: add workspace list display command#54
Merged
Conversation
This feature will display a list of all workspaces with: - Associated project information (name and alias) - Target directories from enkan-repl-target-directories - Additional workspace metadata 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
- Add enkan-repl-workspace-list command to display all workspaces - Show current workspace indicator and project information - Display target directories and session counts - Implement special mode with keybindings for workspace switching - Add tests for list display functionality Workspace list features: - RET: Switch to workspace at point - g: Refresh the list - q: Quit the buffer - Shows active/inactive status for each workspace 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
The error "Wrong type argument: listp" occurred because project-info could be either a cons cell (project-name . project-path) or a string. Now properly extract the directory path from the cons cell structure. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Remove unnecessary session counts and other verbose information. Display only workspace ID, project name, and directory. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
…ectories Fixed the condition check to use (stringp (cdr project-info)) instead of (consp (cdr project-info)) to properly detect (project-name . path) format. Now correctly displays both string values and path values. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
…y list Show '<no project>' when no project is assigned to workspace, and '<not found>' when project is assigned but not in target-directories. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- First try to find by alias (e.g., 'er', 'os') - If not found, try to find by project name in target-directories - Fix enkan-repl--get-project-path-from-directories to handle string values correctly - Show '<not found>' when project exists but not in target-directories 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Remove incorrect lookup logic for 'eronly'/'ofonly' which are project config names, not aliases. The current-project should contain the actual alias like 'er' or 'os'. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Stop reinventing the wheel and use the existing function that properly handles project name -> aliases -> directories lookup chain. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Tests now use enkan-repl-projects for project config names - Workspace :current-project contains project config name (not alias) - Use enkan-repl--get-project-paths-for-current to resolve directories - Remove unused project-aliases variable All tests pass without warnings. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
When a project has multiple aliases (e.g., web-dev with er, pt, cc), display all target directories separated by commas. Added test case for multiple targets to ensure all paths are shown. All 6 tests pass. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Use fixed-width format with consistent spacing: - Active workspace: ' ▶ 01 - ...' - Inactive workspace: ' 02 - ...' This ensures all workspace IDs align properly regardless of selection status. Added tests to verify proper alignment. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Update test expectations to match the user's preferred format: - Active workspace: '▶ 01 - ...' - Inactive workspace: ' 02 - ...' All 6 tests pass. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
github-actions Bot
pushed a commit
that referenced
this pull request
Sep 8, 2025
# [0.16.0](v0.15.10...v0.16.0) (2025-09-08) ### Features * add workspace list display command ([#54](#54)) ([7fc628e](7fc628e))
|
🎉 This PR is included in version 0.16.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implementation Plan
Display Information
Required:
enkan-repl-target-directoriesAdditional suggestions:
Command Features
Test plan
🤖 Generated with Claude Code