For now, only one `pyodide` can be loaded in a given global scope because we need to use a global variable for package loading. This changes `loadPyodide` so that it no longer defines `globalThis.pyodide` in favor of just returning `pyodide` from `loadPyodide`. `Module` is placed on the global object as `globalThis.__pyodide_module` for the purposes of package loading. I added documentation to explicitly state that `loadPyodide` can only be run once in each global Javascript context, an error will be thrown otherwise.

This has two advantages: 
1. The returned `pyodide` module is understandable to Typescript and other code analysis, whereas the global variable is not understandable.
2. If in the future we are able to remove the global variable, no further API change will be needed (other than relaxing the documented "pyodide must be globally-unique" constraint). I figure it's better to make this API change earlier rather than later.

It will be possible to completely remove the global variable dependence when Firefox implements support for using ES6 modules in webworkers (see https://bugzilla.mozilla.org/show_bug.cgi?id=1247687).
API Don't expose pyodide as a global variable in loadPyodide. For now, only one `pyodide` can be loaded in a given global scope because we need to use a global variable for package loading. This changes `loadPyodide` so that it no longer defines `globalThis.pyodide` in favor of just returning `pyodide` from `loadPyodide`. `Module` is placed on the global object as `globalThis.__pyodide_module` for the purposes of package loading. I added documentation to explicitly state that `loadPyodide` can only be run once in each global Javascript context, an error will be thrown otherwise.

This has two advantages: 
1. The returned `pyodide` module is understandable to Typescript and other code analysis, whereas the global variable is not understandable.
2. If in the future we are able to remove the global variable, no further API change will be needed (other than relaxing the documented "pyodide must be globally-unique" constraint). I figure it's better to make this API change earlier rather than later.

It will be possible to completely remove the global variable dependence when Firefox implements support for using ES6 modules in webworkers (see https://bugzilla.mozilla.org/show_bug.cgi?id=1247687).
#ifndef PYTHON2JS_H
#define PYTHON2JS_H

/** Translate Python objects to Javascript.
 */
// clang-format off
#define PY_SSIZE_T_CLEAN
#include "Python.h"
// clang-format on
#include "hiwire.h"

/**
 * Do a shallow conversion from python to Javascript. Convert immutable types
 * with equivalent Javascript immutable types, but all other types are proxied.
 */
JsRef
python2js(PyObject* x);

/**
 * Like python2js except in the handling of PyProxy creation.
 *
 * If proxies is NULL, will throw an error instead of creating a PyProxy.
 * Otherwise, proxies should be an Array and python2js_track_proxies will add
 * all proxies created to the proxies array.
 */
JsRef
python2js_track_proxies(PyObject* x, JsRef proxies);

/**
 * Convert a Python object to a Javascript object, copying standard collections
 * into javascript down to specified depth
 * \param x The Python object
 * \param depth The maximum depth to copy
 * \param proxies If this is Null, will raise an error if we have no Javascript
 *        conversion for a Python object. If not NULL, should be a Javascript
 *        list. We will add all PyProxies created to the list.
 * \return The Javascript object -- might be an Error object in the case of an
 *         exception.
 */
JsRef
python2js_with_depth(PyObject* x, int depth, JsRef proxies);

int
python2js_init(PyObject* core);

#endif /* PYTHON2JS_H */

#ifndef PYTHON2JS_H
#define PYTHON2JS_H

/** Translate Python objects to Javascript.
 */
// clang-format off
#define PY_SSIZE_T_CLEAN
#include "Python.h"
// clang-format on
#include "hiwire.h"

/**
 * Do a shallow conversion from python to Javascript. Convert immutable types
 * with equivalent Javascript immutable types, but all other types are proxied.
 */
JsRef
python2js(PyObject* x);

/**
 * Like python2js except in the handling of PyProxy creation.
 *
 * If proxies is NULL, will throw an error instead of creating a PyProxy.
 * Otherwise, proxies should be an Array and python2js_track_proxies will add
 * all proxies created to the proxies array.
 */
JsRef
python2js_track_proxies(PyObject* x, JsRef proxies);

/**
 * Convert a Python object to a Javascript object, copying standard collections
 * into javascript down to specified depth
 * \param x The Python object
 * \param depth The maximum depth to copy
 * \param proxies If this is Null, will raise an error if we have no Javascript
 *        conversion for a Python object. If not NULL, should be a Javascript
 *        list. We will add all PyProxies created to the list.
 * \return The Javascript object -- might be an Error object in the case of an
 *         exception.
 */
JsRef
python2js_with_depth(PyObject* x, int depth, JsRef proxies);

int
python2js_init(PyObject* core);

#endif /* PYTHON2JS_H */
