Remove legacy skill definitions from .claude/skills directory #182
Conversation
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
This refactoring changes how DeepWork generates skills for Claude Code: **Before:** Each job generated a meta-skill + separate step skills - `/job_name` (meta-skill that routes to step skills) - `/job_name.step1` → `/job_name.step2` (step skills calling each other) **After:** Each job generates a single agent file with embedded skills - `/job_name` (agent with all skills embedded) - No separate step skill files Key changes: - Added agent-job.md.jinja template for generating agent files - Added `supports_agent_mode` flag to adapters (enabled for Claude) - Added `generate_agent()` method to SkillGenerator - Updated sync command to use agent mode when supported - Sync now cleans up old step skill directories when using agent mode - Updated tests to expect agent-based structure Benefits: - Simpler architecture: one file per job instead of many - Better alignment with Claude Code's Task tool for subagents - Easier to understand: all job skills visible in one file - Cleaner skills directory structure Gemini continues to use legacy mode (meta-skill + step skills) as it doesn't support the agent pattern yet. https://claude.ai/code/session_01GWfanjqoS4i54AWANk7UJ2
- Updated doc/architecture.md to describe agent-based skill generation - Added agent-job.md.jinja template to directory structure - Added supports_agent_mode to ClaudeAdapter example - Updated SkillGenerator section to describe two generation modes - Updated target project structure to show agent-mode skill layout - Updated CHANGELOG.md with agent-based architecture changes https://claude.ai/code/session_01GWfanjqoS4i54AWANk7UJ2
- Add macros for repeated patterns: output_files(), workflow_desc(), completion_msg() - Consolidate redundant workflow/step listings in Agent Overview and How to Use sections - Remove duplicate Workflow Detection and Standalone Skill Detection sections - Simplify On Completion sections using completion_msg macro - Net reduction of 116 lines across template and generated files https://claude.ai/code/session_01GWfanjqoS4i54AWANk7UJ2
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
Removes outdated skill definition files from the
.claude/skillsdirectory that were part of an earlier platform integration workflow system. These files are no longer used and have been superseded by the current job-based workflow implementation.Changes
.claude/skills/add_platform.research/SKILL.md- Research phase documentation.claude/skills/add_platform.add_capabilities/SKILL.md- Capability addition phase documentation.claude/skills/add_platform.implement/SKILL.md- Implementation phase documentation.claude/skills/add_platform.verify/SKILL.md- Verification phase documentationContext
These skill definitions were part of an earlier iteration of the platform integration workflow. The current system uses job definitions (located in
.deepwork/jobs/) instead of skill definitions. This cleanup removes technical debt and eliminates confusion between the two workflow systems.Impact
https://claude.ai/code/session_01GWfanjqoS4i54AWANk7UJ2