Skip to content

fix: repair exact image index assembly - #23

Merged
ndizazzo merged 1 commit into
mainfrom
codex/fix-oci-index
Aug 8, 2026
Merged

fix: repair exact image index assembly#23
ndizazzo merged 1 commit into
mainfrom
codex/fix-oci-index

Conversation

@ndizazzo

@ndizazzo ndizazzo commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • fix the invalid jq scope in exact tested-image index assembly
  • execute the workflow's exact filter in regression tests
  • add an optional fail-closed expected upstream SHA input for manual recovery

Validation

  • full TypeScript suite: 93/93
  • focused workflow tests: 13/13
  • matrix validation
  • actionlint
  • git diff --check

v0.75.0 recovery

After merge, an image-only main dispatch will pin expected_mesh_sha=0ddaa87d4a24e5ec7c891afccf515748298b7027, set dry_run=false, publish images only, and consume checksum-verified v0.75.0 release artifacts without rebuilding MeshLLM source.

Summary by CodeRabbit

  • Bug Fixes

    • Improved release validation to prevent image releases from using an incorrect upstream revision.
    • Strengthened image-index checks to ensure all staged results match the selected release matrix.
    • Fixed exact image-index assembly issues that could affect published releases.
  • Tests

    • Added coverage for valid, invalid, mismatched, uppercase, and malformed revision values.
    • Added regression checks for image-index filtering and matrix consistency.

@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The release workflow now validates an optional immutable upstream tag SHA and uses explicit matrix-row binding for image-index checks. Regression tests cover SHA validation and mismatched staged image results.

Changes

Release workflow validation

Layer / File(s) Summary
Expected upstream SHA validation
.github/workflows/images-release.yml, tests/release-workflow.test.ts, TODO.md
The workflow accepts an optional lowercase 40-character SHA and verifies it against the resolved upstream tag commit. Tests cover empty, matching, uppercase, malformed, and mismatched values.
Image-index matrix validation
.github/workflows/images-release.yml, tests/release-workflow.test.ts
The jq filter binds each matrix row before comparing it with staged results. Tests verify matching rows pass and backend mismatches fail.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: repairing exact image index assembly.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/fix-oci-index

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ndizazzo
ndizazzo merged commit 1b47fef into main Aug 8, 2026
2 checks passed
@ndizazzo
ndizazzo deleted the codex/fix-oci-index branch August 8, 2026 07:44
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