Skip to content

fix(build): remove sandbox-dependent generated overlay paths#729

Merged
mstykow merged 3 commits intomainfrom
refactor/build-support-version-format
Apr 17, 2026
Merged

fix(build): remove sandbox-dependent generated overlay paths#729
mstykow merged 3 commits intomainfrom
refactor/build-support-version-format

Conversation

@mstykow
Copy link
Copy Markdown
Owner

@mstykow mstykow commented Apr 17, 2026

Summary

  • move the shared build version formatter out of src/ into build_support/ so build-only shared code does not masquerade as a runtime module
  • include build_support/**/* in packaged sources so crates.io source builds still contain the shared helper
  • generate bundled overlay manifests with inlined string contents instead of include_str! absolute paths so Cargo and Buck builds do not depend on matching rustc/build-script sandboxes

Scope and exclusions

  • Included: build script portability fixes, shared helper relocation, and source-package include list updates
  • Explicit exclusions: no runtime license overlay semantics changes, no overlay fixture content changes, and no Buck-specific repository wiring beyond removing the sandbox path dependency

Follow-up work

  • Created or intentionally deferred: CI or Buck-native validation can now verify the sandbox-safe manifest generation on the downstream integration side

mstykow and others added 3 commits April 17, 2026 12:37
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@mstykow mstykow merged commit 7c5c920 into main Apr 17, 2026
14 checks passed
@mstykow mstykow deleted the refactor/build-support-version-format branch April 17, 2026 11:04
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