Skip to content

content dry-run の検証を本実行に近づける - #40

Merged
mrmtsu merged 4 commits into
mainfrom
codex/harden-content-dry-run-validation
May 17, 2026
Merged

content dry-run の検証を本実行に近づける#40
mrmtsu merged 4 commits into
mainfrom
codex/harden-content-dry-run-validation

Conversation

@mrmtsu

@mrmtsu mrmtsu commented May 17, 2026

Copy link
Copy Markdown
Owner

概要

  • schema に type: string のヒントがある single select でも、["value"] 形式を validate で落とさないようにしました
  • content create/update --dry-run で schema validation を行い、入力そのままの payload と本実行時に送る normalizedPayload を出力するようにしました
  • JSON mode の microCMS API error で、取得できる API response body を error.details に含めるようにしました
  • バージョンを 0.7.4 に上げ、CHANGELOG / CHANGELOG.ja / README / README.ja に今回の挙動を追記しました

背景

0.7.3 のフィードバックでは、validate / dry-run / 本実行で payload shape の期待値が一致せず、dry-run の .ok == true を信じたあとに本実行で 400 になるケースがありました。

特に危なかったのは、dry-run が schema validation や本実行時の select 正規化を反映せず、単に入力 payload を表示して成功扱いにしていた点です。このPRでは、直接の content create/update --dry-run を本実行に近い preflight に寄せています。

確認

  • pnpm run check:ci
  • push 時の pre-push hook でも npm run check:ci が成功

@mrmtsu mrmtsu changed the title [codex] Harden content dry-run validation content dry-run の検証を本実行に近づける May 17, 2026
@mrmtsu
mrmtsu marked this pull request as ready for review May 17, 2026 05:59
@mrmtsu
mrmtsu merged commit 10d3324 into main May 17, 2026
13 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