extphprs / ext-php-rs

66%

master: 66%

LAST BUILD ON BRANCH fix/184-property-type-stubs

branch: fix/184-property-type-stubs

CHANGE BRANCH

x

Reset

• fix/184-property-type-stubs
• 102_refs
• 138_inheritance
• 173_simple_extends_and_implements
• 176_getter_test
• 182_objects_in_props
• 184_missing_property_types_in_stubs
• 186_stubs_constants
• 191_zend_callable
• 199_union_intersection_dnf
• 237_zval_flags
• 252_static_props_and_methods
• 310_performance
• 325_static_str
• 369_better_stubs
• 424_zval_heap_corruption
• 492_lit_suffixes_in_defaults
• 493_feat_sets_in_arguments
• 499_document_superglobals
• 502_returning_self
• 508_php_write
• 515_ref_mutability
• 518_cargo_php_install
• 522_indexmap
• 523_custom_allocator_and_stubs
• 525_zend_ht_entry
• 536_identifiers
• 537_zend_bailout
• 538_nullable_param
• 590_interfaces
• 599-add-performance-test-infrastructure
• abstract_methods
• add-grpc-php-rs
• add-sapi-send-headers
• add-sapi-zts-test
• cargo_watch
• chore/add-coc
• chore/bump-edition-2024
• chore/cargo-php-stubs-tests
• chore/context7
• chore/php-8-5
• chore/php-86-nightly
• ci/musl
• ci/zts-embed
• codspeed-wizard-1776277848593
• customize-guide-logo
• dependabot/cargo/convert_case-0.10.0
• dependabot/cargo/convert_case-0.11.0
• dependabot/cargo/convert_case-0.9.0
• dependabot/cargo/darling-0.23
• dependabot/cargo/libloading-0.9
• dependabot/cargo/zip-7.0
• dependabot/cargo/zip-8.0
• dependabot/github_actions/JamesIves/github-pages-deploy-action-4.7.4
• dependabot/github_actions/JamesIves/github-pages-deploy-action-4.7.6
• dependabot/github_actions/JamesIves/github-pages-deploy-action-4.8.0
• dependabot/github_actions/actions/cache-5
• dependabot/github_actions/actions/checkout-6
• dependabot/github_actions/actions/github-script-8
• dependabot/github_actions/actions/github-script-9
• dependabot/github_actions/actions/upload-artifact-7
• dependabot/github_actions/dawidd6/action-download-artifact-19
• dependabot/github_actions/dawidd6/action-download-artifact-20
• dependabot/github_actions/docker/build-push-action-7
• dependabot/github_actions/docker/setup-buildx-action-4
• docs/move-showcase-to-awesome
• embed-zts
• feat/cache-slot-property-lookup
• feat/cargo-llvm-cov
• feat/embed-php-code
• feat/error-observer
• feat/interface_impl
• feat/module-globals
• feat/observer-api
• feat/sapi-trait-and-zts
• feat/separated-phpref-types
• feat/static-property-descriptors
• feat/zend-extension
• feat/zero-alloc-fast-path
• feat/zval-ref-deref-inline
• feat_immutable_empty_array
• feat_zval_coercion
• fix-valid-method-name
• fix-windows-symbols
• fix/729-binary-interned-heap-corruption
• fix/bencher-host
• fix/binary-return-memory-leak
• fix/cargo-php-install-musl
• fix/ci-badge-workflow-reference
• fix/clang-arm64-preserve-none
• fix/clippy--
• fix/clippy-1-93-0
• fix/clippy-1-95-0
• fix/clippy-hasher
• fix/clippy-rust-1-91
• fix/cloning-class
• fix/deploy-doc
• fix/disable-zts-embed
• fix/github-templates
• fix/mock-unint
• fix/module-entry-memory-leak
• fix/musl-8-5-build
• fix/null-pointer-sapi-trampolines
• fix/parking-lot-min-version
• fix/releaze-plz-workflow
• fix/releaze-plz-workflow-permission
• fix/segfault-empy-array
• fix/use-fork-bindgen
• fix/windows-bindgen-nommx
• fix/windows-dev-pack
• fix/windows-link-attr-without-rustfmt
• fix_prop_visibility
• fix_stub_gen
• hotload
• json_serializable
• lazy
• master
• named_arguments
• patch-1
• perf/precompute-mangled-property-names
• perf/zend-string-init-fast
• raw-dylib-fix
• readonly_class
• release-plz-2025-10-29T06-23-17Z
• release-plz-2025-10-29T13-44-17Z
• release-plz-trigger
• release-plz-trigger-v2
• release-plz-trigger-v3
• remove-php80-testing
• showcase
• smartstring
• trusted-publishing

Build # 23735045124

Build Type

Pull #705

github

Committed by ptondereau

Commit Message

fix(stubs): add type declarations for properties (#184)

Properties declared via `#[prop]` or `#[getter]`/`#[setter]` now emit
typed PHP stubs instead of untyped `public $foo`.

- Extract `<T as IntoZval>::TYPE` and `::NULLABLE` (compile-time consts)
  in the proc macros for field properties and getter/setter methods
- Thread type info through `PropertyInfo` → `ClassBuilder` → `describe::Property`
- Emit `?type` prefix for nullable properties (`Option<T>`)
- Emit `readonly` keyword for getter-only properties (no setter)
- Emit `@var` PHPDoc tag only when Rust doc comments are present

Closes #184

Pull Request Pull Request #705: fix(stubs): add type declarations for properties

Coverage Stats

281 of 315 new or added lines in 6 files covered. (89.21%)

8068 of 12242 relevant lines covered (65.9%)

32.05 hits per line