feat(pm): PM skill pack fast-follows — single /pm command, portfolio roll-up, real health bottlenecks - #474
Conversation
Agentflare-Agent: claude-code_2-1-228_harness Agentflare-Branch: feat/pm-fast-follows
Agentflare-Agent: claude-code_2-1-228_harness Agentflare-Branch: feat/pm-fast-follows
Project override (name or UUID) on the read-only item groom/standup/health actions, plus project(action=list), backs a new /pm:portfolio workflow that rolls reports up across every project in the workspace. Adds an item_assignment_events log (migration 0011), written by item::update whenever assignee_agent changes (covers claim too, since claim assigns through update). item_health now computes real bottlenecks (items handed between agents >=2x in the window) instead of the always-empty placeholder, with a caveat that history starts at this migration. Splits crates/agentflare-backend/src/item.rs (over the LOC gate's frozen limit) into item/mod.rs + item/tests.rs, and src/mcp_server/tests/item_tests.rs (over the plain limit) into item_tests.rs + item_tests_reporting.rs. Also fixes a pre-existing Windows-only clippy warning: an unused `use super::*` in a unix-only test file. Agentflare-Agent: claude-code_2-1-228_harness Agentflare-Branch: feat/pm-fast-follows
…446-pm-skill-pack-fast-follows-pm-shims-port # Conflicts: # crates/agentflare-backend/src/item/mod.rs # src/mcp_server/tests/item_commit_failure_tests.rs # src/mcp_server/tests/item_tests.rs # src/mcp_server/tests/mod.rs Agentflare-Agent: claude-code_2-1-229_agent Agentflare-Branch: task/446-pm-skill-pack-fast-follows-pm-shims-port Agentflare-Item: 446
…ck detection
item_health_reports_velocity_wip_and_bottleneck_placeholder asserted the
old stub message ("no handoff history"), which no longer applies now
that assignment_events wires up real bottleneck detection. The identical
scenario is already covered by item_tests_reporting's
item_health_reports_velocity_wip_and_empty_bottlenecks.
Agentflare-Agent: claude-code_2-1-229_agent
Agentflare-Branch: task/446-pm-skill-pack-fast-follows-pm-shims-port
Agentflare-Item: 446
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 24 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (11)
Comment |
…ack-fast-follows-pm-shims-port Agentflare-Agent: claude-code_2-1-229_agent Agentflare-Branch: task/446-pm-skill-pack-fast-follows-pm-shims-port Agentflare-Item: 446
Rebased/integrated version of #469 onto current master.
#469 predates two subsequent master changes that conflicted with it:
crates/agentflare-backend/src/item.rswas independently split intoitem/{claim,crud,relations,search}.rssubmodules on master (item feat(pm): PM skill pack fast-follows — single /pm command, portfolio roll-up, real health bottlenecks #469's own scope also included this exact split — now superseded, kept master's structure).assignment_eventsrecording call (the actual new feature — wires item::update's assignee-change detection into the health bottleneck signal) has been re-applied on top of master's crud.rs.0011_item_assignment_events.sqlis byte-identical on both sides (already picked up by fix: recover schema-ahead DB migration + worktree branch-resolution collision #471's DB-recovery fix) — no real conflict there.item_health_reports_velocity_wip_and_bottleneck_placeholder) whose "no handoff history" placeholder assertion is superseded by the real bottleneck detection this PR enables; the identical scenario is already covered byitem_health_reports_velocity_wip_and_empty_bottlenecks.Full local verification:
cargo build,cargo fmt --check,cargo clippy --workspace -- -A unsafe_code -A clippy::pedantic -D warnings,cargo test --workspace --bins(1286 passed, 0 failed).Supersedes #469 — closing that one in favor of this rebased version.