docgrity 0.1.2 → 0.1.3

package/test/json.test.ts

@@ -1,87 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { extractJson, num, str, validateEvidence } from '../src/core/json';
-
-describe('extractJson', () => {
-  it('parses a bare JSON object', () => {
-    expect(extractJson('{"a":1}')).toEqual({ a: 1 });
-  });
-
-  it('parses JSON wrapped in prose and code fences', () => {
-    const text = 'Sure! Here is the result:\n```json\n{"is_duplicate": true, "confidence": 0.9}\n```\nLet me know.';
-    expect(extractJson(text)).toEqual({ is_duplicate: true, confidence: 0.9 });
-  });
-
-  it('handles nested objects and braces inside strings', () => {
-    const text = 'prefix {"summary": "uses {braces} and \\"quotes\\"", "detail": {"x": 1}} suffix {ignored}';
-    expect(extractJson(text)).toEqual({
-      summary: 'uses {braces} and "quotes"',
-      detail: { x: 1 },
-    });
-  });
-
-  it('throws when there is no JSON object', () => {
-    expect(() => extractJson('no json here')).toThrow(/No JSON object/);
-  });
-
-  it('throws on unbalanced JSON', () => {
-    expect(() => extractJson('{"a": 1')).toThrow(/Unbalanced/);
-  });
-});
-
-describe('num', () => {
-  it('clamps to [0,1] by default', () => {
-    expect(num(2)).toBe(1);
-    expect(num(-1)).toBe(0);
-    expect(num(0.5)).toBe(0.5);
-  });
-
-  it('coerces non-numbers to 0', () => {
-    expect(num('not a number')).toBe(0);
-    expect(num(undefined)).toBe(0);
-    expect(num(null)).toBe(0);
-  });
-
-  it('accepts numeric strings', () => {
-    expect(num('0.7')).toBe(0.7);
-  });
-});
-
-describe('str', () => {
-  it('passes strings through', () => {
-    expect(str('hello')).toBe('hello');
-  });
-
-  it('coerces null/undefined to empty string', () => {
-    expect(str(null)).toBe('');
-    expect(str(undefined)).toBe('');
-  });
-
-  it('stringifies other values', () => {
-    expect(str(42)).toBe('42');
-  });
-});
-
-describe('validateEvidence', () => {
-  it('normalises page to A unless explicitly B', () => {
-    const out = validateEvidence([
-      { page: 'B', excerpt: 'x' },
-      { page: 'C', excerpt: 'y' },
-      { excerpt: 'z' },
-    ]);
-    expect(out.map((e) => e.page)).toEqual(['B', 'A', 'A']);
-  });
-
-  it('drops entries with empty excerpts', () => {
-    expect(validateEvidence([{ page: 'A', excerpt: '' }, { page: 'A' }])).toEqual([]);
-  });
-
-  it('truncates excerpts to 1000 chars', () => {
-    const out = validateEvidence([{ page: 'A', excerpt: 'a'.repeat(2000) }]);
-    expect(out[0].excerpt).toHaveLength(1000);
-  });
-
-  it('tolerates non-array input', () => {
-    expect(validateEvidence(null)).toEqual([]);
-    expect(validateEvidence('nope')).toEqual([]);
-  });
-});

package/test/prefilter.test.ts

@@ -1,65 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { hasOpenQuestionSignals, mapLimit } from '../src/core/prefilter';
-
-describe('hasOpenQuestionSignals', () => {
-  it('detects TODO/TBD/FIXME markers', () => {
-    expect(hasOpenQuestionSignals('Deploy steps\nTODO: add rollback')).toBe(true);
-    expect(hasOpenQuestionSignals('Owner: TBD')).toBe(true);
-    expect(hasOpenQuestionSignals('FIXME later')).toBe(true);
-    expect(hasOpenQuestionSignals('This is to be decided by the team')).toBe(true);
-  });
-
-  it('detects placeholders', () => {
-    expect(hasOpenQuestionSignals('Region: ???')).toBe(true);
-    expect(hasOpenQuestionSignals('Contact: <add here>')).toBe(true);
-  });
-
-  it('detects unanswered question lines', () => {
-    expect(hasOpenQuestionSignals('# Notes\nWho owns the billing service?\n')).toBe(true);
-  });
-
-  it('ignores questions inside fenced code blocks', () => {
-    expect(hasOpenQuestionSignals('```\nis this ok?\n```\nAll settled.')).toBe(false);
-  });
-
-  it('returns false for clean docs', () => {
-    expect(hasOpenQuestionSignals('# Runbook\nRestart the service with systemctl.')).toBe(false);
-  });
-
-  it('does not match marker substrings inside words', () => {
-    expect(hasOpenQuestionSignals('The mastodon population is stable.')).toBe(false);
-  });
-});
-
-describe('mapLimit', () => {
-  it('preserves order and maps all items', async () => {
-    const res = await mapLimit([1, 2, 3, 4, 5], 2, async (n) => n * 10);
-    expect(res.map((r) => (r.ok ? r.value : -1))).toEqual([10, 20, 30, 40, 50]);
-  });
-
-  it('captures per-item errors without aborting the batch', async () => {
-    const res = await mapLimit([1, 2, 3], 2, async (n) => {
-      if (n === 2) throw new Error('boom');
-      return n;
-    });
-    expect(res[0]).toEqual({ ok: true, value: 1 });
-    expect(res[1].ok).toBe(false);
-    expect(res[2]).toEqual({ ok: true, value: 3 });
-  });
-
-  it('respects the concurrency limit', async () => {
-    let active = 0;
-    let peak = 0;
-    await mapLimit([1, 2, 3, 4, 5, 6], 2, async () => {
-      active++;
-      peak = Math.max(peak, active);
-      await new Promise((r) => setTimeout(r, 5));
-      active--;
-    });
-    expect(peak).toBeLessThanOrEqual(2);
-  });
-
-  it('handles empty input', async () => {
-    expect(await mapLimit([], 4, async () => 1)).toEqual([]);
-  });
-});

package/test/slug.test.ts

@@ -1,31 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { parseGithubSlug } from '../src/core/slug';
-
-describe('parseGithubSlug', () => {
-  it('parses https remotes', () => {
-    expect(parseGithubSlug('https://github.com/ujjavala/docgrity-vscode.git')).toBe(
-      'ujjavala/docgrity-vscode'
-    );
-    expect(parseGithubSlug('https://github.com/ujjavala/docgrity-vscode')).toBe(
-      'ujjavala/docgrity-vscode'
-    );
-  });
-
-  it('parses ssh remotes', () => {
-    expect(parseGithubSlug('git@github.com:owner/repo.git')).toBe('owner/repo');
-    expect(parseGithubSlug('ssh://git@github.com/owner/repo.git')).toBe('owner/repo');
-  });
-
-  it('keeps dots in repo names', () => {
-    expect(parseGithubSlug('git@github.com:owner/my.repo.name.git')).toBe('owner/my.repo.name');
-  });
-
-  it('handles trailing slash and whitespace', () => {
-    expect(parseGithubSlug(' https://github.com/owner/repo/ \n')).toBe('owner/repo');
-  });
-
-  it('returns undefined for non-GitHub remotes', () => {
-    expect(parseGithubSlug('https://gitlab.com/owner/repo.git')).toBeUndefined();
-    expect(parseGithubSlug('')).toBeUndefined();
-  });
-});

package/test/verify.test.ts

@@ -1,47 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { verifyExcerpts } from '../src/core/verify';
-
-const docA = 'The deploy pipeline runs   on every merge to main.\nReleases are weekly.';
-const docB = 'Releases happen monthly after QA sign-off.';
-
-describe('verifyExcerpts', () => {
-  it('accepts excerpts present verbatim', () => {
-    expect(verifyExcerpts([{ excerpt: 'Releases are weekly.' }], [docA, docB])).toBe(true);
-  });
-
-  it('is whitespace-insensitive', () => {
-    expect(
-      verifyExcerpts([{ excerpt: 'deploy pipeline runs on every merge' }], [docA])
-    ).toBe(true);
-  });
-
-  it('is case-insensitive', () => {
-    expect(verifyExcerpts([{ excerpt: 'RELEASES HAPPEN MONTHLY' }], [docB])).toBe(true);
-  });
-
-  it('rejects hallucinated excerpts', () => {
-    expect(verifyExcerpts([{ excerpt: 'Releases are daily.' }], [docA, docB])).toBe(false);
-  });
-
-  it('rejects when any excerpt fails (all-or-nothing)', () => {
-    expect(
-      verifyExcerpts(
-        [{ excerpt: 'Releases are weekly.' }, { excerpt: 'invented claim' }],
-        [docA, docB]
-      )
-    ).toBe(false);
-  });
-
-  it('rejects empty excerpts', () => {
-    expect(verifyExcerpts([{ excerpt: '   ' }], [docA])).toBe(false);
-  });
-
-  it('accepts an empty excerpt list (nothing to verify)', () => {
-    expect(verifyExcerpts([], [docA])).toBe(true);
-  });
-
-  it('only compares the first 200 normalised chars of long excerpts', () => {
-    const long = 'x'.repeat(300);
-    expect(verifyExcerpts([{ excerpt: long }], [`prefix ${'x'.repeat(200)} suffix`])).toBe(true);
-  });
-});

package/tsconfig.json

@@ -1,15 +0,0 @@
-{
-  "compilerOptions": {
-    "module": "commonjs",
-    "target": "ES2022",
-    "lib": ["ES2022"],
-    "outDir": "out",
-    "rootDir": "src",
-    "strict": true,
-    "sourceMap": true,
-    "esModuleInterop": true,
-    "types": ["node"],
-    "skipLibCheck": true
-  },
-  "include": ["src"]
-}

package/vitest.config.mts

@@ -1,9 +0,0 @@
-import { defineConfig } from 'vitest/config';
-
-// action/ is a zero-dependency package tested with node:test (`cd action && npm test`);
-// vitest covers the extension sources only.
-export default defineConfig({
-  test: {
-    exclude: ['**/node_modules/**', 'action/**', 'out/**'],
-  },
-});