Skip to content

chore(deps): clear 24 Dependabot alerts across aws/cdk, docs, examples - #370

Merged
joshuali925 merged 5 commits into
opensearch-project:mainfrom
kylehounslow:deps/security-sweep-2026-07-28
Jul 29, 2026
Merged

chore(deps): clear 24 Dependabot alerts across aws/cdk, docs, examples#370
joshuali925 merged 5 commits into
opensearch-project:mainfrom
kylehounslow:deps/security-sweep-2026-07-28

Conversation

@kylehounslow

Copy link
Copy Markdown
Collaborator

What

Clears 24 open Dependabot alerts across four manifests, plus a Dependabot coverage-gap fix. One commit per manifest group, all DCO-signed.

Alerts cleared (24)

aws/cdk (fix(aws/cdk))

docs (fix(docs))

docs/starlight-docs (fix(docs/starlight-docs))

examples (fix(examples), 3 sample apps)

Coverage gap (chore(deps))

.github/dependabot.yml: add aws/cdk, aws/cli-installer, examples/agent-evals/genai-sdk, docker-compose/agent-eval-canary. These manifests held open alerts with no bot PR.

Validation

  • aws/cdk: tsc --noEmit clean; fast-uri absent from lockfile; esbuild 0.28.1
  • docs: 557/557 vitest, 135-page build, links validator clean; sharp 0.35.3
  • docs/starlight-docs: 135-page build, links validator clean; astro 6.4.8, vite 7.3.6, all overrides applied
  • examples: all 3 uv lock --check pass; idna 3.18, starlette 1.3.1 in every lock

Vulnerable ranges confirmed absent from each regenerated lockfile, not from a green Dependabot check.

Deferred (5)

aws-cdk-lib ^2.262.1 drops the vulnerable fast-uri transitive out of the
tree entirely (opensearch-project#319, opensearch-project#321). esbuild override ^0.28.1 clears
GHSA-67mh-4wv8-2f99 (opensearch-project#180, range >=0.27.3 <0.28.1). tsc passes.

brace-expansion (opensearch-project#278, opensearch-project#329) is bundled inside aws-cdk-lib and not
reachable by an override; tracked as deferred.

Signed-off-by: Kyle Hounslow <kylhouns@amazon.com>
sharp override ^0.35.0 (resolves 0.35.3) clears GHSA-f88m-g3jw-g9cj
(opensearch-project#324, range <0.35.0). astro is already 7.x on main; docs deps
otherwise unchanged. 557/557 vitest pass, 135-page build, links valid.

Signed-off-by: Kyle Hounslow <kylhouns@amazon.com>
…ss/vite/esbuild to clear CVEs

astro ^6.4.8 clears GHSA-2pvr-wf23-7pc7, GHSA-jrpj-wcv7-9fh9 (opensearch-project#207, opensearch-project#208,
range <6.4.6). Overrides: sharp 0.35.3 (opensearch-project#327), svgo 4.0.2 (opensearch-project#326),
postcss 8.5.24 (opensearch-project#332), vite 7.3.6 (opensearch-project#198, opensearch-project#199), esbuild 0.28.1 (opensearch-project#179).

astro 7.x-only CVEs (opensearch-project#281, opensearch-project#293, opensearch-project#294) require a major upgrade, deferred.

Signed-off-by: Kyle Hounslow <kylhouns@amazon.com>
…canary

Unwatched manifests held open alerts with no bot PR; add them so
Dependabot cuts PRs going forward.

Signed-off-by: Kyle Hounslow <kylhouns@amazon.com>
@codecov

codecov Bot commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.62%. Comparing base (53be4f5) to head (649ae8b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #370   +/-   ##
=======================================
  Coverage   55.62%   55.62%           
=======================================
  Files           4        4           
  Lines         169      169           
  Branches       47       47           
=======================================
  Hits           94       94           
  Misses         74       74           
  Partials        1        1           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@joshuali925
joshuali925 merged commit c6d5505 into opensearch-project:main Jul 29, 2026
10 checks passed
@joshuali925

Copy link
Copy Markdown
Member

@joshuali925-osdbot check if there's any unnecessary change in the CVE fix PR, if so send a revert PR

@joshuali925-osdbot

Copy link
Copy Markdown
Contributor

On it — reviewing the merged CVE fix PR for any unnecessary changes. Will report back and open a revert PR if I find any.

@joshuali925-osdbot

Copy link
Copy Markdown
Contributor

Reviewed all changes in #370 against the alerts it set out to clear. Found one unnecessary change, and opened a revert: #382.

Unnecessary — reverted in #382

Checked and kept (necessary / intended)

  • requires-python >=3.9>=3.10 in weather-agent: necessary — I verified reverting it makes uv lock unsatisfiable, since starlette>=1.3.1 forces fastapi>=0.129.0 which requires Python >=3.10. (Under-documented in the PR, but correct.)
  • .github/dependabot.yml additions: the intended coverage-gap fix.
  • All astro/sharp/svgo/postcss/vite/esbuild version bumps and overrides: each maps to a listed advisory.

Revert PR: #382

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.

3 participants