fix(build): stable generated_profile header path (#36) #64

Merged

timur merged 1 commit from 36-build-header-path into development

2026-04-15 14:23:32 +00:00

timur commented

2026-04-15 14:23:30 +00:00

Owner

Fourth slice on #36. The header line of crates/hero_os_app/src/generated_profile.rs embedded a CARGO_MANIFEST_DIR-derived absolute path, so fresh builds on different dev machines produced a spurious diff on every commit (e.g. /Users/mahmoud/… ↔ /Users/timurgordon/…).

build.rs now emits a stable path — HERO_CONFIG verbatim if set, otherwise just config.toml.

Verification

$ head -1 crates/hero_os_app/src/generated_profile.rs
// Auto-generated by build.rs from config.toml [profiles.default] — do not edit

Test plan

Another developer runs cargo build -p hero_os_app → git diff shows no change to the header line.

Refs #36.

🤖 Generated with Claude Code

Fourth slice on #36. The header line of `crates/hero_os_app/src/generated_profile.rs` embedded a `CARGO_MANIFEST_DIR`-derived absolute path, so fresh builds on different dev machines produced a spurious diff on every commit (e.g. `/Users/mahmoud/…` ↔ `/Users/timurgordon/…`). `build.rs` now emits a stable path — `HERO_CONFIG` verbatim if set, otherwise just `config.toml`. ## Verification ``` $ head -1 crates/hero_os_app/src/generated_profile.rs // Auto-generated by build.rs from config.toml [profiles.default] — do not edit ``` ## Test plan - [ ] Another developer runs `cargo build -p hero_os_app` → `git diff` shows no change to the header line. Refs #36. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

timur added 1 commit

2026-04-15 14:23:30 +00:00

fix(build): emit stable config path in generated_profile header (#36 )

Build and Test / test (pull_request) Failing after 39s

Details

33d66fde59

Previously the header embedded CARGO_MANIFEST_DIR-derived absolute path,
so fresh builds on different dev machines produced a spurious diff on
every commit (e.g. /Users/mahmoud/… vs /Users/timurgordon/…).

Emit a stable, developer-agnostic path instead: use HERO_CONFIG verbatim
if set, otherwise just "config.toml".

Refs #36.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

timur merged commit d011d6e9c6 into development

2026-04-15 14:23:32 +00:00

timur referenced this pull request from a commit

2026-04-15 14:23:33 +00:00

fix(build): stable generated_profile header path (#36) (#64)

timur referenced this pull request

2026-04-15 15:33:21 +00:00

Cleanup: dead code, competing impls, and refactor debris audit #36