Pomo timer shows incorrect time left

[A-raj468]

🐛 Describe the bug

The issue

When I set a pomo timer of more than 60s, when displaying the timer additional 30m are added to the minute fields.
Eg: If I set a timer for 20m, it will show 50m, if I set timer for 1m 15s, it shows 31m 15s, ets. (One example included in the screenshot below)
The actual timer works correctly. Only the display, both in noice display box and lualine, are wrong.

Config(using lazy package manager):

return {
    {
        -- Pomodoro timer
        'epwalsh/pomo.nvim',
        version = '*',
        lazy = true,
        cmd = { 'TimerStart', 'TimerRepeat', 'TimerSession' },
        dependencies = {
            'rcarriga/nvim-notify',
        },
    },
}

My nvim config: nvim-config

Example screenshot

Setting pomo timer for 75s:

Timer showing 31m 10s

Timer goes to 59s once 16s have passed from original timer

Versions

❯ nvim --version
NVIM v0.10.0
Build type: RelWithDebInfo
LuaJIT 2.1.1713484068
Run "nvim -V1 -v" for more info
❯ nvim --headless -c 'lua print(require("pomo.version"))' -c q
0.7.0

My lazy-lock.json file.

[tonnycheruiyot]

I guess the minutes bit was fixed. The issue now is with the hour display. If the time left is beyond 1h, the time left is shown as x+3h, eg, If you set the timer to 1h30m, both the time left is displayed as 4h30m

[A-raj468]

For me, both the issues are there minutes showing additional 30m and hour showing additional 5h.

[A-raj468]

I realized the issue is that when formatting the time string, in the util.lua module the formatting function uses os.date("%Mm %Ss", time_left). This uses the local time zone to display the time(which in my case is UTC+0530, additional 5hr and 30min). This is exactly the additional time added in the display.

Using UTC to format instead of local time zone fixes the issue. Changing the format string in os.date as os.date("!%Mm %Ss", time_left) forces the os.date module to use UTC instead of local time zone.

I have requested this change in the PR #32. Hopefully the maintainer merges this change after reviewing it.

[tonnycheruiyot]

That actually makes sense. I hope the pr gets merged soon.

[dfenerski]

Hey, I experience this too on a fresh install. Any progress with the PR?

[A-raj468]

Nah, he hasn't replied yet to the PR. For now, I am using my fork. You can also use that if you want.