Skip to content

t3491: group local declarations in supervisor-cron auto-batch block #4426

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
alex-solovyev merged 1 commit into from
Mar 13, 2026
Merged

t3491: group local declarations in supervisor-cron auto-batch block #4426

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 7 additions & 6 deletions .agents/scripts/supervisor-archived/cron.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1032,23 +1032,24 @@ cmd_auto_pickup() {

if [[ -n "$still_unbatched" ]]; then
# Create a new auto-batch for remaining unbatched tasks (t1314)
local auto_batch_name
# Declarations grouped; assigned separately per SC2155.
local auto_batch_name auto_cores auto_base_concurrency
local task_csv still_unbatched_count auto_batch_id

auto_batch_name="auto-$(date +%Y%m%d-%H%M%S)"
local task_csv
task_csv=$(echo "$still_unbatched" | tr '\n' ',' | sed 's/,$//')

# Derive base concurrency from CPU cores (cores / 2, min 2)
# A 10-core Mac gets 5, a 32-core server gets 16, etc.
# The adaptive scaling in calculate_adaptive_concurrency() then
# adjusts up/down from this base depending on actual load.
local auto_cores
auto_cores="$(get_cpu_cores)"
local auto_base_concurrency=$((auto_cores / 2))
auto_base_concurrency=$((auto_cores / 2))
if [[ "$auto_base_concurrency" -lt 2 ]]; then
auto_base_concurrency=2
fi
local still_unbatched_count

still_unbatched_count=$(echo "$still_unbatched" | wc -l | tr -d ' ')
local auto_batch_id
auto_batch_id=$(cmd_batch "$auto_batch_name" --concurrency "$auto_base_concurrency" --tasks "$task_csv" 2>>"${SUPERVISOR_LOG:-/dev/null}")
if [[ -n "$auto_batch_id" ]]; then
log_success "Auto-batch: created '$auto_batch_name' ($auto_batch_id) with $still_unbatched_count tasks (t1314)"
Expand Down
Loading