From c1b6a93624d8bc83af927b5229c23bc85e72aae9 Mon Sep 17 00:00:00 2001 From: luojiyin Date: Mon, 19 Jan 2026 12:57:47 +0800 Subject: [PATCH] fix: use pop() instead of del in process_none_page_numbers Using del item_copy['page'] raises KeyError if 'page' key doesn't exist. Use pop('page', None) for safe removal without exception. --- pageindex/page_index.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pageindex/page_index.py b/pageindex/page_index.py index 882fb5dea..89ff1b919 100644 --- a/pageindex/page_index.py +++ b/pageindex/page_index.py @@ -674,11 +674,11 @@ def process_none_page_numbers(toc_items, page_list, start_index=1, model=None): continue item_copy = copy.deepcopy(item) - del item_copy['page'] + item_copy.pop('page', None) result = add_page_number_to_toc(page_contents, item_copy, model) if isinstance(result[0]['physical_index'], str) and result[0]['physical_index'].startswith('').strip()) - del item['page'] + item.pop('page', None) return toc_items