Skip to content

fix(lint): daily lint fixes (2026-07-25)#701

Merged
aggre merged 1 commit into
mainfrom
fix/lint-20260725-1513
Jul 25, 2026
Merged

fix(lint): daily lint fixes (2026-07-25)#701
aggre merged 1 commit into
mainfrom
fix/lint-20260725-1513

Conversation

@aggre

@aggre aggre commented Jul 25, 2026

Copy link
Copy Markdown
Member

Daily lint fix

Cap: 14

Fixed

  • relay/server.ts: Remove 2 eslint-disable comments
    • @typescript-eslint/no-unsafe-returnJSON.parse(s) as unknown
    • functional/no-expression-statements → capture with const _ = R.ifElse(...)(import.meta.url)

Justification improvements

  • sdk/disclose.ts: Added explanation for Promise<void> WASM initialization awaits

Remaining

  • 108 existing disable comments remain (mostly imperative boundary code: Merkle tree, crypto, HTTP servers, wallet RPC, polyfills, LLM retry loops)
  • packages/web: 84 lint errors + 20+ type errors (excluded per policy)

Typecheck

  • All non-web packages: ✅ clean

@cursor

cursor Bot commented Jul 25, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
fetcher 47553c2 Commit Preview URL

Branch Preview URL
Jul 25 2026, 03:14 PM

@aggre

aggre commented Jul 25, 2026

Copy link
Copy Markdown
Member Author

Reviewed and approved. Lint-only changes, clean diff. Foundry rate-limit flake on first CI job is pre-existing infra issue (not code-related). Second test job passed. Merging.

@aggre
aggre merged commit 78161f9 into main Jul 25, 2026
2 of 3 checks passed
@aggre
aggre deleted the fix/lint-20260725-1513 branch July 25, 2026 20:02
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