mirror of
https://github.com/abhigyanpatwari/GitNexus.git
synced 2026-09-11 22:53:04 +00:00
bf09eab95b
* feat: configure prettier with pre-commit hook integration Add prettier, lint-staged, and prettier-plugin-tailwindcss at the repo root with husky pre-commit hook integration. Moves husky from gitnexus/ to root package.json for reliable hook installation. - Root package.json with prepare/format/format:check scripts - .prettierrc with endOfLine:lf and tailwindStylesheet for TW v4 - .prettierignore excluding fixtures, vendor, generated, *.d.ts, *.md - .gitattributes enforcing LF line endings for Windows consistency - Pre-commit hook uses direct node_modules/.bin/ paths (no npx) * style: apply prettier formatting to entire codebase One-time bulk format. No logic changes. Use .git-blame-ignore-revs to skip this commit in git blame. * chore: add .git-blame-ignore-revs for prettier format commit * perf: pre-commit hook runs only tests related to staged files Use vitest --related to scope test execution to tests that import the changed files, instead of running the full suite on every commit. * perf: remove vitest from pre-commit hook, keep in CI only Pre-commit now runs lint-staged + tsc only. Tests run in CI (ci-tests.yml) where they belong — keeps commits fast. * ci: add prettier format check to quality workflow PRs will now fail if code isn't formatted with prettier.
88 lines
3.4 KiB
TypeScript
88 lines
3.4 KiB
TypeScript
import { describe, it, expect, beforeAll } from 'vitest';
|
|
import path from 'path';
|
|
import {
|
|
FIXTURES,
|
|
getRelationships,
|
|
getNodesByLabelFull,
|
|
runPipelineFromRepo,
|
|
type PipelineResult,
|
|
} from './helpers.js';
|
|
|
|
describe('shape_check integration', () => {
|
|
let result: PipelineResult;
|
|
|
|
beforeAll(async () => {
|
|
result = await runPipelineFromRepo(path.join(FIXTURES, 'shape-check-integration'), () => {});
|
|
}, 60000);
|
|
|
|
// --- Route & response shape extraction ---
|
|
|
|
it('creates Route nodes with responseKeys', () => {
|
|
const routes = getNodesByLabelFull(result, 'Route');
|
|
const searchRoute = routes.find((r) => r.name === '/api/search');
|
|
expect(searchRoute).toBeDefined();
|
|
expect(searchRoute!.properties.responseKeys).toBeDefined();
|
|
expect(searchRoute!.properties.responseKeys.length).toBeGreaterThan(0);
|
|
});
|
|
|
|
it('extracts quoted property keys without wrapping quotes', () => {
|
|
const routes = getNodesByLabelFull(result, 'Route');
|
|
const searchRoute = routes.find((r) => r.name === '/api/search');
|
|
expect(searchRoute).toBeDefined();
|
|
for (const key of searchRoute!.properties.responseKeys) {
|
|
expect(key).not.toMatch(/['"]/);
|
|
}
|
|
expect(searchRoute!.properties.responseKeys).toContain('courses');
|
|
expect(searchRoute!.properties.responseKeys).toContain('articles');
|
|
});
|
|
|
|
it('separates responseKeys from errorKeys by HTTP status', () => {
|
|
const routes = getNodesByLabelFull(result, 'Route');
|
|
const usersRoute = routes.find((r) => r.name === '/api/users');
|
|
expect(usersRoute).toBeDefined();
|
|
expect(usersRoute!.properties.responseKeys).toEqual(expect.arrayContaining(['data']));
|
|
expect(usersRoute!.properties.errorKeys).toEqual(expect.arrayContaining(['error']));
|
|
});
|
|
|
|
// --- Consumer key extraction ---
|
|
|
|
it('creates FETCHES edges with accessedKeys in reason', () => {
|
|
const edges = getRelationships(result, 'FETCHES');
|
|
const searchFetch = edges.find(
|
|
(e) => e.sourceFilePath.includes('SearchBar') && e.target === '/api/search',
|
|
);
|
|
expect(searchFetch).toBeDefined();
|
|
expect(searchFetch!.rel.reason).toContain('keys:');
|
|
const keysStr = searchFetch!.rel.reason!.match(/keys:([^|]+)/)?.[1] ?? '';
|
|
const keys = keysStr.split(',');
|
|
expect(keys).toContain('courses');
|
|
expect(keys).toContain('articles');
|
|
});
|
|
|
|
it('does not include DOM methods in consumer accessedKeys', () => {
|
|
const edges = getRelationships(result, 'FETCHES');
|
|
const gdprFetch = edges.find(
|
|
(e) => e.sourceFilePath.includes('GdprExport') && e.target === '/api/gdpr/export',
|
|
);
|
|
expect(gdprFetch).toBeDefined();
|
|
const keysStr = gdprFetch!.rel.reason!.match(/keys:([^|]+)/)?.[1] ?? '';
|
|
const keys = keysStr.split(',');
|
|
expect(keys).not.toContain('appendChild');
|
|
expect(keys).not.toContain('removeChild');
|
|
expect(keys).not.toContain('createElement');
|
|
expect(keys).not.toContain('click');
|
|
expect(keys).toContain('url');
|
|
});
|
|
|
|
it('captures error-path key access from consumers', () => {
|
|
const edges = getRelationships(result, 'FETCHES');
|
|
const userFetch = edges.find(
|
|
(e) => e.sourceFilePath.includes('UserList') && e.target === '/api/users',
|
|
);
|
|
expect(userFetch).toBeDefined();
|
|
const keysStr = userFetch!.rel.reason!.match(/keys:([^|]+)/)?.[1] ?? '';
|
|
const keys = keysStr.split(',');
|
|
expect(keys).toContain('data');
|
|
expect(keys).toContain('error');
|
|
});
|
|
});
|