fix: add NO_RAIQUB_SOURCEGENERATORS_POLYFILL flag to disable embedded polyfills

[skarllot]

Summary

• Guards the embedded InternalPolyfill types (SpanExtensions, SpanLineEnumerator, StringBuilderMemory) behind a #if !NO_RAIQUB_SOURCEGENERATORS_POLYFILL compile-time flag
• Fixes duplicate type definition errors for projects that already provide these types (e.g. via the PolyFill library)
• Documents the flag in both the root README.md and src/InterpolationCodeWriter.CSharp.Sources/README.md

Test plan

• Verify the project builds normally (without the flag) — polyfill types should be compiled as before
• Verify the project builds with NO_RAIQUB_SOURCEGENERATORS_POLYFILL defined — polyfill types should be excluded
• Verify no regression in existing tests

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • Added configuration option to disable embedded polyfill types via compiler constant, preventing duplicate type definition errors when targeting older frameworks.

• Documentation

  • Added "Advanced Configuration" and "Configuration" sections to README files with instructions for disabling built-in polyfills.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.88%. Comparing base (768b7ab) to head (6ea8e8f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #261   +/-   ##
=======================================
  Coverage   88.88%   88.88%           
=======================================
  Files          19       19           
  Lines        1332     1332           
  Branches      101      101           
=======================================
  Hits         1184     1184           
  Misses        104      104           
  Partials       44       44           

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 93ed83a5-0ae9-4ade-ba56-3aaee81cffb8

📥 Commits

Reviewing files that changed from the base of the PR and between 768b7ab and 6ea8e8f.

📒 Files selected for processing (5)

• README.md
• src/InternalPolyfill/SpanExtensions.cs
• src/InternalPolyfill/SpanLineEnumerator.cs
• src/InternalPolyfill/StringBuilderMemory.cs
• src/InterpolationCodeWriter.CSharp.Sources/README.md

---

📝 Walkthrough

Walkthrough

This PR adds compile-time guards (NO_RAIQUB_SOURCEGENERATORS_POLYFILL) around three internal polyfill types to allow consuming projects to disable embedded polyfills. Documentation updates to two README files explain how to define this compiler constant to prevent duplicate type definition errors.

Changes

Cohort / File(s)	Summary
Documentation Updates README.md, src/InterpolationCodeWriter.CSharp.Sources/README.md	Added "Advanced Configuration" and "Configuration" sections documenting the NO_RAIQUB_SOURCEGENERATORS_POLYFILL compiler constant for disabling embedded polyfills.
Internal Polyfill Conditional Compilation src/InternalPolyfill/SpanExtensions.cs, src/InternalPolyfill/SpanLineEnumerator.cs, src/InternalPolyfill/StringBuilderMemory.cs	Wrapped internal polyfill types with #if !NO_RAIQUB_SOURCEGENERATORS_POLYFILL / #endif directives to conditionally exclude them from compilation when the constant is defined.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• refactor: Move internal polyfills to Internals namespace #250: Both PRs modify the same internal polyfill types (SpanExtensions, SpanLineEnumerator, StringBuilderMemory); this PR adds conditional compilation guards while the related PR refactors/renames them to an Internals namespace.
• chore: Create read-me files for each package #225: Both PRs update the src/InterpolationCodeWriter.CSharp.Sources/README.md file—the related PR introduces the file while this PR adds configuration guidance.
• feat: Replace T4 template support with string interpolation #221: Directly related as it introduces the internal polyfill types (SpanExtensions, SpanLineEnumerator, StringBuilderMemory) that this PR now conditionally compiles.

Suggested labels

enhancement

Suggested reviewers

• fgodoy-lepaho

Poem

🐰 With polyfills wrapped in guards so tight,
Consumers choose what fits just right,
A constant flips the switch with ease,
No duplicate types to displease,
The sources dance to a gentler beat! ✨

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/disable-polyfill-flag

---

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.}