Skip to content

Add symlink rules for .cursor/rules - #6529

Open
Sankalp-Mittal wants to merge 4 commits into
mainfrom
sankalp-mittal/symlink-lint-rules
Open

Add symlink rules for .cursor/rules#6529
Sankalp-Mittal wants to merge 4 commits into
mainfrom
sankalp-mittal/symlink-lint-rules

Conversation

@Sankalp-Mittal

@Sankalp-Mittal Sankalp-Mittal commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Changes

Add a linter that keeps every agent's config symlinks in sync with the canonical
AGENTS.md + .agents/ setup (Claude, Cursor, and GitHub Copilot). It creates
any missing symlinks and drops stale Cursor ones; wired into task checks, so
CI's post-checks git diff --exit-code fails if the tree wasn't already in sync.

Run it with:

./tools/validate_agents_setup.py

Why

Help keep the AI rules for different agents in sync

Tests

Not Required

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Waiting for approval

Based on git history, these people are best suited to review:

  • @pietern -- recent work in ./, tools/
  • @janniklasrose -- recent work in ./, tools/
  • @denik -- recent work in ./, tools/

Eligible reviewers: @andrewnester, @anton-107, @lennartkats-db, @rclarey, @renaudhartert-db, @rugpanov, @shreyas-goenka, @simonfaltum

Suggestions based on git history. See OWNERS for ownership rules.

@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 5f5c654

Run: 33887816163

Env 💚​RECOVERED ✅​pass 🙈​skip Time
💚​ aws linux 1 275 15 6:08
💚​ aws windows 1 277 13 4:08
💚​ azure linux 1 274 15 6:25
💚​ azure windows 1 276 13 4:13
💚​ gcp linux 1 275 15 6:43
💚​ gcp windows 1 277 13 4:27
Test Name aws linux aws windows azure linux azure windows gcp linux gcp windows
💚​ TestAccept 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
Top 6 slowest tests (at least 2 minutes):
duration env testname
4:11 azure windows TestAccept
4:06 aws windows TestAccept
4:00 gcp windows TestAccept
3:57 aws linux TestAccept
3:54 gcp linux TestAccept
3:44 azure linux TestAccept

Comment thread Taskfile.yml Outdated
Comment thread tools/validate_cursor_rules.py Outdated
Sankalp-Mittal and others added 2 commits September 4, 2026 15:00
Rename validate_cursor_rules.py to validate_agents_setup.py and extend it to
sync every agent's symlinks (Claude, Cursor, GitHub Copilot) with the canonical
AGENTS.md + .agents/ setup, not just Cursor's .mdc mirrors.

Drop the --fix flag (fixing is now the default) and instead exit non-zero when a
symlink was missing or wrong: a created symlink is untracked, so CI's
`git diff --exit-code` gate alone would let the common missing-link case pass.

Co-authored-by: Isaac <no-reply@databricks.com>
The Claude and GitHub Copilot symlinks are committed, static, and directory-level
(new rules appear automatically), so any drift is already caught by CI's
`git diff --exit-code`. Only Cursor's per-rule .mdc mirrors need active syncing,
since adding a rule creates a new untracked symlink that a plain git diff misses.

Co-authored-by: Isaac <no-reply@databricks.com>
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.

3 participants