fix(ci): remove accidental .claude/skills submodule gitlink - #138
Merged
Conversation
Commit 015fd5e ("fix: add charts CSS to base layout") inadvertently committed .claude/skills as a submodule (mode 160000) without adding a .gitmodules entry. Every subsequent actions/checkout invocation on main has failed its submodule cleanup with: fatal: No url found for submodule path '.claude/skills' in .gitmodules This caused OpenSSF Scorecard runs to fail outright and Gitleaks / Trivy to emit checkout warnings on every push and on the weekly scheduled run. Drop the gitlink from the index and ignore .claude/skills/ and .claude/worktrees/ so future agent state doesn't slip back into the tree. .claude/commands/ and .claude/settings.local.json.example remain tracked as before. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Removes an accidental .claude/skills submodule gitlink that was added without a .gitmodules entry, which had been breaking actions/checkout (and thus OpenSSF Scorecard, Gitleaks, and Trivy) on every workflow run on main. Also extends .gitignore to prevent per-developer Claude Code state from being re-committed.
Changes:
- Drops the stray
160000gitlink at.claude/skillsfrom the index. - Adds
.claude/skills/and.claude/worktrees/to.gitignorewith a comment clarifying that.claude/commands/and.claude/settings.local.json.exampleremain intentionally tracked.
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
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.
Summary
015fd5e(CSS layout fix) accidentally added.claude/skillsto the tree as a submodule gitlink (mode160000) without a corresponding.gitmodulesentry.main. The submodule-cleanup step in checkout v4/v6 hard-fails withfatal: No url found for submodule path '.claude/skills' in .gitmodules..claude/skills/+.claude/worktrees/to.gitignoreso per-developer agent state can't slip back into the tree. The intentionally-tracked.claude/commands/slash commands and.claude/settings.local.json.exampleare unaffected.Example failing run: https://github.com/sarg3nt/gearbox/actions/runs/25999377392
Test plan
main(after merge) succeeds end-to-end.git ls-tree HEAD .claude/no longer shows a160000 commitentry.🤖 Generated with Claude Code