goscript 0.0.49 → 0.0.50

package/gs/context/context.ts

@@ -1,22 +1,15 @@
 import * as $ from '@goscript/builtin/index.js'
 
-export const Canceled = new Error('context canceled')
-Canceled.name = 'CanceledError'
+export const Canceled = $.newError('context canceled')
 
-export class DeadlineExceededError extends Error {
-  constructor() {
-    super('context deadline exceeded')
-    this.name = 'DeadlineExceededError'
-  }
-}
-export const DeadlineExceeded = new DeadlineExceededError()
+export const DeadlineExceeded = $.newError('context deadline exceeded')
 
 // Function types
 export type CancelFunc = () => void
-export type CancelCauseFunc = (cause: Error | null) => void
+export type CancelCauseFunc = (cause: $.GoError) => void
 
 // Context interface matching Go's context.Context
-export interface Context {
+export type Context = null | {
   // Deadline returns the time when work done on behalf of this context should be canceled
   Deadline(): [Date | null, boolean]
 
@@ -24,17 +17,20 @@ export interface Context {
   Done(): $.Channel<{}>
 
   // Err returns a non-nil error value after Done is closed
-  Err(): Error | null
+  Err(): $.GoError
 
   // Value returns the value associated with this context for key, or null
   Value(key: any): any
 }
 
+// ContextNonNil is a non-nil context
+export type ContextNonNil = Exclude<Context, null>
+
 // Base implementation for all contexts
-abstract class baseContext implements Context {
+abstract class baseContext implements ContextNonNil {
   abstract Deadline(): [Date | null, boolean]
   abstract Done(): $.Channel<{}>
-  abstract Err(): Error | null
+  abstract Err(): $.GoError
   abstract Value(key: any): any
 }
 
@@ -54,7 +50,7 @@ class backgroundContext extends baseContext {
     return backgroundContext.neverClosedChannel
   }
 
-  Err(): Error | null {
+  Err(): $.GoError {
     return null
   }
 
@@ -66,14 +62,14 @@ class backgroundContext extends baseContext {
 // Value context wraps a parent and adds a key-value pair
 class valueContext extends baseContext {
   constructor(
-    private parent: Context,
+    private parent: ContextNonNil,
     private key: any,
     private val: any,
   ) {
     super()
   }
 
-  getParent(): Context {
+  getParent(): ContextNonNil {
     return this.parent
   }
 
@@ -85,7 +81,7 @@ class valueContext extends baseContext {
     return this.parent.Done()
   }
 
-  Err(): Error | null {
+  Err(): $.GoError {
     return this.parent.Err()
   }
 
@@ -100,14 +96,14 @@ class valueContext extends baseContext {
 // Cancel context that can be canceled
 class cancelContext extends baseContext {
   protected doneChannel: $.Channel<{}>
-  protected err: Error | null = null
-  protected cause: Error | null = null
+  protected err: $.GoError = null
+  protected cause: $.GoError = null
   protected children: Set<cancelContext> = new Set()
-  protected parent: Context
+  protected parent: ContextNonNil
   protected parentCancelCtx: cancelContext | null = null
   protected removeFromParent: (() => void) | null = null
 
-  constructor(parent: Context) {
+  constructor(parent: ContextNonNil) {
     super()
     this.parent = parent
     this.doneChannel = $.makeChannel<{}>(0, {}, 'both')
@@ -121,7 +117,7 @@ class cancelContext extends baseContext {
     return this.doneChannel
   }
 
-  Err(): Error | null {
+  Err(): $.GoError {
     return this.err
   }
 
@@ -129,14 +125,11 @@ class cancelContext extends baseContext {
     return this.parent.Value(key)
   }
 
-  getCause(): Error | null {
-    if (this.cause !== null) {
-      return this.cause
-    }
-    return this.err
+  getCause(): $.GoError {
+    return this.cause ?? this.err
   }
 
-  cancel(removeFromParent: boolean, err: Error, cause: Error | null): void {
+  cancel(removeFromParent: boolean, err: $.GoError, cause: $.GoError): void {
     if (this.err !== null) {
       return // Already canceled
     }
@@ -205,7 +198,7 @@ class timerContext extends cancelContext {
   private deadline: Date
   private timer: any
 
-  constructor(parent: Context, deadline: Date) {
+  constructor(parent: ContextNonNil, deadline: Date) {
     super(parent)
     this.deadline = deadline
   }
@@ -229,7 +222,7 @@ class timerContext extends cancelContext {
     }, duration)
   }
 
-  cancel(removeFromParent: boolean, err: Error, cause: Error | null): void {
+  cancel(removeFromParent: boolean, err: $.GoError, cause: $.GoError): void {
     super.cancel(removeFromParent, err, cause)
     if (this.timer) {
       clearTimeout(this.timer)
@@ -240,7 +233,7 @@ class timerContext extends cancelContext {
 
 // Without cancel context - inherits values but not cancellation
 class withoutCancelContext extends baseContext {
-  constructor(private parent: Context) {
+  constructor(private parent: ContextNonNil) {
     super()
   }
 
@@ -252,7 +245,7 @@ class withoutCancelContext extends baseContext {
     return backgroundContext.getNeverClosedChannel()
   }
 
-  Err(): Error | null {
+  Err(): $.GoError {
     return null
   }
 
@@ -276,7 +269,10 @@ export function TODO(): Context {
 }
 
 // WithCancel returns a copy of parent with a new Done channel
-export function WithCancel(parent: Context): [Context, CancelFunc] {
+export function WithCancel(parent: Context): [ContextNonNil, CancelFunc] {
+  if (parent === null) {
+    throw new Error('cannot create context from nil parent')
+  }
   const ctx = new cancelContext(parent)
   ctx.propagateCancel()
 
@@ -289,20 +285,28 @@ export function WithCancel(parent: Context): [Context, CancelFunc] {
 }
 
 // WithCancelCause returns a copy of parent with a new Done channel and cause recording
-export function WithCancelCause(parent: Context): [Context, CancelCauseFunc] {
+export function WithCancelCause(
+  parent: Context,
+): [ContextNonNil, CancelCauseFunc] {
+  if (parent === null) {
+    throw new Error('cannot create context from nil parent')
+  }
   const ctx = new cancelContext(parent)
   ctx.propagateCancel()
 
   return [
     ctx,
-    (cause: Error | null) => {
+    (cause: $.GoError) => {
       ctx.cancel(true, Canceled, cause)
     },
   ]
 }
 
 // WithDeadline returns a copy of parent with the deadline adjusted to be no later than d
-export function WithDeadline(parent: Context, d: Date): [Context, CancelFunc] {
+export function WithDeadline(
+  parent: Context,
+  d: Date,
+): [ContextNonNil, CancelFunc] {
   return WithDeadlineCause(parent, d, null)
 }
 
@@ -310,8 +314,11 @@ export function WithDeadline(parent: Context, d: Date): [Context, CancelFunc] {
 export function WithDeadlineCause(
   parent: Context,
   d: Date,
-  cause: Error | null,
-): [Context, CancelFunc] {
+  cause: $.GoError,
+): [ContextNonNil, CancelFunc] {
+  if (parent === null) {
+    throw new Error('cannot create context from nil parent')
+  }
   // Check if parent deadline is already earlier
   const [parentDeadline, ok] = parent.Deadline()
   if (ok && parentDeadline && parentDeadline <= d) {
@@ -335,7 +342,7 @@ export function WithDeadlineCause(
 export function WithTimeout(
   parent: Context,
   timeout: number,
-): [Context, CancelFunc] {
+): [ContextNonNil, CancelFunc] {
   return WithDeadline(parent, new Date(Date.now() + timeout))
 }
 
@@ -343,23 +350,31 @@ export function WithTimeout(
 export function WithTimeoutCause(
   parent: Context,
   timeout: number,
-  cause: Error | null,
-): [Context, CancelFunc] {
+  cause: $.GoError,
+): [ContextNonNil, CancelFunc] {
   return WithDeadlineCause(parent, new Date(Date.now() + timeout), cause)
 }
 
 // WithValue returns a copy of parent with the value associated with key
-export function WithValue(parent: Context, key: any, val: any): Context {
+export function WithValue(parent: Context, key: any, val: any): ContextNonNil {
+  if (parent === null) {
+    throw new Error('cannot create context from nil parent')
+  }
   return new valueContext(parent, key, val)
 }
 
 // WithoutCancel returns a context that inherits values but not cancellation
-export function WithoutCancel(parent: Context): Context {
+export function WithoutCancel(parent: Context): ContextNonNil {
+  if (parent === null) {
+    throw new Error('cannot create context from nil parent')
+  }
   return new withoutCancelContext(parent)
 }
 
 // Cause returns the underlying cause of the context's cancellation
-export function Cause(ctx: Context): Error | null {
+export function Cause(ctx: Context): $.GoError {
+  if (!ctx) return null
+
   let c = ctx
   // Unwrap value contexts
   while (c instanceof valueContext) {
@@ -375,6 +390,9 @@ export function Cause(ctx: Context): Error | null {
 
 // AfterFunc runs f in a separate goroutine after ctx is done
 export function AfterFunc(ctx: Context, f: () => void): () => boolean {
+  if (ctx === null) {
+    throw new Error('cannot create context from nil parent')
+  }
   let stopped = false
   let done = false
 

package/gs/fmt/fmt.ts

@@ -202,7 +202,11 @@ let stdout = {
   write: (data: string) => {
     // Use process.stdout.write if available (Node.js), otherwise fallback to console.log
     // but we need to avoid adding extra newlines that console.log adds
-    if (typeof process !== 'undefined' && process.stdout && process.stdout.write) {
+    if (
+      typeof process !== 'undefined' &&
+      process.stdout &&
+      process.stdout.write
+    ) {
       process.stdout.write(data)
     } else {
       // In browser environments, we need to use console.log but handle newlines carefully

package/gs/reflect/type.ts

@@ -15,12 +15,7 @@ export class rtype {
 
   Pointers(): boolean {
     const k = this.kind
-    return (
-      k === Ptr ||
-      k === Map ||
-      k === Slice ||
-      k === Interface
-    )
+    return k === Ptr || k === Map || k === Slice || k === Interface
   }
 }
 
@@ -127,13 +122,13 @@ export const BothDir: ChanDir = 3
 export function ChanDir_String(d: ChanDir): string {
   switch (d) {
     case RecvDir:
-      return "RecvDir"
+      return 'RecvDir'
     case SendDir:
-      return "SendDir"  
+      return 'SendDir'
     case BothDir:
-      return "BothDir"
+      return 'BothDir'
     default:
-      return "ChanDir(" + d + ")"
+      return 'ChanDir(' + d + ')'
   }
 }
 

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.50",
   "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