Skip to content

fix: COO brief accuracy -- skeleton templates, NPS source, ALICE windows, timestamps#2372

Closed
gabrielratner-debug wants to merge 76 commits into
nanocoai:mainfrom
gabrielratner-debug:fix/coo-brief-accuracy-anis-feedback
Closed

fix: COO brief accuracy -- skeleton templates, NPS source, ALICE windows, timestamps#2372
gabrielratner-debug wants to merge 76 commits into
nanocoai:mainfrom
gabrielratner-debug:fix/coo-brief-accuracy-anis-feedback

Conversation

@gabrielratner-debug
Copy link
Copy Markdown

Summary

Addresses accuracy issues flagged by Austin GM on the May 2026 COO Brief.

  • Locked property detail and executive summary to fixed fill-in skeletons -- prevents model from inventing extra sections (P&L tables, KPI header cards, full-month OTB tables)
  • Fixed NPS source filter: was 'Survey', data uses 'Revinate Surveys' -- was matching nothing and producing wrong counts/scores
  • Fixed mid-month forecast accuracy table: replaced raw MTD-vs-full-month delta with Actual % of Fcst + Month Elapsed benchmark so the numbers are interpretable at any point in the month
  • Added 3:00am cache timestamps to P.Fcst and Duetto OTB section headers
  • Expanded ALICE from last-7d only to Last 7/30/60/90d windows; removed MTD/YTD counts (wrong by construction -- prefetch only caches recent data)
  • Added bundled Overall rating row to platform reviews table
  • Required exact computed values in narrative bullets -- no rounding or approximation
  • Clarified STLY definition as OTB at same date last year, not full LY actuals

Test plan

  • Run /coo-prefetch then /coo-briefing and verify property sections match skeleton structure
  • Confirm NPS shows ~39 survey responses and NPS ~+72 for L7d (was 7 responses / +86)
  • Confirm 3c shows Actual % of Fcst + Month Elapsed columns, not raw delta
  • Confirm no P&L, KPI header, or full-month OTB sections appear
  • Confirm ALICE shows 4 rows (Last 7/30/60/90d) per property

🤖 Generated with Claude Code

gavrielc and others added 30 commits March 8, 2026 22:52
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…penAI Whisper

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Syncs with upstream main (on schedule, dispatch, or push), then
merges main into all skill/* branches with build+test validation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
github-actions Bot and others added 24 commits March 11, 2026 10:26
# Conflicts:
#	package-lock.json
#	package.json
#	repo-tokens/badge.svg
# Conflicts:
#	package-lock.json
#	package.json
#	repo-tokens/badge.svg
# Conflicts:
#	.env.example
#	.github/workflows/fork-sync-skills.yml
#	package-lock.json
#	repo-tokens/badge.svg
Adds outlook-mcp (75+ tools) to container agents — email, calendar,
contacts, tasks, and Teams via Microsoft Graph API. Mounts credentials
and token file, injects Azure OAuth env vars, and includes SKILL.md
for guided setup.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Outlook channel (src/channels/outlook.ts) with MSAL auth and polling
- Voice transcription dependency and channel updates
- Operational scripts: weekly restart, Docker startup, OAuth token sync,
  Outlook login, Claude session launcher
- Package dependency updates (Azure, googleapis, grammy)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ows, timestamps

- Replace property detail and executive summary with locked fill-in
  skeletons to prevent model from improvising extra sections (P&L tables,
  full-month OTB tables, KPI header cards)
- Fix NPS filter: 'Revinate Surveys' not 'Survey' (was matching nothing,
  producing wrong counts and NPS scores)
- Replace MTD-vs-full-month forecast comparison with Actual % of Fcst
  + Month Elapsed benchmark so mid-month data is interpretable
- Add 3:00am cache timestamps to P.Fcst and Duetto OTB sections
- Expand ALICE from last-7d to Last 7/30/60/90d windows; remove MTD/YTD
  counts (wrong by construction)
- Add bundled Overall rating row to platform reviews table
- Require exact computed values in narrative bullets (no rounding)
- Clarify STLY = OTB at same date last year, not full LY actuals

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@gabrielratner-debug
Copy link
Copy Markdown
Author

Landed directly on main (3f8c214) due to branch conflict with local ahead-of-origin state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants