Repository navigation
Artifact walker: keep Kotlin-public declarations in *.internal packages out of the bindings #192
Description
What
The artifact walker registers every Kotlin-public declaration (blacklist exposure, AGENTS.md §12.7). It has no package filter: excludePackages reaches only the KSP processor, not the walker. So a JVM- and Kotlin-public declaration in a package such as androidx.compose.ui.internal is bound and gets a stub.
This now matters: the core-extended fork (fork PR #19, AWT removal for pointer icons) puts embedder hooks in androidx.compose.ui.internal. They are public in bytecode but left out of the .api dump on purpose, i.e. not public API by the upstream convention.
Proposal (spec change first, SPEC B-section)
- The walker skips packages with an
internalsegment (*.internal,*.internal.*), matching the.apidump convention, and honourspythonBindings.excludePackageslike the KSP side. - This stays a blacklist: the rule is "what the library itself declares non-public API", not a whitelist.
Done when
SPEC states the rule; a fixture artefact with a public class in an .internal package produces no binding and no stub (red first); the three fixture modules pass as separate invocations.
python-multiplatform's turn in the priority order; related to #191.
Activity
b-re-w commentedon Oct 4, 2026
Update (fork #19 is now fork #56, 2026-10-05): the pointer-icon hooks in
androidx.compose.ui.internal(StandardPointerIconKind,PointerIcon.standardKind,PointerIconSink,ExtendedPointerIcons) now also carry@InternalComposeUiApi, because the.apidump does not drop the package by itself.Scope for the spec:
@InternalComposeUiApi,@InternalComposeApi,@InternalFoundationApi,@InternalTextApiand the like, i.e. RequiresOptIn markers whose name says Internal.@ExperimentalMaterial3Api,@ExperimentalFoundationApiand other Experimental markers guard API that pythonx-compose uses (for example Material 3 top app bars). Skipping them would remove bindings the notebook needs.@Internal...Apideclaration and one@Experimental...Apideclaration; the first has no binding, the second keeps its binding.Test: checkIndexOverflow is now skipped with its internal package; ch…
b-re-w commentedon Oct 17, 2026
Landed in develop via #199