Skip to content

feat: bug fixes, tests, CI, and --json agent output#7

Merged
tiero merged 3 commits intomasterfrom
feat/json-output
Feb 12, 2026
Merged

feat: bug fixes, tests, CI, and --json agent output#7
tiero merged 3 commits intomasterfrom
feat/json-output

Conversation

@brg444
Copy link
Copy Markdown
Contributor

@brg444 brg444 commented Feb 12, 2026

Summary

  • Remove stale MoltBot references from README, CLI, and skill modules
  • Fix exchange rate and fee percentage calculations in lendaswap skill
  • Fix getAvailablePairs type intersection for SDK type gap
  • Fix as any cast in lightning.ts with proper type assertion
  • Expand .gitignore, fix README Quick Start, add missing CLI commands
  • Add vitest with 17 unit tests for lendaswap utilities
  • Add GitHub Actions CI pipeline (Node 20 + 22, lint/build/test)
  • Add --json flag on every CLI command for machine-readable output
  • Add --version / -v flag

Test plan

  • pnpm build passes
  • pnpm test passes (17 tests)
  • pnpm lint passes
  • arkade balance --json returns valid JSON
  • arkade --version prints version
  • CI runs on push

brg444 and others added 3 commits February 12, 2026 22:06
- Remove MoltBot references from README, CLI, and skill modules
- Fix exchangeRate and fee percentage calculations in lendaswap skill
- Fix getAvailablePairs with proper type intersection for SDK gap
- Fix 'as any' cast in lightning.ts with proper type assertion
- Fix README Quick Start (init no longer requires private key)
- Add missing CLI commands to README table
- Expand .gitignore for full coverage
- Add patches/README.md documenting the ESM import patch
- Fix VHTL typo to VHTLC in lendaswap skill

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add vitest with 17 unit tests for lendaswap utilities
  (mapSwapStatus, isTerminalStatus, TOKEN_DECIMALS, exchange rate math)
- Export mapSwapStatus, isTerminalStatus, TOKEN_DECIMALS from lendaswap
  module for testability
- Add GitHub Actions CI workflow (Node 20 + 22, lint/build/test)
- Add vitest.config.ts to scope test discovery to src/

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The pnpm/action-setup@v4 auto-detects the version from the
packageManager field in package.json. Specifying version: 10
in the workflow conflicted with pnpm@10.20.0 in package.json.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@tiero tiero merged commit 1b63ea1 into master Feb 12, 2026
2 checks passed
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