dev-contrib

[toxic wren]

wdym like a guideline on how to use AI in the server

!rule 10

[stable mountain]

Rules

10. Do not copy and paste answers from ChatGPT or similar AI tools.

[toxic wren]

that works

[toxic wren]

[snow yacht]

To me "answers" implies something else

I mean when people come on voice and ask for assistance with their project/scripts or w/e which is slop they don't understand

[patent pivot]

that is probably something to be written as a policy on site or an extension to the asking questions guide -- if that's something people feel is necessary then opening an issue on the meta repo is a good idea and staff will discuss it

[patent pivot]

i would look at the issues first, there are a lot of well specified feature requests and enhancements for a range of functionality

[snow yacht]

I ended up writing the following to do the message separation a bit more ergonomically:

with ordered_messages as (
    select id, created_at, ROW_NUMBER() over (order by created_at) - 1
    as message_index
from messages where author_id = '158020687850766338'),
base_time as (
    select created_at as start_time from ordered_messages where message_index = 0
),
adjusted_values as (
    select om.id, greatest(om.created_at, bt.start_time + om.message_index * INTERVAL '15 minutes') as new_timestamp
    from ordered_messages om
    cross join base_time bt
)
update messages
set created_at = av.new_timestamp
from adjusted_values av
where messages.id = av.id

still not quite happy with it though

[snow yacht]

is it worth cleaning this script up and putting it somewhere? I suspect the cleaner variant would be just having a script to replace the metricity data with known data, parameterized over author_id would be cleaner so you don't have to go to the server and spam messages to populate the table in the first place

[toxic wren]

!source exec

[stable mountain]

Bad argument

Unable to convert 'exec' to valid command, tag, or Cog.

[toxic wren]

!source eval

[stable mountain]

Command: eval

Run Python code and get the results.

Source Code

Go to GitHub

bot/exts/utils/snekbox/_cog.py:572

[toxic wren]

Is this is the change needed in bot#3390 ?

[dusky shore]

GitHub

[bot] #3390 Add !exec alias to clarify command behavior

[fallen patrol]

Hm

IMO command name should be exec and alias should be eval

[celest charm]

eval should be like exec but if the input is not a single expression, it permabans you

[toxic wren]

@patent pivot https://github.com/python-discord/bot/issues/3390#issuecomment-3341737512

[patent pivot]

for now just add an alias

[toxic wren]

ok making PR

[toxic wren]

bot#3392

[dusky shore]

GitHub

[bot] #3392 Add !exec alias to clarify command behavior

[toxic wren]

Works!

[vital escarp]

Hi guys , i wanted to help with the open source projects , any guidence (im a newbie)?

[toxic wren]

!contrib

[stable mountain]

Contribute to Python Discord's open source projects

Looking to contribute to Open Source Projects for the first time? Want to add a feature or fix a bug on the bots on this server? We have on-going projects that people can contribute to, even if you've never contributed to open source before!

Projects to Contribute to

• Sir Lancebot - our fun, beginner-friendly bot
• Python - our utility & moderation bot
• Site - resources, guides, and more

Where to start

1. Read our contribution guide
2. Chat with us in #dev-contrib if you're ready to jump in or have any questions
3. Open an issue or ask to be assigned to an issue to work on

[vital escarp]

thanks!

[lime stream]

What is python

[dim pelican]

A programming language. If you'd like to learn more you could check out https://www.python.org/about/

[brittle needle]

Hi guys, i am exploring preview environments for django/fastAPI during development, i have few question and would love insights from the folks who have done this in thier workflow

[rapid swallow]

Hey, maybe you should ask this in #web-development
. This channel is for discussion about this server's OSS projects (https://git.pydis.com/)

[brittle needle]

@rapid swallow thank you

[wise marten]

I have noticed, that the eval command gives a redo button when the non-code portions of the message are edited

often times I include an explanation around the code that should run (which is in a codeblock).
However, if I make a typo, then edit it, it prompts me to rerun

I wonder, what would be strategy to stop that from happening? I mean, how would you check programatically?

[fallen patrol]

Run prepare_input on both of the contents before comparing

Or no, after comparing

If before.content != after.content and prepare_input(before.content) != prepare_input(after.content): ...

Checking the first one because we don't need to run two regexes if it wasn't content that changed

[vital escarp]

Can someone help me set up docker in desktop cause ive done installed and set up wsl but for some reason it says its version its too old for docker. Dm me private please!

[fallen patrol]

#sir-lancebot-playground message
Yeah, because Lance isn't sending names for the day

[small jetty]

hi, i have some experience programming. Would anyone be interested if I try adding a !lint command to the bot which I guess would use something like ruff, run it on a code snippet and give the output of ruff?

[toxic wren]

I don't know if you will use this but, coderabbitai is free(The Pro tier is free) for OSS projects like the Python Bot and Sir Lancebot.

Background: Code rabbit is like GitHub Copilot and summarizes, and review PR's. This gives context for reviewers and contributors. It can also be configured for the repositories specific needs(ex: the exact format on how to write code). It also gives reviews in your IDE.

Btw I am not from code rabbit or associated with them, I just came across this and thought it would be cool for projects like this.

[patent pivot]

We did play with it on a few repos, whilst it was alright it didn't generate anything I'd call particularly useful so not sure it's worth it for PyDis projects

[toxic wren]

It might be useful, a temporary integration maybe?

[patent pivot]

this sounds cool -- I like both the idea of flagging lint issues and also running ruff format. in theory we have ruff in the container or can add it easily

[patent pivot]

we did try it, it's been used on our repos

[toxic wren]

ok then

[toxic wren]

so like in the snelkbox containers we also run ruff check?

[small jetty]

Ok, I can try adding a !lint function. If I understand correctly !e runs code, so I can look into that and adapt it to use ruff instead of eval or something like that?

[stiff path]

Hello guys i want to start python how i can be better fast ?

[toxic wren]

maybe look into the snekbox repo too

[toxic wren]

I have a question for the snekbox. I want to use snekbox for my project(running python code in a website(code will run on the server side, api maybe?) How do I do this and set it up?

[patent pivot]

yeah

we just need to bundle it as a dependency

in the same way we already have added numpy and similar

and then just need to check the entrypoints get mounted to the jail

[toxic wren]

Im building snekbox and it's been 10 minutes and its not done yet

probably my computer being slow

stuck on step 27 of build

@vital escarp !contib

!contrib

[stable mountain]

Contribute to Python Discord's open source projects

Looking to contribute to Open Source Projects for the first time? Want to add a feature or fix a bug on the bots on this server? We have on-going projects that people can contribute to, even if you've never contributed to open source before!

Projects to Contribute to

• Sir Lancebot - our fun, beginner-friendly bot
• Python - our utility & moderation bot
• Site - resources, guides, and more

Where to start

1. Read our contribution guide
2. Chat with us in #dev-contrib if you're ready to jump in or have any questions
3. Open an issue or ask to be assigned to an issue to work on

[modest ruin]

can someone rate my code?

i started python 5 minutes ago

[toxic wren]

this channel is for projects of python-discord ask in #python-discussion or open a #1035199133436354600 post

[timid sentinel]

There is an existing issue for that - https://github.com/python-discord/bot/issues/1880

Doesn't specifically mention ruff but only because it's from before ruff existed :)

A good start would be coming up with a specification for how the command would be implemented, and the user interface for it (e.g. how could it support passing options for the linter) and posting that on the issue

[small jetty]

Oh it seems neither ruff nor black have a Python API... That might make things tricky.

[small jetty]

I suppose a call to subprocess.run could be used to execute ruff. And it would follow a similar style to the eval command

[stable mountain]

bot/exts/utils/internal.py line 231

async def eval(self, ctx: Context, *, code: str) -> None:```

[patent pivot]

yeah it would be a matter of running the commands -- which is why i said we'd need to expose the entrypoints into the nsjail that actually runs the code

[small jetty]

ah i'm not that familliar with nsjail.

[toxic wren]

If I add snekbox to a website(for running code) would it still be a static site(Hosted with Cloudflare pages/github pages) or dynamic?

[fallen patrol]

This seems like a scope creep for snekbox

I've had a black format command for a while on Monty

[timid sentinel]

iirc the snekbox endpoint lets you specify the path to the executable you want to use, could this be as simple as adding the package you want to the requirements.txt and then just working out the path to pass for the executable?

[patent pivot]

what wookie said, it doesn't require any changes to snekbox

[vale ibex]

If ruff provides a pre-built executable it would be even more trivial

[patent pivot]

it just requires us to expose entrypoints (or check they're exposed), you can pass the path to the executable in to snekbox alongside files that snekbox will put in an execution dir

[fallen patrol]

It does

[fallen patrol]

Ah gotcha. In that case I think it's cool

[vale ibex]

iirc it should be a case of putting it in /snekbin somewhere and then specifying that path when calling the api

[patent pivot]

yea

[fallen patrol]

Remember to set the env var to have ruff not write a cache

[vale ibex]

it'll be a tmpfs inside the nsjail container anyway so shouldn't matter

[harsh hill]

looks like there hasnt been much contribution to sir lancelot this year

[fallen patrol]

Yeah but why have the performance penalty of writing a useless cache

[outer oasis]

Then get started on it

[fallen patrol]

Remember WPC: Write Performant Code

[harsh hill]

say less twin come with me

[fallen patrol]

So when are we rewriting snekbox to use rust? /s

[toxic wren]

Soon..........

[patent pivot]

we're a programming community

[toxic wren]

Guys, How would a !stats command be for showing contribution stats of the bot, lancebot, site, snekbox, etc in a certain timeframe.
Something in this format:
Showing stats for the Last 10 months:
Issues Opened: 10
PR's Opened: 15
Issues Closed:5
PR's closed: 2
Total Commits:100

Something like that?

[patent pivot]

like the details on https://www.pythondiscord.com/ ?

[toxic wren]

The stats there are total stars, and PR's I think

though we could add those too

Ill check the Github API docs

if it's possible

[patent pivot]

we already get that from the GitHub API

those stats on the homepage are pulled from the GitHub api

[toxic wren]

I don't know if you are talking about what i am talking are you talking about this:

This shows:
Lang, Stars, and forks

I want to show:
PR's opened in a timeframe, Issues opened in a timeframe, PR's closed in a timeframe, Issues closed in a timeframe and total commits in a timeframe

so the command is like this !stats <timestamp_start> <timestamp_end>

[fallen patrol]

Yeah seems cool

Hmmmm

[patent pivot]

uhhh

can you open an issue for this

[toxic wren]

Sure gimme a few minutes

[patent pivot]

i'm again not sure if implementing this in discord is very useful given you can get more useful statistics straight out of GitHub with your own queries and have them be presented in a much better way

[fallen patrol]

Consider: recognition within discord

[patent pivot]

not sure what you mean

[toxic wren]

I mean like if you were to show someone stats for bot in Discord it would be cool

Like all the contribution efforts

from the community

[patent pivot]

I think doing that via numbers doesn't really do it justice, highlighting individual features is what does that justice and we have the auto-embedding of issues and PRs

[toxic wren]

wdym

[patent pivot]

and also, being completely transparent, our repos are a lot less active than they used to be and I'm not sure it very well highlights the development efforts of our community projects if you focus solely on the most recent issues and PRs

[toxic wren]

its just a number statisitc

[toxic wren]

we can probably add on to it in the future

[harsh hill]

wait a sec... am i only able to contribute to sir lance if i purchase Ona (formerly Gitpod)?

i forked it and cloned it to pycharm but then i actually read the contribution page on pythondiscord.com

[toxic wren]

Issue opened: bot#3394

[dusky shore]

GitHub

[bot] #3394 Add a !stats command for stats on Python discord repos

[harsh hill]

ope

they have a free plan. maaaaaan this is why my ADHD ass needs to read the instructions before grabbing the hammer

[royal prawn]

can you share the exact page with the exact instructions

been a while since i've done anything with lancebot, but i did not interact with gitpod

[harsh hill]

sure 🙂 https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/sir-lancebot/

[cold island]

git pod is one method to work with sir lancebot, you don't have to use it. Can run everything locally

What you need is under the requirements section

[royal prawn]

yeah i used the docker path iirc, back in the poetry days

[harsh hill]

poetry?

[fallen patrol]

!pypi poetry

[stable mountain]

poetry v2.2.1

Python dependency management and packaging made easy.

Released on <t:1758466309:D>.

[royal prawn]

another package-etc manger, like uv

[toxic wren]

yea pydis used to use poetry until they chaged into uv

[fallen patrol]

Ahem, "pep517 frontend, environment management, and package manager." A project manager if you will.

Package manager slightly misnames it due to all of the other tools that exist

[toxic wren]

!pypi uv

[stable mountain]

uv v0.8.23

An extremely fast Python package and project manager, written in Rust.

Released on <t:1759602233:D>.

[toxic wren]

in Rust

[fallen patrol]

@harsh hill you don't need to use gitpod

[toxic wren]

why does everything need to be in rust

[fallen patrol]

You can literally clone the repo, run docker compose up, and boom running bot
-# after adding bot token

[outer oasis]

I said so

[outer oasis]

Always with the fine print

[toxic wren]

huh?

[patent pivot]

[fallen patrol]

Go on

[patent pivot]

i have nothing else to add

I think you could use slightly more polite and less pedantic ways

[toxic wren]

very nice:https://git.pydis.wtf

[fallen patrol]

True.

[fallen patrol]

Sorry for being pedantic about the description of uv

[royal prawn]

meh
did not perceive as even slightly hostile

[harsh hill]

okay i got the venv installed and ill just say thats as far as im going to go rn. thats wayyyy out of my league haha

ill be back for you sir lanceBOT. wait for me senpai

[fallen patrol]

fair

and that's totally fine

[harsh hill]

now im just blasting baroque music in my ear while going thru tutorial hell. lets get this bread and grind

[tame pebble]

why would you say that's out of your league ?

The reason why I'm asking is that we'd also like to simplify this as much as possible

and make it even more accessible

[harsh hill]

for sure! vocab (knowing what venv and precommit and push, etc.,), how to navigate pycharm(the difference between the terminal and the text editor. do they work together or are they separate?) and simply just like... how do you run code, read code, how is it divided into different files in the project...

all sorts of stuff

its not you, its me

[fallen patrol]

would this be your first ever contribution?

[harsh hill]

yee

[fallen patrol]

Fair and I did not mean to not reply

Before i forgot but @celest charm got me interested in seeing about adding basedpyright to @stable mountain

And I'm down to submit the pull

[fallen patrol]

I don't have the contributing docs in front of me atm but which part of getting a local copy of sir lancebot are you having trouble with? Maybe I can help walk you through it.

The tldr is

• clone the repo
• open in an IDE/editor
• install dependencies
• install git hook
• create discord application and discord server
• add bot to server
• add token to new .env file in editor
• add other configuration variables

[patent pivot]

I can't remember if the last time we discussed this was in a public channel, but the way we'd like to do it is as a completely separate non-required check using the baseline

so it's much more an advisory thing

there are things we'll need to undoubtedly silence/ignore because there are things we can contextually assess to be impossible that the typing system cannot, and at this point we don't wish to constrain contributors to having to add a pile of typing stuff

so importantly at this stage not part of the main lint pipeline and not part of pre-commit/commit hooks

[tough berry]

Hey everyone! I just launched Digamma Prime — a symbolic framework for auditing ML models.

It uses metrics like φ and Δφ to compare models by behavior, not just accuracy.

Docs, CLI, and tutorials are live: https://github.com/Cerene-Salt/Digamma-Prime-Framework

Would love feedback or collaborators — especially around visual modules and dashboards!

I did it with the help of Copilot.

[toxic wren]

This channel is for projects related to python discord not personal projects

Read the description for more info

[fallen patrol]

!d disnake.Role.colour is this on my end?

[stable mountain]

disnake.Role.colour

Unable to parse the requested symbol due to a network error.

[fallen patrol]

Seems its the inventory configuration here

base url should be https://docs.disnake.dev/en/latest/

there's a second API in there for some reason

these two commands !d removedoc disnake
!d setdoc disnake https://docs.disnake.dev/en/latest/objects.inv

should fix it

[oblique cairn]

!d removedoc disnake

[toxic wren]

Inf typing loop

[stable mountain]

I'm just typing a very long message..

[toxic wren]

wait what

[oblique cairn]

Yeah, it's trying to look it up in the removedoc command ._.

[toxic wren]

it does not exist?

[oblique cairn]

The current disnake docs endpoint is wrong, apparently.

(see above)

[toxic wren]

ah

makes sense

[toxic wren]

still typing......

[outer oasis]

Poor @stable mountain

[toxic wren]

its still going

its gonna be permanent

[royal prawn]

shen what did you do

[outer oasis]

Lol, it just needs to be be restarted

[toxic wren]

shen?

[outer oasis]

M-x rollout restart bots/bot or whatever it is

[outer oasis]

hi

me here

[toxic wren]

oh i just saw your username

[outer oasis]

[wise marten]

Python has been stuck typing for a while in multiple channels... I sent this message 14 minutes ago

[toxic wren]

It is becuase of this command

[toxic wren]

its calling the wrong API

and its stuck in a loop

allegedly

[wise marten]

i c

[fallen patrol]

Lmfao

@late hedge we did it again we broke @stable mountain

First ruff and now pydis

[outer oasis]

this is no laughing matter
LOOK WHAT YOU'VE DONE

[fallen patrol]

(and yes, disnake once broke ruff CI)

[outer oasis]

y

[fallen patrol]

Off topic

[royal prawn]

why has no one fixed this yet
do i have to ping devs oops in the dev oops channel

[outer oasis]

yes

They're the ones with permission to restart the bot

[royal prawn]

heaven forbid the former oopser do it

[outer oasis]

@fallen patrol a restart is all that's needed to get it to stop, right?

[fallen patrol]

It's also typing in #bot-commands

[fallen patrol]

The doc cache also probably needs to be reset

[outer oasis]

M-x rollout restart bot/bots

[royal prawn]

<@&409416496733880320> pybot has been typing in multiple channels for a while, may need restarted.
some commands, like !docs, also appear to be failing

[outer oasis]

nuthin

[fallen patrol]

Oh wait this probably why this command didn't finish

[outer oasis]

this is not that channel

[royal prawn]

i noticed. this does demo the problem, though

[outer oasis]

directions, thuri, directions

[cold island]

If the only problem is that it's typing then it's not really urgent, someone will get to it

[outer oasis]

!docs is dead

[fallen patrol]

!d

[stable mountain]

All inventories (`0` total)

Hmmm, seems like there's nothing here yet.

[fallen patrol]

Yeah it's a little more urgent I think

[royal prawn]

apparently there may be other problems
i no have sentry access so cannot give full details

[fallen patrol]

Lol

This is why I need to rewrite the docs command on monty

It works but it doesn't sometimes

[outer oasis]

What makes you think that?

Last Sentry event was 6h ago

you don't say

[rustic venture]

doesn't the Python bot have any kind of sane timeout, maximum retries and maximum redirects that it follows when doing lookups and retrieval for the !d commands?

[cold island]

doesn't seem so

[fallen patrol]

How do you view these?

I didn't think you had access

[outer oasis]

I login to Sentry

[outer oasis]

They never removed it
They probably will now that they see it though

[royal prawn]

i technically have access to pydis's sentry, but i'm not on any of the teams, so i can't see anything

[fallen patrol]

Lmao

Wait you used to be devops?

[outer oasis]

yea

[outer oasis]

Yeah I think anyone in the org can just login

[fallen patrol]

O.o

Why did you leave?

[cold island]

probably lol

So this is the get request https://github.com/python-discord/bot/blob/main/bot/exts/info/doc/_batch_parser.py#L111

But looking at the aiohttp documentation it seems to set 30s timeout and 10 max redirects by default

[stable mountain]

bot/exts/info/doc/_batch_parser.py line 111

async with bot.instance.http_session.get(doc_item.url, raise_for_status=True) as response:```

[fallen patrol]

Its this. https://github.com/python-discord/bot/blob/f0e4d45cb30208c35fa55372e15ea524ea573123/bot/exts/info/doc/_cog.py#L410

[stable mountain]

bot/exts/info/doc/_cog.py line 410

@lock(NAMESPACE, COMMAND_LOCK_SINGLETON, raise_error=True)```

[fallen patrol]

it got stuck either in redis or in refreshing the inventories

and based on the state of !d

it was either here https://github.com/python-discord/bot/blob/f0e4d45cb30208c35fa55372e15ea524ea573123/bot/exts/info/doc/_cog.py#L216 or https://github.com/python-discord/bot/blob/f0e4d45cb30208c35fa55372e15ea524ea573123/bot/exts/info/doc/_inventory_parser.py#L89-L90 (probably)

[stable mountain]

bot/exts/info/doc/_cog.py line 216

await self.item_fetcher.clear()```
`bot/exts/info/doc/_inventory_parser.py` lines 89 to 90
```py
timeout = aiohttp.ClientTimeout(sock_connect=5, sock_read=5)
async with bot.instance.http_session.get(url, timeout=timeout, raise_for_status=True) as response:```

[fallen patrol]

wow

its still going

[wise marten]

Wao

[celest charm]

Yikes, this means that there's no total timeout

there's a timeout for each individual socket.read() and for connecting, but not for the total request time

[cold island]

be the change etc.

[timid sentinel]

btw there is an issue for this docs problem https://github.com/python-discord/bot/issues/3326

[fallen patrol]

Wait a moment

@timid sentinel are you able to dump some logs from yesterday, and are you able to check if python broke the urls on its own or if someone modified the values?

It looks fixed now, but there was another broken package as well

[timid sentinel]

There is nothing interesting in the logs, I don't think anything changed recently.

It seems disnake had the wrong base url in january #bot-commands message

But not in November 2024 #bot-commands message

So I'm guessing something happened between then. logs don't go that far back and I probably don't have access to whatever channel the command was run in if it was manual

[toxic wren]

Umm.. guys Python 3.14 dropped a stable release. should we make the bot use 3.14 instead of 3.13?

nvm Ill just open an issue

[wind ruin]

for !eval or for the bot itself?
I don't think discord.py supports it nor will soon

I'd also wait until most whitelisted libraries support 3.14 before updating eval tbh

[timid sentinel]

Yeah we'll need to wait for dependencies, and we'll also probably want to wait for 3.14.1 to help ensure any major bugs are ironed out.

[toxic wren]

bot#3397

[dusky shore]

GitHub

[bot] #3397 Update the bot to use Python 3.14.

[toxic wren]

for !eval

[toxic wren]

ok lemme edit the issue

[timid sentinel]

!e 3.14 import sys;print(sys.version_info)

[stable mountain]

:white_check_mark: Your 3.14 pre-release eval job has completed with return code 0.

sys.version_info(major=3, minor=14, micro=0, releaselevel='candidate', serial=1)

[timid sentinel]

Already there it seems

though needs updating

[toxic wren]

wdym

[toxic wren]

Isn't the point of a stable release that major bugs are ironed out?

or are we waiting for general publics issues?

[fallen patrol]

@timid sentinel do you have logs around 1424854698460184700 about whether or not there was a concurrent documentation invoke?

[patent pivot]

this

also I think it's probably a better idea to look at it at a higher organisation level and push multiple of our projects to newer python versions where we can

what's the situation with the docs command? is this just because the dissnake endpoint was configured wrong or something

[timid sentinel]

https://github.com/python-discord/bot/issues/3326#issuecomment-3378793022

[patent pivot]

ahh gotcha thanks

so fixing the URL has actually fixed the immediate problem?

[timid sentinel]

yeah I think it should have

[patent pivot]

why was the original problem the !d discord though?

oh was this whilst dissnake was hijacking the discord namespace

[timid sentinel]

hmmm

It may have been a temporary outage of the discord.py docs?

which I think could trigger the same issue

[patent pivot]

hmmm yeah

[fallen patrol]

Our docs didn't

Not in the past 3 years.

[fallen patrol]

yesterday discord also had the same bug.

huh. No it didn't #bot-commands message maybe

odd

[timid sentinel]

!d discord

[stable mountain]

discord

In order to work with the library and the Discord API in general, we must first create a Discord Bot account.

Creating a Bot account is a pretty straightforward process.

Similar names: disnake.discord

[timid sentinel]

!d disnake.discord

[stable mountain]

disnake.discord

[timid sentinel]

disnake's "discord" is just a page in the docs, which is valid, it seems like the docs command just handles it weirdly sometimes

[patent pivot]

yeah

[toxic wren]

snekbox#244

[dusky shore]

GitHub

[snekbox] #244 Update the bot to use Python 3.14

[royal prawn]

it would not surprise me if that gets blocked by third-parties until 2026

[toxic wren]

what gets blocked?

[royal prawn]

migrating to 3.14

[toxic wren]

they prefer high stability over features.

thats why they wait

[royal prawn]

alright i stand corrected. my own major environment members (coverage, hypothesis, jinja2, matplotlib, numpy, pandas, pillow, pytest, ruff, scipy and sympy) already got successfully resolved for 3.14

[fallen patrol]

wait did i comment on the wrong issue

[patent pivot]

okay I think we can maybe calm it with these issues, this doesn't even reference snekbox

@fallen patrol @toxic wren we already have 3.14 in snekbox, this issue just needs to be about moving from the rc to the new release (which is a one line change).

https://github.com/python-discord/snekbox/blob/main/Dockerfile#L45

the bot still needs to be updated to default to 3.14, that is not controlled snekbox side, but again it's literally updating the order of the literal that stores version info

https://github.com/python-discord/bot/blob/main/bot%2Fexts%2Futils%2Fsnekbox%2F_cog.py#L90

[stable mountain]

Dockerfile line 45

RUN /build_python.sh 3.14.0rc1```
`bot%2Fexts%2Futils%2Fsnekbox%2F_cog.py` line 90
```py
SupportedPythonVersions = Literal["3.13", "3.13t", "3.14"]```

[patent pivot]

I appreciate the enthusiasm from both of you generally with contributions here but this caused a bunch of issue noise for 2 lines of changes and I think maybe just stepping off the gas a touch and thinking a bit more slowly / asking Core Devs might be better

for example we've now closed that bot issue when there are actually bot changes necessary, and the snekbox issue mentions bot when snekbox has nothing to do with bot and is a completely separate and independent project

[fallen patrol]

I mean fwiw at the time of that issue that change wasn't in snekbox, which to me is a different project from the bot, I also don't see the issue with well, making issues: its a pretty good way of tracking things that need to be done!

Enough discussion happened here that it seems to have needed an issue anyhow. eg these #dev-contrib message #dev-contrib message definitely make it sound like it needs time before full support

[patent pivot]

Sure, maybe it needed issues, but either way it at least needed an issue in both places instead of one issue that was then edited to fundamentally change scope and then eventually closed.

I've made the snekbox changes now anyway. Anyone else is free to pick up the reordering of the literal and remove the extra bit of copy about it being pre-release.

[fallen patrol]

yeah didn't mean the issue in bot needed to be closed

just that it also needed one in snekbox

[patent pivot]

one that actually describes the snekbox changes instead of a copy paste from bot would've been better for that

[fallen patrol]

yeah. I didn't open that.

[patent pivot]

I know, just replying to your comment, I agree issues are great for tracking things that need to be done but if they're fundamentally misspecified and talk about the wrong things it's about as useful as having no issues

[fallen patrol]

fair

[patent pivot]

Any issue that proposes refactoring/changing something should include some sort of guidance to where those changes would be, like the GitHub links I posted above.

[fallen patrol]

ye

in some cases it may just be a feature request though

[patent pivot]

yea but that's not refactoring or a change

[fallen patrol]

im arguing an apples to oranges please ignore me

[patent pivot]

if something's brand new sure, else you link where the thing needs adding or changing

feature requests are either enhancements or brand new things, the former still has the same rules

snekbox CI should be done in 20 mins or so, so bot will start using 3.14.0 when !eval 3.14 ... is used, the copy just needs removing and the default version changing

depending on what's cached and what needs rebuilding because the python versions changed it might be 50 minutes left in snekbox we'll see

[fallen patrol]

why do we build the python versions for snekbox?

rather than using prebuilts?

[patent pivot]

was mostly so we can support pre-releases I think, the way we have python versions in snekbox isn't permanent

[patent pivot]

hahaha 3.14.0 test suite fails in CI

[patent pivot]

now we see why we wait for .1

[subtle remnant]

hello! can someone here suggest me beginner repo that i can contribute to

i have knowledge on ml,fastapi,mongodb,firebase,nlp stuff

[tame pebble]

You can have a look at our repos

We usually tag issues for people who don't know the repos with ' good first issue'

!contribute

[stable mountain]

Contribute to Python Discord's open source projects

Looking to contribute to Open Source Projects for the first time? Want to add a feature or fix a bug on the bots on this server? We have on-going projects that people can contribute to, even if you've never contributed to open source before!

Projects to Contribute to

• Sir Lancebot - our fun, beginner-friendly bot
• Python - our utility & moderation bot
• Site - resources, guides, and more

Where to start

1. Read our contribution guide
2. Chat with us in #dev-contrib if you're ready to jump in or have any questions
3. Open an issue or ask to be assigned to an issue to work on

[subtle remnant]

i checked the repo most of the good first issue have been taken or done with a pr

should i create a new issue by myself

how do i start

[tame pebble]

Yeah good first issues can be taken, but if you feel they have just been picked up and ignored, you can try posting a message here and we will look into and and maybe reassign, because it does happen.

You don't really need to work on good first issues per se, if something picks up your interest and do some research to understand how things currently work and propose something, you can of course request to be assigned, but that can be too much to dive into right from the start.

You can always create issues if you notice something is wrong/could use improvement/have ideas of course.

Is this you ?

[subtle remnant]

uhhh

i would like to be anonymous

[tame pebble]

it was just to know about the assignment, didn't mean it to sound like exposing you, sorry

[patent pivot]

@toxic wren they're just spam issues, generally from people who are new to GitHub and don't entirely understand things. core devs see and delete them when they're opened, if the user closes them there's no need to really try interrogate why they were opened, there generally is no reason other than spam

[oblique cairn]

Hey @lilac merlin, this channel is for contributions to repositories of the Python Discord organizations, not any random projects.

[patent pivot]

!eval 3.14 ```py
import sys
print(sys.version_info)

[stable mountain]

:white_check_mark: Your 3.14 pre-release eval job has completed with return code 0.

sys.version_info(major=3, minor=14, micro=0, releaselevel='final', serial=0)

[patent pivot]

https://github.com/python-discord/bot/pull/3406

[toxic wren]

#dev-log message why?

[patent pivot]

you can see from the PR body

[toxic wren]

#dev-log message ?

[patent pivot]

[toxic wren]

so... you use an emac? I don't get the point?

[patent pivot]

do you know what emacs is

https://www.gnu.org/software/emacs/

[toxic wren]

yes

[patent pivot]

the PR was opened from emacs and had a title of --- because i configured something wrong

i did not mean to title the PR just ---

not a very descriptive title

[toxic wren]

oh ok

[toxic wren]

so you uh use an emac?

[patent pivot]

i don't know what you mean by "an emac"

the software is called emacs

it is a text editor, like vscode

[toxic wren]

I mean you use the software

[toxic wren]

my mind keeps thinking about Macbooks when typing that.

so I used "an emac"

instead of emac

[patent pivot]

yea i use the software

[toxic wren]

um... do you like it? It looks old.

[patent pivot]

doesn't look too old

[toxic wren]

that and this look very different:

[patent pivot]

"Emacs is the extensible, customizable, self-documenting real-time display editor" 🙂

[toxic wren]

that looks amazing🤩

[patent pivot]

similar to vim you can have it look however you want

[toxic wren]

I did not think it was that customizable, I just use a vanilla Zed install.

[fallen patrol]

Yay snekbox supports pi-thon now

[patent pivot]

!e ```py
import sys
print(sys.version_info)

[stable mountain]

:white_check_mark: Your 3.14 eval job has completed with return code 0.

sys.version_info(major=3, minor=14, micro=0, releaselevel='final', serial=0)

[patent pivot]

!e 3.14t ```py
import sysconfig
print(sysconfig.get_config_var("Py_GIL_DISABLED"))

[stable mountain]

:white_check_mark: Your 3.14 free threaded eval job has completed with return code 0.

1

[toxic wren]

did you forget about the !exec alias? #changelog message

[toxic wren]

bot#3403 attachment uploading?

[dusky shore]

GitHub

[bot] #3403 Disable automatic attachment uploading to pastebin in DMs

[toxic wren]

bot#3196 shouldn't this be a snekbox issue?

[dusky shore]

GitHub

[bot] #3196 Bot runs !eval when re-running a !timeit command

[patent pivot]

no

snekbox doesn't know anything about !timeit, !timeit is literally just a template that the bot applies before sending it off to snekbox for eval

anything that is actually about how code runs, available versions, the execution environment, etc. goes on snekbox

anything about the frontend to that (the !eval command, features like !timeit) need to go on bot because that's actually where the commands are

the logic to rerun commands (and the current problem where !timeit reruns use !eval) is something that is entirely on bot

snekbox has no idea

[toxic wren]

oh ok

[dusky shore]

GitHub

[bot] #3350 Automated temporary voice access for new users

[fallen patrol]

what needs to be done to have support to allow internet access in snekbox if allowed with an evaluation request? eg providing "permissions": {"network": true} in an API request and having the environment allow that

because I have a usecase for allowing network access in a specific snekbox container as a still otherwise controlled and restrained environment

[patent pivot]

fairly trivial

I'd imagine the macvlan stuff would just need setting

there might be some filtering necessary to make sure your jailed procs can't then access snekbox API themselves or other services running locally or in a private network

https://serverfault.com/a/1114563

[fallen patrol]

neat, is this something that could be considered for upstreaming? (as in, I plan to implement this either way for my own use)

[patent pivot]

seems fine to have upstream, but obviously documented with the caveat that snekbox itself cannot handle that network filtering

[fallen patrol]

ye

[patent pivot]

if an eval process can access snekbox API then it can cause a denial of service quite easily

[fallen patrol]

in my case I don't really care about that, I just need to download a few files from the internet every so often and parse them safely in an environment
code is controlled anyhow

[patent pivot]

if going upstream it probably needs an issue to specify what the interface looks like for users

[fallen patrol]

.repo python-discord/snekbox

[patent pivot]

I think there's a chance that because we can't necessarily protect it well then we shouldn't add it upstream so we need to discuss that

[fallen patrol]

.repository python-discord/snekbox

ok.

[patent pivot]

snekbox right now is very hard to make dangerous

[fallen patrol]

https://github.com/python-discord/snekbox/issues/new/choose

[patent pivot]

none of the config options we expose would allow for any sort of compromise or resource exhaustion

[fallen patrol]

Fair

[patent pivot]

allowing network immediately adds a lot of risk of internal network pivoting and snekbox API access means people can launch infinite denial of service problems against snekbox, and we can't really add protection against those

[fallen patrol]

wonder if we can add RFC1918 IP filtering somehow to nsjail

[patent pivot]

nsjail outsources most of that to macvlan and the interface there is fairly simple and low level

[fallen patrol]

possibly https://github.com/google/nsjail#isolation-of-network-services-inetd-style

[patent pivot]

in my mind it'd be easy with netns

like ip netns exec jailns iptables -A OUTPUT -d 10.0.0.0/8 -j REJECT

or something

but I don't think it'd be easy to achieve from solely nsjail

[fallen patrol]

fair

if it were to be added it could probably require an environment variable in configuration to ensure the environment is secured

[fallen patrol]

bot#3408

[dusky shore]

GitHub

[bot] #3408 Fix unbound error for code snippet fetch errors

[fallen patrol]

oh so close

@patent pivot I don't see a reason to surface the issue to the user because this could possibly be a ratelimit has been reached vs anything else

we don't want to surface 404s, 401s, 403s, but we could surface 429 as a marker to the devs

granted, it logs as an error in that case. I don't see a log of when this happened but I'm sort of interested about it

[patent pivot]

yes but something happened that was an error and then we blindly return other results, that's bad UX

we don't need to surface error specifics but we should say the results are incomplete

anytime we present incomplete information we should tell the user or retry

[fallen patrol]

i mean, not necessarily

there's a lot of detection that would need to go on then

[patent pivot]

what do you mean

[fallen patrol]

https://github.com/python-discord/bot/blob/8e8d5b0c4023e957324a4e69f9704393e0e29fe2/bot/exts/info/code_snippets.py#L284
https://github.com/onerandomusername/private-repo/blob/main/README.md#L2-4
what should behavior be on this message? it shouldn't tell me that its half of the messages, I know its half of the messages

[stable mountain]

bot/exts/info/code_snippets.py line 284

continue```

[fallen patrol]

at the very least it shouldn't stop the one half because of that private repo link

[patent pivot]

I feel if it's a transient error code we should note and tell the user that

permanent error codes obviously less important

obviously the ideal would be to retry but that's probably overcomplicating it

[fallen patrol]

Could I take a look at the source logs/what caused this specific issue? bot#3404

[dusky shore]

GitHub

[bot] #3404 Fix UnboundLocalError in code_snippets.py

[patent pivot]

there's actually not a lot useful

that's all sentry is surfacing

& in the actual logs

[fallen patrol]

huh

[fallen patrol]

what message was it?

or was that not even logged?

even just the date those timestamps were from means I could figure out the message through snowflake searching

[patent pivot]

it wasn't logged because it was a 404 but it was #ot0-psvm’s-eternal-disapproval message

[fallen patrol]

ah yeah

that is an intentional break the bot string

<https://github.com/onerandomusername/monty-python/blob/main/../../../../wntrblm/nox/blob/main/README.md#L1>

it normalises to a completely different path and 404s because of that

bot#3315

[dusky shore]

GitHub

[bot] #3315 Unsafe URL construction in the CodeSnippets cog

[fallen patrol]

I can submit a fix if desired

right now it has this sort of bug

https://github.com/onerandomusername/monty-python/blob/main/../../../../rate_limit#L1

[stable mountain]

../../../../rate_limit line 1

{"resources":{"code_search":{"limit":60,"remaining":45,"reset":1760365703,"used":15,"resource":"code_search"},"core":{"limit":60,"remaining":45,"reset":1760365703,"used":15,"resource":"core"},"graphql":{"limit":0,"remaining":0,"reset":1760368260,"used":0,"resource":"graphql"},"integration_manifest":{"limit":5000,"remaining":5000,"reset":1760368260,"used":0,"resource":"integration_manifest"},"search":{"limit":10,"remaining":10,"reset":1760364720,"used":0,"resource":"search"}},"rate":{"limit":60,"remaining":45,"reset":1760365703,"used":15,"resource":"core"}}```

[patent pivot]

do we authenticaate those requests

[fallen patrol]

you don't

which is why a 403/404 is more possible for python codesnippets: you only get 60 requests in an hour

[patent pivot]

well that's good at least lol

[fallen patrol]

i auth mine but it doesn't have any permissions other than upping the ratelimit

[patent pivot]

yea i'd just be worried that it'd allow for some nasty permission escalation into private repos

like <https://github.com/onerandomusername/monty-python/blob/main/../../../../repos/python-discord/admin-tasks#L1> or something

[fallen patrol]

it doesn't

[patent pivot]

i think only arthur has real permissions

M-x github add jb3

[radiant merlin]

:white_check_mark: Successfully invited jb3 to the staff team.

[fallen patrol]

this is my scope

literally just: exists

oh wait

lemme check

[fallen patrol]

either way I can patch 3315 if you want

I fixed this on my bot last night

[fallen patrol]

ok yeah just tested, without that there's nothing that isn't public aside from it showing public details of the logged in github user which i wouldn't consider a security hole since that is public knowledge anyhow

you can't get anything private

[patent pivot]

yea i don't mind reviewing that

[fallen patrol]

bot#3409

[dusky shore]

GitHub

[bot] #3409 Implement URL normalization in code snippet handler

[fallen patrol]

I haven't tested this exact fix on @stable mountain but I tested it within Monty and ported the fix over

so if it doesn't work that's on my port

[patent pivot]

okay fixed the stochastic snekbox tests a little bit I thnk

there is still this that pops up every now and then but I don't think this is related to any recent changes https://github.com/python-discord/snekbox/actions/runs/18422697909/job/52500287007

[patent pivot]

!e 3.14j ```py
import sys

data = []

for number in range(0, 100_000):
data.append(number * 2)
if sys._jit.is_active():
print(f"JIT activated on round {number}")
break

[stable mountain]

:white_check_mark: Your 3.14 JIT-compilation enabled eval job has completed with return code 0.

JIT activated on round 4096

[fallen patrol]

Huh

[toxic wren]

Hello,
I want to make most of the general use commands into slash(/)( !commands are also there) commands. Does anyone want to work with me on this. I can work alone if needed.

[royal prawn]

to clarify, you mean adding slash functionality, right? not switching to slash-only?

[toxic wren]

yes

[patent pivot]

if we're doing that can we open an issue to discuss priorities and implementation

it's going to be an absolute pain to review if that's all done at once

I suggest breaking it down into commands that are best suited to become hybrid commands (particularly ones where autocomplete is possible) and then open a PR for each individual cog prioritising the commands that make most sense

at least for this iteration no commands that are helper+ or moderator+ should be moved

[patent pivot]

huh to which bit

[tame pebble]

!help

[stable mountain]

Command Help

HelpForum
!help-forum close
Make the help post this command was called in dormant.
!help-forum
A group of commands that help manage our help forum system.
!help-forum title <title>
Rename the help post to the provided title.

Page 2/5

[tame pebble]

docs could be one maybe

@patent pivot i suggest an ai command that sends a troll email to volcy, signed with "Love, claude"

[patent pivot]

Hi Bella, Absolutely 😊. Love from all my family to yours

[patent pivot]

this is true, docs would be a very good one to move

[patent pivot]

@toxic wren I'm suggesting each cog rather than each command

I might reword this issue a bit -- but I can assign you to it if you want to take it

[toxic wren]

Sure.

[toxic wren]

ah ok

[toxic wren]

can you reword?

[patent pivot]

issue is updated

[patent pivot]

cogs are the full modules of commands

like !eval and !timeit are part of the Eval/Snekbox cog (can't remember exactly what it's called)

we should use that as a unit of migration

(though we won't migrate those two commands)

[toxic wren]

ok, is't the codebase for the commands made pre-slash. Will it still be possible to migrate?

[patent pivot]

yes, hybrid commands

it's a discord.py feature

also we won't need to migrate the help command I wouldn't have thought -- the command works as is for text commands and the idea of slash commands is that typing a / becomes your new help command (as all commands are displayed there to the user)

Bella is right that migrating docs is something that instantly improves the user experience and works better as a slash command

I'll have a look through for others when I'm at a laptop

[patent pivot]

https://discordpy.readthedocs.io/en/stable/ext/commands/commands.html#hybrid-commands

[toxic wren]

Ok, Ill start working on !docs later today or tomorrow.

[fallen patrol]

jit is cool

[fallen patrol]

yeah. after looking through discord.py as well I do think that hybrid is cool.

[toxic wren]

!src docs

[stable mountain]

Command: docs

Look up documentation for Python symbols.

Source Code

Go to GitHub

bot/exts/info/doc/_cog.py:304

[toxic wren]

!contrib

[stable mountain]

Contribute to Python Discord's open source projects

Looking to contribute to Open Source Projects for the first time? Want to add a feature or fix a bug on the bots on this server? We have on-going projects that people can contribute to, even if you've never contributed to open source before!

Projects to Contribute to

• Sir Lancebot - our fun, beginner-friendly bot
• Python - our utility & moderation bot
• Site - resources, guides, and more

Where to start

1. Read our contribution guide
2. Chat with us in #dev-contrib if you're ready to jump in or have any questions
3. Open an issue or ask to be assigned to an issue to work on

[toxic wren]

!docs aiohttp

[stable mountain]

aiohttp

Common data structures used by aiohttp internally...

[toxic wren]

!src tag

!src tags

probably should do this in #bot-commands

[toxic wren]

are all the docs not in the bot repo?

!docs aiohttp does not work on my local version

[stable mountain]

aiohttp

Common data structures used by aiohttp internally...

[fallen patrol]

#bot-commands message

use that command

what the heck

[cold island]

What do you mean by docs being on the bot repo?

[outer oasis]

I am curious.
Is the list of available inventories stored somewhere? Is it in the DB?

[cold island]

It's in redis iirc

[vale ibex]

Yeah, it's cached in redis

[fallen patrol]

its in redis????

I swear that used to be in the database

[patent pivot]

the symbols cache is yeah

[fallen patrol]

!src docs

[stable mountain]

Command: docs

Look up documentation for Python symbols.

Source Code

Go to GitHub

bot/exts/info/doc/_cog.py:304

[patent pivot]

the inventory links are stored in the database

[fallen patrol]

oh yeah the symbol cache which is redisable

but the inventories need persistence

[outer oasis]

No
You have to go to each website and find the URL for the Sphinx “inventory” file, and then run !docs setdoc with the URL, and that will download all the symbols and put the URL in the database for future reference.

For example:
!docs setdoc disnake https://docs.disnake.dev/en/latest/objects.inv

Meta: a list of a the ones we have could be added to the bot docs to give contributors a starting point

[fallen patrol]

hmmm

sorry for the sentry issues

[stable mountain]

Click here to see this code in our pastebin.

[patent pivot]

that's a dump of what we currently have in the bot documentation storage

[outer oasis]

Oh hey

Tanks

[toxic wren]

wait why can't .src be used in #1035199133436354600 channels

would be useful here: https://discord.com/channels/267624335836053506/1428518628055187497

[fallen patrol]

.src src

[dusky shore]

Command: source

Display information and a GitHub link to the source code of a command, tag, or cog.

Source Code

Go to GitHub

bot/exts/core/source.py:19

[fallen patrol]

https://github.com/python-discord/sir-lancebot/blob/85771ca1f6b5ef01016ba3661553fad5216ebf70/bot/exts/core/source.py#L20

[stable mountain]

bot/exts/core/source.py line 20

@whitelist_override(channels=WHITELISTED_CHANNELS+(Channels.community_meta, Channels.dev_contrib))```

[outer oasis]

And that’s

https://github.com/python-discord/sir-lancebot/blob/85771ca1f6b5ef01016ba3661553fad5216ebf70/bot/constants.py#L313-L322

[stable mountain]

bot/constants.py lines 313 to 322

# Whitelisted channels
WHITELISTED_CHANNELS = (
    Channels.bot_commands,
    Channels.sir_lancebot_playground,
    Channels.off_topic_0,
    Channels.off_topic_1,
    Channels.off_topic_2,
    Channels.voice_chat_0,
    Channels.voice_chat_1,
)```

[outer oasis]

Nice comment

[toxic wren]

huh /docs worked and then I restrted bot and now it's gone

I got it on video

but I deleted it

😭

Here's a chat log

now it does not even work /docs isnt even a command now

oh and I ended up making another command called docs_slash because hybrid seems to not be working? Probably will try to make hybrid after separate command works.

[toxic wren]

#community-meta message
make .src available in #1035199133436354600

[oblique cairn]

There's an existing issue for this, regarding .latex

https://github.com/python-discord/sir-lancebot/issues/1695

[tame pebble]

I don't see why it shouldn't be allowed either

[fallen patrol]

Which is waiting on one of the devops members of pydis to approve ...

https://github.com/python-discord/bot-core/pull/305

That seems a little overarching, requiring devops to approve every release seems flawed

[patent pivot]

seems fine

changes a matching file

[patent pivot]

I suppose we actually need to add python help to the whitelisted channels

latex should work fine

[fallen patrol]

Yah but this is one of those times having a _version.py file where the version is dynamically grabbed from would make sense

Joe.... https://github.com/python-discord/bot-core/pull/305/files#diff-b7f8a8784d5eff6e6b1b2c2529b831cd86132c49699fe48e0d23c17ad4e2d7deR7

[patent pivot]

yea at the time we started this poetry did not support this

fixed

[fallen patrol]

Poetry still doesn't support it

At least poetry now sorta allows using a different backend

[patent pivot]

well there is https://pypi.org/project/poetry-dynamic-versioning/

you used to have to install it as a global plugin and now you can have it be project specific

mark built something excellent for snekbox

https://github.com/python-discord/snekbox/blob/main/scripts/version.py

fixed

[patent pivot]

@fallen patrol relevant to the multiple interpreters issue https://github.com/python-discord/snekbox/pull/248

also takes snekbox CI time from like 30-40 minutes to like 3-4

[fallen patrol]

Why are they pre built custom rather than using PBS?

[patent pivot]

no JIT builds from PBS last I checked (without just using their script to build stuff, which is a more complex version of what we are currently doing)

[outer oasis]

Why does only PyDis get more than one Python version?

[patent pivot]

for this specific case where we've got a fairly select few versions and pyenv builds them easily with pretty simple direct access to the flags we may as well just build them ourselves — and we're doing it on a schedule anyway

[patent pivot]

because we shouldn't pick versions on behalf of downstream users

downstream users can do what they want, or just copy our version (and our image is public anyway)

right now there is no rhyme nor reason to why a version appears in snekbox

[fallen patrol]

Now I gotta make an image builder repo lol but actually probably better than what I have now

I think snekbox should support python versions

Or it needs to be documented very cleanly how to add a new tool + use it

[outer oasis]

@fallen patrol You can use the PyDis image

[patent pivot]

for reference -- the python-builds repo is just what snekbox already does but pulled out into a separate repo

[patent pivot]

which is why there are README changes about adding new interpreters to /snekbin/ in that PR

[fallen patrol]

Python builds doesn't add snekbox, yeah?

Cause I am slightly confused about wait NVM I forget that python is python and not rust. Python isn't a self contained binary, it requires the stdlib lol

[patent pivot]

no, python-builds is generic but does build into a snekbox format, so the binaries are dumped into /snekbin/ and have an ELF runpath of /snekbin/

[toxic wren]

Should we make the paste bin DM's for invocation of !eval on non-snekbox channels opt-in?

Ive seen lots of people complain

[patent pivot]

do you have any links to this? I'm curious what people are complaining about

and it's probably not worth having as an opt-in, we either have it or we scrap that, it's not something worth tracking and we don't hold the context long enough to have a delete button that people can opt-in to post-redirection (and I think adding that would be massively overcomplicated)

[toxic wren]

I think I saw someone in #python-discussion a while ago though I don't exactly remember when

I got DMed by the bot a few minutes ago and remembered this

[patent pivot]

i found one complaint with a bit of searching, I think generally we shouldn't roll things back based solely on one complaint of new behaviour. if people are that pressed they can ignore/block the bot but generally it remains a useful (and requested) feature

[toxic wren]

wait what why?

[thorny obsidian]

It's often dropped without any other context to people with questions. So it creates a not-great environment.

[toxic wren]

the link itself has the context needed

[thorny obsidian]

In-person, it would be rude to just walk in to a group and ask a question. On the internet that's not the case, but the change in expectation is not always clear. We'd rather people explain rather than just drop a link and then get no help

[thorny obsidian]

Sure, but if a person is like "I have a question, does anyone know X?" and then only gets a link to "dontasktoask", that's not the server culture we want.

It's off putting and not welcoming

[toxic wren]

how did you make it like this?

can I have the config?

btw I am new to emacs

[patent pivot]

https://github.com/doomemacs/doomemacs

[toxic wren]

ah, thanks! Ill look into it

[fallen patrol]

Does pydis still participate in hacktoberfest?

[cold island]

No, it seemed positive on paper but doesn't create the right incentives and places a burden on maintainers

[fervent sage]

hacktoberfest in a nutshell lol

[outer oasis]

Oh hai Alec

What do I actually get from Hacktoberfest anyways?

[patent pivot]

well there's no shirt anymore iirc lol

[fallen patrol]

There is

[patent pivot]

ah they are incentivizing crap again

excellent

[fallen patrol]

I was only wondering cause I noticed I'd made some pulls to pydis this month and was wondering if I could get the hacktoberfest-accepted label on them

[patent pivot]

oh yeah that's fine

[fallen patrol]

https://github.com/python-discord/bot/pull/3408 is the only one

Thanks joe

[patent pivot]

i added to 3409 as well

[fallen patrol]

-# I might as well go submit a patch for https://github.com/python-discord/bot/issues/3403

[patent pivot]

can i get myself a free shirt here let's see

[fallen patrol]

You are a maintainer at pydis so your pulls don't count

[patent pivot]

devastating

[fallen patrol]

Not seeing what you expect here? Hacktoberfest profiles only show contributor activity, not maintainer activity (we calculate this after Hacktoberfest ends). Profiles update once every 15 minutes if you're loading the page often, or once every 6 hours in the background.

I have 40 pending pulls that I made to my own repos so they won't count once they check that which is hilarious

[fallen patrol]

But ye, do you mind if I submit a quality patch for this?

[patent pivot]

yea go for it

if it lets you just review request me when you submit

i can't remember if you can or not as a non-maintainer

either way I'll look out for it

[fallen patrol]

Sounds good. Lemme get a test bot configured again, pain

Iirc it's been a while, botstrap should make it easier

[patent pivot]

yea botstrap does make it not so bad

[fallen patrol]

why does @radiant merlin and @slow bone have the presence intent?

also if you see this joe, how do you want https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot.md/#appendix-full-env-file-options to be updated?

Its missing fakeredis and a lot of other variables

[patent pivot]

i'm not sure why modmail needs it, may just be that it was run at a time where the bot was requesting all intents and we enabled it by default

probably same with big brother though less sure why that's necessary

[fallen patrol]

(I'm already preparing a MR to site due to a broken link + bot uses python 3.13 now)

eh tbf you could probably fix the broken link on your end

https://blog.pythondiscord.com/post/statistics-infrastructure

[patent pivot]

huh

interesting

that's now https://blog.pythondiscord.com/post/statistics-infrastructure-at-python-discord/

[fallen patrol]

ye

hey joe you mind if i submit a pull to fix that and the python 3.12 -> 3.13 for bot?

[patent pivot]

go for it

[fallen patrol]

I'm collecting the changes locally before I pr it

[patent pivot]

nice nice

[fallen patrol]

yay

good lord

[patent pivot]

ratelimited.me

[fallen patrol]

discord that's impressive:

you gave me a 403 to fetch a webhook with admin

oh

because the IDs are hardcoded in botstrap..... hm

[patent pivot]

huh

that sounds

against the idea of botstrap lol

[fallen patrol]

its because I wasn't working with big brother

when i got this line i thought that botstrap completed because "its just a warning"

its actually fatal

https://github.com/python-discord/bot/blob/d21c2c7d6660a547a3c232cbf0b10be971c32fea/botstrap.py#L54-L60

[stable mountain]

botstrap.py lines 54 to 60

log.warning(f"Couldn't find key: {item} in dict: {self.name} ")
log.warning(
    "Please make sure to follow our contribution guideline "
    "https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot/ "
    "to guarantee a successful run of botstrap "
)
sys.exit(-1)```

[patent pivot]

lol

[fallen patrol]

changed it to fatal so i don't forget for now

now i just get this lol

do templates include emojis?

[patent pivot]

hmmm

they might not actually

[fallen patrol]

i mean

fwiw I didn't use the template

[patent pivot]

ah lol

[fallen patrol]

or well, I did.... 3 years ago

[patent pivot]

looks like just channels, roles, perms and some settings

[fallen patrol]

are emojis stored in a github repo somewhere?

cause we could upgrade to app emojis

[green oriole]

I was looking for code that I wrote a while ago but I couldn't find it, maybe someone would remember better

It was to I believe check code jam qualifiers, it did some very cursed stuff to load the code by embedding it in a larger snippet, does that ring a bell to anybody?

There's no way it's the next PR I check lmfao

Oooh it was on forms

https://github.com/python-discord/forms-backend/pull/63

Thank you for y'all's magical brain waves

[fallen patrol]

I see conflicting information on this guide:

Note: This script will overwrite the .env.server file. We suggest you put any configuration not generated by this script in to .env instead

In your .env.server file, set urls_site_api="http://localhost:8000/api". If you wish to keep using http://web:8000/api, then COMPOSE_PROJECT_NAME has to be set.

To configure the bot manually, you will only need to set inside the .env.server file the values for the channels, roles, categories, etc. that are used by the component you are developing.

[tame pebble]

they are in a separate discord server

and the bootstrap script doesn't set the emojis, iirc

[tame pebble]

yeah that looks like it should point to .env instead

[fallen patrol]

aight I'll add it to my pr

how can i add an adomination to a page on the site?

[tame pebble]

to the contribution docs ?

[fallen patrol]

yeah

[tame pebble]

Add {: .notification .is-warning } underneath the warning

https://raw.githubusercontent.com/python-discord/site/refs/heads/main/pydis_site/apps/content/resources/guides/pydis-guides/contributing/sir-lancebot.md

[fallen patrol]

thanks

[tame pebble]

If it's multiline, i think you need to denote it with ---

There should be another example in the same file

[fallen patrol]

pain

I have a really old test guild

i'm sure its fine

Pydis Core Developers -> Core Developers
Domain Lead -> Domain Leads
Moderation Team -> Mod Team

but just for test server

what could be going on here?

[tame pebble]

did you use the template ?

[fallen patrol]

I did.... 3 years ago

:^)

[tame pebble]

Yeah that's invalid

use the new one

https://discord.new/zmHtscpYN9E3

[fallen patrol]

looking into https://github.com/python-discord/bot/issues/3402 and I actually can't figure out how to trigger that error

[stable mountain]

Please react with ✅ to upload your file(s) to our paste bin, which is more accessible for some users.

[fallen patrol]

🤔

on my test server it deleted my message and dmed me as well

which was confusing UI

is this no longer used on @stable mountain?

[cold island]

you can upload txt files now

is your clone out of date?

[fallen patrol]

Nope

my most recent commit is 88574889

https://github.com/python-discord/bot/blob/88574889/bot/exts/filtering/_filter_lists/extension.py

is this file disabled in prod?

or no, I see

so apparently Attachment.content_type seems to be just from the file extension

[fallen patrol]

it seems that this filter is no longer used here, seems I can also upload binary files

-# if you're wondering, that is the tr binary from gnu coreutils

[patent pivot]

lol yea i thought it was

hmmm

that's probably not intended but also isn't inherently harmful

uploading a UNIX binary is kind of pointless because to run it you'd need to set the +x bit when you download it

whereas .exe etc automatically have execution available

[fallen patrol]

or ./tr

[patent pivot]

not without the execute bit

[fallen patrol]

oh good .exe is still blocked

[fallen patrol]

i looked more into it and its anything with text/plain that gets previewed in discord

[patent pivot]

yea

[fallen patrol]

and discord sets it so weirdly

the same binary, with .txt appended

[patent pivot]

elf my beloved

[fallen patrol]

so let me see...

huh okay

can't recreate that locally

-# that took more work than I expected
https://github.com/python-discord/bot/pull/3413

breaking attachment types was somewhat difficult lmao

[fallen patrol]

...can i submit a pull to quiet bot strap?

@tame pebble used the new guild:

this would be in the template needing an update

[patent pivot]

lol yeah that is excessive

but presumably yeah it's just third party loggers being set to debug

[fallen patrol]

except because its botstrap I haven't configured anything yet

I decided for fixing stuff on the contributing guide i would return to the roots of a confused new developer trying to set up the bot for the first time and would see how easy it is to use

[patent pivot]

yea no I agree it needs fixing -- just shouldn't be too crazy of a fix

[patent pivot]

this is a nice idea

[fallen patrol]

its difficult lol

there's at least a few other things that can be fixed/provided in botstrap

i will create an issue for this lol

-# oh the things I'll do for a t-shirt

seems to properly work on https://github.com/onerandomusername/ghre-tester/blob/9be3025d879813db5f6524338002539ad0264e45/README.md%3Fplain%3D1#L1

[stable mountain]

README.md%3Fplain%3D1 line 1

# oh no```

[fallen patrol]

except for that text not being decoded

that.... hm

@timid sentinel with patch

[tame pebble]

yes

@patent pivot can you show me where that founders role lies in the role hierarchy in pydis ?

I will add it to the template

it shouldn't matter tbh

[patent pivot]

uhhh I think it's below admins above mods

[tame pebble]

cheers bg

[patent pivot]

no problems bby

[fallen patrol]

holy shit there's 18250 people with the announcements role????

and lol the cloudflare bot being here

-# oh wait no thats a cloudflare employee

[timid sentinel]

Try with %3f instead of %3F the URL you paste though.

[patent pivot]

!role Announcements

[stable mountain]

Announcements info

ID

463658397560995840

Colour (RGB)

#000000

Colour (HSV)

0.00 0.00 0

Member count

18249

Position

43

Permission code

6442450944

[fallen patrol]

context: user installed app

used it to get the role order

[tame pebble]

can you try again

[timid sentinel]

oh

I think I just misread the code

[fallen patrol]

@tame pebble it needs to be synced

this can be seen with https://discord.new/zmHtscpYN9E3

[tame pebble]

I did sync

[fallen patrol]

aside from one tiny error

[tame pebble]

Did it again now

[fallen patrol]

@patent pivot hey joeeeee can you run uv lock and commit the change to bot :^)

thoughts on adding the following

🥲

if not that can we add the --locked flag to ci

[timid sentinel]

yeaaaah we should probably do that...

[fallen patrol]

if we update the flag to --locked this would be caught in ci too

and we don't need the pre-commit config

not sure if the dockerfile should use --locked or --frozen

looks like everything uses locked in the documentation from astral

...so this would already sort of catch it

ye that works without adding another hook

your call wookie lol

[timid sentinel]

too much power

adding the hook would actually lock the file though while changing to --locked would just error right?

[fallen patrol]

@patent pivot noticing lance and site as well, was there a reason that you went with --frozen over --locked for uv?

[fallen patrol]

yes

[patent pivot]

consulting a higher power, chris is telling me now

[timid sentinel]

I'd probably just go belts and braces and change to --locked as well as adding the hook.

[fallen patrol]

lmfao

i got an error

lmfao

[patent pivot]

yea we should add the hook

[fallen patrol]

because uv just released a new version

[patent pivot]

but we should use frozen in docker

but locked for local dev

i will relock now

[fallen patrol]

and locked in ci

[patent pivot]

yea

main is now updated relocked

[patent pivot]

i don't know that we need to pull something external in for this though

chris has pointed me to something we did for polonium ```yaml
- id: uv-check
name: uv lock check
description: Checks the validity of the uv.lock file.
entry: uv lock --check
language: system
files: pyproject.toml
pass_filenames: false
require_serial: true

[fallen patrol]

ok, won't

yeah its literally just https://github.com/astral-sh/uv-pre-commit/blob/571189337017c8bdbf22a0977b9e44d33b5e5ae9/.pre-commit-hooks.yaml#L11C1-L20C38

[stable mountain]

.pre-commit-hooks.yaml line 11

- id: uv-lock```

[fallen patrol]

siiiigh

bot#3415

[dusky shore]

GitHub

[bot] #3415 code snippet additional character support

[timid sentinel]

lol

[fallen patrol]

also that trashcan not working,,,, did I do that?

ah that's #dev-log message

-# I swear I'm not hacktoberfest farming

bot#3416

[dusky shore]

GitHub

[bot] #3416 chore: add pre-commit hook for checking uv is locked

[fallen patrol]

@patent pivot could I please have the hacktoberfest-accepted label added to bot#3416, bot#3413, site#1566?

[dusky shore]

GitHub

[bot] #3416 chore: add pre-commit hook for checking uv is locked
[bot] #3413 fix: check for unknown/None content_type for pastebin uploader
[site] #1566 fix: update bot's required version and contrib deadlinks

[patent pivot]

added 👍

[fallen patrol]

thanks! if you want me to go ahead now, I can make a pull for lance + site, unless I should hold off until bot#3416 is merged

[dusky shore]

GitHub

[bot] #3416 chore: add pre-commit hook for checking uv is locked

[patent pivot]

i am going to uhhhh

merge

d

[fallen patrol]

sounds good

[timid sentinel]

hold up

[timid sentinel]

BIG mistake my friend

[patent pivot]

uh oh

UH OH

IT'S OVER FOR ME

[timid sentinel]

to be fair uv also messed up EDIT: it makes sense in the uv precommit repo because that's how it installs uv, tho would be more efficient to just install it directly without python

language: python will make it initialise a python environment for no reason I think

[patent pivot]

ahhh

i assumed we'd added chris's one

skimmed that

hmmm

[fallen patrol]

I copied that from uv

....listen its not my fault I broke ci twice

ci is just... breakable

[patent pivot]

i do think this will work fine, even if it won't automatically relock ```yaml

• id: uv-check
  name: uv lock check
  description: Checks the validity of the uv.lock file.
  entry: uv lock --check
  language: system
  files: pyproject.toml
  pass_filenames: false
  require_serial: true

[fallen patrol]

why not auto-relock?

this is what I get with this config:

    - id: uv-lock
      name: uv lock check
      description: Checks the validity of the uv.lock file.
      entry: uv lock --check
      language: system
      files: ^(uv\.lock|pyproject\.toml|uv\.toml)$
      pass_filenames: false

[timid sentinel]

I think what was merged but just with language: system is perfect

[fallen patrol]

(require_serial isn't needed afaict)

the extra arguments while I know we don't have a uv.toml it probably doesn't hurt to check it

[patent pivot]

i will update to language: system

changed

[fallen patrol]

number of times I've made joe commit directly to main today: two

[fallen patrol]

could i get https://github.com/python-discord/bot/issues/3414 approved so I can start work on it?

forum channels now seem to be supported in templates!

[patent pivot]

approved both

[fallen patrol]

thank you

@tame pebble now the only two errors from the new template are about rule_alerts and nomination_discussion

[patent pivot]

rule_alerts is a problem because it's a thread under mod-alerts and so we don't have it in the template -- but could add it to fix those errors

[fallen patrol]

oh

its that open issue

gotcha

hm

should map rule_alerts to mod_alerts somehow

[fallen patrol]

oh bet

like Webhook and _Webhooks

....botstrap doesn't work for your internal test guild, huh

because of the webhook handling

[fallen patrol]

woo

changes so far:

• httpcore filter
• response caching (so each endpoint is only hit once)
• flag updating (adds the message content intent automatically)
• assert we're in the guild, provide an invite link if we're not
• log fatal errors as fatal

[fallen patrol]

do we have a short url for https://www.pythondiscord.com/pages/guides/pydis-guides/contributing/bot/ ?

maybe https://www.pydis.com/contributing/bot ?

[toxic wren]

why tho?

[fallen patrol]

Its a long url to print in a console

[toxic wren]

well, doing that would require making a change to the site

[fallen patrol]

It wouldn't: https://pydis.com/contributing.md isn't in the source code. There's a short url system somewhere

but that link is specifically for the CONTRIBUTING.md files in bot, lance, and site

hmmm.... I could add support for upgrading existing guilds with botstrap

....might as well

...eh no that's different

[patent pivot]

yea short URLs are a worker

[fallen patrol]

bot#3418

[dusky shore]

GitHub

[bot] #3418 Support upgrading existing test guilds with botstrap

[patent pivot]

worker not happy with slashes in short paths it seems lol

i forget how this works

https://github.com/python-discord/workers/blob/main/short-urls/src/index.ts oh it's OSS

have added:

• https://pydis.com/contributing-bot
• https://pydis.com/contributing-site
• https://pydis.com/contributing-lance

i was sure we had a bot that could manage this

no i'm hallucinating that

i thought it was in arthur's cloudflare extensions

in theory it has an API key that can write to KV zones

[fallen patrol]

yeah probbaly

[patent pivot]

ah i did it for a bot in anotehr project and that's what i'm misremembering

[fallen patrol]

oh cool... discord made new apps default to public...

[patent pivot]

classic

[fallen patrol]

now i really want to add a cli to botstrap

lol

like fix-app

I wonder if sir lancebot has something like botstrap

[patent pivot]

nah it doesn't from memory

but we also don't have the hard dependencies on channels like we do for bot for most of lance

[outer oasis]

yeah

Lance is way less picky