Skip to content

fix: size report#65

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

fix: size report#65
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: mainly adjusts profiling metadata (size and phase naming) and updates docs/types; runtime behavior of isolated execution is unchanged aside from added/reshaped profiling output and debug logging.

Overview
Improves execution profiling output by adding a profiling.size field (computed via Buffer.byteLength of the bundled content) and adjusting debug logging to emit the full profiling result with rounded phase timings.

Updates public docs and typings to describe nodePaths for pre-resolved dependencies and to clarify profiling phase breakdown (notably install/build and bundled size).

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

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Size uses string not bytes
    • Changed compiled.content.length to Buffer.byteLength(compiled.content) to correctly calculate UTF-8 byte length instead of UTF-16 code units.

Create PR

Or push these changes by commenting:

@cursor push bd97cc78bf
Preview (bd97cc78bf)
diff --git a/src/index.js b/src/index.js
--- a/src/index.js
+++ b/src/index.js
@@ -69,7 +69,7 @@
         const { run, ...rest } = profiling
         const result = {
           ...rest,
-          size: compiled.content.length,
+          size: Buffer.byteLength(compiled.content),
           phases: {
             ...compiled.phases,
             spawn: spawnMs - run,

You can send follow-ups to the cloud agent here.

Reviewed by Cursor Bugbot for commit a609050. Configure here.

Comment thread src/index.js Outdated
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 26157909164

Warning

No base build found for commit aafebd8 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: 94.089%

Details

  • Patch coverage: 3 of 3 lines across 1 file 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: 694
Covered Lines: 656
Line Coverage: 94.52%
Relevant Branches: 118
Covered Branches: 108
Branch Coverage: 91.53%
Branches in Coverage %: Yes
Coverage Strength: 45.23 hits per line

💛 - Coveralls

@coveralls
Copy link
Copy Markdown

coveralls commented May 20, 2026

Coverage Report for CI Build 26158054391

Warning

No base build found for commit aafebd8 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: 94.089%

Details

  • Patch coverage: 3 of 3 lines across 1 file 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: 694
Covered Lines: 656
Line Coverage: 94.52%
Relevant Branches: 118
Covered Branches: 108
Branch Coverage: 91.53%
Branches in Coverage %: Yes
Coverage Strength: 45.23 hits per line

💛 - Coveralls

@Kikobeats Kikobeats merged commit 6f83550 into master May 20, 2026
3 checks passed
@Kikobeats Kikobeats deleted the next branch May 20, 2026 10:59
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