Skip to content

fix: Batch bounty fixes — issues #2618 #2564 #2559 #2591 #2544 #2600 #2588 #2608 #2583 #2572 #2615#1

Open
crowniteto wants to merge 1 commit into
mainfrom
bounty-batch-5-22
Open

fix: Batch bounty fixes — issues #2618 #2564 #2559 #2591 #2544 #2600 #2588 #2608 #2583 #2572 #2615#1
crowniteto wants to merge 1 commit into
mainfrom
bounty-batch-5-22

Conversation

@crowniteto
Copy link
Copy Markdown
Owner

Batch bounty fixes for 11 issues. All 22 existing tests pass. Upstream PR: https://github.com/orchestration-agent/AgentOrchestration/pull/2633

… #2588, #2608, #2583, #2572, #2615

Fixes included in this batch PR:

- #2618: Validate blank/whitespace agent names in API registration
- #2564: Reject NaN/Inf observations in MetricsCollector.observe()
- #2559: Include min/max in histogram snapshots
- #2591: Fix metrics.stop_timer() deadlock (reentrant lock on observe)
- #2544: Protect config.set() against None values with TypeError
- #2600: Add workflow step timeout with step_timeout parameter
- #2588: SDK client raises ValueError on empty agent_id
- #2608: Prevent worker heartbeat from reviving completed runs
- #2583: Keep delayed jobs out of ready scans (only promote when ready_at passed)
- #2572: Validate batch acknowledgement ownership via worker_id
- #2615: Enforce least-privilege scopes on run cancellation

Additional fixes:
- Export AgentStatus from src.agent.__init__ (fixes import error)
- Fix reentrant Lock deadlock in MetricsCollector.stop_timer()

All 22 existing tests pass.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant