Skip to content

feat: xseed-improvements consolidation (18 commits)#37

Closed
bakerboy448 wants to merge 18 commits into
mainfrom
xseed-improvements
Closed

feat: xseed-improvements consolidation (18 commits)#37
bakerboy448 wants to merge 18 commits into
mainfrom
xseed-improvements

Conversation

@bakerboy448
Copy link
Copy Markdown
Collaborator

Summary

Consolidation PR for the xseed-improvements branch (18 commits ahead of main). Feature work that never landed:

  • xseed.sh v2.0.0 + .env-driven config
  • zfsburn.sh env support
  • omegabrr_upgrade.sh v2
  • notifiarr-branch-builder.sh v3
  • dupe.sh v2 + dupe fixes
  • qbm-qbit.sh v2 + better logging
  • jdupes support in .env.sample

Why this PR exists

Branch never had a PR opened. Reviewer should:

  • Decide which commits are still relevant (some files may have moved on in main)
  • Cherry-pick a curated subset or squash the lot
  • Close without merge if the work has been superseded by other paths

Test plan

  • Inspect 18-commit log: git log --oneline main..xseed-improvements
  • Cherry-pick or squash curated subset
  • Smoke-test each script against a sandbox dataset

🤖 Auto-opened during cross-repo branch consolidation sweep.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 10, 2026

Warning

Rate limit exceeded

@bakerboy448 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 59 minutes and 57 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 3b600424-eba0-433c-8796-ce5e013dfc2f

📥 Commits

Reviewing files that changed from the base of the PR and between 77a0845 and 43aba9f.

📒 Files selected for processing (8)
  • .env.sample
  • .gitignore
  • dupe.sh
  • notifiarr-branch-builder.sh
  • omegabrr_upgrade.sh
  • qbm-qbit.sh
  • xseed.sh
  • zfsburn.sh
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch xseed-improvements

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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 and usage tips.

bakerboy448 added a commit that referenced this pull request May 11, 2026
Cherry-picked from xseed-improvements branch (PR #37) — keeping only
the qbittorrent-manage + notifiarr-deploy helpers, not the xseed.sh
work superseded by qui-xseed.sh on main.

Inline fixes during cherry-pick:
- qbm-qbit.sh: aligned QBQBM_* var names to consumer line (was
  defining QBQBM_VENV_PATH etc. but invoking unset $VENV_PATH etc.)
- qbm-qbit.sh: removed unused QBQBM_LOCK_TIME
- qbm-qbit.sh: fixed $LOCK reference in remove_lock (was unset)
- notifiarr-branch-builder.sh: replaced unicode smart quotes (“/”, –)
  with ASCII equivalents so bash parses correctly

Co-Authored-By: Claude <noreply@anthropic.com>
@bakerboy448
Copy link
Copy Markdown
Collaborator Author

Superseded by #38 (qbm-qbit + notifiarr cherry-picks) and #39 (bulk shfmt). xseed.sh + .env reworks intentionally dropped — main has migrated to qui-xseed.sh.

@bakerboy448 bakerboy448 deleted the xseed-improvements branch May 11, 2026 01:31
bakerboy448 added a commit that referenced this pull request May 11, 2026
## Summary
- Cherry-picks 2 standalone scripts from the stale PR #37
(xseed-improvements)
- Drops xseed.sh and its env reworks (superseded by qui-xseed.sh on
main)
- Inline-fixes pre-existing bugs from the branch:
- qbm-qbit.sh: var name mismatch (QBQBM_* defined, $VENV_PATH etc.
invoked); fixed
  - qbm-qbit.sh: $LOCK reference in remove_lock (was unset); fixed
  - qbm-qbit.sh: unused QBQBM_LOCK_TIME removed
- notifiarr-branch-builder.sh: unicode smart quotes ("/", –) replaced
with ASCII

## Test plan
- [ ] CI green (pre-commit shellcheck --severity=warning)
- [ ] qbm-qbit.sh shellcheck-clean
- [ ] notifiarr-branch-builder.sh shellcheck-clean
- [ ] AJ to close PR #37 after this lands (xseed.sh + .env reworks
intentionally dropped)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Chores**
* Added automated source build and deployment script enabling
interactive repository cloning, dependency verification, branch
selection, project compilation, binary installation, and systemd service
lifecycle management including start, stop, and status verification.
* Added background execution wrapper script implementing exclusive
process locking to prevent concurrent service execution, with
configurable virtual environment paths and environment variable support.

[![Review Change
Stack](https://storage.googleapis.com/coderabbit_public_assets/review-stack-in-coderabbit-ui.svg)](https://app.coderabbit.ai/change-stack/baker-scripts/StarrScripts/pull/38)

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants