pinecrew / pined-django / 33444429776

95%

push

github

repo: write down what a read of the source turned up

Twenty items, from a pass over `src/`. Eight look like bugs: the admin
reordering mutates the dict it was handed and so takes its order from
whichever user loaded the page first; a `PydanticField` writes NULL when it
cannot make sense of an expression; `create_admin` prints its failures and
exits 0, which in a start-up chain hides an unmigrated database as readily as
an existing user; `chain` substitutes an unset `$VAR` with the empty string,
so the two together can create an admin with an empty password and report
success.

The rest is what will bite later — a computed field that makes directories
while being serialized, a stamp file in `/tmp`, django's own defaults restated
where a release changing one of them would be silently overridden — and three
places where the docstrings do not follow the guide the repo carries.

Two items are questions rather than instructions: whether a user-written
`Case` can reach `get_db_prep_value` at all, and what a data migration should
do with a NULL row, which the code has been carrying as `idk if this approach
is correct` for a while.

Not shipped: the sdist takes `src`, the README and the LICENSE.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

201 of 246 branches covered (81.71%)

Branch coverage included in aggregate %.

1113 of 1302 relevant lines covered (85.48%)

0.85 hits per line