Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added .cursor/rules folder and update .cursorrules promts #58

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

skulidropek
Copy link
Contributor

Issue

Towards #57 .

Progress

I updated the prompt system and integrated key components to improve the structure and functionality of the project:

@skulidropek
Copy link
Contributor Author

This is an example, he always starts writing code by viewing the available documentation. I am still finalizing the documentation that he has. If you have useful information, drop it here and I will add it to the agent's knowledge

изображение

Copy link
Member

@novusnota novusnota left a comment

Choose a reason for hiding this comment

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

LGTM, @i582 @Shvandre would you add/remove anything?

@skulidropek
Copy link
Contributor Author

LGTM, @i582 @Shvandre would you add/remove anything?

This is not the final version yet. I will let you know when I finish with the prompts. I noticed that the context window has shrunk a lot due to the addition of a large number of prompts.

@skulidropek
Copy link
Contributor Author

LGTM, @i582 @Shvandre would you add/remove anything?

This is not the final version yet. I will let you know when I finish with the prompts. I noticed that the context window has shrunk a lot due to the addition of a large number of prompts.

I think I'll have to remove most of the unimportant hints. Make them more universal.

@skulidropek
Copy link
Contributor Author

I think we'll also need to tell the AI ​​about CLI commands that will allow it to browse node module folders to see additional information about the TS and TACT library source code.

ls, cat, tree

@skulidropek
Copy link
Contributor Author

I think that we should leave information about all standards (tep). About what the source code of Jeton and NFT looks like.

Everything else is not that important information. He can get it by decompiling and thinking in agent mode.

@novusnota novusnota self-requested a review February 25, 2025 12:13
@@ -0,0 +1,92 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

Sure, I'll mark this as a draft. Also, let's remove this contract.

@novusnota novusnota marked this pull request as draft February 25, 2025 14:46
@skulidropek
Copy link
Contributor Author

I think we'll also need to tell the AI ​​about CLI commands that will allow it to browse node module folders to see additional information about the TS and TACT library source code.

ls, cat, tree

Add support in System Promt for commands for working with a GitHub repository and a local project.

git log -p -G "hello" --no-merges --pretty=format:"COMMIT: %h | %s%nAUTHOR: %an | DATE: %ad%nDESCRIPTION: %b%n" | grep -C3 "hello"
grep -r "hello" .
git blame

@skulidropek
Copy link
Contributor Author

I think we'll also need to tell the AI ​​about CLI commands that will allow it to browse node module folders to see additional information about the TS and TACT library source code.

ls, cat, tree

Add support in System Promt for commands for working with a GitHub repository and a local project.

git log -p -G "hello" --no-merges --pretty=format:"COMMIT: %h | %s%nAUTHOR: %an | DATE: %ad%nDESCRIPTION: %b%n" | grep -C3 "hello"
grep -r "hello" .
git blame

For now, the current plan is as follows: All documentation/contracts must be kept in a separate folder. AI has access to work with the project. Access to git changes and search across the entire project.

AI always checks the relevance of its information by comparing data with local folders in which documentation and an example of best contract practices will be located.

Two additional folders will be created: .docs .contrasts

or .information or and it will have subfolders like .docs and .contrasts

For a regular user, these are optional folders, they will serve for the CLI AI to work with the agent mode

@anton-trunov
Copy link
Member

@skulidropek btw, not sure if this helps, maybe you can tell: we have some recommendations on gas optimizations for Tact contracts, is it possible to somehow incorporate these: tact-lang/tact#1868?

@skulidropek
Copy link
Contributor Author

@skulidropek btw, not sure if this helps, maybe you can tell: we have some recommendations on gas optimizations for Tact contracts, is it possible to somehow incorporate these: tact-lang/tact#1868?

Yes, we can write system prompts taking into account these preferences. But we must understand that AI is a probability. If we want to get rid of certain bad practices in the code, it is best to write a static analyzer that will scold the AI ​​for any bad misdeed.

Moved some hints from .cursor/rules into them.

Improved the prompt in the .cursorrules file
Made it more agent-friendly, added info about different commands
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