Feature implementation from commits 2beb8b0..b34810b

[yashuatla]

PR Summary

Add importlib_metadata package with APIs for accessing third-party package metadata

Overview

This PR introduces a new Python module importlib_metadata that provides APIs for accessing metadata from third-party Python packages, including classes for package discovery, metadata extraction, and entry point handling. It also includes utility modules, test fixtures, and documentation configuration.

Change Types

Type	Description
Feature	New importlib_metadata module with APIs for package metadata access
Feature	Utility modules for collections, functions, and path handling
Feature	Diagnostic tools for inspecting package distributions
Enhancement	Sphinx documentation configuration with improved linking
Test	Performance testing functions and test fixtures

---

Affected Modules

Module / File	Change Description
__init__.py	Core importlib_metadata module with APIs for package metadata access
_collections.py	FreezableDefaultDict and Pair classes for collections handling
_functools.py	Utility functions from jaraco.functools including method_cache
_meta.py	Protocol classes for package metadata and path handling
_path.py	Filesystem utilities for symlinks and file structure building
_context.py	'suppress' class for exception suppression with decorator support
diagnose.py	Diagnostic functions for inspecting package distributions
fixtures.py	Test fixtures and utilities for testing environments
exercises.py	Performance testing functions for importlib_metadata functionality
conf.py	Sphinx documentation configuration with Python issue linking

[codeowlai]

🐛 Correctness Issue

Potential double function call in missing method.

If _frozen attribute doesn't exist and super().missing returns a callable, it will be called twice with the same key, potentially causing unexpected behavior.

Current Code (Diff):

-         return getattr(self, '_frozen', super().__missing__)(key)
+         if hasattr(self, '_frozen'):
+             return self._frozen(key)
+         return super().__missing__(key)

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	return getattr(self, '_frozen', super().__missing__)(key)
	if hasattr(self, '_frozen'):
	return self._frozen(key)
	return super().__missing__(key)

[codeowlai]

🐛 Correctness Issue

Missing error handling for malformed input.

The parse method will raise ValueError if text doesn't contain '=' character, causing potential runtime exceptions when processing malformed input.

Current Code (Diff):

-         return cls(*map(str.strip, text.split("=", 1)))
+         try:
+             return cls(*map(str.strip, text.split("=", 1)))
+         except ValueError:
+             return cls(text.strip(), "")

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	return cls(*map(str.strip, text.split("=", 1)))
	try:
	return cls(*map(str.strip, text.split("=", 1)))
	except ValueError:
	return cls(text.strip(), "")

🔄 Dependencies Affected

importlib_metadata/init.py

Function: Unknown

Issue: Calls to Pair.parse with malformed input will cause runtime exceptions

Suggestion: Add error handling around the Pair.parse call or ensure input is properly formatted

---

importlib_metadata/_collections.py

Function: Unknown

Issue: Calls to Pair.parse with malformed input will cause runtime exceptions

Suggestion: Add error handling around the Pair.parse call or ensure input is properly formatted

---

[codeowlai]

🐛 Correctness Issue

Missing implementation in Protocol method.

The second get_all overload is missing the required '...' or 'pass' implementation, which will cause a syntax error when the file is executed.

Current Code (Diff):

-    def get_all(self, name: str, failobj: _T) -> Union[List[Any], _T]:
-        """
-        Return all values associated with a possibly multi-valued key.
-        """
+    def get_all(self, name: str, failobj: _T) -> Union[List[Any], _T]:
+        """
+        Return all values associated with a possibly multi-valued key.
+        """
+        ...  # pragma: no cover

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	def get_all(self, name: str, failobj: _T) -> Union[List[Any], _T]:
	def get_all(self, name: str, failobj: _T) -> Union[List[Any], _T]:
	"""
	Return all values associated with a possibly multi-valued key.
	"""
	... # pragma: no cover