Skip to content

Commit c276d6e

Browse files
committed
fix(ci): exclude _attic/ from deno test; guard CFL upload-sarif on file existence
deno test -A was picking up legacy Node.js files in _attic/ that use require() (not valid in Deno), causing test failures. Add test.exclude to deno.json to match the existing lint/fmt exclusions. ClusterFuzzLite upload-sarif step was failing with "path does not exist" when the fuzzer found no bugs (no vulnerabilities.sarif produced). Guard the step with hashFiles() so it only runs when the SARIF report actually exists. https://claude.ai/code/session_01EJLZKDtcF1RGdKx6TcUTQG
1 parent 143567a commit c276d6e

2 files changed

Lines changed: 6 additions & 1 deletion

File tree

.github/workflows/cflite_pr.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,6 @@ jobs:
2626
mode: code-change
2727
output-sarif: true
2828
- uses: github/codeql-action/upload-sarif@662472033e021d55d94146f66f6058822b0b39fd # v3
29-
if: always()
29+
if: always() && hashFiles('vulnerabilities.sarif') != ''
3030
with:
3131
sarif_file: vulnerabilities.sarif

deno.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,10 @@
88
"exclude": [
99
"playground/experiments/_attic/"
1010
]
11+
},
12+
"test": {
13+
"exclude": [
14+
"playground/experiments/_attic/"
15+
]
1116
}
1217
}

0 commit comments

Comments
 (0)