Repository navigation
chore(ci): remove the remaining HarmonyOS CI and script leftovers - #3373
Merged
stamp-build[bot] merged 1 commit intoOct 21, 2026
Merged
Conversation
HarmonyOS was removed from the codebase in #3181; these were the last OHOS pieces in CI and scripts. - Delete the `OHOS Release` dispatch stub workflow (nothing references it). - prepare_android_hands_symbols.sh: drop the DevEco/openharmony llvm-readelf fallback paths. - i18n literal scanner: scan Kotlin only (drop .ets/ArkTS parsing, the ArkTS resource prefixes and the "OHOS" technical term); drop the ArkTS test cases and the two Harmony/OHOS allowlist entries. - review_agent.mjs: no `ohos` platform branch; fix_queue.mjs: no `ohosTest` test root; cmp_migrate_imports.py: comment no longer names ohosArm64Main. - Kuikly consumer and no-local-Ktor contracts: drop the OHOS-only assertions (kuiklyOhos pin, ktor-ohos fork paths/versions, ohosArm64 variant); their other checks are unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 21, 2026
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.
Summary
HarmonyOS was removed from the codebase in #3181. This PR removes the OHOS pieces that were still left in CI and scripts. This changes CI and workflows on artin's explicit instruction ("鸿蒙部分都删了": remove all the HarmonyOS parts). The docs cleanup is a separate PR, #3371.
.github/workflows/ohos-release.yml. This was the "OHOS Release" dispatch stub. Nothing referenced it: no other workflow,workflow_runtrigger, ruleset required check, path filter or contract script.scripts/prepare_android_hands_symbols.sh: dropped the two DevEco/openharmonyllvm-readelffallback paths. The Android NDK,llvm-readelfandreadelfcandidates are still there.scripts/i18n_code_literal_scan.py: it now scans.ktonly. Removed the.ets/ArkTS parsing (single-quote and backtick strings), the ArkTSapp.string./sys.string.prefixes and the"OHOS"technical term.scripts/qa/test_i18n_code_literal_scan.py: removed the two ArkTS-only tests and the ArkTS half of the resource-reference test.scripts/i18n_code_literal_allowlist.json: removed the "Harmony smoke preview" and "generated OHOS paths" entries.scripts/qa/review_agent.mjs: removed theohosplatform branch, so the platform isiosorandroid.scripts/patrol/fix_queue.mjs: removedohosTestfrom the test-root regex.scripts/cmp_migrate_imports.py: the comment no longer namesohosArm64Main.scripts/ci/test_kuikly_raft_consumer_contract.py: removed thekuiklyOhoscatalog-pin assertion. The Kuikly UI checks are unchanged.scripts/ci/test_no_local_ktor_ci_contract.py: removed the OHOS-only assertions: theprepare_ktor_ohos_fork.shandthird_party/ktor-ohospaths, the2.3.7-slock-ohosversion, thektor-client-core-ohosarm64variant and the.json5(oh-package) suffix. The mavenLocal andio.ktorchecks and the NetworkKMM version check are unchanged. Both contracts still guard non-OHOS things, so they stay intest.yml.Tests run locally
python3 -m unittest test_i18n_code_literal_scan(inscripts/qa): 18 tests, OKpython3 scripts/ci/test_kuikly_raft_consumer_contract.py: PASSpython3 scripts/ci/test_no_local_ktor_ci_contract.py: PASSnode --test scripts/qa/*.test.mjs: 169 pass, 0 failnode --test scripts/patrol/fix_queue.test.mjs: passnode --checkonreview_agent.mjsandfix_queue.mjs: OKbash -n scripts/prepare_android_hands_symbols.sh: OKpy_compile cmp_migrate_imports.py: OKcheck_gradle_cache_policy_wiring.py --self-testand the real run: OKtest_gradle_cache_policy.sh: OKcheck-merge-command-policy.shandtest_merge_command_policy.sh: OKcheck_source_constraints.py: OKrubyand CI will run them in full:test_ci_tiering_contract.sh,test_git_diff_path_state.shandtest_android_hands_release_contract.sh. Their non-ruby parts passed, and none of them reference the deleted workflow.Notes / follow-ups (not in this PR)
i18n_code_literal_scan.pyon the real repo already fails onorigin/main: a stale allowlist entry ("Floating Dock" inSettingsHomePage.kt) no longer matches anything. This is unrelated to this PR, and the scan is advisory and not wired into CI. With the two Harmony allowlist entries gone, those two literals (still inSettingsHomePage.ktandRaftLucideIconGalleryPage.kt) will show up as findings until the compose code cleanup removes them.compose/shared/src/ohosArm64Main/.../RaftStoreConfinement.ohos.kt, a source set that no target compiles; OHOS/Harmony comments incompose/**; andTHIRD_PARTY_NOTICES.md.🤖 Generated with Claude Code