Skip to content

Add inline script environment lifecycle telemetry (PEP 723 PR 15/16) - #4

Closed
StellaHuang95 wants to merge 1 commit into
mainfrom
pep723-pr15-telemetry
Closed

Add inline script environment lifecycle telemetry (PEP 723 PR 15/16)#4
StellaHuang95 wants to merge 1 commit into
mainfrom
pep723-pr15-telemetry

Conversation

@StellaHuang95

@StellaHuang95 StellaHuang95 commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Draft fork-review PR. Do not send upstream yet.

Summary

  • add inlineScript.envCreated
  • add inlineScript.envReuseHit
  • add inlineScript.envError
  • define typed GDPR-safe schemas and stable low-cardinality error categories

Semantics

  • emit creation only after verified environment creation and sidecar persistence
  • measure actual build/rebuild duration, excluding lock wait and cache inspection
  • emit reuse only after complete cache validation
  • emit one lifecycle result for an underlying coalesced operation rather than per waiter
  • distinguish discovery failure, no compatible Python, user decline, install failure, package cancellation, lock timeout, and unavailable retained/orphaned locks
  • send no URI, path, version, dependency value, requirement, package name, or error-message content
  • preserve existing uv prompts, installation behavior, and uv telemetry

Scope

This is telemetry-only. It does not add routing, discovery, cleanup, UI, project registration, retries, or cache algorithm changes.

Validation

  • TypeScript test compilation
  • ESLint and diff checks
  • focused unit suite: 137 passing
  • two manual telemetry/code-path review passes; no significant findings remain

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3cb82ae9-7424-40a4-9156-8c54ac6e0895
@StellaHuang95
StellaHuang95 marked this pull request as ready for review August 17, 2026 17:07
@StellaHuang95 StellaHuang95 added review-auto:changes-requested Automated review: posted blocking findings to address. and removed review-auto:changes-requested Automated review: posted blocking findings to address. labels Aug 17, 2026
@StellaHuang95

Copy link
Copy Markdown
Owner Author

Closing this PR in favor of a clean-history replacement with the same current reviewed file changes.

@StellaHuang95

Copy link
Copy Markdown
Owner Author

Superseded by clean-history replacement #10.

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