Repository navigation
Windows: short project name for the app's chat folders, and a path budget - #2082
Draft
Brian Krabach (bkrabach) wants to merge 5 commits into
Draft
Brian Krabach (bkrabach) wants to merge 5 commits into
Brian Krabach (bkrabach) wants to merge 5 commits into
Conversation
…dget On Windows the app's own chat folders (<any>\chats\<UUID>\files) get a 21-character project name derived from the chat UUID. The shared CLI name of such a folder repeats the installation path inside every session path, which exceeds the Windows path limit when long paths are off (the default). A chat that already has history under the CLI name keeps it; nothing is moved. Folders a person chose and every other platform keep the CLI name. Adds the path budget as a test, and a native Windows CI case that turns long paths off on its runner and runs the real stores under the longest supported installation folder.
…pt Python's own length refusal The native canary showed that the runner honours LongPathsEnabled=0 for a new process: 259 characters accepted, 260 refused in the same folder. Python's open reports that refusal as errno 2 without a Windows code.
… open A chat with history under the CLI name opens where it is. When both names hold history for one chat ID (only after an update rollback) the chat still opens under the short name and the other folder stays listed read-only.
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.
Draft while the native Windows job is brought to green. The full description follows before this leaves draft.
Part of the Windows 11 out-of-the-box program (path length, parts A and C).