Skip to content

Add Workspace Management Commands#5

Merged
fredvisser merged 3 commits intomainfrom
users/fvisser/add-workspace-command
Jul 25, 2025
Merged

Add Workspace Management Commands#5
fredvisser merged 3 commits intomainfrom
users/fvisser/add-workspace-command

Conversation

@fredvisser
Copy link
Copy Markdown
Collaborator

Adds comprehensive workspace management capabilities with read-only operations and administrative functions.

Key Changes:

  • New workspace commands: list, info, and disable for workspace administration
  • Consistent table formatting: Updated all list commands (templates, workflows, notebooks, workspaces) to use unified box-drawing character styling
  • Command naming: Renamed all commands from plural to singular forms (templatestemplate, workflowsworkflow, etc.)
  • Enhanced utilities: Added workspace mapping functions and improved error handling
  • Documentation: Updated README with workspace management section and CLI best practices
  • Test coverage: Added comprehensive test suite for workspace operations (9 tests)

Workspace Features:

  • List workspaces with filtering (by name, include disabled)
  • Get detailed workspace info (contents: templates, workflows, notebooks)
  • Disable workspaces with confirmation prompts
  • JSON and table output formats for all commands

@fredvisser fredvisser requested a review from Copilot July 25, 2025 15:30
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds comprehensive workspace management functionality to the SystemLink CLI with read-only operations and administrative capabilities, while modernizing command naming and visual formatting across all CLI modules.

  • New workspace management commands: list, info, and disable for workspace administration with filtering and comprehensive content details
  • Command naming modernization: Changed all command groups from plural to singular forms (templatestemplate, workflowsworkflow)
  • Visual consistency improvements: Unified table formatting with box-drawing characters across all list commands (templates, workflows, notebooks, workspaces)

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
slcli/workspace_click.py Implements new workspace command group with list, info, and disable operations
slcli/main.py Registers workspace commands with CLI
slcli/utils.py Updates workspace mapping utilities with enhanced error handling
slcli/workflows_click.py Renames command group and updates table formatting
slcli/templates_click.py Renames command group and updates table formatting
slcli/notebook_click.py Updates table formatting for consistency
tests/unit/* Updates test references to use new singular command names
README.md Updates documentation with workspace management section and new command names

@fredvisser fredvisser merged commit 4fbc18e into main Jul 25, 2025
1 check passed
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.

2 participants