Modernise packaging and clean up Python 2 residue (#100)

* Modernise packaging and clean up Python 2 residue

- Replace setup.py with pyproject.toml (hatchling build backend, uv-native)
- Add uv.lock and dependency-groups for dev dependencies (pytest, pytest-cov)
- Drop tox.ini in favour of direct `uv run pytest`
- Replace .travis.yml with GitHub Actions CI (uv, Python 3.10–3.13 matrix)
- Drop Python <3.10 support (3.6–3.9 all EOL)
- Remove (object) base class and super() call on ArabicReshaper
- Switch _ligatures_re from @Property with hidden mutation to @cached_property
- Add -> str type hint and sentinel comment to reshape()
- Add ArabicReshaperConfigurationError(ValueError); replace bare Exception raises
- Add docstring and ImportError to config_for_true_type_font()
- Fix LETTERS -> letters parameter naming in letters.py helper functions
- Add comment explaining the RIAL SIGN regex pattern in ligatures.py
- Remove unused import os from __init__.py; export ArabicReshaperConfigurationError
- Remove debug print() from test_003_reshaping.py

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Complete remaining plan items: coverage and type hints

- Wire pytest-cov into addopts so `uv run pytest` always reports coverage
- Add type hints to auto_config() and config_for_true_type_font()

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Strengthen font config tests; gitignore fonts/

- Add fonts/ to .gitignore so local test fonts are never committed
- Skip tests gracefully when fonts/ is absent or contains no .otf files
- Replace smoke tests with correctness tests: _expected_config() independently
  reads the font's cmap via fonttools and computes the expected config, then
  asserts config_for_true_type_font() == expected — catches any mis-detection
  of glyph support rather than just checking the function doesn't crash
- Add fonttools to dev dependency group so the tests run out of the box

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Replace Travis CI badge with GitHub Actions badge in README

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Bump to 3.0.1: remove obsolete files and update version history

Remove .travis.yml, MANIFEST.in, meta.yaml, and upload-to-anaconda.sh
which are all superseded by the modernised pyproject.toml/hatchling setup
and GitHub Actions CI. Update README version history accordingly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Add GitHub Actions release workflow for PyPI publishing

Uses OIDC trusted publishing (no API token needed). Runs the full test
matrix first, then builds and publishes on success.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Add Python 3.14 to CI and release matrices

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: mpcabd

.github/workflows/ci.yml

@@ -0,0 +1,32 @@
+name: CI
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+    branches: [master]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ["3.10", "3.11", "3.12", "3.13", "3.14"]
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v5
+        with:
+          enable-cache: true
+
+      - name: Set up Python ${{ matrix.python-version }}
+        run: uv python install ${{ matrix.python-version }}
+
+      - name: Install dependencies
+        run: uv sync --group dev
+
+      - name: Run tests
+        run: uv run pytest --cov=arabic_reshaper --cov-report=term-missing

.github/workflows/release.yml

@@ -0,0 +1,33 @@
+name: Release
+
+on:
+  release:
+    types: [published]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: true
+      matrix:
+        python-version: ["3.10", "3.11", "3.12", "3.13", "3.14"]
+    steps:
+      - uses: actions/checkout@v4
+      - uses: astral-sh/setup-uv@v5
+        with:
+          enable-cache: true
+      - run: uv python install ${{ matrix.python-version }}
+      - run: uv sync --group dev
+      - run: uv run pytest
+
+  publish:
+    needs: test
+    runs-on: ubuntu-latest
+    environment: pypi
+    permissions:
+      id-token: write
+    steps:
+      - uses: actions/checkout@v4
+      - uses: astral-sh/setup-uv@v5
+      - run: uv build
+      - uses: pypa/gh-action-pypi-publish@release/v1

.gitignore

@@ -47,3 +47,6 @@ venv/
 # ignore
 .ignore/
 .DS_Store
+
+# local test fonts — not committed
+fonts/

.travis.yml

@@ -1,6 +1,6 @@
 ## Python Arabic Reshaper
 
-[![Build Status](https://app.travis-ci.com/mpcabd/python-arabic-reshaper.svg?branch=master)](https://app.travis-ci.com/mpcabd/python-arabic-reshaper)
+[![CI](https://github.com/mpcabd/python-arabic-reshaper/actions/workflows/ci.yml/badge.svg)](https://github.com/mpcabd/python-arabic-reshaper/actions/workflows/ci.yml)
 
 Reconstruct Arabic sentences to be used in applications that don't support
 Arabic script.
@@ -248,6 +248,14 @@ https://github.com/mpcabd/python-arabic-reshaper/tarball/master
 
 ## Version History
 
+### 3.0.1
+
+* Modernised packaging: migrated from `setuptools` to `hatchling` via `pyproject.toml`
+* Dropped support for Python < 3.10
+* Switched CI from Travis CI to GitHub Actions
+* Added type hints and improved test coverage for font configuration
+* Removed obsolete Anaconda publishing scripts
+
 ### 3.0.0
 * Stop supporting Python 2.7
 * Remove dependency on `future`. See #88.

MANIFEST.in

@@ -1,7 +1,6 @@
-import os
-
 from .arabic_reshaper import reshape, default_reshaper, ArabicReshaper
 from .reshaper_config import (config_for_true_type_font,
+                              ArabicReshaperConfigurationError,
                               ENABLE_NO_LIGATURES,
                               ENABLE_SENTENCES_LIGATURES,
                               ENABLE_WORDS_LIGATURES,

README.md

@@ -9,6 +9,7 @@
 
 import re
 
+from functools import cached_property
 from itertools import repeat
 
 from .ligatures import LIGATURES
@@ -35,7 +36,7 @@
 )
 
 
-class ArabicReshaper(object):
+class ArabicReshaper:
     """
     A class for Arabic reshaper, it allows for fine-tune configuration over the
     API.
@@ -56,7 +57,7 @@ class ArabicReshaper(object):
     """
 
     def __init__(self, configuration=None, configuration_file=None):
-        super(ArabicReshaper, self).__init__()
+        super().__init__()
 
         self.configuration = auto_config(configuration, configuration_file)
         self.language = self.configuration.get('language')
@@ -68,33 +69,25 @@ def __init__(self, configuration=None, configuration_file=None):
         else:
             self.letters = LETTERS_ARABIC
 
-    @property
+    @cached_property
     def _ligatures_re(self):
-        if not hasattr(self, '__ligatures_re'):
-            patterns = []
-            re_group_index_to_ligature_forms = {}
-            index = 0
-            FORMS = 1
-            MATCH = 0
-            for ligature_record in LIGATURES:
-                ligature, replacement = ligature_record
-                if not self.configuration.getboolean(ligature):
-                    continue
-                re_group_index_to_ligature_forms[index] = replacement[FORMS]
-                patterns.append('({})'.format(replacement[MATCH]))
-                index += 1
-            self._re_group_index_to_ligature_forms = (
-                re_group_index_to_ligature_forms
-            )
-            self.__ligatures_re = re.compile('|'.join(patterns), re.UNICODE)
-        return self.__ligatures_re
+        patterns = []
+        self._re_group_index_to_ligature_forms = {}
+        index = 0
+        FORMS = 1
+        MATCH = 0
+        for ligature, replacement in LIGATURES:
+            if not self.configuration.getboolean(ligature):
+                continue
+            self._re_group_index_to_ligature_forms[index] = replacement[FORMS]
+            patterns.append(f'({replacement[MATCH]})')
+            index += 1
+        return re.compile('|'.join(patterns), re.UNICODE)
 
     def _get_ligature_forms_from_re_group_index(self, group_index):
-        if not hasattr(self, '_re_group_index_to_ligature_forms'):
-            return self._ligatures_re
         return self._re_group_index_to_ligature_forms[group_index]
 
-    def reshape(self, text):
+    def reshape(self, text: str) -> str:
         if not text:
             return ''
 
@@ -217,6 +210,8 @@ def reshape(self, text):
                 if not forms[ligature_form]:
                     continue
                 output[a] = (forms[ligature_form], NOT_SUPPORTED)
+                # Pad the replaced positions with empty sentinels so that
+                # Harakat position indices remain aligned with the output list.
                 output[a+1:b] = repeat(('', NOT_SUPPORTED), b - 1 - a)
 
         result = []

arabic_reshaper/__init__.py

@@ -506,22 +506,22 @@
     ZWJ: (ZWJ, ZWJ, ZWJ, ZWJ),
 }
 
-def connects_with_letter_before(letter,LETTERS):
-    if letter not in LETTERS:
+def connects_with_letter_before(letter, letters):
+    if letter not in letters:
         return False
-    forms = LETTERS[letter]
+    forms = letters[letter]
     return forms[FINAL] or forms[MEDIAL]
 
 
-def connects_with_letter_after(letter,LETTERS):
-    if letter not in LETTERS:
+def connects_with_letter_after(letter, letters):
+    if letter not in letters:
         return False
-    forms = LETTERS[letter]
+    forms = letters[letter]
     return forms[INITIAL] or forms[MEDIAL]
 
 
-def connects_with_letters_before_and_after(letter,LETTERS):
-    if letter not in LETTERS:
+def connects_with_letters_before_and_after(letter, letters):
+    if letter not in letters:
         return False
-    forms = LETTERS[letter]
+    forms = letters[letter]
     return forms[MEDIAL]

arabic_reshaper/arabic_reshaper.py

@@ -71,6 +71,7 @@
         '\u0648\u0633\u0644\u0645', ('\uFDF8', '', '', ''),
     )),
     ('RIAL SIGN', (
+        # Regex (not a literal string): matches both Farsi YEH (U+06CC) and Arabic YEH (U+064A).
         '\u0631[\u06CC\u064A]\u0627\u0644', ('\uFDFC', '', '', ''),
     )),
 )