feat: PACKET_ASK_EFFORT 로 기본 effort 를 두되 출처를 함께 기록한다

[ictechgy]

왜

--effort 를 매번 치지 않아도 되게 한다.

왜 사용자 TOML 이 아닌가

[providers.glm] effort = "max" 는 거절 목록 두 군데를 어긴다.

1. _assert_safe_id 가 내장 id 를 이름으로 쓰는 것 자체를 거절한다 (provider_builtin_override)
2. 사용자 overlay 는 paste 별명뿐이다 (mode != "paste" 거절, 금지 키 15개)

env 는 기존 8개와 같은 결이고 그 규칙과 무관하다.

무엇

$ PACKET_ASK_EFFORT=high packet-ask review --provider glm --preview ...
... timeout=1800s(auto,applies) ... effort=high(env) ...

$ PACKET_ASK_EFFORT=max packet-ask review --provider glm --preview --effort low ...
... effort=low(explicit) ...

$ PACKET_ASK_EFFORT=ultra packet-ask review --provider glm --preview ...
PACKET_ASK_EFFORT must be one of low, medium, high, xhigh, max.   # exit 2

우선순위는 플래그 > env > 벤더 기본값. --timeout 의 explicit > auto 와 같다.

effort_source 에 env 값이 하나 더 붙는다. #56 에서 값과 출처를 나눠 둔 덕에
additive 다. GLM 이 그 리뷰에서 제안한 설계가 바로 다음 배치를 싸게 만들었다.

조용한 기본값이 되지 않게

출처 기록	effort_source 가 explicit/env/vendor-default. 이게 없으면 "왜 이 실행이 751초 걸렸나" 를 못 푼다
오타 거절	argparse choices 는 플래그만 본다. env 를 따로 안 막으면 오타가 조용히 벤더 기본값으로 떨어진다. exit 2
빈 값	"설정하지 않음". 거절하면 unset 하기가 어려워진다
우회로 차단	프로바이더 검증이 해석된 값을 본다. 플래그 경로만 막으면 env 가 우회로가 된다
자식 격리	minimal_child_env 화이트리스트라 벤더가 이 변수를 자기 설정으로 읽지 못한다. 테스트로 고정

거절 메시지가 --effort 를 지목하고 있었는데 env 로 설정한 경우엔 부정확해서
둘 다 덮게 고쳤다.

검증

uv run pytest      507 passed   (기존 501 + 신규 6)
uv build           wheel + sdist
smoke ×2           ok / ok

변이	결과
env 값 검증 제거	test_invalid_env_effort_is_rejected_not_ignored 실패
우선순위 뒤집기	test_flag_wins_over_env 외 5건 실패
프로바이더 검증이 해석 전 플래그를 봄	test_env_effort_is_rejected_for_providers_that_cannot_take_it 실패

같이 고친 표면

docs/AGENTS.md 의 환경 변수 표대로 전부 갱신했다 — .env.example,
SECURITY 영/한 변수 표, README 영/한, 번들 SKILL.md, docs/design.md 44.

알아 둘 것

SECURITY 영/한 변수 표가 이미 갈라져 있었다. 한국어 쪽에 PACKET_ASK_LEDGER
와 PACKET_ASK_LANG 이 없다. 이번 변수는 양쪽에 넣었고 기존 누락은 범위 밖이라
손대지 않았다. 별도로 정리할 대상이다.

🤖 Generated with Claude Code