FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Split plugin dev guide from root CLAUDE.md into module-level files by wu-sheng · Pull Request #790 · apache/skywalking-java · GitHub

/ skywalking-java Public

Split plugin dev guide from root CLAUDE.md into module-level files - #790

Merged
wu-sheng merged 1 commit into
mainfrom
split-claude-md-plugin-dev
Feb 12, 2026
ConversationCommits (1)ChecksFiles changed
Merged

Split plugin dev guide from root CLAUDE.md into module-level files#790
wu-sheng merged 1 commit into
mainfrom
split-claude-md-plugin-dev

Conversation

wu-sheng commented Feb 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Move plugin development content (V2/V1 APIs, class matching, tracing concepts, meter APIs, plugin test framework, etc.) from root CLAUDE.md into dedicated module-level files:
    • apm-sniffer/apm-sdk-plugin/CLAUDE.md — full SDK plugin development guide (~280 lines)
    • apm-sniffer/bootstrap-plugins/CLAUDE.md — bootstrap plugin specifics (~50 lines)
  • Root CLAUDE.md reduced from ~700 lines to ~300 lines, keeping only project-level info (build, architecture overview, code style, PR guidelines) with cross-references to the plugin guides

Test plan

  • Verify root CLAUDE.md still contains project overview, build, code style, testing, PR guidelines
  • Verify apm-sniffer/apm-sdk-plugin/CLAUDE.md contains full plugin dev guide (APIs, rules, tracing, meters, test framework)
  • Verify apm-sniffer/bootstrap-plugins/CLAUDE.md contains bootstrap-specific guidance with cross-references
  • Verify cross-references between files are correct

Split plugin development guide from root CLAUDE.md into module-level …
fa5805c 
…files

Move plugin development content (V2/V1 APIs, class matching, tracing concepts,
meter APIs, plugin test framework, etc.) into dedicated CLAUDE.md files under
apm-sniffer/apm-sdk-plugin/ and apm-sniffer/bootstrap-plugins/, reducing the
root CLAUDE.md from ~700 lines to ~300 lines for better maintainability.
wu-sheng requested a review from wankai123 February 12, 2026 12:14
wu-sheng added this to the 9.6.0 milestone Feb 12, 2026
wu-sheng merged commit ea25bfe into main Feb 12, 2026
9 of 10 checks passed
wu-sheng deleted the split-claude-md-plugin-dev branch February 12, 2026 15:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

wankai123 wankai123 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

9.6.0

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL