Conversation
Reduces target/ disk usage while keeping backtrace file:line info. Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
There was a problem hiding this comment.
Code Review
This pull request updates the Cargo.toml file to include a [profile.dev] section with a reduced debug level. Feedback suggests improving the clarity of the configuration by using the named value "line-tables-only" instead of the numeric level 1 and recommends grouping this new section with other profile definitions for better organization.
| [profile.dev] | ||
| debug = 1 |
There was a problem hiding this comment.
For better maintainability and clarity, consider the following improvements:
- Organization: Group this
[profile.dev]section with other profile definitions, such as[profile.serious](line 87), to keep the configuration organized. - Readability: Use the named value
"line-tables-only"instead of the numeric1. It is more explicit and easier to understand for developers who may not be familiar with Cargo's numeric debug levels.
Note that while this change reduces disk usage, it will limit the ability to inspect local variables in a debugger. If you wish to preserve full debugging for the local crate while still saving space, you could use package-specific overrides for dependencies (e.g., [profile.dev.package."*"]).
| [profile.dev] | |
| debug = 1 | |
| [profile.dev] | |
| debug = "line-tables-only" |
Greptile SummarySets Confidence Score: 5/5Safe to merge — single-line config change with no functional impact on tests or backtraces. The change is a well-understood Rust build tuning option. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[cargo build / cargo test] --> B{Profile?}
B -- dev --> C[profile.dev]
B -- release --> D[profile.release]
C --> E["debug = 1 (line tables only)\nPreviously: debug = 2 (full DWARF)"]
E --> F[Smaller target/ directory\nFaster linking]
E --> G[Backtraces: file:line ✅\nDebugger variable inspection: ❌]
D --> H[Unchanged release profile]
Reviews (1): Last reviewed commit: "chore: set dev profile debug to 1" | Re-trigger Greptile |
### 🚀 Features - **(builtins)** add `buildifier` format and lint built-ins by [@plx](https://github.com/plx) in [#896](#896) ### 🐛 Bug Fixes - **(step)** only auto-batch when rendered command exceeds ARG_MAX by [@jdx](https://github.com/jdx) in [#901](#901) ### 📚 Documentation - thank Namespace for GitHub Actions runner support by [@jdx](https://github.com/jdx) in [#895](#895) ### 🔍 Other Changes - **(ci)** use !cancelled() instead of always() for final job by [@jdx](https://github.com/jdx) in [#906](#906) - **(docs)** remove shrill.en.dev analytics script by [@jdx](https://github.com/jdx) in [#903](#903) - remove rust-cache from release jobs by [@jdx](https://github.com/jdx) in [#893](#893) - invert CLAUDE.md/AGENTS.md so AGENTS.md is canonical by [@jdx](https://github.com/jdx) in [#905](#905) - set dev profile debug to 1 by [@jdx](https://github.com/jdx) in [#907](#907) ### 📦️ Dependency Updates - update anthropics/claude-code-action digest to fefa07e by [@renovate[bot]](https://github.com/renovate[bot]) in [#897](#897) - update jdx/mise-action digest to 1648a78 by [@renovate[bot]](https://github.com/renovate[bot]) in [#898](#898) - update apple-actions/import-codesign-certs action to v7 by [@renovate[bot]](https://github.com/renovate[bot]) in [#900](#900) - update autofix-ci/action action to v1.3.4 by [@renovate[bot]](https://github.com/renovate[bot]) in [#899](#899) - lock file maintenance by [@renovate[bot]](https://github.com/renovate[bot]) in [#908](#908) ### New Contributors - @plx made their first contribution in [#896](#896) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk release bookkeeping: primarily version string updates across manifests and docs with no functional code changes in this diff. > > **Overview** > Updates the project for the `v1.45.0` release by bumping the crate/CLI version (`Cargo.toml`, `Cargo.lock`, `hk.usage.kdl`, generated CLI docs) and adding the `1.45.0` entry to `CHANGELOG.md`. > > Refreshes documentation and example configs to reference the new versioned Pkl package URLs (`docs/*.md`, `docs/public/*.pkl`, `hk-example.pkl`). > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit cfe2da5. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by: mise-en-dev <[email protected]>
Summary
Reduces
target/disk usage by setting[profile.dev] debug = 1, while keeping backtrace file:line info intact.The default
debug = 2writes full DWARF — including variable/type info that's only needed when stepping through code in a debugger (lldb/gdb). Withdebug = 1, panics,RUST_BACKTRACE=1, and error chains still resolve to file:line; only debugger variable inspection is reduced.In practice this typically shrinks
target/by a meaningful percentage, which adds up across worktrees.Test plan
cargo buildstill succeedsThis PR was generated by an AI coding assistant.
Note
Low Risk
Low risk: build configuration-only change that reduces debug info in
devbuilds and may only affect stepping/variable inspection in debuggers.Overview
Adjusts Rust
devbuild settings by adding[profile.dev] debug = 1inCargo.toml, reducing the amount of generated debug symbols (and typicallytarget/size) while keeping file/line backtrace information.Reviewed by Cursor Bugbot for commit e7f300b. Bugbot is set up for automated code reviews on this repo. Configure here.