Added ordered question inserting feature and more configuration options

[HarrySu123]

Overview

• renaming of Module to Set
• implemented ordered question inserting
• added more configuration options for the api
• added relevant and correct python docstring where needed
• fixed test.yml to pass properly

renaming of Module to Set

The file/class Module.py has been renamed to set.py.
All reference to "module" has been changed to "set", to match the terminology of lambda feedback.

implemented ordered question inserting

Done inside json_convert.py

def _zip_sorted_folder(folder_path, zip_path):
"""Zips the contents of a folder, preserving the directory structure.
Args:
folder_path: The path to the folder to zip.
zip_path: The path where the zip file will be created.
"""
with zipfile.ZipFile(zip_path, "w") as zf:
for root, dirs, files in os.walk(folder_path):
# Sort files for deterministic, alphabetical order
for file in sorted(files):
abs_path = os.path.join(root, file)
rel_path = os.path.relpath(abs_path, folder_path)
zf.write(abs_path, arcname=rel_path)

used to create the zip folder and files, and order the files in deterministic, alphabetical order to achieve ordered question inserting.
called inside converter(),

added more configuration options for the api

lambda feedback, apart from the question.json file also uses a set.json file with extra settings like the name, description... etc.
json_convert/minimal_template_set.json contains the minimal_template_set.json
it is used similar to how monimal_template_question.json is used

The settings used for workedSolutionVisibility, structuredTutorialVisibility and chatbotVisibility all uses the same three options, OPEN, HIDE and OPEN_WITH_WARNINGS.

These three options is referred to using enum classes to avoid magic numbers. This can be found in api/visibility_status.py, along with a Controller that wraps around it to allow for custom methods and ease of conversion between the 3.

The relevant settings are also added to the class Set.py as attributes

fixed test.yml to pass properly

updates setuptools, wheel and virtualenv when installing python dependencies, ensuring it is always up to date.
fixed Linting Checks and pytests by running all the scripts inside the poetry virtual environment.

[Copilot]

Pull Request Overview

This pull request renames the Module class to Set to align with Lambda Feedback terminology, implements ordered question insertion for deterministic output, and adds configuration options for question visibility settings.

Key changes:

• Comprehensive rename from "Module" to "Set" across the codebase
• Added deterministic ZIP file creation with alphabetical ordering for question insertion
• Introduced visibility status management with enum-based configuration options

Reviewed Changes

Copilot reviewed 18 out of 19 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
pyproject.toml	Added requests version constraint and isort configuration
in2lambda/main.py	Updated imports and references from Module to Set
in2lambda/json_convert/minimal_template_set.json	New template file for set configuration
in2lambda/json_convert/minimal_template_question.json	Restructured question template with new fields
in2lambda/json_convert/json_convert.py	Implemented ordered ZIP creation and Set-based conversion
in2lambda/filters/markdown.py	Updated filter signatures from Module to Set
in2lambda/filters/*/filter.py	Updated filter implementations to use Set instead of Module
in2lambda/api/visibility_status.py	New visibility status enum and controller classes
in2lambda/api/set.py	New Set class replacing Module with visibility configuration
in2lambda/api/question.py	Updated documentation reference from module to set
in2lambda/api/module.py	Removed original Module class file
docs/source/contributing/documentation.md	Updated documentation references
.github/workflows/test.yml	Fixed CI configuration to use poetry virtual environment

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The VisibilityStatus class is missing a docstring. Add a comprehensive docstring explaining the purpose and usage of the three visibility states (OPEN, HIDE, OPEN_WITH_WARNINGS).

Suggested change

	"""Enum representing the visibility status of a question or set."""
	"""
	Enumrepresentingthevisibilitystatusofaquestionorset.
	Thevisibilitystatusdetermineshowaquestionorsetispresentedtousers:
	-OPEN: Thequestionorsetisfullyvisibleandaccessibletouserswithoutanyrestrictions.
	Usethisstatuswhenthecontentisreadyandshouldbeavailabletoallintendedusers.
	-HIDE: Thequestionorsetishiddenfromusersandcannotbeaccessed.
	Usethisstatustotemporarilyorpermanentlyremovecontentfromview, suchasduringrevieworifitisdeprecated.
	-OPEN_WITH_WARNINGS: Thequestionorsetisvisibleandaccessible, butusersareshownwarnings.
	Usethisstatuswhenthecontentisavailablebutmayhaveissues, suchasincompleteinformation, potentialerrors, orothercaveatsthatusersshouldbeawareof.
	Thisenumisusedtocontrolthedisplayandaccessibilityofquestionsandsetsthroughouttheapplication.
	"""

[Copilot]

The VisibilityController class is missing a docstring. Add a docstring explaining its role as a wrapper around VisibilityStatus and how it provides convenient methods for status management.

Suggested change

	"""Controller for managing visibility status with easy-to-use methods."""
	"""
	AwrapperaroundtheVisibilityStatusenumthatprovidesconvenientmethodsformanaging
	thevisibilitystatusofquestionsandsets.
	Thiscontrollerallowsyoutoeasilychangethestatususingmethodssuchas`to_open()`,
	`to_hide()`, and`to_open_with_warnings()`, andaccessthecurrentstatusviathe`status`
	property. Itisdesignedtosimplifystatusmanagementandserializationforvisibilitylogic.
	"""

[Copilot]

The function parameters lack type hints. Add type annotations: folder_path: str, zip_path: str.

Suggested change

	def_zip_sorted_folder(folder_path, zip_path):
	def_zip_sorted_folder(folder_path: str, zip_path: str) ->None:

[Copilot]

The filename generation doesn't handle special characters or empty titles properly. Consider using re.sub(r'[^\w\-_.]', '_', output['title']) to sanitize the title and provide a fallback for empty titles.

[peterbjohnson]

@HarrySu123 all looks good to me, thanks. Very thorough, and neat and tidy. I've checked all the changes but no comments.

Check the copilot comments. The docstrings probably aren't necessary, but catching characters in file names might be helpful: #19 (comment).

Assuming you've tested this (!) I approve this for merging when you're ready.