Odchudź shimy PBN (usuń pbn_api.const/utils/client.transport)#609
Open
mpasternak wants to merge 1 commit into
Open
Odchudź shimy PBN (usuń pbn_api.const/utils/client.transport)#609mpasternak wants to merge 1 commit into
mpasternak wants to merge 1 commit into
Conversation
Trzy pliki w pbn_api były czystymi re-eksportami z pakietów PBN (`from pbn_client.X import *`) — zbędna warstwa pośrednia po tym jak kod klienta wyszedł na PyPI. Usunięte + repoint callerów wprost na paczkę: - pbn_api/const.py → callery: from pbn_client.const import ... (22 plików) - pbn_api/utils.py → from pbn_client.dict_utils import ... (2) - pbn_api/client/transport.py → from pbn_client.transport import ... (1) Spójne z resztą kodu po ekstrakcji (mongodb_ops itd. już importują wprost z pbn_client/django_pbn_client). ZOSTAWIONE świadomie: - pbn_api/exceptions.py — hybryda: wyjątki BPP-specific + alias BrakIDPracyPoStroniePBN = PublicationNotFound. - pbn_api/models/base.py — seam pod przyszłą Fazę 3 (abstrakcyjne modele). Czysto mechaniczne (tylko podmiana ścieżki modułu, nazwy bez zmian). Review: fable PASS + codex SHIP (brak defektów). manage.py check OK, pbn_api 339 passed, zero nowych naruszeń ruff. Pre-existing lint (F401/I001/UP012 w tkniętych plikach) świadomie NIE ruszany — poza zakresem. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pierwszy z trzech zaplanowanych elementów (kolejność: shimy → P4 → Faza 3).
Stack na #607.
Co i dlaczego
Po ekstrakcji kodu klienta PBN na PyPI trzy pliki w
pbn_apizostały jakoczyste re-eksporty (
from pbn_client.X import *) — zbędna warstwa pośrednia.Usunięte, a callery repointowane wprost na paczkę (spójnie z resztą kodu, który
już importuje z
pbn_client/django_pbn_client):pbn_api/const.pyfrom pbn_client.const import …pbn_api/utils.pyfrom pbn_client.dict_utils import …pbn_api/client/transport.pyfrom pbn_client.transport import …Zostawione świadomie
pbn_api/exceptions.py— hybryda: definiuje wyjątki BPP-specific + hostujealias
BrakIDPracyPoStroniePBN = PublicationNotFound(z P3: BrakIDPracyPoStroniePBN = alias PublicationNotFound (rozpoznanie 422 w paczce) #606).pbn_api/models/base.py— seam pod przyszłą Fazę 3 (abstrakcyjne modeleencji); kasowanie teraz = churn delete-then-recreate.
Charakter
Czysto mechaniczne — tylko podmiana ścieżki modułu, importowane nazwy bez zmian.
Wszystkie 25 nazw istnieją publicznie w docelowych modułach
pbn_client.Review (sub-agenty, jak w reszcie stacka)
istnieją, zero churn,
base.py/exceptions.pynietknięte).Weryfikacja
manage.py check: OKpytest src/pbn_api/tests/: 339 passedplikach świadomie nie ruszane — poza zakresem shimów; „Lint changed files"
jest non-blocking).
🤖 Generated with Claude Code