Pass cell metadata and cell id as metadata for execute_request (microsoft#13288)

* Expose cell metadata * pass cell metadata as the metadata for execute request, * add cellId to cell metadata when cell is generated * don't add the metadata, avoids messing up the file * news entry * reword news entry * JSONObject -> Record<string, any>

Author: stisa

news/1 Enhancements/13252.md

@@ -0,0 +1,2 @@
+Cell id and cell metadata are now passed as the metadata field for execute_request messages.
+(thanks [stisa](https://github.com/stisa/))

src/client/datascience/jupyter/jupyterNotebook.ts

@@ -859,7 +859,9 @@ export class JupyterNotebookBase implements INotebook {
 
  private generateRequest = (
  code: string,
- silent?: boolean
+ silent?: boolean,
+ // tslint:disable-next-line: no-any
+ metadata?: Record<string, any>
  ): Kernel.IShellFuture<KernelMessage.IExecuteRequestMsg, KernelMessage.IExecuteReplyMsg> | undefined => {
  //traceInfo(`Executing code in jupyter : ${code}`);
  try {
@@ -873,7 +875,8 @@ export class JupyterNotebookBase implements INotebook {
  allow_stdin: true, // Allow when silent too in case runStartupCommands asks for a password
  store_history: !silent // Silent actually means don't output anything. Store_history is what affects execution_count
  },
- silent // Dispose only silent futures. Otherwise update_display_data doesn't finda future for a previous cell.
+ silent, // Dispose only silent futures. Otherwise update_display_data doesn't find a future for a previous cell.
+ metadata
  )
  : undefined;
  } catch (exc) {
@@ -1098,7 +1101,10 @@ export class JupyterNotebookBase implements INotebook {
  subscriber.error(this.sessionStartTime, exitError);
  subscriber.complete(this.sessionStartTime);
  } else {
- const request = this.generateRequest(concatMultilineStringInput(subscriber.cell.data.source), silent);
+ const request = this.generateRequest(concatMultilineStringInput(subscriber.cell.data.source), silent, {
+ ...subscriber.cell.data.metadata,
+ ...{ cellId: subscriber.cell.id }
+ });
 
  // Transition to the busy stage
  subscriber.cell.state = CellState.executing;