Conversation
Bumps the go-dependencies group with 1 update: [github.com/mattn/go-shellwords](https://github.com/mattn/go-shellwords). Updates `github.com/mattn/go-shellwords` from 1.0.12 to 1.0.13 - [Commits](mattn/go-shellwords@v1.0.12...v1.0.13) --- updated-dependencies: - dependency-name: github.com/mattn/go-shellwords dependency-version: 1.0.13 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: go-dependencies ... Signed-off-by: dependabot[bot] <support@github.com>
|
Claude finished @dependabot[bot]'s task in 1m 38s —— View job Dependency Review:
|
| Commit | Change | Risk |
|---|---|---|
f6737fe |
Parse \t as TAB instead of escaped t |
Behavior change — but not used in this repo |
e2951fc |
Fix dollarQuote state management | Bug fix |
b074fa0 + 9a78803 |
Treat bare ) as syntax error (CVE fix) |
Behavior change — safe for this repo (see below) |
551a1d0 |
Update CI (Go 1.25/1.26) | No code impact |
fd1aa6c |
gofmt formatting | No code impact |
Usage in This Codebase
go-shellwords is used in exactly one place: cmd/kosli/testHelpers.go:46
args, err := shellwords.Parse(cmd)This is test-only infrastructure — it splits CLI command strings in cmdTestCase structs into argument arrays for test execution. It is not used in production code paths.
The code uses the package-level Parse() function (default parser: ParseBacktick=false, ParseEnv=false).
Risk Assessment
Low risk. Here's why:
-
\tbehavior change: No test command strings in the repo contain literal\tescape sequences. Verified via grep — zero matches across all*_test.gofiles. -
Bare
)syntax error: The one test case that includes)in a command string (createAttestationType_test.go:57) uses it inside single quotes ('.name | startswith("B")'), so shellwords treats it as a literal character — not affected by the bare)change. -
Test-only dependency: Even if a behavior change did surface, it would manifest as a test failure, not a production bug.
-
Security fix included: The CVE fix (
9a78803) for unmatched)handling is a good reason to take this update.
Verdict
Safe to merge. This is a minor-version patch with security fixes, and the one place this library is used in the codebase is unaffected by the behavior changes. No follow-up action needed.
Bumps the go-dependencies group with 1 update: [github.com/mattn/go-shellwords](https://github.com/mattn/go-shellwords). Updates `github.com/mattn/go-shellwords` from 1.0.12 to 1.0.13 - [Commits](mattn/go-shellwords@v1.0.12...v1.0.13) --- updated-dependencies: - dependency-name: github.com/mattn/go-shellwords dependency-version: 1.0.13 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: go-dependencies ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the go-dependencies group with 1 update: github.com/mattn/go-shellwords.
Updates
github.com/mattn/go-shellwordsfrom 1.0.12 to 1.0.13Commits
fd1aa6cRun gofmt: add missing //go:build directives and trailing newlinese73986eTreat bare ')' as syntax error regardless of ParseBacktick9a78803Merge pull request #60 from scumfrog/security-fix-cveb074fa0fix: preserve parser compatibility for unmatched ')' handling735b5e8Implement tests for shellwords parser functionalitye2951fcFix dollarQuote state management in shellwords.go551a1d0Update CI: Go 1.25/1.26 and latest GitHub Actionsf3bbb6fMerge pull request #53 from ndeloof/masterf6737feparse\tas TAB, not escapedtDependabot 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 rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions