fix: merge 4 mobileSyncConflict scenarios into 1 to fit CI budget, fix git process leaks and error handling

- Consolidate 4 separate mobileSyncConflict scenarios into a single scenario
  with 4 parts, saving 3 app restarts (~2min on CI)
- Add Observable teardown to kill git child processes on unsubscribe
- Propagate stdin errors to subscriber instead of silently logging
- Validate git service parameter against allowlist
- Reorder readOnly check before expensive git operations
- Add error event handler to all git child process spawns
This commit is contained in:
lin onetwo 2026-03-07 22:48:07 +08:00
parent a4669132c6
commit 7ca460ea5e
4 changed files with 55 additions and 26 deletions

View file

@ -52,7 +52,7 @@ jobs:
# Set Chinese locale for i18n testing
LANG: zh_CN.UTF-8
LC_ALL: zh_CN.UTF-8
timeout-minutes: 25
timeout-minutes: 15
# Upload test artifacts (screenshots, logs)
- name: Upload test artifacts