Skip to content

fix(python): separate package symbol scope from file identity - #2528

Open
DeusData wants to merge 6 commits into
mainfrom
fix/python-init-symbol-scope-v2
Open

DeusData wants to merge 6 commits into
mainfrom
fix/python-init-symbol-scope-v2

Conversation

@DeusData

@DeusData DeusData commented Oct 3, 2026 •

Copy link
Copy Markdown
Owner

Functions and classes declared in pkg/__init__.py have names such as pkg.Field, but resolution used the file identity pkg.__init__ as their symbol scope. This could lose precise call edges and choose a same-named base class from another module.

Keep the raw file identity for Module nodes and top-level callers, and use package scope for Python symbols. Propagate explicit language through resolution, include Python package definitions in import filtering, preserve complete long qualified names, and report scope allocation failures explicitly. Other languages and the public raw/lineage resolver APIs retain their scope semantics.

Regression coverage includes ordinary-module controls, package-local and imported calls, inheritance collisions, sequential and parallel indexing, scope allocation failures, import-filter fallback, and resolver cache/long-name boundaries.

Stacked on #2525 (which includes #2522 and #2498). This PR targets main, so its diff includes those dependencies until they land. Existing indexes need reindexing to acquire corrected edges.

Parallel resolver error slots use matching tracked allocation and cleanup. Import-filter failures share cleanup while preserving the full-registry fallback. The existing raw-allocation allowances are unchanged.

Validation: source review, formatting, and diff checks. Builds, tests, compiler/syntax checks, revert proof, and local CI are UNRUN, as requested. GitHub CI is pending.

Carry declared instance-field types through extraction and result
compaction into the shared Python type registry so chained calls through
imported classes resolve their receiver types.

Refs #1277

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
Copy inherited field-type pointer arrays into the writable registry arena
before Python refines an existing field. Keep shared type objects intact
and avoid retaining per-file scratch types in the sealed fallback.

Add deterministic shared-state and later-overlay coverage, retaining the
sequential Python field regression and adding a forced-parallel variant.

Refs #1277

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
Memoize C expression evaluation, index repeated scope and registry queries,
and report incomplete walks explicitly on memo or depth failure.

Refs #1527

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
Preserve the arena-owned raw Module caller name so top-level resolutions
join the extractor in both pipeline drivers. Add occurrence and ownership
controls, including package modules and named lambdas.

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
Resolve symbols declared in __init__.py under their package while retaining
the raw module identity for file and top-level caller nodes. Carry explicit
language through registry callers, select Python package imports, preserve
long exact names, and report scope allocation failures without partial output.

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
Use tracked zeroed storage for the parallel LSP failure array and
release it with the matching memory class after worker joins.
Share import-filter failure cleanup without changing fallback state.
Keep existing raw-allocation allowances unchanged.

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
@DeusData

DeusData commented Oct 3, 2026

Copy link
Copy Markdown
Owner Author

Checkpoint and handover (2026-10-03 UTC)

Published head: 71337a2853c1aef198dc0b882565a817d3a1c7f5. The remote head was verified.

Python init handling is published across 14 files with 22 new tests, unrun locally. Prerequisite: #2525. The allocator follow-up preserves shared-filter failure cleanup and checked parallel-array allocation without relaxing the baseline. #2529 is the next dependent PR.

Hosted snapshot at 2026-10-03 21:54:22 UTC: 3 queued. Confirm the required checks on this exact head before treating it as ready.

No additional local build, test, lint, sanitizer, benchmark or CI runs were performed at this checkpoint, as requested. Earlier executed evidence remains historical; prepared tests and the newer source-reviewed changes must still be validated by the hosted gate.

The campaign is paused at the maintainer’s request. Local monitoring has stopped; hosted jobs remain running. No merge was performed. Thanks for reviewing this change.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant