@mintlify/cli 4.0.1797 → 4.0.1798

package/__test__/feedback.test.ts

@@ -1,38 +1,30 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 
-import { authenticatedFetch } from '../src/authenticatedFetch.js';
 import { API_URL } from '../src/constants.js';
 import { sendFeedback } from '../src/feedback.js';
 
-vi.mock('../src/authenticatedFetch.js');
+const fetchMock = vi.fn();
+vi.stubGlobal('fetch', fetchMock);
 
 const input = { message: 'mint dev crashed', clientId: 'c1', category: 'bug' as const };
-const respond = (status: number) =>
-  vi.mocked(authenticatedFetch).mockResolvedValue(new Response('{}', { status }));
+const respond = (status: number) => fetchMock.mockResolvedValue(new Response('{}', { status }));
 
-beforeEach(() => vi.resetAllMocks());
+beforeEach(() => {
+  fetchMock.mockReset();
+});
 
 describe('mint feedback', () => {
-  it('posts the feedback with the cli source', async () => {
+  it('posts the feedback with the cli source and no login', async () => {
     respond(201);
     expect(await sendFeedback(input)).toBeUndefined();
-    const [url, init] = vi.mocked(authenticatedFetch).mock.calls[0]!;
+    const [url, init] = fetchMock.mock.calls[0]!;
     expect(url).toBe(`${API_URL}/api/cli/feedback`);
     expect(init?.method).toBe('POST');
+    expect(init?.headers).not.toHaveProperty('Authorization');
     expect(JSON.parse(String(init?.body))).toEqual({ ...input, source: 'cli' });
   });
-  it('asks for login on 401', async () => {
-    respond(401);
-    expect(await sendFeedback(input)).toContain('mint login');
-  });
-  it('asks for login when no credentials are stored', async () => {
-    vi.mocked(authenticatedFetch).mockRejectedValue(
-      new Error('Not authenticated. Run `mint login` to authenticate.')
-    );
-    expect(await sendFeedback(input)).toContain('mint login');
-  });
   it('reports a network failure', async () => {
-    vi.mocked(authenticatedFetch).mockRejectedValue(new Error('fetch failed'));
+    fetchMock.mockRejectedValue(new Error('fetch failed'));
     expect(await sendFeedback(input)).toBe('could not reach the server');
   });
   it('reports a server error', async () => {

package/bin/cli.js

@@ -561,7 +561,7 @@ export const cli = ({ packageName = 'mint' }) => {
         description: 'Feedback category',
     })
         .example('$0 feedback "mint dev crashes on a missing image" --client-id 7f3a --category bug', 'Report a bug')
-        .epilog('Requires mint login. Never include API keys, tokens, passwords, or other secrets in the message.'), (argv) => __awaiter(void 0, void 0, void 0, function* () {
+        .epilog('Never include API keys, tokens, passwords, or other secrets in the message.'), (argv) => __awaiter(void 0, void 0, void 0, function* () {
         const sent = yield feedback({
             message: argv.message,
             clientId: argv.clientId,

package/bin/feedback.js

@@ -9,26 +9,21 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 };
 import { jsx as _jsx } from "react/jsx-runtime";
 import { addLog, ErrorLog, SuccessLog } from '@mintlify/previewing';
-import { authenticatedFetch } from './authenticatedFetch.js';
 import { API_URL } from './constants.js';
 export const FEEDBACK_CATEGORIES = ['bug', 'docs', 'idea'];
-const LOGIN_REQUIRED = 'not logged in. Run `mint login` first, then retry.';
 export function sendFeedback(input) {
     return __awaiter(this, void 0, void 0, function* () {
         let res;
         try {
-            res = yield authenticatedFetch(`${API_URL}/api/cli/feedback`, {
+            res = yield fetch(`${API_URL}/api/cli/feedback`, {
                 method: 'POST',
                 headers: { 'Content-Type': 'application/json' },
                 body: JSON.stringify(Object.assign(Object.assign({}, input), { source: 'cli' })),
             });
         }
-        catch (error) {
-            const notLoggedIn = error instanceof Error && /not authenticated/i.test(error.message);
-            return notLoggedIn ? LOGIN_REQUIRED : 'could not reach the server';
+        catch (_a) {
+            return 'could not reach the server';
         }
-        if (res.status === 401)
-            return LOGIN_REQUIRED;
         return res.ok ? undefined : `could not send feedback (HTTP ${res.status})`;
     });
 }