fix(nano_banana_dashboard): inject current date into prompt and cache key#116
fix(nano_banana_dashboard): inject current date into prompt and cache key#116natesilva wants to merge 1 commit into
Conversation
… key When the plugin refreshes after midnight, source plugin data (e.g. calendar’s today_in_tz) may still contain yesterday's date. The Gemini prompt had no independent date reference, causing the LLM to render calendars and date-dependent visuals for the wrong day. - Add current_date method returning user’s timezone-aware date - Inject today’s date into prompt before source data, with override instruction to prefer it over stale dates in source data - Prepend current_date to cache key so the image regenerates on date change Fixes usetrmnl#111
|
apologies for missing this @natesilva , tagging @ikraamg to review. |
|
Hi @natesilva 👋 Apologies once again for missing this. I have made two changes, one a couple weeks back, and one yesterday that might resolve this bug. Please let me know if this is still a relevant issue for you and I will double down on an improved fix. |
|
Hi @ikraamg. We can close this, and it’s because of TRMNL’s new plugin refresh strategy. In the past, the calendar’s I still think it would be good for these prompts to have the current date injected. The new refresh logic does solve the problem, but it’s an incidental fix rather than an explicit direct fix. Closing this. |
3 participants
When the plugin refreshes after midnight, source plugin data (e.g. calendar’s today_in_tz) may still contain yesterday's date. The Gemini prompt had no independent date reference, causing the LLM to render calendars and date-dependent visuals for the wrong day.
Fixes #111