Skip to content

chore(deps): bump SocketDev/socket-basics to v3.1.0 - #2

Merged
william-u10d merged 1 commit into
mainfrom
chore/bump-socket-basics-v3.1.0
Sep 3, 2026
Merged

chore(deps): bump SocketDev/socket-basics to v3.1.0#2
william-u10d merged 1 commit into
mainfrom
chore/bump-socket-basics-v3.1.0

Conversation

@william-u10d

@william-u10d william-u10d commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Bumps the pinned Socket Basics digest from 6b71cdf (v3.0.0) to 5b615c8 (v3.1.0).

Why now

v3.1.0 shipped 2026-09-03. It is a minor, backwards-compatible release, but two of its fixes bear directly on how this workflow invokes the action:

  • #105changed_files resolved to zero files inside the Docker container action due to a git dubious-ownership error.
  • #98changed_files was not honored from every config source, and failed silently when it wasn't.

Both bugs can cause a scan to quietly cover nothing while still reporting success. Because this workflow runs continue-on-error: true and is informational-only, a silent no-op scan would be invisible here. That makes this a correctness fix for PR-scoped scanning, not just a version refresh.

Also included upstream: pr_comment_enabled / pr_comment_collapse_all PR-comment controls (#97) — not adopted in this PR, no input changes.

Full upstream diff: SocketDev/socket-basics@v3.0.0...v3.1.0

Verification

  • Digest 5b615c89d17764ab46de2f2de63122ca2ef6a4be confirmed via the GitHub API to be the chore(release): 3.1.0 commit carrying the v3.1.0 tag in SocketDev/socket-basics.
  • No breaking changes in the release notes; no action inputs added, removed, or renamed, so the with: block is unchanged.
  • Workflow YAML re-parsed clean.

Scope

One line. No change to triggers, permissions, the fork-PR guard, or the audit-logging step.

Review in cubic

Bumps the pinned digest from 6b71cdf (v3.0.0) to 5b615c8 (v3.1.0).

v3.1.0 is a minor, backwards-compatible release. Notable fixes relevant
to how this workflow invokes the action:

- changed_files resolved to zero files inside the Docker container action
  because of a git dubious-ownership error (SocketDev/socket-basics#105)
- changed_files was not honored from every config source, and failed
  silently when it was not (SocketDev/socket-basics#98)

Both bugs could cause the scan to silently cover no files, so this is a
correctness fix for PR-scoped scanning, not just a version refresh.

Digest verified against the v3.1.0 tag via the GitHub API.
@william-u10d
william-u10d marked this pull request as ready for review September 3, 2026 14:14

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Shadow auto-approve: would not auto-approve. This PR does not meet the repository auto-approval settings.

Re-trigger cubic

@william-u10d
william-u10d merged commit 36203d8 into main Sep 3, 2026
4 checks passed
@william-u10d
william-u10d deleted the chore/bump-socket-basics-v3.1.0 branch September 3, 2026 15:35
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.

2 participants