Add BinaryReaderExtensions and BinaryWriterExtensions

[Copilot]

Ports BinaryReader and BinaryWriter extension methods from Tyrrrz/Ressy.

New extensions

BinaryReaderExtensions

• SkipZeroes(long? maxSkipLength = null) — advances past zero bytes, optionally bounded
• ReadNullTerminatedString() — reads chars until \0, returns the accumulated string

BinaryWriterExtensions

• WriteNullTerminatedString(string value) — writes each char then a null terminator

using var reader = new BinaryReader(stream);
reader.SkipZeroes();
var name = reader.ReadNullTerminatedString();

using var writer = new BinaryWriter(stream);
writer.WriteNullTerminatedString("hello");

[Copilot]

Pull request overview

Ports null-terminated string / padding-related BinaryReader and BinaryWriter extensions (and tests) into PowerKit.

Changes:

• Added BinaryReaderExtensions.SkipZeroes(long? maxSkipLength = null) and ReadNullTerminatedString().
• Added BinaryWriterExtensions.WriteNullTerminatedString(string value).
• Added unit tests for the new extensions.

Reviewed changes

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

File	Description
PowerKit/Extensions/BinaryWriterExtensions.cs	Adds null-terminated string writing extension for BinaryWriter.
PowerKit/Extensions/BinaryReaderExtensions.cs	Adds zero-padding skipping and null-terminated string reading extensions for BinaryReader.
PowerKit.Tests/BinaryWriterExtensionsTests.cs	Verifies null-terminated string writing behavior.
PowerKit.Tests/BinaryReaderExtensionsTests.cs	Verifies zero skipping and null-terminated string reading behavior.

---

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

[Tyrrrz]

Can we use Peek to avoid relying on seeking? @copilot

[Copilot]

Done in 7cab462. SkipZeroes now uses PeekChar() to look ahead before consuming a byte — no seeking at all. The CanSeek guard, BaseStream.Length, and Seek(-1, Current) are all gone. The bounded variant tracks a skipped counter instead of comparing stream positions. The NonSeekableStream test and helper class were also removed since they're no longer relevant.

[Tyrrrz]

Can we use an u8 string here? @copilot

[Copilot]

Done in f45cd92. The assertion now uses "hello\0"u8.ToArray().