#0087 Safety Features #151

Merged
fuzzy merged 1 commit from feature/0087-safety-features into main 2026-07-06 07:25:04 +00:00
Owner

Adds safety features to prevent accidental data loss.

Changes:

  • src/hottea/commit.pywarn_recent_end_date() function
  • src/hottea/cli.py — calls warn_recent_end_date(), backup docs in help epilog
  • tests/test_commit.py — 3 tests for new function
  • tests/test_cli.py — 1 test for backup procedure docs

Safety features:

  • Dry-run mode (--dry-run): preview SQL without writes or API calls
  • Transaction wrapper: SQL wrapped in BEGIN/COMMIT for atomicity
  • Informative comments: per-record commit hash, author, timestamp
  • Recent end-date warning: warns if --end-date is within 30 days
  • Backup documentation: step-by-step backup/restore procedure in --help

Closes: #0087, #0082, #0083, #0084, #0085, #0086

Adds safety features to prevent accidental data loss. **Changes:** - `src/hottea/commit.py` — `warn_recent_end_date()` function - `src/hottea/cli.py` — calls `warn_recent_end_date()`, backup docs in help epilog - `tests/test_commit.py` — 3 tests for new function - `tests/test_cli.py` — 1 test for backup procedure docs **Safety features:** - **Dry-run mode** (`--dry-run`): preview SQL without writes or API calls - **Transaction wrapper**: SQL wrapped in BEGIN/COMMIT for atomicity - **Informative comments**: per-record commit hash, author, timestamp - **Recent end-date warning**: warns if `--end-date` is within 30 days - **Backup documentation**: step-by-step backup/restore procedure in `--help` **Closes:** #0087, #0082, #0083, #0084, #0085, #0086
feat(safety): add recent end-date warning and backup docs
All checks were successful
CI / lint-and-test (pull_request) Successful in 1m26s
209da052da
Add safety features for end-date usage and SQL application.

- warn_recent_end_date(): warns if end-date is within 30 days
- Backup procedure documented in --help epilog
- Dry-run mode, transaction wrapper, comments already existed

Closes #82, #83, #84, #85, #86, #87
fuzzy scheduled this pull request to auto merge when all checks succeed 2026-07-06 07:23:38 +00:00
the.auditor left a comment

Audit Review — PR #151

Clean safety features PR. Recent end-date warning correctly excludes today and dates older than 30 days. Backup procedure is comprehensive (5 steps). Remaining safety features (dry-run, transaction wrapper, comments) were already implemented in earlier PRs.

No suggestions — clean PR.

Summary

Result: COMMENT — no issues found.

## Audit Review — PR #151 Clean safety features PR. Recent end-date warning correctly excludes today and dates older than 30 days. Backup procedure is comprehensive (5 steps). Remaining safety features (dry-run, transaction wrapper, comments) were already implemented in earlier PRs. No suggestions — clean PR. ## Summary **Result:** COMMENT — no issues found.
the.auditor left a comment

APPROVED. Safety features complete. Merge when ready.

APPROVED. Safety features complete. Merge when ready.
fuzzy merged commit 209da052da into main 2026-07-06 07:25:04 +00:00
fuzzy deleted branch feature/0087-safety-features 2026-07-06 07:25:04 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
thwap/hottea!151
No description provided.