Skip to content

Fix more puter.js typescript types#2181

Merged
ProgrammerIn-wonderland merged 8 commits intoHeyPuter:mainfrom
reynaldichernando:types
Dec 16, 2025
Merged

Fix more puter.js typescript types#2181
ProgrammerIn-wonderland merged 8 commits intoHeyPuter:mainfrom
reynaldichernando:types

Conversation

@reynaldichernando
Copy link
Copy Markdown
Member

  • networking
    • rename the types
    • adjust on error param type
  • os (we don't expose this, but might as well)
    • fix import
    • remove constructor and global function
  • ui
    • add missing params and methods
  • util (we don't expose this aswell)
  • workers
    • remove constructor and global function
  • puter global
    • remove constructor
    • fix weird networking import
  • apps
    • update AppRecord -> App
    • remove constructor and global function

notes:

  • i'm doing changes that are obvious here, haven't gotten into the actual implementation, where usually there are more params and overloads, etc
  • will do another pass on this to make sure i didn't miss any
  • other than this, i haven't touched perms and drivers

Copy link
Copy Markdown
Collaborator

@ProgrammerIn-wonderland ProgrammerIn-wonderland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Internally it is called PTLSSocket, and TLSSocket generally means something else (node's socket) not sure if this is a good idea. Same with Socket/Psocket

@reynaldichernando
Copy link
Copy Markdown
Member Author

thanks for the review @ProgrammerIn-wonderland ,i've gone ahead and updated them

@ProgrammerIn-wonderland ProgrammerIn-wonderland merged commit dc827f9 into HeyPuter:main Dec 16, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants