feat(status): temporarily remove markFinalWithCheckmark feature due to inconsistencies
This commit is contained in:
parent
6906c7ecf6
commit
ec52035285
@ -67,7 +67,7 @@ const DEFAULT_CONFIG: Required<StatusUpdateConfig> = {
|
||||
updateIntervalMs: 5000,
|
||||
showPhases: true,
|
||||
showElapsedTime: true,
|
||||
markFinalWithCheckmark: true,
|
||||
markFinalWithCheckmark: false,
|
||||
elapsedTimeThresholdMs: 3000,
|
||||
};
|
||||
|
||||
@ -200,14 +200,7 @@ export class StatusUpdateController {
|
||||
|
||||
if (!this.isEnabled()) return finalText;
|
||||
|
||||
// If we have a final text and config says to mark with checkmark
|
||||
if (finalText && this.config.markFinalWithCheckmark) {
|
||||
// NOTE: We rely on Telegram delivery to do the edit/replacement, so we just format
|
||||
|
||||
const marked = `${finalText.trimEnd()} ✅`;
|
||||
|
||||
return marked;
|
||||
}
|
||||
// The final message is returned, relying on message deletion/cleanup.
|
||||
|
||||
// Clean up status message if it exists and we're not editing
|
||||
if (this.state.statusMessageId && this.callbacks.deleteStatus) {
|
||||
|
||||
@ -268,7 +268,7 @@ const FIELD_LABELS: Record<string, string> = {
|
||||
"agents.defaults.statusUpdates.updateIntervalMs": "Status Update Interval (ms)",
|
||||
"agents.defaults.statusUpdates.showPhases": "Show Processing Phases",
|
||||
"agents.defaults.statusUpdates.showElapsedTime": "Show Elapsed Time",
|
||||
"agents.defaults.statusUpdates.markFinalWithCheckmark": "Mark Final with ✅",
|
||||
|
||||
"agents.defaults.statusUpdates.elapsedTimeThresholdMs": "Elapsed Time Threshold (ms)",
|
||||
"agents.defaults.cliBackends": "CLI Backends",
|
||||
"commands.native": "Native Commands",
|
||||
|
||||
@ -184,8 +184,7 @@ export type StatusUpdateConfig = {
|
||||
showPhases?: boolean;
|
||||
/** Show elapsed time in status messages (default: true). */
|
||||
showElapsedTime?: boolean;
|
||||
/** Mark final responses with ✅ (default: true). */
|
||||
markFinalWithCheckmark?: boolean;
|
||||
|
||||
/** Minimum duration before showing elapsed time (ms, default: 3000). */
|
||||
elapsedTimeThresholdMs?: number;
|
||||
};
|
||||
|
||||
@ -525,8 +525,7 @@ export const StatusUpdateSchema = z
|
||||
showPhases: z.boolean().optional(),
|
||||
/** Show elapsed time in status messages (default: true). */
|
||||
showElapsedTime: z.boolean().optional(),
|
||||
/** Mark final responses with ✅ (default: true). */
|
||||
markFinalWithCheckmark: z.boolean().optional(),
|
||||
|
||||
/** Minimum duration before showing elapsed time (ms, default: 3000). */
|
||||
elapsedTimeThresholdMs: z.number().int().nonnegative().optional(),
|
||||
})
|
||||
|
||||
Loading…
Reference in New Issue
Block a user