Extract repo-intel into standalone repo#15
Merged
Merged
Conversation
repo-intel now lives in its own repo with a CLI, GitHub Action, and Homebrew formula. This removes the in-tree source and build targets: - Remove src/repo-intel/ and the repo-intel-* Makefile targets - Untrack + gitignore stow/bin/repo-intel (kept locally so ~/bin keeps working) - setup.sh: fetch the artifact into stow/bin via curl when not installed by brew - brew.sh: install tyom/tap/repo-intel non-fatally (won't block core packages) - README: point at the standalone repo
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (2)
📜 Recent review details⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
🧰 Additional context used🧠 Learnings (1)📚 Learning: 2026-01-01T21:11:09.058ZApplied to files:
🪛 Shellcheck (0.11.0)scripts/install/brew.sh[info] 59-59: Note that A && B || C is not if-then-else. C may run when A is true. (SC2015) 🔇 Additional comments (1)
📝 WalkthroughWalkthroughThe PR removes in-repository repo-intel build infrastructure whilst updating configuration and documentation to describe repo-intel as a standalone tool. It then adds installation wiring in brew.sh to fetch repo-intel via Homebrew formula, with a non-fatal fallback to direct download if the formula is unavailable. ChangesMove repo-intel to standalone installation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@scripts/install/brew.sh`:
- Around line 59-60: Update the log message passed to print_step so its tap name
matches the actual brew install command; currently print_step shows
"tyom/homebrew-tap" while the install line uses "tyom/tap/repo-intel" (see
print_step invocation and the brew install command), so change the message to
reference "tyom/tap/repo-intel" (or vice-versa if you intended a different tap)
to keep them consistent.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: eb840cdf-a3da-4e4e-8d32-14439371efae
📒 Files selected for processing (12)
.gitignoreMakefileREADME.mdscripts/install/brew.shscripts/setup.shsrc/repo-intel/README.mdsrc/repo-intel/build.pysrc/repo-intel/gen_techdata.pysrc/repo-intel/repo-intel.pysrc/repo-intel/techdata.jsonsrc/repo-intel/template.htmlstow/bin/repo-intel
💤 Files with no reviewable changes (5)
- src/repo-intel/README.md
- src/repo-intel/techdata.json
- src/repo-intel/build.py
- src/repo-intel/gen_techdata.py
- src/repo-intel/repo-intel.py
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: test
🧰 Additional context used
🧠 Learnings (2)
📚 Learning: 2026-01-01T21:11:09.058Z
Learnt from: tyom
Repo: tyom/dotfiles PR: 6
File: scripts/setup.sh:33-41
Timestamp: 2026-01-01T21:11:09.058Z
Learning: In dotfiles repositories, when running installers during setup (e.g., Volta or similar tools) from shell scripts, pass the appropriate skip/setup flag (such as --skip-setup or an equivalent) to avoid modifying shell profiles. Shell configuration should be managed centrally in the version-controlled dotfiles, so external modifications should be prevented or minimized by default. Ensure setup.sh and related scripts do not alter shell init files outside the repo.
Applied to files:
scripts/install/brew.shscripts/setup.sh
📚 Learning: 2026-01-01T21:17:03.846Z
Learnt from: tyom
Repo: tyom/dotfiles PR: 6
File: scripts/setup.sh:64-66
Timestamp: 2026-01-01T21:17:03.846Z
Learning: In personal dotfiles repositories, setup scripts can rely on sequential execution guarantees and omit redundant availability checks, since failures are immediately visible during interactive setup and the context is personal, not production. Apply this guideline to setup.sh files or any similarly named scripts in personal repos, but do not generalize to non-interactive or production environments or scripts that may run in parallel. Ensure the script still handles essential errors gracefully and remains readable.
Applied to files:
scripts/setup.sh
🪛 Shellcheck (0.11.0)
scripts/install/brew.sh
[info] 59-59: Note that A && B || C is not if-then-else. C may run when A is true.
(SC2015)
🔇 Additional comments (1)
scripts/setup.sh (1)
70-72: ⚡ Quick winProvide the original review comment (including any diff/context) so I can rewrite it in the required format.
repo-intel is now published (tap tyom/tap/repo-intel + a standalone installer), and brew.sh runs on both macOS and Linux (Linuxbrew), so the Homebrew install covers every platform. Remove the now-redundant stow-tree curl fallback: - setup.sh: drop the curl-into-stow/bin fetch block - brew.sh: this is the sole install path; on failure hint at the manual installer instead of the removed fallback - .gitignore: drop stow/bin/repo-intel; ignore __pycache__/ and *.pyc - README: installed via Homebrew or the standalone repo's installer
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
repo-intelnow lives in its own repository — tyom/repo-intel — with a CLI, a GitHub Action, and a Homebrew formula. This removes the in-tree source and build wiring from dotfiles.Changes
src/repo-intel/and therepo-intel-build/repo-intel-techdata/repo-intel-devMakefile targets (nowmake build/make techdata/make devin the standalone repo).stow/bin/repo-intel. The local file is kept, so an existing~/bin/repo-intelsymlink keeps working until the next setup run.setup.sh: before stowing, fetch the single-file artifact intostow/bin/repo-intelvia curl whenrepo-intelisn't already installed (covers Linux and macOS-without-brew). Non-fatal.brew.sh: installtyom/tap/repo-intelin a separate, non-fatal step so a missing/unpublished tap can't abort the core package install.Notes
tyom/homebrew-tap) and a tagged release are not published yet, so thebrew installstep is intentionally non-fatal; the curl fallback installsrepo-intelregardless.Summary by CodeRabbit
New Features
repo-inteltool is now available as a standalone Homebrew package for independent installation.Documentation
repo-intelas a standalone tool.VARIANT=minimaltesting option.Chores