Skip to content

Commit b4297fc

Browse files
nhortonclaude
andauthored
fix: Remove unnecessary restart/reload instructions from sync output (#178)
The restart/reload instructions shown after `deepwork sync` are not needed as skills become available immediately without restarting the CLI session. - Remove reload_instructions class variable from all adapters - Remove "To use the new skills" output section from sync command - Remove Step 6 (Relay Reload Instructions) from implement.md - Simplify Step 7 in learn.md to just run sync without relay instructions https://claude.ai/code/session_01FD9ALAxSQ6x4QWHqXJiJJR Co-authored-by: Claude <noreply@anthropic.com>
1 parent 3cfef10 commit b4297fc

4 files changed

Lines changed: 8 additions & 39 deletions

File tree

src/deepwork/cli/sync.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,6 @@ def sync_skills(project_path: Path) -> None:
117117
# Sync each platform
118118
generator = SkillGenerator()
119119
stats = {"platforms": 0, "skills": 0, "hooks": 0}
120-
synced_adapters: list[AgentAdapter] = []
121120

122121
for platform_name in platforms:
123122
try:
@@ -184,7 +183,6 @@ def sync_skills(project_path: Path) -> None:
184183
console.print(f" [red]✗[/red] Failed to sync skill permissions: {e}")
185184

186185
stats["platforms"] += 1
187-
synced_adapters.append(adapter)
188186

189187
# Summary
190188
console.print()
@@ -202,10 +200,3 @@ def sync_skills(project_path: Path) -> None:
202200

203201
console.print(table)
204202
console.print()
205-
206-
# Show reload instructions for each synced platform
207-
if synced_adapters and stats["skills"] > 0:
208-
console.print("[bold]To use the new skills:[/bold]")
209-
for adapter in synced_adapters:
210-
console.print(f" [cyan]{adapter.display_name}:[/cyan] {adapter.reload_instructions}")
211-
console.print()

src/deepwork/core/adapters.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,6 @@ class AgentAdapter(ABC):
5858
skill_template: ClassVar[str] = "skill-job-step.md.jinja"
5959
meta_skill_template: ClassVar[str] = "skill-job-meta.md.jinja"
6060

61-
# Instructions for reloading skills after sync (shown to users)
62-
# Subclasses should override with platform-specific instructions.
63-
reload_instructions: ClassVar[str] = "Restart your AI assistant session to use the new skills."
64-
6561
# Mapping from generic SkillLifecycleHook to platform-specific event names.
6662
# Subclasses should override this to provide platform-specific mappings.
6763
hook_name_mapping: ClassVar[dict[SkillLifecycleHook, str]] = {}
@@ -296,12 +292,6 @@ class ClaudeAdapter(AgentAdapter):
296292
display_name = "Claude Code"
297293
config_dir = ".claude"
298294

299-
# Claude Code doesn't have a reload command - must restart session
300-
reload_instructions: ClassVar[str] = (
301-
"Type 'exit' to leave your current session, then run "
302-
"'claude --resume' (your history will be maintained)."
303-
)
304-
305295
# Claude Code uses PascalCase event names
306296
hook_name_mapping: ClassVar[dict[SkillLifecycleHook, str]] = {
307297
SkillLifecycleHook.AFTER_AGENT: "Stop",
@@ -576,11 +566,6 @@ class GeminiAdapter(AgentAdapter):
576566
skill_template = "skill-job-step.toml.jinja"
577567
meta_skill_template = "skill-job-meta.toml.jinja"
578568

579-
# Gemini CLI can reload with /memory refresh
580-
reload_instructions: ClassVar[str] = (
581-
"Run '/memory refresh' to reload skills, or restart your Gemini CLI session."
582-
)
583-
584569
# Gemini CLI does NOT support skill-level hooks
585570
# Hooks are global/project-level in settings.json, not per-skill
586571
hook_name_mapping: ClassVar[dict[SkillLifecycleHook, str]] = {}

src/deepwork/standard_jobs/deepwork_jobs/steps/implement.md

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -126,11 +126,7 @@ This will:
126126
- Generate skills for each step
127127
- Make the skills available in `.claude/skills/` (or appropriate platform directory)
128128

129-
### Step 6: Relay Reload Instructions
130-
131-
After running `deepwork sync`, look at the "To use the new skills" section in the output. **Relay these exact reload instructions to the user** so they know how to pick up the new skills. Don't just reference the sync output - tell them directly what they need to do (e.g., "Type 'exit' then run 'claude --resume'" for Claude Code, or "Run '/memory refresh'" for Gemini CLI).
132-
133-
### Step 7: Consider Rules for the New Job
129+
### Step 6: Consider Rules for the New Job
134130

135131
After implementing the job, consider whether there are **rules** that would help enforce quality or consistency when working with this job's domain.
136132

@@ -221,8 +217,7 @@ Before marking this step complete, ensure:
221217
- [ ] Each instruction file is complete and actionable
222218
- [ ] `deepwork sync` executed successfully
223219
- [ ] Skills generated in platform directory
224-
- [ ] User informed to follow reload instructions from `deepwork sync`
225-
- [ ] Considered whether rules would benefit this job (Step 7)
220+
- [ ] Considered whether rules would benefit this job (Step 6)
226221
- [ ] If rules suggested, offered to run `/deepwork_rules.define`
227222
228223
## Quality Criteria

src/deepwork/standard_jobs/deepwork_jobs/steps/learn.md

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -233,14 +233,12 @@ If instruction files were modified:
233233
changes: "Improved [step] instructions based on execution learnings: [brief description]"
234234
```
235235

236-
### Step 7: Sync and Relay Instructions
236+
### Step 7: Sync Skills
237237

238-
1. **Run deepwork sync** (if instructions were modified)
239-
```bash
240-
deepwork sync
241-
```
242-
243-
2. **If skills were regenerated**, look at the "To use the new skills" section in the `deepwork sync` output and **relay these exact reload instructions to the user** (e.g., "Type 'exit' then run 'claude --resume'" for Claude Code)
238+
**Run deepwork sync** (if instructions were modified)
239+
```bash
240+
deepwork sync
241+
```
244242

245243
## File Reference Patterns
246244

@@ -330,7 +328,7 @@ I found the following job executions:
330328

331329
**Summary**
332330

333-
Updated job instructions and created AGENTS.md with bespoke learnings. To get the updated skills, type 'exit' then run 'claude --resume'.
331+
Updated job instructions and created AGENTS.md with bespoke learnings.
334332
```
335333
336334
## Handling Edge Cases

0 commit comments

Comments
 (0)