Skip to content

ci: scope CI jobs by changed language directory using path filters#256

Open
aubreyquinn wants to merge 6 commits intomainfrom
users/aubreyquinn/fixedBuildIsses
Open

ci: scope CI jobs by changed language directory using path filters#256
aubreyquinn wants to merge 6 commits intomainfrom
users/aubreyquinn/fixedBuildIsses

Conversation

@aubreyquinn
Copy link
Copy Markdown
Contributor

@aubreyquinn aubreyquinn commented Mar 25, 2026

Add dorny/paths-filter to ci-orchestrator.yml so dotnet/nodejs/python jobs only run when files in their respective directories change.

Fixed build issues raised in this PR: #255

aubreyquinn and others added 4 commits March 25, 2026 17:18
Add dorny/paths-filter to ci-orchestrator.yml so dotnet/nodejs/python
jobs only run when files in their respective directories change.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
correlationId does not exist on the BaggageBuilder type. Remove the
calls from all affected nodejs sample agents.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
correlationId does not exist on BaggageBuilder. Replace with sessionId
using the Bot Framework activity ID, which is the unique per-turn
identifier and serves as the natural correlation ID for tracing.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 25, 2026

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 2 package(s) with unknown licenses.
See the Details below.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 6284384.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

License Issues

.github/workflows/ci-orchestrator.yml

PackageVersionLicenseIssue Type
actions/checkout4.*.*NullUnknown License
dorny/paths-filter3.*.*NullUnknown License
Denied Licenses: GPL-3.0-only, AGPL-3.0-only

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/checkout 4.*.* 🟢 6
Details
CheckScoreReason
Code-Review🟢 10all changesets reviewed
Maintained⚠️ 23 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 2
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
License🟢 10license file detected
Fuzzing⚠️ 0project is not fuzzed
Packaging⚠️ -1packaging workflow not detected
Signed-Releases⚠️ -1no releases found
Pinned-Dependencies🟢 3dependency not pinned by hash detected -- score normalized to 3
Security-Policy🟢 9security policy file detected
Branch-Protection🟢 6branch protection is not maximal on development and all release branches
SAST🟢 8SAST tool detected but not run on all commits
actions/dorny/paths-filter 3.*.* 🟢 4.6
Details
CheckScoreReason
Maintained🟢 109 commit(s) and 16 issue activity found in the last 90 days -- score normalized to 10
Binary-Artifacts🟢 10no binaries found in the repo
Packaging⚠️ -1packaging workflow not detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Code-Review🟢 3Found 7/21 approved changesets -- score normalized to 3
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Security-Policy⚠️ 0security policy file not detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0

Scanned Files

  • .github/workflows/ci-orchestrator.yml

aubreyquinn and others added 2 commits March 25, 2026 17:32
callerId and callerName do not exist on BaggageBuilder. Replace with
userId and userName which map to activity.from.id and activity.from.name.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
recordResponseId does not exist on InferenceScope. Replace with
recordAttributes using the gen_ai.response.id semantic convention key.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@aubreyquinn aubreyquinn marked this pull request as ready for review March 25, 2026 17:38
@aubreyquinn aubreyquinn requested a review from a team as a code owner March 25, 2026 17:38
Copilot AI review requested due to automatic review settings March 25, 2026 17:38
@aubreyquinn aubreyquinn enabled auto-merge (squash) March 25, 2026 17:39
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