Skip to content

Add deserialization_exception field for programmatic error handling#316

Open
bjester wants to merge 2 commits intolearningequality:release-v0.8.xfrom
bjester:des-exc
Open

Add deserialization_exception field for programmatic error handling#316
bjester wants to merge 2 commits intolearningequality:release-v0.8.xfrom
bjester:des-exc

Conversation

@bjester
Copy link
Copy Markdown
Member

@bjester bjester commented Apr 14, 2026

Summary

This allows programmatic handling of deserialization errors instead of parsing error messages

  • Adds a new deserialization_exception nullable field to Store model that stores the fully-qualified exception class path when deserialization fails
  • Updates existing deserialization_error field to be nullable
  • Updates handling of deserialization errors to set both fields, or otherwise null both of them
  • Adds tests for changes
  • Removes migration tests that verified the squash migration, which now cause issues with new migrations

References

Followup to #306
Closes #315

AI Usage

Directed an AI agent to tackle the issue, iterated on its work for a loop, then did a small refactor myself. Had the AI agent add test coverage, then reviewed.

@bjester bjester requested review from ozer550 and rtibbles April 14, 2026 18:31
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