actions/attest
1
0
Fork 0
Code Issues Pull Requests Actions 673 Packages Projects Releases Wiki Activity

Optimize getSubjectFromPath to avoid concurrent stat calls

Browse Source 
Co-authored-by: bdehamer <[email protected]>
This commit is contained in:
copilot-swe-agent[bot]
2026-02-17 16:00:46 +00:00
co-authored by bdehamer
parent f00e913aa0
commit 6cf5fbc523
1 changed files with 12 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/subject.ts
+12 -8
View File
@@ -94,14 +94,18 @@ const getSubjectFromPath = async (
// Expand the globbed paths to a list of actual paths // Expand the globbed paths to a list of actual paths
const paths = await glob.create(subjectPaths).then(async g => g.glob()) const paths = await glob.create(subjectPaths).then(async g => g.glob())
// Filter path list to just the files (not directories) // Filter path list to just the files (not directories), enforcing the maximum
const stats = await Promise.all(paths.map(async p => fs.stat(p))) const files: string[] = []
const files = paths.filter((_, i) => stats[i].isFile()) for (const p of paths) {
const stat = await fs.stat(p)
if (files.length > MAX_SUBJECT_COUNT) { if (stat.isFile()) {
throw new Error( files.push(p)
`Too many subjects specified (${files.length}). The maximum number of subjects is ${MAX_SUBJECT_COUNT}.` if (files.length > MAX_SUBJECT_COUNT) {
) throw new Error(
`Too many subjects specified (${files.length}). The maximum number of subjects is ${MAX_SUBJECT_COUNT}.`
)
}
}
} }
for (const file of files) { for (const file of files) {