fix: reconcile required-tools.txt smoke checks (S04.5)
Deploy Claude Dev (Dev) / deploy-claude-dev-dev (push) Successful in 1m32s
Run Tests / Secret Detection (pull_request) Failing after 1m0s
Run Tests / Backend Tests (pull_request) Failing after 2m33s
Run Tests / Frontend Tests (pull_request) Failing after 27s

- Replace openssh-client with ssh (checkable binary name)
- Remove socat (not a claude-dev container dependency)
- Handle ca-certificates as special package check in smoke-tools.sh
- Add SPECIAL_CHECKS array for packages without matching binaries

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
Gan, Jimmy
2026-05-03 18:13:59 +08:00
parent 0c3d2f854b
commit 59bdcf392c
2 changed files with 12 additions and 4 deletions
+1 -3
View File
@@ -1,6 +1,6 @@
bash bash
ca-certificates ca-certificates
openssh-client ssh
gh gh
git git
jq jq
@@ -19,5 +19,3 @@ make
tmux tmux
tea tea
claude claude
socat
+11 -1
View File
@@ -8,11 +8,21 @@ if [[ ! -f "$TOOLS_FILE" ]]; then
exit 1 exit 1
fi fi
# Special packages that aren't standalone binaries
declare -A SPECIAL_CHECKS=(
["ca-certificates"]="test -f /etc/ssl/certs/ca-certificates.crt"
)
missing=() missing=()
while IFS= read -r tool; do while IFS= read -r tool; do
[[ -z "$tool" ]] && continue [[ -z "$tool" ]] && continue
[[ "$tool" =~ ^# ]] && continue [[ "$tool" =~ ^# ]] && continue
if ! command -v "$tool" >/dev/null 2>&1; then
if [[ -n "${SPECIAL_CHECKS[$tool]:-}" ]]; then
if ! eval "${SPECIAL_CHECKS[$tool]}" >/dev/null 2>&1; then
missing+=("$tool")
fi
elif ! command -v "$tool" >/dev/null 2>&1; then
missing+=("$tool") missing+=("$tool")
fi fi
done < "$TOOLS_FILE" done < "$TOOLS_FILE"