Skip to content

fix: validate structured image prompt blocks#110

Merged
vincentkoc merged 3 commits intomainfrom
vincentkoc-code/live-image-validation
Mar 11, 2026
Merged

fix: validate structured image prompt blocks#110
vincentkoc merged 3 commits intomainfrom
vincentkoc-code/live-image-validation

Conversation

@vincentkoc
Copy link
Copy Markdown
Member

Summary

  • validate structured ACP image prompt blocks for non-empty image/* MIME types and base64 payloads
  • surface human-readable CLI usage errors for invalid structured prompt JSON in text mode too
  • add explicit image validation coverage plus an opt-in pnpm run test:live Cursor ACP smoke test outside the default CI suite
  • add an Unreleased changelog note covering image MIME support and the live test path

Testing

  • pnpm run build:test && node --test dist-test/test/prompt-content.test.js dist-test/test/cli.test.js dist-test/test/integration.test.js
  • pnpm run check:docs
  • pnpm run check (fails locally at the existing test:coverage threshold report: 37.42% line / 67.60% branch / 38.02% function; targeted tests above pass)
  • pnpm run test:live (fails locally with Cursor auth: Authentication required)

@vincentkoc vincentkoc merged commit 1b8b148 into main Mar 11, 2026
7 checks passed
@vincentkoc vincentkoc deleted the vincentkoc-code/live-image-validation branch March 11, 2026 03:17
vincentkoc added a commit to MikeChongCan/acpx that referenced this pull request Mar 11, 2026
…-gemini-version-check

* origin/main:
  fix(windows): support .cmd/.bat agent commands (openclaw#102)
  chore(deps-dev): bump @typescript/native-preview (openclaw#107)
  chore(deps-dev): bump tsdown from 0.21.0-beta.2 to 0.21.0 (openclaw#106)
  fix: validate structured image prompt blocks (openclaw#110)
  chore: switch releases to tag-based publishing
  chore: bump version to 0.2.0

# Conflicts:
#	src/client.ts
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