Skip to content

fix: report invalid Claude settings JSON - #1530

Open
yoldaolmak wants to merge 1 commit into
anthropics:mainfrom
yoldaolmak:codex/settings-json-errors
Open

fix: report invalid Claude settings JSON#1530
yoldaolmak wants to merge 1 commit into
anthropics:mainfrom
yoldaolmak:codex/settings-json-errors

Conversation

@yoldaolmak

Copy link
Copy Markdown

Summary

  • distinguish JSON-shaped settings input from a settings-file path
  • report invalid inline JSON, settings input files, and existing Claude settings distinctly
  • await and strengthen the asynchronous error-path tests

This addresses the Claude settings JSON validation portion of #242. GitHub Actions workflow YAML is parsed by GitHub before the action runs, so it remains outside this action-level change.

Verification

  • bun test (801 passing)
  • bun run typecheck
  • bun run format:check

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