VaclavObornik / mongodash / 24609472314

91%

main: 92%

Build Type

push

github

Committed by VaclavObornik

Commit Message

refactor(cron): two small cleanups surfaced via LSP findReferences

1. getRegisteredCronTaskIds is documented in docs/cron-tasks.md as
   a public helper but was never re-exported from src/index.ts. LSP
   findReferences on the declaration returned just itself (0 real
   usages). grep would show the same but with no way to distinguish
   'dead code' from 'external API that consumers call'. The docs
   told me it should be public, so add the missing re-export so
   users can actually import it.

2. triggerCronTask has a @deprecated JSDoc steering users to
   scheduleCronTaskImmediately. LSP findReferences revealed that our
   own OperationalTaskController (the built-in dashboard bridge)
   was still calling the deprecated alias - a bad example for
   consumers grep-ing the repo. Switch the internal call and its
   test mock to scheduleCronTaskImmediately.

Both issues are the kind of inconsistency that symbol-aware search
surfaces cleanly where plain text search would have drowned in
false positives (planner / trigger are common substrings).

Coverage Stats

1492 of 1747 branches covered (85.4%)

Branch coverage included in aggregate %.

3 of 3 new or added lines in 2 files covered. (100.0%)

1 existing line in 1 file now uncovered.

2316 of 2428 relevant lines covered (95.39%)

370.28 hits per line