chore: Remove redundant comments and fix formatting

  - Apply ruff formatting fixes
  - Update example documentation with terraform-docs

Author: pedropinho7

examples/build-package/README.md

@@ -54,6 +54,7 @@ Note that this example may create resources which cost money. Run `terraform des
 | <a name="module_package_src_poetry2"></a> [package\_src\_poetry2](#module\_package\_src\_poetry2) | ../../ | n/a |
 | <a name="module_package_with_commands_and_patterns"></a> [package\_with\_commands\_and\_patterns](#module\_package\_with\_commands\_and\_patterns) | ../../ | n/a |
 | <a name="module_package_with_docker"></a> [package\_with\_docker](#module\_package\_with\_docker) | ../../ | n/a |
+| <a name="module_package_with_docker_shell_commands"></a> [package\_with\_docker\_shell\_commands](#module\_package\_with\_docker\_shell\_commands) | ../../ | n/a |
 | <a name="module_package_with_npm_lock_in_docker"></a> [package\_with\_npm\_lock\_in\_docker](#module\_package\_with\_npm\_lock\_in\_docker) | ../../ | n/a |
 | <a name="module_package_with_npm_requirements_in_docker"></a> [package\_with\_npm\_requirements\_in\_docker](#module\_package\_with\_npm\_requirements\_in\_docker) | ../../ | n/a |
 | <a name="module_package_with_patterns"></a> [package\_with\_patterns](#module\_package\_with\_patterns) | ../../ | n/a |

package.py

@@ -920,11 +920,11 @@ def execute(self, build_plan, zip_stream, query):
                         "DOCKER TAG ID: %s -> %s", docker_image, docker_image_tag_id
                     )
                 else:
-                    # Image not found locally, try to pull it
-                    log.info("Docker image not found locally, pulling: %s", docker_image)
+                    log.info(
+                        "Docker image not found locally, pulling: %s", docker_image
+                    )
                     try:
                         check_call(docker_pull_command(docker_image))
-                        # Get the image ID after pulling
                         output = check_output(docker_image_id_command(docker_image))
                         if output:
                             docker_image_tag_id = output.decode().strip()
@@ -1022,7 +1022,6 @@ def execute(self, build_plan, zip_stream, query):
                     script = action[1]
 
                     if docker and docker_image_tag_id:
-                        # Execute shell commands in Docker container
                         if log.isEnabledFor(DEBUG2):
                             log.debug("exec shell script in docker...")
                             for line in script.splitlines():
@@ -1033,22 +1032,22 @@ def execute(self, build_plan, zip_stream, query):
                             [
                                 script,
                                 "retcode=$?",
-                                "pwd",  # Output final working directory to stdout
+                                "pwd",
                                 "exit $retcode",
                             ]
                         )
 
                         # Add chown to fix file ownership (like pip at line 1150-1154)
                         chown_mask = "{}:{}".format(os.getuid(), os.getgid())
                         full_script = "{} && {}".format(
-                            enhanced_script, shlex_join(["chown", "-R", chown_mask, "."])
+                            enhanced_script,
+                            shlex_join(["chown", "-R", chown_mask, "."]),
                         )
 
                         shell_command = [full_script]
 
-                        # Execute in Docker
                         docker_cmd = docker_run_command(
-                            sh_work_dir,  # build_root = current working directory
+                            sh_work_dir,
                             shell_command,
                             query.runtime,
                             image=docker_image_tag_id,
@@ -1076,16 +1075,17 @@ def execute(self, build_plan, zip_stream, query):
                             )
 
                         # Extract final working directory from stdout
-                        # The 'pwd' command output is in the stdout, but we need to parse it
+                        # The 'pwd' command output is in stdout, but we need to parse it
                         # because there might be other output from the script
                         output_lines = result.stdout.strip().split("\n")
                         if output_lines:
-                            # The last line should be the pwd output
                             final_pwd = output_lines[-1]
                             # Map container path back to host path
                             # Container path structure: /var/task = sh_work_dir (via volume mount)
                             if final_pwd.startswith("/var/task"):
-                                relative_path = final_pwd[len("/var/task") :].lstrip("/")
+                                relative_path = final_pwd[len("/var/task") :].lstrip(
+                                    "/"
+                                )
                                 sh_work_dir = (
                                     os.path.join(sh_work_dir, relative_path)
                                     if relative_path
@@ -1096,7 +1096,7 @@ def execute(self, build_plan, zip_stream, query):
                         log.debug("WORKDIR: %s", sh_work_dir)
 
                     else:
-                        # Execute shell commands on host (existing behavior)
+                        # Execute shell commands on host
                         with tempfile.NamedTemporaryFile(
                             mode="w+t", delete=True
                         ) as temp_file:

tests/test_zip_source.py

@@ -24,9 +24,9 @@ def test_sh_with_docker_basic():
     bpm = BuildPlanManager(args=Mock())
 
     # Mock subprocess functions
-    with patch("package.check_output") as mock_check_output, \
-         patch("package.subprocess.run") as mock_run:
-
+    with patch("package.check_output") as mock_check_output, patch(
+        "package.subprocess.run"
+    ) as mock_run:
         # Mock docker image ID lookup
         mock_check_output.return_value = b"sha256:abc123"
 
@@ -88,10 +88,9 @@ def capture_write_dirs(path, *args, **kwargs):
         subdir = os.path.join(tmpdir, "subdir")
         os.makedirs(subdir, exist_ok=True)
 
-        with patch("package.check_output") as mock_check_output, \
-             patch("package.subprocess.run") as mock_run, \
-             patch("os.getcwd", return_value=tmpdir):
-
+        with patch("package.check_output") as mock_check_output, patch(
+            "package.subprocess.run"
+        ) as mock_run, patch("os.getcwd", return_value=tmpdir):
             mock_check_output.return_value = b"sha256:abc123"
 
             # Mock docker execution returning changed working directory
@@ -163,9 +162,9 @@ def test_sh_docker_error_handling():
 
     bpm = BuildPlanManager(args=Mock())
 
-    with patch("package.check_output") as mock_check_output, \
-         patch("package.subprocess.run") as mock_run:
-
+    with patch("package.check_output") as mock_check_output, patch(
+        "package.subprocess.run"
+    ) as mock_run:
         mock_check_output.return_value = b"sha256:abc123"
 
         # Mock docker execution failure