refactor: Remove workspace infrastructure from core#948
Open
saraichinwag wants to merge 3 commits intoExtra-Chill:mainfrom
Open
refactor: Remove workspace infrastructure from core#948saraichinwag wants to merge 3 commits intoExtra-Chill:mainfrom
saraichinwag wants to merge 3 commits intoExtra-Chill:mainfrom
Conversation
- Remove duplicated Build system and Testing sections from AGENTS.md - Remove workspace and github CLI commands from README.md (moved to extension) - Update wp-cli.md to remove workspace and github command sections - Add notes about data-machine-code extension across all docs - Fix agent memory file commands to use correct CLI syntax - Update tools documentation to reflect extension moves The wp datamachine workspace and wp datamachine github commands, along with related tools, have been moved to the data-machine-code extension plugin. All documentation now reflects this change.
The workspace infrastructure (handlers, abilities, tools) and GitHub integration remain in core Data Machine. Only the WP-CLI commands (wp datamachine workspace, wp datamachine github) moved to the data-machine-code extension. Updated README to clarify this distinction: - Workspace feature notes CLI is in extension - GitHub issues task notes it's in extension - Handler tables clarify which are in extensions
Completely removes all workspace-related code from Data Machine core. The workspace infrastructure has been migrated to the data-machine-code extension plugin. Removed: - Workspace fetch handler (Fetch/Handlers/Workspace/) - Workspace publish handler (Publish/Handlers/Workspace/) - WorkspaceScopedTools (Workspace/Tools/) - Workspace service files (FilesRepository/Workspace*.php) - Workspace directory method from DirectoryManager - All workspace-related tests - Handler instantiations from data-machine.php 12 files changed, 2758 deletions(-)
saraichinwag
force-pushed
the
remove-workspace-core
branch
from
3d61fbb to
9770785
Compare
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
1 participant
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
Completely removes all workspace-related code from Data Machine core. The workspace infrastructure has been fully migrated to the data-machine-code extension plugin.
Changes
Removed Files (12 files, 2758 deletions)
Workspace Service Layer:
Workspace Handlers:
Workspace Tools:
Tests:
Updated Files:
Migration Path
Users needing workspace functionality should install the data-machine-code extension:
All workspace CLI commands are available via:
Related