Skip to content

chore: remove serena leftovers (hooks, permissions, docs, templates)#8518

Merged
MarkusNeusinger merged 2 commits into
mainfrom
chore/remove-serena-leftovers
Jun 9, 2026
Merged

chore: remove serena leftovers (hooks, permissions, docs, templates)#8518
MarkusNeusinger merged 2 commits into
mainfrom
chore/remove-serena-leftovers

Conversation

@MarkusNeusinger

Copy link
Copy Markdown
Owner

Summary

Serena MCP was removed from the project, but config and docs still referenced it. The most visible symptom: every Claude Code session failed on startup with

SessionStart:startup hook error
Failed with non-blocking status code: /bin/sh: 1: serena-hooks: not found

This completes the removal:

  • .claude/settings.json: drop all serena-hooks hooks (PreToolUse remind/auto-approve, SessionStart activate, Stop cleanup) and the mcp__serena__* permission entry
  • .serena/: delete the tracked project files and memories (were already deleted locally, now committed)
  • CLAUDE.md: remove the Serena tool section; Context7 guidance kept
  • agentic/commands/: replace Serena tool references (mcp__serena__*, list_dir, find_file, search_for_pattern, think_about_collected_information) with built-in Glob/Grep/Read/Edit equivalents across the audit auditors, audit.md, agentic.md, prime.md, prime-deep.md, and audit.workflow.js

Historical artifacts (agentic/runs/, agentic/audits/, agentic/specs/) intentionally untouched.

Verification

  • grep -rni serena over the repo (excluding historical artifacts) returns zero hits
  • Both settings files validate as JSON
  • The local untracked .claude/settings.local.json was cleaned separately (not part of this PR)

🤖 Generated with Claude Code

Serena MCP was removed from the project, but config and docs still
referenced it — the SessionStart hook failed every session with
"serena-hooks: not found".

- .claude/settings.json: drop serena-hooks hooks (PreToolUse,
  SessionStart, Stop) and the mcp__serena__* permission
- .serena/: delete tracked project files and memories
- CLAUDE.md: drop the Serena tool section, keep Context7
- agentic/commands/: replace serena tool references with built-in
  Glob/Grep/Read/Edit equivalents in the audit/prime/agentic templates

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 9, 2026 18:30
@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 finishes removing the previously-deleted Serena MCP integration by deleting the remaining Serena project artifacts and updating Claude/agentic command documentation to rely on built-in navigation/editing tools instead of mcp__serena__* and serena-hooks.

Changes:

  • Removed Serena hook/permission configuration from .claude/settings.json to stop Claude Code session startup failures.
  • Deleted tracked .serena/ project files and memories.
  • Updated CLAUDE + agentic audit/prime command docs to replace Serena tool references with Glob/Grep/Read/Edit guidance.

Reviewed changes

Copilot reviewed 25 out of 25 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.claude/settings.json Removes mcp__serena__* allowlist entry and all serena-hooks startup/tool hooks.
CLAUDE.md Drops Serena MCP tool documentation; keeps Context7 guidance only.
agentic/commands/prime.md Replaces Serena activation/tooling section with built-in tool guidance.
agentic/commands/prime-deep.md Replaces Serena tool catalog with built-in tool guidance.
agentic/commands/audit/security-auditor.md Rewrites Serena/list_dir references into Glob/Grep/Read equivalents.
agentic/commands/audit/quality-auditor.md Rewrites Serena/list_dir/search_for_pattern guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/observability-auditor.md Rewrites Serena/list_dir/find_symbol guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/llm-pipeline-auditor.md Rewrites Serena/list_dir/find_symbol guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/infra-auditor.md Rewrites list_dir/find_file/search_for_pattern guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/frontend-auditor.md Rewrites list_dir/get_symbols_overview/search_for_pattern guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/design-auditor.md Rewrites list_dir/get_symbols_overview guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/db-auditor.md Rewrites list_dir/get_symbols_overview/find_referencing_symbols guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/catalog-auditor.md Rewrites list_dir references into Glob guidance.
agentic/commands/audit/backend-auditor.md Rewrites Serena/list_dir/find_symbol guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/agentic-auditor.md Rewrites list_dir/get_symbols_overview guidance into Glob/Grep/Read equivalents.
agentic/commands/audit/audit.workflow.js Removes “Serena” mention from the cross-validation prompt instructions.
agentic/commands/audit.md Updates orchestrator guidance to remove Serena tool assumptions and references.
agentic/commands/agentic.md Updates audit-team guidance to remove Serena tool assumptions and references.
.serena/project.yml Deletes the Serena project configuration file.
.serena/memories/task_completion.md Deletes Serena memory file (task completion checklist).
.serena/memories/style_guide.md Deletes Serena memory file (design system notes).
.serena/memories/project_overview.md Deletes Serena memory file (repo overview).
.serena/memories/commands.md Deletes Serena memory file (commands reference).
.serena/memories/code_style.md Deletes Serena memory file (code style/conventions).
.serena/.gitignore Deletes Serena-local ignore file.

Comment thread .claude/settings.json
Comment on lines 10 to 12
"mcp__plugin_playwright_playwright__*",
"mcp__serena__*",
"Bash(git commit *)",
"Bash(git push *)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Implemented in 502401e by removing the redundant Bash(git commit *) and Bash(git push *) entries while keeping the existing "Bash" permission.

Comment on lines 20 to 23
@agentic/docs/project-guide.md
@agentic/commands/context.md
@docs/concepts/vision.md
@pyproject.toml
The file was deleted in #5648; the include has been a silent no-op
since. Flagged by Copilot review on #8518.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@MarkusNeusinger MarkusNeusinger merged commit 94c03c9 into main Jun 9, 2026
11 checks passed
@MarkusNeusinger MarkusNeusinger deleted the chore/remove-serena-leftovers branch June 9, 2026 19:49
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