Add SkipPadding to BinaryWriterExtensions and fix missing guard in BinaryReaderExtensions

[Copilot]

BinaryWriter lacked a SkipPadding counterpart to the existing BinaryReader.SkipPadding. Additionally, BinaryReaderExtensions.SkipPadding was missing an argument guard, which could cause a divide-by-zero for non-positive boundaryBytes values.

Changes

• BinaryWriterExtensions — adds SkipPadding(int boundaryBytes = 4): writes 0x00 bytes until BaseStream.Position is aligned to the given boundary. Includes a guard against non-positive boundaryBytes to prevent divide-by-zero / infinite loops.
• BinaryReaderExtensions — adds the same ArgumentOutOfRangeException guard to the existing SkipPadding for consistency and to prevent divide-by-zero on non-positive boundaryBytes.
• BinaryWriterExtensionsTests — adds tests for the unaligned and already-aligned cases.

Usage

using var writer = new BinaryWriter(stream);
writer.Write((byte)0x01); // position = 1
writer.SkipPadding(boundaryBytes: 4); // writes 3 zero bytes → position = 4

[codecov]

Codecov Report

❌ Patch coverage is 80.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.08%. Comparing base (1a2fcb3) to head (cad6506).
⚠️ Report is 1 commits behind head on prime.

Files with missing lines	Patch %	Lines
PowerKit/Extensions/BinaryReaderExtensions.cs	0.00%	1 Missing and 1 partial ⚠️
PowerKit/Extensions/BinaryWriterExtensions.cs	60.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##            prime      #36      +/-   ##
==========================================
- Coverage   94.28%   94.08%   -0.20%     
==========================================
  Files          76       76              
  Lines        1469     1489      +20     
  Branches      121      124       +3     
==========================================
+ Hits         1385     1401      +16     
- Misses         65       67       +2     
- Partials       19       21       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Tyrrrz]

@copilot formatting failure

[Copilot]

Pull request overview

Adds a BinaryWriter counterpart to the existing BinaryReader.SkipPadding, enabling callers to write zero-byte padding until the underlying stream position is aligned to a specified boundary.

Changes:

• Add BinaryWriterExtensions.SkipPadding(int boundaryBytes = 4) to write 0x00 bytes until BaseStream.Position aligns to boundaryBytes.
• Add tests covering the unaligned and already-aligned scenarios for SkipPadding.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
PowerKit/Extensions/BinaryWriterExtensions.cs	Introduces SkipPadding for BinaryWriter with argument validation.
PowerKit.Tests/BinaryWriterExtensionsTests.cs	Adds tests verifying padding behavior for aligned/unaligned positions.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Tyrrrz]

@copilot do we have the same argument check in the other SkipPadding method?

[Copilot]

@copilot formatting failure

Fixed in 62c3ffd — ran CSharpier on the file and merged the single-argument ArgumentOutOfRangeException form you preferred. The build now passes the format check.

[Copilot]

@copilot do we have the same argument check in the other SkipPadding method?

No — BinaryReaderExtensions.SkipPadding didn't have the guard, so a non-positive boundaryBytes would cause a divide-by-zero. Added the same check in cad6506.