Skip to content

Commit 029c535

Browse files
logaretmclaude
andcommitted
fix: restore original ERROR_EMPTY_MESSAGE_TEXT wording
Keeps the default error string identical to the pre-PR version so standalone sendFeedback consumers matching on the exact message string see no behavior change. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 4333b92 commit 029c535

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

packages/feedback/src/constants/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ export const HIGHLIGHT_TOOL_TEXT = 'Highlight';
2626
export const HIDE_TOOL_TEXT = 'Hide';
2727
export const REMOVE_HIGHLIGHT_TEXT = 'Remove';
2828

29-
export const ERROR_EMPTY_MESSAGE_TEXT = 'Unable to submit feedback with an empty message';
29+
export const ERROR_EMPTY_MESSAGE_TEXT = 'Unable to submit feedback with empty message';
3030
export const ERROR_NO_CLIENT_TEXT = 'No client setup, cannot send feedback.';
3131
export const ERROR_TIMEOUT_TEXT = 'Unable to determine if Feedback was correctly sent.';
3232
export const ERROR_FORBIDDEN_TEXT =

packages/feedback/test/core/sendFeedback.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ describe('sendFeedback', () => {
269269

270270
it('throws when message is empty', () => {
271271
mockSdk();
272-
expect(() => sendFeedback({ message: '' })).toThrow('Unable to submit feedback with an empty message');
272+
expect(() => sendFeedback({ message: '' })).toThrow('Unable to submit feedback with empty message');
273273
});
274274

275275
it('throws when no client is set up', async () => {

0 commit comments

Comments
 (0)