434: Static Type Checker Mypy added to CI & as a poe task in pyproject.toml

[adigidh]

Which issue(s) does this pull-request address?

Closes: #434

Description

Added MyPy type checking as a poe task and as a CI workflow action. Added in tasks for different type checking scenarios.

type-check = "mypy ."
type-check-strict = "mypy --strict ."
type-check-verbose = "mypy --verbose --show-error-context --pretty ."

Captured examples of failures in the Git Actions: https://github.com/i-am-bee/beeai-framework/actions/runs/13707535290/job/38336396042?pr=507

Example failures while running these locally: poetry run poe type-check-strict

Checklist

• I have read the contributor guide
• I have signed off on my commit
• Linting passes: yarn lint or yarn lint:fix
• Formatting is applied: yarn format or yarn format:fix
• Unit tests pass: yarn test:unit
• E2E tests pass: yarn test:e2e
• Tests are included
• Documentation is changed or added
• Commit messages and PR title follow conventional commits

[ajbozarth]

This looks great, would you be willing to look into addressing the top level mypy failures while you're at it. We should be able to config mypy in pyproject.toml to allow duplicate module names in separate subdirectories.

[adigidh]

This looks great, would you be willing to look into addressing the top level mypy failures while you're at it. We should be able to config mypy in pyproject.toml to allow duplicate module names in separate subdirectories.

@ajbozarth : Initially I updated mypy’s configuration to allow for duplicate module names in separate subdirectories.

namespace_packages = true: Allows mypy to recognize namespaces without requiring __init__.py files in every directory.

explicit_package_bases = true: Ensures mypy uses the directory structure (relative to the root).

But it complained about other things with "Source file found twice under different module names". Therefore added empty __init__.py files to enforce a namespace, offering a cleanest way to fix both issues spotted earlier. Let me know what you think!

[ajbozarth]

But it complained about other things with "Source file found twice under different module names". Therefore added empty init.py files to enforce a namespace, offering a cleanest way to fix both issues spotted earlier. Let me know what you think!

Thanks, I checked it out and played with it a bit and I think you only need the init.py at the root of examples to fix it, so you can delete the others. Otherwise I think this is g2g

[ajbozarth]

I think this should work like poe lint and poe format, so this would be [tool.poe.tasks.type-check] and reformat the following lines to work similar to the other poe.tasks section above