Skip to content

chore: compact logging#63

Merged
Kikobeats merged 1 commit into
masterfrom
next
May 20, 2026
Merged

chore: compact logging#63
Kikobeats merged 1 commit into
masterfrom
next

Conversation

@Kikobeats
Copy link
Copy Markdown
Owner

@Kikobeats Kikobeats commented May 20, 2026

Note

Low Risk
Low risk: changes only affect debug/profiling logging and remove duration-wrapped timing around install/build without altering execution flow or outputs.

Overview
Removes duration/debug instrumentation from compilation (npm install and esbuild phases) so these steps run directly without extra log wrapping.

Compacts runtime debug logs by rounding cpu and phase timings to whole milliseconds via a new roundMs helper, reducing log verbosity while keeping the same profiling fields.

Reviewed by Cursor Bugbot for commit 32c15f5. Bugbot is set up for automated code reviews on this repo. Configure here.

@Kikobeats Kikobeats merged commit 9484e45 into master May 20, 2026
3 checks passed
@coveralls
Copy link
Copy Markdown

coveralls commented May 20, 2026

Coverage Report for CI Build 26151465302

Warning

No base build found for commit 26faabb on master.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 95.807%

Details

  • Patch coverage: 10 of 10 lines across 2 files are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 671
Covered Lines: 646
Line Coverage: 96.27%
Relevant Branches: 116
Covered Branches: 108
Branch Coverage: 93.1%
Branches in Coverage %: Yes
Coverage Strength: 47.06 hits per line

💛 - Coveralls

1 similar comment
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 26151465302

Warning

No base build found for commit 26faabb on master.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 95.807%

Details

  • Patch coverage: 10 of 10 lines across 2 files are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 671
Covered Lines: 646
Line Coverage: 96.27%
Relevant Branches: 116
Covered Branches: 108
Branch Coverage: 93.1%
Branches in Coverage %: Yes
Coverage Strength: 47.06 hits per line

💛 - Coveralls

@Kikobeats Kikobeats deleted the next branch May 20, 2026 08: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.

2 participants