fix: electron app launch with wayland protocol by default - #19
Merged
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Groveer The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
TAG Bot TAG: 2026.08.27 |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR fixes Electron application startup on Wayland by installing per-application Application Manager overrides that select the Wayland protocol by default for eleven affected applications, and documents the change in the Debian changelog. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
deepin pr auto review🤖 AI 代码审查报告总体评分： 📊 总体评价
🔍 详细分析1. 语法逻辑 ✅评价： 优秀 ✅ 通过（语法正确，逻辑清晰） 潜在问题： 建议： 无 2. 代码质量 ✅评价： 优秀 ✅ 通过（代码结构清晰，注释完整） 潜在问题：
建议： 统一所有 JSON 配置文件的尾部换行符，建议所有文件均以换行符结尾（POSIX 标准） 3. 代码性能 ✅评价： 优秀 ✅ 通过（性能良好，资源使用合理） 潜在问题： 建议： 无 4. 代码安全 🔒存在0个安全漏洞 评价： 优秀 ✅ 通过（安全合规）
漏洞对比统计：新增漏洞 0 个，减少漏洞 0 个，持平 0 个 安全漏洞详情： 安全分析：
建议： 无 📝 代码目的与审查结论代码目的： 修复 Bug 371259（腾讯元宝、千问等 Electron 应用程序首次无法打开启动运行），通过为 10 个 Electron 应用添加 Wayland 协议默认启动配置覆盖。 修改文件清单（11个文件，新增 114 行）：
审查结论： 代码实现与 commit 目的完全一致。通过添加 dsg.config.override 格式的 JSON 配置文件，为各 Electron 应用设置 Wayland 协议启动参数，有效解决 Wayland 环境下 Electron 应用首次无法启动的问题。所有 JSON 文件结构正确，参数配置合理，无安全风险。 💡 改进建议统一文件尾部换行符： com.bytedance.feishu 和 com.doubao.uos 两个配置文件末尾缺少换行符，建议统一添加： {
"magic": "dsg.config.override",
"version": "1.0",
"contents": {
"Exec": {
"value": "!AM_FULL! --enable-features=UseOzonePlatform --ozone-platform=wayland --enable-wayland-ime",
"permissions": "readwrite"
}
}
}（文件末尾以换行符 本报告由 AI 代码审查工具自动生成 |
Contributor
|
/integrate |
|
AutoIntegrationPr Bot |
|
TAG Bot ✅ Tag created successfully 📋 Tag Details
|
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.
Bug: https://pms.uniontech.com/bug-view-371259.html
Summary by Sourcery
Enable Wayland-by-default launching for supported Electron applications.
Bug Fixes:
Chores: