Add cerebrum

[DewSecOff]

Add Cerebrum plugin to the Caido store

I am submitting a new Plugin Package

Repository URL

https://github.com/DewSecOff/Caido-Plugin-Cerebrum

Link to my plugin package: https://github.com/DewSecOff/Caido-Plugin-Cerebrum/releases/tag/1.0.1

Release Checklist

• I have tested the plugin on
  • Windows
  • macOS
  • Linux
• My GitHub release contains all required files
  • plugin_package.zip
  • plugin_package.zip.sig
• GitHub Tag name matches the exact version number specified in my manifest.json
• The id in my manifest.json matches the id in the plugin_packages.json file.
• My README.md describes the plugin package purpose and provides clear usage instructions.
• I have read the developer policy at https://developer.caido.io/policy.html, and have assessed my plugin package adherence to this policy.
• I have added a license in the LICENSE file and it matches the license field in the plugin_packages.json file.
• My project respects and is compatible with the original license of any third-party code that I'm using.
  I have given proper attribution to these other projects in my README.md and/or LICENSE.

[CLAassistant]

All committers have signed the CLA.

[Sytten]

@DewSecOff Started to check your plugin, first thing would be to add a gitignore because your are adding the dist folder and the node_modules to your repo which is not good 😅

[Sytten]

Analysis:

• You probably dont want to run the dump on every saved request: https://github.com/DewSecOff/Caido-Plugin-Cerebrum/blob/bf5082ed51a289710b9d659be6c70f4ac8dae6de/packages/backend/src/index.ts#L18
• You could only run https://github.com/DewSecOff/Caido-Plugin-Cerebrum/blob/bf5082ed51a289710b9d659be6c70f4ac8dae6de/packages/backend/src/index.ts#L79-L94 on plugin init, no need to run it on every saved request, also do remember that if you need to add fields in the future you will need to do a database migrations since the table will already exist
• You should be able to do import { Database } from 'sqlite' instead of https://github.com/DewSecOff/Caido-Plugin-Cerebrum/blob/bf5082ed51a289710b9d659be6c70f4ac8dae6de/packages/backend/src/index.ts#L4 also you dont seem to use that variable anywhere

The frontend is good enough, just improve the small points above and we will get it merged :)

[DewSecOff]

Hi @Sytten ! Thank you for your advices. I just modified the code.

• Moved table creation from saveRequest to plugin init() to avoid unnecessary execution
• Removed unused libs, function dumpAllRequests and others log call
• Added .gitignore to exclude dist/, node_modules/, and build artifacts

[Sytten]

Hi @DewSecOff

• There are still dist and node_modules in your subfolders (frontend/backend)
• It is better to keep using the same global db instead of re-creating it since it has a performance cost: https://github.com/DewSecOff/Caido-Plugin-Cerebrum/blob/main/packages/backend/src/index.ts#L44 (I created sdk.meta.db subsequent calls should return the same object caido#1636 to improve that)

Once that is done I am ready to merge it!

[DewSecOff]

Hi @Sytten
Sorry for this, seems that I have to upgrade my basic development skills :)

• I deleted the nodes modules and the dists folders.
• The db is now creating just one time at line https://github.com/DewSecOff/Caido-Plugin-Cerebrum/blob/main/packages/backend/src/index.ts#L9 and no more in lines https://github.com/DewSecOff/Caido-Plugin-Cerebrum/blob/main/packages/backend/src/index.ts#L44 and https://github.com/DewSecOff/Caido-Plugin-Cerebrum/blob/main/packages/backend/src/index.ts#L83

So the new release is the 1.3 :)

[Sytten]

Hey! I was on vacation, just got back and will look at it this week @darklotuskdb

[Sytten]

Merged, thank you!

[DewSecOff]

Thank you ! I try to unlock the CPTS currently, but I will add some features when I finish it :)