Skip to content

Commit e3628ee

Browse files
nhortonclaude
andcommitted
fix: address review findings from first review pass
- Remove unnecessary `gh repo edit` command from new_user skill - Add new_user/ and record/ to skill directory listings in CLAUDE.md and doc/architecture.md - Fix incorrect requirement ID mapping in record DeepSchema (workflow-name-prompt was mistagged as PLUG-REQ-002.1.1) - Add PLUG-REQ-002.7 requirement for learn-after-record redirect - Update CHANGELOG.md with unreleased changes Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9840f48 commit e3628ee

5 files changed

Lines changed: 18 additions & 2 deletions

File tree

CHANGELOG.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
99

1010
### Added
1111

12+
- `/record` skill: "watch and learn" approach to creating DeepWork workflows — users do their work normally, then `/deepwork learn` turns it into a repeatable job
13+
- `/new_user` skill: guided onboarding that introduces DeepWork, offers review rule setup for code projects, and offers to record a first workflow
14+
- `/deepwork learn` now routes to the `new_job` workflow when invoked after `/deepwork:record`
15+
- Requirements specs PLUG-REQ-002 (record skill) and PLUG-REQ-003 (new user skill)
16+
- Anonymous DeepSchemas for both new skills
17+
1218
### Changed
1319

20+
- README install commands consolidated into a single `&&`-joined command ending with `/deepwork:new_user`
21+
1422
### Fixed
1523

1624
### Removed

doc/architecture.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,8 @@ deepwork/ # DeepWork tool repository
113113
│ │ │ ├── deepreviews/SKILL.md
114114
│ │ │ ├── deepwork/SKILL.md
115115
│ │ │ ├── deepschema/SKILL.md
116+
│ │ │ ├── new_user/SKILL.md
117+
│ │ │ ├── record/SKILL.md
116118
│ │ │ └── review/SKILL.md
117119
│ │ ├── hooks/ # hooks.json, post_commit_reminder.sh, post_compact.sh, startup_context.sh, deepschema_write.sh
118120
│ │ └── .mcp.json # MCP server config

plugins/claude/skills/new_user/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ If `gh` is available, use `AskUserQuestion` to say something like:
2121
If they agree, run:
2222

2323
```bash
24-
gh repo edit Unsupervisedcom/deepwork --visibility public 2>/dev/null; gh api -X PUT /user/starred/Unsupervisedcom/deepwork
24+
gh api -X PUT /user/starred/Unsupervisedcom/deepwork
2525
```
2626

2727
If `gh` is not installed, skip this entirely — do not mention it.

plugins/claude/skills/record/.deepschema.SKILL.md.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ requirements:
1010
If the user agrees to star the repo, the skill MUST run
1111
`gh api -X PUT /user/starred/Unsupervisedcom/deepwork`.
1212
13-
# PLUG-REQ-002.1.1
13+
# PLUG-REQ-002.3.1 (workflow name collection is part of the handoff flow)
1414
workflow-name-prompt: >
1515
The skill MUST ask the user for a workflow name before proceeding.
1616

specs/deepwork/cli_plugins/PLUG-REQ-002-record-skill.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,3 +44,9 @@ This skill targets new users who know *what* they want to automate but haven't y
4444
2. If `gh` is available, the skill MUST use `AskUserQuestion` to offer to star the DeepWork repo on GitHub so the user gets update notifications.
4545
3. If the user agrees, the skill MUST run `gh api -X PUT /user/starred/Unsupervisedcom/deepwork`.
4646
4. If `gh` is not installed, the skill MUST skip this step entirely without mentioning it.
47+
48+
### PLUG-REQ-002.7: Learn After Record Redirect
49+
50+
1. When `/deepwork learn` is invoked in a conversation where `/deepwork:record` was used, the `/deepwork` skill MUST start the `new_job` workflow (`job_name: "deepwork_jobs"`, `workflow_name: "new_job"`) instead of the `learn` workflow.
51+
2. The `learn` step in the `deepwork_jobs` standard job MUST check for a prior `/deepwork:record` invocation and redirect to `new_job` if found.
52+
3. The conversation history from the recording session MUST be used as the primary input for defining the new job's steps, inputs, and outputs.

0 commit comments

Comments
 (0)