Update import statement to include Type

[nyimbi]

Otherwise cannot import semantica

Description

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Performance improvement
• Code refactoring

Related Issues

Closes #
Fixes #

Changes Made

Testing

• Tested locally
• Added tests for new functionality
• Package builds successfully (python -m build)

Test Commands

# Build the package
pip install build
python -m build

# Optional: Run your own tests
pytest tests/

# Optional: Format code
black semantica/
isort semantica/

Documentation

• Updated relevant documentation
• Added code examples if applicable
• Updated API reference if adding new APIs
• Updated cookbook if adding new examples
• No documentation changes needed

Breaking Changes

Breaking Changes: [Yes/No]

Checklist

• My code follows the project's style guidelines
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• Package builds successfully

Additional Notes

[qodo-code-review]

Review Summary by Qodo

Add missing Type import to semantica.utils.helpers

🐞 Bug fix

Walkthroughs

Description

• Add missing Type import to fix import error
• Enables proper module import functionality

Diagram

flowchart LR
  A["typing imports"] -->|"add Type"| B["semantica/utils/helpers.py"]
  B -->|"enables"| C["Module imports successfully"]

Loading

File Changes

1. semantica/utils/helpers.py 🐞 Bug fix +1/-1

Add Type to typing imports

• Added Type to the typing imports on line 68
• Fixes import error that prevented module from being imported

semantica/utils/helpers.py

---

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📎 Requirement gaps (0)

1. Unused Type import 🐞 Bug ⛯ Reliability

Description

Type was added to semantica/utils/helpers.py but is not used anywhere in that file, which will
trigger flake8’s unused-import check (F401) in this repo’s pre-commit pipeline and can block
merging/CI.

Code

semantica/utils/helpers.py[68]

+from typing import Any, Dict, List, Optional, Type, Tuple, Union

Evidence

The PR adds Type to the typing import list, but the rest of the module contains no references to
Type. The repository enforces both isort and flake8 via pre-commit; flake8 will fail on unused
imports by default (F401 is not in the ignore list).

semantica/utils/helpers.py[59-69]
semantica/utils/helpers.py[220-520]
.pre-commit-config.yaml[28-39]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`semantica/utils/helpers.py` imports `Type` but does not use it anywhere in the module. This will fail flake8 (unused import) under the repo’s pre-commit configuration.

### Issue Context
Pre-commit runs both `isort` and `flake8` and does not ignore unused-import errors.

### Fix Focus Areas
- semantica/utils/helpers.py[68-68]
- .pre-commit-config.yaml[28-39]

### Suggested change
- Remove `Type` from the `from typing import ...` list unless you are adding code in this module that uses `Type[...]`.
- Run `isort semantica/utils/helpers.py` (or `pre-commit run -a`) to ensure import ordering is correct.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

---

ⓘ The new review experience is currently in Beta. Learn more

[qodo-code-review]

1. Unused type import 🐞 Bug ⛯ Reliability

Type was added to semantica/utils/helpers.py but is not used anywhere in that file, which will
trigger flake8’s unused-import check (F401) in this repo’s pre-commit pipeline and can block
merging/CI.

Agent Prompt

### Issue description
`semantica/utils/helpers.py` imports `Type` but does not use it anywhere in the module. This will fail flake8 (unused import) under the repo’s pre-commit configuration.

### Issue Context
Pre-commit runs both `isort` and `flake8` and does not ignore unused-import errors.

### Fix Focus Areas
- semantica/utils/helpers.py[68-68]
- .pre-commit-config.yaml[28-39]

### Suggested change
- Remove `Type` from the `from typing import ...` list unless you are adding code in this module that uses `Type[...]`.
- Run `isort semantica/utils/helpers.py` (or `pre-commit run -a`) to ensure import ordering is correct.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

[KaifAhmad1]

@nyimbi
Thanks for the contribution — really appreciate the effort to address the import issue 🙌

Currently, Type is added but not used in semantica/utils/helpers.py, which will fail our flake8 pre-commit check (unused import F401). So we can’t merge it as-is.

Please either use Type where needed or remove it and update the PR, and we’ll be happy to review again 👍