Conversation
Verifies the full Symphony lifecycle: - Webhook signature verification (accept valid, reject invalid) - Issue transition to In Progress triggers workspace + agent spawn - Duplicate issue webhooks are deduplicated - Issue leaving In Progress removes active workspace - Non-issue webhook events are skipped - Prompt template variable rendering Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 852005c4eb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| // WORKFLOW.md already exists in the repo root — orchestrator reads it from cwd | ||
| await orchestrator.start() |
There was a problem hiding this comment.
Because orchestrator.start() happens inside the first test instead of beforeAll, every later endpoint/webhook test only passes if Bun executes that setup test first. Bun explicitly supports focused runs via bun test --test-name-pattern, and bun test tests/todo-lifecycle.e2e.test.ts -t 'status endpoint shows orchestrator state' already fails with ConnectionRefused because the server was never started. Please move the server bootstrap into shared setup so each test can run independently.
Useful? React with 👍 / 👎.
1 participant
Summary
FakeAgentSessionto mock agent execution without real subprocess or Linear API callsTest plan
bun test tests/todo-lifecycle.e2e.test.ts(32 assertions)🤖 Generated with Claude Code