test(kun): align LLM debug recorder trace attempts#997
Closed
luoye520ww wants to merge 1 commit into
Closed
Conversation
luoye520ww
force-pushed
the
codex/fix-llm-debug-recorder-typecheck
branch
from
July 22, 2026 07:10
14712c2 to
2fde0c3
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The runtime test gate was failing before the Issue PRs could reach their own checks because two tests still asserted retired LLM debug behavior.
Root cause
LlmDebugRecordermoved fromcaptureRequest()to structured HTTP attempts, and Agent Perspective now creates a recorder for every HTTP-backed runtime. The legacy tests were not migrated with those production changes.Scope
Test-contract repair only. This PR does not change runtime behavior, public APIs, settings semantics, or trace retention.
Changes
beginHttpAttempt()and awaitfinish().llmDebugoptions and{ enabled: false }, which no longer disable the recorder.Tests
Actual validation
The runtime-factory test creates a real temp data directory, records an HTTP attempt, persists it, and reads the thread trace back through the runtime-owned recorder.
Review performed
PR size
CI note
A local
npm.cmd run check:extensionsrun reaches extension schema/docs/example validation, then hits the existing Windows symlink-permission baseline inexamples/extensions/kun-video-editor. GitHub CI remains the authoritative cross-platform gate.