Skip to content

fix(outbound): ignore schema-padded poll metadata on send#93441

Merged
vincentkoc merged 3 commits into
mainfrom
clownfish/ghcrawl-157058-autonomous-smoke
Jun 16, 2026
Merged

fix(outbound): ignore schema-padded poll metadata on send#93441
vincentkoc merged 3 commits into
mainfrom
clownfish/ghcrawl-157058-autonomous-smoke

Conversation

@openclaw-clownfish

Copy link
Copy Markdown
Contributor

Summary

Validation

  • pnpm -s vitest run src/poll-params.test.ts src/infra/outbound/message-action-runner.send-validation.test.ts src/infra/outbound/message-action-runner.core-send.test.ts --maxWorkers=1
  • pnpm check:changed

Fixes #43015.
Source PR credit: #65301 by @weichengdeng.

Clownfish 🐠 replacement reef notes:

fish notes: model gpt-5.5, reasoning medium; reviewed against 4d9329c.

@openclaw-clownfish openclaw-clownfish Bot added the clownfish Tracked by Clownfish automation label Jun 16, 2026
@vincentkoc

Copy link
Copy Markdown
Member

Land-ready maintainer verification:

  • Fresh branch update onto fixed main: head 97a4a13e0da0c136fc7b01be16bbe9f7c61340f1.
  • Hosted PR CI is clean: 137 passing checks, 26 skipped, 0 failing checks.
  • The previous inherited blockers are resolved on this head: checks-node-core-fast and checks-node-agentic-gateway-core passed.
  • Diff sanity checked: hasPollCreationParams only gates action === "send"; explicit action="poll" still goes through poll validation, so schema-padded send metadata no longer falsely trips poll intent.

Merging this PR now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clownfish Tracked by Clownfish automation size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

message.send schema overexposes poll/components/modal causing GPT auto-population breakages

1 participant