Skip to content

feat(account): add csv and json output for transactions#2

Merged
geavenx merged 1 commit intomasterfrom
feat/transactions-output-formats
Feb 11, 2026
Merged

feat(account): add csv and json output for transactions#2
geavenx merged 1 commit intomasterfrom
feat/transactions-output-formats

Conversation

@geavenx
Copy link
Owner

@geavenx geavenx commented Feb 11, 2026

Summary by CodeRabbit

Release Notes

  • New Features

    • Added machine-readable export formats for account transactions: CSV and JSON options via new --output flag, enabling scripting and spreadsheet workflows.
  • Documentation

    • Updated documentation to describe transaction listing with export format options and usage examples.
  • Tests

    • Expanded test coverage for transaction output formats and command behaviors.

@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

The PR extends the CLI transaction command to support multiple output formats (table, csv, json) via a new --output option, adds corresponding formatting functions to render transactions in CSV and JSON, and includes comprehensive test coverage for the new functionality.

Changes

Cohort / File(s) Summary
Documentation
AGENTS.md, README.md
Updated transaction listing documentation to reflect new --output (-o) option and supported output formats (table, csv, json). Increased test count guidance from 109 to 119 tests.
CLI Output Formats
src/arcter/cli.py
Added TransactionOutputFormat enum with table/csv/json members. Introduced _print_transaction_csv() and _print_transaction_json() functions. Extended account_transactions() command with output_format option to enable runtime format selection. Added imports for csv, json, and sys modules.
Test Coverage
tests/test_account_cli.py
Added 10 new tests covering CSV and JSON output formatting, field validation, filter respect, null handling, empty states, and edge cases. Tests verify default table behavior remains unchanged and validate proper output format rendering.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 Hop along, dear transactions, new formats to explore!
CSV hops, JSON prances, tables shine like never before,
A rabbit's dream of export—where data dances free,
In spreadsheets and scripts, oh what joy for thee! 📊✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 3.57% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately describes the main change: adding CSV and JSON output format support for the transactions command.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Docstrings were successfully generated.
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/transactions-output-formats

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

Note

Docstrings generation - SUCCESS
Generated docstrings for this pull request at #3

coderabbitai bot added a commit that referenced this pull request Feb 11, 2026
Docstrings generation was requested by @geavenx.

* #2 (comment)

The following files were modified:

* `src/arcter/cli.py`
* `tests/test_account_cli.py`
@geavenx geavenx merged commit e7cbcbc into master Feb 11, 2026
2 checks passed
geavenx pushed a commit that referenced this pull request Feb 12, 2026
Docstrings generation was requested by @geavenx.

* #2 (comment)

The following files were modified:

* `src/arcter/cli.py`
* `tests/test_account_cli.py`
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