Files
jackwener__opencli/clis/linux-do/topic-content.test.ts
jakevin 70b1145b5e refactor: migrate all CLI adapters from YAML to TypeScript (#887)
* refactor: remove version field from GenerateOutcome and EarlyHint

All consumers are in the same repo and evolve together — version field
adds ceremony without practical value at this stage.

Keeps schema_version in VerifiedArtifactMetadata (sidecar file format).

* refactor: migrate all 123 CLI adapters from YAML to TypeScript

Remove YAML as an adapter format entirely. All adapters now use
TypeScript with cli() from @jackwener/opencli/registry.

- Convert 123 YAML adapter files to TypeScript via batch script
- Remove YAML scanning from discovery.ts (registerYamlCli, yaml import)
- Remove scanYaml() and shouldReplaceManifestEntry() from build-manifest.ts
- Change synthesize.ts to output JSON candidates (internal format)
- Change generate-verified.ts to write .ts adapter files instead of .yaml
- Delete yaml-schema.ts (dead code) and scripts/yaml-to-ts.mjs (one-time tool)
- Update all tests to match new format

Closes discussion in #OpenCLI thread 47ddba82.

* fix: close YAML migration gaps in plugin scaffold, validation, and scan

- plugin-scaffold.ts: generate hello.ts (TS pipeline) instead of hello.yaml
- plugin.ts validatePluginStructure: no longer accept .yaml as valid command file
- plugin.ts scanPluginCommands: remove .yaml/.yml from scanned extensions
- discovery.ts: add explicit log.warn() when YAML files detected in clis/ or plugins/
- plugin.test.ts: update all test fixtures from .yaml to .js
- plugin-scaffold.test.ts: update hello.yaml references to hello.ts
- Delete dead src/yaml-schema.ts

Resolves PR #887 review blockers from @mbp-codex-pr0.

* refactor: complete YAML removal across docs, skills, record, and binance adapters

Code changes:
- record.ts: candidate output changed from .yaml (yaml.dump) to .json (JSON.stringify), removed js-yaml import
- src/clis/binance: convert all 11 YAML adapters to TypeScript cli() format
- binance/commands.test.ts: rewrite to use registry instead of yaml.load
- skill-generate.test.ts, diagnostic.test.ts: update mock paths from .yaml to .ts
- build-manifest.ts, synthesize.ts: update stale YAML comments

Documentation:
- README.md: remove .yaml from Dynamic Loader, fix plugin types, fix synthesize comment
- README.zh-CN.md: fix synthesize comment
- CONTRIBUTING.md: replace YAML Adapter section with Pipeline Adapter (TS), update arg examples
- docs/developer/yaml-adapter.md: replaced with deprecation redirect
- docs/developer/architecture.md: remove YAML pipeline references
- docs/developer/contributing.md: remove YAML adapter section
- docs/developer/ai-workflow.md: YAML → TS in synthesize description
- docs/guide/getting-started.md: remove .yaml from loader, update engine description
- docs/guide/plugins.md: remove YAML plugin option, update plugin types
- docs/index.md, docs/comparison.md: remove YAML adapter references
- docs/zh/guide/plugins.md: remove .yaml from scan description

Skills:
- opencli-explorer/SKILL.md: rewrite YAML vs TS decision tree to TS-only
- opencli-oneshot/SKILL.md: replace YAML templates with TS cli() templates
- opencli-generate/SKILL.md: YAML artifact path → TS artifact path
- opencli-usage/SKILL.md, plugins.md: update adapter format references

* fix: clean up remaining YAML adapter references in docs

- docs/zh/guide/plugins.md: replace YAML plugin example with TS pipeline
- docs/developer/testing.md: YAML Adapter heading → Adapter, remove validate line
- TESTING.md: same fix in root testing doc
- CONTRIBUTING.md: remove "YAML validation" comment
- docs/.vitepress/config.mts: mark YAML Adapter Guide as (Deprecated) in nav
- docs/advanced/download.md: remove "YAML Adapters" from pipeline step heading
2026-04-08 23:01:08 +08:00

68 lines
2.3 KiB
TypeScript

import { getRegistry } from '@jackwener/opencli/registry';
import fs from 'node:fs';
import { describe, expect, it } from 'vitest';
import { __test__ } from './topic-content.js';
describe('linux-do topic-content', () => {
it('prefers raw markdown when the topic payload includes it', () => {
const result = __test__.extractTopicContent({
title: 'Hello Linux.do',
post_stream: {
posts: [
{
post_number: 1,
username: 'neo',
raw: '## Heading\n\n- one\n- two',
cooked: '<h2>Heading</h2><ul><li>one</li><li>two</li></ul>',
like_count: 7,
created_at: '2025-04-05T10:00:00.000Z',
},
],
},
}, 1234);
expect(result.content).toContain('---');
expect(result.content).toContain('title: Hello Linux.do');
expect(result.content).toContain('author: neo');
expect(result.content).toContain('likes: 7');
expect(result.content).toContain('url: https://linux.do/t/1234');
expect(result.content).toContain('## Heading');
expect(result.content).toContain('- one');
});
it('falls back to cooked html and converts it to markdown', () => {
const result = __test__.extractTopicContent({
title: 'Converted Topic',
post_stream: {
posts: [
{
post_number: 1,
username: 'trinity',
cooked: '<p>Hello <strong>world</strong></p><blockquote><p>quoted</p></blockquote>',
like_count: 3,
created_at: '2025-04-05T10:00:00.000Z',
},
],
},
}, 42);
expect(result.content).toContain('Hello **world**');
expect(result.content).toContain('> quoted');
});
it('registers topic-content with plain default output for markdown body rendering', () => {
const command = getRegistry().get('linux-do/topic-content');
expect(command?.defaultFormat).toBe('plain');
expect(command?.columns).toEqual(['content']);
});
it('keeps topic adapter as a summarized first-page reader after the split', () => {
const topicTs = fs.readFileSync(new URL('./topic.ts', import.meta.url), 'utf8');
expect(topicTs).not.toContain('main_only');
expect(topicTs).toContain('slice(0, 200)');
expect(topicTs).toContain('帖子首页摘要和回复');
});
});