Fix unreachable str handling in string_view wrapper methods

[Copilot]

The getStringViewLength and stringViewToString methods declared bytes parameter types, but contained str handling logic that was unreachable since Cython enforces type constraints at the C boundary.

Changes

• Remove bytes type annotation from sv parameter (now untyped/object)
• Update docstrings to reflect Union[bytes, str] acceptance
• Existing isinstance checks and .encode() logic now reachable

# Before: str never reaches function body
def getStringViewLength(self, bytes sv):
    if isinstance(sv, str):  # unreachable
        v0 = sv.encode('utf-8')

# After: both types reach function body
def getStringViewLength(self, sv):
    if isinstance(sv, str):  # now works
        v0 = sv.encode('utf-8')

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[timosachsenberg]

Ah I see Sam merged it. Wasn’t sure - I think cleanup is not tested so nothing failed and I was not sure how to properly test