fix(protocol): allow paperclip field in AgentParamsSchema#75023
fix(protocol): allow paperclip field in AgentParamsSchema#75023Linux2010 wants to merge 1 commit into
Conversation
Paperclip agent injects a 'paperclip' property into agent invocation payloads, but gateway validation rejects unknown fields due to additionalProperties: false. Add optional paperclip field to allow third-party integrations without breaking validation. Fixes openclaw#74635
|
Thanks for the context here. I swept through the related work, and this is now duplicate or superseded. Close as duplicate/superseded. The Paperclip root-field rejection is still real on current main, but this PR is a later, narrower duplicate of older open PRs that already cover the same compatibility fix with tests and maintainer-review context. So I’m closing this here and keeping the remaining discussion on the canonical linked item. Review detailsBest possible solution: Close this duplicate PR and consolidate review on one canonical implementation, preferably an additive adapter metadata contract or an explicitly approved Paperclip compatibility field with tests, generated protocol artifacts, and follow-up closure of duplicate reports after merge. Do we have a high-confidence way to reproduce the issue? Yes. Sending a Gateway Is this the best way to solve the issue? No. This PR fixes the immediate validation failure in principle, but it is not the best current path because older open PRs already cover the same work with tests and maintainer context, and this branch leaves protocol tests/generated artifacts out of sync. Security review: Security review cleared: The diff only adds an optional ignored schema field and does not touch workflows, dependencies, lockfiles, scripts, secrets, or package publishing surfaces. What I checked:
Likely related people:
Codex review notes: model gpt-5.5, reasoning high; reviewed against 9cb71f767242. |
Summary
Paperclip agent injects a
paperclipproperty into agent invocation payloads when using the openclaw_gateway adapter, but the gateway validation rejects it with:AgentParamsSchema has
additionalProperties: false, so any unknown field fails validation.Fix
Add
paperclip: Type.Optional(Type.Unknown())to AgentParamsSchema to allow Paperclip and other third-party integrations to pass runtime context without breaking gateway validation. The gateway accepts but ignores this field.Test Plan
paperclipfieldFixes #74635