core: rename some internal symbols

It's no longer necessary to prefix them with an underscore so they don't
get auto-exported.

src/_sqlite3.c

@@ -547,5 +547,5 @@ void tjs__mod_sqlite3_init(JSContext *ctx, JSValue ns) {
     JSValue obj = JS_NewObjectProto(ctx, JS_NULL);
     JS_SetPropertyFunctionList(ctx, obj, tjs_sqlite3_funcs, countof(tjs_sqlite3_funcs));
 
-    JS_DefinePropertyValueStr(ctx, ns, "_sqlite3", obj, JS_PROP_C_W_E);
+    JS_DefinePropertyValueStr(ctx, ns, "sqlite3", obj, JS_PROP_C_W_E);
 }

src/fs.c

@@ -1294,8 +1294,8 @@ static const JSCFunctionListEntry tjs_fs_funcs[] = {
     TJS_CFUNC_DEF("lchown", 3, tjs_fs_lchown),
     TJS_CFUNC_DEF("chmod", 2, tjs_fs_chmod),
     /* Internal */
-    TJS_CFUNC_DEF("_mkdirSync", 2, tjs_fs_mkdir_sync),
-    TJS_CFUNC_DEF("_statSync", 1, tjs_fs_stat_sync),
+    TJS_CFUNC_DEF("mkdirSync", 2, tjs_fs_mkdir_sync),
+    TJS_CFUNC_DEF("statSync", 1, tjs_fs_stat_sync),
 };
 
 void tjs__mod_fs_init(JSContext *ctx, JSValue ns) {

src/js/core/env.js

@@ -2,33 +2,33 @@ const core = globalThis[Symbol.for('tjs.internal.core')];
 
 const env = new Proxy({}, {
     ownKeys() {
-        return core._envKeys();
+        return core.envKeys();
     },
     get(_, prop) {
         if (prop === Symbol.toStringTag) {
-            return JSON.stringify(core._environ(), null, 2);
+            return JSON.stringify(core.environ(), null, 2);
         } else if (prop === 'toJSON') {
-            return () => core._environ();
+            return () => core.environ();
         } else if (typeof prop === 'string') {
             try {
-                return core._getenv(prop);
+                return core.getenv(prop);
             } catch (_) { /* Ignored. */ }
         }
 
         return undefined;
     },
     set(_, prop, val) {
-        core._setenv(prop, val);
+        core.setenv(prop, val);
 
         return true;
     },
     deleteProperty(_, prop) {
-        core._unsetenv(prop);
+        core.unsetenv(prop);
 
         return true;
     },
     has(_, key) {
-        return key in core._envKeys();
+        return key in core.envKeys();
     }
 });
 

src/js/core/index.js

@@ -183,21 +183,21 @@ Object.defineProperty(tjs, 'stderr', {
 // Interface for the garbage collection
 const _gc_state = {
     enabled: true,
-    threshold: core._gc.getThreshold()
+    threshold: core.gc.getThreshold()
 };
 
 Object.defineProperty(tjs, 'gc', {
     enumerable: true,
     configurable: false,
     writable: false,
     value: {
-        run: () => core._gc.run(),
+        run: () => core.gc.run(),
 
         set enabled(value) {
             if (value) {
-                core._gc.setThreshold(_gc_state.threshold);
+                core.gc.setThreshold(_gc_state.threshold);
             } else {
-                core._gc.setThreshold(-1);
+                core.gc.setThreshold(-1);
             }
 
             _gc_state.enabled=value;
@@ -208,13 +208,13 @@ Object.defineProperty(tjs, 'gc', {
 
         set threshold(value) {
             if (_gc_state.enabled) {
-                core._gc.setThreshold(value);
+                core.gc.setThreshold(value);
             }
 
             _gc_state.threshold = value;
         },
         get threshold() {
-            const tmp = core._gc.getThreshold();
+            const tmp = core.gc.getThreshold();
 
             if (tmp !== -1) {
                 _gc_state.threshold = tmp;

src/js/polyfills/storage.js

@@ -1,7 +1,7 @@
 /* global tjs */
 
 const core = globalThis[Symbol.for('tjs.internal.core')];
-const sqlite3 = core._sqlite3;
+const sqlite3 = core.sqlite3;
 
 const kStorageMap = Symbol('kStorageMap');
 
@@ -220,7 +220,7 @@ function mkdirSync(path, options = { mode: 0o777, recursive: false }) {
     const pathModule = globalThis[Symbol.for('tjs.internal.modules.path')];
 
     if (!options.recursive) {
-        return core._mkdirSync(path, options.mode);
+        return core.mkdirSync(path, options.mode);
     }
 
     const parent = pathModule.dirname(path);
@@ -232,11 +232,11 @@ function mkdirSync(path, options = { mode: 0o777, recursive: false }) {
     mkdirSync(parent, options);
 
     try {
-        return core._mkdirSync(path, options.mode);
+        return core.mkdirSync(path, options.mode);
     } catch (e) {
         // Cannot rely on checking for EEXIST since the OS could throw other errors like EROFS.
 
-        const st = core._statSync(path);
+        const st = core.statSync(path);
 
         if (!st.isDirectory) {
             throw e;

src/js/stdlib/sqlite.js

@@ -1,5 +1,5 @@
 const core = globalThis[Symbol.for('tjs.internal.core')];
-const sqlite3 = core._sqlite3;
+const sqlite3 = core.sqlite3;
 
 const kSqlite3Handle = Symbol('kSqlite3Handle');
 let controllers;

src/os.c

@@ -486,11 +486,11 @@ static const JSCFunctionListEntry tjs_os_funcs[] = {
     TJS_CFUNC_DEF("uname", 0, tjs_uname),
     TJS_CFUNC_DEF("uptime", 0, tjs_uptime),
     TJS_CFUNC_DEF("guessHandle", 1, tjs_guess_handle),
-    TJS_CFUNC_DEF("_getenv", 0, tjs_getenv),
-    TJS_CFUNC_DEF("_setenv", 2, tjs_setenv),
-    TJS_CFUNC_DEF("_unsetenv", 1, tjs_unsetenv),
-    TJS_CFUNC_DEF("_envKeys", 0, tjs_envKeys),
-    TJS_CFUNC_DEF("_environ", 0, tjs_environ),
+    TJS_CFUNC_DEF("getenv", 0, tjs_getenv),
+    TJS_CFUNC_DEF("setenv", 2, tjs_setenv),
+    TJS_CFUNC_DEF("unsetenv", 1, tjs_unsetenv),
+    TJS_CFUNC_DEF("envKeys", 0, tjs_envKeys),
+    TJS_CFUNC_DEF("environ", 0, tjs_environ),
     TJS_CFUNC_DEF("chdir", 1, tjs_chdir),
     TJS_CFUNC_DEF("random", 3, tjs_random),
     TJS_CFUNC_DEF("cpuInfo", 0, tjs_cpu_info),

src/sys.c

@@ -274,7 +274,7 @@ void tjs__mod_sys_init(JSContext *ctx, JSValue ns) {
 
     JSValue gc = JS_NewObjectProto(ctx, JS_NULL);
     JS_SetPropertyFunctionList(ctx, gc, tjs_gc_funcs, countof(tjs_gc_funcs));
-    JS_DefinePropertyValueStr(ctx, ns, "_gc", gc, 0);
+    JS_DefinePropertyValueStr(ctx, ns, "gc", gc, JS_PROP_C_W_E);
 
     JS_DefinePropertyValueStr(ctx, ns, "versions", versions, JS_PROP_C_W_E);
     JS_DefinePropertyValueStr(ctx, ns, "platform", JS_NewString(ctx, TJS__PLATFORM), JS_PROP_C_W_E);