Search results

And a fix for several require bugs. You can test with svn checkout https://github.com/phulin/bean-daily/branches/main/build and test-require. Incidentally, the project I stuck this in is an example of how you can set up modern JS tooling for relay script stuff. Everything is written in...

This patch should handle relay scripts correctly, both override scripts and those from the menu. Notably, in order to avoid breaking backwards compatibility with e.g. Guide, I've added a new GET argument to the relay menu scripts that triggers execution of any relay_x.js script vs simply...

Here is another patch to implement require() for both ASH + JS scripts. Can be tested with require("zlib.ash").getvar("verbosity") and require("test-require.js").main(0, Monster.get("zobmie"), "") with the below version of test-require.js installed.

This is a big patch to set up consult scripts, "lifecycle" scripts (choiceAdventureScript, betweenBattleScript, etc.), and combat filter functions. I think this mostly completes the refactoring to pull all the non-ASH-specific stuff apart from the ASH code, but it's still a little bit messy...

When updated, scripts that use file_to_buffer to load data files from disk will continue to load old versions until Mafia is closed, because the cache isn't cleared until Mafia is restarted. The shorts GUI uses a data file to store the HTML snippet it inserts into the page. If that's outdated...

Yeah, that's a frequent problem that results from Mafia's file cache, which should probably just get emptied any time there's an SVN update. I'll put submitting a patch for that on my todo list.

This fixes the bad error message when using a nonexistent function on the runtime library.

Can you double check that you are running the latest version?

I wasn't planning on adding more pockets directly to the GUI to keep the UI uncluttered - at least for the in-run view. I think ~all the items you mention should be available in the aftercore view, though. If you have an idea for how UI should work to add additional pocket buttons, I'd be happy...

Here is an actually correct updated .classpath file. The line endings should be OK....

MCroft has helpfully committed as 20509

This is due to the bug in https://kolmafia.us/threads/javascript-bugs.25638/#post-160189

Fairly critical bug that breaks the Ezandora choice override script arrangement. Fixed in this patch. Parent ash interpreters weren't being passed down to children run via cli_execute, so the children could not access the relay request. This patch rewires that together. This was an error in my...

This is a tracking thread for early JavaScript bugs, since I expect there to be quite a few. Error messages when a value is missing on one of the globals are garbage. Translate record types for e.g. maximize return value. jsref command like ashref. Wiki documentation. (first stab here) Rhino...

Thanks so much, frono. I will start a separate thread to track bugs.

Is this ready to merge in your assessment? Are we still waiting on input from veracity?

Hm, it applied cleanly for me, but on double check there were some issues with the build.xml changes. Here's an updated patch. I executed the following commands to apply it. Please let me know if this does not work for you. svn checkout https://svn.code.sf.net/p/kolmafia/code/ kolmafia-code cd...

Oh, and I am quite sure I used svn add before generating the latest couple of patch files. If something is still not applying cleanly, please let me know.

Thanks, frono, and thanks for all the review you've done so far. Here's a patch that has rhino in a src/jar/ directory (placing the jar still needs to be done manually). Can split this up into a sequence of patches if you'd like, but i suspect it would be easier for someone with commit access to...

It is not only needed at runtime. The code to run JS is deeply intertwined with Rhino and needs to know about Rhino changes at both compile time and runtime. Re: MCroft, I think the risk of namespace collisions is pretty limited. Everything still needs to be imported via its fully-qualified...