Conversation
|
✨ Fix all issues with BitsAI or with Cursor
|
Codecov Report❌ Patch coverage is ❌ Your patch status has failed because the patch coverage (84.81%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## master #3581 +/- ##
==========================================
+ Coverage 62.18% 68.64% +6.45%
==========================================
Files 141 165 +24
Lines 13352 18774 +5422
Branches 1746 1768 +22
==========================================
+ Hits 8303 12887 +4584
- Misses 4259 5086 +827
- Partials 790 801 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 3 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
f4914c3 to
2fc2654
Compare
bbc593e to
192a4a1
Compare
Benchmarks [ appsec ]Benchmark execution time: 2026-03-04 15:16:10 Comparing candidate commit 1098f3c in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics. |
Benchmarks [ tracer ]Benchmark execution time: 2026-03-04 12:25:07 Comparing candidate commit e162af9 in PR branch Found 2 performance improvements and 0 performance regressions! Performance is the same for 190 metrics, 2 unstable metrics. scenario:MessagePackSerializationBench/benchMessagePackSerialization
scenario:MessagePackSerializationBench/benchMessagePackSerialization-opcache
|
fb4432d to
8d1029e
Compare
|
This PR is so large that github will only permit me to review it one file at a time. I didn't even know that was a thing! You're going to need to break it down into a series of smaller PRs, probably. |
a218cd6 to
332fd93
Compare
|
@morrisonlevi I've had success for very big PRs with the PHPStorm/CLion github integrations in the past. Doesn't matter for small PRs, but can definitely recommend it for extra-large PRs :-) |
fc9100c to
718f800
Compare
d68a588 to
5d247e2
Compare
Changes the request_exec message format from [rasp_rule, data] to [data, options_map]. The options map supports: - rasp_rule: string (same as before) - subctx_id: optional string (accepted but ignored) - subctx_last_call: optional bool (accepted but ignored) This prepares the protocol for curl/subcontext support while maintaining backwards compatibility. The subcontext fields are accepted by the protocol but not implemented in the business logic. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Upgrade testcontainers - Add some assertions - Better debug output for metrics
719e83e to
c100434
Compare
46b5f2f to
ec93c63
Compare
ec93c63 to
bb60003
Compare
0385bb7 to
d06640c
Compare
Compile the test fixture library via build.rs using the target's C compiler (cc crate), rather than invoking rustc at test runtime. The runtime rustc approach produced a binary with dependencies incompatible with musl's dynamic linker, causing dlopen to fail with "No such file or directory" even though the .so was present. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The function searched /proc/self/maps for "/libddappsec-helper.so" but the Rust helper loads as "libddappsec-helper-rust.so". The suffix "-rust" before ".so" means the old substring never matched, so the fallback path resolution failed causing client_init to return an error. Match "/libddappsec-helper" as a prefix so both the C++ and Rust variants are found. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sets up a PHP 8.5/Apache container with the Datadog tracer installed from locally-downloaded CI artifacts, plus a Datadog agent container pointed at the staging site (datad0g.com). AppSec is left at its default (unset) so its enabled state is managed by Remote Configuration. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Description
Passing integration and system-tests.
Further integration into sidecar and protocol changes pending.
Reviewer checklist