[PR #84] [MERGED] Add per-hat max_activations limit #127

New issue

Closed

opened 2026-02-27 10:22:19 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 10:22:19 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/mikeyobrien/ralph-orchestrator/pull/84
Author: @godnight10061
Created: 1/21/2026
Status: ✅ Merged
Merged: 1/21/2026
Merged by: @mikeyobrien

Base: main ← Head: fix/issue-66

📝 Commits (1)

b9e17f8 Add per-hat max_activations limit

📊 Changes

7 files changed (+352 additions, -23 deletions)

View changed files

📝 Cargo.toml (+0 -1)
📝 crates/ralph-adapters/src/cli_executor.rs (+13 -1)
📝 crates/ralph-adapters/src/pty_executor.rs (+13 -0)
📝 crates/ralph-core/src/config.rs (+6 -0)
📝 crates/ralph-core/src/event_loop.rs (+239 -21)
📝 crates/ralph-core/src/summary_writer.rs (+2 -0)
➕ specs/per-hat-max-activations.spec.md (+79 -0)

📄 Description

Summary:

Enforce per-hat max_activations in the event loop.
When the limit is reached, emit <hat_id>.exhausted once per run and drop would-trigger events for that hat.

Tests:

bash ./scripts/sync-embedded-files.sh check
cargo fmt --all -- --check
CARGO_TARGET_DIR=target_ci cargo clippy --all-targets --all-features -- -D warnings
CARGO_TARGET_DIR=target_ci cargo test
CARGO_TARGET_DIR=target_ci cargo test -p ralph-core smoke_runner
CARGO_TARGET_DIR=target_ci cargo test -p ralph-core kiro
cargo package -p ralph-cli --allow-dirty --no-verify

References:

mikeyobrien/ralph-orchestrator#66

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/mikeyobrien/ralph-orchestrator/pull/84 **Author:** [@godnight10061](https://github.com/godnight10061) **Created:** 1/21/2026 **Status:** ✅ Merged **Merged:** 1/21/2026 **Merged by:** [@mikeyobrien](https://github.com/mikeyobrien) **Base:** `main` ← **Head:** `fix/issue-66` --- ### 📝 Commits (1) - [`b9e17f8`](https://github.com/mikeyobrien/ralph-orchestrator/commit/b9e17f8f35b57c21898c90c20027335570385610) Add per-hat max_activations limit ### 📊 Changes **7 files changed** (+352 additions, -23 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.toml` (+0 -1) 📝 `crates/ralph-adapters/src/cli_executor.rs` (+13 -1) 📝 `crates/ralph-adapters/src/pty_executor.rs` (+13 -0) 📝 `crates/ralph-core/src/config.rs` (+6 -0) 📝 `crates/ralph-core/src/event_loop.rs` (+239 -21) 📝 `crates/ralph-core/src/summary_writer.rs` (+2 -0) ➕ `specs/per-hat-max-activations.spec.md` (+79 -0) </details> ### 📄 Description Summary: - Enforce per-hat max_activations in the event loop. - When the limit is reached, emit <hat_id>.exhausted once per run and drop would-trigger events for that hat. Tests: - bash ./scripts/sync-embedded-files.sh check - cargo fmt --all -- --check - CARGO_TARGET_DIR=target_ci cargo clippy --all-targets --all-features -- -D warnings - CARGO_TARGET_DIR=target_ci cargo test - CARGO_TARGET_DIR=target_ci cargo test -p ralph-core smoke_runner - CARGO_TARGET_DIR=target_ci cargo test -p ralph-core kiro - cargo package -p ralph-cli --allow-dirty --no-verify References: - mikeyobrien/ralph-orchestrator#66 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>