goscript 0.0.49 → 0.0.51

package/gs/syscall/index.ts

@@ -1,212 +1,17 @@
-import * as $ from '@goscript/builtin/index.js'
+// Re-export all types
+export * from './types.js'
 
-// Essential type aliases
-export type uintptr = number
+// Re-export all constants
+export * from './constants.js'
 
-// Errno type for syscall errors
-export interface Errno {
-  Error(): string
-  Is(target: $.GoError): boolean
-  Errno(): number
-}
+// Re-export environment functions
+export * from './env.js'
 
-// Essential syscall constants
-export const O_RDONLY = 0
-export const O_WRONLY = 1
-export const O_RDWR = 2
-export const O_APPEND = 8
-export const O_CREATE = 64
-export const O_EXCL = 128
-export const O_SYNC = 256
-export const O_TRUNC = 512
+// Re-export file system structures and functions
+export * from './fs.js'
 
-export const Stdin = 0
-export const Stdout = 1
-export const Stderr = 2
+// Re-export error constants
+export * from './errors.js'
 
-export const SIGINT = 2
-export const SIGTERM = 15
-
-// File mode constants
-export const S_IFMT = 0o170000
-export const S_IFREG = 0o100000
-export const S_IFDIR = 0o040000
-export const S_IFLNK = 0o120000
-export const S_IFBLK = 0o060000
-export const S_IFCHR = 0o020000
-export const S_IFIFO = 0o010000
-export const S_IFSOCK = 0o140000
-export const S_ISUID = 0o004000
-export const S_ISGID = 0o002000
-export const S_ISVTX = 0o001000
-
-// Environment variable functions using Node.js/browser APIs
-export function Getenv(key: string): [string, boolean] {
-  if (typeof process !== 'undefined' && process.env) {
-    const value = process.env[key]
-    return value !== undefined ? [value, true] : ['', false]
-  }
-  return ['', false]
-}
-
-export function Setenv(key: string, value: string): $.GoError {
-  if (typeof process !== 'undefined' && process.env) {
-    process.env[key] = value
-    return null
-  }
-  return { Error: () => 'setenv not supported' }
-}
-
-export function Unsetenv(key: string): $.GoError {
-  if (typeof process !== 'undefined' && process.env) {
-    delete process.env[key]
-    return null
-  }
-  return { Error: () => 'unsetenv not supported' }
-}
-
-export function Clearenv(): void {
-  if (typeof process !== 'undefined' && process.env) {
-    for (const key in process.env) {
-      delete process.env[key]
-    }
-  }
-}
-
-export function Environ(): $.Slice<string> {
-  if (typeof process !== 'undefined' && process.env) {
-    const env: string[] = []
-    for (const [key, value] of Object.entries(process.env)) {
-      if (value !== undefined) {
-        env.push(`${key}=${value}`)
-      }
-    }
-    return $.arrayToSlice(env)
-  }
-  return $.arrayToSlice([])
-}
-
-// Dirent structure with Reclen field
-export class Dirent {
-  public Name: $.Bytes = new Uint8Array(0)
-  public Reclen: number = 0
-  constructor(init?: any) {
-    if (init?.Name) this.Name = init.Name
-    if (init?.Reclen) this.Reclen = init.Reclen
-  }
-}
-
-// Stat_t structure stub
-export class Stat_t {
-  public Dev: number = 0
-  public Ino: number = 0
-  public Mode: number = 0
-  public Nlink: number = 0
-  public Uid: number = 0
-  public Gid: number = 0
-  public Rdev: number = 0
-  public Size: number = 0
-  public Blksize: number = 0
-  public Blocks: number = 0
-  public Atime: number = 0
-  public Mtime: number = 0
-  public Ctime: number = 0
-  public AtimeNsec: number = 0
-  public MtimeNsec: number = 0
-  public CtimeNsec: number = 0
-
-  constructor(init?: any) {
-    if (init) {
-      Object.assign(this, init)
-    }
-  }
-
-  public clone(): Stat_t {
-    return new Stat_t(this)
-  }
-}
-
-// RawConn interface - stub implementation for JavaScript environment
-export interface RawConn {
-  Control(f: (fd: uintptr) => void): $.GoError
-  Read(f: (fd: uintptr) => boolean): $.GoError
-  Write(f: (fd: uintptr) => boolean): $.GoError
-}
-
-// Stub implementation of RawConn that always returns ErrUnimplemented
-export class StubRawConn implements RawConn {
-  Control(_f: (fd: uintptr) => void): $.GoError {
-    return {
-      Error: () => 'operation not implemented in JavaScript environment',
-    }
-  }
-
-  Read(_f: (fd: uintptr) => boolean): $.GoError {
-    return {
-      Error: () => 'operation not implemented in JavaScript environment',
-    }
-  }
-
-  Write(_f: (fd: uintptr) => boolean): $.GoError {
-    return {
-      Error: () => 'operation not implemented in JavaScript environment',
-    }
-  }
-}
-
-// Additional error constants - implement as Errno type
-export const ENOSYS: Errno = {
-  Error: () => 'function not implemented',
-  Is: (target: $.GoError) => target === ENOSYS,
-  Errno: () => 38,
-}
-
-export const EISDIR: Errno = {
-  Error: () => 'is a directory',
-  Is: (target: $.GoError) => target === EISDIR,
-  Errno: () => 21,
-}
-
-export const ENOTDIR: Errno = {
-  Error: () => 'not a directory',
-  Is: (target: $.GoError) => target === ENOTDIR,
-  Errno: () => 20,
-}
-
-export const ERANGE: Errno = {
-  Error: () => 'result too large',
-  Is: (target: $.GoError) => target === ERANGE,
-  Errno: () => 34,
-}
-
-export const ENOMEM: Errno = {
-  Error: () => 'out of memory',
-  Is: (target: $.GoError) => target === ENOMEM,
-  Errno: () => 12,
-}
-
-export const ESRCH: Errno = {
-  Error: () => 'no such process',
-  Is: (target: $.GoError) => target === ESRCH,
-  Errno: () => 3,
-}
-
-// Additional missing syscall functions
-export function Open(
-  _path: string,
-  _flag: number,
-  _perm: number,
-): [number, $.GoError] {
-  return [-1, ENOSYS]
-}
-
-export function Sysctl(_name: string): [string, $.GoError] {
-  return ['', ENOSYS]
-}
-
-// Getpagesize returns the underlying system's memory page size.
-export function Getpagesize(): number {
-  // Return a standard page size for JavaScript environment
-  // Most systems use 4096 bytes as the default page size
-  return 4096
-}
+// Re-export RawConn implementation
+export * from './rawconn.js'

package/gs/syscall/rawconn.ts

@@ -0,0 +1,23 @@
+import * as $ from '@goscript/builtin/index.js'
+import { RawConn, uintptr } from './types.js'
+
+// Stub implementation of RawConn that always returns ErrUnimplemented
+export class StubRawConn implements RawConn {
+  Control(_f: (fd: uintptr) => void): $.GoError {
+    return {
+      Error: () => 'operation not implemented in JavaScript environment',
+    }
+  }
+
+  Read(_f: (fd: uintptr) => boolean): $.GoError {
+    return {
+      Error: () => 'operation not implemented in JavaScript environment',
+    }
+  }
+
+  Write(_f: (fd: uintptr) => boolean): $.GoError {
+    return {
+      Error: () => 'operation not implemented in JavaScript environment',
+    }
+  }
+} 

package/gs/syscall/types.ts

@@ -0,0 +1,18 @@
+import * as $ from '@goscript/builtin/index.js'
+
+// Essential type aliases
+export type uintptr = number
+
+// Errno type for syscall errors
+export interface Errno {
+  Error(): string
+  Is(target: $.GoError): boolean
+  Errno(): number
+}
+
+// RawConn interface - stub implementation for JavaScript environment
+export interface RawConn {
+  Control(f: (fd: uintptr) => void): $.GoError
+  Read(f: (fd: uintptr) => boolean): $.GoError
+  Write(f: (fd: uintptr) => boolean): $.GoError
+} 

package/gs/time/time.ts

@@ -5,21 +5,32 @@ export class Time {
   private _monotonic?: number // high-resolution monotonic timestamp in nanoseconds
   private _location: Location // timezone location
 
-  constructor(
+  constructor(_props?: {}) {
+    // Default constructor creates a zero time (Unix epoch in UTC)
+    this._date = new globalThis.Date(0)
+    this._nsec = 0
+    this._monotonic = undefined
+    this._location = UTC
+  }
+
+  // create is a static factory method that creates a Time instance with specific parameters
+  public static create(
     date: globalThis.Date,
     nsec: number = 0,
     monotonic?: number,
     location?: Location,
-  ) {
-    this._date = new globalThis.Date(date.getTime())
-    this._nsec = nsec
-    this._monotonic = monotonic
-    this._location = location || UTC
+  ): Time {
+    const time = new Time()
+    time._date = new globalThis.Date(date.getTime())
+    time._nsec = nsec
+    time._monotonic = monotonic
+    time._location = location || UTC
+    return time
   }
 
   // clone returns a copy of this Time instance
   public clone(): Time {
-    return new Time(this._date, this._nsec, this._monotonic, this._location)
+    return Time.create(this._date, this._nsec, this._monotonic, this._location)
   }
 
   // Unix returns t as a Unix time, the number of seconds elapsed since January 1, 1970 UTC
@@ -492,7 +503,7 @@ export class Time {
     const newNsec = this._nsec + (durationNs % 1000000)
     const newMonotonic =
       this._monotonic !== undefined ? this._monotonic + durationNs : undefined
-    return new Time(newDate, newNsec, newMonotonic, this._location)
+    return Time.create(newDate, newNsec, newMonotonic, this._location)
   }
 
   // Equal reports whether t and u represent the same time instant
@@ -531,7 +542,7 @@ export class Time {
   public Round(_d: Duration): Time {
     // Implementation would round to nearest duration
     // For now, simplified version that strips monotonic reading
-    return new Time(this._date, this._nsec, undefined, this._location)
+    return Time.create(this._date, this._nsec, undefined, this._location)
   }
 
   // Truncate returns the result of rounding t down to a multiple of d
@@ -539,7 +550,7 @@ export class Time {
   public Truncate(_d: Duration): Time {
     // Implementation would truncate to duration
     // For now, simplified version that strips monotonic reading
-    return new Time(this._date, this._nsec, undefined, this._location)
+    return Time.create(this._date, this._nsec, undefined, this._location)
   }
 
   // String returns the time formatted as a string
@@ -571,8 +582,11 @@ export function Duration_lt(receiver: Duration, other: Duration): boolean {
   return receiver < other
 }
 
-// Duration multiplication function 
-export function Duration_multiply(receiver: Duration, multiplier: number): Duration {
+// Duration multiplication function
+export function Duration_multiply(
+  receiver: Duration,
+  multiplier: number,
+): Duration {
   return receiver * multiplier
 }
 
@@ -775,7 +789,7 @@ export function Now(): Time {
     monotonic = performance.now() * 1000000
   }
 
-  return new Time(date, 0, monotonic)
+  return Time.create(date, 0, monotonic)
 }
 
 // Date returns the Time corresponding to
@@ -820,7 +834,7 @@ export function Date(
       Math.floor(nsec / 1000000),
     )
   }
-  return new Time(date, nsec % 1000000000, undefined, loc) // No monotonic reading
+  return Time.create(date, nsec % 1000000000, undefined, loc) // No monotonic reading
 }
 
 // Common locations
@@ -871,13 +885,13 @@ export const Kitchen = '3:04PM'
 export function Unix(sec: number, nsec: number = 0): Time {
   const ms = sec * 1000 + Math.floor(nsec / 1000000)
   const remainingNsec = nsec % 1000000
-  return new Time(new globalThis.Date(ms), remainingNsec, undefined, UTC)
+  return Time.create(new globalThis.Date(ms), remainingNsec, undefined, UTC)
 }
 
 // UnixMilli returns the local Time corresponding to the given Unix time,
 // msec milliseconds since January 1, 1970 UTC
 export function UnixMilli(msec: number): Time {
-  return new Time(new globalThis.Date(msec), 0, undefined, UTC)
+  return Time.create(new globalThis.Date(msec), 0, undefined, UTC)
 }
 
 // UnixMicro returns the local Time corresponding to the given Unix time,
@@ -885,7 +899,7 @@ export function UnixMilli(msec: number): Time {
 export function UnixMicro(usec: number): Time {
   const ms = Math.floor(usec / 1000)
   const nsec = (usec % 1000) * 1000
-  return new Time(new globalThis.Date(ms), nsec, undefined, UTC)
+  return Time.create(new globalThis.Date(ms), nsec, undefined, UTC)
 }
 
 // UnixNano returns the local Time corresponding to the given Unix time,
@@ -893,7 +907,7 @@ export function UnixMicro(usec: number): Time {
 export function UnixNano(nsec: number): Time {
   const ms = Math.floor(nsec / 1000000)
   const remainingNsec = nsec % 1000000
-  return new Time(new globalThis.Date(ms), remainingNsec, undefined, UTC)
+  return Time.create(new globalThis.Date(ms), remainingNsec, undefined, UTC)
 }
 
 // ParseDuration parses a duration string
@@ -965,7 +979,7 @@ export function ParseInLocation(
         `parsing time "${value}" as "${layout}": cannot parse`,
       )
     }
-    return new Time(date, 0, undefined, loc)
+    return Time.create(date, 0, undefined, loc)
   }
 
   if (layout === DateTime || layout === '2006-01-02 15:04:05') {
@@ -979,7 +993,7 @@ export function ParseInLocation(
         `parsing time "${value}" as "${layout}": cannot parse`,
       )
     }
-    return new Time(date, 0, undefined, loc)
+    return Time.create(date, 0, undefined, loc)
   }
 
   // Fallback to standard Date parsing
@@ -993,7 +1007,7 @@ export function ParseInLocation(
       `parsing time "${value}" as "${layout}": cannot parse`,
     )
   }
-  return new Time(date, 0, undefined, loc)
+  return Time.create(date, 0, undefined, loc)
 }
 
 // After waits for the duration to elapse and then returns the current time

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "goscript",
   "description": "Go to TypeScript transpiler",
-  "version": "0.0.49",
+  "version": "0.0.51",
   "author": {
     "name": "Aperture Robotics LLC.",
     "email": "support@aperture.us",
@@ -61,7 +61,7 @@
     "release:commit": "git reset && git add package.json && git commit -s -m \"release: v$npm_package_version\" && git tag v$npm_package_version",
     "release:publish": "git push && git push --tags && npm run build && npm publish",
     "lint": "npm run lint:go && npm run lint:js",
-    "lint:go": "make lint",
+    "lint:go": "golangci-lint run .",
     "lint:js": "eslint -c eslint.config.mjs ./",
     "lint:js:fix": "eslint -c eslint.config.mjs ./ --fix",
     "prepare": "husky",

package/gs/TODO.md

@@ -1,129 +0,0 @@
-# TODO: Wrapper Type Refactor for gs/ Directory
-
-## Overview
-The compiler has been refactored to generate wrapper types as type aliases instead of classes, with methods as standalone functions. This requires corresponding updates to the gs/ directory to remove the old class-based implementations and valueOf type parameters.
-
-## Required Changes
-
-### 1. Remove Wrapper Type Classes
-All wrapper type classes in gs/ need to be converted to type aliases with standalone functions:
-
-**Current pattern (to be removed):**
-```typescript
-export class FileMode {
-  constructor(private _value: number) {}
-  
-  valueOf(): number {
-    return this._value;
-  }
-  
-  String(): string {
-    return fileModeString(this._value);
-  }
-  
-  IsDir(): boolean {
-    return (this._value & ModeDir) !== 0;
-  }
-}
-```
-
-**New pattern (to implement):**
-```typescript
-export type FileMode = number;
-
-export function FileMode_String(receiver: FileMode): string {
-  return fileModeString(receiver);
-}
-
-export function FileMode_IsDir(receiver: FileMode): boolean {
-  return (receiver & ModeDir) !== 0;
-}
-```
-
-### 2. Files that need wrapper type refactoring:
-- `gs/os/types_js.gs.ts` - FileMode and other os types
-- `gs/time/time.ts` - Duration and other time types
-- `gs/syscall/` - Various syscall wrapper types
-- Any other files that implement wrapper types as classes
-
-### 3. Remove valueOf Type Parameters
-The `@builtin` package needs to be updated to remove valueOf type parameters from builtin functions.
-
-**Current pattern (to be removed):**
-```typescript
-function int<T extends { valueOf(): number }>(value: T): number {
-  return value.valueOf();
-}
-```
-
-**New pattern (to implement):**
-```typescript
-function int(value: number): number {
-  return value;
-}
-```
-
-### 4. Update Method Call Sites
-Any existing method call sites in gs/ files need to be updated:
-
-**Current pattern (to be updated):**
-```typescript
-mode.String()  // method call on wrapper class
-```
-
-**New pattern:**
-```typescript
-FileMode_String(mode)  // function call with receiver
-```
-
-### 5. Remove .valueOf() Calls
-The compiler still generates `.valueOf()` calls for wrapper types. These need to be eliminated:
-
-**Current generated pattern (to be fixed):**
-```typescript
-return receiver.valueOf() & 0o111;
-```
-
-**Target pattern:**
-```typescript
-return receiver & 0o111;
-```
-
-## Implementation Notes
-
-### Analysis Changes Completed
-- ✅ Modified analysis to detect wrapper types ahead of time
-- ✅ Added receiver mapping for wrapper function identifiers
-- ✅ Removed shadowingContext dependency
-
-### Compiler Changes Completed
-- ✅ Updated `WriteNamedTypeWithMethods()` to generate type aliases
-- ✅ Updated method call handling to use function calls
-- ✅ Updated type conversion logic for wrapper types
-- ✅ Removed constructor generation for wrapper types
-
-### Remaining Compiler Issues
-- ❌ Stop generating `.valueOf()` calls for wrapper types in:
-  - `compiler/expr.go` - `needsValueOfForBitwiseOp()` function
-  - `compiler/expr-call-type-conversion.go` - various conversion functions
-  - Need to update logic to not add `.valueOf()` for wrapper types since they're now primitive types
-
-### Testing
-Use the `wrapper_type_args` compliance test to verify changes:
-```bash
-go test -timeout 30s -run ^TestCompliance/wrapper_type_args$ ./compiler
-```
-
-## Migration Strategy
-1. First complete the compiler changes to stop generating `.valueOf()` calls
-2. Update each gs/ package incrementally 
-3. Test each package after refactoring
-4. Update builtin functions to remove valueOf type parameters
-5. Run full test suite to ensure compatibility
-
-## Benefits After Completion
-- Cleaner generated TypeScript code
-- Better performance (no wrapper class overhead)
-- More idiomatic TypeScript types
-- Simplified runtime type system
-- Easier debugging and maintenance