@jackwener/opencli 1.7.22 → 1.8.1

package/clis/gitee/search.js

@@ -123,8 +123,8 @@ cli({
             rows.push({
                 rank: rows.length + 1,
                 name,
-                language: normalizeText(getFirstText(fields.langs)) || '-',
-                description: normalizeText(getFirstText(fields.description)) || '-',
+                language: normalizeText(getFirstText(fields.langs)) || '',
+                description: normalizeText(getFirstText(fields.description)) || '',
                 stars: normalizeStars(fields['count.star']),
                 url: repoUrl,
             });

package/clis/gitee/search.test.js

@@ -0,0 +1,65 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { getRegistry } from '@jackwener/opencli/registry';
+import './search.js';
+
+function mockGiteeResponse(hits) {
+    return {
+        ok: true,
+        json: () => Promise.resolve({ hits: { hits } }),
+    };
+}
+
+function makePage() {
+    return {
+        goto: vi.fn().mockResolvedValue(undefined),
+        wait: vi.fn().mockResolvedValue(undefined),
+    };
+}
+
+describe('gitee search', () => {
+    beforeEach(() => {
+        vi.restoreAllMocks();
+    });
+    afterEach(() => {
+        vi.unstubAllGlobals();
+    });
+
+    it('emits empty-string for missing language / description instead of a sentinel', async () => {
+        const cmd = getRegistry().get('gitee/search');
+        expect(cmd?.func).toBeTypeOf('function');
+        const fetchMock = vi.fn().mockResolvedValue(mockGiteeResponse([
+            {
+                fields: {
+                    title: 'someuser/no-meta-repo',
+                    url: 'https://gitee.com/someuser/no-meta-repo',
+                },
+            },
+        ]));
+        vi.stubGlobal('fetch', fetchMock);
+        const rows = await cmd.func(makePage(), { keyword: 'test', limit: 10 });
+        expect(rows).toHaveLength(1);
+        expect(rows[0].name).toBe('someuser/no-meta-repo');
+        expect(rows[0].language).toBe('');
+        expect(rows[0].description).toBe('');
+    });
+
+    it('passes through populated language / description verbatim', async () => {
+        const cmd = getRegistry().get('gitee/search');
+        const fetchMock = vi.fn().mockResolvedValue(mockGiteeResponse([
+            {
+                fields: {
+                    title: 'org/repo-a',
+                    url: 'https://gitee.com/org/repo-a',
+                    langs: 'TypeScript',
+                    description: 'A test repo',
+                    'count.star': '42',
+                },
+            },
+        ]));
+        vi.stubGlobal('fetch', fetchMock);
+        const rows = await cmd.func(makePage(), { keyword: 'test', limit: 10 });
+        expect(rows[0].language).toBe('TypeScript');
+        expect(rows[0].description).toBe('A test repo');
+        expect(rows[0].stars).toBe('42');
+    });
+});

package/clis/jike/post.js

@@ -1,4 +1,5 @@
 import { cli } from '@jackwener/opencli/registry';
+import { CommandExecutionError } from '@jackwener/opencli/errors';
 cli({
     site: 'jike',
     name: 'post',
@@ -16,16 +17,16 @@ cli({
         },
     ],
     columns: ['type', 'author', 'content', 'likes', 'time'],
-    pipeline: [
-        { navigate: 'https://m.okjike.com/originalPosts/${{ args.id }}' },
-        { evaluate: `(() => {
+    func: async (page, args) => {
+        await page.goto(`https://m.okjike.com/originalPosts/${args.id}`);
+        const data = await page.evaluate(`(() => {
+  const el = document.querySelector('script[type="application/json"]');
+  if (!el) return { ok: false, reason: 'missing-data-script' };
   try {
-    const el = document.querySelector('script[type="application/json"]');
-    if (!el) return [];
-    const data = JSON.parse(el.textContent);
+    const data = JSON.parse(el.textContent || '{}');
     const pageProps = data?.props?.pageProps || {};
     const post = pageProps.post || {};
-    const comments = pageProps.comments || [];
+    const comments = Array.isArray(pageProps.comments) ? pageProps.comments : [];
 
     const result = [{
       type: 'post',
@@ -47,16 +48,25 @@ cli({
 
     return result;
   } catch (e) {
-    return [];
+    return { ok: false, reason: 'parse-error', message: e?.message || String(e) };
   }
 })()
-` },
-        { map: {
-                type: '${{ item.type }}',
-                author: '${{ item.author }}',
-                content: '${{ item.content }}',
-                likes: '${{ item.likes }}',
-                time: '${{ item.time }}',
-            } },
-    ],
+`);
+        if (Array.isArray(data)) {
+            return data.map((item) => ({
+                type: item.type ?? '',
+                author: item.author ?? '',
+                content: item.content ?? '',
+                likes: item.likes ?? 0,
+                time: item.time ?? '',
+            }));
+        }
+        if (data?.reason === 'missing-data-script') {
+            throw new CommandExecutionError('Jike post page did not expose the expected data script');
+        }
+        if (data?.reason === 'parse-error') {
+            throw new CommandExecutionError(`Failed to parse Jike post data: ${data.message || 'unknown error'}`);
+        }
+        throw new CommandExecutionError('Jike post returned an unreadable payload');
+    },
 });

package/clis/jike/read.test.js

@@ -0,0 +1,86 @@
+import { describe, expect, it, vi } from 'vitest';
+import { getRegistry } from '@jackwener/opencli/registry';
+import { CommandExecutionError, EmptyResultError } from '@jackwener/opencli/errors';
+import './post.js';
+import './topic.js';
+import './user.js';
+
+function makePage(evaluateResult) {
+    return {
+        goto: vi.fn().mockResolvedValue(undefined),
+        evaluate: vi.fn().mockResolvedValue(evaluateResult),
+    };
+}
+
+describe('jike read commands', () => {
+    it('maps post rows from the browser-side extractor', async () => {
+        const command = getRegistry().get('jike/post');
+        const page = makePage([
+            { type: 'post', author: 'alice', content: 'hello', likes: 3, time: '2026-05-16' },
+            { type: 'comment', author: 'bob', content: 'nice', likes: 1, time: '2026-05-16' },
+        ]);
+
+        await expect(command.func(page, { id: 'post-1' })).resolves.toEqual([
+            { type: 'post', author: 'alice', content: 'hello', likes: 3, time: '2026-05-16' },
+            { type: 'comment', author: 'bob', content: 'nice', likes: 1, time: '2026-05-16' },
+        ]);
+        expect(page.goto).toHaveBeenCalledWith('https://m.okjike.com/originalPosts/post-1');
+    });
+
+    it('maps topic rows and applies limit on the Node side', async () => {
+        const command = getRegistry().get('jike/topic');
+        const page = makePage([
+            { id: 'a', content: 'one', author: 'alice', likes: 1, comments: 2, time: 't1' },
+            { id: 'b', content: 'two', author: 'bob', likes: 3, comments: 4, time: 't2' },
+        ]);
+
+        await expect(command.func(page, { id: 'topic-1', limit: 1 })).resolves.toEqual([
+            {
+                content: 'one',
+                author: 'alice',
+                likes: 1,
+                comments: 2,
+                time: 't1',
+                url: 'https://web.okjike.com/originalPost/a',
+            },
+        ]);
+        expect(page.goto).toHaveBeenCalledWith('https://m.okjike.com/topics/topic-1');
+    });
+
+    it('maps user rows and applies limit on the Node side', async () => {
+        const command = getRegistry().get('jike/user');
+        const page = makePage([
+            { id: 'a', content: 'one', type: 'post', likes: 1, comments: 2, time: 't1' },
+            { id: 'b', content: 'two', type: 'repost', likes: 3, comments: 4, time: 't2' },
+        ]);
+
+        await expect(command.func(page, { username: 'alice', limit: 1 })).resolves.toEqual([
+            {
+                id: 'a',
+                content: 'one',
+                type: 'post',
+                likes: 1,
+                comments: 2,
+                time: 't1',
+                url: 'https://web.okjike.com/originalPost/a',
+            },
+        ]);
+        expect(page.goto).toHaveBeenCalledWith('https://m.okjike.com/users/alice');
+    });
+
+    it('throws CommandExecutionError for malformed browser-side payloads', async () => {
+        await expect(getRegistry().get('jike/post').func(makePage({ reason: 'missing-data-script' }), { id: 'post-1' }))
+            .rejects.toBeInstanceOf(CommandExecutionError);
+        await expect(getRegistry().get('jike/topic').func(makePage({ reason: 'parse-error', message: 'bad json' }), { id: 'topic-1' }))
+            .rejects.toBeInstanceOf(CommandExecutionError);
+        await expect(getRegistry().get('jike/user').func(makePage(null), { username: 'alice' }))
+            .rejects.toBeInstanceOf(CommandExecutionError);
+    });
+
+    it('throws EmptyResultError when topic or user extractors return no posts', async () => {
+        await expect(getRegistry().get('jike/topic').func(makePage([]), { id: 'topic-1' }))
+            .rejects.toBeInstanceOf(EmptyResultError);
+        await expect(getRegistry().get('jike/user').func(makePage([]), { username: 'alice' }))
+            .rejects.toBeInstanceOf(EmptyResultError);
+    });
+});

package/clis/jike/topic.js

@@ -1,4 +1,5 @@
 import { cli } from '@jackwener/opencli/registry';
+import { CommandExecutionError, EmptyResultError } from '@jackwener/opencli/errors';
 cli({
     site: 'jike',
     name: 'topic',
@@ -17,15 +18,16 @@ cli({
         { name: 'limit', type: 'int', default: 20, help: 'Number of posts' },
     ],
     columns: ['content', 'author', 'likes', 'comments', 'time', 'url'],
-    pipeline: [
-        { navigate: 'https://m.okjike.com/topics/${{ args.id }}' },
-        { evaluate: `(() => {
+    func: async (page, args) => {
+        await page.goto(`https://m.okjike.com/topics/${args.id}`);
+        const limit = Number(args.limit) || 20;
+        const data = await page.evaluate(`(() => {
+  const el = document.querySelector('script[type="application/json"]');
+  if (!el) return { ok: false, reason: 'missing-data-script' };
   try {
-    const el = document.querySelector('script[type="application/json"]');
-    if (!el) return [];
-    const data = JSON.parse(el.textContent);
+    const data = JSON.parse(el.textContent || '{}');
     const pageProps = data?.props?.pageProps || {};
-    const posts = pageProps.posts || [];
+    const posts = Array.isArray(pageProps.posts) ? pageProps.posts : [];
     return posts.map(p => ({
       content: (p.content || '').replace(/\\n/g, ' ').slice(0, 80),
       author: p.user?.screenName || '',
@@ -35,18 +37,29 @@ cli({
       id: p.id || '',
     }));
   } catch (e) {
-    return [];
+    return { ok: false, reason: 'parse-error', message: e?.message || String(e) };
   }
 })()
-` },
-        { map: {
-                content: '${{ item.content }}',
-                author: '${{ item.author }}',
-                likes: '${{ item.likes }}',
-                comments: '${{ item.comments }}',
-                time: '${{ item.time }}',
-                url: 'https://web.okjike.com/originalPost/${{ item.id }}',
-            } },
-        { limit: '${{ args.limit }}' },
-    ],
+`);
+        if (Array.isArray(data)) {
+            if (data.length === 0) {
+                throw new EmptyResultError('jike topic', `No posts were returned for topic ${args.id}. Confirm the topic ID and login state.`);
+            }
+            return data.slice(0, limit).map((item) => ({
+                content: item.content ?? '',
+                author: item.author ?? '',
+                likes: item.likes ?? 0,
+                comments: item.comments ?? 0,
+                time: item.time ?? '',
+                url: `https://web.okjike.com/originalPost/${item.id ?? ''}`,
+            }));
+        }
+        if (data?.reason === 'missing-data-script') {
+            throw new CommandExecutionError('Jike topic page did not expose the expected data script');
+        }
+        if (data?.reason === 'parse-error') {
+            throw new CommandExecutionError(`Failed to parse Jike topic data: ${data.message || 'unknown error'}`);
+        }
+        throw new CommandExecutionError('Jike topic returned an unreadable payload');
+    },
 });

package/clis/jike/user.js

@@ -1,4 +1,5 @@
 import { cli } from '@jackwener/opencli/registry';
+import { CommandExecutionError, EmptyResultError } from '@jackwener/opencli/errors';
 cli({
     site: 'jike',
     name: 'user',
@@ -17,14 +18,15 @@ cli({
         { name: 'limit', type: 'int', default: 20, help: 'Number of posts' },
     ],
     columns: ['id', 'content', 'type', 'likes', 'comments', 'time', 'url'],
-    pipeline: [
-        { navigate: 'https://m.okjike.com/users/${{ args.username }}' },
-        { evaluate: `(() => {
+    func: async (page, args) => {
+        await page.goto(`https://m.okjike.com/users/${args.username}`);
+        const limit = Number(args.limit) || 20;
+        const data = await page.evaluate(`(() => {
+  const el = document.querySelector('script[type="application/json"]');
+  if (!el) return { ok: false, reason: 'missing-data-script' };
   try {
-    const el = document.querySelector('script[type="application/json"]');
-    if (!el) return [];
-    const data = JSON.parse(el.textContent);
-    const posts = data?.props?.pageProps?.posts || [];
+    const data = JSON.parse(el.textContent || '{}');
+    const posts = Array.isArray(data?.props?.pageProps?.posts) ? data.props.pageProps.posts : [];
     return posts.map(p => ({
       content: (p.content || '').replace(/\\n/g, ' ').slice(0, 80),
       type: p.type === 'ORIGINAL_POST' ? 'post' : p.type === 'REPOST' ? 'repost' : p.type || '',
@@ -34,19 +36,30 @@ cli({
       id: p.id || '',
     }));
   } catch (e) {
-    return [];
+    return { ok: false, reason: 'parse-error', message: e?.message || String(e) };
   }
 })()
-` },
-        { map: {
-                id: '${{ item.id }}',
-                content: '${{ item.content }}',
-                type: '${{ item.type }}',
-                likes: '${{ item.likes }}',
-                comments: '${{ item.comments }}',
-                time: '${{ item.time }}',
-                url: 'https://web.okjike.com/originalPost/${{ item.id }}',
-            } },
-        { limit: '${{ args.limit }}' },
-    ],
+`);
+        if (Array.isArray(data)) {
+            if (data.length === 0) {
+                throw new EmptyResultError('jike user', `No posts were returned for user ${args.username}. Confirm the username and login state.`);
+            }
+            return data.slice(0, limit).map((item) => ({
+                id: item.id ?? '',
+                content: item.content ?? '',
+                type: item.type ?? '',
+                likes: item.likes ?? 0,
+                comments: item.comments ?? 0,
+                time: item.time ?? '',
+                url: `https://web.okjike.com/originalPost/${item.id ?? ''}`,
+            }));
+        }
+        if (data?.reason === 'missing-data-script') {
+            throw new CommandExecutionError('Jike user page did not expose the expected data script');
+        }
+        if (data?.reason === 'parse-error') {
+            throw new CommandExecutionError(`Failed to parse Jike user data: ${data.message || 'unknown error'}`);
+        }
+        throw new CommandExecutionError('Jike user returned an unreadable payload');
+    },
 });

package/clis/jira/attachments.js

@@ -0,0 +1,28 @@
+import { cli, Strategy } from '@jackwener/opencli/registry';
+import { fetchIssue, jiraConfig, jiraRowsOrEmpty, normalizeAttachment, requireIssueKey } from './shared.js';
+import { requirePayloadArray } from '../_atlassian/shared.js';
+
+cli({
+    site: 'jira',
+    name: 'attachments',
+    access: 'read',
+    description: 'Jira issue attachment metadata',
+    domain: 'atlassian.net',
+    strategy: Strategy.PUBLIC,
+    browser: false,
+    args: [
+        { name: 'key', positional: true, required: true, help: 'Jira issue key, e.g. PROJ-123' },
+    ],
+    columns: ['id', 'filename', 'mimeType', 'size', 'url'],
+    func: async (args) => {
+        const key = requireIssueKey(args.key);
+        const config = jiraConfig();
+        const issue = await fetchIssue(config, key, ['attachment']);
+        const attachments = requirePayloadArray(issue.fields?.attachment, `jira attachments ${key}`);
+        return jiraRowsOrEmpty(
+            attachments.map(normalizeAttachment),
+            `jira attachments ${key}`,
+            `Jira issue ${key} has no attachments.`,
+        );
+    },
+});