Sync CE: drop the UOS desktop line and refresh from full upstream - #121
Closed
Luhaozhu wants to merge 1 commit into
Closed
Sync CE: drop the UOS desktop line and refresh from full upstream#121Luhaozhu wants to merge 1 commit into
Luhaozhu wants to merge 1 commit into
Conversation
Remove `desktop-uos/` from the Community Edition tree. The UOS 1070 line (Electron on aarch64/glibc 2.28, built for a domestic-stack delivery target) was never meant to ship with CE, but its manifest exclusion only ever existed on an unmerged branch while the code itself reached the mainline through a different commit. The CE generator therefore treated it as an ordinary directory and copied all 30 files here. The exclusion now lives on the mainline manifest, so this sync deletes them. The Tauri 2 `desktop/` line is unaffected: `desktop-uos` depended on `desktop/` in one direction only, so removing it leaves nothing dangling. This sync also carries the upstream work accumulated since the last one: - force connector invocation when the user explicitly selects one - retire the home-screen shortcut row; first-run guidance is now carried entirely by suggested questions - stop the streaming view from snapping back to the bottom while the user is scrolling - stop interrupted plans from being resurrected after a run is cancelled - keep automation reasoning out of the visible message body - scope site editing to the active project - keep the automation page subtitle on a single line
Contributor
Author
|
Closing in favour of #120, which lands a byte-for-byte identical tree against the same base. The manifest-level fix that keeps |
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
1 participant
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.
Summary
desktop-uos/(30 files) from the Community Edition tree — the UOS 1070 line is a domestic-stack delivery target that was never meant to ship with CEWhy
desktop-uoswas hereThe exclusion for
desktop-uos/**was written when the UOS line was opened, but that commit only ever existed on an unmerged branch. The directory itself reached the mainline through a different commit that did not carry the manifest change, soscripts/build_ce.pytreated it as an ordinary directory and copied it into this repository. The exclusion now lives on the mainline manifest and the files are deleted here.The Tauri 2
desktop/line is unaffected.desktop-uosdepended ondesktop/in one direction only, so removing it leaves no dangling references — confirmed by the CE required-runtime-file gate.Diff shape
desktop-uos/)Validation
python scripts/build_ce.py --import-check --pytest-checkdesktop-uosfiles, with no other additions or deletions