Skip to content

Require kompo-vfs 0.7.0 - #67

Merged
ahogappa merged 1 commit into
masterfrom
claude/kompo-gem-path-generation-3okpzr
Aug 1, 2026
Merged

Require kompo-vfs 0.7.0#67
ahogappa merged 1 commit into
masterfrom
claude/kompo-gem-path-generation-3okpzr

Conversation

@ahogappa

@ahogappa ahogappa commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Bumps the required kompo-vfs version from 0.6.0 to 0.7.0.

Why this touches four places

KOMPO_VFS_MIN_VERSION is not only a floor. kompo_vfs_path.rb builds the release tarball URL and the install directory name from it, and CI checks out the same tag to build from source. Bumping it moves all three behaviours at once, so every site that pinned 0.6.0 has to move together.

File Change
lib/kompo/version.rb KOMPO_VFS_MIN_VERSION0.7.0
.github/workflows/ci.yml tag checked out before cargo build --releasev0.7.0
test/tasks/kompo_vfs_path_test.rb fixtures feeding verify!, which now raises on a 0.6.0 KOMPO_VFS_VERSION file
CHANGELOG.md Unreleased entry noting 0.6.x installs must be upgraded

Generated by Claude Code

Summary by CodeRabbit

  • Compatibility

    • Updated the minimum supported Kompo VFS version to 0.7.0.
    • Older Kompo VFS versions, including 0.6.0, are no longer accepted.
  • Documentation

    • Added upgrade guidance for Homebrew and source-built installations.
    • Updated the unreleased changelog with the new requirement.

@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The project raises its minimum supported kompo-vfs version from 0.6.0 to 0.7.0. CI, version checks, test fixtures, and the unreleased changelog now use the new requirement.

Changes

Kompo VFS Version Update

Layer / File(s) Summary
Version contract and validation
lib/kompo/version.rb, test/tasks/kompo_vfs_path_test.rb
The minimum version is now 0.7.0. Tests accept 0.7.0 and reject 0.6.0.
CI and release documentation
.github/workflows/ci.yml, CHANGELOG.md
CI checks out kompo-vfs v0.7.0. The changelog documents the requirement and upgrade commands.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Poem

A rabbit updates the version line,
From zero-six to zero-seven fine.
CI hops to the newer tag,
Tests guard the old one in a bag.
The changelog tells the upgrade tale.

🚥 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 primary change: requiring kompo-vfs version 0.7.0.
✨ 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 claude/kompo-gem-path-generation-3okpzr

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.

@ahogappa
ahogappa force-pushed the claude/kompo-gem-path-generation-3okpzr branch from 4a8c0fe to 04192f1 Compare August 1, 2026 02:05
KOMPO_VFS_MIN_VERSION is not only a floor: kompo_vfs_path.rb builds the release
tarball URL and the install directory name from it, and CI checks out the same
tag to build from source. So bumping it moves all three at once, and every place
that pinned 0.6.0 has to move together.

- lib/kompo/version.rb: the constant
- .github/workflows/ci.yml: the tag checked out before cargo build
- test/tasks/kompo_vfs_path_test.rb: fixtures feeding verify!, which would now
  raise on a 0.6.0 KOMPO_VFS_VERSION file

The too-old case now uses 0.6.0 rather than 0.5.1, so it pins that the bar
actually moved - it fails against the previous constant. The remaining 0.6.0
literals are deliberate: get_version tests whitespace stripping and
version_satisfies? tests comparison, neither of which reads the constant.

The cargo target cache key in ci.yml is left alone. Its "-v6-" marker does not
track the kompo-vfs version in the file's history, and editing ci.yml already
invalidates the key through hashFiles.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@ahogappa
ahogappa force-pushed the claude/kompo-gem-path-generation-3okpzr branch from 04192f1 to b25781e Compare August 1, 2026 02:09
@ahogappa
ahogappa merged commit a26b516 into master Aug 1, 2026
5 checks passed
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.

2 participants