create_package_from_pep610 breaks without "requested_revision"

• [ x] I am on the latest Poetry version.
• [ x ] I have searched the issues of this repo and believe that this is not a duplicate.
• [ x ] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

• OS version and name: macOS 10.15
• Poetry version: 1.2.0a1

Issue

Any poetry command which hits create_package_from_pep610 when a URL installed package is installed, and the direct_url.json lacks “requested_revision”, causes a KeyError. This is not PEP610 compliant:

https://github.com/python-poetry/poetry/blob/527882380fd7aecd7c34922a7bc888226a5737ff/poetry/repositories/installed_repository.py#L206

A requested_revision key (type string) MAY be present naming a branch/tag/ref/commit/revision/etc (in a format compatible with the VCS) to install.

https://www.python.org/dev/peps/pep-0610/#:~:text=a requested_revision key (type string) may be present naming a branch%2Ftag%2Fref%2Fcommit%2Frevision%2Fetc (in a format compatible with the vcs) to install.

The offending direct_url.json:

{"url": "https://github.com/encode/uvicorn.git", "vcs_info": {"commit_id": "62825d3c1c2897e414b72318a079d0d8657ade34", "vcs": "git"}}

Steps to reproduce

docker run -it python:3.9.6 bash  # or some other fresh env
pip install poetry-version-plugin  # install a plugin and poetry 1.2.0a1
poetry plugin show
# works
pip install 'git+https://github.com/encode/uvicorn.git'  # install offending package
poetry plugin show 

Trace:

  KeyError

  'requested_revision'

  at /usr/local/lib/python3.9/site-packages/poetry/repositories/installed_repository.py:201 in create_package_from_pep610
      197│         elif "vcs_info" in url_reference:
      198│             # VCS distribution
      199│             source_type = url_reference["vcs_info"]["vcs"]
      200│             source_url = url_reference["url"]
    → 201│             source_reference = url_reference["vcs_info"]["requested_revision"]
      202│             source_resolved_reference = url_reference["vcs_info"]["commit_id"]
      203│
      204│         package = Package(
      205│             distribution.metadata["name"],
root@d98543c446e4:/# poetry plugin show -vvv

  Stack trace:

  10  /usr/local/lib/python3.9/site-packages/cleo/application.py:314 in run
       312│
       313│             try:
     → 314│                 exit_code = self._run(io)
       315│             except Exception as e:
       316│                 if not self._catch_exceptions:

   9  /usr/local/lib/python3.9/site-packages/poetry/console/application.py:167 in _run
       165│         self._load_plugins(io)
       166│
     → 167│         return super()._run(io)
       168│
       169│     def _configure_io(self, io: IO) -> None:

   8  /usr/local/lib/python3.9/site-packages/cleo/application.py:409 in _run
       407│                 io.set_input(ArgvInput(argv))
       408│
     → 409│         exit_code = self._run_command(command, io)
       410│         self._running_command = None
       411│

   7  /usr/local/lib/python3.9/site-packages/cleo/application.py:451 in _run_command
       449│
       450│         if error is not None:
     → 451│             raise error
       452│
       453│         return event.exit_code

   6  /usr/local/lib/python3.9/site-packages/cleo/application.py:435 in _run_command
       433│
       434│             if event.command_should_run():
     → 435│                 exit_code = command.run(io)
       436│             else:
       437│                 exit_code = ConsoleCommandEvent.RETURN_CODE_DISABLED

   5  /usr/local/lib/python3.9/site-packages/cleo/commands/base_command.py:118 in run
       116│         io.input.validate()
       117│
     → 118│         status_code = self.execute(io)
       119│
       120│         if status_code is None:

   4  /usr/local/lib/python3.9/site-packages/cleo/commands/command.py:85 in execute
        83│
        84│         try:
     →  85│             return self.handle()
        86│         except KeyboardInterrupt:
        87│             return 1

   3  /usr/local/lib/python3.9/site-packages/poetry/console/commands/plugin/show.py:42 in handle
        40│
        41│         system_env = EnvManager.get_system_env(naive=True)
     →  42│         installed_repository = InstalledRepository.load(
        43│             system_env, with_dependencies=True
        44│         )

   2  /usr/local/lib/python3.9/site-packages/poetry/repositories/installed_repository.py:247 in load
       245│                     continue
       246│
     → 247│                 package = cls.create_package_from_distribution(distribution, env)
       248│
       249│                 if with_dependencies:

   1  /usr/local/lib/python3.9/site-packages/poetry/repositories/installed_repository.py:111 in create_package_from_distribution
       109│             and path.joinpath("direct_url.json").exists()
       110│         ):
     → 111│             return cls.create_package_from_pep610(distribution)
       112│
       113│         is_standard_package = env.is_path_relative_to_lib(path)

  KeyError

  'requested_revision'

  at /usr/local/lib/python3.9/site-packages/poetry/repositories/installed_repository.py:201 in create_package_from_pep610
      197│         elif "vcs_info" in url_reference:
      198│             # VCS distribution
      199│             source_type = url_reference["vcs_info"]["vcs"]
      200│             source_url = url_reference["url"]
    → 201│             source_reference = url_reference["vcs_info"]["requested_revision"]
      202│             source_resolved_reference = url_reference["vcs_info"]["commit_id"]
      203│
      204│         package = Package(
      205│             distribution.metadata["name"],

[hwalinga]

This is not present in 1.1.7, but is in the preview release 1.2.0a2.

[dimbleby]

fixed at #5274