| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
0.82: fix the DCP CodeCheck findings (codemars, cmetrics) Fixes the Code Issues DCP reported for MR 134 (run 6ace179564650f998bb6c97c, issue #99) under the codemars and cmetrics rules, one commit per kind of finding: - codemars 禁用显示的any类型声明 (23): explicit any replaced with concrete types (a declared TraceEvent for the callback tracer, RNGH event types for the old-API recorders, a readAnimatedValue() util for the one __getValue read, Platform.OS widened to string); - codemars G.MET.07 (1): explicit return undefined in the error-forwarding effect of AutomatedApp; - cmetrics 冗余代码 (8): commented-out code deleted (nested-scroll demo, hover example, deploy.js); - cmetrics 超大函数 / 超大深度函数 / 超大圈复杂度 (13): adb-server.js translate()/startBridge(), e2e-android.js main(), format-cpp.js main(), GestureGuides resolveGuides(), chatHeads constructor/render(), Driver.measure() and deploy.js runDeployment() split into smaller functions, no behaviour change. Left as is: createHandler.tsx render() (upstream RNGH code, marked ignored in DCP). Checks: tsc --noEmit (no new errors; the 4 pre-existing ones in src/issues/* stay), prettier, node --check, old vs new translate() compared on 17 inputs (identical), node scripts/format-cpp.js --check (24 files ok), e2e-android.js without a device fails the same way as before. No e2e run. Signed-off-by: Radosław Rolka <radoslaw.rolka@swmansion.com> | 3 天前 | |
0.82: fix the DCP CodeCheck findings (codemars, cmetrics) Fixes the Code Issues DCP reported for MR 134 (run 6ace179564650f998bb6c97c, issue #99) under the codemars and cmetrics rules, one commit per kind of finding: - codemars 禁用显示的any类型声明 (23): explicit any replaced with concrete types (a declared TraceEvent for the callback tracer, RNGH event types for the old-API recorders, a readAnimatedValue() util for the one __getValue read, Platform.OS widened to string); - codemars G.MET.07 (1): explicit return undefined in the error-forwarding effect of AutomatedApp; - cmetrics 冗余代码 (8): commented-out code deleted (nested-scroll demo, hover example, deploy.js); - cmetrics 超大函数 / 超大深度函数 / 超大圈复杂度 (13): adb-server.js translate()/startBridge(), e2e-android.js main(), format-cpp.js main(), GestureGuides resolveGuides(), chatHeads constructor/render(), Driver.measure() and deploy.js runDeployment() split into smaller functions, no behaviour change. Left as is: createHandler.tsx render() (upstream RNGH code, marked ignored in DCP). Checks: tsc --noEmit (no new errors; the 4 pre-existing ones in src/issues/* stay), prettier, node --check, old vs new translate() compared on 17 inputs (identical), node scripts/format-cpp.js --check (24 files ok), e2e-android.js without a device fails the same way as before. No e2e run. Signed-off-by: Radosław Rolka <radoslaw.rolka@swmansion.com> | 3 天前 | |
0.82: fix the DCP CodeCheck findings (codemars, cmetrics) Fixes the Code Issues DCP reported for MR 134 (run 6ace179564650f998bb6c97c, issue #99) under the codemars and cmetrics rules, one commit per kind of finding: - codemars 禁用显示的any类型声明 (23): explicit any replaced with concrete types (a declared TraceEvent for the callback tracer, RNGH event types for the old-API recorders, a readAnimatedValue() util for the one __getValue read, Platform.OS widened to string); - codemars G.MET.07 (1): explicit return undefined in the error-forwarding effect of AutomatedApp; - cmetrics 冗余代码 (8): commented-out code deleted (nested-scroll demo, hover example, deploy.js); - cmetrics 超大函数 / 超大深度函数 / 超大圈复杂度 (13): adb-server.js translate()/startBridge(), e2e-android.js main(), format-cpp.js main(), GestureGuides resolveGuides(), chatHeads constructor/render(), Driver.measure() and deploy.js runDeployment() split into smaller functions, no behaviour change. Left as is: createHandler.tsx render() (upstream RNGH code, marked ignored in DCP). Checks: tsc --noEmit (no new errors; the 4 pre-existing ones in src/issues/* stay), prettier, node --check, old vs new translate() compared on 17 inputs (identical), node scripts/format-cpp.js --check (24 files ok), e2e-android.js without a device fails the same way as before. No e2e run. Signed-off-by: Radosław Rolka <radoslaw.rolka@swmansion.com> | 3 天前 | |
0.82: fix the DCP CodeCheck findings (codemars, cmetrics) Fixes the Code Issues DCP reported for MR 134 (run 6ace179564650f998bb6c97c, issue #99) under the codemars and cmetrics rules, one commit per kind of finding: - codemars 禁用显示的any类型声明 (23): explicit any replaced with concrete types (a declared TraceEvent for the callback tracer, RNGH event types for the old-API recorders, a readAnimatedValue() util for the one __getValue read, Platform.OS widened to string); - codemars G.MET.07 (1): explicit return undefined in the error-forwarding effect of AutomatedApp; - cmetrics 冗余代码 (8): commented-out code deleted (nested-scroll demo, hover example, deploy.js); - cmetrics 超大函数 / 超大深度函数 / 超大圈复杂度 (13): adb-server.js translate()/startBridge(), e2e-android.js main(), format-cpp.js main(), GestureGuides resolveGuides(), chatHeads constructor/render(), Driver.measure() and deploy.js runDeployment() split into smaller functions, no behaviour change. Left as is: createHandler.tsx render() (upstream RNGH code, marked ignored in DCP). Checks: tsc --noEmit (no new errors; the 4 pre-existing ones in src/issues/* stay), prettier, node --check, old vs new translate() compared on 17 inputs (identical), node scripts/format-cpp.js --check (24 files ok), e2e-android.js without a device fails the same way as before. No e2e run. Signed-off-by: Radosław Rolka <radoslaw.rolka@swmansion.com> | 3 天前 | |
Fix the hidden bar not sliding out when dragging a nested list (demo 2) Signed-off-by: Mateusz Pawliczek <mateusz.pawliczek@swmansion.com> | 1 个月前 | |
0.82: fix the DCP CodeCheck findings (codemars, cmetrics) Fixes the Code Issues DCP reported for MR 134 (run 6ace179564650f998bb6c97c, issue #99) under the codemars and cmetrics rules, one commit per kind of finding: - codemars 禁用显示的any类型声明 (23): explicit any replaced with concrete types (a declared TraceEvent for the callback tracer, RNGH event types for the old-API recorders, a readAnimatedValue() util for the one __getValue read, Platform.OS widened to string); - codemars G.MET.07 (1): explicit return undefined in the error-forwarding effect of AutomatedApp; - cmetrics 冗余代码 (8): commented-out code deleted (nested-scroll demo, hover example, deploy.js); - cmetrics 超大函数 / 超大深度函数 / 超大圈复杂度 (13): adb-server.js translate()/startBridge(), e2e-android.js main(), format-cpp.js main(), GestureGuides resolveGuides(), chatHeads constructor/render(), Driver.measure() and deploy.js runDeployment() split into smaller functions, no behaviour change. Left as is: createHandler.tsx render() (upstream RNGH code, marked ignored in DCP). Checks: tsc --noEmit (no new errors; the 4 pre-existing ones in src/issues/* stay), prettier, node --check, old vs new translate() compared on 17 inputs (identical), node scripts/format-cpp.js --check (24 files ok), e2e-android.js without a device fails the same way as before. No e2e run. Signed-off-by: Radosław Rolka <radoslaw.rolka@swmansion.com> | 3 天前 | |
fix(codecheck): 修复 codecheck 检查问题 Signed-off-by: chenhaoyue <chenhaoyue1@h-partners.com> AI[99%] Human Fixed[0%] Human[1%] AI Adopted[100%] Signed-off-by: chenhaoyue <chenhaoyue1@h-partners.com> Co-authored-by: opencode (glm-5.3) <ai@local> | 3 天前 | |
0.82: fix the DCP CodeCheck findings (codemars, cmetrics) Fixes the Code Issues DCP reported for MR 134 (run 6ace179564650f998bb6c97c, issue #99) under the codemars and cmetrics rules, one commit per kind of finding: - codemars 禁用显示的any类型声明 (23): explicit any replaced with concrete types (a declared TraceEvent for the callback tracer, RNGH event types for the old-API recorders, a readAnimatedValue() util for the one __getValue read, Platform.OS widened to string); - codemars G.MET.07 (1): explicit return undefined in the error-forwarding effect of AutomatedApp; - cmetrics 冗余代码 (8): commented-out code deleted (nested-scroll demo, hover example, deploy.js); - cmetrics 超大函数 / 超大深度函数 / 超大圈复杂度 (13): adb-server.js translate()/startBridge(), e2e-android.js main(), format-cpp.js main(), GestureGuides resolveGuides(), chatHeads constructor/render(), Driver.measure() and deploy.js runDeployment() split into smaller functions, no behaviour change. Left as is: createHandler.tsx render() (upstream RNGH code, marked ignored in DCP). Checks: tsc --noEmit (no new errors; the 4 pre-existing ones in src/issues/* stay), prettier, node --check, old vs new translate() compared on 17 inputs (identical), node scripts/format-cpp.js --check (24 files ok), e2e-android.js without a device fails the same way as before. No e2e run. Signed-off-by: Radosław Rolka <radoslaw.rolka@swmansion.com> | 3 天前 | |
add upstream app examples Signed-off-by: Radosław Rolka <radoslaw.rolka@swmansion.com> | 1 个月前 | |
Get the HarmonyOS e2e suite green: e2e fixes, attach order and review fixes ## Summary The HarmonyOS e2e suite on the 0.82 line now passes in full on the phone and on emulators, mostly through fixes in the native gesture layer and the e2e harness; one HarmonyOS-only case is skipped. The last remaining intermittent red, a Pressable press lost on some mounts, came from a node's handlers being kept in attach-success order; they now keep upstream's attach order. Issue/Task: # ## Changes * **Fixes from the e2e fix commits** - a node context left on a reused ArkUI handle is discarded, fingers that lift before the session opens are replayed into the touch stream, an ACTIVE native view emits an update per move (RectButton drag-out), a wait target's END in the same touch counts, a lift with no position lifts no finger, the inner scroll pan pairs with a node the scroll waits for, the two NativeView handlers of one ScrollView act as one scroll, a pan follows its nominated finger by proximity, and two unrelated taps on one node settle in attach order. * **Attach order** - a node's handlers now keep the order their attach messages arrived in, like upstream's registry. Before, a retried attach could put a Pressable's LongPress ahead of its Native handler and the Pressable state machine dropped every press on that mount (the intermittent hitSlop and delayLongPress Pressable reds). * **Review fixes** - the scroll judge prefers the gated NativeView also before a session opens, replaying early-lifted fingers no longer lifts a finger still down, a session begun from JS stamps its down time, the END time is stamped on END only, and the diagnostic hold/pairing logs are dropped. * **E2E harness** - the app starts only after the bridge acknowledges the multi-finger driver, so two-finger cases no longer drop out as manual; single-finger multi-pointer requests are accepted; npm run test:e2e -- --only <ids> limits a run to categories or suites. The app-side wait that the acknowledgement made redundant is removed. * **Tests** - the ten-finger case pins nine (ArkUI's node touch event cannot place a tenth), the averageTouches grip is widened, the overscroll check allows half a physical pixel of float residue, and the same-axis scroll hand-off case is skipped on HarmonyOS (Android and iOS do not chain a nested drag). Two scroll-conflict cases whose asserts need the driver's exact sequence show as skipped in a manual run. * Added comments cut to one line; NATIVE-CODE-MAP.md updated. ## E2E results - Final branch: full runs green on every device below, 0 fail, 0 broken. - Before the review commit (the e2e fix commits alone) the suite still lost a Pressable press now and then; those runs are listed below for comparison. - Host: OHOS clang -fsyntax-only on all 17 TUs. ### HarmonyOS (npm run test:e2e) - Full run - HarmonyOS phone - finished=466 pass=457 fail=0 broken=0 skipped=9 examples=0 - Full run - HarmonyOS emulator - finished=466 pass=457 fail=0 broken=0 skipped=9 examples=0 Before the review commit: - Full run - HarmonyOS phone - finished=466 pass=456 fail=2 broken=0 skipped=8 examples=0 - Full run - HarmonyOS emulator - finished=466 pass=457 fail=1 broken=0 skipped=8 examples=0 - Full run - HarmonyOS phone - finished=466 pass=458 fail=0 broken=0 skipped=8 examples=0 ### Android (npm run test:e2e:android) Not run on this branch. Android runs upstream RNGH from node_modules, so only the tester changes reach it. Signed-off-by: Mateusz Pawliczek <mateusz.pawliczek@swmansion.com> | 16 天前 | |
tester: group the test suites into category folders Signed-off-by: Radosław Rolka <radoslaw.rolka@swmansion.com> | 1 个月前 |