build: update template to avoid extra directory

[alandtse]

Summary by CodeRabbit

• Chores

  • Added a new binary directory setting to the automated-deploy build preset.
  • Adjusted the release script to always run the configure step (stamp-file check removed).

• Documentation

  • Updated build and deployment command examples (including editor run-on-save and shader workflow docs) to use the standard build directory for shader copy/deploy tasks.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b9b425d and f90c45f.

📒 Files selected for processing (2)

• .claude/CLAUDE.md
• BuildRelease.bat

---

📝 Walkthrough

Walkthrough

Added binaryDir to the ALL-WITH-AUTO-DEPLOYMENT configure preset and changed references from build/ALL-WITH-AUTO-DEPLOYMENT to build/ALL across documentation and tooling; also removed --check-stamp-file from the initial configure step in BuildRelease.bat.

Changes

Cohort / File(s)	Summary
CMake Preset Configuration CMakeUserPresets.json.template	Added binaryDir property with value "${sourceDir}/build/ALL" to the ALL-WITH-AUTO-DEPLOYMENT configure preset.
Documentation updates docs/development/README.md, docs/development/shader-workflow.md, docs/development/vscode-setup.md	Replaced occurrences of build/ALL-WITH-AUTO-DEPLOYMENT with build/ALL for COPY_SHADERS / DEPLOY_ALL examples and VSCode run-on-save config.
Dev tooling / guidance .github/copilot-instructions.md, .claude/CLAUDE.md	Updated example CMake build commands to use ./build/ALL --target COPY_SHADERS / DEPLOY_ALL instead of ./build/ALL-WITH-AUTO-DEPLOYMENT.
Build script BuildRelease.bat	Removed the --check-stamp-file argument from the initial CMake configure step; left subsequent build and error handling unchanged.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• build(cmake): restore copy_shaders fast deployment #1756 — Related changes to CMake presets and deployment workflow involving ALL vs ALL-WITH-AUTO-DEPLOYMENT.
• build: add cmake install commands #1372 — Changes to presets and build-target packaging that touch the same presets/targets and build-directory usage.
• docs: add AI instructions #1332 — Documentation changes referencing the build directory that overlap with .claude/CLAUDE.md edits.

Suggested reviewers

• doodlum

Poem

🐰 I nudged a path from old to new,
A tiny preset and docs too,
The build now points where outputs play,
A cleaner hop, a brighter day,
I twitch my nose — the pipeline's true!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly describes the main change: updating a build template to avoid an extra directory (ALL-WITH-AUTO-DEPLOYMENT → ALL). This aligns with the primary modification across multiple build configuration files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@CMakeUserPresets.json.template`:
- Line 6: The "binaryDir" line in CMakeUserPresets.json.template uses spaces for
indentation while the rest of the file uses tabs; update the indentation for the
line containing "binaryDir": "${sourceDir}/build/ALL" to use a single tab
(matching the surrounding entries) so the file's indentation is consistent with
the rest of the file.

[Copilot]

Pull request overview

Updates the user preset template to reuse the existing build/ALL binary directory so using ALL-WITH-AUTO-DEPLOYMENT doesn’t create an additional build/ALL-WITH-AUTO-DEPLOYMENT folder.

Changes:

• Set binaryDir for ALL-WITH-AUTO-DEPLOYMENT to ${sourceDir}/build/ALL to avoid generating an extra build directory.

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

docs/development/vscode-setup.md (2)

1-5: ⚠️ Potential issue | 🟡 Minor

Conventional commit title suggestion (scope missing).

Suggested title: build(presets): avoid extra build directory.
Current title lacks a scope per the repo’s conventional commit guidelines.

---

40-50: ⚠️ Potential issue | 🟡 Minor

Clarify build directory created by the preset.

The command now builds build/ALL, but step 4 still references ALL-WITH-AUTO-DEPLOYMENT without explaining it generates build/ALL. Consider clarifying to avoid confusion for new contributors.

✏️ Suggested doc tweak

-4. Run `cmake --preset ALL-WITH-AUTO-DEPLOYMENT` once to create the build directory
+4. Run `cmake --preset ALL-WITH-AUTO-DEPLOYMENT` once to create `build/ALL`