feat(remote_agent): implement adjacent_possible guardrails and telemetry integration

[PiyushDatta]

Summary

• Apply WorkerPal completion bd0f07ea-b263-4246-a696-574e797dc02d to main_agents.
• Integrate commit 74a13bb7dcb0479398e1d567a085fb7eca46a833 from refs/pushpals/agent/workerpal-c1f57c76/bd0f07ea-b263-4246-a696-574e797dc02d.
• Worker workerpal-c1f57c76 reported: Executed task and modified 3 file(s)
• Canonical task request: Implement adjacent_possible inside apps/remotebuddy/src/autonomous_engine.ts to recombine proven motifs with current bottlenecks, emit clear telemetry and guardrails for its decisions, update or add supporting tests within apps/remotebud...

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): Implement adjacent_possible inside apps/remotebuddy/src/autonomous_engine.ts to recombine pro...

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-c1f57c76/bd0f07ea-b263-4246-a696-574e797dc02d

• Completion ref: refs/pushpals/agent/workerpal-c1f57c76/bd0f07ea-b263-4246-a696-574e797dc02d

• Commit: 74a13bb7dcb0479398e1d567a085fb7eca46a833

• Completion ID: 40b187a8-7619-42e4-8ff6-b46dcd13fe43

[PiyushDatta]

ReviewAgent: Changes Rejected (score 7.4/10)

Verdict: The feature implementation is substantial and mostly coherent, but the new test suite is likely not executing and there are a couple of correctness/telemetry robustness gaps that should be fixed before treating this as production-ready.

Issues:

• Critical test execution gap: apps/remotebuddy/src/autonomous_engine.adjacent_possible.test.ts wraps all tests in registerAdjacentPossibleTestSuite() and only invokes it under if (import.meta.main), which is typically false under Bun test runners, so these tests may never run in CI; move describe(...) blocks to top-level or invoke the registration unconditionally in test context.
• Telemetry accuracy bug: enforceMixGapDiversity reports duplicatesDropped as mixes.length - finalMixes.length, which includes truncation-by-limit and not just duplicate removal; this can emit misleading mix_gap_diversity_enforced data and should distinguish duplicate removals from limit truncation.
• Queue telemetry extraction is overly permissive: QUEUE_PENDING_KEY_PATHS includes generic keys like pending, which can match unrelated metadata fields and produce incorrect queue pending values; tighten accepted paths to queue-scoped keys or require queue-specific container context before accepting generic keys.

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

[PiyushDatta]

ReviewAgent: Changes Rejected (score 7.2/10)

Verdict: Feature logic is substantial and mostly coherent, but the new test file is structured in a way that likely prevents the suite from running, leaving critical guardrail behavior effectively unvalidated.

Issues:

• Critical test harness issue: apps/remotebuddy/src/autonomous_engine.adjacent_possible.test.ts defines all tests inside registerAdjacentPossibleTestSuite() and only calls it under if (import.meta.main), which is typically false under bun test; this can result in zero executed tests for this file.
• Coverage gap: there is no integration-level test asserting buildEngineInspirationContext() actually wires adjacent_possible output into adjacent_possible context and mutates the adjacent_possible building block score/evidence as intended.
• Coverage gap on failure/edge paths: new parsing logic in extractQueueTelemetryFromSignals() lacks tests for malformed/negative/non-numeric schema values and conflicting typed/evidence inputs, so sanitization and source attribution behavior is not defensibly validated.

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