Add typing to StoreType

[NolanTrem]

Important

Refactor StoreType usage by moving its definition to graph.py and updating imports across the codebase, while removing unused telemetry event classes.

• Refactoring:
  • Move StoreType from graphs.py to graph.py.
  • Update imports for StoreType in documents_router.py, graph_router.py, graph_service.py, and management_service.py.
• Code Cleanup:
  • Remove StoreType definition from graphs.py.
  • Remove unused telemetry event classes from events.py.

^{This description was created by for 99ba3c0. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 99ba3c0 in 55 seconds

More details

• Looked at 479 lines of code in 10 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. py/shared/abstractions/graph.py:140

• Draft comment:
  The StoreType enum is correctly defined here and used consistently across the codebase. This change improves type safety and clarity.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The PR introduces a new enum StoreType in py/shared/abstractions/graph.py and removes its previous definition from py/core/database/graphs.py. This change is consistent across the codebase, as seen in the imports and usage in various files. The enum is used to specify the type of storage, either GRAPHS or DOCUMENTS. The change is correctly reflected in the imports and usage across the codebase, ensuring type safety and clarity.

Workflow ID: wflow_gbnIm7q4pBcomfYR

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.