Skip to content

v0.1.1

Choose a tag to compare

View all tags
@Yaki-Blake Yaki-Blake released this 01 Oct 11:50
0.1.1
33b886f

Fixes all findings from the community review report.

Fixed

  • package-lock.json out of sync with package.json (root cause of the failed review): the lock file did not record @types/node and builtin-modules, so npm ci aborted with EUSAGE and both the source review and the build verification reported a dependency-install failure. The lock file is now in sync — npm ci succeeds.
  • Removed the unused builtin-modules and @types/node dev dependencies (this project builds with the esbuild CLI only).

Changed

  • Replaced the require()-style imports with a typed import { remote } from 'electron' (still built as external; runtime behaviour unchanged).
  • Removed every explicit any; added minimal ambient typings for the Electron remote API and the Obsidian internals used.
  • Empty catch blocks now log the error; unused catch bindings removed.
  • Command name no longer repeats the plugin name.
  • Settings tab migrated to the declarative settings API (getSettingDefinitions), so the settings are also reachable from the settings search.
  • README.md is now bilingual (English + 中文).
  • Removed the empty heading element created with createEl('h2').

Verification

  • npm ci --dry-run passes
  • tsc --noEmit — 0 errors
  • eslint src/ package.json (obsidianmd recommended config) — 0 problems

Note

  • minAppVersion is now 1.13.0, required by the declarative settings API.
Assets 5
Loading