Workspace plugin should read location from its config section (workspace.location) #176

Closed
opened 2026-02-28 05:43:24 +00:00 by Zeus · 0 comments
Collaborator

Problem

The workspace plugin reads its path from config.get("workspace") as a top-level string, which conflicts with the plugin config pattern where each plugin gets a config section as a dict.

Expected

The workspace location should be configurable under the workspace plugin's config section:

workspace:
  location: /home/alpha/workspace

The plugin should read:

workspace_config = config.get("workspace", {})
if isinstance(workspace_config, dict):
    config_workspace = workspace_config.get("location")
else:
    config_workspace = workspace_config  # backward compat for string

Current Code

workspace/plugin.py configure():

config_workspace = config.get("workspace")  # reads top-level key as string

This doesn't follow the plugin config convention and would break if workspace: is a dict with sub-keys.

Also Needed

  • Wizard should ask for workspace location (default: ~/.cobot/workspace)
  • Document the config option in the workspace plugin README
  • #174 — exec tool should use workspace path as cwd (so the agent naturally operates in its workspace)
## Problem The workspace plugin reads its path from `config.get("workspace")` as a top-level string, which conflicts with the plugin config pattern where each plugin gets a config section as a dict. ## Expected The workspace location should be configurable under the workspace plugin's config section: ```yaml workspace: location: /home/alpha/workspace ``` The plugin should read: ```python workspace_config = config.get("workspace", {}) if isinstance(workspace_config, dict): config_workspace = workspace_config.get("location") else: config_workspace = workspace_config # backward compat for string ``` ## Current Code `workspace/plugin.py` configure(): ```python config_workspace = config.get("workspace") # reads top-level key as string ``` This doesn't follow the plugin config convention and would break if `workspace:` is a dict with sub-keys. ## Also Needed - Wizard should ask for workspace location (default: `~/.cobot/workspace`) - Document the config option in the workspace plugin README ## Related - #174 — exec tool should use workspace path as cwd (so the agent naturally operates in its workspace)
k9ert closed this issue 2026-02-28 05:52:32 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ultanio/cobot#176
No description provided.