[RSDK-10385] Remove unhelpful typedef as well

Author: acmorrow

src/viam/sdk/services/motion.hpp

@@ -115,11 +115,8 @@ class Motion : public Service {
 
     /// @brief An individual "step", representing the state each component (keyed as a fully
     /// qualified component name) should reach while executing that step of the plan.
-    using step = std::unordered_map<std::string, pose>;
-
-    /// @brief An ordered list of plan steps.
     /// @ingroup Motion
-    using steps = std::vector<step>;
+    using step = std::unordered_map<std::string, pose>;
 
     /// @struct plan
     /// @brief Describes a motion plan.
@@ -136,7 +133,7 @@ class Motion : public Service {
         std::string execution_id;
 
         /// @brief An ordered list of plan steps.
-        steps steps;
+        std::vector<step> steps;
 
         friend bool operator==(const plan& lhs, const plan& rhs);
     };

src/viam/sdk/services/private/motion_client.cpp

@@ -137,9 +137,9 @@ Motion::plan_status_with_id from_proto(const service::motion::v1::PlanStatusWith
     return pswi;
 }
 
-Motion::steps steps_from_proto(
+std::vector<Motion::step> steps_from_proto(
     const google::protobuf::RepeatedPtrField<service::motion::v1::PlanStep>& proto) {
-    Motion::steps steps;
+    std::vector<Motion::step> steps;
     for (const auto& ps : proto) {
         Motion::step step;
         for (const auto& component : ps.step()) {