ci: rewrite deploy scripts with python3

[ElectricalBoy]

Summary

Previous discussion: https://discord.com/channels/321003439431745537/321641940879802368/1468407645391163432

This PR rewrites shellscript versions of current deploy scripts with Python.

How did you test this change?

• https://github.com/ElectricalBoy/Lua-Modules/actions/runs/21696696866
• https://github.com/ElectricalBoy/Lua-Modules/actions/runs/21737290044

[hjpalpha]

i think we should test this on a fork first ;)

on first glance seems reasonable

[ElectricalBoy]

i think we should test this on a fork first ;)

running: https://github.com/ElectricalBoy/Lua-Modules/actions/runs/21672292444
cancelled for now; will run during EU night time

update: https://github.com/ElectricalBoy/Lua-Modules/actions/runs/21696696866

[Rathoz]

Would it be worth using pywikibot directly?

[hjpalpha]

Would it be worth using pywikibot directly?

imo too much overhead and would likely increase runtime even more
additionally it would not generate the same "log" outputs as currently

[FO-nTTaX]

Do we feel the need to have a linter and style checker for the deploy scripts? I'm not aware of such things for bash, but in the python world tools like Flake8 exist

[mbergen]

Do we feel the need to have a linter and style checker for the deploy scripts? I'm not aware of such things for bash, but in the python world tools like Flake8 exist

nowadays people use https://astral.sh/ruff instead

[ElectricalBoy]

Do we feel the need to have a linter and style checker for the deploy scripts? I'm not aware of such things for bash, but in the python world tools like Flake8 exist

I didn't pass the scripts through a linter but everything is style checked with psf/black already
I'm not opposed to adding a linter if there's a suggestion for one

[mbergen]

Suggested change

	on: [pull_request, workflow_dispatch]
	on:
	pull_request:
	paths:
	- '**.py'
	workflow_dispatch:

Not sure if we even need the dispatch?

[ElectricalBoy]

Not sure if we even need the dispatch?

it's there only because luacheck workflow has it
and I'm not opposed to removing dispatch trigger from both

[mbergen]

Could also move this into a dedicated context manager to avoid repetition of cookie reading, but is also a bit overkill

@contextmanager
def authenticated_session(wiki: str):
    token = get_token(wiki)
    with requests.Session() as session:
        session.headers.update(HEADERS)
        session.cookies = read_cookie_jar(wiki)
        yield session, token

Suggested change

	token = get_token(wiki)
	with requests.Session() as session:
	session.cookies = read_cookie_jar(wiki)
	with authenticated_session(wiki) as session, token:

In case we'd then also move the opening of this context manager to be in each main function, and then passed everywhere, we could also get rid of the cookie jar as the session will persist the cookies.

[ElectricalBoy]

yeah I think it's a bit overkill

[FO-nTTaX]

Do we feel the need to have a linter and style checker for the deploy scripts? I'm not aware of such things for bash, but in the python world tools like Flake8 exist

nowadays people use https://astral.sh/ruff instead

Fair enough, I'm not super up top date with the wider python ecosystem, flake8 is what I use for the LP Discord Bot, hence that suggestion. I'm not married to either tool, or to how far we'd want to go with implementing style checking and/or linting for this, I just wanted to start the discussion if this was relevant.