carloea2 opened a new pull request, #8216:
URL: https://github.com/apache/texera/pull/8216

   ### What changes were proposed in this PR?
   
   Reject non-string state dictionary keys before JSON serialization instead of 
coercing them and risking collisions.
   
   The regression covers both top-level and nested mappings.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8215
   
   ### How was this PR tested?
   
   ```text
   python -c "import sys,pytest; 
sys.path[:0]=[r'<worktree>ambersrcmainpython',r'<main-checkout>ambersrcmainpython'];
 raise 
SystemExit(pytest.main([r'amber/src/test/python/core/models/test_state.py','-q','-p','no:cacheprovider']))"
   15 passed
   
   ruff check amber/src/main/python amber/src/test/python
   All checks passed
   
   ruff format --check amber/src/main/python amber/src/test/python
   213 files already formatted
   ```
   
   A direct runtime probe now raises:
   
   ```text
   TypeError: State dictionary keys must be strings
   ```
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Codex


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to