Set PYTHONPATH in olbase image to avoid manually specifying it for scripts

[cdrini]

I got tired of having to specify PYTHONPATH=. everywhere ; we can bake this into the image and then in theory never specify it again.

Technical

Testing

To test it, I added the ENV line to our oldev file, and then rebuilt it and confirmed copydocs worked without having to set the environment variable.

# Before
$ docker compose exec web ./scripts/copydocs.py --help
Traceback (most recent call last):
  File "/openlibrary/./scripts/copydocs.py", line 12, in <module>
    from scripts.solr_builder.solr_builder.fn_to_cli import FnToCLI
ModuleNotFoundError: No module named 'scripts'

# After
# Add the ENV file to oldev for testing; last line is fastest
docker compose build web
docker compose up -d --no-deps web
$ docker compose exec web ./scripts/copydocs.py --help
usage: copydocs.py [-h] [--src SRC] [--dest DEST] [--comment COMMENT]
                   [--recursive | --no-recursive] [--editions | --no-editions]
                   [--lists [LISTS ...]] [--search SEARCH] [--search-limit SEARCH_LIMIT]
                   [keys ...]
...

Needs another pass of testing to confirm the site comes up normally with this set though.

Screenshot

Stakeholders

@RayBB

[Copilot]

Pull request overview

This PR updates the olbase Docker image to set a default PYTHONPATH so repository scripts (e.g., scripts/copydocs.py) can be executed inside the container without manually prefixing commands with PYTHONPATH=..

Changes:

• Set ENV PYTHONPATH=/openlibrary in the base Docker image to make local modules importable by default.

[mekarpeles]

LGTM; @cdrini + @RayBB let me know if there's any further testing we should do to ensure it runs smoothly on prod

[RayBB]

We build and tested locally on the call.