It's only used from __init__.py

So that's where it should live.

find_bad_motion_tracks/__init__.py

@@ -26,7 +26,7 @@
     UILayout,
 )
 
-from .logic import get_active_clip, find_bad_tracks, find_duplicate_tracks
+from .logic import find_bad_tracks, find_duplicate_tracks
 
 FIND_BAD_TRACKS = "Find Bad Tracks"
 
@@ -125,6 +125,12 @@ def draw_item(
         layout.label(text=f"{duplicateItem.track1_name} & {duplicateItem.track2_name}")
 
 
+def get_active_clip(context: bpy.types.Context):
+    spaces = cast(bpy.types.AreaSpaces, context.area.spaces)
+    active = cast(bpy.types.SpaceClipEditor, spaces.active)
+    return active.clip
+
+
 class OP_Tracking_find_bad_tracks(bpy.types.Operator):
     """
     Identify bad tracks by looking at how they move relative to other tracks.

find_bad_motion_tracks/logic.py

@@ -11,7 +11,6 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 
-import bpy
 import statistics
 
 from dataclasses import dataclass
@@ -329,9 +328,3 @@ def find_duplicate_tracks(clip: MovieClip) -> Iterable[Duplicate]:
 
     # Return the track pairs that come close enough at some point
     return filter(lambda dup: dup.are_dups(), dups.values())
-
-
-def get_active_clip(context: bpy.types.Context):
-    spaces = cast(bpy.types.AreaSpaces, context.area.spaces)
-    active = cast(bpy.types.SpaceClipEditor, spaces.active)
-    return active.clip