Skip to content

fix: remove trpc dist index#1660

Merged
catrielmuller merged 1 commit intomainfrom
catrielmuller/remove-trpc-dist
Mar 27, 2026
Merged

fix: remove trpc dist index#1660
catrielmuller merged 1 commit intomainfrom
catrielmuller/remove-trpc-dist

Conversation

@catrielmuller
Copy link
Copy Markdown
Contributor

No description provided.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot bot commented Mar 27, 2026

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0

Fix these issues in Kilo Cloud

Issue Details (click to expand)

No inline comments were posted because the PR only deletes packages/trpc/dist/index.d.ts, and GitHub does not allow comments on deleted lines.

Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
packages/trpc/package.json 12 exports["."].types still points at ./dist/index.d.ts, so removing the committed declaration bundle makes imports like kilo-app/src/lib/trpc.ts fail to resolve @kilocode/trpc in clean checkouts and in package-local workflows such as pnpm --filter kilo-app typecheck.
Files Reviewed (1 files)
  • packages/trpc/dist/index.d.ts - 1 issue

Reviewed by gpt-5.4-20260305 · 482,007 tokens

@catrielmuller catrielmuller force-pushed the catrielmuller/remove-trpc-dist branch from 6e908ba to 96d78bd Compare March 27, 2026 18:42
@catrielmuller catrielmuller enabled auto-merge March 27, 2026 18:42
@catrielmuller catrielmuller merged commit 8e48569 into main Mar 27, 2026
26 checks passed
@catrielmuller catrielmuller deleted the catrielmuller/remove-trpc-dist branch March 27, 2026 18:47
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