Skip to content

Teleop should support multiple frontend mounts #20

Description

@MNikoliCC

Moved from SpesRobotics/teleop-pro#9 because the fix belongs in the public teleop core.

Summary

teleop-pro can pass multiple frontend mounts to teleop.Teleop so the backend can serve the operator UI at /, the operator alias at /teleop, and the spectator TV UI at /tv.

The current teleop core expects frontend_dir to be a single path. When teleop-pro passes a mount list, backend startup crashes.

Observed error

TypeError: expected str, bytes or os.PathLike object, not list

Expected behavior

The core teleop.Teleop serving hook should remain backward compatible with a single frontend directory and also support a list of (mount_path, frontend_dir) pairs.

Acceptance criteria

  • frontend_dir="/path/to/dist" still serves the legacy UI at /.
  • frontend_dir=[("/", operator), ("/teleop", operator), ("/tv", tv)] serves each frontend and its assets correctly.
  • /ws and downstream product routers continue to work unchanged.
  • teleop-pro can remove its local compatibility patch once the core branch/commit is used.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingteleop proTeleop Pro product-related integration

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions