Skip to content

fix: resolve Dependabot security alerts - #1623

Merged
thymikee merged 1 commit into
mainfrom
agent/fix-dependabot-alerts
Aug 5, 2026
Merged

fix: resolve Dependabot security alerts#1623
thymikee merged 1 commit into
mainfrom
agent/fix-dependabot-alerts

Conversation

@thymikee

@thymikee thymikee commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

Upgrade the vulnerable transitive dependencies reported by Dependabot:

  • undici to 7.29.0
  • fast-uri to 3.1.5
  • postcss to 8.5.23
  • qs to 6.15.2
  • brace-expansion to 5.0.9

The root and test-app pnpm workspace overrides and lockfiles now resolve all 10 open Dependabot alerts. No runtime source behavior changed; the scope is limited to four dependency configuration/lockfiles.

Validation

Frozen-lockfile checks passed for both workspaces. Formatting, lint, typecheck, and layering checks passed.

The full local aggregate remains environment-blocked: Fallow could not create its temporary base worktree, and the unit aggregate reported 80 failures caused by restricted socket binding, protected Apple/Xcode cache paths, and unavailable registry access.

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-08-05 20:58 UTC

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 1.97 MB 1.97 MB +1.2 kB
JS gzip 631.3 kB 631.7 kB +339 B
npm tarball 761.3 kB 761.6 kB +259 B
npm unpacked 2.67 MB 2.67 MB +1.2 kB

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 28.4 ms 28.7 ms +0.3 ms
CLI --help 68.5 ms 65.5 ms -3.0 ms

Top changed chunks:

Chunk Raw diff Gzip diff
dist/src/internal/daemon.js +836 B +265 B
dist/src/sdk-batch-runner.js -53 B -51 B
dist/src/session.js +58 B +34 B
dist/src/screenshot-geometry.js +103 B +31 B
dist/src/viewport-dimension.js +163 B +21 B

@thymikee

thymikee commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

Reviewed exact head 3c5c3b9a. The five overrides resolve all 10 current Dependabot alerts to their required patched versions across the root and test-app workspaces. The lockfile peer-instance churn is mechanically consistent with those resolutions; frozen-lockfile, package-build, static, coverage, integration, CodeQL, and platform smoke checks are all green. No source/API behavior change and no code finding. Mergeable and technically ready; the PR remains marked draft.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Aug 5, 2026
@thymikee
thymikee marked this pull request as ready for review August 5, 2026 20:58
@thymikee
thymikee merged commit 9dd1cdd into main Aug 5, 2026
30 checks passed
@thymikee
thymikee deleted the agent/fix-dependabot-alerts branch August 5, 2026 20:58
thymikee added a commit to szdziedzic/agent-device that referenced this pull request Aug 6, 2026
* origin/main:
  chore: Update GitHub Sponsors usernames in FUNDING.yml
  fix: resolve Dependabot security alerts (callstack#1623)
  fix: update MCP registry namespace (callstack#1618)
  fix(ios): corroborate recorded tap outcomes (callstack#1605)
  fix(ios): never signal a recycled runner pid from a stale lease (callstack#1621)
  build: eliminate tsdown bundle warnings (callstack#1607)
  refactor(contracts): one viewport-root predicate for the whole repo (callstack#1613)
  refactor(contracts): name façade exports explicitly and retire the pin table (callstack#1614)
  refactor(ios): share one private-XCTest event bridge between gesture and text synthesis (callstack#1608)
  refactor(daemon): give the Maestro fallback and ambiguous-match details real types (callstack#1612)
  docs: clarify iOS drag synthesis profiles (callstack#1616)

# Conflicts:
#	scripts/layering/facade-symbols.ts
#	src/commands/capture/screenshot.ts
#	src/commands/recording/index.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant