Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Align Microsoft.ReactNative.Cxx includes with core public headers

Đang mở
  • Có gì không ổn?
#16,522 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
42/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
cpp, react-native
Lĩnh vực
build-system

Hướng nghiên cứu

Read Microsoft.ReactNative.Cxx.vcxitems alongside the vnext/include/React forwarders and the core <React/*.h> umbrellas, using PR #16513 as the reference for the intended public include shape. Inventory the header closure each umbrella needs, then compare it with the Cxx include path and ClInclude entries. Done means ReactCommon is no longer on the public include path, the NuGet ships only that closure, and exclusions such as Dynamic.h (#11644) remain.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Integration Follow-up Needs: Triage :mag:

Problem Description

Core now treats <React/*.h> as the public C++ entry point and guards direct includes of fine-grained headers. vnext/include/React already forwards those umbrellas, but Microsoft.ReactNative.Cxx.vcxitems still adds all of ReactCommon to the public include path and lists private headers as package includes. Public headers therefore pull in private headers, and the NuGet does not ship the include/React forwarders.

Follow-up from https://github.com/microsoft/react-native-windows/pull/16513. MS.RN.Cxx should match core's public includes.

Scope:

  1. Inventory the header closure required by each core <React/*.h> umbrella.
  2. Compare that closure with the Cxx include path and ClInclude entries.
  3. Stop exposing ReactCommon on the public include path.
  4. Ship only that closure in the Microsoft.ReactNative.Cxx NuGet.
  5. Keep the existing Windows exclusions, including Dynamic.h (#11644).
  6. Do not move headers out of the npm package until that layout is agreed.

Removing ReactCommon from the include path before the closure is packaged would break in-tree and external native code.

Integration PR: https://github.com/microsoft/react-native-windows/pull/16513
Nightly: 0.89.0-nightly-20261002-cd2de187e

Ngôn ngữ chính
C++
Star
17.4k
Fork
1.2k
Merge trung bình
1 ngày 20 giờ
Pull request đã merge (30 ngày)
37

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của microsoft/react-native-windows

Tất cả issue của microsoft/react-native-windows

Issue tương tự

Thêm issue về C++

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.