mirror of
https://github.com/abhigyanpatwari/GitNexus.git
synced 2026-09-07 08:26:11 +00:00
d91428ad9d
* feat(cli): add `gitnexus publish` for opt-in understand-quickly registry Adds a small, opt-in command that fires a single `repository_dispatch` event at `looptech-ai/understand-quickly` to ask the registry for an instant resync of the current repo's entry. No graph file is uploaded; the registry pulls from raw.githubusercontent.com per the protocol at https://github.com/looptech-ai/understand-quickly/blob/main/docs/integrations/protocol.md. - Pure helpers (id parsing, payload construction, validation) live in `gitnexus-shared/src/integrations/understand-quickly.ts` so the package stays Node-free and the same logic is testable in isolation. - The CLI command lives in `gitnexus/src/cli/publish.ts`. Without `UNDERSTAND_QUICKLY_TOKEN` it is a no-op (exits 0 with one informational line); with the token it POSTs the dispatch and surfaces 204 / 401 / 404 / 5xx distinctly. - The id defaults to `<owner>/<repo>` parsed from the `origin` remote and can be overridden with `--id`. - Refuses to publish when no `.gitnexus/` index exists, with a `gitnexus analyze` hint. Tests: a new vitest unit covers the pure helpers (8 + 8 + 2 cases) and the no-token no-op path with a `fetch` spy that fails the test if the network is touched. README gets a one-paragraph "Publishing to understand-quickly" section near the existing CLI docs. * fix(uq-publish): address review blockers + high-severity items Addresses CodeQL polynomial-regex (HIGH), token-gate ordering, distinct 401/403/404/422 response branches, fetch timeout, expanded test coverage, tightened owner/repo validation, and non-GitHub remote rejection. See response thread on PR #1425 for the per-finding rationale. Signed-off-by: amacsmith <alex.mac@looptech.ai> * fix(publish): address Claude review on PR #1425 - AbortError → TimeoutError: AbortSignal.timeout() throws a DOMException with name 'TimeoutError', not Error{name:'AbortError'}. Match the pattern used in core/embeddings/http-client.ts so the user-facing "timed out after 15000ms" message actually fires. Update the regression test to throw a real DOMException — the previous fake was a false-green. - isValidOwnerRepo: forbid trailing hyphen in the owner segment. GitHub rejects this at account-creation time; allowing it here meant hand-typed --id values like 'my-org-/repo' would pass our regex and 422 from GitHub. - Add publish-command coverage to cli-index-help.test.ts (asserts on --id, --skip-git, the registry name, and the token env var) and cli-commands.test.ts (asserts publishCommand is exported as a function). Catches accidental command-registration deletion. --------- Signed-off-by: amacsmith <alex.mac@looptech.ai> Co-authored-by: Gergő Magyar <gergomagyar@icloud.com>
97 lines
3.4 KiB
TypeScript
97 lines
3.4 KiB
TypeScript
import { describe, it, expect, vi } from 'vitest';
|
|
|
|
// Mock all the heavy imports before importing index
|
|
vi.mock('../../src/cli/analyze.js', () => ({
|
|
analyzeCommand: vi.fn(),
|
|
}));
|
|
vi.mock('../../src/cli/mcp.js', () => ({
|
|
mcpCommand: vi.fn(),
|
|
}));
|
|
vi.mock('../../src/cli/setup.js', () => ({
|
|
setupCommand: vi.fn(),
|
|
}));
|
|
vi.mock('../../src/cli/publish.js', () => ({
|
|
publishCommand: vi.fn(),
|
|
}));
|
|
|
|
describe('CLI commands', () => {
|
|
describe('version', () => {
|
|
it('package.json has a valid version string', async () => {
|
|
const pkg = await import('../../package.json', { with: { type: 'json' } });
|
|
expect(pkg.default.version).toMatch(/^\d+\.\d+\.\d+/);
|
|
});
|
|
});
|
|
|
|
describe('package.json scripts', () => {
|
|
it('has test scripts configured', async () => {
|
|
const pkg = await import('../../package.json', { with: { type: 'json' } });
|
|
expect(pkg.default.scripts.test).toBeDefined();
|
|
expect(pkg.default.scripts['test:integration']).toBeDefined();
|
|
expect(pkg.default.scripts['test:unit']).toBeDefined();
|
|
});
|
|
|
|
it('has build script', async () => {
|
|
const pkg = await import('../../package.json', { with: { type: 'json' } });
|
|
expect(pkg.default.scripts.build).toBeDefined();
|
|
});
|
|
});
|
|
|
|
describe('package.json bin entry', () => {
|
|
it('exposes gitnexus binary', async () => {
|
|
const pkg = await import('../../package.json', { with: { type: 'json' } });
|
|
expect(pkg.default.bin).toBeDefined();
|
|
expect(pkg.default.bin.gitnexus || pkg.default.bin).toBeDefined();
|
|
});
|
|
});
|
|
|
|
describe('optional parser dependencies', () => {
|
|
it('uses vendored source for tree-sitter-dart instead of a remote dependency', async () => {
|
|
const pkg = await import('../../package.json', { with: { type: 'json' } });
|
|
expect(pkg.default.optionalDependencies['tree-sitter-dart']).toBe(
|
|
'file:./vendor/tree-sitter-dart',
|
|
);
|
|
});
|
|
|
|
it('uses the vendored official Swift runtime package instead of source-building on install', async () => {
|
|
const pkg = await import('../../package.json', { with: { type: 'json' } });
|
|
const swiftPkg = await import('../../vendor/tree-sitter-swift/package.json', {
|
|
with: { type: 'json' },
|
|
});
|
|
expect(pkg.default.dependencies['tree-sitter']).toBe('^0.21.1');
|
|
expect(pkg.default.optionalDependencies['tree-sitter-swift']).toBe(
|
|
'file:./vendor/tree-sitter-swift',
|
|
);
|
|
expect(pkg.default.scripts.postinstall).not.toContain('tree-sitter-swift');
|
|
expect(swiftPkg.default.version).toBe('0.7.1');
|
|
expect(swiftPkg.default.peerDependencies['tree-sitter']).toContain('^0.21.1');
|
|
});
|
|
});
|
|
|
|
describe('analyzeCommand', () => {
|
|
it('is a function', async () => {
|
|
const { analyzeCommand } = await import('../../src/cli/analyze.js');
|
|
expect(typeof analyzeCommand).toBe('function');
|
|
});
|
|
});
|
|
|
|
describe('mcpCommand', () => {
|
|
it('is a function', async () => {
|
|
const { mcpCommand } = await import('../../src/cli/mcp.js');
|
|
expect(typeof mcpCommand).toBe('function');
|
|
});
|
|
});
|
|
|
|
describe('setupCommand', () => {
|
|
it('is a function', async () => {
|
|
const { setupCommand } = await import('../../src/cli/setup.js');
|
|
expect(typeof setupCommand).toBe('function');
|
|
});
|
|
});
|
|
|
|
describe('publishCommand', () => {
|
|
it('is a function', async () => {
|
|
const { publishCommand } = await import('../../src/cli/publish.js');
|
|
expect(typeof publishCommand).toBe('function');
|
|
});
|
|
});
|
|
});
|