Skip to content

fix: resolve Python dotted imports in dependency analyzer#209

Merged
DevanshuNEU merged 2 commits into
OpenCodeIntel:mainfrom
DevanshuNEU:main
Jan 13, 2026
Merged

fix: resolve Python dotted imports in dependency analyzer#209
DevanshuNEU merged 2 commits into
OpenCodeIntel:mainfrom
DevanshuNEU:main

Conversation

@DevanshuNEU

Copy link
Copy Markdown
Collaborator

Bug

Python imports like starlette.exceptions were being treated as external dependencies, resulting in 0 resolved dependencies even when files existed.

Logs showed:

[DEBUG] Sample file imports file=starlette/_exception_handler.py | imports=['starlette.exceptions', 'starlette.websockets', 'starlette.requests']
[INFO] Import resolution complete resolved=0 | external=603

Root Cause

Early return in _resolve_import_to_file checked for / in import path but not .:

if '/' not in import_path:
    return None  # ← starlette.exceptions has no '/' so returns here

Fix

Only treat single-word imports (os, json, sys) as external. Allow dotted imports to reach resolution logic that converts starlette.exceptionsstarlette/exceptions.py.

Test Results

Import Before After
starlette.exceptions None starlette/exceptions.py
os None None

Bug: imports like 'starlette.exceptions' were treated as external
because early return checked for '/' but not '.' in import path

Fix: only treat single-word imports (os, json) as external
Allow dotted imports to reach resolution logic that converts
'starlette.exceptions' → 'starlette/exceptions.py'
@vercel

vercel Bot commented Jan 13, 2026

Copy link
Copy Markdown

@DevanshuNEU is attempting to deploy a commit to the Dev's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Jan 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
opencodeintel Ready Ready Preview, Comment Jan 13, 2026 0:30am

@DevanshuNEU DevanshuNEU merged commit dd0a34f into OpenCodeIntel:main Jan 13, 2026
5 checks passed
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