Skip to content

fix: remove hard-coded paths and patch overrides#8

Open
noahgift wants to merge 1 commit intomainfrom
fix/remove-hard-coded-paths
Open

fix: remove hard-coded paths and patch overrides#8
noahgift wants to merge 1 commit intomainfrom
fix/remove-hard-coded-paths

Conversation

@noahgift
Copy link
Contributor

@noahgift noahgift commented Mar 4, 2026

Summary

  • Remove path = \"../../../probar/crates/probar\" local path override from crates/ttop/Cargo.toml dev-dependency on jugar-probar — now resolves from crates.io (v1.0.1)
  • Remove all /mnt/nvme-raid0 hard-coded path strings from crates/ttop/src/panels.rs test assertions, comments, and the mount_marker() function logic — replaced with generic /mnt/nvme paths
  • Update doc comment in crates/ttop/src/analyzers/treemap.rs to remove machine-specific path reference

These changes break clean-room CI builds because the local path = override requires the probar repo to be checked out on the CI machine, and the /mnt/nvme-raid0 prefix is specific to this development machine's NVMe RAID mount.

Spec: sovereign-stack-protected-branch-strategy.md (Section 5)

Test plan

  • cargo check passes in /home/noah/src/trueno-viz (root workspace)
  • cargo check passes in /home/noah/src/trueno-viz/crates/ttop (ttop crate, now resolves jugar-probar from crates.io)
  • No remaining /mnt/nvme-raid0 references in any .rs files
  • No remaining path = "../ in any Cargo.toml files

🤖 Generated with Claude Code

Remove [patch.crates-io] path overrides and /mnt/nvme-raid0 references.
These break clean-room CI builds.

Spec: sovereign-stack-protected-branch-strategy.md (Section 5)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant