enable build comparison and stack usage report
What the developer wrote
Authored by Mathias BROUSSET
enable build comparison and stack usage report
What changed, and why it matters
This commit only changes GitHub CI workflow settings. It turns on two build-reporting features: comparing compiled outputs across builds and reporting how much stack memory the app uses during tests. There is no change to the actual Bitcoin app code, no bug fix, and no security patch.
No security action needed. Treat as a normal CI/infrastructure change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies .github/workflows/build_and_functional_tests.yml, passing build_comparison=true and enable_stack_consumption=true to a reusable build workflow, and post_stack_consumption=true to a reusable test workflow. These are observability/CI-quality flags; they do not alter firmware logic, cryptography, parsing, or access controls.
Changed components
.github/workflows/build_and_functional_tests.ymlInspect captured patch +3 / −0
diff --git a/.github/workflows/build_and_functional_tests.yml b/.github/workflows/build_and_functional_tests.yml
index 9a0ed3f..a871b2a 100644
--- a/.github/workflows/build_and_functional_tests.yml
+++ b/.github/workflows/build_and_functional_tests.yml
@@ -34,6 +34,8 @@ jobs:
with:
upload_app_binaries_artifact: "compiled_app_binaries"
flags: "DEBUG=0 COIN=bitcoin_testnet"
+ build_comparison: true
+ enable_stack_consumption: true
ragger_tests:
name: Run ragger tests using the reusable workflow
@@ -45,3 +47,4 @@ jobs:
# when merging a PR, we run the tests with the --enable_slow_tests parameter
test_options: ${{ github.event_name == 'push' && '--enable_slow_tests' || '' }}
regenerate_snapshots: ${{ github.event_name == 'workflow_dispatch' && inputs.golden_run == 'Open a PR' }}
+ post_stack_consumption: true
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.