[skjson] Rename sk_freeze and remove unused export (#806)

I noticed while adding an optional default parameter to Values#getUnique
that it's probably a footgun to export `sk_freeze` from skjson via
`core`'s index.ts, since it doesn't actually _freeze_, just adds the
hidden property that we use to track frozen-ness.

This PR drops that export and renames to avoid confusion.

Author: bennostein

skiplang/skjson/ts/binding/src/index.ts

@@ -34,7 +34,7 @@ export abstract class Frozen implements Managed {
   protected abstract freeze(): void;
 }
 
-export function sk_freeze<T extends object>(x: T): T & Managed {
+function tagSkManaged<T extends object>(x: T): T & Managed {
   return Object.defineProperty(x, sk_managed, {
     enumerable: false,
     writable: false,
@@ -48,7 +48,7 @@ export function isSkManaged(x: any): x is Managed {
 
 export abstract class SkManaged extends Frozen {
   protected freeze() {
-    sk_freeze(this);
+    tagSkManaged(this);
   }
 }
 
@@ -126,12 +126,12 @@ export function deepFreeze<T>(value: T): T & DepSafe {
       for (const elt of value) {
         deepFreeze(elt);
       }
-      return Object.freeze(sk_freeze(value));
+      return Object.freeze(tagSkManaged(value));
     } else {
       for (const val of Object.values(value)) {
         deepFreeze(val);
       }
-      return Object.freeze(sk_freeze(value));
+      return Object.freeze(tagSkManaged(value));
     }
   } else {
     // typeof value == "function" || typeof value == "undefined"
@@ -265,7 +265,7 @@ function interpretPointer<T extends Internal.CJSON>(
       const array = Array.from({ length }, (_, idx) =>
         interpretPointer(binding, binding.SKIP_SKJSON_at(aPtr, idx)),
       );
-      return sk_freeze(array);
+      return tagSkManaged(array);
     }
     case Type.Object: {
       const oPtr = binding.SKIP_SKJSON_asObject(pointer);

skipruntime-ts/core/src/index.ts

@@ -12,12 +12,7 @@ import type {
   JsonConverter,
   JsonObject,
 } from "../skiplang-json/index.js";
-import {
-  sk_freeze,
-  isSkManaged,
-  SkManaged,
-  checkOrCloneParam,
-} from "../skiplang-json/index.js";
+import { SkManaged, checkOrCloneParam } from "../skiplang-json/index.js";
 
 import { sknative } from "../skiplang-std/index.js";
 
@@ -54,7 +49,6 @@ import {
   type FromBinding,
 } from "./binding.js";
 
-export { sk_freeze, isSkManaged };
 export * from "./api.js";
 export * from "./errors.js";