singularity/singularity-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #2609 from gsd-build/refine/extract-planning-state-checks

Browse source 
refactor: extract planning-state validation helpers in detectRogueFileWrites
This commit is contained in:
TÂCHES 2026-03-25 23:13:02 -06:00 • committed by GitHub
commit 07375e4948
1 changed files with 11 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
src/resources/extensions/gsd/auto-post-unit.ts
View file

@ -84,6 +84,15 @@ export interface RogueFileWrite {
* in postUnitPostVerification() eventually ingests rogue files, but explicit
* detection provides immediate diagnostics so operators know the prompt failed.
*/
// eslint-disable-next-line @typescript-eslint/no-explicit-any
function hasNonEmptyFields(row: Record<string, any> | null, fields: string[]): boolean {
if (!row) return false;
return fields.some(f => String(row[f] || "").trim().length > 0);
}
const MILESTONE_PLANNING_FIELDS = ["title", "vision", "requirement_coverage", "boundary_map_markdown"];
const SLICE_PLANNING_FIELDS = ["title", "demo", "risk", "depends"];
export function detectRogueFileWrites(
unitType: string,
unitId: string,
@ -124,12 +133,7 @@ export function detectRogueFileWrites(
if (!roadmapPath || !existsSync(roadmapPath)) return [];
const dbRow = getMilestone(mid);
const hasPlanningState = !!dbRow && (
String(dbRow.title || "").trim().length > 0 ||
String(dbRow.vision || "").trim().length > 0 ||
String(dbRow.requirement_coverage || "").trim().length > 0 ||
String(dbRow.boundary_map_markdown || "").trim().length > 0
);
const hasPlanningState = hasNonEmptyFields(dbRow, MILESTONE_PLANNING_FIELDS);
if (!hasPlanningState) {
rogues.push({ path: roadmapPath, unitType, unitId });
@ -142,12 +146,7 @@ export function detectRogueFileWrites(
if (!planPath || !existsSync(planPath)) return [];
const dbRow = getSlice(mid, sid);
const hasPlanningState = !!dbRow && (
String(dbRow.title || "").trim().length > 0 ||
String(dbRow.demo || "").trim().length > 0 ||
String(dbRow.risk || "").trim().length > 0 ||
String(dbRow.depends || "").trim().length > 0
);
const hasPlanningState = hasNonEmptyFields(dbRow, SLICE_PLANNING_FIELDS);
if (!hasPlanningState) {
rogues.push({ path: planPath, unitType, unitId });