Update pre-commit, do not run integration tests for flaky objectives

.pre-commit-config.yaml

@@ -15,7 +15,7 @@ exclude: ^src/external_dependencies/
 repos:
   # Standard hooks
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v3.4.0
+    rev: v6.0.0
     hooks:
       - id: check-ast
       - id: check-case-conflict
@@ -31,12 +31,12 @@ repos:
       - id: fix-byte-order-marker
 
   - repo: https://github.com/psf/black
-    rev: 22.3.0
+    rev: 25.9.0
     hooks:
       - id: black
 
   - repo: https://github.com/codespell-project/codespell
-    rev: v2.0.0
+    rev: v2.4.1
     hooks:
       - id: codespell
         args: ["--write-changes", "-L", "atleast,inout,ether"] # Provide a comma-separated list of misspelled words that codespell should ignore (for example: '-L', 'word1,word2,word3').
@@ -51,7 +51,7 @@ repos:
         args: ["-fallback-style=none"]
 
   - repo: https://github.com/adrienverge/yamllint
-    rev: v1.27.1
+    rev: v1.37.1
     hooks:
       - id: yamllint
         args:
@@ -64,13 +64,13 @@ repos:
         files: \.(yml|yaml)$
 
   - repo: https://github.com/tcort/markdown-link-check
-    rev: v3.10.3
+    rev: v3.14.1
     hooks:
       - id: markdown-link-check
 
   # NOTE: Broken on arm64. Will need to bump once https://github.com/hadolint/hadolint/issues/840 is fixed.
   - repo: https://github.com/hadolint/hadolint
-    rev: v2.11.0
+    rev: v2.14.0
     hooks:
       - id: hadolint
         name: "Lint Dockerfiles"
@@ -79,7 +79,7 @@ repos:
         entry: hadolint/hadolint:2.11.0 hadolint
 
   - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: "v3.1.0"
+    rev: "v4.0.0-alpha.8"
     hooks:
       # Use Prettier to lint XML files because, well.. its rules are prettier than most linters, as the name implies.
       # Also we use it in the UI, so it's familiar.

src/factory_sim/objectives/drop_bracket_part_on_jig_subtree.xml

@@ -40,7 +40,7 @@
         timeout="-1.000000"
       />
       <Action ID="SetupMTCCurrentState" task="{mtc_task}" />
-      <!--The arm is moved to this intermediate waypoint to give a more predicable arm motion while avoiding the hanging bowls. Even with a collision object modeled the arm would still make large motions at times which influenced how well the part was aligned to the jig posts.-->
+      <!--The arm is moved to this intermediate waypoint to give a more predictable arm motion while avoiding the hanging bowls. Even with a collision object modeled the arm would still make large motions at times which influenced how well the part was aligned to the jig posts.-->
       <Action
         ID="SetupMTCPlanToJointState"
         acceleration_scale_factor="1"

src/hangar_sim/launch/sim/localization_launch.py

@@ -39,6 +39,7 @@
 from launch_ros.descriptions import ComposableNode, ParameterFile
 from nav2_common.launch import RewrittenYaml
 
+
 # This file is adapted from: https://github.com/ros-navigation/navigation2/blob/humble/nav2_bringup/launch/localization_launch.py
 def generate_launch_description():
     # Get the launch directory

src/lab_sim/test/objectives_integration_test.py

@@ -74,8 +74,11 @@
     "Stack Blocks with ICP",  # Skipped because there is no primary ui to switch to in ci
     "Teleoperate",
     "Inspector",
+    "Pick 1 Pill Bottle",
+    "AddBottlesToPlanningScene",
 }
 
+
 # This is a workaround to avoid the test running before ros control is ready
 def wait_for_gripper_action_server(timeout: int = 180):
     node = rclpy.create_node("test_node")