fix: set autoUnload in onLoad()

[qnixsynapse]

Describe Your Changes

The variable was not initialised resulted in always setting true when starting.

This change fixes it.

---

Important

Initialize autoUnload in onLoad() to use configuration value in index.ts.

• Behavior:
  • Initialize autoUnload in onLoad() in index.ts to use this.config.auto_unload instead of defaulting to true.
• Misc:
  • Fixes issue where autoUnload was always set to true on start.

^{This description was created by for b243012. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to b243012 in 1 minute and 0 seconds. Click for details.

• Reviewed 13 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. extensions/llamacpp-extension/src/index.ts:171

• Draft comment:
  Good fix: initializing this.autoUnload from the config ensures it reflects the saved setting rather than always defaulting to true. Consider aligning naming (e.g. auto_unload → autoUnload) if feasible.
• Reason this comment was not posted:
  Comment looked like it was already resolved.

Workflow ID: wflow_HXCHaXycbYly12JU

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[github-actions]

Barecheck - Code coverage report

Total: 38.73%

Your code coverage diff: 0.01% ▴

✅ All code changes are covered

[louis-jan]

LGTM and work great