Skip to content

🔧 Remove refresh option and always force refresh in contract report #2174

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mkmccarty merged 2 commits into from
Feb 7, 2026
Merged

🔧 Remove refresh option and always force refresh in contract report #2174

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3452176
🔧 Remove refresh option and always force refresh in contract report
mkmccarty Feb 7, 2026
b21cb31
🔧 fix: remove trailing whitespace in contract report comments
mkmccarty Feb 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 7 additions & 12 deletions src/boost/contract_report.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ func userMessage(err error) string {
case errors.Is(err, ErrNoChannelContract):
return "No contract found in this channel. Please provide a contract-id."
case errors.Is(err, ErrEvaluationNotFound):
return "Evaluation not found, if you just completed the contract please wait a few minutes and try again with refresh=true."
return "Evaluation not found, if you just completed the contract please wait a few minutes and try again."
Copy link

Copilot AI Feb 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This user-facing string is a run-on sentence and reads awkwardly. Consider splitting it into two sentences (e.g., end after "not found") to improve clarity.

Suggested change
return "Evaluation not found, if you just completed the contract please wait a few minutes and try again."
return "Evaluation not found. If you just completed the contract, please wait a few minutes and try again."

Copilot uses AI. Check for mistakes.
case errors.Is(err, ErrCoopIDMissing):
return "No coop ID found for this contract evaluation."
case errors.Is(err, ErrUnsupportedCXPVersion):
Expand Down Expand Up @@ -150,12 +150,6 @@ func GetSlashContractReportCommand(cmd string) *discordgo.ApplicationCommand {
Required: true,
Autocomplete: true,
},
{
Type: discordgo.ApplicationCommandOptionBoolean,
Name: "refresh",
Description: "If you want to force a refresh due a recent change to your contracts.",
Required: false,
},
{
Type: discordgo.ApplicationCommandOptionBoolean,
Name: "token-details",
Expand Down Expand Up @@ -326,7 +320,7 @@ func processContributors(
// Parameters:
// - s: active Discord session
// - i: the triggering interaction
// - optionMap: slash-command options (e.g., "contract-id", "refresh").
// - optionMap: slash-command options (e.g., "contract-id").
// - userID: Discord user ID of the caller
// - okayToSave: whether API fetches may be cached/persisted.
//
Expand All @@ -346,10 +340,11 @@ func ContractReport(
p := contractReportParameters{}

// --- Options ---
forceRefresh := false
if opt, ok := optionMap["refresh"]; ok {
forceRefresh = opt.BoolValue()
}

// Always want forceRefresh true to ensure we have the latest contract evaluations.
// This is typically run right after a contract completes and needs the refresh anyway.
forceRefresh := true

showTokenDetails := false
if opt, ok := optionMap["token-details"]; ok {
showTokenDetails = opt.BoolValue()
Expand Down
Loading