Skip to content

docs: remove outdated Claude Code hooks section from notifications - #2053

Merged
lawrencecchen merged 1 commit into
manaflow-ai:mainfrom
BillionClaw:fix/notifications-remove-claude-code-hooks
Mar 25, 2026
Merged

lawrencecchen merged 1 commit into
manaflow-ai:mainfrom
BillionClaw:fix/notifications-remove-claude-code-hooks

Conversation

@BillionClaw

@BillionClaw BillionClaw commented Mar 24, 2026 •

Copy link
Copy Markdown
Contributor

The Claude Code hooks section in the Notifications docs referenced pre-0.60 configuration using matchers (idle_prompt, permission_prompt) that are no longer valid in current Claude Code versions.

This removes the outdated section and replaces it with a link to the official Claude Code documentation.

Fixes #2009


Summary by cubic

Removed the outdated Claude Code hooks section from the notifications docs that referenced pre-0.60 matchers (idle_prompt, permission_prompt). Replaced it with a link to the official Claude Code docs to avoid invalid configurations.

Written for commit 26ebda5. Summary will update on new commits.

Summary by CodeRabbit

  • Documentation
    • Updated notifications documentation. Removed specific hook configuration examples, detailed setup instructions, and implementation details previously included. The documentation now directs users to the official Anthropic documentation for comprehensive hook configuration guidance.

The Claude Code hooks section referenced pre-0.60 configuration using
matchers (idle_prompt, permission_prompt) that are no longer valid in
current Claude Code versions. Replace with a link to the official
Claude Code documentation.

Fixes manaflow-ai#2009
@vercel

vercel Bot commented Mar 24, 2026

Copy link
Copy Markdown

@BillionClaw is attempting to deploy a commit to the Manaflow Team on Vercel.

A member of the Team first needs to authorize it.

@cubic-dev-ai cubic-dev-ai 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.

No issues found across 1 file

@greptile-apps

greptile-apps Bot commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes a stale Claude Code hooks configuration example from docs/notifications.md that used pre-0.60 matcher syntax (idle_prompt, permission_prompt) no longer valid in current Claude Code releases, replacing it with a reference to the official documentation. The change is purely cosmetic/docs and carries no code risk.

  • Removed 27-line JSON snippet with deprecated Notification hook matchers
  • Replaced with a single sentence and link to the Claude Code documentation root
  • All other integration sections (OpenAI Codex, OpenCode Plugin) are untouched
  • Minor opportunity: the replacement link targets the Claude Code root page rather than the dedicated hooks sub-page, which would give users a more direct path

Confidence Score: 5/5

  • Safe to merge — documentation-only cleanup with no code changes.
  • Single-file docs change that removes actively misleading content. The only non-blocking suggestion is to use a more targeted URL for the hooks sub-page rather than the root Claude Code docs landing page.
  • No files require special attention.

Important Files Changed

Filename Overview
docs/notifications.md Removed outdated pre-0.60 Claude Code hook config block (using deprecated idle_prompt/permission_prompt matchers) and replaced it with a link to official documentation. The surrounding content is untouched and remains correct.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Claude Code fires a hook event] --> B{Before PR: inline config}
    A --> C{After PR: official docs}
    B --> D["~/.claude/settings.json\nmatchers: idle_prompt / permission_prompt\n⚠️ Deprecated pre-0.60 syntax"]
    C --> E["docs.anthropic.com/en/docs/claude-code\n✅ Always up-to-date"]
    E --> F[User follows current hook setup]
    D -.->|breaks on Claude Code ≥ 0.60| G[Hook silently ignored]
Loading

Reviews (1): Last reviewed commit: "docs: remove outdated Claude Code hooks ..." | Re-trigger Greptile

Comment thread docs/notifications.md
```
### Claude Code

See the [Claude Code documentation](https://docs.anthropic.com/en/docs/claude-code) for hook configuration.

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.

P2 Link points to root docs, not hooks page

The URL https://docs.anthropic.com/en/docs/claude-code lands on the top-level Claude Code documentation page. Users looking specifically for hook configuration will need to navigate through it manually. Linking directly to the hooks sub-page (e.g. https://docs.anthropic.com/en/docs/claude-code/hooks) would give them a much shorter path to the relevant content, and keeps this section as useful as the other integration examples in the file.

Suggested change
See the [Claude Code documentation](https://docs.anthropic.com/en/docs/claude-code) for hook configuration.
See the [Claude Code hooks documentation](https://docs.anthropic.com/en/docs/claude-code/hooks) for hook configuration.

@coderabbitai

coderabbitai Bot commented Mar 24, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f9db5548-e0b6-43e8-952c-3d87588381ba

📥 Commits

Reviewing files that changed from the base of the PR and between 441118b and 26ebda5.

📒 Files selected for processing (1)
  • docs/notifications.md

📝 Walkthrough

Walkthrough

Removed outdated "Claude Code hooks" configuration instructions from the Notifications documentation that became irrelevant after v0.60.0, replacing them with a reference to the official Anthropic documentation for hook configuration.

Changes

Cohort / File(s) Summary
Documentation Cleanup
docs/notifications.md
Removed pre-0.60.0 Claude Code hooks section with macOS-specific osascript implementations (idle_prompt, permission_prompt matchers); replaced with generic "Claude Code" heading and link to official Anthropic documentation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 Old hooks fade to grey and dust,
Outdated docs we had to bust,
Now clean and fresh, the path is clear—
Point to Anthropic's wisdom here! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description addresses the main change but is incomplete against the template; it lacks Testing and Demo Video sections, and the Checklist is not filled out. Fill in the Testing section documenting how the change was verified, and complete the pre-submission Checklist items.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: removing outdated Claude Code hooks section from notifications documentation.
Linked Issues check ✅ Passed The PR removes the outdated pre-0.60 Claude Code hooks section and replaces it with a link to official documentation, directly addressing issue #2009's request.
Out of Scope Changes check ✅ Passed All changes are documentation-only and directly focused on removing outdated hooks information per issue #2009; no extraneous modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

@lawrencecchen
lawrencecchen merged commit 6d9c937 into manaflow-ai:main Mar 25, 2026
4 of 5 checks passed
@lawrencecchen

Copy link
Copy Markdown
Contributor

Thank you for the contribution!

bn-l pushed a commit to bn-l/cmux that referenced this pull request Apr 3, 2026
…anaflow-ai#2053)

The Claude Code hooks section referenced pre-0.60 configuration using
matchers (idle_prompt, permission_prompt) that are no longer valid in
current Claude Code versions. Replace with a link to the official
Claude Code documentation.

Fixes manaflow-ai#2009

Co-authored-by: BillionClaw <267901332+BillionClaw@users.noreply.github.com>
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.

docs (Notifications): Clean up pre-0.60 "Claude Code hooks" section in "Notifications" docs

2 participants