Skip to content

feat: replace ckb-transaction-dumper with ccc-based implementation (#…#431

Merged
RetricSu merged 1 commit intov0.4.xfrom
develop
Apr 10, 2026
Merged

feat: replace ckb-transaction-dumper with ccc-based implementation (#…#431
RetricSu merged 1 commit intov0.4.xfrom
develop

Conversation

@RetricSu
Copy link
Copy Markdown
Collaborator

…420)

  • feat: replace ckb-transaction-dumper with ccc-based implementation
  • Rewrite src/tools/ckb-tx-dumper.ts to use ccc Client and molecule codecs
  • Implement dep_group unpacking using ccc.mol
  • Remove ckb-transaction-dumper from dependencies
  • Eliminates external npm dependency for transaction dumping
  • chore: add changeset for ckb-transaction-dumper replacement

  • fix: address PR review comments for ckb-tx-dumper

  • Remove .sisyphus/boulder.json and add to .gitignore
  • Fix type field serialization: emit null instead of undefined
  • Fix dep_type format: convert camelCase to snake_case (depGroup -> dep_group)
  • Fix depGroup error handling: throw error when refCell not found
  • Fix async race condition: make buildTxFileOptionBy and callers async
  • fix: fix CCC-based transaction dumper for debug command
  • Use cccA.JsonRpcTransformers.transactionTo() to handle snake_case input format
  • Convert all numeric values to hex format (since, index, version, capacity)
  • Add missing dep_group cell to mock_info.cell_deps before expansion
  • Add fs.mkdirSync to ensure output directory exists

Fixes issues where debug command failed with 'undefined is not iterable', 'unprovided cell dep', or 'since is not a legal u64' errors.

  • fix: address Copilot PR review comments for ccc-based tx dumper
  • Select ClientPublicTestnet/Mainnet based on RPC URL pattern
  • Use proper error type checking with instanceof Error

)

* feat: replace ckb-transaction-dumper with ccc-based implementation

- Rewrite src/tools/ckb-tx-dumper.ts to use ccc Client and molecule codecs
- Implement dep_group unpacking using ccc.mol
- Remove ckb-transaction-dumper from dependencies
- Eliminates external npm dependency for transaction dumping

* chore: add changeset for ckb-transaction-dumper replacement

* fix: address PR review comments for ckb-tx-dumper

- Remove .sisyphus/boulder.json and add to .gitignore
- Fix type field serialization: emit null instead of undefined
- Fix dep_type format: convert camelCase to snake_case (depGroup -> dep_group)
- Fix depGroup error handling: throw error when refCell not found
- Fix async race condition: make buildTxFileOptionBy and callers async

* fix: fix CCC-based transaction dumper for debug command

- Use cccA.JsonRpcTransformers.transactionTo() to handle snake_case input format
- Convert all numeric values to hex format (since, index, version, capacity)
- Add missing dep_group cell to mock_info.cell_deps before expansion
- Add fs.mkdirSync to ensure output directory exists

Fixes issues where debug command failed with 'undefined is not iterable',
'unprovided cell dep', or 'since is not a legal u64' errors.

* fix: address Copilot PR review comments for ccc-based tx dumper

- Select ClientPublicTestnet/Mainnet based on RPC URL pattern
- Use proper error type checking with instanceof Error
@RetricSu RetricSu merged commit a078424 into v0.4.x Apr 10, 2026
5 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.

1 participant