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
133 lines
3.6 KiB
TypeScript
133 lines
3.6 KiB
TypeScript
import * as fs from 'node:fs';
|
|
import * as os from 'node:os';
|
|
import * as path from 'node:path';
|
|
import { afterEach, describe, expect, it } from 'vitest';
|
|
import { __test__ } from './feed.js';
|
|
|
|
describe('linux-do feed metadata resolution', () => {
|
|
afterEach(() => {
|
|
__test__.resetMetadataCaches();
|
|
});
|
|
|
|
it('prefers live tag metadata over the bundled snapshot', async () => {
|
|
__test__.setLiveMetadataForTests({
|
|
tags: [{ id: 9999, slug: 'fresh-tag', name: 'Fresh Tag' }],
|
|
});
|
|
|
|
const request = await __test__.resolveFeedRequest(null, {
|
|
tag: 'Fresh Tag',
|
|
view: 'latest',
|
|
limit: 20,
|
|
});
|
|
|
|
expect(request.url).toBe('/tag/fresh-tag/9999.json?per_page=20');
|
|
});
|
|
|
|
it('uses live category metadata with parent paths for subcategories', async () => {
|
|
__test__.setLiveMetadataForTests({
|
|
categories: [
|
|
{
|
|
id: 10,
|
|
name: 'Parent',
|
|
description: '',
|
|
slug: 'parent',
|
|
parentCategoryId: null,
|
|
parent: null,
|
|
},
|
|
{
|
|
id: 11,
|
|
name: 'Fresh Child',
|
|
description: '',
|
|
slug: 'fresh-child',
|
|
parentCategoryId: 10,
|
|
parent: {
|
|
id: 10,
|
|
name: 'Parent',
|
|
description: '',
|
|
slug: 'parent',
|
|
parentCategoryId: null,
|
|
},
|
|
},
|
|
],
|
|
});
|
|
|
|
const request = await __test__.resolveFeedRequest(null, {
|
|
category: 'Fresh Child',
|
|
view: 'hot',
|
|
limit: 20,
|
|
});
|
|
|
|
expect(request.url).toBe('/c/parent/fresh-child/11/l/hot.json?per_page=20');
|
|
});
|
|
|
|
it('accepts parent/name category paths for subcategories', async () => {
|
|
__test__.setLiveMetadataForTests({
|
|
categories: [
|
|
{
|
|
id: 10,
|
|
name: 'Parent',
|
|
description: '',
|
|
slug: 'parent',
|
|
parentCategoryId: null,
|
|
parent: null,
|
|
},
|
|
{
|
|
id: 11,
|
|
name: 'Fresh Child',
|
|
description: '',
|
|
slug: 'fresh-child',
|
|
parentCategoryId: 10,
|
|
parent: {
|
|
id: 10,
|
|
name: 'Parent',
|
|
description: '',
|
|
slug: 'parent',
|
|
parentCategoryId: null,
|
|
},
|
|
},
|
|
],
|
|
});
|
|
|
|
const request = await __test__.resolveFeedRequest(null, {
|
|
category: 'Parent / Fresh Child',
|
|
view: 'latest',
|
|
limit: 20,
|
|
});
|
|
|
|
expect(request.url).toBe('/c/parent/fresh-child/11.json?per_page=20');
|
|
});
|
|
|
|
it('falls back to cached metadata when live metadata is unavailable', async () => {
|
|
const cacheDir = fs.mkdtempSync(path.join(os.tmpdir(), 'opencli-linux-do-cache-'));
|
|
__test__.setCacheDirForTests(cacheDir);
|
|
|
|
fs.writeFileSync(path.join(cacheDir, 'tags.json'), JSON.stringify({
|
|
fetchedAt: new Date(Date.now() - 7 * 24 * 60 * 60 * 1000).toISOString(),
|
|
data: [{ id: 3, slug: 'chatgpt', name: 'ChatGPT' }],
|
|
}));
|
|
fs.writeFileSync(path.join(cacheDir, 'categories.json'), JSON.stringify({
|
|
fetchedAt: new Date(Date.now() - 7 * 24 * 60 * 60 * 1000).toISOString(),
|
|
data: [{
|
|
id: 4,
|
|
name: '开发调优',
|
|
description: '',
|
|
slug: 'develop',
|
|
parentCategoryId: null,
|
|
parent: null,
|
|
}],
|
|
}));
|
|
|
|
const request = await __test__.resolveFeedRequest(null, {
|
|
tag: 'ChatGPT',
|
|
category: '开发调优',
|
|
view: 'top',
|
|
period: 'monthly',
|
|
limit: 20,
|
|
});
|
|
|
|
expect(request.url).toContain('/tags/c/develop/4/chatgpt/3/l/top.json');
|
|
expect(request.url).toContain('per_page=20');
|
|
expect(request.url).toContain('period=monthly');
|
|
});
|
|
});
|