chore(ci): remove unused docker version resolver

[NkwaTambe]

Summary

Removes the old Docker-publish version resolver (resolve-version.sh) and its regression test. The docker-publish workflow no longer uses a workflow_run trigger — release.yml now dispatches it directly with the version passed as an input — so this script (and the CI job that tested it) is dead code.

Related Issue

Closes #282 (contributes the remaining cleanup portion; the auto-publish and build-time work was already merged in commit 20f2c2e — "replace unreliable workflow_run trigger and eliminate QEMU bottleneck").

Intend

Issue #282 asks for a reliable, fast automatic Docker publish on release. The auto-trigger and native-ARM speedup are already merged. This PR completes the picture by removing the obsolete workflow_run-era machinery that nothing references anymore.

Changes

• Delete .github/scripts/resolve-version.sh (unused by any workflow).
• Delete tests/ci/resolve-version-test.sh (tested only the deleted script).
• Remove the shell-tests job from .github/workflows/ci.yml that ran that test.

Validation

• ci.yml re-parses as valid YAML after the edit.
• Confirmed via grep that no workflow or file references resolve-version anywhere else in the repo.
• No Rust code touched, so cargo fmt/clippy are not applicable.
• The affected behavior (auto-publish + version resolution) lives in release.yml/docker-publish.yml, which are unchanged by this PR.

Checklist

• My commit messages are descriptive and scoped to this change.
• This PR is focused on one issue or one coherent change.
• I ran cargo fmt or confirmed formatting was not applicable. (Not applicable — no Rust code changed.)
• I ran cargo clippy or explained why it was not applicable. (Not applicable — no Rust code changed.)
• I added or updated tests for new behavior or bug fixes, or explained why tests were not needed. (Deleted an obsolete test that exercised deleted code.)
• I updated documentation for user-facing, operational, or architectural changes. (None needed — this removes internal CI dead code.)
• I proposed an ADR or called out reviewer attention for architectural changes. (Not an architectural change.)

AI Usage Declaration

• What AI was used for: This change and PR text were produced with AI assistance. AI identified the dead code, performed the deletion and ci.yml edit, validated the result, and drafted this description.
• What was verified by a human (contributor): The contributor reviewed the final diff, confirmed the removed files were genuinely unreferenced (via grep) and that ci.yml remains valid YAML, and confirmed the change does not affect the GHCR publish path.
• Source of truth: Issue Auto-publish Docker image on release and reduce build time #282 — Auto-publish Docker image on release and reduce build time #282
• Verification evidence: git grep resolve-version → no matches; python3 -c "import yaml; yaml.safe_load(open('.github/workflows/ci.yml'))" → OK.

Notes for Reviewers

• The core of Auto-publish Docker image on release and reduce build time #282 (auto-publish trigger + native-ARM multi-arch build) was already merged; this PR is the follow-up cleanup of the now-unused workflow_run resolver. The remaining acceptance criteria (real merge to main producing the image, and a measured build-time figure) require an actual release and are out of scope for a code-only PR — please confirm that's acceptable to close Auto-publish Docker image on release and reduce build time #282.

Confidence Score: 5/5

The PR appears safe to merge; no outstanding publishing dependency on the removed resolver was found.

Summary

The PR removes an unused Docker version resolver, its regression test, and the CI job that ran the test. The current publishing workflow receives the release version directly.

_{Reviews (2) · Last reviewed commit: "Merge branch 'develop' into chore/remove..."}

[greptile-apps]

Your trial has ended. Reactivate Greptile to resume code reviews.

[Christiantyemele]

@greptileai review

[NkwaTambe]

https://github.com/greptileai review

[Christiantyemele]

@greptileai review