Skip to content

chore: remove deprecated LSP hook and cleanup plugin config#71

Merged
amondnet merged 1 commit into
mainfrom
chore/remove-deprecated-lsp-hook
Jan 8, 2026
Merged

chore: remove deprecated LSP hook and cleanup plugin config#71
amondnet merged 1 commit into
mainfrom
chore/remove-deprecated-lsp-hook

Conversation

@amondnet

@amondnet amondnet commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove deprecated LSP hook and clean up plugin configuration as Claude Code now has native LSP support.

Changes

  • hooks.json:
    • Remove deprecated LSP hook (Claude Code has native LSP diagnostics)
    • Remove SessionStart hook for ast-grep-context
    • Add error tolerance to format hook (|| true)
  • plugin.json: Version bump to 0.1.15
  • settings.json: Remove unused plugins (hookify, github, context7)

Rationale

Claude Code now provides native LSP diagnostics through the IDE integration, making the custom LSP hook redundant. The format hook now has error tolerance to prevent failures from blocking the workflow.

Test Plan

  • Verify formatting hook still works with || true fallback
  • Confirm LSP diagnostics work through Claude Code native support
  • Test plugin loads correctly with updated configuration

- Remove deprecated LSP hook (Claude Code has native LSP support)
- Remove SessionStart hook for ast-grep-context
- Add error tolerance to format hook (|| true)
- Bump version to 0.1.15
- Clean up unused plugins from settings (hookify, github, context7)
@amondnet amondnet self-assigned this Jan 8, 2026
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying code-please with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3b35111
Status:🚫  Build failed.

View logs

@codecov

codecov Bot commented Jan 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.71%. Comparing base (e5488e1) to head (3b35111).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
- Coverage   39.35%   38.71%   -0.65%     
==========================================
  Files          59       59              
  Lines        6489     6515      +26     
==========================================
- Hits         2554     2522      -32     
- Misses       3935     3993      +58     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@amondnet
amondnet merged commit e28a2ea into main Jan 8, 2026
8 of 10 checks passed
@amondnet
amondnet deleted the chore/remove-deprecated-lsp-hook branch January 8, 2026 09:29
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