refactor: use core ResourceTable in cli

[bartlomieju]

Follow up to #3134, #3136 and #3150

[bartlomieju]

@ry this is minimal landable patch with only Repl factored out. PTAL while I start working on breaking out child and worker resources

[bartlomieju]

Interestingly resource are now not properly dropped 🤔 investigating...

EDIT:
I added this to core/resources.rs and sadly it's never called. The same for ReplResource and Repl. This is strange...

impl Drop for ResourceTable {
  fn drop(&mut self) {
    eprintln!("dropping resource table");
  }
}

EDIT 2:
Now I'm even more confused... From lazy_static docs we can read:

Apart from the lazy initialization, the resulting "static ref" variables have generally the same properties as regular "static" variables:

- Any type in them needs to fulfill the Sync trait.
- If the type has a destructor, then it will not run when the process exits.

AFAIK BTreeMap has a destructor implementation and resources are dropped, however core's ResourceTable is not... 😕

EDIT 3: Nevermind 😅 I screwed op_close implementation

[piscisaureus]

nice work, lgtm!

[ry]

LGTM too - nice work.