feat: introduce toml-sorter in pre-commit

Makes sure the pyproject.toml is nicely ordered.

Author: nijel

.pre-commit-config.yaml

@@ -31,8 +31,10 @@ repos:
   hooks:
   - id: pretty-format-yaml
     args: [--autofix, --indent, '2']
-  - id: pretty-format-toml
-    args: [--autofix]
+- repo: https://github.com/pappasam/toml-sort
+  rev: v0.24.2
+  hooks:
+  - id: toml-sort-fix
 - repo: https://github.com/abravalheri/validate-pyproject
   rev: v0.24.1
   hooks:

pyproject.toml

@@ -4,51 +4,51 @@ requires = ["setuptools>=78.0.2"]
 
 [dependency-groups]
 dev = [
-  "flake8>=5.0.4",
-  "pytest>=4.5",
-  "responses==0.25.7",
   "coverage>=3.6",
-  "pytest-cov>=2.7.1",
+  "flake8>=5.0.4",
+  "google-auth-stubs==0.3.0",
+  "mypy==1.15.0",
   "pyright==1.1.400",
+  "pytest-cov>=2.7.1",
   "pytest-xdist>=3.6.1",
-  "types-requests==2.32.0.20250328",
+  "pytest>=4.5",
+  "responses==0.25.7",
   "types-defusedxml==0.7.0.20240218",
-  "types-python-jose==3.4.0.20250224",
   "types-oauthlib==3.2.0.20250408",
+  "types-python-jose==3.4.0.20250224",
   "types-requests-oauthlib==2.0.0.20250306",
-  "google-auth-stubs==0.3.0",
-  "mypy==1.15.0"
+  "types-requests==2.32.0.20250328"
 ]
 
 [project]
 authors = [
-  {name = "Matias Aguirre", email = "matiasaguirre@gmail.com"},
+  {email = "matiasaguirre@gmail.com", name = "Matias Aguirre"},
   {email = "michal@weblate.org", name = "Michal Čihař"}
 ]
 classifiers = [
   "Development Status :: 5 - Production/Stable",
   "Environment :: Web Environment",
   "Intended Audience :: Developers",
-  "Programming Language :: Python",
   "Programming Language :: Python :: 3",
   "Programming Language :: Python :: 3.10",
   "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
   "Programming Language :: Python :: 3.13",
   "Programming Language :: Python :: 3.9",
+  "Programming Language :: Python",
   "Topic :: Internet"
 ]
 dependencies = [
-  "PyJWT>=2.7.0",
   "cryptography>=1.4",
   "defusedxml>=0.5.0",
   "oauthlib>=1.0.3",
+  "PyJWT>=2.7.0",
   "python3-openid>=3.0.10",
   "requests-oauthlib>=0.6.1",
   "requests>=2.9.1"
 ]
 description = "Python social authentication made simple."
-keywords = ["openid", "oauth", "saml", "social auth"]
+keywords = ["oauth", "openid", "saml", "social auth"]
 license = "BSD-3-Clause"
 license-files = ["LICENSE"]
 name = "social-auth-core"
@@ -58,10 +58,10 @@ version = "4.5.6"
 
 [project.optional-dependencies]
 all = [
-  "social-auth-core[saml]",
   "social-auth-core[azuread]",
-  "social-auth-core[shopify]",
-  "social-auth-core[google-onetap]"
+  "social-auth-core[google-onetap]",
+  "social-auth-core[saml]",
+  "social-auth-core[shopify]"
 ]
 allpy3 = [
   "social-auth-core[all]"
@@ -72,20 +72,20 @@ azuread = [
 # This is present until pip implements supports for PEP 735
 # see https://github.com/pypa/pip/issues/12963
 dev = [
-  "flake8>=5.0.4",
-  "pytest>=4.5",
-  "responses==0.25.7",
   "coverage>=3.6",
-  "pytest-cov>=2.7.1",
+  "flake8>=5.0.4",
+  "google-auth-stubs==0.3.0",
+  "mypy==1.15.0",
   "pyright==1.1.400",
+  "pytest-cov>=2.7.1",
   "pytest-xdist>=3.6.1",
-  "types-requests==2.32.0.20250328",
+  "pytest>=4.5",
+  "responses==0.25.7",
   "types-defusedxml==0.7.0.20240218",
-  "types-python-jose==3.4.0.20250224",
   "types-oauthlib==3.2.0.20250408",
+  "types-python-jose==3.4.0.20250224",
   "types-requests-oauthlib==2.0.0.20250306",
-  "google-auth-stubs==0.3.0",
-  "mypy==1.15.0"
+  "types-requests==2.32.0.20250328"
 ]
 google-onetap = [
   "google-auth~=2.39.0"
@@ -122,73 +122,73 @@ check_untyped_defs = true
 disallow_untyped_defs = true
 ignore_missing_imports = true
 module = [
-  "openid.*",
+  "google.appengine.*",
   "onelogin.*",
-  "shopify.*",
-  "google.appengine.*"
+  "openid.*",
+  "shopify.*"
 ]
 
 [tool.pyright]
 executionEnvironments = [
-  {root = "social_core/backends/gae.py", reportMissingImports = "none"}
+  {reportMissingImports = "none", root = "social_core/backends/gae.py"}
 ]
 pythonVersion = "3.9"
 reportOptionalMemberAccess = "none"
 reportPossiblyUnboundVariable = "none"
 
 [tool.ruff.lint]
 extend-safe-fixes = [
+  "ANN",
   "D",
-  "TCH",
+  "FA102",
   "FLY",
   "SIM",
-  "ANN",
-  "FA102",
+  "TCH",
   "UP"
 ]
 ignore = [
-  "COM",  # CONFIG: No trailing commas
-  "D203",  # CONFIG: incompatible with D211
-  "D212",  # CONFIG: incompatible with D213
-  "E501",  # WONTFIX: we accept long strings (rest is formatted by ruff)
-  'ISC001',  # CONFIG: formatter
-  "PT",  # CONFIG: Not using pytest
-  "FIX002",  # CONFIG: we use TODO
-  "TD002",  # CONFIG: no detailed TODO documentation is required
-  "TD003",  # CONFIG: no detailed TODO documentation is required
+  "A001",  # TODO: Variable is shadowing a Python builtin
+  "A002",  # TODO: Function argument is shadowing a Python builtin
   "ANN",  # TODO: Missing type annotations
-  "D",  # TODO: Missing documentation
-  "B904",  # TODO: use raise from
-  "PLR2004",  # TODO: Magic value used in comparison
-  "PLW2901",  # TODO: loop variable overwritten by assignment target
-  "N",  # TODO: Naming issues
-  "RUF012",  # TODO: Type annotations
   "ARG001",  # TODO: Unused function argument (mostly for API compatibility)
   "ARG002",  # TODO: Unused method argument (mostly for API compatibility)
   "ARG003",  # TODO: Unused class method argument
   "ARG005",  # TODO: Unused lambda argument
-  "TID252",  # TODO: Prefer absolute imports over relative imports from parent modules
-  "FBT",  # TODO: Boolean in function definition
-  "S105",  # TODO: Possible hardcoded password assigned
   "B018",  # TODO: Found useless expression.
-  "A001",  # TODO: Variable is shadowing a Python builtin
-  "A002",  # TODO: Function argument is shadowing a Python builtin
-  "ERA001",  # TODO: Found commented-out code
+  "B026",  # TODO: Star-arg unpacking after a keyword argument is strongly discouraged
+  "B904",  # TODO: use raise from
+  "BLE001",  # TODO: Do not catch blind exception: `Exception`
+  "COM",  # CONFIG: No trailing commas
+  "D",  # TODO: Missing documentation
+  "D203",  # CONFIG: incompatible with D211
+  "D212",  # CONFIG: incompatible with D213
+  "DTZ001",  # TODO: `datetime.datetime()` called without a `tzinfo` argument
+  "DTZ005",  # TODO: `datetime.datetime.now()` called without a `tz` argument
+  "E501",  # WONTFIX: we accept long strings (rest is formatted by ruff)
   "EM101",  # TODO: Exception must not use a string literal, assign to variable first
   "EM102",  # TODO: Exception must not use an f-string literal, assign to variable first
-  "TRY003",  # TODO: Avoid specifying long messages outside the exception class
+  "ERA001",  # TODO: Found commented-out code
+  "FBT",  # TODO: Boolean in function definition
+  "FIX002",  # CONFIG: we use TODO
+  "ISC003",  # TODO: Explicitly concatenated string should be implicitly concatenated
+  "N",  # TODO: Naming issues
+  "PERF203",  # WONTFIX: This rule is only enforced for Python versions prior to 3.11
+  "PLR2004",  # TODO: Magic value used in comparison
+  "PLW0603",  # TODO: Using the global statement to update `BACKENDSCACHE` is discouraged
+  "PLW2901",  # TODO: loop variable overwritten by assignment target
+  "PT",  # CONFIG: Not using pytest
+  "RUF012",  # TODO: Type annotations
   "S101",  # TODO: Use of `assert` detected
-  "DTZ001",  # TODO: `datetime.datetime()` called without a `tzinfo` argument
-  "DTZ005",  # TODO: `datetime.datetime.now()` called without a `tz` argument
-  "B026",  # TODO: Star-arg unpacking after a keyword argument is strongly discouraged
+  "S105",  # TODO: Possible hardcoded password assigned
+  "S110",  # TODO: `try`-`except`-`pass` detected, consider logging the exception
   "S301",  # TODO: `pickle` and modules that wrap it can be unsafe when used to deserialize untrusted data, possible security issue
   "S318",  # TODO: Using `xml` to parse untrusted data is known to be vulnerable to XML attacks; use `defusedxml` equivalents
-  "PERF203",  # WONTFIX: This rule is only enforced for Python versions prior to 3.11
-  "ISC003",  # TODO: Explicitly concatenated string should be implicitly concatenated
-  "PLW0603",  # TODO: Using the global statement to update `BACKENDSCACHE` is discouraged
+  "TD002",  # CONFIG: no detailed TODO documentation is required
+  "TD003",  # CONFIG: no detailed TODO documentation is required
+  "TID252",  # TODO: Prefer absolute imports over relative imports from parent modules
+  "TRY003",  # TODO: Avoid specifying long messages outside the exception class
   "TRY301",  # TODO: Abstract `raise` to an inner function
-  "BLE001",  # TODO: Do not catch blind exception: `Exception`
-  "S110"  # TODO: `try`-`except`-`pass` detected, consider logging the exception
+  'ISC001'  # CONFIG: formatter
 ]
 select = ["ALL"]
 
@@ -205,3 +205,10 @@ max-branches = 15
 
 [tool.setuptools]
 packages = ["social_core"]
+
+[tool.tomlsort]
+ignore_case = true
+sort_inline_arrays = true
+sort_inline_tables = true
+sort_table_keys = true
+spaces_before_inline_comment = 2