Walkthrough

Introduces architecture-specific build constraints for read-only checks. Excludes readonly.go on s390x/ppc64/ppc64le and adds readonly_strict.go for those architectures with isReadOnly always returning false. No exported API changes; behavior on other architectures remains unchanged.

Changes

Sequence Diagram(s)

sequenceDiagram
    participant Dev as Build System
    participant App as Application
    participant RO as isReadOnly

    rect rgb(240,245,255)
    note over Dev: Compile-time selection
    Dev->>Dev: Check GOARCH
    alt GOARCH in {s390x, ppc64, ppc64le}
        Dev->>App: Include readonly_strict.go
        note over App: isReadOnly always false
    else Other architectures
        Dev->>App: Include readonly.go
        note over App: isReadOnly performs .rodata check
    end
    end

    rect rgb(245,255,245)
    note over App,RO: Runtime call
    App->>RO: isReadOnly(ptr)
    alt Strict-arch build
        RO-->>App: false
    else Other archs
        RO-->>App: result of .rodata check
    end
    end

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A bunny bounds through arches three,
S/390 winds and PPC,
I sniff the bytes—no trips today,
Read-only checks hop out the way.
On friendly fields they still may run,
But strict terrains? We keep it fun. 🐇✨

Pre-merge checks and finishing touches

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.

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

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.64%. Comparing base (b25339d) to head (82b09b3).
⚠️ Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main    #3772      +/-   ##
==========================================
- Coverage   91.70%   91.64%   -0.06%     
==========================================
  Files         113      113              
  Lines       11940    11940              
==========================================
- Hits        10949    10943       -6     
- Misses        728      733       +5     
- Partials      263      264       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.