Repository navigation
ADFA-4851 | Python-Tools: syntax highlighting and code completion for .py files - #119
Merged
Merged
Conversation
… .py files Python-Tools implements LanguageExtension: it ships the tree-sitter Python grammar (0.23.6, built per ABI from the checksummed npm tarball) with its highlights query, and runs pylsp on the device for completion, pyflakes diagnostics and yapf formatting. pylsp and the Flask template's packages come from a bundled wheelhouse pinned in python-bundle.lock (URL + SHA-256), installed with pip --no-index, so neither needs a network connection. min_ide_version moves to 26.41 and native.code is requested for the grammar library.
Daniel-ADFA
marked this pull request as ready for review
October 5, 2026 12:25
dara-abijo-adfa
requested changes
Oct 5, 2026
jatezzz
reviewed
Oct 5, 2026
jatezzz
reviewed
Oct 5, 2026
- A failed or cancelled requirements install no longer re-arms the automatic sync. toolbarActionsToHide() runs on every toolbar refresh, so an offline project re-ran a 300 s pip install and re-showed the failure toast on nearly every refresh. The sync now runs once per project open; the failure toast points at Install requirements for a retry. - downloadTreeSitterGrammar declares the grammar version, URL and sha256 as task inputs, so changing any of them re-runs the download instead of the task staying up to date on a stale grammar. - The wheelhouse is extracted through <name>.partial and renamed into place, so a copy interrupted by a kill or a full disk never leaves a truncated wheel at the final name for later activations to skip and pip to reject.
jatezzz
approved these changes
Oct 5, 2026
toolbarActionsToHide() can only hide the Gradle actions in ToolbarActionIds.BUILD_HIDEABLE, which do not include Quick Build (ide.editor.build.quickBuild), so it stayed on the toolbar of a Python project. Implement UIExtension and return it from getHiddenToolbarActionIds() while a Python project is open, as Go-Tools does.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ADFA-4851
Python-Tools implements
LanguageExtension: tree-sitter highlighting for.pyandpylspcompletion, diagnostics and formatting, installed offline from a pinned wheelhouse.Depends on appdevforall/CodeOnTheGo#2099. Draft until that merges and the
update-libsbot bringsLanguageExtensionintolibs/plugin-api.jar; CI cannot compile it before then.