Skip to content

fix(ci): verify Gitleaks release archives - #610

Merged
Lutar, Stephen P. (stephenlutar2-hash) merged 1 commit into
mainfrom
agent/harden-secret-scan-integrity-v3
Aug 13, 2026
Merged

fix(ci): verify Gitleaks release archives#610
Lutar, Stephen P. (stephenlutar2-hash) merged 1 commit into
mainfrom
agent/harden-secret-scan-integrity-v3

Conversation

@stephenlutar2-hash

Copy link
Copy Markdown
Member

Summary

  • Verify the official Gitleaks 8.21.2 Linux x64 archive before extraction.
  • Use one pinned SHA-256 across GitHub Actions, CircleCI, and the portal workflow.
  • Assert that each installed binary reports the expected version.
  • Make the portal workflow's optional config arguments shell-safe.
  • Align current secret-scanning docs with the active security.yml workflow.

Scope

  • Keep Gitleaks at version 8.21.2.
  • Keep the existing scanners, scan coverage, triggers, and permissions.
  • Keep all action references and immutable action pins unchanged.
  • Keep the root security fan-in job and its dependencies unchanged.
  • Make no dependency or lockfile changes.

Verification

  • git diff --check
  • strict YAML parse for all three changed CI files
  • Actionlint 1.7.12 and ShellCheck 0.11.0
  • Bash syntax checks for six relevant shell bodies
  • official SHA-256 manifest comparison
  • Linux download, checksum, extraction, install, and exact-version probe
  • Gitleaks: no unallowlisted findings in every changed tree
  • project secret scanner: clean; 15/15 tests passed
  • pnpm 10.26.1 typecheck: 182/182 tasks passed

Current-main lineage

This is the clean mapped-identity successor after #606 advanced main.
It is parented directly on e0cb7c61744fb64c15f59464e61c7fde15755e3a.
Its stable patch ID matches #607 and #609: fa769b46dfcfcf6023513282607d1a1b910ecd69.
#609 was rejected before merge because GitHub classified its local placeholder
committer identity as no_user. No published history was rewritten.

Evidence boundary

The nested portal workflow is not a root platform hosted workflow at its current path.
It is hardened as a tracked extraction/template surface.

Merge gate

Require a valid GitHub signature and DCO, terminal exact-head hosted checks,
resolved threads, and a normal protected squash merge.

Workcell: SECRET-SCAN-INTEGRITY

Supersedes: #609, #607
Follow-up to: #604

Verify the official Gitleaks 8.21.2 Linux x64 archive before extraction.

Use one pinned SHA-256 across GitHub Actions, CircleCI, and the portal workflow.
Make the portal optional config argument-safe without changing scan coverage.
Align current secret-scanning docs with the active security.yml workflow.

Validation:
- git diff --check
- strict YAML parse for three CI configs
- Actionlint and ShellCheck
- Linux checksum, extraction, and exact-version probe
- full project secret scan and 15 scanner tests
- pnpm 10.26.1 typecheck (182/182 tasks)

Workcell: SECRET-SCAN-INTEGRITY
Signed-off-by: Lutar, Stephen P. <stephenlutar2@gmail.com>

Copy link
Copy Markdown
Member Author

Codex (@codex) review

Exact-head release review requested for e64d69f5ef1f6064245f8ebf0a87294c6d540a5e. Please review the complete current diff against main after the hosted matrix is terminal.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: e64d69f5ef

ℹ️ 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 (@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 (@codex) address that feedback".

@stephenlutar2-hash
Lutar, Stephen P. (stephenlutar2-hash) merged commit 8745cf0 into main Aug 13, 2026
55 of 56 checks passed
@stephenlutar2-hash
Lutar, Stephen P. (stephenlutar2-hash) deleted the agent/harden-secret-scan-integrity-v3 branch August 13, 2026 11:12
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