Skip to content

Fix Psych::DisallowedClass when using schema_element_name_overrides#1198

Open
myronmarston wants to merge 1 commit into
mainfrom
myron/fix-1021
Open

Fix Psych::DisallowedClass when using schema_element_name_overrides#1198
myronmarston wants to merge 1 commit into
mainfrom
myron/fix-1021

Conversation

@myronmarston
Copy link
Copy Markdown
Collaborator

@myronmarston myronmarston commented May 20, 2026

Summary

  • Fix Psych::DisallowedClass: Tried to load unspecified class: Symbol when schema_element_name_overrides contains symbol keys (the natural Ruby hash syntax)
  • Convert symbol keys/values to strings in to_dumpable_hash so the YAML output is safe_load-compatible

Fixes #1021

Test plan

  • Unit test: SchemaElementNames roundtrips through to_dumpable_hash → from_hash with symbol-keyed overrides
  • Higher-level test: Schema roundtrips through YAML.dump → YAML.safe_load → Schema.from_hash with symbol-keyed overrides
  • All existing tests pass (107 schema_artifacts specs, 17 rake_tasks integration specs)

🤖 Generated with Claude Code

@myronmarston @claude
Fix Psych::DisallowedClass when using schema_element_name_overrides (#…
080918e 
…1021)

`to_dumpable_hash` was dumping symbol-keyed overrides as-is, producing
YAML with Ruby symbol tags that `YAML.safe_load` rejects on re-read.
Convert keys/values to strings so the output is safe_load-compatible.

Fixes #1021

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@myronmarston myronmarston marked this pull request as ready for review May 20, 2026 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ayousufi ayousufi Awaiting requested review from ayousufi ayousufi is a code owner

@BrianSigafoos-SQ BrianSigafoos-SQ Awaiting requested review from BrianSigafoos-SQ BrianSigafoos-SQ is a code owner

@bsorbo bsorbo Awaiting requested review from bsorbo bsorbo is a code owner

@jwils jwils Awaiting requested review from jwils jwils is a code owner

@jwondrusch jwondrusch Awaiting requested review from jwondrusch jwondrusch is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] Psych::DisallowedClass: Tried to load unspecified class: Symbol

1 participant

@myronmarston