Skip to content

fix(examples): scripting-smoke pins labelle-scripting v0.2.1 — the Lua mirror - #658

Merged
apotema merged 2 commits into
mainfrom
fix/scripting-smoke-pin-0.2.1
Aug 7, 2026
Merged

apotema merged 2 commits into
mainfrom
fix/scripting-smoke-pin-0.2.1

Conversation

@apotema

@apotema apotema commented Aug 7, 2026 •

Copy link
Copy Markdown
Contributor

lua.org is down and the Examples integration test has failed five consecutive runs on its single-sourced lua-5.4.8.tar.gz URL (see #656's checks).

The chain: labelle-scripting#61 moved the URL to a byte-identical mirror (release asset; zig fetch verified the same content hash, cross-checked against two independent distro mirrors). v0.2.1 is v0.2.0 + exactly that cherry-picked commit — no code changes, so this example's API surface is untouched and builds are byte-identical.

This PR's own Examples job is the proof: it fetches v0.2.1 and builds the declare tool through the mirror.

🤖 Generated with Claude Code

https://claude.ai/code/session_014pbwN1ytAa8tctZMaTMjAk


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Chores
    • Updated scripting smoke test configuration to reference labelle-scripting version 0.2.1.
    • Updated the accompanying capability information and dependency mirror references.
    • Aligned continuous integration smoke-test comments with the updated scripting plugin version.
    • No user-facing functionality or public API changes were introduced.

…a mirror

lua.org went down today and took the Examples integration test with it, five
runs straight: v0.2.0's manifest fetches lua-5.4.8.tar.gz from lua.org and
nowhere else. labelle-scripting#61 moved the URL to a byte-identical mirror
hosted as a release asset (content hash unchanged and verified, so builds are
identical); v0.2.1 is v0.2.0 plus exactly that commit, cherry-picked, so this
bump changes which server answers and nothing else.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014pbwN1ytAa8tctZMaTMjAk
@coderabbitai

coderabbitai Bot commented Aug 7, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8ceb17cc-0c9b-461e-bf5e-d652eb512554

📥 Commits

Reviewing files that changed from the base of the PR and between 278bacf and c2e5eff.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • examples/scripting-smoke/project.labelle
🚧 Files skipped from review as they are similar to previous changes (2)
  • .github/workflows/ci.yml
  • examples/scripting-smoke/project.labelle

📝 Walkthrough

Walkthrough

The scripting smoke workflow and example project now reference scripting plugin version 0.2.1 instead of 0.2.0. The workflow comments also reference the plugin’s GitHub-hosted Lua dependency mirror.

Changes

Scripting plugin update

Layer / File(s) Summary
Update scripting plugin references
.github/workflows/ci.yml, examples/scripting-smoke/project.labelle
The workflow comments and smoke project dependency now reference scripting plugin version 0.2.1. The workflow comments identify its GitHub-hosted Lua dependency mirror.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

I’m a rabbit with a version to cheer,
0.2.1 is hopping here.
The workflow aligns,
The smoke project shines,
With Lua dependencies clear!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the scripting-smoke example and the update to labelle-scripting v0.2.1 with the Lua mirror.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/scripting-smoke-pin-0.2.1

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

@coderabbitai coderabbitai 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/ci.yml:
- Line 398: Align all scripting dependency comments with v0.2.1: update the
comments at .github/workflows/ci.yml lines 372, 381, 398, and 424, change
.github/workflows/ci.yml line 402 to reference the GitHub Lua mirror instead of
lua.org, and update the corresponding comment in
examples/scripting-smoke/project.labelle line 44.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0159a320-4530-4adc-956d-46c0968e96d4

📥 Commits

Reviewing files that changed from the base of the PR and between 3bce493 and 278bacf.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • examples/scripting-smoke/project.labelle

Comment thread .github/workflows/ci.yml
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014pbwN1ytAa8tctZMaTMjAk
@apotema
apotema merged commit c8c2f27 into main Aug 7, 2026
6 checks passed
@apotema
apotema deleted the fix/scripting-smoke-pin-0.2.1 branch August 7, 2026 22:09
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