Skip to content

chore: remove agent-dotdir mirrors (.cursor/.windsurf/.clinerules) from root#355

Open
JuliusBrussee wants to merge 1 commit into
mainfrom
worktree-agent-a23ab889148296b6b
Open

chore: remove agent-dotdir mirrors (.cursor/.windsurf/.clinerules) from root#355
JuliusBrussee wants to merge 1 commit into
mainfrom
worktree-agent-a23ab889148296b6b

Conversation

@JuliusBrussee
Copy link
Copy Markdown
Owner

Summary

  • Delete CI-generated agent-dotdir mirrors at repo root (.cursor/skills/, .cursor/rules/caveman.mdc, .windsurf/skills/, .windsurf/rules/caveman.md, .clinerules/caveman.md, .github/copilot-instructions.md, caveman/SKILL.md). These self-applied caveman to this repo when devs opened it in those agents β€” the installer never consumed them.
  • Trim .github/workflows/sync-skill.yml: drop the cp/mkdir/git-add lines for the removed mirrors, drop the entire "Sync auto-activation rules" step, drop rules/caveman-activate.md from paths: triggers (workflow no longer reads it). Keep plugins/caveman/ sync, cavecrew sync, compress sync, and caveman.skill ZIP step.
  • Update CONTRIBUTING.md auto-sync note to match the smaller set.

Devs who still want caveman self-applied to this repo in Cursor/Windsurf/Cline can opt in via npx caveman --only cursor (etc.).

Sources of truth (skills/, agents/, rules/caveman-activate.md, hooks/, plugins/caveman/) are untouched. CLAUDE.md is untouched (owned by another cleanup unit). tools/caveman-init.js is untouched β€” it still writes those dotdir paths into a user's repo, which is correct.

Test plan

  • find .cursor .windsurf .clinerules caveman returns nothing (all parent dirs gone)
  • python3 -c "import yaml; yaml.safe_load(open('.github/workflows/sync-skill.yml'))" β€” workflow YAML still parses
  • node tests/test_caveman_init.js β€” 8/8 pass (per-repo init writer untouched)
  • bash install.sh --list β€” 33 providers render correctly
  • grep confirms no remaining refs to deleted mirrors except in tools/caveman-init.js (writes to user repo, correct), tests/test_caveman_init.js (tests user-repo writer, correct), and CLAUDE.md / CLAUDE.original.md (owned by Unit 8)

πŸ€– Generated with Claude Code

Drop CI-generated mirrors that self-applied caveman to this repo when
opened in Cursor/Windsurf/Cline/Copilot. The installer never read them
(bin/install.js consumes only hooks/, rules/, tools/, agents/, skills/,
mcp-servers/, plugins/caveman/), so removing them just declutters the
root. Devs who want self-application can opt in via npx caveman --only
<agent>.

Removed:
- .cursor/skills/, .cursor/rules/caveman.mdc
- .windsurf/skills/, .windsurf/rules/caveman.md
- .clinerules/caveman.md
- .github/copilot-instructions.md
- caveman/SKILL.md (and empty caveman/ parent)

Updated .github/workflows/sync-skill.yml: drop the cp + mkdir lines and
git add paths for those mirrors, drop the entire "Sync auto-activation
rules" step, and drop rules/caveman-activate.md from the paths: trigger
since the workflow no longer consumes it. plugins/caveman/, cavecrew,
compress, and caveman.skill ZIP sync steps are preserved.

Updated CONTRIBUTING.md note to reflect the smaller auto-synced set.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

πŸ’‘ Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: be714a3db8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with πŸ‘.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- name: Sync SKILL.md copies
run: |
cp skills/caveman/SKILL.md caveman/SKILL.md
cp skills/caveman/SKILL.md plugins/caveman/skills/caveman/SKILL.md
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep repo verification aligned with removed mirror files

This change stops maintaining several mirror artifacts from skills/caveman/SKILL.md, but tests/verify_repo.py still requires those exact files in verify_synced_files; on this commit, running python3 tests/verify_repo.py immediately fails with FileNotFoundError for /workspace/caveman/caveman/SKILL.md. Please update the verification expectations (and any CI job using them) in the same change, or keep the mirrors synced, so the repo’s validation pipeline does not break.

Useful? React with πŸ‘Β / πŸ‘Ž.

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

Labels

None yet

1 participant