Skip to content

Symlink agent files instead of telling bots to read other files#9750

Open
Jacobfaib wants to merge 1 commit into
NVIDIA:mainfrom
Jacobfaib:jacobf/2026-07-08/agent-files-cleanup
Open

Symlink agent files instead of telling bots to read other files#9750
Jacobfaib wants to merge 1 commit into
NVIDIA:mainfrom
Jacobfaib:jacobf/2026-07-08/agent-files-cleanup

Conversation

@Jacobfaib

Copy link
Copy Markdown
Contributor

Description

Symlink CLAUDE.md -> AGENTS.md
Symlink .claude/skills -> .agents/skills

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@Jacobfaib Jacobfaib self-assigned this Jul 8, 2026
@github-project-automation github-project-automation Bot moved this to Todo in CCCL Jul 8, 2026
@cccl-authenticator-app cccl-authenticator-app Bot moved this from Todo to In Review in CCCL Jul 8, 2026
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 41ba89e5-ed03-4663-847c-e902413ff772

📥 Commits

Reviewing files that changed from the base of the PR and between 3f87270 and e1ab2d3.

📒 Files selected for processing (5)
  • .claude/skills
  • .claude/skills/cccl-style/SKILL.md
  • .claude/skills/cccl-test/SKILL.md
  • CLAUDE.md
  • CLAUDE.md
💤 Files with no reviewable changes (2)
  • .claude/skills/cccl-test/SKILL.md
  • .claude/skills/cccl-style/SKILL.md

📝 Walkthrough

Summary by CodeRabbit

  • Chores

    • Updated the project’s assistant-related setup to use a shared linked location for skill resources.
    • Removed outdated assistant instruction content from the repository.
  • Documentation

    • Simplified assistant skill references so the current skill definitions are used consistently.

Walkthrough

The .claude/skills directory is replaced with a symlink pointing to ../.agent/skills/, consolidating duplicate skill definition files. The cccl-style and cccl-test SKILL.md files under .claude/skills are removed, and CLAUDE.md content is cleared.

Changes

Skills directory consolidation

Layer / File(s) Summary
Symlink replaces skills directory
.claude/skills, .claude/skills/cccl-style/SKILL.md, .claude/skills/cccl-test/SKILL.md, CLAUDE.md
.claude/skills becomes a symlink to ../.agent/skills/, removing the previously duplicated cccl-style and cccl-test SKILL.md content and clearing CLAUDE.md.

Comment @coderabbitai help to get the list of available commands.

@Jacobfaib Jacobfaib requested a review from NaderAlAwar July 8, 2026 14:05
@github-actions

This comment has been minimized.

@fbusato

fbusato commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

does it work in all OSes?

@Jacobfaib

Copy link
Copy Markdown
Contributor Author

doed it work in all OSes?

Yes. Any recent version of git (IIRC something like v1.5+ (released in 2007)) will determine symlink support on the current platform when downloading the repository.

If the platform doesn't support symlinks (which they all should, windows added full support in windows 10), then git will instead populate the file with the path of the link. So given foo.txt -> /bar/baz.txt, then git will generate foo.txt to contain:

/bar/baz.txt

But this is very very rare, I haven't come across a system other than ancient windows where symlinks haven't worked.

@Jacobfaib Jacobfaib enabled auto-merge (squash) July 8, 2026 17:11
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

😬 CI Workflow Results

🟥 Finished in 3h 26m: Pass: 99%/506 | Total: 16d 18h | Max: 2h 40m | Hits: 37%/1879101

See results here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants