Skip to content

fix(tracing): truncate serialized job payload data#44

Merged
Naoray merged 2 commits intomainfrom
fix/truncate-job-payload
Feb 8, 2026
Merged

fix(tracing): truncate serialized job payload data#44
Naoray merged 2 commits intomainfrom
fix/truncate-job-payload

Conversation

@Naoray
Copy link
Copy Markdown
Owner

@Naoray Naoray commented Feb 8, 2026

Summary

  • Truncates long serialized string values (e.g. payload.data.command) in job payloads to 500 characters with a ... [truncated] marker
  • Applies truncation recursively after sensitive data redaction, keeping useful payload fields like job, uuid, attempts, maxTries, timeout, tags, and displayName intact
  • Adds 3 new tests covering truncation of long strings, preservation of short strings, and preservation of non-string values

Closes #34

Test plan

  • Existing test for job context collection still passes
  • New test verifies long serialized command strings are truncated to 500 chars + marker
  • New test verifies short string values are not truncated
  • New test verifies non-string values (integers, null, arrays) are preserved
  • Full test suite passes (282 tests, 877 assertions)

🤖 Generated with Claude Code

Naoray and others added 2 commits February 8, 2026 13:31
Closes #34

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Naoray Naoray merged commit ca4507b into main Feb 8, 2026
11 checks passed
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.

Truncate or omit serialized job payload data

1 participant