Skip to content

fix(selur-compose): recover poisoned mock locks - #211

Merged
hyperpolymath merged 1 commit into
mainfrom
fix/final-alert-audit
Aug 29, 2026
Merged

fix(selur-compose): recover poisoned mock locks#211
hyperpolymath merged 1 commit into
mainfrom
fix/final-alert-audit

Conversation

@hyperpolymath

Copy link
Copy Markdown
Collaborator

Outcome

Closes the one still-reproducible high/error Hypatia source finding from Burble’s stale alert set:

  • replaces every Mutex::lock().unwrap() in the selur-compose test driver with a typed helper that recovers poisoned fixture state
  • repairs invalid shell-style # comments in two Rust modules; these made the selur-compose workspace fail to parse before tests could run
  • removes the resulting unused-mut warning
  • formats the three touched Rust modules

The old SSL-verification and unsafe-term findings are already fixed on current main; their referenced source path is gone or uses binary_to_term(..., [:safe]). The neural-state @bitCast operations are intentional fixed-width float/integer serialization, not pointer reinterpretation. The credential findings are documentation/module-name false positives rather than embedded values.

Verification

  • cargo test --manifest-path tools/selur-compose/Cargo.toml -p selur-compose-driver: 46/46 unit tests passed
  • doc tests: 2 passed, 1 intentionally ignored
  • git diff --check: pass

@gitar-bot

gitar-bot Bot commented Aug 29, 2026

Copy link
Copy Markdown

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

@codacy-production

Copy link
Copy Markdown
Contributor

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 duplication

Metric Results
Duplication 0

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@hyperpolymath
hyperpolymath merged commit 4271023 into main Aug 29, 2026
18 checks passed
@hyperpolymath
hyperpolymath deleted the fix/final-alert-audit branch August 29, 2026 11:40
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