mirror of
https://github.com/jackwener/OpenCLI.git
synced 2026-09-14 18:25:42 +08:00
80eef46b4e
* refactor: move adapters from src/clis/ to root clis/ for monorepo separation Separates CLI adapters from the core runtime to prepare for independent adapter distribution via postinstall fetch. Key changes: - Move src/clis/ → clis/ (adapters at repo root) - Change tsconfig rootDir from "src" to "." so tsc compiles both - Create root-level shim files (registry.ts, errors.ts, etc.) so adapter relative imports (../../registry.js) resolve correctly - Update build-manifest.ts, main.ts paths for new dist/src/ structure - Expand ensureUserCliCompatShims() to cover all adapter import targets (types, utils, logger, launcher, browser/*, download/*, pipeline/*) - Add scripts/fetch-adapters.js postinstall for ~/.opencli/clis/ sync - Update vitest.config.ts adapter test paths - Add package.json files field to exclude adapters from npm package Official adapter files are unconditionally overwritten on update; user-created files not in the manifest are preserved. * fix: add dist/clis/ and cli-manifest.json to npm files, harden fetch-adapters - Add dist/clis/ and dist/cli-manifest.json to package.json files field so built-in adapters and manifest ship with the npm package - Replace execSync with execFileSync to prevent command injection - Add version check to skip redundant adapter fetches - Track tmpRoot explicitly for reliable cleanup * fix: address review blockers — manifest-based updates, global-only fetch, first-run fallback 1. Manifest-based update strategy: - Read old manifest to identify previously-official files - Clean up files removed upstream (in old manifest but not new) - User-created files (never in any manifest) remain untouched 2. Only run fetch-adapters on global install (npm_config_global=true) or explicit OPENCLI_FETCH=1, preventing heavy side effects for local/dev installs 3. First-run fallback in discovery.ts: - ensureUserAdapters() checks for adapter-manifest.json - If missing and ~/.opencli/clis/ is empty, spawns fetch-adapters.js - Guarantees adapters are available even with --ignore-scripts * fix: remove OPENCLI_FETCH env var, use internal _OPENCLI_FIRST_RUN instead * feat: also support OPENCLI_FETCH=1 for explicit adapter fetch trigger * simplify: replace git clone with local copy from dist/clis/ Adapters already ship in the npm package (dist/clis/), so there's no need to clone from GitHub. Copy directly from the installed package: - Eliminates git, curl, tar dependencies - No network calls in postinstall - No timeout/offline issues - Version always matches the installed CLI - ~65 lines of clone/download code replaced by one cpSync loop
136 lines
4.4 KiB
TypeScript
136 lines
4.4 KiB
TypeScript
import { describe, expect, it } from 'vitest';
|
|
import { extractStoryMediaLinks, parseBloombergRss, renderStoryBody } from './utils.js';
|
|
|
|
describe('Bloomberg utils', () => {
|
|
it('parses Bloomberg RSS items with summary, link, and deduped media links', () => {
|
|
const xml = `<?xml version="1.0" encoding="UTF-8"?>
|
|
<rss xmlns:media="http://search.yahoo.com/mrss/">
|
|
<channel>
|
|
<item>
|
|
<title><![CDATA[Headline One]]></title>
|
|
<description><![CDATA[Summary <b>One</b> & more]]></description>
|
|
<link>https://www.bloomberg.com/news/articles/2026-03-19/example-one</link>
|
|
<media:content url="https://assets.bwbx.io/example-one.jpg" type="image/jpeg">
|
|
<media:thumbnail url="https://assets.bwbx.io/example-one.jpg" />
|
|
</media:content>
|
|
</item>
|
|
<item>
|
|
<title>Headline Two</title>
|
|
<description>Summary Two</description>
|
|
<guid isPermaLink="true">https://www.bloomberg.com/news/articles/2026-03-19/example-two</guid>
|
|
<enclosure url="https://assets.bwbx.io/example-two.png" type="image/png" />
|
|
</item>
|
|
</channel>
|
|
</rss>`;
|
|
|
|
const items = parseBloombergRss(xml);
|
|
expect(items).toHaveLength(2);
|
|
expect(items[0]).toEqual({
|
|
title: 'Headline One',
|
|
summary: 'Summary One & more',
|
|
link: 'https://www.bloomberg.com/news/articles/2026-03-19/example-one',
|
|
mediaLinks: ['https://assets.bwbx.io/example-one.jpg'],
|
|
});
|
|
expect(items[1]).toEqual({
|
|
title: 'Headline Two',
|
|
summary: 'Summary Two',
|
|
link: 'https://www.bloomberg.com/news/articles/2026-03-19/example-two',
|
|
mediaLinks: ['https://assets.bwbx.io/example-two.png'],
|
|
});
|
|
});
|
|
|
|
it('renders Bloomberg story rich-text body into readable text', () => {
|
|
const body = {
|
|
type: 'document',
|
|
content: [
|
|
{ type: 'inline-newsletter', data: { position: 'top' }, content: [] },
|
|
{
|
|
type: 'paragraph',
|
|
data: {},
|
|
content: [
|
|
{ type: 'text', value: 'Lead paragraph with ' },
|
|
{ type: 'entity', content: [{ type: 'text', value: 'linked text' }] },
|
|
{ type: 'text', value: '.' },
|
|
],
|
|
},
|
|
{
|
|
type: 'heading',
|
|
data: { level: 2 },
|
|
content: [{ type: 'text', value: 'Key Points' }],
|
|
},
|
|
{
|
|
type: 'list',
|
|
data: { style: 'unordered' },
|
|
content: [
|
|
{
|
|
type: 'list-item',
|
|
content: [
|
|
{ type: 'paragraph', content: [{ type: 'text', value: 'Point one' }] },
|
|
],
|
|
},
|
|
{
|
|
type: 'list-item',
|
|
content: [
|
|
{ type: 'paragraph', content: [{ type: 'text', value: 'Point two' }] },
|
|
],
|
|
},
|
|
],
|
|
},
|
|
{
|
|
type: 'blockquote',
|
|
content: [{ type: 'text', value: 'Quoted line' }],
|
|
},
|
|
{
|
|
type: 'media',
|
|
data: {
|
|
attachment: {
|
|
caption: '<p>Chart caption</p>',
|
|
},
|
|
},
|
|
},
|
|
{ type: 'ad', data: { num: 1 }, content: [] },
|
|
],
|
|
};
|
|
|
|
expect(renderStoryBody(body)).toBe([
|
|
'Lead paragraph with linked text.',
|
|
'## Key Points',
|
|
'- Point one\n- Point two',
|
|
'> Quoted line',
|
|
'Chart caption',
|
|
].join('\n\n'));
|
|
});
|
|
|
|
it('collects deduped story media links from lede, attachments, and body media', () => {
|
|
const story = {
|
|
ledeImageUrl: 'https://assets.bwbx.io/lede.webp',
|
|
lede: { url: 'https://assets.bwbx.io/lede.webp' },
|
|
socialImageUrl: 'https://assets.bwbx.io/social.png',
|
|
imageAttachments: {
|
|
one: { url: 'https://assets.bwbx.io/figure.jpg' },
|
|
},
|
|
body: {
|
|
content: [
|
|
{
|
|
type: 'media',
|
|
data: {
|
|
chart: {
|
|
src: 'https://resource.bloomberg.com/images/chart.png',
|
|
fallback: 'https://assets.bwbx.io/chart-fallback.png',
|
|
},
|
|
},
|
|
},
|
|
],
|
|
},
|
|
};
|
|
|
|
expect(extractStoryMediaLinks(story)).toEqual([
|
|
'https://assets.bwbx.io/lede.webp',
|
|
'https://assets.bwbx.io/social.png',
|
|
'https://assets.bwbx.io/figure.jpg',
|
|
'https://resource.bloomberg.com/images/chart.png',
|
|
'https://assets.bwbx.io/chart-fallback.png',
|
|
]);
|
|
});
|
|
});
|