Skip to content

Strip empty flash data and token from session#42

Merged
Naoray merged 2 commits intomainfrom
fix/strip-empty-flash-data
Feb 8, 2026
Merged

Strip empty flash data and token from session#42
Naoray merged 2 commits intomainfrom
fix/strip-empty-flash-data

Conversation

@Naoray
Copy link
Copy Markdown
Owner

@Naoray Naoray commented Feb 8, 2026

Summary

  • Strip _flash from session data when both old and new are empty arrays, reducing visual noise
  • Strip _token entirely from session data since it is always redacted and adds no debugging value
  • Remove _flash from the data section (it was duplicated there via Session::all()) so flash info only appears in the dedicated flash key when non-empty

Closes #35

Test plan

  • Verify empty flash data (_flash: { old: [], new: [] }) is stripped from session output
  • Verify non-empty flash data is preserved in the flash key
  • Verify _token is completely removed from session data
  • Verify existing session data collection still works correctly
  • All 282 tests pass

🤖 Generated with Claude Code

Naoray and others added 2 commits February 8, 2026 13:30
Closes #35

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ering

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Naoray Naoray merged commit 3d2dca7 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.

Strip empty flash data from Session section

1 participant