build: drop full debug info from default profiles - #581
Merged
Conversation
Local target/ reached 14 GB, 12 GB of it release artifacts, because [profile.release] carried debug = true while repository policy runs the normal verification suite in release mode. Release now uses line-tables-only, which keeps file:line RUST_BACKTRACE output; a new release-debug profile inherits release with full debug info for debugger sessions. dev/test pin debug = 0 while keeping debug assertions and overflow checks, matching the shared build-artifact hygiene rule (tensor4all/tensor4all-agent-rules#9) and tenferro-rs practice. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
shinaoka
enabled auto-merge (squash)
August 8, 2026 17:14
Repository rules reviewRepository rules review (ae655a9ec08a0c3df8c25369b84be0d04e2a2bf3...9206d9ff7b4137535256862fab4050d77f2c0431) Verdict: pass LLM review: 1 chunk(s) (945 chars) in 5.1s; 0 finding(s) returned, 0 kept, 0 dropped by diff-anchor filtering. No findings. |
This was referenced Aug 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Local
target/measured 14 GB (release 12 GB, debug 2.7 GB) because[profile.release] debug = truecombines with the repository policy of running normal verification in release mode, so every test binary and dependency rlib carries full debug info. Among the 12 GB were 7 orphaned feature/rev variants oflibtenferro_cpualone (596 MB).Changes to the workspace
Cargo.toml:[profile.release] debug = "line-tables-only": keeps readableRUST_BACKTRACEfile:line output (the stated reason for the previousdebug = true) at a fraction of the size.[profile.release-debug]inheriting release withdebug = true, for sessions that attach a debugger:cargo build --profile release-debug.[profile.dev]/[profile.test]pindebug = 0while keepingdebug-assertionsandoverflow-checks, matching tenferro-rs practice. One-command override:CARGO_PROFILE_DEV_DEBUG=2.Follows the shared build-artifact hygiene rule proposed in tensor4all/tensor4all-agent-rules#9. Stale-artifact pruning and CI-profile work (strip, release overflow-checks) are tracked in #566 and not part of this change.
Verification
cargo tree --workspaceandcargo metadataparse the new profiles (profile schema is validated at manifest load).Note: first build after merge is a full rebuild (profile flags change); reclaiming existing bloat needs a one-time
cargo clean --release.🤖 Generated with Claude Code