* REFACTOR: Move Result to the public API
This refactors Result<T> to be part of the public NetscriptDefinitions.
It is not used by anything in this PR, but it is planned to be used in
the autoinfil APIs, or lacking that, future APIs, so this should not be
exposing anything prematurely.
* Add @public as api-extractor suggested and generate docs
* Use import type
---------
Co-authored-by: CatLover <152669316+catloversg@users.noreply.github.com>
"Made with beginners in mind" might be a bit of an overstatement, but I'm not gonna quibble on the wording. (IMO we are less "let me hold your hand while you learn how to cast your fishing rod" and more "I shoved you into the pond, grab some fish or smth idk")
- change BN description to highlight SF levels impact other BN use of singularity functions
- add detail to singularity API description to highlight the same
Subscribing or unsubscribing from within an event handler (possibly
transitively) could cause issues or even infinite loops.
This was initially fixed in #2257, which exhibited such problems.
Currently no code is known to have this issue, but it could easily come
up again in the future.
This moves the implementation entirely into the PortHandle class, which
makes the code a bit more streamlined and will allow for other port
implementations in the future.
* REFACTOR: Rewrite infiltration to pull state out of React
Upcoming projects (auto-infil, the possibility of making infil a work
task, etc.) require infiltration state to transition with predictable
timing and be under our control, instead of inside React. This refactor
accomplishes this by pulling the state out into accompanying model
classes.
After this, infiltration can theoretically run headless (without UI),
although it doesn't actually, and you would quickly be
hospitalized due to failing all the minigames.
There should be no user-visible changes, aside from the progress-bars
scrolling much more smoothly.
* Fix console warning in InfiltrationRoot
It turns out true isn't actually a safe value to use in JSX, only false works.
* Fix up some comments
* Convert purchased server functions to cloud API
- Create `ns.cloud`
- Change `bitnode multipliers` and `server constants` wording for consistency
- change `server`, `ram` and `getting started` docs for consistency
- Added changes to 3.0.0 API Break and `setRemovedFunctions` in NetscriptFunctions.js
Tested by
- running tutorial `purchase-server-8gb.js`, and a more typical player one
- buying manually using vendor (Alpha Ent in Sector 12)
- deleting them all using script, and checked all deleted functions gave correct error
- Imported completed save to ensure auto-transfer of function work
* Revision in line with comments
- changed more `purchased` to `cloud` references
- Added BN mults auto-conversion
* Update getting_started.md
- Corrected function names for new `cloud API`
* Don't show `cloud API` warning
v3.0.0 API break auto-replaces `cloud` functions, not warning suggested.
* API Break correction
- `cloud` affected API break replacement changed to be more descriptive and functional
* Fix typo and add empty lines
* Update many things (check commit's description)
- Comments
- Terminal message
- UI Text
- TSDoc
- md docs
- Improve error messages in src\NetscriptFunctions\Cloud.ts
Michael Ficocelli
catloversg
David Walker
EntenBu
Jonathon Chase
Adam Weeden
Nicholas Colclasure
gmcew
Cica
Paul Setzer
CTN
sg673
maglinvinn
Michael Taylor
imcute_aaaa
Shy
Nick Shelley
Chaddeus