Skip to content

Add automated CI test for Claude code#44

Closed
nhorton wants to merge 8 commits intomainfrom
claude/add-ci-automated-test-bSYyi
Closed

Add automated CI test for Claude code#44
nhorton wants to merge 8 commits intomainfrom
claude/add-ci-automated-test-bSYyi

Conversation

@nhorton
Copy link
Contributor

@nhorton nhorton commented Jan 15, 2026

No description provided.

nhorton and others added 8 commits January 14, 2026 17:30
Replace Markdown with JSON array - CLA Assistant action
expects JSON format for storing signatures.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
CLA Assistant expects { "signedContributors": [] } format,
not a plain array.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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>
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.)
- 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
@nhorton nhorton added this pull request to the merge queue Jan 15, 2026
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 nhorton removed this pull request from the merge queue due to a manual request Jan 15, 2026
@nhorton
Copy link
Contributor Author

nhorton commented Jan 15, 2026

better one merged

@nhorton nhorton closed this Jan 15, 2026
@nhorton nhorton deleted the claude/add-ci-automated-test-bSYyi branch January 15, 2026 23:59
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