Render the Include Walk Once per build_dist.py --check Run

[ptr727]

In --check mode main() computed the include drift and then is_stale() computed it again, so one run rendered every include region twice. is_stale() now takes the drift list main() already holds, and falls back to walking the includes when called without one. Exit codes and messages are unchanged. A new test pins one render per include document for both a clean and a drifted tree, and fails against the previous code.

Closes on promotion: #1379

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: f5262bfb-2caf-4fa8-963f-1f191b4b15fe

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@b7add51). Learn more about missing BASE report.

Additional details and impacted files

@@            Coverage Diff             @@
##             develop    #2241   +/-   ##
==========================================
  Coverage           ?   56.65%           
==========================================
  Files              ?       16           
  Lines              ?     7482           
  Branches           ?        0           
==========================================
  Hits               ?     4239           
  Misses             ?     3243           
  Partials           ?        0           

Flag	Coverage Δ
python-3.13	56.65% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

[Copilot]

Copilot review overview

🟢 Approval recommended

The change is a minimal, backward-compatible refactor whose correctness and single-render behavior are verified and covered by a new regression test, with no objective issues found.

Review effort: Balanced
Findings: None

What changed in this PR

This pull request removes a redundant include-region render in scripts/build_dist.py's --check path. Previously main() computed the drift via include_drift() and then is_stale() computed it again internally, so every check run rendered and compared each include region twice. The change threads the already-computed drift list into is_stale(), eliminating the second walk while keeping exit codes and messages unchanged. This supports the fleet's skill-distribution tooling, where build_dist.py --check is the gate that detects when generated .github/skills/ and .claude-plugin/fleet-skills/ trees are behind their .agents/skills/ sources.

Changes:

• is_stale() gains an optional drift parameter and reuses a passed list instead of re-walking, falling back to include_drift() when called without one.
• main() passes its already-computed drift into is_stale(drift).
• A new test pins exactly one render per include document for both a clean and a drifted tree, failing against the prior double-walk code.

File	Description
scripts/​build_dist.py	Adds optional drift param to is_stale() and reuses main()'s computed drift to avoid a second include walk.
tests/​test_build_dist.py	Adds a regression test asserting one filled_bytes render per include document across clean and drifted --check runs.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.