The pipe into validate_examples_parallel creates a subshell, so
TOTAL_VALIDATED and TOTAL_FAILED set inside it are lost. Now we
call aggregate_results again in the main shell after the pipeline
completes to properly count the results from the JSON files.
The script was checking for legacy UNSANDBOX_API_KEY but CI has
UNSANDBOX_PUBLIC_KEY and UNSANDBOX_SECRET_KEY set. Updated the
credential detection to recognize both HMAC and legacy auth.
This fixes the misleading warning 'UNSANDBOX_API_KEY not set'
when examples ARE actually running with valid HMAC credentials.
The wait -n + pid array removal was buggy - it removed the first
pid from the array when any job finished, not the one that actually
completed. This caused the final wait loop to miss some processes.
Fix: Use bare 'wait' at the end which waits for ALL background
processes, regardless of what's in the pid array.
lint-all-sdks.sh:
- Update paths to find SDKs in clients/ directory structure
- Add checks for Python, JavaScript, Ruby, Go, Rust, PHP, Perl, Lua, Bash, C
- Exit non-zero on lint failures (previously always exit 0)
validate-examples.sh:
- Fix race condition with parallel execution - aggregate results from temp files
after all jobs complete (subshell variables don't propagate to parent)
- Add aggregate_results() function to collect stats from result JSON files
Python async SDK:
- Make aiohttp import optional with DependencyError exception
- Add _check_aiohttp() helper for clear error messages
Python examples (async + sync):
- Exit with code 0 when API keys missing (CI-friendly skip)
- Change "Error:" to "Skipping:" for missing credentials
- Wrap un_async imports in try/except for aiohttp ImportError
