LearnNewsExamplesServices
Frontmatter
id9046
titleFix: DevRank Tracker Timestamp Update Logic
stateClosed
labels
bugai
assigneestobiu
createdAtFeb 7, 2026, 11:16 PM
updatedAtFeb 7, 2026, 11:18 PM
githubUrlhttps://github.com/neomjs/neo/issues/9046
authortobiu
commentsCount1
parentIssue8930
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtFeb 7, 2026, 11:18 PM

Fix: DevRank Tracker Timestamp Update Logic

Closed v12.0.0 bugai
tobiu
tobiu commented on Feb 7, 2026, 11:16 PM

Fix a logic bug in Storage.updateTracker where null timestamps prevent updates.

The Bug: The current comparison logic (update.lastUpdate && update.lastUpdate > existingTime) fails when existingTime is null. In JavaScript, comparing a string to null (e.g., '2026-01-01' > null) evaluates to false.

Consequence: Users with lastUpdate: null (newly discovered users) never get their timestamp updated after a successful scan. They remain null in tracker.json and are re-scanned indefinitely by the Updater.

Fix: Update the condition in apps/devrank/services/Storage.mjs to explicitly treat null as an update-able state.

if (existingTime == null || (update.lastUpdate && update.lastUpdate > existingTime))

(using == null covers both null and undefined)

tobiu added the bug label on Feb 7, 2026, 11:16 PM
tobiu added the ai label on Feb 7, 2026, 11:16 PM
tobiu added parent issue #8930 on Feb 7, 2026, 11:17 PM
tobiu assigned to @tobiu on Feb 7, 2026, 11:17 PM
tobiu referenced in commit 92ad68a - "fix: DevRank Tracker Timestamp Update Logic (#9046)" on Feb 7, 2026, 11:18 PM
tobiu
tobiu Feb 7, 2026, 11:18 PM

Input from Gemini 3 Pro:

✦ Fixed the updateTracker logic bug that prevented updates for users with null timestamps.

Fix

  • Changed the condition existingTime === undefined to existingTime == null (which covers both null and undefined).
  • This ensures that users with lastUpdate: null (new candidates) are now correctly updated with the new timestamp instead of being skipped.

Code

} else if (existingTime == null || (update.lastUpdate && update.lastUpdate > existingTime)) {

Code committed to dev branch.

tobiu closed this issue on Feb 7, 2026, 11:18 PM