chore: resolve multiple CI, CLI refactor, and WASM size/test issues#936
Merged
Gbangbolaoluwagbemiga merged 1 commit intoJun 26, 2026
Merged
Conversation
Closes HyperSafeD#331 Closes HyperSafeD#537 Closes HyperSafeD#540 Closes HyperSafeD#547
|
@ritik4ever is attempting to deploy a commit to the gbangbolaoluwagbemiga's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@ritik4ever Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
5be7c9b
into
HyperSafeD:main
11 of 23 checks passed
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
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.
This PR bundles fixes, size optimization patches, test coverage, and documentation for several distinct scale/polish priorities across the repository.
Changes Included:
[ci] Release workflow: build pre-compiled binaries for Linux, macOS, and Windows
Modified .github/workflows/release.yml to automatically inject CHANGELOG.md directly into the release body using body_path.
Updated README.md to list explicit direct download URLs for pre-compiled binaries alongside the cargo install instruction.
tooling/sanctifier-cli: Refactor for clearer module boundaries (crates.io metadata polish)
Hardened module boundaries in main.rs by stripping internal duplicate mod declarations and correctly importing them from sanctifier_cli::*.
Polished crates.io publishing metadata in Cargo.toml by including accurate attribution (authors).
tooling/sanctifier-wasm: Harden input validation + error messages (size optimization (wasm + JS glue))
Implemented strict size optimization for release WASM bindings using the [profile.release] directives in Cargo.toml (opt-level = "z", codegen-units = 1, lto = true, strip = true).
tooling/sanctifier-wasm: Add unit tests + fixtures (source-map support for diagnostics)
Added test_source_map_diagnostics_fixture inside analysis.rs tests to assert finding structures correctly populate properties mapped by upcoming source-map pipelines.
Closes Issues:
Closes #331
Closes #537
Closes #540
Closes #547