fix: declare runes properly in filepath

Signed-off-by: Christian Stewart <christian@aperture.us>

Author: paralin

compliance/tests/rune_const_import/expected.log

@@ -0,0 +1,8 @@
+separator: 47
+newline: 10
+space: 32
+separator matches '/'
+newline matches '\n'
+space matches ' '
+separator + 1: 48
+space - 1: 31

compliance/tests/rune_const_import/index.ts

@@ -0,0 +1,30 @@
+package main
+
+import "github.com/aperturerobotics/goscript/compliance/tests/rune_const_import/subpkg"
+
+func main() {
+	// Test importing rune constants from another package
+	const separator = subpkg.Separator
+	const newline = subpkg.Newline
+	const space = subpkg.Space
+
+	// Print the imported rune constants
+	println("separator:", separator)
+	println("newline:", newline)
+	println("space:", space)
+
+	// Use them in comparisons to ensure they're actually numbers
+	if separator == '/' {
+		println("separator matches '/'")
+	}
+	if newline == '\n' {
+		println("newline matches '\\n'")
+	}
+	if space == ' ' {
+		println("space matches ' '")
+	}
+
+	// Test arithmetic operations (only works with numbers)
+	println("separator + 1:", separator+1)
+	println("space - 1:", space-1)
+}

compliance/tests/rune_const_import/rune_const_import.go

@@ -0,0 +1,34 @@
+// Generated file based on rune_const_import.go
+// Updated when compliance tests are re-run, DO NOT EDIT!
+
+import * as $ from "@goscript/builtin/index.js";
+
+import * as subpkg from "@goscript/github.com/aperturerobotics/goscript/compliance/tests/rune_const_import/subpkg/index.js"
+
+export async function main(): Promise<void> {
+	// Test importing rune constants from another package
+	let separator: number = subpkg.Separator
+	let newline: number = subpkg.Newline
+	let space: number = subpkg.Space
+
+	// Print the imported rune constants
+	console.log("separator:", 47)
+	console.log("newline:", 10)
+	console.log("space:", 32)
+
+	// Use them in comparisons to ensure they're actually numbers
+	if (47 == 47) {
+		console.log("separator matches '/'")
+	}
+	if (10 == 10) {
+		console.log("newline matches '\\n'")
+	}
+	if (32 == 32) {
+		console.log("space matches ' '")
+	}
+
+	// Test arithmetic operations (only works with numbers)
+	console.log("separator + 1:", 47 + 1)
+	console.log("space - 1:", 32 - 1)
+}
+

compliance/tests/rune_const_import/rune_const_import.gs.ts

@@ -0,0 +1 @@
+export { Newline, Separator, Space } from "./subpkg.gs.js"

compliance/tests/rune_const_import/subpkg/index.ts

@@ -0,0 +1,6 @@
+package subpkg
+
+// Rune constants that should be compiled as numbers, not strings
+const Separator = '/'
+const Newline = '\n'
+const Space = ' '

compliance/tests/rune_const_import/subpkg/subpkg.go

@@ -0,0 +1,11 @@
+// Generated file based on subpkg/subpkg.go
+// Updated when compliance tests are re-run, DO NOT EDIT!
+
+import * as $ from "@goscript/builtin/index.js";
+
+export let Separator: number = 47
+
+export let Newline: number = 10
+
+export let Space: number = 32
+

compliance/tests/rune_const_import/subpkg/subpkg.gs.ts

@@ -1023,6 +1023,19 @@ export const stringToRunes = (str: string): number[] => {
   return Array.from(str).map((c) => c.codePointAt(0) || 0)
 }
 
+/**
+ * Converts a single-character string to its Unicode code point (rune).
+ * Used for readable rune constants like $.stringToRune('/') instead of 47.
+ * @param str A single-character string.
+ * @returns The Unicode code point as a number.
+ */
+export const stringToRune = (str: string): number => {
+  if (str.length === 0) {
+    return 0
+  }
+  return str.codePointAt(0) || 0
+}
+
 /**
  * Converts an array of Unicode code points (runes) to a string.
  * @param runes The input array of numbers representing Unicode code points.

gs/builtin/slice.ts

@@ -3,8 +3,8 @@
 import * as $ from "@goscript/builtin/index.js"
 
 // Path separator constants
-export const Separator = '/'
-export const ListSeparator = ':'
+export const Separator = $.stringToRune('/')
+export const ListSeparator = $.stringToRune(':')
 
 // Error constants
 export const SkipDir = $.newError('skip this directory')
@@ -234,7 +234,7 @@ export function SplitList(path: string): string[] {
   if (path === '') {
     return []
   }
-  return path.split(ListSeparator)
+  return path.split(String.fromCharCode(ListSeparator))
 }
 
 // HasPrefix tests whether the path p begins with prefix.