removesuffix helper returns an empty string for an empty suffix

Offen Anfängerfreundlich

#182 erstellt am 15.8.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 82/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Ruhig
Tech-Stack: python
Bereich: tooling

Rechercherichtung

Beginne bei src/browserbase/_utils/_utils.py:368-375 und untersuche die removesuffix-Implementierung zusammen mit der zugehörigen Präfix-Hilfsfunktion. Führe die vorhandene Testsuite aus und füge anschließend Tests für Fälle mit leerem Präfix und Suffix hinzu; fertig ist es, wenn removesuffix("abc", "") "abc" zurückgibt und dem Verhalten der Standardbibliothek entspricht.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Description

The internal removesuffix helper claims to backport str.removesuffix, but it returns an empty string when the requested suffix is empty.

Code reference

src/browserbase/_utils/_utils.py:368-375

The implementation executes string[:-len(suffix)]; for an empty suffix this becomes string[:-0], which is string[:0].

Reproduction

from browserbase._utils._utils import removesuffix

print(repr(removesuffix("abc", "")))
print(repr("abc".removesuffix("")))

Actual:

''
'abc'

Expected behavior

To match the documented stdlib behavior, an empty suffix should leave the original string unchanged.

Why it matters

This shared helper silently corrupts input for a valid edge case and has no direct test coverage. Even though current in-repo call sites are limited, it is exported through browserbase._utils and may be reused by generated or manual code later. The fix is small (if suffix and string.endswith(suffix): ...) and should include empty-string tests for both prefix/suffix helpers.

Vorherrschende Sprache: Python
Sterne: 93
Forks: 16
Ø Merge: 11 Min.
Gemergte PRs (30 T.): 3

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus browserbase/sdk-python

Extension example catches its own assertions and cannot detect failed deletion Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100

browserbase/sdk-python#180 · Erstellt am 15.8.2026
Query serializer silently drops empty-string parameters Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100

browserbase/sdk-python#179 · Erstellt am 15.8.2026
Mypy fails on Python 3.12 TypeAliasType compatibility branch Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

browserbase/sdk-python#178 · Erstellt am 15.8.2026
Fix malformed README quickstart fence and non-runnable timeout snippet Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100

browserbase/sdk-python#176 · Erstellt am 15.8.2026
Async streaming response wrappers leak an unawaited coroutine when not entered Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100

browserbase/sdk-python#175 · Erstellt am 15.8.2026

Alle Issues in browserbase/sdk-python