chore(deps-dev): Bump brace-expansion from 1.1.12 to 1.1.13

[dependabot]

Bumps brace-expansion from 1.1.12 to 1.1.13.

Commits

• 6c353ca 1.1.13
• 7fd684f Backport fix for GHSA-f886-m6hf-6m8v (#95)
• See full diff in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  You can disable automated security fix PRs for this repo from the Security Alerts page.

[claude]

CI Failure Analysis: CI

Run: #23666397460 | Commit: ffa14eb

Summary

Two jobs failed: Dead Code Check found 3 unused exported types, and Test passed all tests but crashed during coverage report generation due to a recursive directory copy (coverage → coverage/coverage).

Failures Found

Job	File	Issue
Dead Code Check	src/stages/3-execution/sdk-client.ts:51	Unused exported type PermissionMode
Dead Code Check	src/stages/3-execution/sdk-client.ts:72	Unused exported type SDKResultSuccess
Dead Code Check	src/stages/3-execution/sdk-client.ts:72	Unused exported type SDKResultError
Test	Vitest coverage reporter	EINVAL: cannot copy coverage to a subdirectory of self coverage/coverage

How to Fix

Dead code (knip): Remove or mark as used the 3 unused exported types in sdk-client.ts. If they are part of the public API, add them to the knip ignore config:

# Option A: Remove unused exports
# Edit src/stages/3-execution/sdk-client.ts and remove PermissionMode, SDKResultSuccess, SDKResultError exports

# Option B: Run knip to verify
npx knip

Test coverage crash: The Vitest UI reporter is trying to copy coverage/ into coverage/coverage/, creating a self-referencing loop. This is likely caused by the coverage output directory (reportsDirectory) being set to coverage/ while the Vitest UI outputDir also resolves to or includes coverage/. Fix by ensuring the @vitest/ui reporter output directory does not overlap with the coverage directory:

// vitest.config.ts — ensure coverage output doesn't conflict with UI output
coverage: {
  reportsDirectory: './coverage',  // check this doesn't nest inside itself
}

Also check knip.json — the hints suggest removing dist/**, coverage/**, and node_modules/** from the ignore array (these are already ignored by default).

---

Analyzed by Claude