Remove sha mapping which isn't working well

Author: adamtheturtle

.github/workflows/publish-notion.yml

@@ -34,7 +34,6 @@ jobs:
             --parent-type "page" \
             --file "./build-sample/index.json" \
             --title "Sphinx-Notionbuilder Sample" \
-            --sha-mapping "./sample/notion-sha-mapping.json" \
             --icon "🐍"
         env:
           NOTION_TOKEN: ${{ secrets.NOTION_TOKEN }}

README.rst

@@ -299,7 +299,6 @@ Arguments:
 - ``--parent-id``: The ID of the parent page or database in Notion (must be shared with your integration)
 - ``--parent-type``: "page" or "database"
 - ``--title``: Title for the new page in Notion
-- ``--sha-mapping``: Optional JSON file mapping file SHAs to Notion block IDs for efficient file re-uploads (use one file per document). The command will update this file.
 
 The command will create a new page if one with the given title doesn't exist, or update the existing page if one with the given title already exists.
 

pyproject.toml

@@ -35,7 +35,6 @@ dependencies = [
     "beartype>=0.21.0",
     "click>=8.0.0",
     "docutils>=0.21",
-    "notion-client>=2.5.0",
     "sphinx>=8.2.3",
     "sphinx-immaterial>=0.13.7",
     "sphinx-simplepdf>=1.6.0",

sample/notion-sha-mapping.json

@@ -16,7 +16,6 @@
 
 import click
 from beartype import beartype
-from notion_client.errors import APIResponseError
 from ultimate_notion import Emoji, Session
 from ultimate_notion.blocks import PDF as UnoPDF  # noqa: N811
 from ultimate_notion.blocks import Audio as UnoAudio
@@ -46,40 +45,11 @@ def _calculate_file_sha(*, file_path: Path) -> str:
     return sha256_hash.hexdigest()
 
 
-@beartype
-def _clean_deleted_blocks_from_mapping(
-    *,
-    sha_to_block_id: dict[str, str],
-    session: Session,
-) -> dict[str, str]:
-    """Remove deleted blocks from ``SHA`` mapping.
-
-    Returns a new dictionary with only existing blocks.
-    """
-    cleaned_mapping = sha_to_block_id.copy()
-    deleted_block_shas: set[str] = set()
-
-    for sha, block_id_str in sha_to_block_id.items():
-        block_id = UUID(hex=block_id_str)
-        try:
-            session.api.blocks.retrieve(block=block_id)
-        except APIResponseError:
-            deleted_block_shas.add(sha)
-            msg = f"Block {block_id} does not exist, removing from SHA mapping"
-            click.echo(message=msg)
-
-    for deleted_block_sha in deleted_block_shas:
-        del cleaned_mapping[deleted_block_sha]
-
-    return cleaned_mapping
-
-
 @beartype
 def _find_last_matching_block_index(
     *,
     existing_blocks: list[Block] | tuple[Block, ...],
     local_blocks: list[Block],
-    sha_to_block_id: dict[str, str],
 ) -> int | None:
     """Find the last index where existing blocks match local blocks.
 
@@ -92,7 +62,6 @@ def _find_last_matching_block_index(
             _is_existing_equivalent(
                 existing_page_block=existing_page_block,
                 local_block=local_blocks[index],
-                sha_to_block_id=sha_to_block_id,
             )
         ):
             last_matching_index = index
@@ -215,20 +184,6 @@ class _ParentType(Enum):
     help="Icon of the page",
     required=False,
 )
-@click.option(
-    "--sha-mapping",
-    help=(
-        "JSON file mapping file SHAs to Notion block IDs "
-        "(use one file per document)"
-    ),
-    required=False,
-    type=click.Path(
-        exists=True,
-        path_type=Path,
-        file_okay=True,
-        dir_okay=False,
-    ),
-)
 @beartype
 def main(
     *,
@@ -237,23 +192,12 @@ def main(
     parent_type: _ParentType,
     title: str,
     icon: str | None = None,
-    sha_mapping: Path | None = None,
 ) -> None:
     """
     Upload documentation to Notion.
     """
     session = Session()
 
-    sha_mapping_content = (
-        sha_mapping.read_text(encoding="utf-8") if sha_mapping else "{}"
-    )
-    sha_to_block_id: dict[str, str] = dict(json.loads(s=sha_mapping_content))
-
-    sha_to_block_id = _clean_deleted_blocks_from_mapping(
-        sha_to_block_id=sha_to_block_id,
-        session=session,
-    )
-
     blocks = json.loads(s=file.read_text(encoding="utf-8"))
 
     parent: Page | Database
@@ -291,7 +235,6 @@ def main(
     last_matching_index = _find_last_matching_block_index(
         existing_blocks=page.children,
         local_blocks=block_objs,
-        sha_to_block_id=sha_to_block_id,
     )
 
     click.echo(
@@ -310,36 +253,4 @@ def main(
     ]
     page.append(blocks=block_objs_to_upload)
 
-    if sha_mapping:
-        for uploaded_block_index, uploaded_block in enumerate(
-            iterable=block_objs_to_upload
-        ):
-            if isinstance(uploaded_block, _FILE_BLOCK_TYPES):
-                pre_uploaded_block = block_objs[
-                    delete_start_index + uploaded_block_index
-                ]
-                assert isinstance(pre_uploaded_block, _FILE_BLOCK_TYPES)
-                parsed = urlparse(url=pre_uploaded_block.url)
-                if parsed.scheme == "file":
-                    # Ignore ``mypy`` error as the keyword arguments are
-                    # different across Python versions and platforms.
-                    file_path = Path(url2pathname(parsed.path))  # type: ignore[misc]
-                    file_sha = _calculate_file_sha(file_path=file_path)
-                    sha_to_block_id[file_sha] = str(object=uploaded_block.id)
-
-                    sha_mapping.write_text(
-                        data=json.dumps(
-                            obj=sha_to_block_id, indent=2, sort_keys=True
-                        )
-                        + "\n",
-                        encoding="utf-8",
-                    )
-
-                    click.echo(
-                        message=(
-                            f"Updated SHA mapping for {file_path.name}:"
-                            f"{uploaded_block.id}"
-                        )
-                    )
-
     click.echo(message=f"Updated existing page: '{title}' ({page.url})")