Skip to content

fix: resolve TypeScript build errors + add CI - #6

Merged
Hiksang merged 3 commits into
mainfrom
fix/build-errors
Mar 10, 2026
Merged

fix: resolve TypeScript build errors + add CI#6
Hiksang merged 3 commits into
mainfrom
fix/build-errors

Conversation

@Hiksang

@Hiksang Hiksang commented Mar 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Fix 3 TypeScript build errors that broke pnpm run build on main
    • alert.ts: replace non-existent getAccountInfo() with getBalance()
    • hyperliquid.ts: cast tif to HL SDK Tif literal type
    • pacifica.ts: cast tif to Pacifica TimeInForce literal type
  • Add GitHub Actions CI workflow (build check on push/PR to main)

Test plan

  • pnpm run build passes locally
  • GitHub Actions CI passes on this PR

🤖 Generated with Claude Code

Hiksang and others added 3 commits March 11, 2026 00:47
- alert.ts: replace non-existent getAccountInfo() with getBalance()
- hyperliquid.ts: cast tif string to HL SDK Tif literal type
- pacifica.ts: cast tif string to Pacifica TimeInForce literal type

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Runs pnpm install + tsc on every push to main and PR,
preventing build-breaking changes from being merged.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Hiksang
Hiksang merged commit 11d66e3 into main Mar 10, 2026
1 check passed
@Hiksang
Hiksang deleted the fix/build-errors branch March 10, 2026 15:57
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