Skip to content

[codex] verify OpenClaw ACP detection - #38

Merged
NitayRabi merged 1 commit into
mainfrom
codex/verify-openclaw-acp-detection
May 13, 2026
Merged

[codex] verify OpenClaw ACP detection#38
NitayRabi merged 1 commit into
mainfrom
codex/verify-openclaw-acp-detection

Conversation

@NitayRabi

@NitayRabi NitayRabi commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • verify OpenClaw detection by proving acpx openclaw can initialize ACP, not just by finding an executable
  • try each openclaw PATH candidate and persist the first working PATH ordering
  • document the broken-wrapper failure mode and add regression coverage

Root Cause

OpenClaw detection accepted executable presence even when the first openclaw on PATH was a wrapper that failed before ACP initialize.

Validation

  • go test ./internal/app ./internal/acpx
  • go test ./...

@NitayRabi
NitayRabi marked this pull request as ready for review May 13, 2026 18:42
@NitayRabi
NitayRabi enabled auto-merge (squash) May 13, 2026 18:42
@NitayRabi
NitayRabi merged commit 8fe7d1e into main May 13, 2026
3 checks passed
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