Conf42: observable incident demo with approval and recovery - #166
Conf42: observable incident demo with approval and recovery#166EmersonBraun wants to merge 5 commits into
Conversation
4af42a7 to
1a12720
Compare
|
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. |
|
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. |
|
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 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. |
Implements the approved Conf42 recording demo for issue #165.
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