chore: stop tracking local editor configuration

[gamaware]

Stops tracking local editor configuration files and removes references to them from required-file checks and documentation. The files remain available locally but are no longer part of the repository.

Summary by CodeRabbit

• Documentation

  • Removed the project’s Claude-specific instructions and guidance for repository overrides, exclusions, and settings audits. The README no longer lists those files or tools in the repository structure.

• Chores

  • Removed automatic post-edit formatting and executable-permission handling for shell scripts.
  • Repository checks no longer require CLAUDE.md; other required-file checks remain unchanged.
  • Updated the .gitignore section label. Ignore rules are unchanged.

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

🧰 Additional context used

📚 Code guidelines (1)

.github/copilot-instructions.md — auto-discovered

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: gamaware/github-org-settings/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 978f6984-8d7a-478e-87db-dee3689026f2

📥 Commits

Reviewing files that changed from the base of the PR and between 1d122b9 and fe6ac61.

📒 Files selected for processing (10)

• .claude/hooks/post-edit.sh
• .claude/settings.json
• .claude/skills/add-repo-override/SKILL.md
• .claude/skills/audit/SKILL.md
• .claude/skills/exclude-repo/SKILL.md
• .github/workflows/quality-checks.yml
• .gitignore
• CLAUDE.md
• README.md
• config/baseline.json

💤 Files with no reviewable changes (8)

• .claude/skills/audit/SKILL.md
• .claude/settings.json
• .claude/skills/exclude-repo/SKILL.md
• CLAUDE.md
• .claude/hooks/post-edit.sh
• .claude/skills/add-repo-override/SKILL.md
• README.md
• config/baseline.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The repository removes Claude Code hooks, skills, and project instructions. It also removes CLAUDE.md from the required-file lists and repository structure documentation. The gitignore comment changes, but its ignore patterns remain unchanged.

Changes

Claude Code tooling removal

Layer / File(s)	Summary
Remove hooks and skills .claude/hooks/post-edit.sh, .claude/settings.json, .claude/skills/*/SKILL.md	The post-edit hook, its settings entry, and the add-repo-override, audit, and exclude-repo skills are deleted.
Remove project instructions CLAUDE.md, README.md	CLAUDE.md is deleted. The README repository-structure diagram no longer lists the .claude/ directory or CLAUDE.md.
Update repository requirements .github/workflows/quality-checks.yml, config/baseline.json, README.md, .gitignore	The required-file lists no longer include CLAUDE.md. The gitignore heading changes to Editor session files; ignore patterns are unchanged.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to fe6ac

This PR removes repository Claude Code tooling as intended, while keeping the required-file checks and documentation aligned. No material merge risk remains.

Security Architecture Review

Security architecture risk: ⚪ Minimal · up to fe6ac

The removed automation provides editing assistance rather than access-control enforcement. The policy change stops requiring the project instructions file while preserving the remaining repository checks. No material security risk was identified in this change.

Retained concerns
No architecture-level concerns identified.

Security review details

Security Blast Radius

• inferred — The policy effect extends to governed repositories whose effective required_files list inherits the updated baseline: absence of CLAUDE.md will no longer contribute to reported drift. Per-repository overrides can change that effective list. This is broader than the repository-local CI requirement but does not itself disable repository security settings.

Trust Boundaries and Controls

• inferred — The changed required-file value flows into a repository-content read and report, not an authority-changing API operation. Removing the agent hook eliminates its local command invocation rather than adding a privileged sink. The inspected change therefore supplies no new path to broader repository-management or agent permissions; external host permissions remain outside the inspected scope.

🚥 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 the main change: stop tracking local editor configuration and remove its repository references.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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 docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[Copilot]

Copilot review overview

🟡 Changes recommended

Missing ignore rules allow retained or recreated local configuration to be accidentally committed again.

Review effort: Balanced
Findings: None

What changed in this PR

Removes local Claude Code configuration from version control and updates the repository’s governance checks and documentation.

Changes:

• Deletes tracked instructions, settings, hooks, and skills.
• Removes CLAUDE.md from required-file checks and README references.
• Renames a .gitignore comment without changing ignore rules.

File	Description
README.md	Removes configuration references.
config/​baseline.json	Stops requiring CLAUDE.md.
CLAUDE.md	Deletes tracked project instructions.
.gitignore	Renames the session-files comment.
.github/​workflows/​quality-checks.yml	Stops checking for CLAUDE.md.
.claude/​skills/​exclude-repo/​SKILL.md	Deletes the exclusion skill.
.claude/​skills/​audit/​SKILL.md	Deletes the audit skill.
.claude/​skills/​add-repo-override/​SKILL.md	Deletes the override skill.
.claude/​settings.json	Deletes editor hook settings.
.claude/​hooks/​post-edit.sh	Deletes the formatting hook.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.