mirror of
https://github.com/mauriceboe/TREK.git
synced 2026-10-18 21:16:47 +00:00
Maurice
81ec7dd28d
The SDK accepted any number as apiVersion while the host refuses anything but a positive integer no newer than its plugin API, a false green for an author. The manifest patterns, sets and caps now live in the host's protocol/manifest-rules.ts, and gen-plugin-facts writes them and PLUGIN_API_VERSION into the SDK, so check:plugin-facts covers them. One manifest corpus is judged by both validators in a server test (the SDK may only be stricter, case by case), and a typecheck test holds the SDK's plugin API types to the host child's copy, which gains rawBodyBase64.
46 lines
2.0 KiB
TypeScript
46 lines
2.0 KiB
TypeScript
/**
|
|
* The SDK half of the manifest parity corpus (test/fixtures/manifest-corpus.ts). It
|
|
* runs standalone; the server's manifest-parity test runs the same corpus through
|
|
* the host's loader as well, so the two cannot drift apart unseen.
|
|
*/
|
|
import { describe, expect, it } from 'vitest';
|
|
import { validateManifest } from '../src/manifest.js';
|
|
import { PLUGIN_API_VERSION } from '../src/index.js';
|
|
import { MANIFEST_CORPUS } from './fixtures/manifest-corpus.js';
|
|
|
|
describe('manifest corpus — validate', () => {
|
|
it.each(MANIFEST_CORPUS)('$name', ({ manifest, host, sdk }) => {
|
|
const result = validateManifest(manifest);
|
|
expect(result.ok ? 'accept' : 'reject', result.errors.join('; ')).toBe(sdk ?? host);
|
|
});
|
|
|
|
it('marks every stricter-than-host case as one, with its reason', () => {
|
|
for (const c of MANIFEST_CORPUS) {
|
|
if (c.sdk === undefined) continue;
|
|
expect(c.sdk, `${c.name}: the SDK may only ever be stricter than the host`).toBe('reject');
|
|
expect(c.host, c.name).toBe('accept');
|
|
expect(c.why, `${c.name} needs a reason`).toBeTruthy();
|
|
}
|
|
});
|
|
});
|
|
|
|
describe('apiVersion', () => {
|
|
const base = { id: 'demo-plugin', name: 'Demo', version: '1.0.0', type: 'integration', trek: '>=4.0.0 <5.0.0' };
|
|
|
|
it.each([0, -1, 1.5, '1', null])('refuses %p, which is not a positive integer', (apiVersion) => {
|
|
expect(validateManifest({ ...base, apiVersion }).errors).toContain('apiVersion must be a positive integer');
|
|
});
|
|
|
|
it('refuses a version newer than the plugin API TREK implements', () => {
|
|
const next = PLUGIN_API_VERSION + 1;
|
|
expect(validateManifest({ ...base, apiVersion: next }).errors).toEqual([
|
|
`apiVersion ${next} is newer than the plugin API TREK implements (v${PLUGIN_API_VERSION})`,
|
|
]);
|
|
});
|
|
|
|
it('keeps an accepted version and defaults a missing one to 1', () => {
|
|
expect(validateManifest({ ...base, apiVersion: PLUGIN_API_VERSION }).manifest?.apiVersion).toBe(PLUGIN_API_VERSION);
|
|
expect(validateManifest(base).manifest?.apiVersion).toBe(1);
|
|
});
|
|
});
|