Before You Begin

• I confirm that I have downloaded the latest version of the addon.
• I am not playing on a private server.
• I checked for an existing, open ticket for this request and was not able to find one.
• I edited the title of this feature request (above) so that it describes the issue I am reporting.

Feature Request

I'd like to see the ability to generate action packs from priorities without needing the WoW client to be open. This helps improve maintenance of the addon and with easing updates to the latest SimC priorities.

Additional Information

I think the following steps need to be taken:

• Move the action pack description into a header comment in the associated priority file. This makes the priority file hold all of the information in the action pack.
• Move the actual action packs from the various spec files into a separate directory, e.g., TheWarWithin/ActionPacks/DeathKnightBlood.lua, etc. This is tremendously helpful for minimizing diffs to the actual spec files as the actual action pack lines are just generated noise in the diffs.
• Separate the importer and exporter into a standalone module.
• Write a linter that can strictly verify the syntax of a priority and also eventually check that all abilities and auras named in the priority are defined.
• Write a Lua script that can generate a Lua file containing an action pack for an input file with the corresponding priority.

Contact Information

No response

Ah, yes, not sure what I was thinking there. We can only load Lua files, so the SimC APL would just be a string inside the Lua file, much like the current action pack, so the rest of the bits would just be key/value pairs inside a table that gets registered with the addon.

I'm opening this issue as a discussion point. I think this will be very useful for the project. I'm happy to devote a lot of time to submitting patches for review and getting this done.

That works, too. The import does cause extra loading time, so people will see longer loading screens as a result, but it has the benefit of simplicity, and not needing to synchronize between the APL and the action pack string.

As far as I know, we can only load .lua files, but your point stands. We can load them as .lua files.

We can make header comments in each Priority/<spec>.simc file that hold all of the information for a single action pack in a structured comment, then parse that header comment when auto-importing.

I'll work on this for a pull request, and then a later one as well for a linter to verify the priority file.

I've debated going this route for quite a while.

Instead of ActionPacks, the folder should be Priorities.

Honestly, the shortest implementation of this just requires:

• Priority Name
• Priority Specialization
• Priority Version #
• Priority Description / Notes
• SimC APL as a long string [[...]]

Then, the work of RestoreDefaults in Classes.lua can just do the importing if (1) there isn't an existing priority, (2) the existing priority is missing version info, or (3) the existing priority's version # (date) is less than the file's version.

That doesn't solve for offline serialization, but we could still do validation of the priority itself later.