Skip to content

fix: stop tracking personal .cursor/skills symlinks - #21

Merged
nnkogift merged 1 commit into
mainfrom
fix/cursor-skills-broken-symlinks
Aug 3, 2026
Merged

nnkogift merged 1 commit into
mainfrom
fix/cursor-skills-broken-symlinks

Conversation

@nnkogift

@nnkogift nnkogift commented Aug 3, 2026 •

Copy link
Copy Markdown
Owner

Summary

.cursor/skills/coding-standards and .cursor/skills/dev-tooling were committed as symlinks pointing outside the repo (../../../../personal/personal-skills/...) — a machine-local personal skills directory that only resolves on the original committer's machine. Every other entry under .cursor/skills/ is a real, portable directory with its own SKILL.md.

This surfaced as a CI failure in the fallow-badge job's "Publish badge" step (see #20): actions-gh-pages copies the whole working directory to publish the badge, and fails with ENOENT trying to stat the dangling symlink target on a fresh CI checkout.

Fix

  • git rm --cached the two symlinks — they stay on disk locally (your own Cursor setup is unaffected), they're just no longer version-controlled.
  • Added both paths to .gitignore so they don't get accidentally re-added.

Test plan

  • Confirmed the symlinks still exist and resolve correctly on disk locally
  • Confirmed git status/git ls-files no longer track them
  • Fresh clone no longer contains a dangling symlink for actions-gh-pages to trip over

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Removed obsolete local development skill links.
    • Added ignore rules to prevent machine-specific development files from being tracked.

.cursor/skills/coding-standards and .cursor/skills/dev-tooling were
committed as symlinks pointing outside the repo to a machine-local
personal skills directory (../../../../personal/personal-skills/...).
Every other entry under .cursor/skills is a real, portable directory —
these two only ever resolved on the original committer's machine.

In CI, actions-gh-pages's publish step (fallow-badge job) copies the
whole working directory and fails with ENOENT trying to stat the
dangling symlink target. Untracking them (and ignoring the path so they
don't get re-added) fixes that without touching anyone's local Cursor
setup — the symlinks stay on disk, just no longer version-controlled.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 3, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: bcccb7a7-26a0-47ac-b0ac-b6a5462a25e8

📥 Commits

Reviewing files that changed from the base of the PR and between 1a4ca7a and 12de2bd.

📒 Files selected for processing (3)
  • .cursor/skills/coding-standards
  • .cursor/skills/dev-tooling
  • .gitignore
💤 Files with no reviewable changes (2)
  • .cursor/skills/dev-tooling
  • .cursor/skills/coding-standards

📝 Walkthrough

Walkthrough

The PR removes two local Cursor skill reference files and adds .gitignore rules for their machine-local paths.

Changes

Cursor skill cleanup

Layer / File(s) Summary
Ignore machine-local skill links
.gitignore, .cursor/skills/coding-standards, .cursor/skills/dev-tooling
The two local Cursor skill reference files were removed. .gitignore now excludes their machine-local paths.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes removing personal .cursor/skills symlinks from version control.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/cursor-skills-broken-symlinks

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.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Cursor Security Agent completed successfully with no findings requiring human review; Cursor Bugbot was not present on this PR, so that signal was skipped. No reviewers assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

@nnkogift
nnkogift merged commit 51c03c7 into main Aug 3, 2026
14 checks passed
@nnkogift
nnkogift deleted the fix/cursor-skills-broken-symlinks branch August 3, 2026 18:40
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