-
Notifications
You must be signed in to change notification settings - Fork 102
ci(orchestrator): pytest workflow scoped to orchestrator/** - #1135
Draft
najmuzzaman-mohammad wants to merge 1 commit into
Draft
ci(orchestrator): pytest workflow scoped to orchestrator/** #1135najmuzzaman-mohammad wants to merge 1 commit into
najmuzzaman-mohammad wants to merge 1 commit into
Conversation
A standalone GitHub Actions workflow that runs the orchestrator's pytest suite, path-filtered to orchestrator/** (+ the workflow file) so Go/web/docs PRs pay nothing. Fires on every PR regardless of base branch (stacked PRs included), matching ci.yml's rationale; permissions are read-only. Installs the package's base + dev deps in a venv (PEP 668-safe) and deliberately OMITS the `claude` extra: claude-agent-sdk is absent in CI, so the SDK-absent degrade tests run here (they skip locally once the SDK is installed for live runs). Verified by reproducing the CI env (fresh venv, no claude extra): 96 passed, 1 skipped. actions/checkout is SHA-pinned with a ratchet comment to match the repo's supply-chain policy; no third-party actions and no untrusted input in run steps. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
Important
Review skipped
Draft detected.
Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 88b957ec-f6b2-4838-9448-b24c5da8af94
You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.
Use the checkbox below for a quick retry:
- 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
deepagents/16-orchestrator-ci
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CI scoped to
orchestrator/**(stacked on #1134)A standalone GitHub Actions workflow running the orchestrator's pytest suite, path-filtered to
orchestrator/**(+ the workflow file) so Go/web/docs PRs pay nothing. Fires on every PR regardless of base branch (stacked PRs included), matchingci.yml's rationale;permissions: contents: read.claudeextra on purpose —claude-agent-sdkis absent in CI, so the SDK-absent degrade tests run here (they skip locally once the SDK is installed for live runs).actions/checkoutis SHA-pinned with a ratchet comment; no third-party actions; no untrusted input inrun:steps.Verified
Reproduced the CI environment locally (fresh venv, no
claudeextra): 96 passed, 1 skipped (the SDK-present complement skips; the 3 SDK-absent degrade tests run and pass).🤖 Generated with Claude Code