@jackwener/opencli 1.7.7 → 1.7.9

package/clis/zhihu/comment.js

@@ -2,13 +2,12 @@ import { CliError, CommandExecutionError } from '@jackwener/opencli/errors';
 import { cli, Strategy } from '@jackwener/opencli/registry';
 import { assertAllowedKinds, parseTarget } from './target.js';
 import { buildResultRow, requireExecute, resolveCurrentUserIdentity, resolvePayload } from './write-shared.js';
-const COMMENT_AUTHOR_SCOPE_SELECTOR = '.CommentItemV2-head, .CommentItem-head, .CommentItemV2-meta, .CommentItem-meta, .CommentItemV2-metaSibling, [data-comment-author], [itemprop="author"]';
 cli({
     site: 'zhihu',
     name: 'comment',
     description: 'Create a top-level comment on a Zhihu answer or article',
     domain: 'zhihu.com',
-    strategy: Strategy.UI,
+    strategy: Strategy.COOKIE,
     browser: true,
     args: [
         { name: 'target', positional: true, required: true, help: 'Zhihu target URL or typed target' },
@@ -16,7 +15,7 @@ cli({
         { name: 'file', help: 'Comment text file path' },
         { name: 'execute', type: 'boolean', help: 'Actually perform the write action' },
     ],
-    columns: ['status', 'outcome', 'message', 'target_type', 'target', 'author_identity', 'created_url', 'created_proof'],
+    columns: ['status', 'outcome', 'message', 'target_type', 'target', 'author_identity', 'created_url'],
     func: async (page, kwargs) => {
         if (!page)
             throw new CommandExecutionError('Browser session required for zhihu comment');
@@ -25,311 +24,31 @@ cli({
         const target = assertAllowedKinds('comment', parseTarget(rawTarget));
         const payload = await resolvePayload(kwargs);
         await page.goto(target.url);
+        await page.wait(3);
         const authorIdentity = await resolveCurrentUserIdentity(page);
-        const entryPath = await page.evaluate(`(() => {
-      const targetKind = ${JSON.stringify(target.kind)};
-      const targetQuestionId = ${JSON.stringify(target.kind === 'answer' ? target.questionId : null)};
-      const targetAnswerId = ${JSON.stringify(target.kind === 'answer' ? target.id : null)};
-      const restoredDraft = !!document.querySelector('[contenteditable="true"][data-draft-restored], textarea[data-draft-restored]');
-      let scope = document;
-      if (targetKind === 'answer') {
-        const block = Array.from(document.querySelectorAll('article, .AnswerItem, [data-zop-question-answer]')).find((node) => {
-          const dataAnswerId = node.getAttribute('data-answerid') || node.getAttribute('data-zop-question-answer') || '';
-          if (dataAnswerId && dataAnswerId.includes(targetAnswerId)) return true;
-          return Array.from(node.querySelectorAll('a[href*="/answer/"]')).some((link) => {
-            const href = link.getAttribute('href') || '';
-            return href.includes('/question/' + targetQuestionId + '/answer/' + targetAnswerId);
-          });
-        });
-        if (!block) return { entryPathSafe: false, wrongAnswer: true };
-        scope = block;
-      } else {
-        scope =
-          document.querySelector('article')
-          || document.querySelector('.Post-Main')
-          || document.querySelector('[itemprop="articleBody"]')
-          || document;
-      }
-      const topLevelCandidates = Array.from(scope.querySelectorAll('[contenteditable="true"], textarea')).map((editor) => {
-        const container = editor.closest('form, .CommentEditor, .CommentForm, .CommentsV2-footer, [data-comment-editor]') || editor.parentElement;
-        const replyHint = editor.getAttribute('data-reply-to') || '';
-        const text = 'value' in editor ? editor.value || '' : (editor.textContent || '');
-        const nestedReply = Boolean(container?.closest('[data-comment-id], .CommentItem'));
-        return { editor, container, replyHint, text, nestedReply };
-      }).filter((candidate) => candidate.container && !candidate.nestedReply);
-      return {
-        entryPathSafe: topLevelCandidates.length === 1
-          && !restoredDraft
-          && !topLevelCandidates[0].replyHint
-          && !String(topLevelCandidates[0].text || '').trim(),
-        wrongAnswer: false,
-      };
-    })()`);
-        if (entryPath.wrongAnswer) {
-            throw new CliError('TARGET_NOT_FOUND', 'Resolved answer target no longer matches the requested answer:<questionId>:<answerId>');
-        }
-        if (!entryPath.entryPathSafe) {
-            throw new CliError('ACTION_NOT_AVAILABLE', 'Comment entry path was not proven side-effect free');
-        }
-        const beforeSubmitSnapshot = await page.evaluate(`(() => {
-      const normalize = (value) => value.replace(/\\s+/g, ' ').trim();
-      const targetKind = ${JSON.stringify(target.kind)};
-      const targetQuestionId = ${JSON.stringify(target.kind === 'answer' ? target.questionId : null)};
-      const targetAnswerId = ${JSON.stringify(target.kind === 'answer' ? target.id : null)};
-      let scope = document;
-      if (targetKind === 'answer') {
-        const block = Array.from(document.querySelectorAll('article, .AnswerItem, [data-zop-question-answer]')).find((node) => {
-          const dataAnswerId = node.getAttribute('data-answerid') || node.getAttribute('data-zop-question-answer') || '';
-          if (dataAnswerId && dataAnswerId.includes(targetAnswerId)) return true;
-          return Array.from(node.querySelectorAll('a[href*="/answer/"]')).some((link) => {
-            const href = link.getAttribute('href') || '';
-            return href.includes('/question/' + targetQuestionId + '/answer/' + targetAnswerId);
-          });
-        });
-        if (!block) return { wrongAnswer: true, rows: [], commentLinks: [] };
-        scope = block;
-      } else {
-        scope =
-          document.querySelector('article')
-          || document.querySelector('.Post-Main')
-          || document.querySelector('[itemprop="articleBody"]')
-          || document;
-      }
-      return {
-        wrongAnswer: false,
-        rows: Array.from(scope.querySelectorAll('[data-comment-id], .CommentItem')).map((node) => ({
-          id: node.getAttribute('data-comment-id') || '',
-          text: normalize(node.textContent || ''),
-        })),
-        commentLinks: Array.from(scope.querySelectorAll('a[href*="/comment/"]'))
-          .map((node) => node.getAttribute('href') || '')
-          .filter(Boolean),
-      };
-    })()`);
-        if (beforeSubmitSnapshot.wrongAnswer) {
-            throw new CliError('TARGET_NOT_FOUND', 'Resolved answer target no longer matches the requested answer:<questionId>:<answerId>');
-        }
-        const composer = await page.evaluate(`(async () => {
-      const targetKind = ${JSON.stringify(target.kind)};
-      const targetQuestionId = ${JSON.stringify(target.kind === 'answer' ? target.questionId : null)};
-      const targetAnswerId = ${JSON.stringify(target.kind === 'answer' ? target.id : null)};
-      let scope = document;
-      if (targetKind === 'answer') {
-        const block = Array.from(document.querySelectorAll('article, .AnswerItem, [data-zop-question-answer]')).find((node) => {
-          const dataAnswerId = node.getAttribute('data-answerid') || node.getAttribute('data-zop-question-answer') || '';
-          if (dataAnswerId && dataAnswerId.includes(targetAnswerId)) return true;
-          return Array.from(node.querySelectorAll('a[href*="/answer/"]')).some((link) => {
-            const href = link.getAttribute('href') || '';
-            return href.includes('/question/' + targetQuestionId + '/answer/' + targetAnswerId);
-          });
-        });
-        if (!block) return { composerState: 'wrong_answer' };
-        scope = block;
-      } else {
-        scope =
-          document.querySelector('article')
-          || document.querySelector('.Post-Main')
-          || document.querySelector('[itemprop="articleBody"]')
-          || document;
-      }
-      const topLevelCandidates = Array.from(scope.querySelectorAll('[contenteditable="true"], textarea')).map((editor) => {
-        const container = editor.closest('form, .CommentEditor, .CommentForm, .CommentsV2-footer, [data-comment-editor]') || editor.parentElement;
-        const replyHint = editor.getAttribute('data-reply-to') || '';
-        const text = 'value' in editor ? editor.value || '' : (editor.textContent || '');
-        const nestedReply = Boolean(container?.closest('[data-comment-id], .CommentItem'));
-        return { editor, container, replyHint, text, nestedReply };
-      }).filter((candidate) => candidate.container && !candidate.nestedReply);
-      if (topLevelCandidates.length !== 1) return { composerState: 'unsafe' };
-      return {
-        composerState: !topLevelCandidates[0].replyHint && !topLevelCandidates[0].text.trim() ? 'fresh_top_level' : 'unsafe',
-      };
-    })()`);
-        if (composer.composerState === 'wrong_answer') {
-            throw new CliError('TARGET_NOT_FOUND', 'Resolved answer target no longer matches the requested answer:<questionId>:<answerId>');
-        }
-        if (composer.composerState !== 'fresh_top_level') {
-            throw new CliError('ACTION_NOT_AVAILABLE', 'Comment composer was not a fresh top-level composer');
-        }
-        const editorCheck = await page.evaluate(`(async () => {
-      const textToInsert = ${JSON.stringify(payload)};
-      const targetKind = ${JSON.stringify(target.kind)};
-      const targetQuestionId = ${JSON.stringify(target.kind === 'answer' ? target.questionId : null)};
-      const targetAnswerId = ${JSON.stringify(target.kind === 'answer' ? target.id : null)};
-      let scope = document;
-      if (targetKind === 'answer') {
-        const block = Array.from(document.querySelectorAll('article, .AnswerItem, [data-zop-question-answer]')).find((node) => {
-          const dataAnswerId = node.getAttribute('data-answerid') || node.getAttribute('data-zop-question-answer') || '';
-          if (dataAnswerId && dataAnswerId.includes(targetAnswerId)) return true;
-          return Array.from(node.querySelectorAll('a[href*="/answer/"]')).some((link) => {
-            const href = link.getAttribute('href') || '';
-            return href.includes('/question/' + targetQuestionId + '/answer/' + targetAnswerId);
-          });
-        });
-        if (!block) return { editorContent: '', mode: 'wrong_answer' };
-        scope = block;
-      } else {
-        scope =
-          document.querySelector('article')
-          || document.querySelector('.Post-Main')
-          || document.querySelector('[itemprop="articleBody"]')
-          || document;
-      }
-      const topLevelCandidates = Array.from(scope.querySelectorAll('[contenteditable="true"], textarea')).map((editor) => {
-        const container = editor.closest('form, .CommentEditor, .CommentForm, .CommentsV2-footer, [data-comment-editor]') || editor.parentElement;
-        const nestedReply = Boolean(container?.closest('[data-comment-id], .CommentItem'));
-        return { editor, container, nestedReply };
-      }).filter((candidate) => candidate.container && !candidate.nestedReply);
-      if (topLevelCandidates.length !== 1) return { editorContent: '', mode: 'missing' };
-      const { editor } = topLevelCandidates[0];
-      editor.focus();
-      if ('value' in editor) {
-        editor.value = '';
-        editor.dispatchEvent(new Event('input', { bubbles: true }));
-        editor.value = textToInsert;
-        editor.dispatchEvent(new Event('input', { bubbles: true }));
-      } else {
-        editor.textContent = '';
-        document.execCommand('insertText', false, textToInsert);
-        editor.dispatchEvent(new InputEvent('input', { bubbles: true, data: textToInsert, inputType: 'insertText' }));
-      }
-      await new Promise((resolve) => setTimeout(resolve, 200));
-      const content = 'value' in editor ? editor.value : (editor.textContent || '');
-      const replyHint = editor.getAttribute('data-reply-to') || '';
-      return { editorContent: content, mode: replyHint ? 'reply' : 'top_level' };
-    })()`);
-        if (editorCheck.mode === 'wrong_answer') {
-            throw new CliError('TARGET_NOT_FOUND', 'Resolved answer target no longer matches the requested answer:<questionId>:<answerId>');
-        }
-        if (editorCheck.mode !== 'top_level' || editorCheck.editorContent !== payload) {
-            throw new CliError('OUTCOME_UNKNOWN', 'Comment editor content did not exactly match the requested payload before submit');
-        }
-        const proof = await page.evaluate(`(async () => {
-      const normalize = (value) => value.replace(/\\s+/g, ' ').trim();
-      const commentAuthorScopeSelector = ${JSON.stringify(COMMENT_AUTHOR_SCOPE_SELECTOR)};
-      const readCommentAuthorSlug = (node) => {
-        const authorScopes = Array.from(node.querySelectorAll(commentAuthorScopeSelector));
-        const slugs = Array.from(new Set(authorScopes
-          .flatMap((scope) => Array.from(scope.querySelectorAll('a[href^="/people/"]')))
-          .map((link) => (link.getAttribute('href') || '').match(/^\\/people\\/([A-Za-z0-9_-]+)/)?.[1] || null)
-          .filter(Boolean)));
-        return slugs.length === 1 ? slugs[0] : null;
-      };
-      const targetKind = ${JSON.stringify(target.kind)};
-      const targetQuestionId = ${JSON.stringify(target.kind === 'answer' ? target.questionId : null)};
-      const targetAnswerId = ${JSON.stringify(target.kind === 'answer' ? target.id : null)};
-      let scope = document;
-      if (targetKind === 'answer') {
-        const block = Array.from(document.querySelectorAll('article, .AnswerItem, [data-zop-question-answer]')).find((node) => {
-          const dataAnswerId = node.getAttribute('data-answerid') || node.getAttribute('data-zop-question-answer') || '';
-          if (dataAnswerId && dataAnswerId.includes(targetAnswerId)) return true;
-          return Array.from(node.querySelectorAll('a[href*="/answer/"]')).some((link) => {
-            const href = link.getAttribute('href') || '';
-            return href.includes('/question/' + targetQuestionId + '/answer/' + targetAnswerId);
-          });
-        });
-        if (!block) return { proofType: 'wrong_answer' };
-        scope = block;
-      } else {
-        scope =
-          document.querySelector('article')
-          || document.querySelector('.Post-Main')
-          || document.querySelector('[itemprop="articleBody"]')
-          || document;
-      }
-
-      const topLevelCandidates = Array.from(scope.querySelectorAll('[contenteditable="true"], textarea')).map((editor) => {
-        const container = editor.closest('form, [role="dialog"], .CommentEditor, .CommentForm, .CommentsV2-footer, [data-comment-editor]') || editor.parentElement;
-        const nestedReply = Boolean(container?.closest('[data-comment-id], .CommentItem'));
-        return { editor, container, nestedReply };
-      }).filter((candidate) => candidate.container && !candidate.nestedReply);
-      if (topLevelCandidates.length !== 1) return { proofType: 'unknown' };
-      const submitScope = topLevelCandidates[0].container || scope;
-      const submit = Array.from(submitScope.querySelectorAll('button')).find((node) => /发布|评论|发送/.test(node.textContent || ''));
-      submit && submit.click();
-      await new Promise((resolve) => setTimeout(resolve, 1200));
-      const createdLink = Array.from(scope.querySelectorAll('a[href*="/comment/"]')).find((node) => {
-        const href = node.getAttribute('href') || '';
-        return href.includes('/comment/') && !${JSON.stringify(beforeSubmitSnapshot.commentLinks ?? [])}.includes(href);
-      });
-
-      if (createdLink) {
-        const card = createdLink.closest('[data-comment-id], .CommentItem, li');
-        const authorSlug = card ? readCommentAuthorSlug(card) : null;
-        const contentNode =
-          card?.querySelector('[data-comment-content], .RichContent-inner, .CommentItemV2-content, .CommentContent')
-          || card;
-        const text = normalize(contentNode?.textContent || '');
-        const nestedReply = Boolean(card?.closest('ul ul, ol ol, li li') || card?.parentElement?.closest('[data-comment-id], .CommentItem'));
-        return {
-          proofType: 'stable_url',
-          createdUrl: new URL(createdLink.getAttribute('href') || '', location.origin).href,
-          commentScope: nestedReply ? 'nested_reply' : 'top_level_only',
-          authorIdentity: authorSlug,
-          targetMatches: text === normalize(${JSON.stringify(payload)}),
-        };
-      }
-
-      const currentUserSlug = ${JSON.stringify(authorIdentity)};
-      const beforeIds = new Set(${JSON.stringify((beforeSubmitSnapshot.rows ?? []).map((row) => row.id).filter(Boolean))});
-      const beforeTexts = new Set(${JSON.stringify((beforeSubmitSnapshot.rows ?? []).map((row) => row.text).filter(Boolean))});
-      const normalizedPayload = normalize(${JSON.stringify(payload)});
-      const after = Array.from(scope.querySelectorAll('[data-comment-id], .CommentItem')).map((node) => {
-        return {
-          id: node.getAttribute('data-comment-id') || '',
-          text: normalize(node.textContent || ''),
-          authorSlug: readCommentAuthorSlug(node),
-          topLevel: !node.closest('ul ul, ol ol, li li') && !node.parentElement?.closest('[data-comment-id], .CommentItem'),
-        };
-      });
-
-      const matching = after.filter((row) =>
-        !beforeIds.has(row.id)
-        && row.authorSlug === currentUserSlug
-        && row.topLevel
-        && row.text === normalizedPayload
-        && !beforeTexts.has(row.text)
-      );
-
-      return matching.length === 1
-        ? {
-            proofType: 'fallback',
-            createdProof: {
-              proof_type: 'comment_fallback',
-              author_scope: 'current_user',
-              target_scope: 'requested_target',
-              comment_scope: 'top_level_only',
-              content_match: 'exact_normalized',
-              observed_after_submit: true,
-              present_in_pre_submit_snapshot: false,
-              new_matching_entries: 1,
-              post_submit_matching_entries: after.filter((row) =>
-                row.authorSlug === currentUserSlug && row.topLevel && row.text === normalizedPayload
-              ).length,
-              snapshot_scope: ${JSON.stringify(target.kind === 'answer'
-            ? 'stabilized_expanded_target_answer_comment_list'
-            : 'stabilized_expanded_target_article_comment_list')},
-            },
-          }
-        : { proofType: 'unknown' };
-    })()`);
-        if (proof.proofType === 'wrong_answer') {
-            throw new CliError('TARGET_NOT_FOUND', 'Resolved answer target no longer matches the requested answer:<questionId>:<answerId>');
-        }
-        if (proof.proofType === 'fallback') {
-            return buildResultRow(`Commented on ${target.kind}`, target.kind, rawTarget, 'created', {
-                author_identity: authorIdentity,
-                created_proof: proof.createdProof,
+        const apiResult = await page.evaluate(`(async () => {
+            var targetKind = ${JSON.stringify(target.kind)};
+            var targetId = ${JSON.stringify(target.id)};
+            var content = ${JSON.stringify(payload)};
+            var resourceType = targetKind === 'answer' ? 'answers' : 'articles';
+            var url = 'https://www.zhihu.com/api/v4/' + resourceType + '/' + targetId + '/comments';
+            var resp = await fetch(url, {
+                method: 'POST',
+                credentials: 'include',
+                headers: { 'Content-Type': 'application/json' },
+                body: JSON.stringify({ content: content }),
             });
-        }
-        if (proof.proofType !== 'stable_url') {
-            throw new CliError('OUTCOME_UNKNOWN', 'Comment submit was dispatched, but the created object could not be proven safely');
-        }
-        if (proof.commentScope !== 'top_level_only' || proof.authorIdentity !== authorIdentity || !proof.targetMatches) {
-            throw new CliError('OUTCOME_UNKNOWN', 'Stable comment URL was found, but authorship or top-level scope could not be proven safely');
-        }
-        return buildResultRow(`Commented on ${target.kind}`, target.kind, rawTarget, 'created', {
+            var data = await resp.json();
+            if (!resp.ok) return { ok: false, status: resp.status, message: data.error ? data.error.message : 'unknown error' };
+            if (!data || !data.id) return { ok: false, status: resp.status, message: 'Comment API response did not include a created comment id' };
+            return { ok: true, id: data.id, url: data.url };
+        })()`);
+        if (!apiResult?.ok) {
+            throw new CliError('COMMAND_EXEC', apiResult?.message || 'Failed to create comment');
+        }
+        return buildResultRow(`Commented on ${target.kind} ${target.id}`, target.kind, rawTarget, 'created', {
             author_identity: authorIdentity,
-            created_url: proof.createdUrl,
+            created_url: apiResult.url || '',
         });
     },
 });

package/clis/zhihu/comment.test.js

@@ -2,53 +2,49 @@ import { describe, expect, it, vi } from 'vitest';
 import { getRegistry } from '@jackwener/opencli/registry';
 import './comment.js';
 describe('zhihu comment', () => {
-    it('rejects composer paths that are not proven side-effect free', async () => {
+    it('registers as a cookie browser command', () => {
+        const cmd = getRegistry().get('zhihu/comment');
+        expect(cmd).toBeDefined();
+        expect(cmd.strategy).toBe('cookie');
+        expect(cmd.browser).toBe(true);
+    });
+    it('creates a comment via API and returns result', async () => {
         const cmd = getRegistry().get('zhihu/comment');
-        expect(cmd?.func).toBeTypeOf('function');
         const page = {
             goto: vi.fn().mockResolvedValue(undefined),
+            wait: vi.fn().mockResolvedValue(undefined),
             evaluate: vi.fn()
                 .mockResolvedValueOnce({ slug: 'alice' })
-                .mockResolvedValueOnce({ entryPathSafe: false })
-                .mockResolvedValueOnce({ wrongAnswer: false, rows: [], commentLinks: [] }),
+                .mockResolvedValueOnce({ ok: true, id: 99, url: 'https://www.zhihu.com/api/v4/comments/99' }),
         };
-        await expect(cmd.func(page, { target: 'answer:1:2', text: 'hello', execute: true })).rejects.toMatchObject({ code: 'ACTION_NOT_AVAILABLE' });
+        const rows = await cmd.func(page, { target: 'answer:1:2', text: 'hello', execute: true });
+        expect(rows).toEqual([
+            expect.objectContaining({ outcome: 'created', author_identity: 'alice' }),
+        ]);
     });
-    it('requires exact editor replacement before accepting fallback proof', async () => {
+    it('throws on API error', async () => {
         const cmd = getRegistry().get('zhihu/comment');
         const page = {
             goto: vi.fn().mockResolvedValue(undefined),
+            wait: vi.fn().mockResolvedValue(undefined),
             evaluate: vi.fn()
                 .mockResolvedValueOnce({ slug: 'alice' })
-                .mockResolvedValueOnce({ entryPathSafe: true })
-                .mockResolvedValueOnce({ wrongAnswer: false, rows: [], commentLinks: [] })
-                .mockResolvedValueOnce({ composerState: 'fresh_top_level' })
-                .mockResolvedValueOnce({ editorContent: 'hello', mode: 'top_level' })
-                .mockResolvedValueOnce({
-                proofType: 'fallback',
-                createdProof: {
-                    proof_type: 'comment_fallback',
-                    author_scope: 'current_user',
-                    target_scope: 'requested_target',
-                    comment_scope: 'top_level_only',
-                    content_match: 'exact_normalized',
-                    observed_after_submit: true,
-                    present_in_pre_submit_snapshot: false,
-                    new_matching_entries: 1,
-                    post_submit_matching_entries: 1,
-                    snapshot_scope: 'stabilized_expanded_target_comment_list',
-                },
-            }),
+                .mockResolvedValueOnce({ ok: false, status: 403, message: 'forbidden' }),
         };
-        await expect(cmd.func(page, { target: 'answer:1:2', text: 'hello', execute: true })).resolves.toEqual([
-            expect.objectContaining({ outcome: 'created', author_identity: 'alice', created_proof: expect.any(Object) }),
-        ]);
-        expect(page.evaluate.mock.calls[1][0]).toContain('topLevelCandidates.length === 1');
-        expect(page.evaluate.mock.calls[1][0]).not.toContain('commentTrigger');
-        expect(page.evaluate.mock.calls[2][0]).toContain("node.getAttribute('data-answerid')");
-        expect(page.evaluate.mock.calls[2][0]).toContain("node.getAttribute('data-zop-question-answer')");
-        expect(page.evaluate.mock.calls[5][0]).toContain('const readCommentAuthorSlug = (node) =>');
-        expect(page.evaluate.mock.calls[5][0]).toContain('const commentAuthorScopeSelector = ".CommentItemV2-head, .CommentItem-head, .CommentItemV2-meta, .CommentItem-meta, .CommentItemV2-metaSibling, [data-comment-author], [itemprop=\\"author\\"]"');
-        expect(page.evaluate.mock.calls[5][0]).not.toContain("card?.querySelector('a[href^=\"/people/\"]')");
+        await expect(cmd.func(page, { target: 'answer:1:2', text: 'hello', execute: true }))
+            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
+    });
+    it('requires the comment API response to include the created id', async () => {
+        const cmd = getRegistry().get('zhihu/comment');
+        const page = {
+            goto: vi.fn().mockResolvedValue(undefined),
+            wait: vi.fn().mockResolvedValue(undefined),
+            evaluate: vi.fn()
+                .mockResolvedValueOnce({ slug: 'alice' })
+                .mockResolvedValueOnce({ ok: false, status: 200, message: 'Comment API response did not include a created comment id' }),
+        };
+        await expect(cmd.func(page, { target: 'answer:1:2', text: 'hello', execute: true }))
+            .rejects.toMatchObject({ code: 'COMMAND_EXEC' });
+        expect(page.evaluate.mock.calls[1][0]).toContain('Comment API response did not include a created comment id');
     });
 });