fix(predictor): Fix predict_threadable to properly handle symlinks, especially on NixOS#6115
Merged
Merged
Conversation
for more information, see https://pre-commit.ci
…symlink_predictor
for more information, see https://pre-commit.ci
Member
Author
|
I'm going to do self-release because of tests are here and we want to have this in weekly fix release. |
anki-code
pushed a commit
that referenced
this pull request
Mar 5, 2026
🤖 I have created a release *beep* *boop* --- ## [0.22.5](0.22.4...0.22.5) (2026-03-05) ### Features * Added useful command decorators: `[@lines](https://github.com/lines)`, `[@JSON](https://github.com/json)`, `[@jsonl](https://github.com/jsonl)`, `[@yaml](https://github.com/yaml)` ([#6114](#6114)) ([0f7124e](0f7124e)) ### Bug Fixes * **completion:** Fix os.path.commonprefix deprecation warning in python3.15 ([#6102](#6102)) ([2b5b2bb](2b5b2bb)) * Increase `@$()` stability and now it is also working in case of `$THREAD_SUBPROCS=False` ([#6112](#6112)) ([a3ac036](a3ac036)) * **predictor:** Fix predict_threadable to properly handle symlinks, especially on NixOS ([#6115](#6115)) ([bc01b04](bc01b04)) * **prompt:** Keep original key_bindings in prompt-toolkit shell to easy add new key bindings during session ([#6093](#6093)) ([e9b0aea](e9b0aea)) * replace 12 bare except clauses with except Exception ([#6110](#6110)) ([54b311c](54b311c)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: xonsh-credentials-bot[bot] <97968350+xonsh-credentials-bot[bot]@users.noreply.github.com>
anki-code
added a commit
that referenced
this pull request
Mar 5, 2026
In #5440 we had coreutils support. But in #6115 we introduced additional logic for symlink resolving. This breaks `cat` and `yes` predictions (we have True instead of False). So this PR has additional fixes for logic and tests. ## For community ⬇️ **Please click the 👍 reaction instead of leaving a `+1` or 👍 comment** --------- Co-authored-by: Andy Kipp <123@321.123> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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
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.
Closes #6111
Fixed:
cmdis a script with a non-/usr/bin/envshebang path,@$(cmd)does not capture output, but de-sugaring it does #6111 (comment)cmdis a script with a non-/usr/bin/envshebang path,@$(cmd)does not capture output, but de-sugaring it does #6111 (comment)cc #5440
For community
⬇️ Please click the 👍 reaction instead of leaving a
+1or 👍 comment