Skip to content

Add diffsitter skill documentation#1277

Open
qdrddr wants to merge 1 commit intoafnanenayet:mainfrom
qdrddr:main
Open

Add diffsitter skill documentation#1277
qdrddr wants to merge 1 commit intoafnanenayet:mainfrom
qdrddr:main

Conversation

@qdrddr
Copy link
Copy Markdown

@qdrddr qdrddr commented Mar 9, 2026

Add documentation for the diffsitter skill, including its purpose, supported languages, examples, CLI options, and practical workflows for Coding Agents such as Claude Code, Codex, Gemini etc.

Add documentation for the diffsitter skill, including its purpose, supported languages, examples, CLI options, and practical workflows.
@qdrddr qdrddr requested a review from afnanenayet as a code owner March 9, 2026 19:12
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.50%. Comparing base (3386289) to head (3f9ffaa).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1277   +/-   ##
=======================================
  Coverage   61.50%   61.50%           
=======================================
  Files          15       15           
  Lines        1356     1356           
=======================================
  Hits          834      834           
  Misses        522      522           

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

Copy link
Copy Markdown
Owner

@afnanenayet afnanenayet left a comment

Choose a reason for hiding this comment

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

Is it typical to have these things hardcoded? Was wondering if it would be better to tell Claude to pull the supported languages by running the subcommand to list supported languages.

Also this can be built with support for dynamically loading tree sitter grammars so you're not actually restricted to the grammars I built with, so would mention that as well.

The CLI flags might also be worth pulling dynamically, or at least telling Claude to also spot check against the --help text.

@afnanenayet
Copy link
Copy Markdown
Owner

I'm not sure what the best practices are for things like this, I can also see the argument for not having Claude constantly parse this information

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