Skip to content

feat(scripts): add script to collect function data#163

Open
TheRedDaemon wants to merge 3 commits into
mainfrom
feature/add-function-checker-script
Open

feat(scripts): add script to collect function data#163
TheRedDaemon wants to merge 3 commits into
mainfrom
feature/add-function-checker-script

Conversation

@TheRedDaemon

@TheRedDaemon TheRedDaemon commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Relies on a ton of regex. Since it is a support script, I am okey with it breaking in edge cases.
I just hope our general function declaration will not change anymore. 🙃

EDIT: Added also a simple implemtation file setup script.
I also changed both skills. I am not 100 percent sure if I want this, since now they rely on the powershell scripts running.
But is saves tokens..
What do you think?

@TheRedDaemon
TheRedDaemon requested a review from gynt July 18, 2026 01:32
@TheRedDaemon
TheRedDaemon force-pushed the feature/add-function-checker-script branch from daa3e98 to d0488b5 Compare July 18, 2026 18:39
@gynt

gynt commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

I am wondering why adding a mcp server entry isn't easier. Then the skill can refer to this MCP entry point to do the stuff we want to save tokens.

@TheRedDaemon

TheRedDaemon commented Jul 18, 2026

Copy link
Copy Markdown
Contributor Author

I want to use the functions also manually, mainly. This is not only to aid the AI workflow.

I was thinking about making it at least python scripts, but:

  • I do not like to call them via "python ..."
  • I was premature optimizing by assuming that file scanning inside python should be slower than via a tool like "findstr". Do I have prove? Not at all. That being said, one would also able to call it via python.

Skill vs. MCP is a whole other debate. Some say a good CLI via skills is better for clear tasks, since it limits the initial token usage (since it discovers incremental) and might communicate with shorter returns.

Maybe this is pedantic.
Should I transform them to python scripts? They can then integrate with everything.

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