Skip to content

fix: resolve TypeScript strict errors for Docker build#126

Merged
rz1989s merged 1 commit intomainfrom
fix/docker-build
Apr 9, 2026
Merged

fix: resolve TypeScript strict errors for Docker build#126
rz1989s merged 1 commit intomainfrom
fix/docker-build

Conversation

@rz1989s
Copy link
Copy Markdown
Member

@rz1989s rz1989s commented Apr 9, 2026

Summary

Docker build fails on tsc --noEmit in the agent package. Fixes:

  • Add @anthropic-ai/sdk to agent package.json (was only root-level)
  • Revert as Tool['parameters']as any (TypeBox branded type incompatible with plain JSON Schema)
  • Type annotate implicit any params in agent.ts filter callbacks
  • Fix null vs undefined in activity-logger wallet field
  • Add unknown intermediate cast for PaymentLinkStatsResult
  • Handle Express 5 string | string[] params
  • Clamp sentinel network to valid Cluster values
  • Fix pi-ai tool adapter optional description

Test plan

  • tsc --noEmit passes cleanly
  • 573 tests passing
  • CI Docker build succeeds

- Add @anthropic-ai/sdk to agent package.json (was only in root)
- Revert HERALD tool parameter casts from `as Tool['parameters']` to
  `as any` — TypeBox TSchema is a branded type incompatible with plain
  JSON Schema objects
- Type the `b` parameter in content.filter() callbacks in agent.ts
  to satisfy noImplicitAny under strict mode
- Fix null vs undefined mismatch in activity-logger wallet field
- Add double-cast via unknown for PaymentLinkStatsResult → Record
- Narrow Express 5 req.params values (string | string[]) to string
- Clamp sentinel scanner network to SDK's Cluster type (no 'testnet')
- Widen AnthropicTool.description to optional for pi-ai Tool compat
@rz1989s rz1989s merged commit 158308d into main Apr 9, 2026
3 checks passed
@rz1989s rz1989s deleted the fix/docker-build branch April 10, 2026 04:04
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