Skip to content

fix(dotagents): pin merged skills lock commit - #2221

Merged
shunkakinoki merged 1 commit into
mainfrom
codex/fix-dotagents-merged-gitlink
Aug 3, 2026
Merged

fix(dotagents): pin merged skills lock commit#2221
shunkakinoki merged 1 commit into
mainfrom
codex/fix-dotagents-merged-gitlink

Conversation

@shunkakinoki

@shunkakinoki shunkakinoki commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Summary

Replace the orphaned dotagents PR-head gitlink with the identical commit merged to dotagents main.

Validation

  • old and new dotagents commits have identical trees
  • Kyber base commit 0ef0e70 is an ancestor of bfc6007
  • git diff --check

Refs shunkakinokisoftware-hnnt


Summary by cubic

Pinned the dotagents submodule to the commit merged on its main branch, replacing the orphaned PR-head gitlink. This removes the orphan ref and makes builds deterministic.

Written for commit 9777128. Summary will update on new commits.

Review in cubic

Replace the orphaned pull-request head with the identical commit merged to dotagents main.

Refs shunkakinokisoftware-hnnt
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@indent-zero

indent-zero Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor
PR Summary

Re-pins the dotagents submodule from an orphaned PR-branch commit to the equivalent squash-merged commit on dotagents main (PR #178, "fix(skills): remove stale upstream lock entries"). This eliminates the risk of a broken git submodule update if the orphan commit is ever GC'd/force-pushed, without changing any submodule contents.

  • Updates dotagents gitlink 8a6c236bfc6007 (verified: bfc6007 is ancestor of shunkakinoki/dotagents@main; 8a6c236 is not reachable from any remote branch).
  • git diff 8a6c236 bfc6007 is empty — trees are byte-identical, so there is no behavioral change in the submodule.

Issues

No issues found.

CI Checks

Waiting for CI checks...

@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 02d5e40a-a461-4e9a-9813-9af981eb135d

📥 Commits

Reviewing files that changed from the base of the PR and between 2619e93 and 9777128.

📒 Files selected for processing (1)
  • dotagents

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated the bundled agent configuration to the latest revision.

Walkthrough

The dotagents submodule reference changes from commit 8a6c23679ea82430a85d463728032f185654501b to commit bfc6007ffc35f70cfd793deb05d90871c0d8f925.

Changes

dotagents submodule update

Layer / File(s) Summary
Update pinned submodule commit
dotagents
The submodule pointer updates to commit bfc6007ffc35f70cfd793deb05d90871c0d8f925.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Poem

A rabbit hops past the agents’ gate,
One commit shifts the pointer’s state.
From old hash burrow to new hash bright,
The submodule now points right.
“Hop, hop!” says Bun, “the change is light!”

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/fix-dotagents-merged-gitlink

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.

@shunkakinoki
shunkakinoki merged commit d18906e into main Aug 3, 2026
32 of 35 checks passed
@shunkakinoki
shunkakinoki deleted the codex/fix-dotagents-merged-gitlink branch August 3, 2026 09:50
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.

1 participant