Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@vignesh07
Copy link
Contributor

Summary

Fixes iOS node connectivity by allowing clawdbot-ios as a valid gateway client id.

Context

The iOS app connects in node mode with:

  • clientId: "clawdbot-ios"
  • clientMode: "node"

The gateway handshake schema validates client.id against a strict allowlist (GatewayClientIdSchema). Since clawdbot-ios was not in the allowlist, the connection was rejected with:

  • invalid connect params: at /client/id: must be equal to constant; ... must match a schema in anyOf

Changes

  • Add clawdbot-ios to GATEWAY_CLIENT_IDS
  • Add a gateway test to ensure the handshake no longer fails schema validation for this id
  • Fix TS strictness in auto-reply status formatting (parts.filter(Boolean) narrowing) which was causing pnpm build to fail locally

Testing

  • pnpm test src/gateway/server.ios-client-id.test.ts src/auto-reply/status.test.ts
  • pnpm lint
  • pnpm build

🤖 AI-assisted

vignesh07 and others added 3 commits January 21, 2026 05:07
The iOS app currently identifies as clientId=clawdbot-ios when
connecting in node mode. Add this ID to the allowed gateway client
IDs so the handshake schema accepts it.

Also fixes a TS strictness issue in auto-reply status formatting
(parts filter) that caused
> [email protected] build /Users/vignesh/clawd/clawdbot-upstream
> tsc -p tsconfig.json && node --import tsx scripts/canvas-a2ui-copy.ts && node --import tsx scripts/copy-hook-metadata.ts && node --import tsx scripts/write-build-info.ts

[copy-hook-metadata] Copied boot-md/HOOK.md
[copy-hook-metadata] Copied command-logger/HOOK.md
[copy-hook-metadata] Copied session-memory/HOOK.md
[copy-hook-metadata] Copied soul-evil/HOOK.md
[copy-hook-metadata] Done to fail.
@steipete steipete force-pushed the fix/gateway-ios-client-id branch from 78a754e to 110079d Compare January 21, 2026 05:09
@steipete steipete merged commit 968b967 into openclaw:main Jan 21, 2026
27 of 42 checks passed
@steipete
Copy link
Contributor

Landed via temp rebase onto main.

  • Gate: pnpm lint && pnpm build && pnpm test
  • Land commit: 110079d
  • Merge commit: 968b967

Thanks @vignesh07!

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.

2 participants