Thank you for your submission, an automated scan of your plugin code's revealed the following issues:

### Required

[[1](https://github.com/skye-z/ons/blob/b05803018dcf439424e89626a698486839d34a6b/obsidian-plugin/src/peer-manager.ts#L253)][[2](https://github.com/skye-z/ons/blob/b05803018dcf439424e89626a698486839d34a6b/obsidian-plugin/src/peer-manager.ts#L275)]:Obsidian's configuration directory isn't necessarily `.obsidian`, it can be configured by the user. You can access the configured value from `Vault#configDir`

[[1](https://github.com/skye-z/ons/blob/b05803018dcf439424e89626a698486839d34a6b/obsidian-plugin/src/peer-manager.ts#L441)][[2](https://github.com/skye-z/ons/blob/b05803018dcf439424e89626a698486839d34a6b/obsidian-plugin/src/peer-manager.ts#L450)][[3](https://github.com/skye-z/ons/blob/b05803018dcf439424e89626a698486839d34a6b/obsidian-plugin/main.ts#L85)]:You should change all instances of `var` to either `const` or `let`. `var` has function-level scope, so it can easily lead to bugs if you're not careful. Here's a pretty good guide with examples of why not to use `var`: [why `var` is obsolete](https://javascript.plainenglish.io/4-reasons-why-var-is-considered-obsolete-in-modern-javascript-a30296b5f08f)



---

### Optional

[[1](https://github.com/skye-z/ons/blob/b05803018dcf439424e89626a698486839d34a6b/obsidian-plugin/src/peer-manager.ts#L157)]:Use `app.fileManager.trashFile(file: TAbstractFile)` instead, this will make sure that the file is deleted according to the users preferences.



---



Do <b>NOT</b> open a new PR for re-validation.
Once you have pushed all of the required changes to your repo, the bot will update the labels on this PR within 6 hours.
If you think some of the required changes are incorrect, please comment with `/skip` and the reason why you think the results are incorrect.