Repository navigation
Conversation
This was referenced Oct 3, 2026
tang-vu
force-pushed
the
fix/agent-monitor-persisted-telemetry-key
branch
from
October 4, 2026 09:54
7f1ced0 to
edbc704
Compare
Collaborator
|
Thanks for the careful work. Every installer (Linux, macOS, Windows) writes |
5 of 23 tasks
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.
Summary
Read the generated key when explicit configuration is absent, respecting explicit-key precedence and live key-file rotation.
Reconstructed only the reviewed #7212 scope on current upstream
mainb4f015a921c59eeb20317c5f48adf2dfb43b1430. Local candidate9486919d282aff55c6195b722456477d720ec25f; previous published head7f1ced02c369a90bc7230151c542b4b2845062fdis retained locally underrefs/archive/ods-20261004/pr-7212. No behavioral changes were added during reconstruction; current-main changes outside this scope are preserved.Issue association: Closes #7301.
AI Assistance
The human contributor and Codex collaborated on this PR. The contributor provided project direction, scope decisions and environment setup; Codex assisted with analysis, implementation or reconstruction, tests, review and PR preparation. The validation commands were executed by Codex, with results and limitations documented in this PR. The human contributor remains responsible for the final changes and validation claims. Maintainer review and applicable release gates remain required before merge. Earlier independent AI review receipts apply only to their recorded heads; current-candidate validation is identified separately in this PR.
Release Lane
mainChanged Surface
Risk And Validation
Risk level: High
git diff --check
Focused tests
Commit identity gate against exact current-main base/candidate
4 real local HTTP/file tests passed. Current-main red: 3 failed, 1 explicit-key control passed.
The focused results were run on the exact candidate file tree before committing; the commit only recorded that tested tree. Original-head/full native and combined integration receipts remain archived, and are inherited evidence rather than fresh install or new-head hosted CI results.
CI execution: dashboard.yml discovers test_agent_monitor_persisted_key.py.
Operational Change Check
Notes For Reviewers
Path matches Token Spy producer and shared Compose data mount. Full installed Token Spy service is unverified.
Rollback: revert this scope; preserve existing recovery receipts and configured data. Local native tests and green original-head CI do not replace human review or deployed-runtime gates.
CI follow-up
The new mypy ratchet found one heterogeneous test-state inference error. TypedDict now declares the key and observed authorization list; production code is unchanged. Four behavioral tests, changed-file mypy 2.4.0 and Ruff passed. The 464-error existing baseline is unchanged.