Skip to content

Fix duplicate error prefix on runtime errors - #60

Merged
snelusha merged 1 commit into
mainfrom
fix/duplicate-error-suffix
Aug 15, 2026
Merged

Fix duplicate error prefix on runtime errors#60
snelusha merged 1 commit into
mainfrom
fix/duplicate-error-suffix

Conversation

@snelusha

@snelusha snelusha commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Resolves #59

Summary by CodeRabbit

  • Bug Fixes
    • Runtime initialization errors are now displayed directly in standard error output without the redundant “error:” prefix.

@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b1bf1ec1-f778-4ddc-9165-502f1bc83e2a

📥 Commits

Reviewing files that changed from the base of the PR and between 6c98b49 and fd35cd1.

📒 Files selected for processing (1)
  • packages/ballerina-wasm/main_wasm.go

Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The runtime initialization failure path now writes the raw error message to stderr without adding the error: prefix.

Changes

Runtime error output

Layer / File(s) Summary
Runtime initialization error reporting
packages/ballerina-wasm/main_wasm.go
rt.Init failures now print the raw error message to stderr.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to fd35c

This is a localized runtime error-formatting change, and no actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: removing duplicate error: prefixes from runtime errors.
Linked Issues check ✅ Passed The change directly addresses issue #59 by preventing duplicate error: prefixes on runtime errors.
Out of Scope Changes check ✅ Passed The single-line change is limited to the linked issue objective and contains no unrelated modifications.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/duplicate-error-suffix

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 golangci-lint (2.12.2)

level=error msg="Running error: context loading failed: failed to load packages: failed to load packages: failed to load with go/packages: err: exit status 1: stderr: go: cannot load module ballerina listed in go.work file: open ballerina/go.mod: no such file or directory\n"


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@snelusha
snelusha merged commit 006ab1c into main Aug 15, 2026
2 checks passed
@snelusha
snelusha deleted the fix/duplicate-error-suffix branch August 15, 2026 23:27
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.

Prevent duplicate error: prefix on runtime errors

1 participant