docs: add codebase minimization guideline to AGENTS.md

[euxaristia]

Add a project convention encouraging contributors and agents to unify functions/methods where possible, preferring a single cross-platform function with conditional checks over duplicating helpers per platform.

This principle was validated in PR #660 where we consolidated windowsSandboxChildEnv and sandboxEnvironmentForCommand into a single unified function, saving ~19 lines of code without removing any functionality.

Summary by CodeRabbit

• Documentation
  • Added a project convention to encourage a single cross-platform implementation (with conditional checks) rather than duplicating platform-specific helpers.

[coderabbitai]

Warning

Review limit reached

@euxaristia, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 16 minutes

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5034990a-7af0-42cf-95a8-6d169bfe4873

📥 Commits

Reviewing files that changed from the base of the PR and between 7af0831 and 180a909.

📒 Files selected for processing (1)

• AGENTS.md

Walkthrough

AGENTS.md adds a convention favoring unified cross-platform functions and methods over duplicated platform-specific helpers.

Changes

Project conventions

Layer / File(s)	Summary
Cross-platform function guideline AGENTS.md	Adds guidance to use shared cross-platform functions with conditional logic rather than duplicating platform-specific helpers.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• Gitlawb/zero#438: Both changes update the repository’s AGENTS.md conventions file.

Suggested reviewers: kevincodex1

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 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 AGENTS.md docs update adding a codebase minimization guideline.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

[Vasanthdev2004]

Trivial docs change — adds one bullet to the AGENTS.md example conventions block encouraging contributors to unify cross-platform helpers instead of duplicating per-platform. It's reasonable, reads in the same imperative voice as the surrounding bullets, and names no external tools or competitors. Merges clean. Approving.

[kevincodex1]

please fix conflict @euxaristia

[euxaristia]

Rebased onto main (was 7 commits behind) and force-pushed. Checked with git merge-tree beforehand: there was no actual content conflict, it resolved cleanly. The one-line AGENTS.md addition applies as-is on top of current main.