Skip to content

fix(integrations/google-adk): add py.typed sentinel for PEP 561 compliance#552

Open
FBISiri wants to merge 1 commit into
getzep:mainfrom
FBISiri:fix/adk-py-typed-pep561
Open

fix(integrations/google-adk): add py.typed sentinel for PEP 561 compliance#552
FBISiri wants to merge 1 commit into
getzep:mainfrom
FBISiri:fix/adk-py-typed-pep561

Conversation

@FBISiri

@FBISiri FBISiri commented Jun 30, 2026

Copy link
Copy Markdown

Summary

Add an empty py.typed marker file to integrations/adk/python/src/zep_adk/, enabling downstream mypy/pyright consumers to resolve types from the installed package inline — no stubs required.

What changed

File Change
integrations/adk/python/src/zep_adk/py.typed New empty sentinel file (PEP 561)
integrations/adk/python/pyproject.toml Added clarifying comment in [tool.hatch.build.targets.wheel]

Why

PEP 561 requires a py.typed marker for a package to be considered a typed package. Without it, mypy and pyright skip inline type checking and fall back to stubs or Any. The zep-adk package already has full type annotations and a strict mypy config (disallow_untyped_defs = true) — the marker just makes that signal legible to downstream consumers.

Impact

  • No logic changes — empty marker file only.
  • No new dependencies.
  • CI should pass as-is — hatchling auto-includes all files within the declared package directory (src/zep_adk), so no build configuration changes are required.
  • Downstream users who pip install zep-adk will immediately benefit from inline type resolution.

Related

…iance

Add an empty py.typed marker file to src/zep_adk/ so that downstream
mypy/pyright consumers of the zep-adk package can resolve types inline
without stubs, per PEP 561.

Hatchling automatically includes all files under the declared package
directory (src/zep_adk), so no additional build-config changes are
required beyond the sentinel file itself. A clarifying comment is added
to pyproject.toml.

No logic changes. CI should pass as-is.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant