Nick Rolfe
9aba20e4c9
Merge branch 'main' into update-bundle/codeql-bundle-v2.21.2
2025-05-01 13:16:31 +01:00
Henry Mercer
62fbeb66b3
Merge branch 'main' into henrymercer/fix-diff-informed-multiple-analyze
2025-05-01 12:05:02 +01:00
Henry Mercer
f122d1dc9e
Address test failures from computing temporary directory too early
...
These relied on the RUNNER_TEMP environment variable that does not necessarily exist when running locally.
2025-05-01 12:01:22 +01:00
Henry Mercer
083772aae4
Do not fail diff informed analyses when analyze is run twice in the same job
2025-05-01 12:00:46 +01:00
Nick Rolfe
5db14d0471
Merge branch 'main' into update-bundle/codeql-bundle-v2.21.2
2025-05-01 10:28:59 +01:00
Andrew Eisenberg
3ca9a88941
Add actions-specific telemetry fields
2025-04-29 16:14:46 -07:00
github-actions[bot]
4b7eecf8a7
Update default bundle to codeql-bundle-v2.21.2
2025-04-28 15:20:40 +00:00
Michael B. Gale
6ad5d99ccc
Add goproxy_server to LANGUAGE_TO_REGISTRY_TYPE
2025-04-25 16:56:36 +01:00
Michael B. Gale
6ef9b921b1
Use proxy artifacts for v2.21.1
2025-04-24 18:20:31 +01:00
github-actions[bot]
e63b3f5166
Update default bundle to codeql-bundle-v2.21.1
2025-04-16 16:54:11 +00:00
Chuan-kai Lin
a35ae8c380
Merge pull request #2843 from github/cklin/diff-informed-compat
...
Set checkPresence in diff-range data extension
2025-04-07 13:29:16 -07:00
Andrew Eisenberg
bb59df6c17
Merge pull request #2842 from github/henrymercer/zip64
...
Raise the file limit for debug artifacts by producing zip64 files where necessary
2025-04-07 10:50:46 -07:00
Andrew Eisenberg
a8be43c24e
Don't throw error for ENOENT
2025-04-04 13:42:00 -07:00
Chuan-kai Lin
94102d99b0
Set checkPresence in diff-range data extension
...
This commit updates the diff-range data extension to use the new
checkPresence field being introduced in CodeQL CLI 2.21.0, so that
diff-informed analysis no longer fails when a query pack does not have
the restrictAlertsTo extensible predicate.
2025-04-04 08:41:50 -07:00
Henry Mercer
56feaac968
Raise file limit in debug artifacts by using zip64
2025-04-04 14:40:53 +01:00
Arthur Baars
2b85c00718
Merge branch 'main' into update-bundle/codeql-bundle-v2.21.0
2025-04-03 15:28:09 +02:00
Angela P Wen
92864f48b0
Add logs around status report telemetry in init-post step
2025-04-03 14:37:27 +02:00
Fotis Koutoulakis (@NlightNFotis)
676a422916
review-comments: nest validateSariFileSchema into try-catch block to better discriminate error thrown
2025-04-02 19:06:31 +01:00
Fotis Koutoulakis (@NlightNFotis)
498c7f37e8
review-comments: unwrap error in upload-sarif-action and re-classify as ConfigurationError if in known error category
2025-04-02 15:20:03 +01:00
Fotis Koutoulakis (@NlightNFotis)
efd29bef22
refactor: revert getActionsStatus taking an extra argument
2025-04-02 15:13:00 +01:00
Fotis Koutoulakis (@NlightNFotis)
b53826d56d
review-comments: remove syntax-error handling for SARIF from upload-lib
2025-04-01 15:10:16 +01:00
Fotis Koutoulakis (@NlightNFotis)
55ee663d5f
review-comments: refactor getActionsStatus to accept an extra parameter designating if the analysis is third-party
2025-04-01 14:58:59 +01:00
github-actions[bot]
a69f5113b7
Update default bundle to codeql-bundle-v2.21.0
2025-04-01 13:51:03 +00:00
Fotis Koutoulakis (@NlightNFotis)
01f1a1f2c9
Merge branch 'main' into NlightNFotis/reclassify_upload_sarif_issues
2025-03-31 16:29:02 +01:00
Chuan-kai Lin
efffb483ec
Merge pull request #2831 from github/cklin/diff-informed-query-filtering
...
Respect `exclude-from-incremental` query tag for diff-informed analysis
2025-03-31 08:00:50 -07:00
Fotis Koutoulakis (@NlightNFotis)
f21cf0bbd7
feat: reclassify InvalidSarifUploadError as a user-error when final status report is produced
2025-03-31 12:22:18 +01:00
Fotis Koutoulakis (@NlightNFotis)
72a2b1295e
feat: classify some observed SARIF errors as InvalidSarifUploadError
2025-03-31 12:17:23 +01:00
Fotis Koutoulakis (@NlightNFotis)
a022653e2d
feat: classify more HTTP errors as configuration errors in api-client
2025-03-31 11:54:16 +01:00
Fotis Koutoulakis (@NlightNFotis)
3c42562190
fix: update comment for test to state correct expected outcome
2025-03-31 11:51:11 +01:00
Chuan-kai Lin
e7f67e2e61
Redefine shouldPerformDiffInformedAnalysis()
...
This commit renames the original shouldPerformDiffInformedAnalysis(),
which returns `PullRequestBranches | undefined`, to
getDiffInformedAnalysisBranches(). It also adds a new
shouldPerformDiffInformedAnalysis() function that returns boolean.
Separating these two functions makes it clear what the intended uses and
return values should be for each.
2025-03-28 12:29:28 -07:00
Fotis Koutoulakis (@NlightNFotis)
73c938dbc0
fix: fix issue where wrapApiConfigurationError would fail to regex match a string due to boundary constraints on the regex
2025-03-28 14:38:06 +00:00
Fotis Koutoulakis (@NlightNFotis)
2be6da694a
test: add tests for the wrapApiConfigurationError function
2025-03-28 14:37:10 +00:00
Fotis Koutoulakis (@NlightNFotis)
76f9ed9cd9
test: add tests to validate getActionsStatus' behaviour
2025-03-28 14:37:10 +00:00
Chuan-kai Lin
71ab101d38
Set default query filter for diff-informed analysis
2025-03-27 14:06:40 -07:00
Chuan-kai Lin
da967b1ade
AugmentationProperties: add defaultQueryFilters
...
This commit adds a defaultQueryFilters field to AugmentationProperties
and incorporates its value into the augmented Code Scanning config.
However, in this commit defaultQueryFilters is always empty, so there is
not yet any actual behavior change.
2025-03-27 13:44:47 -07:00
Chuan-kai Lin
3c4533916b
Call shouldPerformDiffInformedAnalysis() outside setupDiffInformedQueryRun()
2025-03-27 10:27:24 -07:00
Chuan-kai Lin
1994ea768e
Move shouldPerformDiffInformedAnalysis()
2025-03-27 10:27:24 -07:00
Chuan-kai Lin
534bc63d5e
Rename diff-filtering-utils.ts to diff-informed-analysis-utils.ts
2025-03-27 10:27:23 -07:00
Chuan-kai Lin
3fbee52426
Extract shouldPerformDiffInformedAnalysis()
2025-03-27 10:27:23 -07:00
Chuan-kai Lin
c1fc897eb2
getFileDiffsWithBasehead(): use CODE_SCANNING_REPOSITORY if present
2025-03-27 08:50:31 -07:00
Chuan-kai Lin
f88459c0a3
Use getRepositoryNwo()
2025-03-26 10:18:40 -07:00
Chuan-kai Lin
b22f3341fe
Add getRepositoryNwo() helper functions
2025-03-26 08:11:16 -07:00
Chuan-kai Lin
b361a91508
Diff-informed analysis: fix empty PR handling
2025-03-21 14:18:25 -07:00
Chuan-kai Lin
bd1d9ab4ed
Merge pull request #2816 from github/cklin/overlay-file-list
...
Overlay databases: use --overlay-changes
2025-03-21 12:30:26 -07:00
Chuan-kai Lin
b98ae6ca52
Add overlay-database-utils tests
2025-03-21 11:31:28 -07:00
Chuan-kai Lin
9825184a0a
Add getFileOidsUnderPath() tests
2025-03-21 10:53:21 -07:00
Chuan-kai Lin
d109dd5d33
Detect PR branches for Default Setup
2025-03-21 09:25:08 -07:00
Chuan-kai Lin
3e5446c3d2
Introduce PullRequestBranches
2025-03-21 09:24:16 -07:00
Chuan-kai Lin
6adda79888
Move PR branch detection into setupDiffInformedQueryRun()
2025-03-20 09:51:17 -07:00
Chuan-kai Lin
6be6984cc1
Overlay databases: use --overlay-changes
...
This commit changes overlay database creation to use the
--overlay-changes flag. It also implements Git-based file change
detection to generate the list of files to extract for the overlay
database.
2025-03-19 11:38:45 -07:00
