refactor(builtin): make BytesView escape_to private and generic over Logger

Address review comments on PR #4196: BytesView::escape_to and its helper write_byte_hex are internal (used only by the Show and ToJson impls in this package), so drop pub, and genericize them over [T : Logger] so writes dispatch statically where a concrete logger is passed. Regenerate pkg.generated.mbti (the pub entry is gone).

Co-Authored-By: SeekMoon <seekmoon@moonbitlang.com>

Author: bobzhang

builtin/bytesview.mbt

@@ -331,7 +331,7 @@ pub fn BytesView::iter2(self : BytesView) -> Iter2[Int, Byte] {
 }
 
 ///|
-fn write_byte_hex(logger : &Logger, byte : Byte) -> Unit {
+fn[T : Logger] write_byte_hex(logger : T, byte : Byte) -> Unit {
   fn hex_digit(value : Int) -> Char {
     if value < 10 {
       (value + '0'.to_int()).unsafe_to_char()
@@ -349,7 +349,7 @@ fn write_byte_hex(logger : &Logger, byte : Byte) -> Unit {
 /// Writes the bytes with printable ASCII kept as-is and everything else
 /// rendered as `\xHH`, without the surrounding `b"`/`"`. Shared by `Show`,
 /// which adds the quotes, and `ToJson`, which does not.
-pub fn BytesView::escape_to(self : BytesView, logger : &Logger) -> Unit {
+fn[T : Logger] BytesView::escape_to(self : BytesView, logger : T) -> Unit {
   for byte in self {
     if byte is (b' '..=b'~') && byte != b'"' && byte != b'\\' {
       logger.write_char(byte.to_char())

builtin/pkg.generated.mbti

@@ -1170,7 +1170,6 @@ pub fn BytesView::compare(Self, Self) -> Int
 pub fn BytesView::data(Self) -> Bytes
 pub fn BytesView::equal(Self, Self) -> Bool
 pub fn BytesView::equal_to_bytes(Self, Bytes) -> Bool
-pub fn BytesView::escape_to(Self, &Logger) -> Unit
 pub fn BytesView::find(Self, Self) -> Int?
 pub fn BytesView::get(Self, Int) -> Byte?
 pub fn BytesView::get_view(Self, start? : Int, end? : Int) -> Self?