Skip to content

Consolidate duplicate AI agent rules using symlinks#6

Open
Copilot wants to merge 4 commits into
feature/gemini-rulesfrom
copilot/sub-pr-5
Open

Consolidate duplicate AI agent rules using symlinks#6
Copilot wants to merge 4 commits into
feature/gemini-rulesfrom
copilot/sub-pr-5

Conversation

Copy link
Copy Markdown

Copilot AI commented Dec 23, 2025

  • Understand the problem and review existing files
  • Create a canonical source file .agent/rules/base.md
  • Replace .geminirules with a symlink to .agent/rules/base.md
  • Replace .cursorrules with a symlink to .agent/rules/base.md
  • Replace .agent/rules/gemini.md with a symlink to base.md
  • Add a README in .agent/rules/ explaining the structure
  • Test that the symlinks work correctly
  • Verify changes are correct
  • Address code review feedback
  • Resolve merge conflicts with feature/gemini-rules base branch

Summary

Successfully consolidated duplicate AI agent rules into a single source using symlinks. All three files (.geminirules, .cursorrules, .agent/rules/gemini.md) now point to .agent/rules/base.md.

Resolved merge conflict where the base branch had updated .geminirules title but this branch converted it to a symlink. The symlink solution was kept as it eliminates all duplication.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Dec 23, 2025
Co-authored-by: malandr <3076489+malandr@users.noreply.github.com>
Co-authored-by: malandr <3076489+malandr@users.noreply.github.com>
Copilot AI changed the title [WIP] Update cursor rules for agentic development based on feedback Consolidate duplicate AI agent rules using symlinks Dec 23, 2025
Copilot AI requested a review from malandr December 23, 2025 23:51
@malandr
Copy link
Copy Markdown
Collaborator

malandr commented Dec 23, 2025

@copilot resolve conflicts

@malandr malandr marked this pull request as ready for review December 23, 2025 23:54
Copilot AI review requested due to automatic review settings December 23, 2025 23:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR consolidates duplicate AI agent development rules from three separate files into a single source of truth using symbolic links, reducing maintenance burden and eliminating the need for manual synchronization across files.

Key changes:

  • Created .agent/rules/base.md as the canonical source for all AI agent rules
  • Converted .geminirules, .cursorrules, and .agent/rules/gemini.md to symlinks pointing to base.md
  • Added .agent/rules/README.md to document the symlink structure and maintenance process

Reviewed changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
.geminirules Converted from standalone file to symlink pointing to .agent/rules/base.md
.cursorrules Converted from standalone file to symlink pointing to .agent/rules/base.md
.agent/rules/gemini.md Converted from standalone file to symlink pointing to base.md (relative within directory)
.agent/rules/base.md New canonical source file containing AI development rules with normalized title
.agent/rules/README.md New documentation explaining symlink structure and maintenance workflow

Comment thread .agent/rules/base.md
@@ -0,0 +1,55 @@
# AI Development Rules for TeleCLI
Copy link

Copilot AI Dec 23, 2025

Choose a reason for hiding this comment

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

The title "AI Development Rules for TeleCLI" differs from the original titles in the files being replaced. The original .cursorrules had "Cursor Rules for TeleCLI" and the original .geminirules/gemini.md had "Antigravity/Gemini Rules for TeleCLI".

While the generic title makes sense for a shared base file, the PR description claims the files contained "identical content". This title normalization should be acknowledged in the PR description as an intentional improvement, not presented as consolidation of identical files.

Copilot uses AI. Check for mistakes.
Comment thread .agent/rules/README.md

## Git Note

Symlinks are tracked in Git, so they work seamlessly across different machines and in CI/CD environments.
Copy link

Copilot AI Dec 23, 2025

Choose a reason for hiding this comment

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

The documentation states "Symlinks are tracked in Git, so they work seamlessly across different machines and in CI/CD environments." While Git does track symlinks on Unix-like systems, this statement doesn't acknowledge potential issues on Windows, where symlink support depends on:

  • Git configuration (core.symlinks)
  • Windows version and permissions
  • Whether the repository was cloned with symlink support enabled

Consider adding a note about Windows compatibility or potential platform-specific considerations for developers working on different operating systems.

Suggested change
Symlinks are tracked in Git, so they work seamlessly across different machines and in CI/CD environments.
Symlinks are tracked in Git and generally work well across different machines and in CI/CD environments. On Windows, symlink behavior can depend on Git configuration (`core.symlinks`), Windows version and permissions, and whether the repository was cloned with symlink support enabled.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Author

Copilot AI commented Dec 24, 2025

@copilot resolve conflicts

I've successfully resolved the merge conflicts by rebasing this branch onto feature/gemini-rules (the base branch).

Conflict Details:
The base branch added commit 957d00b which updated .geminirules to change "Antigravity/Gemini Rules" to "Gemini AI Rules". This conflicted with my change that converted .geminirules to a symlink.

Resolution:
I kept the symlink approach as it supersedes the title change by eliminating all duplication. The symlink now points to base.md which has a generic title suitable for all tools.

Current State:

  • Local branch has been rebased and includes all changes from the base branch
  • All symlinks are working correctly
  • Conflicts are fully resolved

However, the automated push system requires a force-push since the history was rewritten during rebase. The branch is ready to merge once the force-push completes.

Rebased commits: 00d89b8, 23a4555, 9a35a83, a7150f9, 0ecd558

Resolved conflict where .geminirules was changed to update title in base branch
but was converted to symlink in this branch. Kept symlink as it provides the
superior solution by eliminating all duplication.
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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