hotel-manage-stack/wages
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove second stale NI-oncost caveat missed in the earlier cleanup

Browse source 
The Rota vs Actual section's caveat was already fixed, but the
Forecast section had its own separate "rota excludes NI oncosts" note
that referenced the same now-defunct /api/v2/schedules limitation —
confirmed the forward window genuinely carries real oncosts post
roster-migration (e.g. FOH 25/07: £562.75 base vs £641.93 w/ oncosts),
so this was actively telling the model to understate its own forecast
confidence for no reason.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
jtricerolph 2026-07-25 10:38:02 +00:00
parent 1ffaece874
commit ed67a91b52
1 changed files with 0 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

7
backend/src/jobs/ai-insights.js
View file

@ -413,13 +413,6 @@ export function buildPrompt(monthProgress, priorPeriod, rotaVsActual, forecast,
lines.push('') lines.push('')
lines.push(`## Forecast (method: ${forecast.forecastMethod}, ${forecast.remainingDaysCount} days remaining)`) lines.push(`## Forecast (method: ${forecast.forecastMethod}, ${forecast.remainingDaysCount} days remaining)`)
if (forecast.forecastMethod === 'rota') {
lines.push(
"Note: remaining days sourced from rota exclude employer NI oncosts (Workforce's schedules API " +
"doesn't supply them), so this projection may modestly understate the true month-end total — same " +
"known limitation shown on the Monthly page."
)
}
lines.push( lines.push(
`Projected month-end total: ${fmtMoney(forecast.projectedTotal)} vs budget ${fmtMoney(monthProgress.budgetAmount)}` + `Projected month-end total: ${fmtMoney(forecast.projectedTotal)} vs budget ${fmtMoney(monthProgress.budgetAmount)}` +
(monthProgress.budgetAmount != null (monthProgress.budgetAmount != null