Skip to content

chore(cli): remove unused @hono/zod-validator dependency - #11345

Merged
markijbema merged 1 commit into
mainfrom
chore/remove-unused-hono-zod-validator
Jun 17, 2026
Merged

chore(cli): remove unused @hono/zod-validator dependency#11345
markijbema merged 1 commit into
mainfrom
chore/remove-unused-hono-zod-validator

Conversation

@markijbema

Copy link
Copy Markdown
Contributor

What

Removes @hono/zod-validator from packages/opencode and its now-orphaned root catalog entry.

Why

Nothing in the repo imports @hono/zod-validator. Its peer dependency hono was removed in #10993, so keeping it would leave an unmet peer dependency for a package that is never used. opencode was the only consumer, so the catalog definition in the root package.json is dropped as well.

The lockfile change is limited to the four @hono/zod-validator entries — no transitive churn.

Follow-up to the review of #10993.

Nothing imports @hono/zod-validator anywhere in the repo, and its peer dependency hono was removed in #10993, leaving an unmet peer. opencode was the only consumer, so the root catalog entry is dropped too.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@kilo-code-bot

kilo-code-bot Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • bun.lock
  • package.json
  • packages/opencode/package.json

Reviewed by deepseek-v4-pro · 179,867 tokens

Review guidance: REVIEW.md from base branch main

@markijbema
markijbema enabled auto-merge June 17, 2026 09:10
@markijbema
markijbema merged commit e50cd89 into main Jun 17, 2026
21 checks passed
@markijbema
markijbema deleted the chore/remove-unused-hono-zod-validator branch June 17, 2026 09:12
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…ono-zod-validator

chore(cli): remove unused @hono/zod-validator dependency
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