iOS screenshot implementation disagrees with the generated nullable bounds contract
维护者通常 1 天内回复
已经有一个关联 PR 被合并了。
- #256 来自 @mrousavy —— 已合并
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 25/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- react-native, typescript
- 领域
- mobile-dev, testing
调研方向
Start with the screenshot spec and the generated HarnessUISpec.h declaration, then inspect HarnessUI.mm and the playground regression tests. The issue describes moving nullable bounds into a required options object with an optional ViewInfo field; verify full-window, explicit-region, and element captures while keeping screen.screenshot() unchanged. Pull request #256 is already open against this issue.
由索引模型根据 Issue 内容生成。
描述
iOS builds report a conflicting parameter type for captureScreenshot:resolve:reject:: generated HarnessUISpec.h declares JS::NativeHarnessUI::ViewInfo &, while HarnessUI.mm implements ViewInfo *.
The TypeScript spec accepts ViewInfo | null, and screen.screenshot() passes null for a full-window capture. React Native's Objective-C++ generator still emits a reference for that top-level nullable struct. Changing the implementation to a reference alone would leave the full-window case without a valid optional value.
I am preparing a fix that puts optional bounds inside a required options object, using the generator's supported std::optional<ViewInfo> field representation. The public screen.screenshot() API stays the same. The playground regression will cover full-window, explicit-region and element captures.
Observed in Nitro's successful release build: https://github.com/margelo/nitro/actions/runs/39439032533/job/118683815608
- 主要语言
- TypeScript
- 星标
- 333
- 派生
- 20
- 平均合并
- 14 小时 55 分钟
- 30 天内合并 PR
- 44
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
callstackincubator/react-native-harness 的其他 Issue
-
--metroPort is documented, but the CLI rejects it as an unrecognized option可能已有人在做 @YevheniiKotyrlo 于 3 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 72/100
callstackincubator/react-native-harness#199 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
callstackincubator/react-native-harness#128 ·
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 72/100
callstackincubator/react-native-harness#277 ·
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 72/100
callstackincubator/react-native-harness#273 ·
维护者通常 1 天内回复
-
Android Activity recreation during a UI test is not reported as a host lifecycle change可能已有人在做 @mrousavy 于 2 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 35/100
callstackincubator/react-native-harness#271 · 1 条评论 ·
维护者通常 1 天内回复
查看 callstackincubator/react-native-harness 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 66/100
lyswhut/lx-music-desktop#3026 ·
-
area: ci type: test
难度 2/5 1-3 小时 新手友好度 65/100
schlessera/brain-kit#2769 ·
维护者通常 1 天内回复
-
Read a vector index's quantization back on Neo4j 2026.10, which reports vector.quantization.type未关闭
难度 2/5 1-3 小时 新手友好度 78/100
liberation-data/drivine#231 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 65/100
alveusgg/alveusgg#2502 · 3 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复