Skip to content

fix: correct ArtifactRef import path and UI module dependencies - #10

Merged
m2papierz merged 2 commits into
mainfrom
fix/import-bugs
Jan 12, 2026
Merged

m2papierz merged 2 commits into
mainfrom
fix/import-bugs

Conversation

@m2papierz

Copy link
Copy Markdown
Contributor

Summary

Fixes two import bugs that cause ModuleNotFoundError:

  1. ArtifactRef lazy import pointed to non-existent devqubit_engine.core.models instead of devqubit_engine.core.types
  2. devqubit-ui imported from devqubit metapackage instead of devqubit_engine directly, breaking standalone UI installation

Type of change

  • Bug fix
  • New feature
  • Refactor (no user-facing change)
  • Docs only
  • CI/build tooling
  • Breaking change

Changelog (user-facing only)

  • I added a towncrier fragment in changelog.d/

Checklist

  • I ran lint locally (uv run pre-commit run --all-files)
  • I ran tests locally (uv run pytest)
  • I added/updated tests for behavior changes
  • I updated docs/README/examples if needed
  • If I changed dependencies, I updated uv.lock (uv lock) and committed it
  • I considered backward compatibility and security impact

Notes for reviewers

No breaking changes - these are pure bug fixes. The UI now imports directly from devqubit_engine submodules, which avoids potential circular dependency issues.

@m2papierz
m2papierz merged commit d4414e8 into main Jan 12, 2026
7 checks passed
@m2papierz
m2papierz deleted the fix/import-bugs branch January 12, 2026 18:30
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