feat(internal): merge Pi canary, bounded agents, and usage UI by malkovitc · Pull Request #1 · malkovitc/buzz

malkovitc · 2026-08-25T12:01:49Z

Internal downstream patch

Merges the production-proven Pi ACP canary, opt-in Codex efficiency controls, empty-message hardening, and NIP-AM agent-usage UI into the fork only. This is not intended for upstream block/buzz.

Review feedback resolved

pooled adapter now supports multiple ACP sessions;
every inbound task gets a fresh Pi SDK process/session;
BUZZ_PRIVATE_KEY and auth tag stay in the trusted adapter; Pi publications use a typed parent-side broker;
non-event initial/heartbeat prompts work without reply routing;
rejected provider prompts propagate as ACP errors;
forum reply kind is inferred by the hardened Buzz CLI parent lookup;
Pi adapter has an install command and strict minimum 0.2.0 version gate;
autonomous session/history bounds remain opt-in (0/12 product defaults preserved).

Checks

Pi Node tests: 20/20
Biome: PASS
Rust config/ACP metadata tests: PASS
Desktop Pi catalog/version tests: PASS
Forum-kind inference test: PASS
rustfmt and git diff --check: PASS
prior Caliper Pi canary + Codex rollback + packaged smoke: PASS

Pi remains canary-only; no fleet-wide runtime switch.

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

Adds the agent-usage UI feature on top of the merged NIP-AM backend. - `desktop/src/features/agent-usage/`: daily bars chart, 7/30-day range tabs, focused agent view with caveats, hooks, and lib utilities. All state is driven by the `get_agent_usage_series` Tauri command. - Profile panel: usage ingress row + focused view wired into agent profiles; gated on owner-scoped visibility (`viewerIsOwner && isBot`). - `AgentsView`: agent-usage overview section. - `AgentsScreen`/`ProfilePanelContext`: usage navigation state. - `e2eBridge.ts` + `tests/helpers/bridge.ts`: `get_agent_usage_series` mock handler + fixture types, including the M2 cache fields (`cacheReadTokens`, `cacheWriteTokens`, `freshInputTokens`) the wire type carries. The UI absorbs but does not display these fields. - `tests/e2e/agent-usage.spec.ts` + `agent-usage-screenshots.spec.ts`: Playwright coverage for loading, empty state, bars, range switching, error/retry, focused view, caveats, and cache invalidation. - `desktop/playwright.config.ts`: agent-usage specs added to suite. - `desktop/package.json`: recharts dependency. Co-authored-by: Will Pfleger <pfleger.will@gmail.com> Signed-off-by: Will Pfleger <pfleger.will@gmail.com> (cherry picked from commit 3d723a7) Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

…t-usage focused view The plumbing already returned cacheReadTokens/cacheWriteTokens/freshInputTokens at every scope of AgentUsageSeries, but the focused view rendered only the four top-level stats and never surfaced them. Add an Input-breakdown subsection to the totals card (Cache read / Cache write / Fresh input) gated on any known cache value, and a compact per-model breakdown line. Fields follow the existing unknown-not-zero semantics: absent subsets are omitted, a known-but-incomplete value shows its lower bound with a Partial marker, and a scope with no cache data renders no subsection at all. Co-authored-by: Will Pfleger <pfleger.will@gmail.com> Signed-off-by: Will Pfleger <pfleger.will@gmail.com> (cherry picked from commit b01a381) Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

malkovitc · 2026-08-25T12:01:56Z

@codex review

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

malkovitc · 2026-08-25T12:04:54Z

Review audit complete for exact head fb900af1a2c82843224d0fdb23cb4be3f1343912. All prior review threads were checked and addressed:

multi-session pool: session registry + sequential pooled tasks;
fresh task session: fresh SDK process/session per prompt;
signing-key isolation: key/auth tag stripped from Pi child, trusted typed broker;
non-event prompts: routing optional until publication;
adapter install: pinned 0.2.0 fork release command;
prompt rejection: ACP failure, no false delivery;
forum replies: CLI resolves parent kind and emits 45003;
version validation: strict minimum 0.2.0 gate;
product defaults: restored 12/0 defaults, bounds opt-in.

Additional validation: Node 20/20; desktop and Rust targeted tests PASS; live Caliper 0.2.0 broker smoke inbound e1b68bca… → durable reply c3a4452e…, 11s, exactly one reply.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 702812b404

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-08-25T12:06:32Z

+        let reply_to = batch
+            .events
+            .last()
+            .or_else(|| batch.cancelled_events.last())?
+            .event
+            .id
+            .to_hex();


Route brokered replies through the resolved thread anchor

When Pi handles a human reply inside an existing channel thread, this always sets replyTo to the newest event ID. However, format_prompt deliberately uses resolve_reply_anchor to target the thread root and keep human-facing replies flat at layer 1. Because buzz_reply treats this metadata as authoritative, Pi responses become nested under the triggering comment instead of following the routing shown in the prompt; derive this value with the same thread/DM anchor logic used by format_prompt.

Useful? React with 👍 / 👎.

chatgpt-codex-connector · 2026-08-25T12:06:32Z

+      const result = await tool.execute(
+        String(event.id),
+        event.args ?? {},
+        new AbortController().signal,
+      );


Abort brokered tools when the ACP prompt is cancelled

If cancellation or the harness deadline occurs while buzz_reply is running, this creates an untracked controller whose signal is never aborted. session/cancel only aborts the Pi session, while the broker's separately spawned buzz messages send process continues and can publish after the turn has already completed as cancelled. Store a per-prompt controller and abort it from cancellation and teardown so cancelled work cannot produce a late channel message.

Useful? React with 👍 / 👎.

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

malkovitc · 2026-08-25T12:17:28Z

Follow-up exact head 7590cf6a614be60206eb25c63ff6928e486b19ff: discovery metadata/version logic and tests were split into focused modules; local just file-size-check now PASS. Node 20/20, Biome, rustfmt, desktop Pi tests, and live 0.2.0 broker smoke remain PASS.

malkovitc and others added 17 commits August 25, 2026 12:43

docs(agents): plan Pi harness canary integration

453455d

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

feat(agents): spike Pi RPC ACP adapter

8dc1216

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

feat(agents): embed Pi SDK runtime with event budgets

d08e836

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

feat(agents): bind Pi prompts to Buzz event routing

2911e43

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

feat(agents): add event-bound Pi tools

034b79d

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

feat(agents): register Pi pilot runtime and test budgets

c878b21

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

test(agents): benchmark Pi ACP question path

2840463

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

test(agents): compare Pi with managed Codex ACP

2a18ba4

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

test(agents): validate bounded Pi UI and reply flows

4babbdb

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

perf(acp): bound autonomous Codex sessions

5725c66

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

perf(agents): reduce reply ceremony

207fc82

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

fix(cli): reject empty message bodies

fe4f653

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

fix(desktop): restack agent usage UI on current profiles

213a7ca

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

docs(agents): record Pi production canary and rollback

93a030d

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

fix(agents): address Pi pilot review findings

702812b

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

chore(agents): pin Pi installer to reviewed fork release

fb900af

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

chatgpt-codex-connector Bot reviewed Aug 25, 2026

View reviewed changes

refactor(desktop): satisfy discovery file-size policy

7590cf6

Signed-off-by: evgeniy.chernomortsev <j.malkovitc@gmail.com>

malkovitc merged commit d3f90f5 into main Aug 25, 2026
48 of 53 checks passed

malkovitc deleted the local/linza-patched-pi-0.5.18 branch August 25, 2026 12:50

malkovitc mentioned this pull request Aug 25, 2026

fix(internal): close final Pi broker review gaps #2

Merged

Navigation Menu

feat(internal): merge Pi canary, bounded agents, and usage UI - #1

feat(internal): merge Pi canary, bounded agents, and usage UI#1
malkovitc merged 19 commits into
mainfrom
local/linza-patched-pi-0.5.18

malkovitc commented Aug 25, 2026

Uh oh!

malkovitc commented Aug 25, 2026

Uh oh!

malkovitc commented Aug 25, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Uh oh!

chatgpt-codex-connector Bot Aug 25, 2026

Uh oh!

chatgpt-codex-connector Bot Aug 25, 2026

Uh oh!

malkovitc commented Aug 25, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

malkovitc commented Aug 25, 2026

Internal downstream patch

Review feedback resolved

Checks

Uh oh!

malkovitc commented Aug 25, 2026

Uh oh!

malkovitc commented Aug 25, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector Bot Aug 25, 2026

Choose a reason for hiding this comment

Uh oh!

chatgpt-codex-connector Bot Aug 25, 2026

Choose a reason for hiding this comment

Uh oh!

malkovitc commented Aug 25, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant