Skip to content

feature: add ai agent skills#152

Open
TheRedDaemon wants to merge 9 commits into
mainfrom
feature/add-ai-agent-support-files
Open

feature: add ai agent skills#152
TheRedDaemon wants to merge 9 commits into
mainfrom
feature/add-ai-agent-support-files

Conversation

@TheRedDaemon

@TheRedDaemon TheRedDaemon commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Meant to help with support tasks to guide agents a bit better.

  • Feeback on naming wanted
  • "setup-function" could maybe just be a parsing script, although it would need to look into multiple files.
  • "enable-local-function-dev" could also be a script, potentially.
  • "find-function-implementation" might also be redundant in general, but the AI can receive guidance through our codebase this way.

Just testing around. There might be better way to do this. If we keep them, then can also be extended/changed. Especially the transforming scripts, should new unknown cases arise.

Creating an AGENTS.md is planned, as is a collection of MSVC structures I remember (I need to know where to put that one though, since it would be a more general list).
I would keep the scope of the PR to the skills for now, though.

@TheRedDaemon TheRedDaemon added the in progress PR functions as reminder, input not needed yet. label Jul 11, 2026
@TheRedDaemon TheRedDaemon changed the title feature: add ai agent support files feature: add ai agent skills Jul 12, 2026
@TheRedDaemon TheRedDaemon removed the in progress PR functions as reminder, input not needed yet. label Jul 12, 2026
@TheRedDaemon TheRedDaemon requested a review from gynt July 12, 2026 00:09
@TheRedDaemon TheRedDaemon marked this pull request as ready for review July 12, 2026 00:09

Replace member calls using global objects:

`Namespace::Class::Function(&Global, args)` or `Namespace::Class::Function(Global, args)` → `MACRO_CALL_MEMBER(Namespace::Class_Func::Function, Global::ptr)(args)`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Or Namespace::Class::Function(this, args) ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added additional point for member functions of the same class. Moved it under field transform. May not hurt to have the "determine class" part over the other, but no idea if this has any effect.

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.

2 participants