Skip to content

Add OpenClaw orchestration support#173

Open
andyrewlee wants to merge 3 commits intomainfrom
openclaw
Open

Add OpenClaw orchestration support#173
andyrewlee wants to merge 3 commits intomainfrom
openclaw

Conversation

@andyrewlee
Copy link
Owner

@andyrewlee andyrewlee commented Feb 19, 2026

…est suite

Summary

Describe the change and intended behavior.

Quality Checklist

  • Ran make devcheck locally.
  • Ran make lint-strict-new locally for changed code.
  • If UI/rendering changed, ran make harness-presets.
  • If tmux/e2e changed, ran go test ./internal/tmux ./internal/e2e.

Open with Devin

…est suite

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 8 additional findings.

Open in Devin Review

andyrewlee and others added 2 commits February 19, 2026 03:09
…cleanup

- Fix command injection via unvalidated $host with is_valid_hostname()
- Add JSON validation guard on step output in turn loop
- Restrict preexisting-branch check to local refs with gitLocalBranchExists()
- Consolidate duplicate if-gf.JSON blocks and add human-mode session_exited
- Add test for stateErr != nil capture fallthrough path
- Extract handlePromptSendError helper to deduplicate Codex retry
- Shell-quote agent/workspace IDs in suggested commands
- Add upper bound (50 attempts) to watch initial capture loop
- Fix SKILL.md text: remove tautology, prefix legacy script paths
- Document needs_input heuristic limitations on broad question markers

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The workflow tests were appearing to hang (~16s each) because
openclaw-present.sh's 250-line jq transform was invoked 3 times
per workflow dual run (~5s per invocation). These tests verify
workflow logic, not presentation augmentation, so bypass the
present script via OPENCLAW_PRESENT_SCRIPT=/nonexistent.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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

Comments