Exact

Exact Union RegExp

Chirag Nighut f6b92d4f13 fix(resolver): fix for same-directory python imports (#328) ... * fix(resolver): prefer same-directory file for Python bare imports Python's sys.path searches the importing script's own directory first, so `import user` from services/auth.py should resolve to services/user.py even if models/user.py was indexed first in the suffix index. Add a proximity check in resolveImportPath that consults the existing dirMap index (O(1)) before falling back to global suffix matching, for single-segment bare Python imports only. Made-with: Cursor * refactor(resolver): replace dirMap scan with O(1) allFiles.has() for proximity check The previous implementation used index.getFilesInDir() + siblings.find() which had two issues: - dirMap stores all suffix levels, so getFilesInDir('services') matched files from every directory named 'services/' across the repo — false positives in monorepos - siblings.find() was an O(n) linear scan despite the O(1) claim Replace with a direct allFiles.has(importerDir + '/' + name + '.py') lookup. allFiles is a Set<string> of full repo-relative paths, so the lookup is truly O(1) and exact — no suffix ambiguity possible. Also fixes: dead code (the '.rb' branch was unreachable since the outer if gates on Python), and Windows backslash handling via normalize before split. Made-with: Cursor * test: remove flag-based demo from unit tests Made-with: Cursor * fix(resolver): cover package __init__.py in proximity check and add end-to-end CALLS test - Also try importerDir/name/__init__.py as a second O(1) candidate so that `import user` resolves to services/user/__init__.py when the target is a package rather than a bare module file - Add unit tests for package proximity, __init__.py fallback, and Windows backslash path handling - Add end-to-end CALLS assertion to the bare-import integration test: svc.execute() must resolve to UserService#execute in services/user.py, proving the fix propagates correctly through the type inference pipeline Made-with: Cursor * refactor: extract Python import resolution into resolvers/python.ts - Move PEP 328 relative import and proximity-based bare import logic from standard.ts into a dedicated resolvers/python.ts (resolvePythonImport) - Dispatch Python imports from resolveLanguageImport in import-processor.ts, consistent with how Ruby, PHP, and other languages are handled - standard.ts is now language-agnostic (TS/JS aliases, Rust paths, suffix fallback) - Add inline comment on __init__.py vs .py resolution order edge case - Update unit tests to call resolvePythonImport directly Made-with: Cursor * docs: add PEP 302/328/451 references to python.ts comments Made-with: Cursor * fix(python): address reviewer comments on PEP compliance - Guard dirParts.pop() against over-traversal: return null when dot count exceeds directory depth, matching CPython's ImportError for 'attempted relative import beyond top-level package' (PEP 328) - Swap __init__.py / .py check order to match CPython's finder precedence (PEP 451 §4); coexistence is physically impossible so order only matters for spec compliance - Fix overstated PEP 302 comment: proximity check is a static heuristic, not a sys.path[0] implementation - Acknowledge namespace package gap (PEP 420) in docstring - Add unit test for over-traversal guard Made-with: Cursor * test(python): document namespace package resolution behaviour Add two unit tests for PEP 420 namespace packages (directory with no __init__.py): bare import returns null (expected — no file exists to resolve to, CPython sets __file__ = None), while the submodule form (import user.model) resolves correctly via suffixResolve fallback. Made-with: Cursor --------- Co-authored-by: chirag-nighut <chiragnighut@gmail.com>		2026-03-17 16:32:34 +00:00
..
models	fix(resolver): fix for same-directory python imports (#328)	2026-03-17 16:32:34 +00:00
services	fix(resolver): fix for same-directory python imports (#328)	2026-03-17 16:32:34 +00:00