Commit eb5c06b
committed
models: Remove payload re-exports to break import cycle
- remove payload-model re-exports from `pdfrest.models` to break the
`models -> payloads -> _internal -> types -> models` import cycle
- keep payload models available through `pdfrest.models.payloads`
- update encryption and permissions-password unit tests to import payload models
from `pdfrest.models.payloads` instead of `pdfrest.models`
Validation:
- `uv run ruff check src/pdfrest/models/__init__.py tests/test_encrypt_pdf.py tests/test_permissions_password.py`
- `uv run basedpyright src/pdfrest/models/__init__.py src/pdfrest/models/payloads.py src/pdfrest/models/_internal.py src/pdfrest/types/__init__.py src/pdfrest/types/public.py tests/test_encrypt_pdf.py tests/test_permissions_password.py`
- `uv run pytest -n auto --maxschedchunk 2 tests/test_encrypt_pdf.py tests/test_permissions_password.py`
Assisted-by: Codex1 parent d4731ba commit eb5c06b
3 files changed
Lines changed: 2 additions & 14 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
2 | | - | |
3 | | - | |
4 | | - | |
5 | | - | |
6 | | - | |
7 | 1 | | |
8 | 2 | | |
9 | 3 | | |
| |||
18 | 12 | | |
19 | 13 | | |
20 | 14 | | |
21 | | - | |
22 | | - | |
23 | 15 | | |
24 | 16 | | |
25 | 17 | | |
26 | 18 | | |
27 | 19 | | |
28 | 20 | | |
29 | | - | |
30 | | - | |
31 | 21 | | |
32 | 22 | | |
33 | 23 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
13 | | - | |
14 | 12 | | |
15 | 13 | | |
16 | 14 | | |
17 | 15 | | |
| 16 | + | |
18 | 17 | | |
19 | 18 | | |
20 | 19 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
15 | | - | |
16 | | - | |
17 | 15 | | |
| 16 | + | |
18 | 17 | | |
19 | 18 | | |
20 | 19 | | |
| |||
0 commit comments