Merge pull request #330 from bmos/patchright

Direct uploading of builds via API

Author: bmos

README.md

@@ -40,43 +40,41 @@ source .venv/bin/activate # Linux or macOS
 ```
 
 ```shell
-uv pip install -e .
+uv pip install .
 ```
 
 ## Usage
 
-1. Create a `.env` file in the project folder containing the following (but with your information):
-
-> [!NOTE]
-> You can add these values directly to your environment variables.
-
-```env
-# your FG forum username
-FG_USER_NAME=**********
-# your FG forum password
-FG_USER_PASS=**********
-# the item ID of the FG Forge item you want to modify
-FG_ITEM_ID=33
-# the name(s) of the (supported -- ext, pak, mod, etc) file(s) you want to upload (can be comma-separated list)
-FG_UL_FILE=path/to/file.ext
-
-# [OPTIONAL] set this to FALSE to skip build uploading
-FG_UPLOAD_BUILD=TRUE
-# [OPTIONAL] set this to "TEST" or "NONE" if you would rather target those channels
-FG_RELEASE_CHANNEL=LIVE
-
-# [OPTIONAL] set this to TRUE to prevent replacing the description with the contents of README.md
-FG_README_UPDATE=FALSE
-# [OPTIONAL] set this to TRUE to remove images instead of creating links
-FG_README_NO_IMAGES=FALSE
-
-# [OPTIONAL] set this to TRUE to generate a "cumulative-sales.png" image
-FG_GRAPH_SALES=FALSE
-```
+1. Put the ext file to upload into the project folder.
+
+2. [OPTIONAL] Create a `.env` file in the project folder containing the following (but with your information):
+
+    > [!NOTE]
+    > You can add these values directly to your environment variables.
+
+    ```env
+    # your FG forum username
+    FG_USER_NAME=**********
+    # your FG forum password
+    FG_USER_PASS=**********
+    # the item ID of the FG Forge item you want to modify
+    FG_ITEM_ID=33
+    # the name(s) of the (supported -- ext, pak, mod, etc) file(s) you want to upload (can be comma-separated list)
+    FG_UL_FILE=path/to/file.ext
+
+    # [OPTIONAL] set this to FALSE to skip build uploading
+    FG_UPLOAD_BUILD=TRUE
+    # [OPTIONAL] set this to "TEST" or "NONE" if you would rather target those channels
+    FG_RELEASE_CHANNEL=LIVE
+
+    # [OPTIONAL] set this to TRUE to prevent replacing the description with the contents of README.md
+    FG_README_UPDATE=FALSE
+    # [OPTIONAL] set this to TRUE to remove images instead of creating links
+    FG_README_NO_IMAGES=FALSE
+    ```
 
-2. Put an ext file to upload into the project folder.
 3. Run the following command from inside the project folder:
 
 ```shell
-python -m src.main
+fg-forge-uploader
 ```

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "fg-forge-updater"
-version = "1.4.0"
+version = "2.0.0"
 readme = "README.md"
 requires-python = ">=3.11"
 license = { file = "LICENSE" }
@@ -9,23 +9,18 @@ urls = { Repository = "https://github.com/bmos/fg_forge_updater" }
 dependencies = [
     "beautifulsoup4==4.14.3",
     "markdown==3.10",
-    "matplotlib==3.10.7",
     "mdformat-footnote==0.1.2",
     "mdformat-frontmatter==2.0.8",
     "mdformat-gfm==1.0.0",
-    "pandas==2.3.3",
+    "patchright>=1.56.0",
     "python-dotenv==1.2.1",
-    "requestium @ git+https://github.com/tryolabs/requestium.git@38f5b174727795d6d3ee3f6f4508a152ff5fd0ff", #  0.5.0 (unreleased)
-    "seaborn==0.13.2",
-    "selenium==4.38.0",
+    "requests>=2.32.5",
 ]
 
 [project.optional-dependencies]
 dev = [
     "bandit[sarif]==1.9.2",
     "mypy==1.19.0",
-    "pandas-stubs==2.3.3.251201",
-    "polyfactory==3.1.0",
     "pre-commit==4.5.0",
     "pytest-cov==7.0.0",
     "pytest-xdist==3.8.0",
@@ -35,11 +30,21 @@ dev = [
     "types-Markdown==3.10.0.20251106",
     "types-beautifulsoup4==4.12.0.20250516",
     "types-requests==2.32.4.20250913",
-    "types-seaborn==0.13.2.20251214",
 ]
 
+[project.scripts]
+fg-forge-updater = "src.main:main"
+
+[build-system]
+requires = ["setuptools>=80.0"]
+build-backend = "setuptools.build_meta"
+
 [tool]
 
+[tool.setuptools.packages.find]
+where = ["."]
+include = ["src*"]
+
 [tool.ruff]
 line-length = 160
 include = [
@@ -107,14 +112,13 @@ ignore = [
 
 [tool.ruff.lint.per-file-ignores]
 "**/{tests,docs}/*" = [
+    "FBT001",  # boolean-type-hint-positional-argument (FBT001)
     "SLF001",  # private-member-access (SLF001)
     "PLR2004",  # magic-value-comparison (PLR2004)
 ]
 
 [tool.pytest]
 addopts = ["-n", "auto", "--cov=src", "--no-cov-on-fail"]
-asyncio_mode = "auto"
-asyncio_default_fixture_loop_scope = "module"
 testpaths = [
     "tests",
 ]

src/build_processing.py

@@ -44,10 +44,15 @@ def readme_html(markdown_text: str, *, no_images: bool = False) -> str:
 
 
 def get_readme(new_files: list[Path], *, no_images: bool = False, readme_name: str = "README.md") -> str:
-    """Read the first README.md found in the new zip files and call readme_html() returning the result."""
-    return next(
-        (readme_html(ZipFile(file).read(readme_name).decode("UTF-8"), no_images=no_images) for file in new_files if readme_name in ZipFile(file).namelist()), ""
-    )
+    """Extract and convert README file from the first zip file containing one."""
+    for file in new_files:
+        with ZipFile(file) as zf:
+            if readme_name in zf.namelist():
+                markdown_text = zf.read(readme_name).decode("UTF-8")
+                return readme_html(markdown_text, no_images=no_images)
+
+    error_msg = "No README file found."
+    raise FileNotFoundError(error_msg)
 
 
 def get_build(file_path: PurePath, env_file: str) -> Path: