Skip to content

Harden release and packaging supply chain - #16

Merged
rodrigoantonioli merged 1 commit into
mainfrom
codex/release-security-hardening
Aug 11, 2026
Merged

Harden release and packaging supply chain#16
rodrigoantonioli merged 1 commit into
mainfrom
codex/release-security-hardening

Conversation

@rodrigoantonioli

Copy link
Copy Markdown
Owner

Summary

  • validate release tags as data and separate unprivileged builds from protected publication
  • pin GitHub Actions, Python build dependencies, Inno Setup, and WingetCreate to verified immutable inputs
  • redact user-profile prefixes from copied diagnostics and document the release security invariants

Validation

  • ruff check .
  • pytest -q (107 passed)
  • actionlint
  • clean hash-locked installs in disposable Python 3.13 environments
  • local EXE/Inno build
  • unsigned MSIX pack/unpack and repeat-build cleanup
  • verified official Inno Setup and WingetCreate hashes and Authenticode signatures without executing them locally

Safety

No version, tag, GitHub Release, WinGet submission, certificate, or Microsoft Store state was changed.

@rodrigoantonioli
rodrigoantonioli merged commit 6aee461 into main Aug 11, 2026
3 checks passed

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7504f32f2b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


$ErrorActionPreference = "Stop"

$ReleaseTagPattern = '^v(?:0|[1-9]\d*)\.(?:0|[1-9]\d*)\.(?:0|[1-9]\d*)(?:-beta(?:[.-]?(?:0|[1-9]\d*))?)?$'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restrict release-tag digits to ASCII

In PowerShell/.NET regexes, \d matches Unicode decimal digits, so a tag such as v1.1٢.0 passes this supposedly strict validator. The workflow then treats it as a stable release and can build/publish an installer before WinGet or other version consumers reject the non-ASCII version; use [0-9] throughout the pattern and add a Unicode-digit rejection test.

AGENTS.md reference: AGENTS.md:L260-L261

Useful? React with 👍 / 👎.

@rodrigoantonioli
rodrigoantonioli deleted the codex/release-security-hardening branch August 11, 2026 14:04
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.

1 participant