Skip to content

[codex] Add CAN ID filtering to summary CLIs#10

Draft
bobzhu94 wants to merge 1 commit into
mainfrom
codex/can-id-filter
Draft

[codex] Add CAN ID filtering to summary CLIs#10
bobzhu94 wants to merge 1 commit into
mainfrom
codex/can-id-filter

Conversation

@bobzhu94
Copy link
Copy Markdown
Owner

@bobzhu94 bobzhu94 commented Jun 5, 2026

Summary

  • Add repeated --id filtering to can-log-summary, can-batch-summary, and can-batch-check.
  • Reuse the existing CAN ID normalization path so decimal and hex IDs resolve consistently.
  • Document the focused CAN ID workflow in README and changelog.

Why

Focused CAN ID filtering makes bench debugging faster when a maintainer wants to inspect one publisher or message across multiple logs.

Validation

  • npm test
  • npm run smoke
  • node src/can-log-summary.mjs --id 0x180 examples/can-sample.log
  • node src/can-batch-summary.mjs --id 0x180 examples/can-sample.log examples/can-sample-jitter.log
  • node src/can-batch-check.mjs --id 0x180 --max-jitter-ms 8 examples/can-sample.log examples/can-sample-jitter.log

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