Fix uncertainty buttons, add report_private, retro improvements, Friday dates#26
Fix uncertainty buttons, add report_private, retro improvements, Friday dates#26
Conversation
… team report - Add FridayOfWeek() helper to compute Friday from the week's Monday - Change report filenames and email subjects from Monday to Friday date - When generating boss report, skip LLM pipeline if team report already exists for the same week — read, parse, and re-render as boss mode instead - Update findLatestReportBefore to match new Friday-based filenames - Fix loadSectionOptionsForModal to use Monday from ReportWeekRange Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates weekly report generation to use Friday (“week ending”) dates in outputs and introduces a boss-report shortcut that derives directly from an existing team report, reducing LLM usage when possible.
Changes:
- Add boss-mode fast path to derive the boss report from an existing team report file (skips LLM).
- Switch report filenames and upload comments to use the week’s Friday date.
- Add
FridayOfWeekhelper and adjust template-loading logic to use the correct week anchor date.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| slack.go | Adds boss-from-team shortcut, switches generated report filenames/comments to Friday dates, and fixes modal template loading to anchor on the report week Monday. |
| report_builder.go | Updates “current report” filename derivation to match Friday-based report naming. |
| models.go | Introduces FridayOfWeek helper for consistent Friday date derivation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| teamReportPath := filepath.Join(cfg.ReportOutputDir, teamReportFile) | ||
| if content, readErr := os.ReadFile(teamReportPath); readErr == nil && len(content) > 0 { | ||
| log.Printf("generate-report boss: deriving from existing team report %s", teamReportPath) |
There was a problem hiding this comment.
The boss-mode shortcut treats any ReadFile error the same as “no existing team report”, so permission/IO errors will silently fall through to the full (token-consuming) pipeline. Handle readErr explicitly: if os.IsNotExist(readErr) continue, otherwise surface an ephemeral error (and log) and return.
There was a problem hiding this comment.
@copilot open a new pull request to apply changes based on this feedback
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Summary
report_privateconfig option: When true,/generate-reportDMs the report to the caller instead of posting to the channel/retrospectiveto/retrospect: Shorter command name/generate-report boss, skip the LLM pipeline entirely if a team report for the same week already exists — derive the boss report directly from the team markdownTest plan
/reportitems and/listwork as before/generate-report teamproduces a file with Friday date (e.g.MyTeam_20260213.md)/generate-report bossafter team report exists: skips LLM, derives from team report, outputs.emlwith Friday date/generate-report bosswithout existing team report: falls through to full LLM pipeline.emlsubject line uses Friday datereport_private: truesends report as DM/retrospectsuggestions show Apply/Dismiss correctlyCGO_ENABLED=1 go test -v ./...passes (35 tests)🤖 Generated with Claude Code