refactor: keck apis

[fundon]

No description provided.

[darkskygit]

Maybe we can consider encapsulating some transaction apis into with_trx like this:

workspace.with_trx(|trx| {
  trx.get(&block_id);
});

And all the block instances obtained in trx are an instance that encapsulates the original block instance and transaction. At this point, we can continue to use this instance to operate the block:

workspace.with_trx(|trx| {
  let block = trx.get(&block_id);
  block.set("key", "value");
});

[fundon]

Agree, trx already contains workspace reference instance.
This step can be put at the end.

[darkskygit]

Maybe we can consider encapsulating some transaction apis into with_trx like this:

workspace.with_trx(|trx| {
  trx.get(&block_id);
});

And all the block instances obtained in trx are an instance that encapsulates the original block instance and transaction. At this point, we can continue to use this instance to operate the block:

workspace.with_trx(|trx| {
  let block = trx.get(&block_id);
  block.set("key", "value");
});

I added more improvement intents in #235
relevant context, please go to this issue to discuss