Skip to content

Revert "refactor: restructure repo into apps/services/packages monorepo"#2037

Closed
jeanduplessis wants to merge 1 commit intomainfrom
revert-1315-monorepo-refactor
Closed

Revert "refactor: restructure repo into apps/services/packages monorepo"#2037
jeanduplessis wants to merge 1 commit intomainfrom
revert-1315-monorepo-refactor

Conversation

@jeanduplessis
Copy link
Copy Markdown
Contributor

Reverts #1315

"files": [
"cloud-agent/**/*.ts",
"cloud-agent-next/**/*.ts",
"cloudflare-*/**/*.ts",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WARNING: cloudflare-gmail-push loses its lint exception here

This broadened override now applies the strict @typescript-eslint/no-unsafe-* rules to every cloudflare-* package, but the revert also removes the old gmail-push carve-out. cloudflare-gmail-push/src/routes/push.ts still types await emailRes.json() directly, and cloudflare-gmail-push/src/consumer.ts does the same for statusRes.json() and tokenRes.json(), so scripts/lint-all.sh will start failing as soon as this config lands.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot bot commented Apr 5, 2026

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
.oxlintrc.json 167 Removing the cloudflare-gmail-push lint override makes scripts/lint-all.sh fail because that package still relies on unsafe Response.json() typing.

Fix these issues in Kilo Cloud

Other Observations (not in diff)

None.

Files Reviewed (18 files)
  • .github/workflows/ci.yml - 0 issues
  • .oxlintrc.json - 1 issue
  • .gitignore - 0 issues
  • .kilocodeignore - 0 issues
  • .prettierignore - 0 issues
  • package.json - 0 issues
  • pnpm-workspace.yaml - 0 issues
  • scripts/changed-workspaces.sh - 0 issues
  • scripts/lint-all.sh - 0 issues
  • scripts/typecheck-all.sh - 0 issues
  • packages/db/package.json - 0 issues
  • packages/db/src/schema.test.ts - 0 issues
  • packages/trpc/package.json - 0 issues
  • packages/trpc/rollup.config.mjs - 0 issues
  • packages/trpc/src/index.ts - 0 issues
  • packages/trpc/tsconfig.json - 0 issues
  • storybook/package.json - 0 issues
  • storybook/tsconfig.json - 0 issues

Reviewed by gpt-5.4-20260305 · 4,932,673 tokens

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