Skip to content

refactor: align create-expert e2e tests with shared utilities#561

Merged
FL4TLiN3 merged 2 commits intomainfrom
refactor/create-expert-e2e-tests
Feb 17, 2026
Merged

refactor: align create-expert e2e tests with shared utilities#561
FL4TLiN3 merged 2 commits intomainfrom
refactor/create-expert-e2e-tests

Conversation

@FL4TLiN3
Copy link
Contributor

Summary

  • Use withEventParsing from runner.ts instead of raw parseEvents for event parsing
  • Add delegation verification (stopRunByDelegate events) to confirm the createExpert + addDelegate + delegate workflow
  • Add JSDoc header comment and increase timeout to 180s, matching other e2e test conventions
  • Add create-expert/ section to e2e/README.md

Test plan

  • pnpm typecheck passes
  • pnpm biome check --write passes
  • pnpm test:e2e -- create-expert.test.ts (requires API keys)

🤖 Generated with Claude Code

- Use withEventParsing from runner.ts instead of raw parseEvents
- Add delegation verification (stopRunByDelegate events)
- Add JSDoc header comment matching other e2e test conventions
- Increase timeout to 180s for delegation round-trips
- Move createTempDir to module scope
- Add create-expert section to e2e/README.md

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@FL4TLiN3 FL4TLiN3 force-pushed the refactor/create-expert-e2e-tests branch 3 times, most recently from 5a14e7e to 9ee054e Compare February 17, 2026 09:27
…ation

- Add createExpert callback and MCP tool to skill-management for
  dynamically creating expert definitions in memory
- Implement createExpert in SkillManager.fromExpert to parse and store
  experts in the experts map
- Update runtime to pass dynamically created experts to delegation executor
- Update create-expert perstack.toml to use createExpert + addDelegate
  workflow instead of runExpert subprocess
- Remove create-expert-skill dependency from create-expert config
- Add unit tests for createExpert in skill-management and skill-manager

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@FL4TLiN3 FL4TLiN3 force-pushed the refactor/create-expert-e2e-tests branch from 9ee054e to 8a9a0b4 Compare February 17, 2026 09:29
@FL4TLiN3 FL4TLiN3 merged commit bcb7893 into main Feb 17, 2026
11 checks passed
@FL4TLiN3 FL4TLiN3 deleted the refactor/create-expert-e2e-tests branch February 17, 2026 09:31
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.

1 participant