Repository navigation
chore(ios): submit localization and deployment updates - #12
Merged
Merged
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 160464c303
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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.
提交此前保留在本地的六项 iOS 本地化和部署修改。固定设备 ID 失效时，部署入口会回退到唯一可用的 iPhone，并允许设备列表查询短暂失败后重试；续签流程也会等待 Xcode 生成 profile，即使没有旧 profile 可备份，已检测到的新 profile 会在构建失败时保留。
本次复核同时修复了选机缺口：只接受完整的
connected/available状态，避免把disconnected/unavailable误选为可用设备；设备列表查询最终失败时，不使用失败命令留下的部分输出。两个部署入口都按列读取 Identifier，兼容 UDID、ECID 和 CoreDevice UUID，补齐 Xcode 27 设备标识变化后的回退。新增部署 CLI 回归测试并接入 iOS CI。本地化采用与最新 master 的语义合并，保留主干新增的 26 个歌词文案。
Localizable.xcstrings共 233 项；本地新增的 16 项为 Xcode 提取条目，既有翻译值未修改。未改动 BLE 协议或 Android 业务代码。验证：
bash -n；git diff --check通过。201c30e的干净工作区运行完整 iOS smoke。原始 Xcode 日志为BUILD SUCCEEDED，但 Tm iPhone 状态为unavailable，CoreDevice 无法定位设备，安装、启动及后续检查失败。报告整体仍为 FAIL（required 0/6）；报告将 build/install 组合步骤失败同时记为 build 失败，不能以此替代原始构建日志。隔离工作区的代码索引刷新因索引工作进程崩溃失败；合入主工作区后已对
b57ba7f完成 full 索引，未生成仓库内持久化图谱文件。部署目录不在索引范围内，本轮部署结论仍使用真实文件核对和脚本测试验证。GitHub CI 已验证当前 head
201c30e461e84b919d9e2b7495c60035d41ea4ed：testLastNowPlayingSnapshotStoreSaveLoadAndClear中出现load == nil断言失败；工作流重启模拟器并按既有策略重试后，104/104 通过。部署测试 9/9 首次运行通过。已合并到 master：
b57ba7f0893a8fa0cc7353f1ab639b4e39726745。合并提交的主干 CI 最终成功：部署测试 9/9 首次运行通过，Android 单测及 APK 构建成功；iOS 同样在上述既有快照存储测试首次失败，按既有策略重试后 104/104 通过。这项间歇失败未在本 PR 修改，不能将重试成功记为首次运行全通过。
原工作区已快进到该 master 提交，无未提交改动、无 ahead/behind 差异。六项原始本地差异已核对与先前快照完全一致，并保留在可恢复的 Git stash 备份中；备份不代表仍有未交付修改。