feat: add python sys.path to vyper path

[charles-cooper]

this allows users to install packages from pip and import them using a regular python workflow.

What I did

opening pandora's box

How I did it

inject sys.path into the vyper compiler's search path

How to verify it

pip install snekmate
cat > test.vy <<EOF
from auth import AccessControl as acl  # AccessControl.vy is in snekmate
EOF
vyper test.vy

Commit message

this makes it easier to install vyper packages from pip and import them
using a regular python workflow.

misc:
- improve how paths appear in error messages; try hard to make them
  relative paths.
- add `chdir_tmp_path` fixture which chdirs to the `tmp_path` fixture
  for the duration of the test.

Description for the changelog

Cute Animal Picture

[fubuloubu]

Can the package name be used as the base of the namespace?

e.g. from snekmate.auth import ...

[charles-cooper]

Can the package name be used as the base of the namespace?

e.g. from snekmate.auth import ...

that's an issue for the package maintainer -- snekmate is currently installing auth to the top- level of site-packages/. better hygiene would be for it to be installed to site-packages/snekmate/ (cc. @pcaversaccio ), then it would be imported as from snekmate.auth import AccessControl.

not valid concern

[codecov-commenter]

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (a2eb60c) 84.95% compared to head (ee369cc) 69.61%.

❗ Current head ee369cc differs from pull request most recent head 7887ce9. Consider uploading reports for the commit 7887ce9 to get more accurate results

Files	Patch %	Lines
vyper/cli/vyper_compile.py	0.00%	3 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #3763       +/-   ##
===========================================
- Coverage   84.95%   69.61%   -15.34%     
===========================================
  Files          92       92               
  Lines       13595    13135      -460     
  Branches     3061     2935      -126     
===========================================
- Hits        11549     9144     -2405     
- Misses       1560     3319     +1759     
- Partials      486      672      +186     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pcaversaccio]

Can the package name be used as the base of the namespace?
e.g. from snekmate.auth import ...

that's an issue for the package maintainer -- snekmate is currently installing auth to the top- level of site-packages/. better hygiene would be for it to be installed to site-packages/snekmate/ (cc. @pcaversaccio ), then it would be imported as from snekmate.auth import AccessControl.

This is correct - I opened a PR to fix this pattern in snekmate: pcaversaccio/snekmate#204 (@charles-cooper can you quickly have a look). If you use the fixed pattern via:

pip install -i https://test.pypi.org/simple/ snekmate==0.0.5rc2

and the introduced changes in this PR, you will get:

Error compiling: Test.vy
vyper.exceptions.ModuleNotFound: vyper.interfaces.ERC165
  contract "/mnt/c/Users/pasca/Desktop/vyper/venvU/lib/python3.11/site-packages/snekmate/auth/AccessControl.vy:54", line 54:0
       53 # which is a built-in interface of the Vyper compiler.
  ---> 54 from vyper.interfaces import ERC165
  --------^
       55 implements: ERC165

The compilation error currently is due to the recent change introduced here #3741 (I would need to refactor this to from ethereum.ercs import ERC165 in the next release), but it works 😄.

If I adjust the snekmate AccessControl.vy module locally to use from ethereum.ercs import ERC165, I however get the following raise:

Error compiling: Test.vy
vyper.exceptions.CompilerPanic: unhandled exception 'ModuleInfo' object has no attribute '_invalid_locations'
  contract "C:\Users\pasca\Desktop\vyper\venv\Lib\site-packages\snekmate\auth\AccessControl.vy:62", line 62:0
       61 import interfaces.IAccessControl as IAccessControl
  ---> 62 implements: IAccessControl
  --------^
       63


This is an unhandled internal compiler error. Please create an issue on Github to notify the developers!
https://github.com/vyperlang/vyper/issues/new?template=bug.md

[pcaversaccio]

Can we add one test case please?

[charles-cooper]

If I adjust the snekmate AccessControl.vy module locally to use from ethereum.ercs import ERC165, I however get the following raise:

seems like an outstanding issue

[fubuloubu]

Figure out a way to mock this for testing

[charles-cooper]

test added here eac2d2f

[charles-cooper]

If I adjust the snekmate AccessControl.vy module locally to use from ethereum.ercs import ERC165, I however get the following raise:

seems like an outstanding issue

that error message should be fixed in #3775