Skip to content

fix: clear scorecard warnings for 0.4.9#65

Merged
rogerdigital merged 1 commit into
mainfrom
release/0.4.9-scorecard-fixes
Jun 25, 2026
Merged

fix: clear scorecard warnings for 0.4.9#65
rogerdigital merged 1 commit into
mainfrom
release/0.4.9-scorecard-fixes

Conversation

@rogerdigital

Copy link
Copy Markdown
Owner

Summary

  • clear Obsidian Community Scorecard risks and warnings by replacing unsafe Notebook Navigator integration typing with explicit interfaces and guards
  • remove unsafe TFile casts, Buffer usage, and document focus access flagged by the scanner
  • bump release metadata to 0.4.9 and add the Scorecard lint command to CI

Root cause

The latest release still contained a small amount of untyped third-party integration code and browser/global assumptions. Obsidian Community's Scorecard scanner surfaced those as unsafe TypeScript access, disallowed explicit any usage, an undescribed eslint directive, a TFile cast, and activeDocument compatibility warnings.

Test plan

  • npm run lint:obsidian-warnings
  • npm run build
  • npm test
  • git diff --check

@rogerdigital rogerdigital merged commit 4119104 into main Jun 25, 2026
1 check passed
@rogerdigital rogerdigital deleted the release/0.4.9-scorecard-fixes branch June 25, 2026 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant