Skip to content

Merge develop into master#439

Merged
RetricSu merged 8 commits into
masterfrom
develop
Jun 25, 2026
Merged

Merge develop into master#439
RetricSu merged 8 commits into
masterfrom
develop

Conversation

@RetricSu

Copy link
Copy Markdown
Collaborator

No description provided.

RetricSu and others added 8 commits April 10, 2026 15:40
)

* 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
…nd_yarn-85af2c71bb

build(deps): bump follow-redirects from 1.15.9 to 1.16.0 in the npm_and_yarn group across 1 directory
…433)

Bumps the npm_and_yarn group with 1 update in the / directory: [hono](https://github.com/honojs/hono).


Updates `hono` from 4.12.12 to 4.12.14
- [Release notes](https://github.com/honojs/hono/releases)
- [Commits](honojs/hono@v4.12.12...v4.12.14)

---
updated-dependencies:
- dependency-name: hono
  dependency-version: 4.12.14
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…434)

Bumps the npm_and_yarn group with 1 update in the / directory: [hono](https://github.com/honojs/hono).


Updates `hono` from 4.12.14 to 4.12.18
- [Release notes](https://github.com/honojs/hono/releases)
- [Commits](honojs/hono@v4.12.14...v4.12.18)

---
updated-dependencies:
- dependency-name: hono
  dependency-version: 4.12.18
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ry (#435)

Bumps the npm_and_yarn group with 1 update in the / directory: [fast-uri](https://github.com/fastify/fast-uri).


Updates `fast-uri` from 3.1.0 to 3.1.2
- [Release notes](https://github.com/fastify/fast-uri/releases)
- [Commits](fastify/fast-uri@v3.1.0...v3.1.2)

---
updated-dependencies:
- dependency-name: fast-uri
  dependency-version: 3.1.2
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore: bump @ckb-ccc/core to 1.14.0 to fix ws vulnerability

Upgrades @ckb-ccc/core from 1.5.3 to 1.14.0 so the transitive
ws dependency resolves to ^8.21.0, which patches the reported
memory exhaustion DoS advisory.

* fix: add NervosDao to devnet known scripts for ccc 1.14.0 compatibility

ccc >= 1.14.0 calls getKnownScript(NervosDao) during completeFeeBy
for every input. Devnet has no NervosDao deployment, so supplying the
testnet definition lets isNervosDao() return false without throwing.

Also add the missing changeset for the @ckb-ccc/core bump.

Co-Authored-By: Claude <noreply@anthropic.com>

* ci: skip changeset PR comments on fork PRs

The github-script comment steps require pull-requests: write, which
fork PRs do not receive with the pull_request trigger. Skip the
optional comment steps on forks so the required changeset check can
still pass/fail cleanly.

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: use actual devnet DAO script for NervosDao known script

The previous fallback reused the testnet NervosDao definition. Devnet
deploys its own DAO system cell, so map KnownScript.NervosDao to the
devnet dao script derived from ckb list-hashes instead.

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>
* chore: bump default CKB version to 0.207.0

* Revert package version bump per PR feedback - will use changeset bump later
@github-actions

Copy link
Copy Markdown

Missing Changeset

Please add a changeset describing your changes:

pnpm changeset

If your changes do not need a version bump (docs, CI, refactoring),
add the skip-changeset label to this PR.

For dependency updates, use the dependencies label.

@RetricSu RetricSu merged commit 38413c1 into master Jun 25, 2026
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants