Skip to content

Extract message content in mini-playground output to match rpk CLI#360

Merged
JakeSCahill merged 1 commit intomainfrom
fix-mini-playground-output
Feb 3, 2026
Merged

Extract message content in mini-playground output to match rpk CLI#360
JakeSCahill merged 1 commit intomainfrom
fix-mini-playground-output

Conversation

@JakeSCahill
Copy link
Copy Markdown
Contributor

Summary

Fixes inconsistency between mini-playground and main playground output display.

Before:

  • Mini-playground: { "msg": { "message": "hello world" } }
  • Main playground: { "message": "hello world" }
  • rpk CLI: { "message": "hello world" }

After:

  • All display just the message content (extracts .msg field)

Changes

  • Extract .msg field from WASM result before displaying in mini-playground
  • Matches behavior of main playground and rpk connect blobl CLI

Testing

  • ✅ All 13 interactive tests pass
  • ✅ Verified with rpk connect blobl for consistency

The mini-playground was showing the full WASM result structure {"msg": {...}, "meta": {...}} while the main playground and rpk CLI extract just the message content. Now both playgrounds display consistent output.
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 3, 2026

Deploy Preview for docs-ui ready!

Name Link
🔨 Latest commit 7e4aa6c
🔍 Latest deploy log https://app.netlify.com/projects/docs-ui/deploys/69823aa6e1dfaa00083365c1
😎 Deploy Preview https://deploy-preview-360--docs-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 26 (🔴 down 2 from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (no change from production)
SEO: 89 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

The change modifies the result display logic in the Bloblang interactive JavaScript module. When a Bloblang mapping produces a result, the code now checks if the parsed JSON object contains a .msg field. If present, this field value is extracted and formatted for display; otherwise, the entire parsed object is used. This affects only the presentation layer without changing error handling or mapping execution logic.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • paulohtb6
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly reflects the main change: extracting message content in mini-playground output to align with rpk CLI behavior.
Description check ✅ Passed The description comprehensively explains the inconsistency being fixed, shows before/after examples, details the specific changes, and documents testing verification.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-mini-playground-output

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 and usage tips.

@JakeSCahill JakeSCahill merged commit e05d0aa into main Feb 3, 2026
7 checks passed
@JakeSCahill JakeSCahill deleted the fix-mini-playground-output branch February 3, 2026 18:20
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