mirror of
https://github.com/tiennm99/ccs.git
synced 2026-07-16 14:16:43 +00:00
fix(maintainability): scan tracked src files for stable gate
This commit is contained in:
@@ -201,6 +201,8 @@ All criteria achieved:
|
||||
- `bun run maintainability:check`
|
||||
- `npm run maintainability:check`
|
||||
|
||||
The baseline/check scripts enumerate git-tracked files under `src` for deterministic results, with filesystem traversal fallback when git is unavailable.
|
||||
|
||||
The check mode supports a maintainability regression gate that blocks increases in:
|
||||
- `process.exit` references
|
||||
- synchronous fs API references
|
||||
|
||||
@@ -1,5 +1,6 @@
|
||||
#!/usr/bin/env node
|
||||
|
||||
const { execFileSync } = require('child_process');
|
||||
const fs = require('fs');
|
||||
const path = require('path');
|
||||
|
||||
@@ -117,7 +118,7 @@ function parseArgs(argv) {
|
||||
return options;
|
||||
}
|
||||
|
||||
function collectFiles(dirPath) {
|
||||
function collectFilesFromFileSystem(dirPath) {
|
||||
const collected = [];
|
||||
const entries = fs
|
||||
.readdirSync(dirPath, { withFileTypes: true })
|
||||
@@ -126,7 +127,7 @@ function collectFiles(dirPath) {
|
||||
for (const entry of entries) {
|
||||
const fullPath = path.join(dirPath, entry.name);
|
||||
if (entry.isDirectory()) {
|
||||
collected.push(...collectFiles(fullPath));
|
||||
collected.push(...collectFilesFromFileSystem(fullPath));
|
||||
continue;
|
||||
}
|
||||
|
||||
@@ -138,6 +139,49 @@ function collectFiles(dirPath) {
|
||||
return collected;
|
||||
}
|
||||
|
||||
function collectTrackedFilesFromGit() {
|
||||
try {
|
||||
const output = execFileSync('git', ['ls-files', '-z', '--', 'src'], {
|
||||
cwd: PROJECT_ROOT,
|
||||
encoding: 'utf8',
|
||||
stdio: ['ignore', 'pipe', 'ignore'],
|
||||
});
|
||||
|
||||
if (!output) {
|
||||
return [];
|
||||
}
|
||||
|
||||
return output
|
||||
.split('\0')
|
||||
.filter(Boolean)
|
||||
.sort((left, right) => left.localeCompare(right))
|
||||
.map(relativePath => path.resolve(PROJECT_ROOT, relativePath))
|
||||
.filter(filePath => {
|
||||
const relativeToSrc = path.relative(SRC_DIR, filePath);
|
||||
if (relativeToSrc.startsWith('..') || path.isAbsolute(relativeToSrc)) {
|
||||
return false;
|
||||
}
|
||||
|
||||
try {
|
||||
return fs.statSync(filePath).isFile();
|
||||
} catch {
|
||||
return false;
|
||||
}
|
||||
});
|
||||
} catch {
|
||||
return null;
|
||||
}
|
||||
}
|
||||
|
||||
function collectFilesInSrc() {
|
||||
const trackedFiles = collectTrackedFilesFromGit();
|
||||
if (trackedFiles !== null) {
|
||||
return trackedFiles;
|
||||
}
|
||||
|
||||
return collectFilesFromFileSystem(SRC_DIR);
|
||||
}
|
||||
|
||||
function countLines(content) {
|
||||
if (content.length === 0) {
|
||||
return 0;
|
||||
@@ -160,7 +204,7 @@ function collectMetrics() {
|
||||
throw new Error(`Directory not found: ${SRC_DIR}`);
|
||||
}
|
||||
|
||||
const files = collectFiles(SRC_DIR);
|
||||
const files = collectFilesInSrc();
|
||||
|
||||
let typeScriptFileCount = 0;
|
||||
let locInSrc = 0;
|
||||
|
||||
Reference in New Issue
Block a user