refactor(build): replace eslint/prettier with oxlint/oxfmt, add debug…#160
Merged
Conversation
… build variant, remove checksum cache Linting/formatting: - Replace eslint + prettier with oxlint and oxfmt - Add .oxlintrc.json and .oxfmtrc.json config files - Remove eslint.config.mjs, prettier.config.js - Update package.json: lint/format scripts; remove eslint/prettier devDeps; add oxlint Debug/release build variants: - Add PRIVMX_BUILD_TYPE CMake cache variable to all five C++ components (async-engine, webendpoint-cpp, drv-crypto, drv-ecc, drv-net) - Debug: -O0 -g, ASSERTIONS=2, SAFE_HEAP=1, STACK_OVERFLOW_CHECK=2, no LTO - Release: unchanged (-O3, LTO, ASSERTIONS=0, SAFE_HEAP=0) - Add build:debug and build:wasm:debug npm scripts - pipeline.sh reads PRIVMX_BUILD_TYPE from env; passes it to build_api - build_api, build_async_engine, build_webdrivers read build type; reconfigure only when CMakeCache.txt is absent or the build type changed - Add configure_if_needed helper in build_webdrivers to avoid repetition Remove custom checksum cache: - Delete scripts/check_dir_checksum - Remove check_dir_checksum calls, RESULT case/esac blocks, and need_rebuild sentinel logic from build_gmp, build_poco, build_privmx_endpoint, build_pson, build_secp, build_api, build_async_engine, build_webdrivers - Replace with: configure step guarded by CMakeCache.txt / Makefile existence; make always runs (Make's own dependency graph skips unchanged units) Add clean:wasm script: - scripts/clean_wasm removes build-emscripten/ from every first-party C++ component and all dependency_sources/ subdirectories - Add "clean:wasm" npm script README: - Update build scripts table to include build:debug, build:wasm:debug, clean:wasm - Add "Release vs Debug builds" section with flag comparison table
138b137 to
d9c29bd
Compare
zurekz
approved these changes
Apr 28, 2026
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.
… build variant, remove checksum cache
Linting/formatting:
Debug/release build variants:
Remove custom checksum cache:
Add clean:wasm script:
README: