workflows: fix tests when coverage build, mp: workaround for assertion

[mabrarov]

Changes

1. Increased coroutine stack size for CI coverage builds to fix failing tests (refer to https://github.com/fluent/fluent-bit/actions/runs/23456675367/job/68247579444#logs).
2. Workaround for Fixed assertion in mpack_reader_init ludocode/mpack#119 (refer to build: CMake project fixes for MS Visual C++ compiler #11397 (comment)) to fix CI macOS builds.

---

Testing

Before we can approve your change; please submit the following in a comment:

• [N/A] Example configuration file for the change.
• [N/A] Debug log output from testing the change.
• [N/A] Attached Valgrind output that shows no leaks or memory corruption was found.
• [N/A] Run local packaging test showing all targets (including any new ones) build.
• [N/A] Set ok-package-test label to test for all targets (requires maintainer to do).

Documentation

• [N/A] Documentation required for this feature

Backporting

• [N/A] Backport to latest stable release.

Related PRs

1. coro: Extend coro stack size for 38KiB padding of yyjson (2nd attempt) #11608
2. Fixed assertion in mpack_reader_init ludocode/mpack#119

---

Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

Summary by CodeRabbit

• Bug Fixes

  • Enhanced message pack counting logic with improved NULL pointer validation to prevent unsafe operations when input data is unavailable.

• Chores

  • Updated CI/CD workflow to enhance coverage detection and conditionally apply additional build configuration flags during unit testing.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 1bc71fc3-7a70-44f3-ae44-dd9f5f645092

📥 Commits

Reviewing files that changed from the base of the PR and between d4a4cbe and ff235f7.

📒 Files selected for processing (1)

• src/flb_mp.c

🚧 Files skipped from review as they are similar to previous changes (1)

• src/flb_mp.c

---

📝 Walkthrough

Walkthrough

Two changes enhance build reliability and code safety: GitHub Actions workflow logic now detects coverage builds and appends a compiler flag to adjust stack size, while the msgpack function initializes variables and adds a NULL check for safer data handling.

Changes

Cohort / File(s)	Summary
Build System .github/workflows/unit-tests.yaml	Modified Ubuntu unit test job to detect COVERAGE mode in FLB_OPT and append -DFLB_CORO_STACK_SIZE=4194304 flag with proper spacing handling.
Msgpack Utility src/flb_mp.c	Updated flb_mp_count_remaining() to initialize remaining variable and guard msgpack reader initialization/iteration with a data != NULL check, preventing unsafe reader operations on null pointers.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

• edsiper
• patrick-stephens
• niedbalski
• celalettin1286

Poem

🐰 A workflow now catches coverage's call,
With stack sizes tuned to fit all,
While msgpack counts with caution stand,
Null-checked safe across the land,
These guards and flags make code more grand!

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the two main changes: a workflow fix for coverage builds and an mp workaround for assertion issues.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 428865758b

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".