-
-
Notifications
You must be signed in to change notification settings - Fork 0
unity-cli@v1.8.1 #62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
unity-cli@v1.8.1 #62
Conversation
StephenHodgson
commented
Dec 14, 2025
- cleanup unity telemetry logging
- cleanup unity telemetry logging
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR cleans up Unity telemetry logging by refactoring the UTP (Unity Test Platform) type system, improving JSON sanitization to handle ANSI color codes, and enhancing log parsing with proper partial-line handling. The version is bumped to 1.8.1.
- Refactored UTP type definitions into a dedicated module with comprehensive type coverage and validation
- Enhanced telemetry JSON sanitization to handle ANSI escape sequences and properly process partial log lines
- Updated GitHub Actions workflows with latest action versions and improved configuration
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/utp/utp.ts | New file containing expanded UTP type definitions with normalization and validation logic |
| src/utp.ts | Removed - types migrated to src/utp/utp.ts |
| src/unity-logging.ts | Refactored telemetry parsing with ANSI handling, partial line buffering, and improved sanitization |
| tests/unity-logging.test.ts | Added test for ANSI color code removal in telemetry JSON |
| src/logging.ts | Added comment clarifying UTP as minimal logging level |
| src/cli.ts | Removed unused options parameter from editor-logs command |
| package.json | Version bump to 1.8.1 and @types/node update |
| package-lock.json | Lockfile updates for dependencies |
| .github/workflows/unity-build.yml | Updated to actions/checkout@v6, improved shell configuration, and workflow cleanup |
| .github/workflows/publish.yml | Updated to actions/checkout@v6 |
| .github/workflows/integration-tests.yml | Updated to actions/checkout@v6 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 10 out of 11 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.