feat: add before-and-after - #2324
Conversation
|
Warning Indent Zero is shutting down on August 7th. Please migrate over to Indent 2.0 to continue getting PR reviews.
|
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe PR advances the ChangesMaintenance updates
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
Suggested labels: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
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 `@dotagents`:
- Line 1: Revert the dotagents submodule gitlink to its previous reachable
commit, removing the pointer to 9589ed39a7f0dd49c4fd396bbe559a7d4f7bc9b1 until
that commit is available from the upstream submodule remote.
🪄 Autofix
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 Plus
Run ID: f1a6e75a-fccd-412d-9c7b-7a4a3088c27d
⛔ Files ignored due to path filters (1)
bun.lockis excluded by!**/*.lock
📒 Files selected for processing (2)
dotagentspackage.json
| @@ -1 +1 @@ | |||
| Subproject commit fa405a0d3240cdc5b6df38b603d8bab6b1c3d583 | |||
| Subproject commit 9589ed39a7f0dd49c4fd396bbe559a7d4f7bc9b1 | |||
There was a problem hiding this comment.
🔒 Security & Privacy | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
expected="9589ed39a7f0dd49c4fd396bbe559a7d4f7bc9b1"
actual="$(git ls-tree HEAD -- dotagents | awk '$4 == "dotagents" { print $3 }')"
test "$actual" = "$expected"
remote="$(git config -f .gitmodules --get submodule.dotagents.url)"
tmp="$(mktemp -d)"
trap 'rm -rf "$tmp"' EXIT
git clone --filter=blob:none --no-checkout "$remote" "$tmp/dotagents"
git -C "$tmp/dotagents" cat-file -e "$expected^{commit}"
old="$(git rev-parse HEAD^:dotagents 2>/dev/null || true)"
if [ -n "$old" ]; then
git -C "$tmp/dotagents" diff --name-status "$old" "$expected"
fiRepository: shunkakinoki/dotfiles
Length of output: 466
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
printf 'HEAD dotagents gitlink:\n'
git ls-tree HEAD -- dotagents || true
printf '\nPrevious HEAD dotagents gitlink:\n'
git rev-parse --verify HEAD^:dotagents || true
printf '\nHEAD@{1} dotagents gitlink:\n'
git rev-parse --verify HEAD@{1}:dotagents || true
printf '\n.gitmodules dotagents entry:\n'
git config -f .gitmodules --get-regexp 'submodule\.dotagents\.' || true
printf '\nLocal refs/object around dotagents:\n'
git for-each-ref --format='%(refname) %(objectname) %(objecttype)' | rg 'dotagents|dotfiles|main|master' || true
printf '\nLocal object existence check:\n'
expected="9589ed39a7f0dd49c4fd396bbe559a7d4f7bc9b1"
git cat-file -e "$expected" && git cat-file -t "$expected" || trueRepository: shunkakinoki/dotfiles
Length of output: 901
Reject this submodule pointer until the target commit is reachable.
The configured dotagents submodule points to 9589ed39a7f0dd49c4fd396bbe559a7d4f7bc9b1, but that object is not available from the submodule remote. Revert the gitlink change until the intended commit is public in the upstream repository.
🤖 Prompt for 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.
In `@dotagents` at line 1, Revert the dotagents submodule gitlink to its previous
reachable commit, removing the pointer to
9589ed39a7f0dd49c4fd396bbe559a7d4f7bc9b1 until that commit is available from the
upstream submodule remote.
1 participant
Adds the @vercel/before-and-after CLI, trusts its package scripts, and updates the dotagents submodule to the matching locked skill.
Validated with a frozen Bun install and before-and-after --help.
Summary by cubic
Add the
@vercel/before-and-afterCLI (v0.0.4) and trust its package scripts so it can run in our tooling. Update thedotagentssubmodule to the matching locked skill and refreshbun.lock.Written for commit b802705. Summary will update on new commits.