core icon indicating copy to clipboard operation
core copied to clipboard

[Chore] Rename all `stray` arguments in endpoints as `cat`

Open pieroit opened this issue 10 months ago • 4 comments

In plugins hook, tools and forms the StrayCat instance is called cat, while in core and custom endpoints it is named stray. This brought confusion in devx and avoidable punctualizations in the docs.

Let's rename all stray references as simply cat, keeping StrayCat as a class. There should be no conflict with the main application package (which is also called cat) because it is never imported directly.

In case interested in contribution, please comment to get the issue assigned

pieroit avatar Feb 12 '25 18:02 pieroit

Please assign me this issue i will work on this.

MihirKohli avatar Feb 15 '25 17:02 MihirKohli

Should i make these changes across all the files in core?

MihirKohli avatar Feb 15 '25 17:02 MihirKohli

Should i make these changes across all the files in core?

Only endpoints for now! So cat/routes folder

Assigning to you thanks 🙏

pieroit avatar Feb 15 '25 18:02 pieroit

raised pull request for the same @pieroit https://github.com/cheshire-cat-ai/core/pull/1031

MihirKohli avatar Feb 17 '25 15:02 MihirKohli

this issue is closed

MihirKohli avatar Oct 02 '25 15:10 MihirKohli