Skip to content

feat(pi): adopt telemetry conformance, agent-runtime eval, session fencing, daemon leases, diff renderer core - #226

Merged
Patel230 merged 1 commit into
mainfrom
feat/pi-adoption
Aug 22, 2026
Merged

feat(pi): adopt telemetry conformance, agent-runtime eval, session fencing, daemon leases, diff renderer core#226
Patel230 merged 1 commit into
mainfrom
feat/pi-adoption

Conversation

@Patel230

Copy link
Copy Markdown
Contributor

Summary

Implements the approved Pi adoption plan (docs/plans/pi-adoption-plan.md), landing the feasible items across hawk:

P0

  • Telemetry conformance suite (internal/observability/conformance): typed span/attribute schema + runner-independent conformance harness wired to the real span starters; schema-drift, sensitive-key, and passivity tests. CI now fails on telemetry schema drift.
  • Agent-runtime eval harness (internal/feature/evalloop + hawk eval loop): drives the real engine.Session tool loop via an injected ChatClient in an isolated dir, snapshots the transcript, reports events/usage/cost. Runs in CI without external credentials via a mock client.

P1

  • Session writer fencing (internal/session): additive Fence metadata field persisted on save/load + FenceOf/SetFence helpers.
  • Daemon session leases (internal/daemon): POST/DELETE /v1/sessions/{id}/lease acquire/release single-owner leases with fence enforcement; OpenAPI updated (parity test green).
  • Differential renderer core (internal/tui/diff): line-granular diff + synchronized-output emission mirroring pi-tui, fully unit-tested. Full Bubble Tea renderer swap deferred to a runtime-verified follow-up.

P2

  • Kitty graphics deferred (requires the render-path follow-up).

Test plan

  • Full go test ./... green (twice + via pre-push hook)
  • New package tests: conformance, evalloop, diff, daemon lease, session fence
  • make vet, make lint, hawk verify green
  • Boundary guards, submodule drift, govulncheck passed in pre-push hooks
  • markdownlint-cli2 clean on the updated plan doc

@Patel230
Patel230 merged commit f92ebf7 into main Aug 22, 2026
26 checks passed
@Patel230
Patel230 deleted the feat/pi-adoption branch August 22, 2026 05:59
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