fix(fs): carry file_get_contents results as bytes

file_get_contents() and file_get_contents_with_max_length() return
Vec<u8> instead of a from_utf8_lossy'd String.
Call sites whose consumer takes a &str still convert lossily and are
marked TODO(bytes).

file_get_contents_with_max_length() now reads at most the requested
number of bytes instead of the whole file.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/crates/shirabe/src/command/init_command.rs b/crates/shirabe/src/command/init_command.rs
index 8d892bd3..67530008 100644
--- a/crates/shirabe/src/command/init_command.rs
+++ b/crates/shirabe/src/command/init_command.rs
@@ -215,7 +215,9 @@ impl InitCommand {
     fn add_vendor_ignore(&self, ignore_file: &str, vendor: &str) {
         let mut contents = String::new();
         if file_exists(ignore_file) {
-            contents = file_get_contents(ignore_file).unwrap_or_default();
+            // TODO(bytes): the ignore file is edited as a String.
+            contents = String::from_utf8_lossy(&file_get_contents(ignore_file).unwrap_or_default())
+                .into_owned();
 
             if strpos(&contents, "\n") != Some(0) {
                 contents.push('\n');