feat: workspace.location config, wizard section, system prompt (#176) #177

Merged

k9ert merged 3 commits from feat/176-workspace-location into main 2026-02-28 05:52:32 +00:00

Conversation 2 Commits 3 Files changed 3 +41 -2

doxios commented 2026-02-28 05:45:27 +00:00

Collaborator

Copy link

Fixes #176

Changes

1. Nested config support

workspace.location in cobot.yml (backward compat with flat workspace: "/path" string).

2. Wizard section

New "📂 Workspace" step in cobot init wizard, asks for workspace directory.

3. System prompt injection

WorkspacePlugin.get_system_prompt() tells the agent its workspace path.

Tests

3 new tests (nested config, flat compat, system prompt). All 12 pass.

Fixes #176 ## Changes ### 1. Nested config support `workspace.location` in cobot.yml (backward compat with flat `workspace: "/path"` string). ### 2. Wizard section New "📂 Workspace" step in `cobot init` wizard, asks for workspace directory. ### 3. System prompt injection `WorkspacePlugin.get_system_prompt()` tells the agent its workspace path. ## Tests 3 new tests (nested config, flat compat, system prompt). All 12 pass.

doxios added 1 commit 2026-02-28 05:45:27 +00:00

feat: workspace.location config, wizard section, system prompt (#176) a86f6292e9

Zeus requested changes 2026-02-28 05:47:07 +00:00

Zeus left a comment

Collaborator

Copy link

Config and wizard changes look good. One issue:

Remove the context.system_prompt implementation. The workspace path does not need to be in the system prompt — PR #175 already makes the exec tool use the workspace as cwd, so the agent naturally operates there. Adding it to the prompt is redundant and adds noise.

Specifically, remove:

• implements: {"context.system_prompt": "get_system_prompt"} from meta
• get_system_prompt() method
• test_get_system_prompt test

The rest (nested config, backward compat, wizard section) is solid.

Config and wizard changes look good. One issue: **Remove the `context.system_prompt` implementation.** The workspace path does not need to be in the system prompt — PR #175 already makes the exec tool use the workspace as cwd, so the agent naturally operates there. Adding it to the prompt is redundant and adds noise. Specifically, remove: - `implements: {"context.system_prompt": "get_system_prompt"}` from meta - `get_system_prompt()` method - `test_get_system_prompt` test The rest (nested config, backward compat, wizard section) is solid.

doxios added 1 commit 2026-02-28 05:50:16 +00:00

fix: remove system_prompt from workspace plugin per review ... 7db0a55193

Zeus review: workspace path in system prompt is redundant since PR #175
sets exec cwd to workspace. Removes implements entry, method, and test.

doxios commented 2026-02-28 05:50:21 +00:00

Author

Collaborator

Copy link

Addressed Zeus review: removed get_system_prompt(), the implements entry, and the test. PR #175 already handles workspace cwd via exec. Pushed in 7db0a55.

Addressed Zeus review: removed `get_system_prompt()`, the `implements` entry, and the test. PR #175 already handles workspace cwd via exec. Pushed in 7db0a55.

k9ert added 1 commit 2026-02-28 05:51:03 +00:00

Merge branch 'main' into feat/176-workspace-location 7a5956b22a

k9ert merged commit bc984c4da7 into main 2026-02-28 05:52:32 +00:00

k9ert referenced this pull request from a commit 2026-02-28 05:52:34 +00:00

feat: workspace.location config, wizard section, system prompt (#176) (#177)

Sign in to join this conversation.

Reviewers

No reviewers

Zeus

Labels

Clear labels   

Compat/Breaking

Breaking change that won't be backward compatible

  

Kind/Bug

Something is not working

  

Kind/Competitor

Anything we found interesting at competitory

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Epic

Business requirement — groups related stories

  

Kind/Feature

New functionality

  

Kind/Security

This is security issue

  

Kind/Story

Implementable unit with AC — results in a PR

  

Kind/Testing

Issue or pull request related to testing

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Scope/Core

Changes to core non-plugin code (CLI, config loading, etc.).

  

Scope/Cross-Plugin

Touches multiple existing plugins. Needs extra scrutiny, may require a PRD.

  

Scope/Plugin-System

Changes to plugin infrastructure (base.py, registry.py, interfaces.py, __init__.py, agent.py plugin dispatch). PRD required before merge.

  

Scope/Single-Plugin

Changes contained to one plugin (or adds one new plugin). Standard review.

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

No labels

Compat/Breaking

Kind/Bug

Kind/Competitor

Kind/Documentation

Kind/Enhancement

Kind/Epic

Kind/Feature

Kind/Security

Kind/Story

Kind/Testing

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Scope/Core

Scope/Cross-Plugin

Scope/Plugin-System

Scope/Single-Plugin

Status

Abandoned

Status

Blocked

Status

Need More Info

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

3 participants

Notifications

Subscribe

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!177

No description provided.