Skip to content

security: remove .claude/ from repo, add to .gitignore#5

Merged
LEEI1337 merged 1 commit into
mainfrom
security/remove-claude
May 26, 2026
Merged

security: remove .claude/ from repo, add to .gitignore#5
LEEI1337 merged 1 commit into
mainfrom
security/remove-claude

Conversation

@LEEI1337

Copy link
Copy Markdown
Member

Summary

  • Entfernt .claude/skill-registry.json aus dem öffentlichen Repository
  • .gitignore erweitert um .claude/ Pattern

Hintergrund

.claude/ ist als lokale Claude-Code-Konfiguration gemeint und gehört nicht in ein öffentliches Repository.

Reversibilität

Soft-Remove: aktueller HEAD nach Merge sauber, Git-Historie behält die Datei weiterhin sichtbar.

Test plan

  • git ls-files .claude/ → leer auf branch
  • .gitignore enthält .claude/-Pattern
  • Diff: 1 Deletion + 1 .gitignore-Add (= 2 files changed, 2 insertions(+), 224 deletions(-))

🤖 Generated with Claude Code

.claude/ enthielt skill-registry.json als lokale Claude-Code-Konfiguration.
Gehoert nicht in ein oeffentliches Repository.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

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

Copy link
Copy Markdown

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: d7139def2c

ℹ️ 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".

Comment thread .gitignore
self-improving/corrections.md
self-improving/memory.md
# Local Claude Code configuration (interne Doktrin/Workflows)
.claude/

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve registry consistency checks in CI

Ignoring the entire .claude/ directory here guarantees .claude/skill-registry.json is absent on fresh checkouts, and validate_registry_consistency() then exits early with only a warning (scripts/validate.py:276-283) instead of running the mismatch checks (scripts/validate.py:299-316). Because CI still runs python scripts/validate.py (.github/workflows/ci.yml:112), this change silently disables the registry drift detection the validator was meant to enforce, so missing/extra skill entries stop being caught.

Useful? React with 👍 / 👎.

@LEEI1337 LEEI1337 merged commit 8b71976 into main May 26, 2026
7 checks passed
@LEEI1337 LEEI1337 deleted the security/remove-claude branch May 26, 2026 00:23
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.

1 participant