Skip to content

Comments

Add validation script (alt)#5

Closed
magicmark wants to merge 3 commits intographql:mainfrom
magicmark:add-validation-script-2
Closed

Add validation script (alt)#5
magicmark wants to merge 3 commits intographql:mainfrom
magicmark:add-validation-script-2

Conversation

@magicmark
Copy link
Contributor

Alt version of #3 to implement using glob per:

Not ideal; this would find GAP-* (unlikely currently) folders inside node_modules for example. Maybe just incorporate the finding into the validate-structure.js script? Note glob is built in to Node.js now:

https://nodejs.org/api/fs.html#fspromisesglobpattern-options

The script now handles directory discovery internally using Node.js's built-in
globSync. It takes a required directory path argument and discovers all GAP-*
directories within that path using glob patterns, then validates each one.

Usage: ./scripts/validate-structure.js <directory>

This keeps all logic self-contained in the script and eliminates the need for
the find command in the npm task.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@magicmark magicmark force-pushed the add-validation-script-2 branch from 8178292 to 5170b14 Compare February 16, 2026 16:22
@magicmark magicmark closed this Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant