Skip to content

Production-readiness audit: fixes, regression tests, docs - #1

Merged
hellpuffyt merged 3 commits into
mainfrom
audit/production-readiness
Oct 24, 2026
Merged

hellpuffyt merged 3 commits into
mainfrom
audit/production-readiness

Conversation

@hellpuffyt

Copy link
Copy Markdown
Owner

Summary

Commits

  • Add production-readiness report
  • Fix clone URL, document exit codes, env vars and limitations; add SECURITY.md, .editorconfig, CI read-only permissions
  • Fix dotfile data loss, symlink/mode clobbering, --force and --dry-run state bugs, fresh-apt installs, slow-network preflight

Evidence

  • Commands, exit codes, test counts and limitations: docs/PRODUCTION_READINESS.md on this branch.
  • Independently re-verified from a fresh clone of this exact commit (c034fb5).
  • Added lines scanned for secrets, personal data and local paths: none.
  • CI: runs on this PR.

Not merged automatically: maintainer review required.

… state bugs, fresh-apt installs, slow-network preflight

- managed blocks: rewrite in place (keeps symlinks and mode); refuse damaged markers instead of deleting the rest of the file
- shell-profile: an up-to-date block is not an apply failure (--force shell-profile)
- --dry-run --force no longer edits the state file
- apt: apt-get update once per run before installing (noninteractive)
- preflight network probe is a HEAD request with a connect timeout
- argument errors exit 3 before any write; profile names validated; --force must name a profile step
- bats regression tests for each
…URITY.md, .editorconfig, CI read-only permissions
Copilot AI balanced review requested due to automatic review settings October 24, 2026 16:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@coderabbitai

coderabbitai Bot commented Oct 24, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0220ebdd-c638-4d2f-acdc-c12c06a5c337
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hellpuffyt
hellpuffyt merged commit e4538fd into main Oct 24, 2026
7 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.

2 participants