feat: various improvement for claude configuration#297
Open
Conversation
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR addresses the problem that we had very big (300+ lines) AGENTS file, which could become problematic for AI to follow (i.e. to broad concepts, missing general information and a lot of specific knowledge (like type mappings))
This PR introduces:
check-formattinghook basically formats file after each changeAGENTS.mdinto three:AGENTS.md,rust-styleandportingskills.Things that are good: it now follows the instructions much better, spending less time locating the repository and understanding the pluto codebase
Things that are bad: it now follows the instructions very precisely, and we need to play with prompts more. For example, when asking to implement
dkg/syncpackage, it successfully and quickly navigatedcharon, gathered context and made a plan, but it copied the structure of the Go implementation 1-to-1, despite we have different approach to P2P.Specifically to p2p - I will create a separate skill (for now it's empty), explaining the structural differences between rust and go.