Repository navigation
feat: integrate CrewAI automation workspace - #1
Merged
Merged
Conversation
justgithubaccount
marked this pull request as ready for review
October 29, 2026 11:08
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.
Outcome
Add an Automation workspace for the existing CrewAI API in the same UI, without merging backend services.
CREWAI_API_URL/CREWAI_API_TIMEOUT_MSand fixed/api/automationproxy. Shared transport preserves Chat API behavior.Verification
npm run check: strict types, zero-warning ESLint, 21 unit/contract tests.npm run build,npm run format:check,npm audit --audit-level=high: PASS; audit reports zero vulnerabilities.PLAYWRIGHT_CHANNEL=chrome npm run test:e2e: 19 scenarios PASS with no retries.PLAYWRIGHT_CHANNEL=chrome npm run test:integration:automation: real Next proxy → explicit loopback HTTP fixture PASS; target/context, no browser cookie forwarding, origin/override guards verified.647663b3f05c1a6e913eeb3f76778705c8c9abf1..2852b17530f7eac756c3e4b6a3e35b1ef8f7837b: no Critical/Important/Minor change requests; reviewer independently ran all 21 unit tests. This is code evidence, not human approval. Final exact-head CI is linked on this PR.Review evidence limits / decisions
Handoff state
Base:
647663b3f05c1a6e913eeb3f76778705c8c9abf1.Head:
2852b17530f7eac756c3e4b6a3e35b1ef8f7837b.Branch:
feat/crewai-automation. Main is unchanged; PR is not merged.Drift:
local_repo=NONE_DETECTED,routed_artifacts=NOT_CHECKED(not used by this UI task),live_runtime=UNKNOWN(no real CrewAI connected).branch_merge_status: NOT_AUDITED.Boundaries
Verified CrewAI contract:
0835d3102555e6cef4fb700c56969ecddc6f6ddd,src/auto_k8s_pilot/api.py/flow_runner.py.No backend mutation, real flow execution, paid LLM call, cloud resource, merge or deployment. Tests use explicit fixtures, never production fallback. The current API has no persisted run history, live step updates or cancellation; the UI does not invent them. A timeout or leaving the page does not cancel upstream work. An actual CrewAI runtime must be connected separately; source-public does not mean runtime-public.
Design:
docs/superpowers/specs/2026-10-29-automation-design.md.Evidence:
docs/automation-validation.md.