Skip to content

Use structured logging for internal diagnostics - #138

Merged
highbyte merged 1 commit into
mainfrom
feature/structured-diagnostics
Sep 1, 2026
Merged

Use structured logging for internal diagnostics#138
highbyte merged 1 commit into
mainfrom
feature/structured-diagnostics

Conversation

@highbyte

@highbyte highbyte commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • route internal CLI and web diagnostics through typed, structured logging with stable event IDs and levels
  • keep command output, safety prompts, worker events, and worker NDJSON on their existing stdout/stderr protocols
  • add provider fan-out, filtering, stderr-routing, safe-target, and NDJSON-isolation coverage
  • document the central logging and capture contract with POSIX and PowerShell examples

Verification

  • dotnet test Wrighty.slnx --configuration Release --nologo (2,304 passed)
  • npm test (96 passed; coverage thresholds met)
  • npm run test:coverage (96 passed; coverage thresholds met)
  • package-manager manifest tests (6 passed)
  • release smoke tests (2 passed)
  • dotnet list Wrighty.slnx package --vulnerable --include-transitive (no vulnerable packages)
  • changed-file dotnet format ... --verify-no-changes
  • real wrighty web check: 404 diagnostic reached stderr only and omitted the non-allowlisted query value

Closes #136

@sonarqubecloud

sonarqubecloud Bot commented Sep 1, 2026

Copy link
Copy Markdown

@highbyte
highbyte merged commit 329aace into main Sep 1, 2026
6 checks passed
@highbyte
highbyte deleted the feature/structured-diagnostics branch September 1, 2026 14:43
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.

Use ILogger for internal CLI diagnostics

1 participant