fix: replace node -e validation with script, remove events.jsonl from forbidden_paths

autogame-17 · autogame-17 · commit 09da49725831 · 2026-02-28T08:42:01.000Z
- Create scripts/validate-modules.js for safe module validation - Update all genes.json validation commands to use the new script - Update buildValidationCmd and buildAutoGene to generate script-based commands - Remove assets/gep/events.jsonl from gene_gep_innovate_from_opportunity forbidden_paths (was blocking solidify) Fixes: GitHub #145, #148
diff --git a/assets/gep/genes.json b/assets/gep/genes.json
@@ -30,8 +30,8 @@
         ]
       },
       "validation": [
-        "node -e \"require('./src/evolve'); require('./src/gep/solidify'); console.log('ok')\"",
-        "node -e \"require('./src/gep/selector'); require('./src/gep/memoryGraph'); console.log('ok')\""
+        "node scripts/validate-modules.js ./src/evolve ./src/gep/solidify",
+        "node scripts/validate-modules.js ./src/gep/selector ./src/gep/memoryGraph"
       ]
     },
     {
@@ -64,7 +64,7 @@
         ]
       },
       "validation": [
-        "node -e \"require('./src/evolve'); require('./src/gep/prompt'); console.log('ok')\""
+        "node scripts/validate-modules.js ./src/evolve ./src/gep/prompt"
       ]
     },
     {
@@ -96,12 +96,11 @@
         "max_files": 25,
         "forbidden_paths": [
           ".git",
-          "node_modules",
-          "assets/gep/events.jsonl"
+          "node_modules"
         ]
       },
       "validation": [
-        "node -e \"require('./src/evolve'); require('./src/gep/solidify'); console.log('ok')\""
+        "node scripts/validate-modules.js ./src/evolve ./src/gep/solidify"
       ]
     }
   ]
diff --git a/scripts/validate-modules.js b/scripts/validate-modules.js
@@ -0,0 +1,6 @@
+// Usage: node scripts/validate-modules.js ./src/evolve ./src/gep/solidify
+// Requires each module to verify it loads without errors.
+const modules = process.argv.slice(2);
+if (!modules.length) { console.error('No modules specified'); process.exit(1); }
+for (const m of modules) { require(m); }
+console.log('ok');
diff --git a/src/gep/assetStore.js b/src/gep/assetStore.js
@@ -30,8 +30,8 @@ function writeJsonAtomic(filePath, obj) {
 // runValidations() executes with cwd=repoRoot, so require('./src/...')
 // resolves correctly without embedding machine-specific absolute paths.
 function buildValidationCmd(relModules) {
-  const checks = relModules.map(m => `require('./${m}')`);
-  return `node -e "${checks.join('; ')}; console.log('ok')"`;
+  const paths = relModules.map(m => `./${m}`);
+  return `node scripts/validate-modules.js ${paths.join(' ')}`;
 }
 
 function getDefaultGenes() {
diff --git a/src/gep/solidify.js b/src/gep/solidify.js
@@ -924,7 +924,7 @@ function buildAutoGene({ signals, intent }) {
         'skills/git-sync',
       ],
     },
-    validation: ['node -e "require(\'./src/gep/solidify\'); console.log(\'ok\')"'],
+    validation: ['node scripts/validate-modules.js ./src/gep/solidify'],
     epigenetic_marks: [], // Epigenetic marks: environment-specific expression modifiers
   };
   gene.asset_id = computeAssetId(gene);

Original file line number	Diff line number	Diff line change
`@@ -30,8 +30,8 @@`
`30`	`30`	`]`
`31`	`31`	`},`
`32`	`32`	`"validation": [`
`33`		`- "node -e \"require('./src/evolve'); require('./src/gep/solidify'); console.log('ok')\"",`
`34`		`- "node -e \"require('./src/gep/selector'); require('./src/gep/memoryGraph'); console.log('ok')\""`
	`33`	`+ "node scripts/validate-modules.js ./src/evolve ./src/gep/solidify",`
	`34`	`+ "node scripts/validate-modules.js ./src/gep/selector ./src/gep/memoryGraph"`
`35`	`35`	`]`
`36`	`36`	`},`
`37`	`37`	`{`
`@@ -64,7 +64,7 @@`
`64`	`64`	`]`
`65`	`65`	`},`
`66`	`66`	`"validation": [`
`67`		`- "node -e \"require('./src/evolve'); require('./src/gep/prompt'); console.log('ok')\""`
	`67`	`+ "node scripts/validate-modules.js ./src/evolve ./src/gep/prompt"`
`68`	`68`	`]`
`69`	`69`	`},`
`70`	`70`	`{`
`@@ -96,12 +96,11 @@`
`96`	`96`	`"max_files": 25,`
`97`	`97`	`"forbidden_paths": [`
`98`	`98`	`".git",`
`99`		`- "node_modules",`
`100`		`- "assets/gep/events.jsonl"`
	`99`	`+ "node_modules"`
`101`	`100`	`]`
`102`	`101`	`},`
`103`	`102`	`"validation": [`
`104`		`- "node -e \"require('./src/evolve'); require('./src/gep/solidify'); console.log('ok')\""`
	`103`	`+ "node scripts/validate-modules.js ./src/evolve ./src/gep/solidify"`
`105`	`104`	`]`
`106`	`105`	`}`
`107`	`106`	`]`