@maxeonyx have u seen how we handle /init?

I was thinking a cleaner solution here would be to make compact a built in slash command, a user could override the prompt by defining their own compact and it wouldn't require additional config fields

The main change this would require is allowing slash commands to register functions to execute, which ofc couldn't be done in json but perhaps for now that functionality would remain internal

I haven't seen that, no, but I assumed compact has special handling due to the fact that it replaces the full context window. I am very open to doing this a different way.

@rekram1-node in your suggestion, then, I would define my own /handover slash command, with a different prompt, and configured to remove the rest of the context same as /compact (and we would also make /compact be implemented the same way)

@rekram1-node

@maxeonyx have u seen how we handle /init?

I was thinking a cleaner solution here would be to make compact a built in slash command, a user could override the prompt by defining their own compact and it wouldn't require additional config fields

The main change this would require is allowing slash commands to register functions to execute, which ofc couldn't be done in json but perhaps for now that functionality would remain internal

I agree, I think
1.compact should be a default includes slash command
2. Default included slash commands should be able to be overriden by user custom commands
3. User custom commands should be able to run as functions, something like #4411

I would advocate for trying to build the built-in slash commands such as /compact with the sdk, in other words forcing yourself to use the same functionality that a custom plugin developer would use. That would drive a architecture which makes sure that user custom commands can be just as powerful as the ones provided by default.

Yup total agreement, currently jumping between several fixes and then I can come back to these new feature prs

I have the new version mostly implemented but I'm on holiday atm - feel free to take it over or close as I won't able to do the final "runnable as functions" part. But I have done the "custom md file to implement custom compact-type command". I can add precedence for overriding /compact - that's a good idea.

@rekram1-node @einarpersson This PR is ready for your review, hope it's what you are imagining. It has turned out pretty clean I think.

@rekram1-node Keeping this up-to date is becoming a hassle - would you be able to review?

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

1. Open an issue describing the bug/feature (if one doesn't exist)
2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.