fujin/thefuck
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add bashlex dependency with import fallback

Browse Source 
bashlex (GPL-3+) powers upcoming structural command analysis: parsing
pipelines into segments with word offsets so a typo anywhere in a
command can be fixed as confidently as one at the start. Because the
license is incompatible with distributing the MIT-licensed app, the
dependency is import-guarded: thefuck/shell_ast.py exposes
AST_AVAILABLE, which is False when bashlex is missing or under python 2
(the py2.7 CI requirements.txt deliberately omits it), so all new
behavior must gate on it and the fork stays personal-use only.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
This commit is contained in:
Alexander
2026-09-13 20:31:51 +02:00
parent 78f4636a61
commit f7d3b10288
4 changed files with 105 additions and 44 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pyproject.toml
+1
View File
@@ -10,6 +10,7 @@ dependencies = [
"six",
"decorator",
"pyte",
"bashlex==0.18",
]
[project.scripts]
tests/test_shell_ast_guard.py
+35
View File
@@ -0,0 +1,35 @@
import os
import subprocess
import sys
def _run_with_stub_bashlex(tmpdir):
"""Runs a fresh interpreter with a broken bashlex shadowing the real one."""
tmpdir.join('bashlex.py').write("raise ImportError('stub')\n")
repo_root = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
env = os.environ.copy()
old_path = env.get('PYTHONPATH', '')
env['PYTHONPATH'] = os.pathsep.join(
[str(tmpdir), repo_root] + ([old_path] if old_path else []))
code = ("import thefuck.entrypoints.fix_command; "
"from thefuck.shell_ast import AST_AVAILABLE, bashlex; "
"print(AST_AVAILABLE); print(bashlex)")
return subprocess.check_output([sys.executable, '-c', code], env=env)
def test_guard_off_when_bashlex_unimportable(tmpdir):
"""With bashlex broken, the module still imports with the guard off."""
assert _run_with_stub_bashlex(tmpdir).split() == [b'False', b'None']
def test_ast_available_reflects_bashlex():
"""The guard is on exactly when bashlex imports on python 3."""
try:
import bashlex # noqa: F401
bashlex_importable = True
except ImportError:
bashlex_importable = False
from thefuck.shell_ast import AST_AVAILABLE
assert AST_AVAILABLE == (bashlex_importable and sys.version_info[0] == 3)
thefuck/shell_ast.py
+14
View File
@@ -0,0 +1,14 @@
"""Structural analysis of shell commands, backed by bashlex.
Personal fork only: bashlex is GPL-3+ licensed while thefuck is MIT,
so this combination must not be distributed.
"""
import six
try:
import bashlex
except ImportError:
bashlex = None
AST_AVAILABLE = bashlex is not None and six.PY3
uv.lock
Generated
+55 -44
View File
@@ -9,6 +9,15 @@ resolution-markers = [
"python_full_version < '3.8'",
]
[[package]]
name = "bashlex"
version = "0.18"
source = { registry = "https://pypi.org/simple" }
sdist = { url = "https://files.pythonhosted.org/packages/76/60/aae0bb54f9af5e0128ba90eb83d8d0d506ee8f0475c4fdda3deeda20b1d2/bashlex-0.18.tar.gz", hash = "sha256:5bb03a01c6d5676338c36fd1028009c8ad07e7d61d8a1ce3f513b7fff52796ee", size = 68742, upload-time = "2023-01-18T15:21:26.402Z" }
wheels = [
{ url = "https://files.pythonhosted.org/packages/f4/be/6985abb1011fda8a523cfe21ed9629e397d6e06fb5bae99750402b25c95b/bashlex-0.18-py2.py3-none-any.whl", hash = "sha256:91d73a23a3e51711919c1c899083890cdecffc91d8c088942725ac13e9dcfffa", size = 69539, upload-time = "2023-01-18T15:21:24.167Z" },
]
[[package]]
name = "colorama"
version = "0.4.6"
@@ -50,9 +59,9 @@ name = "exceptiongroup"
version = "1.3.1"
source = { registry = "https://pypi.org/simple" }
dependencies = [
{ name = "typing-extensions", version = "4.7.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "typing-extensions", version = "4.7.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8' or python_full_version >= '3.11'" },
{ name = "typing-extensions", version = "4.13.2", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version == '3.8.*'" },
{ name = "typing-extensions", version = "4.15.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9' and python_full_version < '3.13'" },
{ name = "typing-extensions", version = "4.15.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9' and python_full_version < '3.11'" },
]
sdist = { url = "https://files.pythonhosted.org/packages/50/79/66800aadf48771f6b62f7eb014e352e5d06856655206165d775e675a02c9/exceptiongroup-1.3.1.tar.gz", hash = "sha256:8b412432c6055b0b7d14c310000ae93352ed6754f70fa8f7c34141f91c4e3219", size = 30371, upload-time = "2025-11-21T23:01:54.787Z" }
wheels = [
@@ -77,10 +86,10 @@ resolution-markers = [
"python_full_version < '3.8'",
]
dependencies = [
{ name = "importlib-metadata", marker = "python_full_version < '3.8'" },
{ name = "mccabe", marker = "python_full_version < '3.8.1'" },
{ name = "pycodestyle", version = "2.9.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8.1'" },
{ name = "pyflakes", version = "2.5.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8.1'" },
{ name = "importlib-metadata", marker = "python_full_version < '3.8' or python_full_version >= '3.8.1'" },
{ name = "mccabe" },
{ name = "pycodestyle", version = "2.9.1", source = { registry = "https://pypi.org/simple" } },
{ name = "pyflakes", version = "2.5.0", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/ad/00/9808c62b2d529cefc69ce4e4a1ea42c0f855effa55817b7327ec5b75e60a/flake8-5.0.4.tar.gz", hash = "sha256:6fbe320aad8d6b95cec8b8e47bc933004678dc63095be98528b7bdd2a9f510db", size = 145862, upload-time = "2022-08-03T23:21:27.108Z" }
wheels = [
@@ -95,9 +104,9 @@ resolution-markers = [
"python_full_version >= '3.8.1' and python_full_version < '3.9'",
]
dependencies = [
{ name = "mccabe", marker = "python_full_version >= '3.8.1' and python_full_version < '3.9'" },
{ name = "pycodestyle", version = "2.12.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.8.1' and python_full_version < '3.9'" },
{ name = "pyflakes", version = "3.2.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.8.1' and python_full_version < '3.9'" },
{ name = "mccabe" },
{ name = "pycodestyle", version = "2.12.1", source = { registry = "https://pypi.org/simple" } },
{ name = "pyflakes", version = "3.2.0", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/58/16/3f2a0bb700ad65ac9663262905a025917c020a3f92f014d2ba8964b4602c/flake8-7.1.2.tar.gz", hash = "sha256:c586ffd0b41540951ae41af572e6790dbd49fc12b3aa2541685d253d9bd504bd", size = 48119, upload-time = "2025-02-16T18:45:44.296Z" }
wheels = [
@@ -113,9 +122,9 @@ resolution-markers = [
"python_full_version == '3.9.*'",
]
dependencies = [
{ name = "mccabe", marker = "python_full_version >= '3.9'" },
{ name = "pycodestyle", version = "2.14.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9'" },
{ name = "pyflakes", version = "3.4.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9'" },
{ name = "mccabe" },
{ name = "pycodestyle", version = "2.14.0", source = { registry = "https://pypi.org/simple" } },
{ name = "pyflakes", version = "3.4.0", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/9b/af/fbfe3c4b5a657d79e5c47a2827a362f9e1b763336a52f926126aa6dc7123/flake8-7.3.0.tar.gz", hash = "sha256:fe044858146b9fc69b551a4b490d69cf960fcb78ad1edcb84e7fbb1b4a8e3872", size = 48326, upload-time = "2025-06-20T19:31:35.838Z" }
wheels = [
@@ -127,8 +136,8 @@ name = "importlib-metadata"
version = "4.2.0"
source = { registry = "https://pypi.org/simple" }
dependencies = [
{ name = "typing-extensions", version = "4.7.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "zipp", marker = "python_full_version < '3.8'" },
{ name = "typing-extensions", version = "4.7.1", source = { registry = "https://pypi.org/simple" } },
{ name = "zipp" },
]
sdist = { url = "https://files.pythonhosted.org/packages/c7/7c/126a8686399ebe256b5e4343ea80b6f2ee91549969da2eef0bb2891b8d24/importlib_metadata-4.2.0.tar.gz", hash = "sha256:b7e52a1f8dec14a75ea73e0891f3060099ca1d8e6a462a4dff11c3e119ea1b31", size = 38311, upload-time = "2021-05-26T17:37:50.048Z" }
wheels = [
@@ -238,7 +247,7 @@ resolution-markers = [
"python_full_version < '3.8'",
]
dependencies = [
{ name = "importlib-metadata", marker = "python_full_version < '3.8'" },
{ name = "importlib-metadata" },
]
sdist = { url = "https://files.pythonhosted.org/packages/8a/42/8f2833655a29c4e9cb52ee8a2be04ceac61bcff4a680fb338cbd3d1e322d/pluggy-1.2.0.tar.gz", hash = "sha256:d12f0c4b579b15f5e054301bb226ee85eeeba08ffec228092f8defbaa3a4c4b3", size = 61613, upload-time = "2023-06-21T09:12:28.745Z" }
wheels = [
@@ -410,7 +419,7 @@ resolution-markers = [
"python_full_version < '3.8'",
]
dependencies = [
{ name = "wcwidth", version = "0.2.14", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "wcwidth", version = "0.2.14", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/9f/60/442cdc1cba83710770672ef61e186be8746f419a12b2c84ba36e9a96276d/pyte-0.8.1.tar.gz", hash = "sha256:b9bfd1b781759e7572a6e553c010cc93eef58a19d8d1590446d84c19b1b097b0", size = 51657, upload-time = "2022-06-07T19:19:26.269Z" }
wheels = [
@@ -428,7 +437,7 @@ resolution-markers = [
"python_full_version >= '3.8' and python_full_version < '3.8.1'",
]
dependencies = [
{ name = "wcwidth", version = "0.6.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.8'" },
{ name = "wcwidth", version = "0.6.0", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/ab/ab/b599762933eba04de7dc5b31ae083112a6c9a9db15b01d3109ad797559d9/pyte-0.8.2.tar.gz", hash = "sha256:5af970e843fa96a97149d64e170c984721f20e52227a2f57f0a54207f08f083f", size = 92301, upload-time = "2023-11-12T09:33:43.217Z" }
wheels = [
@@ -443,13 +452,13 @@ resolution-markers = [
"python_full_version < '3.8'",
]
dependencies = [
{ name = "colorama", marker = "python_full_version < '3.8' and sys_platform == 'win32'" },
{ name = "exceptiongroup", marker = "python_full_version < '3.8'" },
{ name = "importlib-metadata", marker = "python_full_version < '3.8'" },
{ name = "iniconfig", version = "2.0.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "packaging", version = "24.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "pluggy", version = "1.2.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "tomli", version = "2.0.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "colorama", marker = "sys_platform == 'win32'" },
{ name = "exceptiongroup" },
{ name = "importlib-metadata" },
{ name = "iniconfig", version = "2.0.0", source = { registry = "https://pypi.org/simple" } },
{ name = "packaging", version = "24.0", source = { registry = "https://pypi.org/simple" } },
{ name = "pluggy", version = "1.2.0", source = { registry = "https://pypi.org/simple" } },
{ name = "tomli", version = "2.0.1", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/80/1f/9d8e98e4133ffb16c90f3b405c43e38d3abb715bb5d7a63a5a684f7e46a3/pytest-7.4.4.tar.gz", hash = "sha256:2cf0005922c6ace4a3e2ec8b4080eb0d9753fdc93107415332f50ce9e7994280", size = 1357116, upload-time = "2023-12-31T12:00:18.035Z" }
wheels = [
@@ -465,12 +474,12 @@ resolution-markers = [
"python_full_version >= '3.8' and python_full_version < '3.8.1'",
]
dependencies = [
{ name = "colorama", marker = "python_full_version == '3.8.*' and sys_platform == 'win32'" },
{ name = "exceptiongroup", marker = "python_full_version == '3.8.*'" },
{ name = "iniconfig", version = "2.1.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version == '3.8.*'" },
{ name = "packaging", version = "26.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version == '3.8.*'" },
{ name = "pluggy", version = "1.5.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version == '3.8.*'" },
{ name = "tomli", version = "2.4.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version == '3.8.*'" },
{ name = "colorama", marker = "sys_platform == 'win32'" },
{ name = "exceptiongroup" },
{ name = "iniconfig", version = "2.1.0", source = { registry = "https://pypi.org/simple" } },
{ name = "packaging", version = "26.0", source = { registry = "https://pypi.org/simple" } },
{ name = "pluggy", version = "1.5.0", source = { registry = "https://pypi.org/simple" } },
{ name = "tomli", version = "2.4.1", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/ae/3c/c9d525a414d506893f0cd8a8d0de7706446213181570cdbd766691164e40/pytest-8.3.5.tar.gz", hash = "sha256:f4efe70cc14e511565ac476b57c279e12a855b11f48f212af1080ef2263d3845", size = 1450891, upload-time = "2025-03-02T12:54:54.503Z" }
wheels = [
@@ -486,14 +495,14 @@ resolution-markers = [
"python_full_version == '3.9.*'",
]
dependencies = [
{ name = "colorama", marker = "python_full_version >= '3.9' and sys_platform == 'win32'" },
{ name = "exceptiongroup", marker = "python_full_version >= '3.9' and python_full_version < '3.11'" },
{ name = "iniconfig", version = "2.1.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version == '3.9.*'" },
{ name = "colorama", marker = "sys_platform == 'win32'" },
{ name = "exceptiongroup", marker = "python_full_version < '3.11'" },
{ name = "iniconfig", version = "2.1.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.10'" },
{ name = "iniconfig", version = "2.3.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.10'" },
{ name = "packaging", version = "26.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9'" },
{ name = "pluggy", version = "1.6.0", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9'" },
{ name = "pygments", marker = "python_full_version >= '3.9'" },
{ name = "tomli", version = "2.4.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9' and python_full_version < '3.11'" },
{ name = "packaging", version = "26.0", source = { registry = "https://pypi.org/simple" } },
{ name = "pluggy", version = "1.6.0", source = { registry = "https://pypi.org/simple" } },
{ name = "pygments" },
{ name = "tomli", version = "2.4.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.11'" },
]
sdist = { url = "https://files.pythonhosted.org/packages/a3/5c/00a0e072241553e1a7496d638deababa67c5058571567b92a7eaa258397c/pytest-8.4.2.tar.gz", hash = "sha256:86c0d0b93306b961d58d62a4db4879f27fe25513d4b969df351abdddb3c30e01", size = 1519618, upload-time = "2025-09-04T14:34:22.711Z" }
wheels = [
@@ -510,8 +519,8 @@ resolution-markers = [
"python_full_version < '3.8'",
]
dependencies = [
{ name = "py-cpuinfo", marker = "python_full_version < '3.9'" },
{ name = "pytest", version = "7.4.4", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "py-cpuinfo" },
{ name = "pytest", version = "7.4.4", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version != '3.8.*'" },
{ name = "pytest", version = "8.3.5", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version == '3.8.*'" },
]
sdist = { url = "https://files.pythonhosted.org/packages/28/08/e6b0067efa9a1f2a1eb3043ecd8a0c48bfeb60d3255006dcc829d72d5da2/pytest-benchmark-4.0.0.tar.gz", hash = "sha256:fb0785b83efe599a6a956361c0691ae1dbb5318018561af10f3e915caa0048d1", size = 334641, upload-time = "2022-10-25T21:21:55.686Z" }
@@ -528,8 +537,8 @@ resolution-markers = [
"python_full_version == '3.9.*'",
]
dependencies = [
{ name = "py-cpuinfo", marker = "python_full_version >= '3.9'" },
{ name = "pytest", version = "8.4.2", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9'" },
{ name = "py-cpuinfo" },
{ name = "pytest", version = "8.4.2", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/24/34/9f732b76456d64faffbef6232f1f9dbec7a7c4999ff46282fa418bd1af66/pytest_benchmark-5.2.3.tar.gz", hash = "sha256:deb7317998a23c650fd4ff76e1230066a76cb45dcece0aca5607143c619e7779", size = 341340, upload-time = "2025-11-09T18:48:43.215Z" }
wheels = [
@@ -544,7 +553,7 @@ resolution-markers = [
"python_full_version < '3.8'",
]
dependencies = [
{ name = "pytest", version = "7.4.4", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "pytest", version = "7.4.4", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/d8/2d/b3a811ec4fa24190a9ec5013e23c89421a7916167c6240c31fdc445f850c/pytest-mock-3.11.1.tar.gz", hash = "sha256:7f6b125602ac6d743e523ae0bfa71e1a697a2f5534064528c6ff84c2f7c2fc7f", size = 31251, upload-time = "2023-06-15T23:58:07.082Z" }
wheels = [
@@ -560,7 +569,7 @@ resolution-markers = [
"python_full_version >= '3.8' and python_full_version < '3.8.1'",
]
dependencies = [
{ name = "pytest", version = "8.3.5", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version == '3.8.*'" },
{ name = "pytest", version = "8.3.5", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/71/28/67172c96ba684058a4d24ffe144d64783d2a270d0af0d9e792737bddc75c/pytest_mock-3.14.1.tar.gz", hash = "sha256:159e9edac4c451ce77a5cdb9fc5d1100708d2dd4ba3c3df572f14097351af80e", size = 33241, upload-time = "2025-05-26T13:58:45.167Z" }
wheels = [
@@ -576,7 +585,7 @@ resolution-markers = [
"python_full_version == '3.9.*'",
]
dependencies = [
{ name = "pytest", version = "8.4.2", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.9'" },
{ name = "pytest", version = "8.4.2", source = { registry = "https://pypi.org/simple" } },
]
sdist = { url = "https://files.pythonhosted.org/packages/68/14/eb014d26be205d38ad5ad20d9a80f7d201472e08167f0bb4361e251084a9/pytest_mock-3.15.1.tar.gz", hash = "sha256:1849a238f6f396da19762269de72cb1814ab44416fa73a8686deac10b0d87a0f", size = 34036, upload-time = "2025-09-16T16:37:27.081Z" }
wheels = [
@@ -624,6 +633,7 @@ name = "thefuck"
version = "3.32"
source = { editable = "." }
dependencies = [
{ name = "bashlex" },
{ name = "colorama" },
{ name = "decorator", version = "5.1.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version < '3.8'" },
{ name = "decorator", version = "5.2.1", source = { registry = "https://pypi.org/simple" }, marker = "python_full_version >= '3.8'" },
@@ -655,6 +665,7 @@ dev = [
[package.metadata]
requires-dist = [
{ name = "bashlex", specifier = "==0.18" },
{ name = "colorama" },
{ name = "decorator" },
{ name = "psutil" },