feat(remote_agent): add queue backpressure control and backlog metrics

[PiyushDatta]

Summary

• Apply WorkerPal completion d42bb866-a339-4da9-a323-7f5d26c6c67a to main_agents.
• Integrate commit c49eddb8ef44283d45db218935dad98e5c25bbe4 from refs/pushpals/agent/workerpal-281a46c7/d42bb866-a339-4da9-a323-7f5d26c6c67a.
• Worker workerpal-281a46c7 reported: Executed task and modified 1 file(s)
• Canonical task request: Update apps/remotebuddy/src/autonomous_engine.ts to read queue telemetry (queue_p95 latency threshold 1000ms and idle_worker count) before minting objectives, throttle or pause new objective creation when queue_p95 exceeds threshold or...

Motivation / Context

• Preserve and review autonomous worker output before final merge to base branch.
• Keep integration branch current with queued worker completions.

Planned Scope

• apps/remotebuddy/src/autonomous_engine.ts

Planned Validation

• Planned: bun run test:root

Changes

• Updated apps/remotebuddy/src/autonomous_engine.ts

Testing / Validation

• Planned: bun run test:root
• Worker completion summary did not include explicit command pass/fail output.

Impact / Risk

• Risk level: medium (automated worker-generated change; maintainer review required).
• No secrets or credentials are expected in this PR body.

SourceControlManager Note

• Use this worker-provided PR title/body when creating the integration PR.
• Suggested title: fix(repo): Update apps/remotebuddy/src/autonomous_engine.ts to read queue telemetry (queue_p95 latency...

Checklist

• Tests added/updated where appropriate

• Validation commands run (or noted as not run)

• Docs/comments updated if needed

• No sensitive data (secrets/tokens) committed

• Agent branch: agent/workerpal-281a46c7/d42bb866-a339-4da9-a323-7f5d26c6c67a

• Completion ref: refs/pushpals/agent/workerpal-281a46c7/d42bb866-a339-4da9-a323-7f5d26c6c67a

• Commit: c49eddb8ef44283d45db218935dad98e5c25bbe4

• Completion ID: cb6f52a6-ed90-443e-8f30-7441a03ba079

[PiyushDatta]

ReviewAgent: Changes Rejected (score 7.9/10)

Verdict: The feature is well-structured and mostly correct, but it has a few correctness and test coverage gaps that should be closed before calling it production-ready.

Issues:

• apps/remotebuddy/src/autonomous_engine.ts:1118 can throw if buildQueueBacklogMetrics receives telemetry with a missing/partial pendingByPriority object (it dereferences telemetry.pendingByPriority.interactive directly); harden this path by normalizing each priority with numeric fallback (same pattern used in fetchQueueTelemetry).
• apps/remotebuddy/src/autonomous_engine.ts:1237 marks backlog sampling as truncated when data.requests.length >= clamped; this over-reports truncation when result count exactly equals limit but total pending count is exactly that limit. Use an API-provided total/has_more flag if available, or emit truncated: data.requests.length === clamped with a documented "possible truncation" semantic.
• apps/remotebuddy/src/autonomous_engine.queue.test.ts only validates helper/controller units; there is no integration test for the main engine path to assert that queue throttling posts a blocked objective, skips enqueue, includes backlog evidence, and then resumes dispatch after backpressure clears.

This PR has been re-queued for automated fixes. A worker will address the issues above.

[PiyushDatta]

ReviewAgent: Changes Rejected (score 4.2/10)

Verdict: The PR introduces useful backlog and startup-telemetry ideas, but it contains blocking API/typing/syntax regressions that make it not production-ready.

Issues:

• Blocking type/API mismatch in buildQueueBacklogMetrics: the newly added exported function now accepts QueueBacklogTelemetry | null and returns a different shape, but the new test autonomous_engine.queue.test.ts calls buildQueueBacklogMetrics({... telemetry, pendingAutonomyBySkill, ...}) and asserts skills[...]; this is incompatible and will fail TypeScript and/or tests.
• Likely structural/syntax regression in autonomous_engine.ts around the buildSyntheticResponse return block near the responseHash/tokenUsage/latencyMs section (the diff shows malformed closing braces and extra blank lines), which risks compilation failure or altered control flow.
• Backward-incompatible StartupChecklistContext change: readBunVersion and readDockerVersion were added as required methods, but no callsite updates are included in this diff; existing context constructions will fail to compile or break at runtime.
• New startup version checks are inserted at the top of default checks and are hard fail gates; this can unexpectedly block previously valid environments without opt-out/config migration path, increasing rollout risk.
• Test coverage is incomplete for newly added queue-backlog parsing/evaluation behavior: no direct tests for normalizeQueuePriority aliases, truncation/confidence inference, unreliable telemetry handling, or fetchAutonomyBacklogCounts response-shape variants that this PR heavily depends on.

This PR has been re-queued for automated fixes. A worker will address the issues above.