PyPtt 2.0.9__tar.gz → 2.1.0__tar.gz

{pyptt-2.0.9 → pyptt-2.1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: PyPtt
-Version: 2.0.9
+Version: 2.1.0
 Summary: PyPtt github: https://github.com/PyPtt/PyPtt
 Author-email: CodingMan <pttcodingman@gmail.com>
 License: GNU Lesser General Public License v3 (LGPLv3)
@@ -38,6 +38,7 @@ Provides-Extra: dev
 Requires-Dist: pytest; extra == "dev"
 Requires-Dist: python-dotenv; extra == "dev"
 Requires-Dist: flake8; extra == "dev"
+Requires-Dist: pyte; extra == "dev"
 Requires-Dist: sphinx; extra == "dev"
 Requires-Dist: sphinx-copybutton; extra == "dev"
 Requires-Dist: pygments==2.15.0; extra == "dev"

{pyptt-2.0.9 → pyptt-2.1.0}/PyPtt/__init__.py

@@ -1,4 +1,4 @@
-__version__ = '2.0.9'
+__version__ = '2.1.0'
 __author__ = 'CodingMan'
 __email__ = 'pttcodingman@gmail.com'
 

{pyptt-2.0.9 → pyptt-2.1.0}/PyPtt/_api_get_post_list.py

@@ -72,9 +72,13 @@ def get_post_list(api, board: str, limit: int = 20, offset: int = 0) -> List[Dic
                 # replace the first cursor in the line
                 line = line.replace(api.cursor, ' ' * len(api.cursor.encode('big5uao')), 1)
 
-                if api.cursor == data_type.Cursor.NEW:
-                    # add one space in the beginning to align the old cursor line
-                    line = f' {line}'
+            if api.cursor == data_type.Cursor.NEW:
+                # Pad one leading space so the fixed-column offsets below
+                # apply equally to cursor and non-cursor rows. Before the
+                # VT100Parser cell-tracking fix, the parser inserted this
+                # padding implicitly via its off-by-one column quirk; the
+                # offsets here were tuned to that padded layout.
+                line = f' {line}'
 
             cur_index = int(line[:8].strip())
             status = line[8:10].strip()

{pyptt-2.0.9 → pyptt-2.1.0}/PyPtt/screens.py

@@ -1,5 +1,6 @@
 import re
 import sys
+import unicodedata
 
 from uao import register_uao
 
@@ -8,6 +9,37 @@ from . import log
 register_uao()
 
 
+def _cell_width(ch: str) -> int:
+    """Return the terminal-cell width of a single character.
+
+    PTT is a Chinese BBS rendered on East Asian terminals. Wide ('W'),
+    Fullwidth ('F'), and Ambiguous ('A') characters all occupy 2 cells —
+    Ambiguous includes glyphs like ※, ←, →, ◎ that PTT's protocol treats
+    as full-width when positioning the cursor.
+    """
+    return 2 if unicodedata.east_asian_width(ch) in ('W', 'F', 'A') else 1
+
+
+def _cell_len(text: str) -> int:
+    return sum(_cell_width(c) for c in text)
+
+
+def _str_pos_at_cells(line: str, cells: int) -> int:
+    """Return the string index that corresponds to `cells` visual cells.
+
+    If `cells` lands inside a wide character, the index of the character *after*
+    it is returned (i.e. the wide character is treated as fully consumed).
+    """
+    if cells <= 0:
+        return 0
+    consumed = 0
+    for i, ch in enumerate(line):
+        if consumed >= cells:
+            return i
+        consumed += _cell_width(ch)
+    return len(line)
+
+
 class Target:
     MainMenu = [
         '離開，再見',
@@ -160,7 +192,10 @@ class Target:
 
     CursorToGoodbye = MainMenu.copy()
 
-    content_start = '─── ── ── ── ── ── ── ── ── ── ── ── ── ── ── ── ── ── ──'
+    # PTT separator: ~39 consecutive box-drawing horizontals. The legacy
+    # parser produced cell-padded output ('─── ── ──...'); with cell-aware
+    # cursor tracking the spaces are gone and we see a continuous run.
+    content_start = '───────────────────────────────────────'
     content_end_list = [
         '--\n※ 發信站: 批踢踢實業坊',
         '--\n※ 發信站: 批踢踢兔(ptt2.cc)',
@@ -211,10 +246,6 @@ class VT100Parser:
         self._cursor_x = 0
         self._cursor_y = 0
 
-    def _2j(self):
-        self.screen = [''] * 24
-        self.screen_length = dict()
-
     def _move(self, x, y):
         self._cursor_x = x
         self._cursor_y = min(y, len(self.screen) - 1)
@@ -224,10 +255,10 @@ class VT100Parser:
         self._cursor_y = min(self._cursor_y + 1, len(self.screen) - 1)
 
     def _k(self):
-        if self._cursor_x == 0:
-            # nothing happen but cause error
-            return
-        self.screen[self._cursor_y] = self.screen[self._cursor_y][:self._cursor_x]
+        # Erase from cursor to end of line (VT100 ESC [ K).
+        pos = _str_pos_at_cells(self.screen[self._cursor_y], self._cursor_x)
+        self.screen[self._cursor_y] = self.screen[self._cursor_y][:pos]
+        self.screen_length[self._cursor_y] = self._cursor_x
 
     def __init__(self, bytes_data, encoding):
         # self._data = data
@@ -282,7 +313,8 @@ class VT100Parser:
                 xy_part = xy_result.group(0)
 
                 new_y = int(xy_part[6:xy_part.find(';')]) - 1
-                new_x = int(xy_part[xy_part.find(';') + 1: -1])
+                # VT100 columns are 1-based; convert to 0-based cursor_x.
+                new_x = int(xy_part[xy_part.find(';') + 1: -1]) - 1
                 # log.py.info('xy', xy_part, new_x, new_y)
                 self._move(new_x, new_y)
 
@@ -297,7 +329,7 @@ class VT100Parser:
                 # print(f'-{data[:1]}-{len(data[:1].encode("big5-uao", "replace"))}')
 
                 if self._cursor_y not in self.screen_length:
-                    self.screen_length[self._cursor_y] = len(self.screen[self._cursor_y].encode(encoding, 'replace'))
+                    self.screen_length[self._cursor_y] = _cell_len(self.screen[self._cursor_y])
 
                 current_line_length = self.screen_length[self._cursor_y]
                 replace_mode = False
@@ -318,14 +350,15 @@ class VT100Parser:
                 current_index = min(next_newline, next_esc)
 
                 current_data = data[:current_index]
-                current_data_length = len(current_data.encode(encoding, 'replace'))
-                # print('=', current_data, '=', current_data_length)
+                current_data_length = _cell_len(current_data)
                 if replace_mode:
-                    current_line = self.screen[self._cursor_y][:self._cursor_x]
-                    current_line += current_data
-                    current_line += self.screen[self._cursor_y][self._cursor_x + len(current_data):]
-
-                    self.screen[self._cursor_y] = current_line
+                    line = self.screen[self._cursor_y]
+                    splice_start = _str_pos_at_cells(line, self._cursor_x)
+                    splice_end = _str_pos_at_cells(line, self._cursor_x + current_data_length)
+                    self.screen[self._cursor_y] = line[:splice_start] + current_data + line[splice_end:]
+                    self._cursor_x += current_data_length
+                    if self._cursor_x > self.screen_length[self._cursor_y]:
+                        self.screen_length[self._cursor_y] = self._cursor_x
                 else:
                     self.screen[self._cursor_y] += current_data
                     self._cursor_x += current_data_length

pyptt-2.1.0/PyPtt/ssl_config.py

@@ -0,0 +1,24 @@
+key = """-----BEGIN EC PARAMETERS-----
+BggqhkjOPQMBBw==
+-----END EC PARAMETERS-----
+-----BEGIN EC PRIVATE KEY-----
+MHcCAQEEIO2MHcOPROQP2B/IzywPM+VycdprTWfPpc/khcpekRscoAoGCCqGSM49
+AwEHoUQDQgAEjVADKUyxMVKt7TRXiP3gdfPblx/KsRafJ1SvrZNwXHb0Q1WEqT45
+LJ+2C4/hGUTdCAKYej7vO0ibHHJ3AXF3Zg==
+-----END EC PRIVATE KEY-----
+"""
+
+cert = """-----BEGIN CERTIFICATE-----
+MIIB6TCCAY8CFCmxmdBhhrcpMq15P8fGZ0kvsYp5MAoGCCqGSM49BAMCMHcxCzAJ
+BgNVBAYTAlRXMQ4wDAYDVQQIDAVTdGF0ZTENMAsGA1UEBwwEQ2l0eTEVMBMGA1UE
+CgwMT3JnYW5pemF0aW9uMRwwGgYDVQQLDBNPcmdhbml6YXRpb25hbCBVbml0MRQw
+EgYDVQQDDAtDb21tb24gTmFtZTAeFw0yNjA1MjQwMjU5NDJaFw0zNjA1MjEwMjU5
+NDJaMHcxCzAJBgNVBAYTAlRXMQ4wDAYDVQQIDAVTdGF0ZTENMAsGA1UEBwwEQ2l0
+eTEVMBMGA1UECgwMT3JnYW5pemF0aW9uMRwwGgYDVQQLDBNPcmdhbml6YXRpb25h
+bCBVbml0MRQwEgYDVQQDDAtDb21tb24gTmFtZTBZMBMGByqGSM49AgEGCCqGSM49
+AwEHA0IABI1QAylMsTFSre00V4j94HXz25cfyrEWnydUr62TcFx29ENVhKk+OSyf
+tguP4RlE3QgCmHo+7ztImxxydwFxd2YwCgYIKoZIzj0EAwIDSAAwRQIgfQDJrQCU
+59HONPJD3U6a8z7rOuDk0KiwvKMSgqJZWUYCIQCznuvlbggeMnlm/9EDxyawnGKs
+IZ2nArdE8SMFaxoStQ==
+-----END CERTIFICATE-----
+"""

{pyptt-2.0.9 → pyptt-2.1.0}/PyPtt.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: PyPtt
-Version: 2.0.9
+Version: 2.1.0
 Summary: PyPtt github: https://github.com/PyPtt/PyPtt
 Author-email: CodingMan <pttcodingman@gmail.com>
 License: GNU Lesser General Public License v3 (LGPLv3)
@@ -38,6 +38,7 @@ Provides-Extra: dev
 Requires-Dist: pytest; extra == "dev"
 Requires-Dist: python-dotenv; extra == "dev"
 Requires-Dist: flake8; extra == "dev"
+Requires-Dist: pyte; extra == "dev"
 Requires-Dist: sphinx; extra == "dev"
 Requires-Dist: sphinx-copybutton; extra == "dev"
 Requires-Dist: pygments==2.15.0; extra == "dev"

{pyptt-2.0.9 → pyptt-2.1.0}/PyPtt.egg-info/SOURCES.txt

@@ -50,10 +50,12 @@ PyPtt.egg-info/SOURCES.txt
 PyPtt.egg-info/dependency_links.txt
 PyPtt.egg-info/requires.txt
 PyPtt.egg-info/top_level.txt
+tests/test_bucket.py
 tests/test_change_pw.py
 tests/test_comment.py
 tests/test_del_post.py
 tests/test_exceptions.py
+tests/test_fast_post.py
 tests/test_get_board_info.py
 tests/test_get_board_list.py
 tests/test_get_bottom_post_list.py
@@ -62,6 +64,7 @@ tests/test_get_mail.py
 tests/test_get_newest_index.py
 tests/test_get_post.py
 tests/test_get_post_list.py
+tests/test_get_post_list_parser.py
 tests/test_get_post_parser.py
 tests/test_get_time.py
 tests/test_get_user.py
@@ -70,8 +73,13 @@ tests/test_give_money.py
 tests/test_i18n.py
 tests/test_init.py
 tests/test_logger.py
+tests/test_mail_send_and_del.py
+tests/test_mark_post.py
 tests/test_parse_query_post.py
 tests/test_post.py
+tests/test_pyte_parity.py
 tests/test_reply_post.py
 tests/test_search_user.py
-tests/test_service.py
+tests/test_service.py
+tests/test_set_board_title.py
+tests/test_vt100_parser.py

{pyptt-2.0.9 → pyptt-2.1.0}/PyPtt.egg-info/requires.txt

@@ -13,6 +13,7 @@ uvicorn[standard]
 pytest
 python-dotenv
 flake8
+pyte
 sphinx
 sphinx-copybutton
 pygments==2.15.0

{pyptt-2.0.9 → pyptt-2.1.0}/pyproject.toml

@@ -53,6 +53,7 @@ dev = [
     "pytest",
     "python-dotenv",
     "flake8",
+    "pyte",
     "sphinx",
     "sphinx-copybutton",
     "pygments==2.15.0",

pyptt-2.1.0/tests/test_bucket.py

@@ -0,0 +1,52 @@
+"""Integration tests for bucket (water-bucket / mute, moderator-only).
+
+The success-path test mutes `config.TEST_USER` for 1 day on a board the
+bot moderates (`config.MOD_BOARD`, skipped if unset). PTT's
+`UserHasPreviouslyBeenBanned` guard is treated as an acceptable outcome
+on re-runs (the previous run created the ban record and PTT hasn't
+released it yet) — that still proves the API drove the full command
+sequence and reached PTT's own duplicate-detection.
+
+A no-moderator-permission test for `bucket` is intentionally omitted:
+when called with a real user ID, the function's `check_user_exist` step
+is sensitive to PyPtt's session state and returns `NoSuchUser` instead
+of reaching the permission check on this branch. That's a pre-existing
+quirk, not something this test file should pin down.
+"""
+import pytest
+
+import PyPtt
+from tests import config
+
+
+def test_bucket_nonexistent_user_raises(ptt_bots):
+    """`_api_util.check_user_exist` runs early, so any board works for
+    this assertion. Uses a clearly-invalid ID — not a real account."""
+    for ptt_bot in ptt_bots:
+        with pytest.raises(PyPtt.NoSuchUser):
+            ptt_bot.bucket(
+                board='Test',
+                bucket_days=1,
+                reason='will-not-apply',
+                ptt_id='zz_no_such_user_xx',
+            )
+
+
+def test_bucket_on_moderated_board(ptt_bots):
+    """Bucket config.TEST_USER on config.MOD_BOARD. Tolerates 'already
+    banned' on re-runs."""
+    if not config.MOD_BOARD:
+        pytest.skip('MOD_BOARD env var not set')
+    for ptt_bot in ptt_bots:
+        if ptt_bot.host != PyPtt.HOST.PTT1:
+            continue
+        try:
+            ptt_bot.bucket(
+                board=config.MOD_BOARD,
+                bucket_days=1,
+                reason='PyPtt automated bucket test',
+                ptt_id=config.TEST_USER,
+            )
+        except PyPtt.UserHasPreviouslyBeenBanned:
+            # Re-run hit PTT's duplicate guard — also a success signal.
+            pass

pyptt-2.1.0/tests/test_fast_post.py

@@ -0,0 +1,56 @@
+"""Integration test for fast_post_step0 + fast_post_step1.
+
+These two methods (and the bundled `fast_post`) all call
+`api.connect_core.fast_send(...)`, but `connect_core.API` only defines
+`send` — `fast_send` does not exist. So every invocation raises
+`AttributeError: 'API' object has no attribute 'fast_send'`. This is
+pre-existing broken code, not a regression from the parser overhaul.
+
+The test below is marked xfail so it visibly tracks the bug. Once
+`fast_send` is implemented (or the methods are rewired to `send`), the
+test should switch from xfail to pass.
+"""
+import time
+
+import pytest
+
+import PyPtt
+from PyPtt import PostField, PostStatus, NewIndex
+
+
+@pytest.mark.xfail(
+    reason='fast_post_step0/1 call api.connect_core.fast_send which is not '
+           'defined — see _api_post.py:34, :52, :84',
+    strict=True,
+    raises=AttributeError,
+)
+def test_fast_post_round_trip(ptt_bots):
+    """Drive the two-step API and confirm the article reaches Test board."""
+    for ptt_bot in ptt_bots:
+        marker = f'fast-post-marker-{int(time.time())}-{ptt_bot.host}'
+        title = f'PyPtt fast_post test {marker}'
+        content = (
+            'PyPtt fast_post integration test\n'
+            f'{marker}\n'
+        )
+
+        ptt_bot.fast_post_step0(
+            board='Test',
+            title=title,
+            content=content,
+            post_type=1,
+        )
+        ptt_bot.fast_post_step1(sign_file=0)
+        time.sleep(1)
+
+        newest = ptt_bot.get_newest_index(index_type=NewIndex.BOARD, board='Test')
+        for i in range(10):
+            post = ptt_bot.get_post(board='Test', index=newest - i)
+            if post[PostField.post_status] != PostStatus.EXISTS:
+                continue
+            if post[PostField.author].split(' ')[0] != ptt_bot.ptt_id:
+                continue
+            if marker in (post[PostField.content] or ''):
+                assert title in post[PostField.title]
+                return
+        pytest.fail(f'fast_post article {marker!r} not found on Test')

pyptt-2.1.0/tests/test_get_post_list_parser.py

@@ -0,0 +1,176 @@
+"""
+Unit tests for _api_get_post_list line-parsing logic.
+
+Drive get_post_list() with a mocked api that returns a hand-crafted board
+listing screen, then assert the parsed post records line up with the input.
+No network required.
+
+The fixed-column offsets in _api_get_post_list ([:8], [8:10], [10:12], etc.)
+were calibrated to the "1 extra leading space" layout that VT100Parser used
+to produce. After the cursor-tracking overhaul the implicit pad is gone, so
+the function adds an explicit `f' {line}'` prefix to keep the offsets valid.
+These tests pin that contract down.
+"""
+import threading
+from unittest.mock import MagicMock
+
+import pytest
+
+import PyPtt
+from PyPtt import _api_get_post_list, data_type, i18n
+
+# i18n strings are accessed by _api_get_post_list's error paths; init once.
+i18n.init(PyPtt.Language.ENGLISH)
+
+
+def _build_screen(cursor_line: str, after_cursor_lines: list[str] | None = None) -> str:
+    """Compose a 24-row listing screen with header + cursor + (body) + footer.
+
+    Real PTT layout when paginating to start_index:
+      [0]   board title row
+      [1]   hotkey hints
+      [2]   column header (編號, 日 期, ...)
+      [3]   cursor row (the starting index PTT landed on)
+      [4..] subsequent listing rows
+      [23]  footer hints
+
+    `_api_get_post_list` skips lines until it finds the cursor row, then
+    parses it and every later row. So body rows should come *after* the
+    cursor row to be visible to the parser.
+    """
+    header = [
+        '【徵求中】                      Python 程式語言                  看板《Python》',
+        '[←]離開 [→]閱讀 [Ctrl-P]發表文章 [d]刪除 [z]精華區 [i]看板資訊/設定 [h]說明',
+        '   編號    日 期 作  者       文  章  標  題                           人氣:1  ',
+    ]
+    footer = ' 文章選讀  (y)回應(X)推文(^X)轉錄 (=[]<>)相關主題(/?a)找標題/作者 (b)進板畫面  '
+    rows = [cursor_line] + (after_cursor_lines or [])
+    blank_pad = 24 - len(header) - len(rows) - 1
+    if blank_pad > 0:
+        rows.extend([''] * blank_pad)
+    elif blank_pad < 0:
+        rows = rows[:24 - len(header) - 1]
+    return '\n'.join(header + rows + [footer])
+
+
+def _build_api(screen: str, cursor: str = '>', newest: int = 9458):
+    api = MagicMock()
+    api._is_login = True
+    api._thread_id = threading.get_ident()  # bypass one_thread guard
+    api.cursor = cursor
+    api.get_newest_index = MagicMock(return_value=newest)
+    api.connect_core = MagicMock()
+    api.connect_core.send = MagicMock(return_value=0)
+    api.connect_core.get_screen_queue = MagicMock(return_value=[screen])
+    return api
+
+
+# ── basic row layouts ─────────────────────────────────────────────────────────
+
+class TestLineLayouts:
+    """Each row format PTT can render, verified through the public function."""
+
+    def test_simple_single_digit_push(self):
+        # PTT layout: 3 leading spaces + index(4) + space + push(1) + ' ' + date.
+        cursor = '>  9458   2 5/07 fragmentwing □ [問題] openpyxl 全局Font？'
+        api = _build_api(_build_screen(cursor, []))
+        posts = _api_get_post_list.get_post_list(api, 'Python', limit=1)
+        assert len(posts) == 1
+        p = posts[0]
+        assert p[data_type.PostField.index] == 9458
+        assert p[data_type.PostField.push_number] == '2'
+        assert p[data_type.PostField.list_date] == '5/07'
+        assert p[data_type.PostField.author] == 'fragmentwing'
+        assert '[問題] openpyxl 全局Font？' in p[data_type.PostField.title]
+
+    def test_two_digit_push(self):
+        # 10+ pushes occupy 2 chars, eat into the date column gap.
+        cursor = '>  9447  17 7/17 ok8ok8       □ [閒聊] 各位現在用 os.path'
+        api = _build_api(_build_screen(cursor, []), newest=9447)
+        posts = _api_get_post_list.get_post_list(api, 'Python', limit=1)
+        assert posts[0][data_type.PostField.push_number] == '17'
+        assert posts[0][data_type.PostField.list_date] == '7/17'
+
+    def test_plus_prefix_push(self):
+        # '+N' is the "new pushes this session" marker; PTT packs it tight
+        # against a 2-digit-month date with no separating space.
+        cursor = '>  9449 + 610/09 edward740706 □ [問題] OMIA PLUS 自學嗎?'
+        api = _build_api(_build_screen(cursor, []), newest=9449)
+        posts = _api_get_post_list.get_post_list(api, 'Python', limit=1)
+        assert posts[0][data_type.PostField.push_number] == '6'
+        assert posts[0][data_type.PostField.list_date] == '10/09'
+        assert posts[0][data_type.PostField.author] == 'edward740706'
+
+    def test_tilde_prefix_push(self):
+        # '~N' is the alt push prefix that should slice the same as '+N'.
+        cursor = '>  9456 ~ 7 2/24 leolarrel    □ [問題] super() 與 MRO'
+        api = _build_api(_build_screen(cursor, []), newest=9456)
+        posts = _api_get_post_list.get_post_list(api, 'Python', limit=1)
+        assert posts[0][data_type.PostField.push_number] == '7'
+        assert posts[0][data_type.PostField.list_date] == '2/24'
+
+    def test_no_push_count(self):
+        # New post / no pushes yet → push column is blank spaces.
+        cursor = '>  9457     4/17 Sumiremywife □ [問題] 關於爬蟲的一些問題'
+        api = _build_api(_build_screen(cursor, []), newest=9457)
+        posts = _api_get_post_list.get_post_list(api, 'Python', limit=1)
+        assert posts[0][data_type.PostField.push_number] == ''
+        assert posts[0][data_type.PostField.list_date] == '4/17'
+
+    def test_reply_marker_R_in_title(self):
+        # Title section includes the kind marker ('R:', '□', '轉', '鎖').
+        cursor = '>  9440   4 1/15 ddavid       R: [問題] @property 真正的運用是啥'
+        api = _build_api(_build_screen(cursor, []), newest=9440)
+        posts = _api_get_post_list.get_post_list(api, 'Python', limit=1)
+        assert posts[0][data_type.PostField.title].lstrip().startswith('R:')
+
+
+# ── multi-row listing (non-cursor rows align too) ─────────────────────────────
+
+class TestMultiRowListing:
+    """The fix's key claim: non-cursor rows pick up the same leading-space
+    pad as cursor rows, so the column offsets work uniformly."""
+
+    def test_non_cursor_rows_parse_with_same_offsets(self):
+        # Cursor lands on the starting index; later rows follow.
+        cursor = '>  9449   610/09 edward740706 □ [問題] OMIA PLUS 自學嗎?'
+        after = [
+            '   9450 + 110/22 dosiris      □ [問題] 關於正規表示法',
+            '   9451 ~ 110/29 kakar0to     □ [問題] python 3.14 free thread',
+            '   9452 ~ 112/13 iread        □ 執行 Python 程式的六種方法',
+        ]
+        api = _build_api(_build_screen(cursor, after), newest=9452)
+        posts = _api_get_post_list.get_post_list(api, 'Python', limit=4)
+        assert [p[data_type.PostField.index] for p in posts] == [9449, 9450, 9451, 9452]
+        assert [p[data_type.PostField.list_date] for p in posts] == \
+            ['10/09', '10/22', '10/29', '12/13']
+        assert [p[data_type.PostField.author] for p in posts] == \
+            ['edward740706', 'dosiris', 'kakar0to', 'iread']
+        # Non-cursor rows must NOT have the '□' marker bleeding into author.
+        for p in posts[1:]:
+            assert '□' not in p[data_type.PostField.author]
+
+
+# ── regression: master before the fix would mis-slice these ───────────────────
+
+class TestRegressionAgainstMasterBug:
+    """Before the prepend-space fix, non-cursor rows lost one character per
+    column: '10/22' → '0/22', 'dosiris' → 'osiris', and the '□' marker
+    leaked into the author slot. These tests would have failed on master."""
+
+    @pytest.mark.parametrize('cursor_line,want', [
+        ('>  9450 + 110/22 dosiris      □ [問題] 關於正規表示法',
+         {'index': 9450, 'date': '10/22', 'author': 'dosiris', 'push': '1'}),
+        ('>  9447  17 7/17 ok8ok8       □ [閒聊] os.path',
+         {'index': 9447, 'date': '7/17', 'author': 'ok8ok8', 'push': '17'}),
+        ('>  9453 + 212/23 tamacugi     □ [閒聊] 合購股票pythony資料庫',
+         {'index': 9453, 'date': '12/23', 'author': 'tamacugi', 'push': '2'}),
+    ])
+    def test_no_off_by_one_on_packed_layouts(self, cursor_line, want):
+        api = _build_api(_build_screen(cursor_line, []), newest=want['index'])
+        posts = _api_get_post_list.get_post_list(api, 'Python', limit=1)
+        p = posts[0]
+        assert p[data_type.PostField.index] == want['index']
+        assert p[data_type.PostField.list_date] == want['date']
+        assert p[data_type.PostField.author] == want['author']
+        assert p[data_type.PostField.push_number] == want['push']

pyptt-2.1.0/tests/test_mail_send_and_del.py

@@ -0,0 +1,55 @@
+"""Integration test for mail + del_mail.
+
+Round-trip: send a mail to self, locate it in the mailbox, verify its
+content, then delete it. Tests both `mail` (untested elsewhere) and
+`del_mail` (also untested), and exercises `get_mail` against a freshly
+posted item.
+"""
+import time
+
+import PyPtt
+from PyPtt import MailField, NewIndex
+
+
+def test_mail_send_to_self_then_delete(ptt_bots):
+    for ptt_bot in ptt_bots:
+        title = f'PyPtt mail test {int(time.time())}'
+        content = (
+            'PyPtt 自動化測試\n'
+            'mail round-trip integration test\n'
+            f'host={ptt_bot.host}\n'
+        )
+
+        # Mail to self so we don't spam another account.
+        ptt_bot.mail(
+            ptt_id=ptt_bot.ptt_id,
+            title=title,
+            content=content,
+            sign_file=0,
+            backup=True,
+        )
+        time.sleep(2)  # let PTT deliver the mail
+
+        newest = ptt_bot.get_newest_index(index_type=NewIndex.MAIL)
+        assert newest > 0, 'mailbox empty after send'
+
+        # Walk back from the newest mail until we find ours (newest entry
+        # is usually it, but PTT sometimes interleaves system mail).
+        target_index = None
+        for i in range(5):
+            mail_info = ptt_bot.get_mail(newest - i)
+            if mail_info[MailField.title].endswith(title):
+                target_index = newest - i
+                break
+        assert target_index is not None, f'sent mail {title!r} not found in mailbox'
+
+        # Verify the content survived the round-trip.
+        mail_info = ptt_bot.get_mail(target_index)
+        assert 'PyPtt 自動化測試' in mail_info[MailField.content]
+        assert 'mail round-trip integration test' in mail_info[MailField.content]
+
+        # Clean up — del_mail must succeed (raises on failure).
+        # We don't assert on the post-delete newest_index because PyPtt
+        # caches that value (`api._newest_index_data`) and the immediate
+        # next read can return stale data.
+        ptt_bot.del_mail(target_index)