Skip to content

chore(release): v0.11.4#215

Merged
akiojin merged 8 commits into
mainfrom
develop
Jun 2, 2026
Merged

chore(release): v0.11.4#215
akiojin merged 8 commits into
mainfrom
develop

Conversation

@akiojin

@akiojin akiojin commented Jun 2, 2026

Copy link
Copy Markdown
Owner

Summary

v0.11.4 リリース。前回リリース v0.11.3 以降の CI 安定化・Git hooks 整備・skill ドキュメント修正をまとめた patch リリースです。機能追加 (feat) はありません。

Changes

  • fix(ci): cargo test--test-threads=1 で実行し、環境変数 race による test の不安定さを解消
  • chore(hooks): husky を auto-install し、commit-msg 検証を CI と一致させる
  • chore(hooks): pre-push を fmt check のみに絞り、push を高速化
  • docs(skills): unity-asset-management skill に有効な get_asset_info action を明記 (unity-asset-management skill uses an invalid manage_asset_database action in its preferred flow #206)
  • docs(lessons): gwt-spec の集約 close 例外条項を追記

Version

v0.11.4 (patch)

Closing Issues

Closes #206

Related Issues / Links

Summary by CodeRabbit

🎉 リリースノート v0.11.4

  • Bug Fixes

    • CI テスト実行の安定性を向上(スレッド競合による失敗を解消)
  • Documentation

    • Unity Asset Management スキルドキュメントを更新
    • テスト実行手順を明確化
  • Chores

    • バージョンを 0.11.4 に更新
    • ローカルビルド前チェックを簡素化
    • 依存関係を更新

akiojin and others added 8 commits May 13, 2026 09:41
別の canonical な gwt-spec に統合されて canonical 性が新 SPEC に
移行した旧 SPEC は close 可、という例外を「正式な gwt-spec を close
しない」rule に追記。今日 Phase 分割の #185 / #187 / #188 / #191 /
#192 をドメイン SPEC #204 に集約した運用に対応する。

Refs #204

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
pnpm install 時に husky で core.hookspath=.husky/_ を自動セット
アップし、ローカル commit でも .husky/commit-msg 経由で commitlint
が走るようにする。これまでは .husky/ ディレクトリは存在するのに
husky が install されておらず hookspath が未設定で、ローカルでは
footer-max-line-length 等の違反を CI でしか検知できなかった
(PR #205 で実害発生)。

Refs #204
cargo test / dotnet test を pre-push から削除し、format check だけ
残す。重いテストは CI に任せて push loop を高速化し、既存の並列実行
race condition (flaky integration tests) で push がブロックされる
事象を避ける。

Refs #204
Rust の std::env::set_var は process-global で thread-unsafe、
複数 test が UNITY_CLI_REGISTRY_PATH / HOME を mutate すると
library 内部の getenv 読み出しと衝突して flaky に失敗していた
(PR #205 で `write_pid_and_cleanup_use_temp_home` / `execution_
context_uses_defaults` / `run_with_cli_set_active_text_output_
when_reachable` が intermittent に fail)。

CI workflow と CLAUDE.md の品質ゲートを cargo llvm-cov 側に既に
存在した --test-threads=1 と同じ運用に揃え、test 実行を serialize
する。tasks/lessons.md に再発防止 rule を追加。

Refs #204
docs(lessons): gwt-spec の集約 close 例外条項を追加
Preferred Flow の "Inspect the target asset" 手順が `manage_asset_database`
の有効な action を示しておらず、例も `{"action":"refresh"}` のみだったため、
agent/user が未サポートの `inspect` action を推測しやすかった。

- 手順1の文言に `{"action":"get_asset_info","assetPath":"..."}` を明記
- コード例に get_asset_info の呼び出しを追加
- metadata.version 0.3.0 -> 0.3.1

有効な action は src/tooling/tool_catalog.rs と AssetDatabaseHandler.cs で
get_asset_info (assetPath 必須) を確認済み。skills lint --severity error は
0 violations。

Closes #206

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
docs(skills): unity-asset-management の get_asset_info action を明記 (#206)
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@akiojin akiojin added the release Release PR label Jun 2, 2026
@akiojin
akiojin enabled auto-merge June 2, 2026 11:36
@akiojin
akiojin merged commit c448944 into main Jun 2, 2026
15 of 16 checks passed
@coderabbitai

coderabbitai Bot commented Jun 2, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 0b38c172-814d-4955-a162-3bd81fd8275e

📥 Commits

Reviewing files that changed from the base of the PR and between cf84cdc and 41fa9d5.

⛔ Files ignored due to path filters (2)
  • Cargo.lock is excluded by !**/*.lock
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (9)
  • .claude-plugin/plugins/unity-cli/skills/unity-asset-management/SKILL.md
  • .github/workflows/test.yml
  • .husky/pre-push
  • CHANGELOG.md
  • CLAUDE.md
  • Cargo.toml
  • UnityCliBridge/Packages/unity-cli-bridge/package.json
  • package.json
  • tasks/lessons.md

Walkthrough

Release version 0.11.4 bumps package versions, fixes CI test reliability by serializing test execution with --test-threads=1, removes local test execution from pre-push hooks, clarifies the unity-asset-management skill documentation with explicit get_asset_info action, and adds corresponding changelog and operational documentation.

Changes

Version 0.11.4 Release

Layer / File(s) Summary
Manifest version updates
Cargo.toml, package.json, UnityCliBridge/Packages/unity-cli-bridge/package.json
Crate and package versions incremented from 0.11.3 to 0.11.4.
CI test reliability and single-threaded execution
.github/workflows/test.yml, CLAUDE.md
Cargo test and cargo llvm-cov steps now run with --test-threads=1 to prevent environment-related race conditions; quality gate procedure documentation updated to match.
Pre-push hook optimization and husky dependency update
.husky/pre-push, package.json
Pre-push hook simplified to run only cargo fmt --all -- --check instead of full test suite; husky dependency updated to ^9.1.7; comment clarifies that heavy tests are CI responsibility.
Unity asset management skill documentation clarification
.claude-plugin/plugins/unity-cli/skills/unity-asset-management/SKILL.md
Skill version bumped to 0.3.1; preferred flow step 1 now explicitly requires manage_asset_database with {"action":"get_asset_info","assetPath":"..."} JSON payload and provides matching bash example.
Changelog and operational documentation
CHANGELOG.md, tasks/lessons.md
CHANGELOG.md records v0.11.4 with bug fixes, documentation updates, and miscellaneous tasks; tasks/lessons.md refines gwt-spec closure policy to permit closing old specs only when consolidated into canonical successors with cross-reference in final comment.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related issues

  • #206: unity-asset-management skill uses an invalid manage_asset_database action in its preferred flow — This PR directly addresses the issue by updating the skill documentation to specify the correct get_asset_info action with explicit JSON payload format and concrete example.

Possibly related PRs

  • akiojin/unity-cli#214: Both PRs modify the same unity-asset-management SKILL.md preferred-flow step to require manage_asset_database with {"action":"get_asset_info","assetPath":"..."} and bump version to 0.3.1, alongside overlapping CI/hook/test-threading doc changes.

Poem

🐰 Version bumps and racing threads,
Format checks keep CI thread-safe beds,
Asset actions now crystal clear,
Release 0.11.4 draws near! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

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.

arghhhhh pushed a commit to arghhhhh/unity-cli that referenced this pull request Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release Release PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

unity-asset-management skill uses an invalid manage_asset_database action in its preferred flow

1 participant