Skip to content

feat: add Linux server diagnostics wave - #20

Merged
RomeoRaven merged 1 commit into
masterfrom
feat/linux-server-diagnostics-wave-1
Jul 25, 2026
Merged

feat: add Linux server diagnostics wave#20
RomeoRaven merged 1 commit into
masterfrom
feat/linux-server-diagnostics-wave-1

Conversation

@RomeoRaven

Copy link
Copy Markdown
Owner

Summary

  • add ten standalone Linux/Ubuntu diagnostics for filesystem, memory, services, listeners, updates, backups, endpoints, TLS, containers, and log storage
  • preserve read-only operation and privacy-reduced defaults with explicit metadata opt-ins
  • add catalog, checksum, launcher, README, and mock-driven behavioral coverage
  • harden decimal parsing, TLS identity/chain verification, target syntax, executable path quoting, curl configuration isolation, and collection failure semantics

Verification

  • ./tests/test-scripts.sh — 183/183
  • ./tests/test-linux-modules.sh — 148/148
  • ./tests/test-launcher.sh — 102/102
  • ./onescript doctor — catalog, 14 modules, files, checksums OK
  • sha256sum -c --strict SHA256SUMS — 16/16
  • Bash syntax, pinned ShellCheck, and staged diff checks clean
  • exact-tree fail-closed independent rereview passed with no security concerns, logic errors, or suggestions

Add ten standalone read-only diagnostics with privacy-reduced defaults, catalog integration, checksums, and behavioral coverage for Linux and Ubuntu hosts.

@RomeoRaven RomeoRaven left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Summary

Verdict: Merge-ready

Findings

  • Blocking: none
  • Warnings: none
  • Suggestions: none

Reviewed

  • exact head 232deb58e517573a7211211999124bb40a24f80b against master
  • all 16 changed files and full production-script context
  • read-only and privacy-reduced boundaries
  • option, arithmetic, path, endpoint, TLS, package, container, and failure semantics
  • README, catalog, checksum, launcher, tests, and CI consistency

Proof

  • ./tests/test-scripts.sh: 183/183
  • focused Linux suite: 148/148
  • launcher suite: 102/102
  • doctor and strict checksums: clean
  • pinned ShellCheck: clean
  • PR diff hygiene and targeted security sweeps: clean
  • both GitHub Actions checks: passed

No merge was performed. GitHub does not permit an author to approve their own PR, so this review is submitted as a non-blocking comment.

@RomeoRaven
RomeoRaven merged commit a9067d1 into master Jul 25, 2026
2 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