fix: httpie failing due to missing binary

Author: jpcaparas

.github/workflows/validate-skills.yml

@@ -9,20 +9,20 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Check Out Repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v6
 
       - name: Set Up pnpm
-        uses: pnpm/action-setup@v4
+        uses: pnpm/action-setup@v6
         with:
           run_install: false
 
       - name: Set Up Python
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@v6
         with:
           python-version: "3.11"
 
       - name: Set Up Node
-        uses: actions/setup-node@v4
+        uses: actions/setup-node@v6
         with:
           node-version: "20"
           cache: "pnpm"
@@ -31,11 +31,11 @@ jobs:
       - name: Install Node Dependencies
         run: pnpm install --frozen-lockfile
 
-      - name: Install ripgrep
-        # Required by skills/ripgrep/scripts/probe_ripgrep.py, which shells
-        # out to `rg --version` during validation. ubuntu-latest does not
-        # ship ripgrep preinstalled.
-        run: sudo apt-get update && sudo apt-get install -y ripgrep
+      - name: Install Validation Tools
+        # Required by skill probe suites that shell out to external CLIs:
+        # - skills/ripgrep/scripts/probe_ripgrep.py -> `rg --version`
+        # - skills/httpie/scripts/probe_httpie.py -> `http`, `https`, `httpie`
+        run: sudo apt-get update && sudo apt-get install -y ripgrep httpie
 
       - name: Validate Skills
         shell: bash

skills/httpie/scripts/probe_httpie.py

@@ -29,15 +29,24 @@ def run_command(
     if env:
         merged_env.update(env)
 
-    proc = subprocess.run(
-        args,
-        env=merged_env,
-        capture_output=True,
-        text=True,
-        encoding="utf-8",
-        errors="replace",
-        check=False,
-    )
+    try:
+        proc = subprocess.run(
+            args,
+            env=merged_env,
+            capture_output=True,
+            text=True,
+            encoding="utf-8",
+            errors="replace",
+            check=False,
+        )
+    except OSError as exc:
+        return {
+            "args": args,
+            "returncode": 127,
+            "stdout": "",
+            "stderr": str(exc),
+            "ok": False,
+        }
     return {
         "args": args,
         "returncode": proc.returncode,
@@ -81,6 +90,36 @@ def run_suite(*, keep_temp: bool = False) -> dict[str, object]:
     payload_path.write_text('{"name":"JP","active":true}\n', encoding="utf-8")
 
     try:
+        required_commands = ["http", "https", "httpie"]
+        missing_commands = [command for command in required_commands if shutil.which(command) is None]
+        if missing_commands:
+            detail = (
+                "Missing required HTTPie CLI commands on PATH: "
+                + ", ".join(missing_commands)
+                + ". Install HTTPie before running the probe suite."
+            )
+            checks.append(
+                {
+                    "name": "dependencies",
+                    "passed": False,
+                    "command": [],
+                    "returncode": 127,
+                    "detail": detail,
+                    "stdout": "",
+                    "stderr": detail,
+                }
+            )
+            return {
+                "passed": False,
+                "summary": {
+                    "checks_total": 1,
+                    "checks_passed": 0,
+                    "checks_skipped": 0,
+                },
+                "checks": checks,
+                "tempdir": str(temp_root) if keep_temp else None,
+            }
+
         version = run_command(["http", "--version"], env=env)
         checks.append(
             check(