Skip to content

fix: align harness.toml version with VERSION/plugin.json (4.13.1) - #178

Merged
Chachamaru127 merged 1 commit into
mainfrom
claude/fix-harness-toml-version-413
May 29, 2026
Merged

Chachamaru127 merged 1 commit into
mainfrom
claude/fix-harness-toml-version-413

Conversation

@Chachamaru127

@Chachamaru127 Chachamaru127 commented May 29, 2026 •

Copy link
Copy Markdown
Owner

Problem

The v4.13.1 release (PR #176) bumped VERSION and .claude-plugin/plugin.json to 4.13.1 but left harness.toml at 4.13.0. The project versioning rules require all three to stay in sync.

Because harness sync (scripts/sync-plugin-cache.sh → bin/harness sync) regenerates plugin.json's version from harness.toml, every sync reverts plugin.json to 4.13.0. The CI validate job runs validate-plugin.sh (which triggers the sync) and then check-consistency.sh, whose version-parity gate then fails on VERSION=4.13.1 vs plugin.json=4.13.0.

This is why the validate job is currently red on main and on every open PR branched from it (e.g. #177, #167).

Fix

Set harness.toml version to 4.13.1, restoring the 3-file sync.

Verification

# after the fix:
bash scripts/sync-plugin-cache.sh   # plugin.json stays 4.13.1 (no revert)
git diff --stat                     # only harness.toml changed
bash scripts/ci/check-consistency.sh  # ✅ すべてのチェックに合格しました

No code behavior change — version metadata only. Recorded under CHANGELOG.md [Unreleased].

Once this merges, open PRs whose validate was failing on this gate (#177, #167) should go green after a rebase on main.

https://claude.ai/code/session_01MNT1hAQw9AzGapXdEA3BFp


Generated by Claude Code

Summary by CodeRabbit

リリースノート

  • Bug Fixes
    • バージョン管理における整合性の問題を修正しました。

バージョン: 4.13.1

Review Change Stack

The v4.13.1 release bumped VERSION and .claude-plugin/plugin.json to
4.13.1 but left harness.toml at 4.13.0. Because 'harness sync' regenerates
plugin.json's version from harness.toml, every sync reverted plugin.json
to 4.13.0, which made the CI 'validate' job fail at the check-consistency
version-parity gate. Restore the 3-file version sync required by the
project versioning rules.

https://claude.ai/code/session_01MNT1hAQw9AzGapXdEA3BFp
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

@coderabbitai

coderabbitai Bot commented May 29, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 70718ca5-04b9-4c3e-9658-1db30aa67204

📥 Commits

Reviewing files that changed from the base of the PR and between 887cec7 and 3938007.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • harness.toml

Walkthrough

harness.toml のプロジェクトバージョンを 4.13.0 から 4.13.1 に更新し、VERSION/.claude-plugin/plugin.json との同期ずれを解決しました。CHANGELOG.md に修正内容を記載し、harness sync によるバージョン巻き戻りと CI 検証失敗の経緯を説明しています。

Changes

バージョン同期修正

Layer / File(s) Summary
Version bump and fix documentation
harness.toml, CHANGELOG.md
harness.toml の [project].version を 4.13.1 に更新し、VERSION/plugin.json との同期ずれ(harness sync による巻き戻りと CI validate 失敗)を修正。CHANGELOG.md の [Unreleased] セクションに修正内容を記載。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 Version numbers dance in harmony,
Four-point-one-three-one they now agree,
No more rollbacks on harness sync—
Changelog marks the fix, quick as a wink! ✨

🚥 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 PRのタイトルは、harness.tomlのバージョンを4.13.1に揃える修正という変更内容を正確かつ簡潔に表現している。
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/fix-harness-toml-version-413

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 and usage tips.

@Chachamaru127
Chachamaru127 merged commit 0737016 into main May 29, 2026
9 checks passed
Chachamaru127 added a commit that referenced this pull request May 29, 2026
Resolve conflicts:
- CHANGELOG [Unreleased]: union of Opus 4.8 tuning (Changed), scaffolder
  removal #170 (Removed), and #171/#178 (Fixed).
- opus-4-7-prompt-audit.md: drop agents/scaffolder.md glob (#170 removed
  the agent), keep docs/team-composition.md path fix.

Rebuild binaries from merged source (main guardrail fix #171 + opus-4-8
allowlist). agents now 3 (scaffolder removed).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Chachamaru127 added a commit that referenced this pull request May 29, 2026
Conflict resolution:
- Plans.md: Phase 82 → 86 (Harness Duplication Cleanup), Phase 83 → 87
  (Cursor Internal-Compatible Promotion). Renumbered to avoid collision
  with the archived Phase 80-84 numbering space (main is at 4.13.1 +
  PR #180 archive).
- CHANGELOG.md: integrated Removed (#170 scaffolder) + Fixed (#171 no-verify
  bypass, #178 harness.toml drift) from main, plus PR #174's Changed
  entries with Phase 86/87 references.
- docs/research/cursor-adapter-candidate.md: adopted origin/main content
  (PR #180's network smoke / write-capability / containment / ACP not
  adopted research).
- docs/bootstrap-routing-contract.md:119: "Phase 83" → "Phase 87".
- spec.md:3: "Phase 72 through Phase 83" → "Phase 72 through Phase 87".

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Chachamaru127 added a commit that referenced this pull request May 30, 2026
Cursor (Composer) 委譲経路の整備 + breezing UX 軽量化 を release。
Cursor support tier は internal-compatible のまま不変、正式機能としては
まだ実験段階のため patch bump。

主な内容:
- /cursor:ask: Cursor 読取専用デリゲート (worktree 不要、3 秒以内応答)
- /cursor:do: 1 タスク write 委譲 (専用 worktree + Lead diff + cherry-pick)
- /harness-review --cursor: cursor を second-opinion として並走 (Opus reviewer 必須併走)
- /breezing UX 軽量化 (Narration Rules + Cursor Backend Fast Path、baseline 15-35s → target 3-7s)
- .claude/rules/cursor-cli-only.md に Read mode delegation (lean path) section 追加
- Opus 4.8 向けプロンプトチューニング (effort tier 統一、code review coverage 優先)
- Scaffolder agent 削除 (#170)、--no-verify bypass fix (#171)、harness.toml drift fix (#178)

VERSION / plugin.json / harness.toml / marketplace.json / codex / cursor
plugin の 7 ファイル同期 (./scripts/sync-version.sh bump patch)。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This was referenced May 30, 2026
Chachamaru127 added a commit that referenced this pull request May 30, 2026
Cursor (Composer) 委譲経路の整備 + breezing UX 軽量化 を release。Cursor support tier は internal-compatible のまま不変、正式機能としてはまだ実験段階のため patch bump。

主な内容:
- Added: /cursor:ask (読取専用、worktree 不要、3 秒応答) / /cursor:do (1 タスク write 委譲、worktree + Lead diff + cherry-pick)
- Changed: /harness-review --cursor (second-opinion only) / /breezing --cursor --reviewer-only (read=lean) / /breezing Narration Rules + Cursor Fast Path (15-35s → 3-7s) / cursor-cli-only.md Read mode section / Opus 4.8 prompt tuning
- Removed: Scaffolder agent #170 (unused)
- Fixed: R10 guardrail shell metachar bypass #171, harness.toml drift #178

VERSION/plugin.json/harness.toml/marketplace.json/codex/cursor の 7 ファイル同期。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Chachamaru127 Chachamaru127 mentioned this pull request May 30, 2026
3 tasks done
Chachamaru127 added a commit that referenced this pull request May 30, 2026
Cursor (Composer) 委譲経路の整備 + breezing UX 軽量化 を release。
Cursor support tier は internal-compatible のまま不変、正式機能としては
まだ実験段階のため patch bump。

主な内容:
- /cursor:ask: Cursor 読取専用デリゲート (worktree 不要、3 秒以内応答)
- /cursor:do: 1 タスク write 委譲 (専用 worktree + Lead diff + cherry-pick)
- /harness-review --cursor: cursor を second-opinion として並走 (Opus reviewer 必須併走)
- /breezing UX 軽量化 (Narration Rules + Cursor Backend Fast Path、baseline 15-35s → target 3-7s)
- .claude/rules/cursor-cli-only.md に Read mode delegation (lean path) section 追加
- Opus 4.8 向けプロンプトチューニング (effort tier 統一、code review coverage 優先)
- Scaffolder agent 削除 (#170)、--no-verify bypass fix (#171)、harness.toml drift fix (#178)

VERSION / plugin.json / harness.toml / marketplace.json / codex / cursor
plugin の 7 ファイル同期 (./scripts/sync-version.sh bump patch)。

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This was referenced Jun 19, 2026
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