fix: Implement qodo-merge-pro safety improvements

Addressed two suggestions from qodo-merge-pro review:

1. **Add explicit error handling to wrapper** (Importance: 2)
   - Changed: cd "#{prefix}" && exec
   - To: cd "#{prefix}" || exit 1; exec
   - Ensures script fails fast if directory change fails
   - Prevents execution in wrong directory if prefix is missing

2. **Use lighter test command** (Importance: 7)
   - Changed: mfc run (full simulation)
   - To: mfc count (case file parsing only)
   - Much faster test execution
   - More reliable (doesn't depend on simulation completing)
   - Still validates: wrapper, toolchain, case parsing, Python env
   - Reduces test time from minutes to seconds

Both changes improve robustness and testing efficiency.

Author: sbryngelson

packaging/homebrew/mfc.rb

@@ -48,7 +48,8 @@ def install
     (bin/"mfc").write <<~EOS
       #!/bin/bash
       export BOOST_INCLUDE="#{Formula["boost"].opt_include}"
-      cd "#{prefix}" && exec "#{libexec}/mfc.sh" "$@"
+      cd "#{prefix}" || exit 1
+      exec "#{libexec}/mfc.sh" "$@"
     EOS
     chmod 0755, bin/"mfc"
   end
@@ -88,8 +89,8 @@ def caveats
     assert_predicate libexec/"mfc.sh", :exist?
     assert_predicate prefix/"toolchain", :exist?
 
-    # Test running a simple example case to verify full toolchain
-    system bin/"mfc", "run", "#{pkgshare}/examples/1D_sodshocktube/case.py"
+    # Test that mfc can parse a case file (lighter than full simulation)
+    system bin/"mfc", "count", "#{pkgshare}/examples/1D_sodshocktube/case.py"
   end
 end