Skip to content

Conf42: observable incident demo with approval and recovery - #166

Open
EmersonBraun wants to merge 5 commits into
mainfrom
codex/conf42-observable-demo
Open

Conf42: observable incident demo with approval and recovery#166
EmersonBraun wants to merge 5 commits into
mainfrom
codex/conf42-observable-demo

Conversation

@EmersonBraun

@EmersonBraun EmersonBraun commented Sep 5, 2026

Copy link
Copy Markdown
Member

Implements the approved Conf42 recording demo for issue #165.

  • Real AgentsKit controller and fixture tools; no LLM or production actions.
  • Scripted agent and deterministic runbook, two timeouts, fallback, native human approval, simulated rollback and health verification.
  • Interactive input/output evidence, export, pause/replay, focus and responsive light/dark themes.
  • Typed adapter substitution test, including adapter-originated approval, and English rehearsal guide.

Preview: https://agentskit-chat-qp7s6ebzy-nox-code-a70334f9.vercel.app/demo/agent-observability

Deployment note: VERCEL_PREVIEW_FEEDBACK_ENABLED=0 is scoped to the preview branch codex/conf42-observable-demo. The application CSP is unchanged. Superseded preview URLs are not the final verification target.

Focused integration and real-browser flows passed. Full build and lint passed. All 30 CLI tests passed with pinned pnpm 10.33.0. The combined verification harness is still running; remote validation is being repeated after the branch-scoped toolbar configuration. Explicit human visual approval remains required. No automatic merge.

Refs #165

@EmersonBraun
EmersonBraun force-pushed the codex/conf42-observable-demo branch from 4af42a7 to 1a12720 Compare September 5, 2026 02:39
@EmersonBraun

Copy link
Copy Markdown
Member Author

Code review: five independent read-only passes covered upstream/rules compliance, logic, API history, prior review context and documentation promises. Findings repaired: inspectable tool inputs, real pause/resume assertions, a typed adapter-substitution regression test preserving the original rollback proposal, and failed-network-request collection. Follow-up inspection found no residual issues at confidence >=80 in those findings. This is not human approval.

The final UI adjustment keeps follow-latest behavior after filtering. Current deployment uses the same application source tree as commit 987a094. The preview disables only Vercel feedback injection (VERCEL_PREVIEW_FEEDBACK_ENABLED=0); CSP remains unchanged.

Current preview: https://agentskit-chat-krir077em-nox-code-a70334f9.vercel.app/demo/agent-observability

Combined acceptance/harness execution is still running. No merge authorized by this review.

@EmersonBraun

Copy link
Copy Markdown
Member Author

Verification update: all code, build, CLI, documentation UI and report UI gates executed successfully in run 1788576200926-3604 except the superseded remote-toolbar check and stale ADR count. The count is now synchronized to 34 and its seven tests pass. Remote run observability-remote-1788576426155 passes with the final branch-scoped Vercel configuration.

A complete fresh harness run is now executing against commit ea5630c and https://agentskit-chat-qp7s6ebzy-nox-code-a70334f9.vercel.app. This update does not override the earlier BLOCKED run and does not claim completion. Human visual approval remains outstanding.

@EmersonBraun

Copy link
Copy Markdown
Member Author

Final local verification: AWAITING_HUMAN_APPROVAL, run 1788576684163-16305, commit ea5630c. All 22 configured checks returned passed, including fixture integration, local/remote real-browser flows, responsive/contrast/keyboard checks, build/lint/tests, docs and HTTP/CLI checks. Source and screenshot hashes are retained in the project verification evidence. No human approval recorded.

MERGE REMAINS BLOCKED: the additional CI quality job failed in pack-release. Registry inspection confirms @agentskit/memory@0.11.9 dependencies contain @agentskit/core: workspace:*. Failure: https://github.com/AgentsKit-io/agentskit-chat/actions/runs/33940180238/job/101235925567 . No upstream package was modified/published as part of this demo task.

Final preview: https://agentskit-chat-qp7s6ebzy-nox-code-a70334f9.vercel.app/demo/agent-observability
Guide: https://agentskit-chat-qp7s6ebzy-nox-code-a70334f9.vercel.app/docs/examples/agent-observability

Three superseded previews and the downloaded temporary preview env file were cleaned. Original untracked report/workflow artifacts were preserved. Human visual approval and separately scoped upstream packaging remediation remain outstanding; no completion or merge claimed.

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