Skip to content

Enhance validation, batch, and deserialization error messages#1035

Closed
gjmulhol wants to merge 1 commit intomainfrom
improve/validation-batch-error-messages
Closed

Enhance validation, batch, and deserialization error messages#1035
gjmulhol wants to merge 1 commit intomainfrom
improve/validation-batch-error-messages

Conversation

@gjmulhol
Copy link
Copy Markdown

Summary

  • Batcher collision error explains what a collision means and suggests unique UIDs
  • Batcher dependency error shows count and first 10 dependency names
  • Collection uid=None error explains likely cause and suggests .register()
  • Deserialization errors truncate data dicts (200 chars) and value reprs (100 chars)

Test plan

  • All 1342 existing tests pass (no assertions on exact error text in these paths)

🤖 Generated with Claude Code

PR 9 of 11. Independent — no dependencies on other PRs.

- Batcher collision error now explains what a collision means
  and suggests ensuring unique UIDs
- Batcher dependency error now shows count and first 10
  dependency names, suggests increasing batch_size
- Collection uid=None error now explains the likely cause
  (unregistered object) and suggests calling .register()
- Deserialization errors truncate data dicts to 200 chars and
  value reprs to 100 chars to avoid overwhelming output

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@gjmulhol
Copy link
Copy Markdown
Author

Consolidated into docs/improve-sdk-documentation branch.

@gjmulhol gjmulhol closed this Mar 25, 2026
@gjmulhol gjmulhol deleted the improve/validation-batch-error-messages branch March 25, 2026 22:26
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