Skip to content

Comments

Claude/add learn command#27

Merged
nhorton merged 4 commits intomainfrom
claude/add-learn-command-npWIp
Jan 15, 2026
Merged

Claude/add learn command#27
nhorton merged 4 commits intomainfrom
claude/add-learn-command-npWIp

Conversation

@nhorton
Copy link
Contributor

@nhorton nhorton commented Jan 15, 2026

No description provided.

claude and others added 4 commits January 14, 2026 22:10
…rovement

The learn command reflects on conversations where DeepWork jobs were run,
identifies confusion or inefficiencies, and improves job instructions. It also
captures bespoke learnings specific to the current run into AGENTS.md files
in the appropriate working folder.

Key features:
- Analyzes conversation for DeepWork job executions
- Classifies learnings as generalizable (→ instructions) or bespoke (→ AGENTS.md)
- Uses file references in AGENTS.md to keep things in sync
- Creates learning_summary.md documenting all changes

Changes:
- Removed refine step from deepwork_jobs
- Added learn step with comprehensive instructions
- Updated job version to 0.2.0
- Regenerated all command files
…ction

Updates to the learn command:
- Simplified working folder identification (use conversation history, fall back to git diff)
- Added quality criteria for instruction conciseness (no redundancy/verbosity)
- Added Step 4b for extracting shared/lengthy content into referenced files
- Instructions can now reference shared files in steps/shared/ directory

This helps keep job instructions maintainable and DRY.
@github-actions
Copy link
Contributor

github-actions bot commented Jan 15, 2026

All contributors have signed the CLA. ✅
Posted by the DCO Assistant Lite bot.

@nhorton
Copy link
Contributor Author

nhorton commented Jan 15, 2026

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Jan 15, 2026
@nhorton nhorton merged commit e3c5beb into main Jan 15, 2026
2 of 3 checks passed
@nhorton nhorton deleted the claude/add-learn-command-npWIp branch January 15, 2026 00:58
github-merge-queue bot pushed a commit that referenced this pull request Jan 15, 2026
* Fix CLA signatures file format

Replace Markdown with JSON array - CLA Assistant action
expects JSON format for storing signatures.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Fix CLA signatures JSON structure

CLA Assistant expects { "signedContributors": [] } format,
not a plain array.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Remove manually created signatures file

Per CLA Assistant docs: "You do not need to create this file manually.
Our workflow will create the signature file if it does not already exist.
Manually creating this file will cause the workflow to fail."

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Creating file for storing CLA Signatures

* @nhorton has signed the CLA in #27

* @tylerwillis has signed the CLA in #31

* Add automated CI test for Claude Code integration

Add a deterministic 'fruits' test job and comprehensive CI tests to validate
that deepwork-generated commands work correctly with Claude Code.

Changes:
- Add fruits job fixture (identify + classify steps) for CI testing
- Add integration tests for fruits workflow (8 tests)
- Add e2e tests for Claude Code execution (3 tests, skipped without API key)
- Add GitHub Actions workflow for automated testing:
  - validate-generation: Always runs, tests command generation
  - claude-code-e2e: Runs with ANTHROPIC_API_KEY, tests actual execution

The fruits job is designed to be deterministic:
- Input: comma-separated list of items (e.g., "apple, car, banana")
- Step 1: Identify which items are fruits
- Step 2: Classify fruits by category (citrus, tropical, etc.)

* Fix CI workflow: use deepwork install with --path, add concurrency rules

- Add concurrency rules to ensure only one instance runs per PR
- Fix test to use 'deepwork install --platform claude --path test_project'
- Create .claude directory before install for platform detection
- Run commands from repo root with --path flag instead of cd'ing

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
nhorton added a commit that referenced this pull request Jan 15, 2026
* Fix CLA signatures file format

Replace Markdown with JSON array - CLA Assistant action
expects JSON format for storing signatures.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Fix CLA signatures JSON structure

CLA Assistant expects { "signedContributors": [] } format,
not a plain array.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Remove manually created signatures file

Per CLA Assistant docs: "You do not need to create this file manually.
Our workflow will create the signature file if it does not already exist.
Manually creating this file will cause the workflow to fail."

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Creating file for storing CLA Signatures

* @nhorton has signed the CLA in #27

* @tylerwillis has signed the CLA in #31

* Add automated CI test for Claude Code integration

Add a deterministic 'fruits' test job and comprehensive CI tests to validate
that deepwork-generated commands work correctly with Claude Code.

Changes:
- Add fruits job fixture (identify + classify steps) for CI testing
- Add integration tests for fruits workflow (8 tests)
- Add e2e tests for Claude Code execution (3 tests, skipped without API key)
- Add GitHub Actions workflow for automated testing:
  - validate-generation: Always runs, tests command generation
  - claude-code-e2e: Runs with ANTHROPIC_API_KEY, tests actual execution

The fruits job is designed to be deterministic:
- Input: comma-separated list of items (e.g., "apple, car, banana")
- Step 1: Identify which items are fruits
- Step 2: Classify fruits by category (citrus, tropical, etc.)

* Fix CI workflow: use deepwork install with --path, add concurrency rules

- Add concurrency rules to ensure only one instance runs per PR
- Fix test to use 'deepwork install --platform claude --path test_project'
- Create .claude directory before install for platform detection
- Run commands from repo root with --path flag instead of cd'ing

* Add full e2e test: define -> implement -> execute workflow

Update the claude-code-e2e job to test the COMPLETE DeepWork workflow:
1. /deepwork_jobs.define - Create a new job from scratch
2. /deepwork_jobs.implement - Generate step instruction files
3. /fruits.identify - Execute the generated identify command
4. /fruits.classify - Execute the generated classify command

This tests the actual user experience rather than just validating
pre-existing fixtures. The test provides deterministic instructions
for creating a 'fruits' job with identify and classify steps.

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
nhorton added a commit that referenced this pull request Jan 15, 2026
* Fix CLA signatures file format

Replace Markdown with JSON array - CLA Assistant action
expects JSON format for storing signatures.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Fix CLA signatures JSON structure

CLA Assistant expects { "signedContributors": [] } format,
not a plain array.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Remove manually created signatures file

Per CLA Assistant docs: "You do not need to create this file manually.
Our workflow will create the signature file if it does not already exist.
Manually creating this file will cause the workflow to fail."

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Creating file for storing CLA Signatures

* @nhorton has signed the CLA in #27

* @tylerwillis has signed the CLA in #31

* Fix CI workflow to run on all PRs

Remove the `branches: ["*"]` filter from the pull_request trigger.
The glob pattern `*` doesn't match `/`, so PRs targeting branches
with slashes (like feature/...) were not triggering the workflow.

By removing the filter, the workflow now runs on all PRs regardless
of target branch, while still not running on post-merge to main
(since there's no push trigger).

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
nhorton added a commit that referenced this pull request Jan 15, 2026
* Fix CLA signatures file format

Replace Markdown with JSON array - CLA Assistant action
expects JSON format for storing signatures.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Fix CLA signatures JSON structure

CLA Assistant expects { "signedContributors": [] } format,
not a plain array.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Remove manually created signatures file

Per CLA Assistant docs: "You do not need to create this file manually.
Our workflow will create the signature file if it does not already exist.
Manually creating this file will cause the workflow to fail."

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Creating file for storing CLA Signatures

* @nhorton has signed the CLA in #27

* @tylerwillis has signed the CLA in #31

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants