Skip to content

ci: scope Bloom Shell entrypoint cppcheck - #586

Merged
boburning merged 1 commit into
mainfrom
perf/ci-runtime-scope
Aug 30, 2026
Merged

boburning merged 1 commit into
mainfrom
perf/ci-runtime-scope

Conversation

@boburning

Copy link
Copy Markdown
Owner

Summary

  • route a pull-request change confined to src/bloomShell/main.c through cppcheck's owning translation unit instead of the entire src/bloomShell directory
  • retain the complete Bloom Shell target for ARM compilation
  • retain affected Bloom Shell and frontend native and AddressSanitizer tests
  • retain the five focused Shell integration files and all exhaustive merge-group/manual behavior
  • extend CI-scope regression coverage for entrypoint-only and mixed Bloom Settings/entrypoint changes

This is a focused continuation of the affected-component CI work in #571 and #573.

Measured cause and result

PR #584 changed four lines in src/bloomShell/main.c, but its static-analysis resolver selected the entire Bloom Shell directory. Hosted cppcheck then spent 190 seconds in the analysis step and 3m24s in the job overall: https://github.com/boburning/BloomOS/actions/runs/33327223569

With this branch's exact PR-only resolver output, reduced cppcheck 2.19 analyzes src/bloomShell/main.c in 12.25 seconds locally. The equivalent #573 owning-file policy produced a 14-second hosted static-analysis job, so this removes the same avoidable directory expansion without changing compile or integration coverage.

For the exact #584-shaped path set (main.c plus its Shell regression test), the resolver now returns:

  • risk: 2
  • ARM build: bloomShell
  • cppcheck: src/bloomShell/main.c
  • native and ASan: BloomShell*:BloomFrontend*
  • Shell: runtime, Safe Mode, Settings, typography, and GameSwitcher control suites
  • visual gallery: none

Validation

Safety and limitations

Related: #518, #520, #571, #573, #584.

@boburning
boburning merged commit 94104f6 into main Aug 30, 2026
5 checks passed
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