fix(add-account): unhang oauth login via fifo stdin
Логин Claude.ai подвисал навсегда: claude auth login ждёт код из браузера в stdin, а хук запускал его в фоне с </dev/null, куда код вставить невозможно. Теперь /add-account поднимает логин со stdin из fifo и печатает ссылку, а /add-account <код> отдаёт код, дожидается обмена и сохраняет новый аккаунт по email из токена. Провал логина больше не маскируется под успешное сохранение текущего аккаунта, а фоновые процессы гасятся строго по своим PID вместо широкого pkill -f. Поведение закреплено пятью тестами в tests/test_fixes.sh.
This commit is contained in:
@@ -7,6 +7,7 @@ set -euo pipefail
|
||||
|
||||
SCRIPT="$(cd "$(dirname "$0")/.." && pwd)/scripts/ai-setup.sh"
|
||||
GLOBAL_RULES_SOURCE="$(cd "$(dirname "$0")/.." && pwd)/home-configs/GLOBAL_RULES.md"
|
||||
ADD_HOOK="$(cd "$(dirname "$0")/.." && pwd)/home-configs/claude/hooks/add-account-hook.sh"
|
||||
PASS=0; FAIL=0
|
||||
|
||||
ok() { echo "[PASS] $1"; PASS=$((PASS+1)); }
|
||||
@@ -231,6 +232,50 @@ test_opencode_models() {
|
||||
fi
|
||||
}
|
||||
|
||||
# ── add-account: логину нужен tty, фоновый запуск висит вечно ────────────────
|
||||
test_add_account_no_headless_login() {
|
||||
if grep -q 'claude auth login.*</dev/null' "$ADD_HOOK"; then
|
||||
fail "add-account: логин запускается с закрытым stdin — код из браузера вставить некуда"
|
||||
else
|
||||
ok "add-account: логин не запускается в фоне с закрытым stdin"
|
||||
fi
|
||||
}
|
||||
|
||||
test_add_account_single_terminal_flow() {
|
||||
if grep -q 'mkfifo' "$ADD_HOOK" \
|
||||
&& grep -q 'add-account <код>' "$ADD_HOOK" \
|
||||
&& ! grep -q 'gnome-terminal' "$ADD_HOOK"; then
|
||||
ok "add-account: двухшаговый флоу в одном терминале, без второго окна"
|
||||
else
|
||||
fail "add-account: логин должен идти через fifo в текущем терминале, без отдельного окна"
|
||||
fi
|
||||
}
|
||||
|
||||
test_add_account_kills_by_pid() {
|
||||
if grep -v '^[[:space:]]*#' "$ADD_HOOK" | grep -q 'pkill'; then
|
||||
fail "add-account: pkill -f выкосит любой процесс с этой подстрокой в cmdline"
|
||||
else
|
||||
ok "add-account: процессы гасятся по своим PID, без широкого pkill -f"
|
||||
fi
|
||||
}
|
||||
|
||||
test_add_account_checks_login_result() {
|
||||
if grep -q 'Login failed' "$ADD_HOOK" \
|
||||
&& ! grep -q 'SAVED:' "$ADD_HOOK"; then
|
||||
ok "add-account: результат логина проверяется, провал не выдаётся за успех"
|
||||
else
|
||||
fail "add-account: провал логина не должен маскироваться под успешное сохранение"
|
||||
fi
|
||||
}
|
||||
|
||||
test_add_hook_syntax() {
|
||||
if bash -n "$ADD_HOOK" 2>&1; then
|
||||
ok "syntax: add-account-hook.sh passes 'bash -n'"
|
||||
else
|
||||
fail "syntax: add-account-hook.sh has syntax errors"
|
||||
fi
|
||||
}
|
||||
|
||||
# ── run all tests ─────────────────────────────────────────────────────────────
|
||||
test_script_syntax
|
||||
test_gpt_autoinstall
|
||||
@@ -252,6 +297,11 @@ test_opencode_claude_launcher
|
||||
test_opencode_routatic_proxy
|
||||
test_opencode_key_management
|
||||
test_opencode_models
|
||||
test_add_hook_syntax
|
||||
test_add_account_no_headless_login
|
||||
test_add_account_single_terminal_flow
|
||||
test_add_account_kills_by_pid
|
||||
test_add_account_checks_login_result
|
||||
|
||||
echo ""
|
||||
echo "Results: $PASS passed, $FAIL failed"
|
||||
|
||||
Reference in New Issue
Block a user