Skip to content

build: use patched Go toolchain and UI dependency#2927

Merged
RossTarrant merged 3 commits into
mainfrom
loganrosen-prepare-v1-6-1-release
Jul 22, 2026
Merged

build: use patched Go toolchain and UI dependency#2927
RossTarrant merged 3 commits into
mainfrom
loganrosen-prepare-v1-6-1-release

Conversation

@loganrosen

@loganrosen loganrosen commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Ensures release binaries are built with Go 1.25.12 and removes the remaining high-severity UI dependency finding.

Why

Main's Docker builder was updated by #2867, but GoReleaser reads the exact Go version from go.mod, which remained at 1.25.0. That could leave future release archives built with an older, vulnerable Go standard library even though the container image was patched. The broader Go 1.26 upgrade in #2705 can proceed separately from this focused correction.

No linked issue. Related: #2705.

What changed

  • Raise the go.mod minimum from Go 1.25.0 to 1.25.12 so setup-go and GoReleaser use the patched toolchain.
  • Update transitive fast-uri from 3.1.2 to 3.1.4 to resolve its high-severity host-confusion advisories.

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

This only changes build inputs; MCP tools and behavior are unchanged.

Prompts tested (tool changes only)

  • N/A — no tool changes.

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

This is a build and dependency security update. It does not change runtime authentication, data exposure, filtering, or limits.

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Note: if you're renaming tools, you must add the tool aliases. For more information on how to do so, please refer to the official docs.

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Also verified govulncheck, license generation, UI typecheck/audit/build, GoReleaser snapshot artifacts, and amd64/arm64 container builds.

Docs

  • Not needed
  • Updated (README / docs / examples)

Ensure setup-go and GoReleaser use the patched Go toolchain for release binaries.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0b2aa497-b47d-464a-b132-af0dcaf2b621
Resolve the high-severity host-confusion advisories reported by npm audit.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0b2aa497-b47d-464a-b132-af0dcaf2b621
Copilot AI review requested due to automatic review settings July 22, 2026 15:10
@loganrosen
loganrosen requested a review from a team as a code owner July 22, 2026 15:10
@loganrosen loganrosen changed the title build: prepare v1.6.1 security fixes build: use patched Go toolchain and UI dependency Jul 22, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Prepares v1.6.1 security fixes by aligning release builds with patched Go 1.25.12 and updating the vulnerable UI dependency.

Changes:

  • Sets the Go minimum/toolchain input to 1.25.12.
  • Updates fast-uri from 3.1.2 to 3.1.4.
Show a summary per file
File Description
go.mod Pins release workflows to patched Go 1.25.12.
ui/package-lock.json Updates the transitive fast-uri dependency.

Review details

Files not reviewed (1)
  • ui/package-lock.json: Generated file
  • Files reviewed: 1/2 changed files
  • Comments generated: 0
  • Review effort level: Medium

@RossTarrant
RossTarrant merged commit d3cd405 into main Jul 22, 2026
19 checks passed
@RossTarrant
RossTarrant deleted the loganrosen-prepare-v1-6-1-release branch July 22, 2026 15:57
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