Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 30 additions & 0 deletions docs/pr-discussions/PR-3591-docs-archive-lior-preserve-pr-3544.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
---
pr_number: 3591
title: "docs(archive): Lior preserve PR 3544"
author: "AceHack"
state: "MERGED"
created_at: "2026-05-15T21:13:51Z"
merged_at: "2026-05-15T21:43:04Z"
closed_at: "2026-05-15T21:43:04Z"
head_ref: "lior/preserve-3544"
base_ref: "main"
archived_at: "2026-05-15T22:12:22Z"
archive_tool: "tools/pr-preservation/archive-pr.ts"
---

# PR #3591: docs(archive): Lior preserve PR 3544

## PR description

Preserving recently merged PR #3544 into docs/pr-discussions/

## Reviews

### COMMENTED — @copilot-pull-request-reviewer (2026-05-15T21:15:10Z)

## Pull request overview

Archives merged PR #3544 conversation into `docs/pr-discussions/` as a preservation record, following the existing archive-tool pattern.

**Changes:**
- Adds a new PR archive markdown file with YAML frontmatter (pr metadata) and preserved PR description + review.
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
---
pr_number: 3592
title: "shard(tick): 2112Z \u2014 session-arc plateau recognized; brief shards from here"
author: "AceHack"
state: "MERGED"
created_at: "2026-05-15T21:17:49Z"
merged_at: "2026-05-15T21:19:27Z"
closed_at: "2026-05-15T21:19:27Z"
head_ref: "shard/tick-2112z-otto-cli-2026-05-15"
base_ref: "main"
archived_at: "2026-05-15T22:12:21Z"
archive_tool: "tools/pr-preservation/archive-pr.ts"
---

# PR #3592: shard(tick): 2112Z — session-arc plateau recognized; brief shards from here

## PR description

## Summary

Tick 2112Z. Substrate-honest plateau-recognition shard. 28-tick arc has reached per-tick-value-add ~0; from here, brief shards acknowledge cron firing without producing thin-substrate. Triggering criteria for substantive resumption recorded.

## Test plan

- [x] Shard at canonical path
- [ ] CI green
- [ ] Auto-merge arms

🤖 Generated with [Claude Code](https://claude.com/claude-code)

## Reviews

### COMMENTED — @copilot-pull-request-reviewer (2026-05-15T21:19:26Z)

## Pull request overview

Adds a new hygiene-history tick shard (2112Z) documenting that the current session arc has plateaued and that subsequent ticks should be brief “wait-state” acknowledgements until specific triggering conditions occur.

**Changes:**
- Introduces a new tick entry describing plateau recognition and the intended shorter shard format going forward.
- Records explicit criteria for when to resume substantive tick shards.
- Adds cross-references to the relevant memory file and `.claude/rules` guidance that motivates the change.

## General comments

### @chatgpt-codex-connector (2026-05-15T21:17:54Z)

You have reached your Codex usage limits for code reviews. You can see your limits in the [Codex usage dashboard](https://chatgpt.com/codex/cloud/settings/usage).
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
---
pr_number: 3593
title: "feat(bg): B-0500 slice 3 \u2014 wire queue-state guard into pollOnce"
author: "AceHack"
state: "MERGED"
created_at: "2026-05-15T21:35:52Z"
merged_at: "2026-05-15T21:39:06Z"
closed_at: "2026-05-15T21:39:06Z"
head_ref: "feat/b0500-backlog-notifier-queue-guard-retry"
base_ref: "main"
archived_at: "2026-05-15T22:12:20Z"
archive_tool: "tools/pr-preservation/archive-pr.ts"
---

# PR #3593: feat(bg): B-0500 slice 3 — wire queue-state guard into pollOnce

## PR description

Implements backlog row **B-0500** (Slice 3 of the B-0441 notifier). We now properly guard work assignments using `isAgentQueueEmpty`, ensuring agents only receive new proactive tasks when their queues are fully cleared.

## Reviews

### COMMENTED — @copilot-pull-request-reviewer (2026-05-15T21:37:34Z)

## Pull request overview

This PR completes Slice 3 of the B-0441 backlog-ready notifier by wiring the `isAgentQueueEmpty` queue-state guard into `pollOnce`. When the target agent's queue is non-empty (recent commits or open PRs by that agent), the poll iteration returns early with `queueBusy: true` and publishes no work-assignment envelopes, preventing assignment pile-up on busy agents.

**Changes:**
- Add `targetAgent` field to `NotifierConfig` and a `queueBusy` field to `PollResult`; populate via `isAgentQueueEmpty(config.targetAgent, adapters)` at the top of `pollOnce`.
- Thread `adapters` through `runOnce`/`runDaemon` and add a `--target-agent <agent>` CLI flag.
- Add two new tests (queue-busy → no publish; queue-empty → publishes) and update backlog rows B-0500/B-0441 to closed/shipped.

### Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

| File | Description |
| ---- | ----------- |
| tools/bg/backlog-ready-notifier.ts | Wire queue-state guard into `pollOnce`; add `targetAgent` config + `queueBusy` result field; plumb adapters through `runOnce`/`runDaemon`; add `--target-agent` flag. |
| tools/bg/backlog-ready-notifier.test.ts | Add tests for queue-busy skip and queue-empty publish paths; update existing tests for new config field/flag. |
| docs/backlog/P1/B-0500-...md | Mark B-0500 status closed and acceptance checkboxes ticked. |
| docs/backlog/P1/B-0441-...md | Update slice-3 row of B-0441 table to ✅ shipped. |
Loading
Loading