Skip to content

feat(read): add --raw-bytes to emit packet bytes as hex#29

Merged
higebu merged 1 commit intomainfrom
add-packet-bytes-cli-option
Apr 23, 2026
Merged

feat(read): add --raw-bytes to emit packet bytes as hex#29
higebu merged 1 commit intomainfrom
add-packet-bytes-cli-option

Conversation

@higebu
Copy link
Copy Markdown
Owner

@higebu higebu commented Apr 23, 2026

Summary

  • Add --raw-bytes flag to dsct read that appends the full original packet bytes (link-layer included) as a lowercase hex string under raw_bytes in each JSONL record.
  • Mirror the option as raw_bytes: bool in the MCP read_packets tool.
  • Update read_schema() and README.

Without the flag, output is byte-identical to before (backward compatible).

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 23, 2026

🐰 Bencher Report

Branchadd-packet-bytes-cli-option
Testbedubuntu-latest
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
milliseconds (ms)
(Result Δ%)
Upper Boundary
milliseconds (ms)
(Limit %)
target/release/dsct read bench_input.pcap📈 view plot
🚷 view threshold
5.93 ms
(-2.06%)Baseline: 6.05 ms
6.92 ms
(85.72%)
target/release/dsct stats bench_input.pcap📈 view plot
🚷 view threshold
72.06 ms
(+4.13%)Baseline: 69.21 ms
85.85 ms
(83.95%)
🐰 View full continuous benchmarking report in Bencher

@higebu higebu merged commit eab82cd into main Apr 23, 2026
15 of 16 checks passed
@higebu higebu deleted the add-packet-bytes-cli-option branch April 23, 2026 11:38
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