Skip to content

Enabled common lua modules#751

Draft
adriengivry wants to merge 1 commit intomainfrom
feature/enable_lua_modules
Draft

Enabled common lua modules#751
adriengivry wants to merge 1 commit intomainfrom
feature/enable_lua_modules

Conversation

@adriengivry
Copy link
Copy Markdown
Member

@adriengivry adriengivry commented Apr 21, 2026

Description

This PR enables the following modules:

  • sol::lib::base
  • sol::lib::math
  • sol::lib::coroutine
  • sol::lib::io
  • sol::lib::os
  • sol::lib::package
  • sol::lib::string
  • sol::lib::table
  • sol::lib::debug

To-Do

  • Validate that we want to allow scripts to use os and package (potential security issues?)
  • Investigate if we can/should modify the workdir for lua scripts (e.g. touch foo.txt creates a files next to the OvEditor executable)

Related Issue(s)

Fixes #246

Review Guidance

Write here.

Screenshots/GIFs

Write here.

AI Usage Disclosure

None

Checklist

  • My code follows the project's code style guidelines
  • When applicable, I have commented my code, particularly in hard-to-understand areas
  • When applicable, I have updated the documentation accordingly
  • My changes don't generate new warnings or errors
  • I have reviewed and take responsibility for all code in this PR (including any AI-assisted contributions)

@adriengivry adriengivry self-assigned this Apr 21, 2026
@adriengivry
Copy link
Copy Markdown
Member Author

@copilot Can you check how we could control what the workdir of lua script is? Like when we execute os.execute("touch foo.txt"), foo.txt is being created next to the executable, but it would be better if it was created under the project folder maybe?

@adriengivry adriengivry force-pushed the feature/enable_lua_modules branch from 278c935 to 5b67362 Compare April 23, 2026 14:58
@Overload-Technologies Overload-Technologies deleted a comment from Copilot AI Apr 23, 2026
@Overload-Technologies Overload-Technologies deleted a comment from Copilot AI Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Add more Lua libraries

1 participant