Skip to main content

CI tool to detect indirect import

Project description

pyproject-indirect-import-detector: Indirect import detector

PyPI Project License Supported Python versions CircleCI Code style: black

Motivation

Indirect import is bad.

  • The biggest reason is requirements are not protected by semantic versioning.
  • Tests cannot check import-problem correctly.
  • Virtual environment is not strictly synced with pyproject.toml nor poetry.lock by poetry, so far. It is possible that you delete a dependency but it still remains in the virtual environment. This means, tests can accidentally pass.
  • FYI, indirect import is not allowed in rust/cargo.

We'll describe the first reason a little more. There are two examples here.

First, assume that

  • You are developping a library or application X.
  • X depends on a library A ^1 and indirectly depends on a library B ^1, meaning, X imports B but does not declare the dependency in pyproject.toml.
  • The latest A is version 1.0.0 and it depends on B 1.0.0.
      Indirect import; imported but no requirement in pyproject.toml
      |
  +--------> B
  |          ^
  |          |
  |          | ---+
  |          |    |
  X -------> A    Requirement in pyproject.toml
      |
      Requirement in pyproject.toml

Consider the case that the authers of A refactor A and publish A 1.1.0 and this does not depends on B anymore. Then,

  • If X is a library, user of library X can face ModuleNotFoundError.
  • If X is an application and you're using a "blessed" lockfile, you'll get no errors.
  • If X is an application and you're not lucky, you'll get ModuleNotFoundError.

The second example is almost the same to the first. Consider the case that the authers of A refactor A and publish A 1.1.0 and this depends on B ^2 and B has breaking chanegs. Then, you may not have an explicit errors but the behavior can differ from the expected.

It's horrible, isn't it? The most reliable and simple way is maintaining dependencies correctly. Because the resource of human brain is limited and most precious, this should be checked in CI. pyproject-indirect-import-detector is a tool for this purpose.

Limitation

Currently, this tool only suuport pyproject.toml using poetry. PEP 631 support is planed if python community agree this motivation and use this tool.

How to use

Install

poetry add --dev pyproject-indirect-import-detector

Note that this tool is intended to use in the virtual environment created by poetry install. See also: Why only works in venv?

Usage

poetry run pyproject-indirect-import-detector -v

See also CI config, especially the job check-indirect-import.

Configuration

You can configure by pyproject.toml as the following:

[tool.pyproject-indirect-import-detector]
exclude_projects = [
    "dataclasses",           # If you use compat trick like https://github.com/kenoss/pyproject-indirect-import-detector/tree/main/tests/integration_test/case/ok_exclude_projects
]
exclude_modules = [
    "dataclasses",           # Corresponding "dataclasses" in `exclude_projects`.
    "tests",                 # If your test suite make `tests` module importable and use it like https://github.com/andreoliwa/nitpick/blob/v0.26.0/tests/test_json.py#L6 .
    "dummy_module_for_test", # If you use dummy modules in tests like https://github.com/PyCQA/isort/blob/5.8.0/tests/unit/example_crlf_file.py#L1-L2 .
]
  • tool.pyproject-indirect-import-detector.exclude_projects

    Omit searching the distributions corresponding designated package names. Notice that this also prevent making "project to modules map". You should also specify exclude_modules manually. For example, see tests OK, NG.

  • tool.pyproject-indirect-import-detector.exclude_modules

    Omit checks for the designated modules. For example, see tests OK, NG.


You can find more examples in CI config, the jobs test-external-project-*.

FAQ

It failes with not reasonable errors.

Report an issue and let me know your case. The core logic is not yet well-tested with real packages. We need edge cases.

Why only works in venv?

This tool makes a correspondence from package names to module names. This use importlib and requires an environment that has all packages installed. This tool is designed to be used in CI. So, runnable under poetry run is enough.

Why installable with python >= 3.6 and runnable only in python >= 3.9?

See the comment in pyproject.toml. I tried to make it runnable in old pythons, but the cost is high. This tool is designed to be used in CI. So, this restriction is reasonable.

How to develop

How to release

  1. Bump version, PR and merge.
  2. git tag <version> then git push origin <version>.
  3. CI will pubish the package to PyPI: https://pypi.org/project/pyproject-indirect-import-detector/

Project details


Download files

Download the file for your platform. If you're not sure which to choose, learn more about installing packages.

Source Distribution

pyproject-indirect-import-detector-0.1.2.tar.gz (12.7 kB view details)

Uploaded Source

Built Distribution

If you're not sure about the file name format, learn more about wheel file names.

File details

Details for the file pyproject-indirect-import-detector-0.1.2.tar.gz.

File metadata

File hashes

Hashes for pyproject-indirect-import-detector-0.1.2.tar.gz
Algorithm Hash digest
SHA256 27d5c893cd737ff420871fae67e5a84bdf7d46de5108bad8c42c0e4e96deda2b
MD5 42a886a480b3009771d61480924fb380
BLAKE2b-256 defc4b303f35581044ebe07681f9a459d5f7dcaac61e315f25dfa6d0202bb366

See more details on using hashes here.

File details

Details for the file pyproject_indirect_import_detector-0.1.2-py3-none-any.whl.

File metadata

File hashes

Hashes for pyproject_indirect_import_detector-0.1.2-py3-none-any.whl
Algorithm Hash digest
SHA256 26d4d5038a050801c825d0801c4eab3bdd2a73fec6ded1c75a6f21c760926d47
MD5 8664445d09a1c6611f4fade8e092efe2
BLAKE2b-256 fec9cd244d1b260c4bbff28600df80dc4cc359a8f2bce22ab9224fc1dacb36d9

See more details on using hashes here.

Supported by

AWS Cloud computing and Security Sponsor Datadog Monitoring Depot Continuous Integration Fastly CDN Google Download Analytics Pingdom Monitoring Sentry Error logging StatusPage Status page