Skip to content

fix(deps): zip 9 へ移行し cargo group 更新を取り込む (#525) - #526

Merged
akiojin merged 1 commit into
developfrom
work/issue-525
Oct 19, 2026
Merged

akiojin merged 1 commit into
developfrom
work/issue-525

Conversation

@akiojin

@akiojin akiojin commented Oct 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • zip を 8 → 9 に移行し、Dependabot chore(deps): bump the cargo group with 5 updates #523 の cargo group 更新(serde_json 1.0.154 / tokio 1.53.3 / toml_edit 0.25.17 / sha2 0.11.1)を同時に取り込む
  • zip 9 で ZipFile::name() が Result<Cow<str>, ZipError> を返すようになったため、src/reference/fetcher.rs の extract_zip_to / detect_top_level_prefix で anyhow の context 付きエラーとして伝播(panic しない)

Supersedes #523

Closes #525

Notes / Assumptions

Verification (gwtd verify.run PASS: vrr-938ccd10a60148c8b1eeaefc10728e49)

  • cargo fmt --all -- --check ✅
  • cargo clippy --workspace --all-targets --all-features -- -D warnings ✅
  • RUSTDOCFLAGS="-D warnings" cargo doc --workspace --no-deps --document-private-items ✅
  • TMPDIR=/tmp cargo test --workspace --all-features -- --test-threads=1 ✅(606 unit + integration 全 pass)
    • verify ハーネスの長い TMPDIR では Unix socket パスが SUN_LEN を超えて unityd/lsp テストが失敗するため、短い TMPDIR を指定(コード起因ではない。直接実行の cargo test --all-targets -- --test-threads=1 も全 pass)
  • cargo run -- skills lint --severity error ✅

User Verification Result: n/a (autonomous)
Agent Visual Check: n/a (no UI surface)

🤖 Generated with Claude Code

Agent: ミク (Claude Code)

Supersedes Dependabot #523: zip 8 -> 9, serde_json 1.0.154,
tokio 1.53.3, toml_edit 0.25.17, sha2 0.11.1.

zip 9 returns Result from ZipFile::name(); propagate failures with
anyhow context instead of panicking.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 19, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 907c970f-bd54-494b-979b-3e1e18355df1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@akiojin
akiojin enabled auto-merge October 19, 2026 01:33
@akiojin
akiojin merged commit 1e36987 into develop Oct 19, 2026
19 checks passed
@akiojin
akiojin deleted the work/issue-525 branch October 19, 2026 01:51
@akiojin akiojin mentioned this pull request Oct 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.

1 participant